Rails Refactor Update (and Merb 1.0.7.1)
Over the past few days, I've been working on refactoring ActionController and ActionView to clean up the interactions between them. You can follow along on my github fork. Some principles I'm following:
- ActionController and ActionView should work well standalone
- All request-related information should be calculated in ActionController and passed through into ActionView
- ActionView should be responsible for figuring out what path to render; ActionController should pass enough information for ActionView to figure it out.
- Based on the previous, the information passed from ActionController to ActionView will likely be: template_name, list of acceptable extensions (i.e. [:html, :xml]), prefix (usually the controller name), and partial (a boolean indicating whether the template is a partial). This is not nailed down yet, but it has so far served well.
- So far, I have unified render(:template), render(:file), and render(:action) to use this new conduit, and will be working on partials tomorrow. Partials are quite complicated so my current plan may have to change slightly when I tackle them.
- Interesting info: ActionPack has a lot of methods that call each other (somewhat circularly), so it wasn't really possible to just replace the existing conduit in a straight-forward manner. Instead, I created the new method (currently called find_by_parts, which finds a template based on the components I discussed above) and slowly (very slowly) moved existing callers of find_by_path over to use find_by_parts. Thankfully, the Rails test suite caught the initial errors I made, a huge saving grace of the entire effort.
- A nice side-effect of moving to a single, clean API between AC and AV is that the final code should be easier to understand. Once I'm further along, I'll post some details of how exactly the interaction works.
- Templates should reload again in development mode
- An issue in the bundler where gems in your local repository were getting greedily installed is fixed
- An issue that was preventing bin/thor merb:gem:install use is fixed
- The error message when gems could not be found was slightly improved
- gem install merb (make sure merb-gen 1.0.7.1 is installed)
- in your app, rm -rf tasks/merb.thor
- in your app, run merb-gen thor
- you should receive a prompt asking you to override bin/common.rb. Accept the prompt.
- you're done