| Issue 46: | SAMPLE Simpler navigation bar highlight code. | |
| 2 people starred this issue and may be notified of changes. | Back to list |
I substituted all methods that does the highlight by just one, as nothing is broken this is not a fix. I think this way its much more simpler to understand, the functionality should be the same. It was changed directly inside /vendor/plugins/substruct. Follows a diff, my working copy is so messed with changes not committed that its hard to separate issues that I filled to make a diff just with one :P It needs that app/controllers/application.rb be this: class ApplicationController < ActionController::Base include SubstructApplicationController before_filter :set_navigation_tabs before_filter :find_customer end This change is out of the scope of the subversion repository.
Apr 8, 2008
Not really the way I'd like to see this solved. By cleaning up I meant removing more crap from the application_controller. Don't like this approach, so I'm rejecting & setting to a low priority.
Labels:
-Priority-Medium Priority-Low
Apr 9, 2008
I don't understood why, its two different things, is really issue 45 that you are talking about? Issue 45 is just a change in a "find" call to not break a test or break the interface when the database wasn't populated using the administration interface (I normally import fixtures in the development environment with random ids (foxy fixtures dont need to have ids) to start to work with the interface). |
6.9 KB View Download