Refactoring of `arel_attributes_values` method #5294

Merged
merged 2 commits into from Mar 12, 2012

2 participants

@robinroestenburg

When browsing through the ActiveRecord code I came across this method and I thought it could be made a bit more clear, so I refactored it. I think this is the most I could do safely within the boundaries of this method.

Let me know if you think it is useful.

@robinroestenburg

I think can take it one step further and get rid of the flag parameters in arel_attributes_values as well, will add it to the pull request.

@tenderlove
Ruby on Rails member

@robinroestenburg yes please. I'll merge after you've updated the request.

@robinroestenburg

@tenderlove it's already there - I updated it right after the comment, check the a02d237 commit.

@tenderlove tenderlove merged commit 0bb6fcb into rails:master Mar 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment