My favorites | Sign in
Project Home Downloads Wiki Issues Source
READ-ONLY: This project has been archived. For more information see this post.
Search
for
  Advanced search   Search tips   Subscriptions
Issue 194: Refactor "OrderHelper" into an OrderPresenter
  Back to list
 
Project Member Reported by subim...@gmail.com, Dec 25, 2009
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.
Aug 25, 2010
Project Member #1 subim...@gmail.com
(No comment was entered for this change.)
Labels: -Priority-High Priority-Critical Milestone-v1.4
Aug 28, 2010
Project Member #2 subim...@gmail.com
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?

Powered by Google Project Hosting