This project is archived and is in readonly mode.
Don't share attribute method matchers between classes
Reported by Sam Pohlenz | September 16th, 2009 @ 11:23 AM | in 3.0.2
Currently, any class that includes ActiveModel::AttributeMethods will share its attribute matchers with another class including the AttributeMethods module.
This patch fixes this by using an inheritable_reader/writer for attribute_method_matchers in ActiveModel::AttributeMethods.
Comments and changes to this ticket
-
Sam Pohlenz September 16th, 2009 @ 11:04 PM
- Tag changed from activemodel to activemodel, patch
-
CancelProfileIsBroken September 25th, 2009 @ 12:06 PM
- Tag changed from activemodel, patch to activemodel, bugmash, patch
-
josh October 7th, 2009 @ 05:22 AM
- Milestone cleared.
- State changed from new to open
- Assigned user set to josh
-
josh October 7th, 2009 @ 03:29 PM
- State changed from open to committed
-
Repository October 7th, 2009 @ 03:57 PM
- State changed from committed to resolved
(from [f8e91bda9c171dd0c31dbea95ba4e9ced4ee2547]) Don't share attribute matchers between classes [#3216 state:resolved]
Allows separate models that include ActiveModel::AttributeMethods to
use different sets of attribute matchers.Signed-off-by: Joshua Peek josh@joshpeek.com
http://github.com/rails/rails/commit/f8e91bda9c171dd0c31dbea95ba4e9... -
Jeremy Kemper October 15th, 2010 @ 11:01 PM
- Milestone set to 3.0.2
- Importance changed from to
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Attachments
Tags
Referenced by
- 3220 Fix for ActiveModel::AttributeMethods#attribute_methods_generated? This patch fixes this and adds some verification tests fo...
- 3216 Don't share attribute method matchers between classes (from [f8e91bda9c171dd0c31dbea95ba4e9ced4ee2547]) Don't s...