paxxxtothemax / substruct

Automatically exported from code.google.com/p/substruct
0 stars 0 forks source link

Refactor "OrderHelper" into an OrderPresenter #194

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
order_helper.rb was written a looong time ago and really isn't the best way to 
share code between 
controllers.

This code should be refactored into a presenter, as outlined here:

http://blog.jayfields.com/2007/03/rails-presenter-pattern.html

It will be much easier to understand and test then. Right now it's a 
clusterfuck.

Original issue reported on code.google.com by subim...@gmail.com on 26 Dec 2009 at 6:06

GoogleCodeExporter commented 8 years ago

Original comment by subim...@gmail.com on 26 Aug 2010 at 5:46

GoogleCodeExporter commented 8 years ago
Updating this code will most likely break StoreController, and require a 
refactoring. This could break existing Substruct installs, and perhaps is 
something that warrants a v2 release?

Original comment by subim...@gmail.com on 29 Aug 2010 at 12:24