I started on a larger refactoring of the
issue_report_details method today but found a potential security bug so I threw it away. It’s better to be secure than refactored :)
So instead, I performed a common Rails refactoring for removing duplicate code: extracting some code to a before filter.
1 2 3 4 5 6 7 8 9 10 11 12 13
# app/controllers/reports_controller.rb class ReportsController 'position') # ... end def issue_report_details @statuses = IssueStatus.find(:all, :order => 'position') # ... end end
1 2 3 4
# app/controllers/reports_controller.rb class ReportsController 'position') end end
This is a good refactoring when your controller actions need some data but that data isn’t their core responsibility. For example the
authorize before filters in Redmine will find the current project, user, and check that the permissions allow access. These don’t have anything to do with the actions, but they are still required.