This repository has been archived by the owner. It is now read-only.

fix signature of sanitize_for_mass_assignment to match the one in Active... #50

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
1 participant

lhm commented Oct 23, 2012

...Model.

The problem being that the strong_parameter's version passes an empty hash up to ActiveModel's sanitize_for_mass_assignment, where a role is expected.

Merge remote-tracking branch 'upstream/master' into fix-sanitize-for-…
…mass-assignment-when-attr-accessibleg

Conflicts:
	test/active_model_mass_assignment_taint_protection_test.rb

lhm commented Jan 10, 2013

this seems to be fixed by 0d21cc5

@lhm lhm closed this Jan 10, 2013

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.