Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rails 4 compatibility #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adam-at-mobomo
Copy link

This is similar to #62, but designed not only to support Rails 4 (without the 'protected_attributes' gem), but also works with Rails 3.2 and with Rails 4+ when using the 'protected_attributes' gem. Also, when checking out this project, the tests should all pass with the default configuration, and the documentation has been updated to mention filtered parameters.

The following changes enable Rails 4 support, while also maintaining compatibility with Rails 3.2+

1. Call attr_accessible only when using ActiveRecord < 4 or when using the 'protected attributes' gem
2. Rename the longest database index to allow tests to run in Rails 4
3. Update the README regarding the use of filtered params
4. Change gemspec and require statements to remove references to protected_attributes, and allow ActiveRecord > 3.2
@RyanRusnak
Copy link

+1!

@xlgmokha
Copy link

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants