Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | let this regexp be stricter | Xavier Noria | 2011-04-19 | 1 | -1/+1 |
| | | | | | | | | | The purpose of this regexp is to detect strings that are valid identifiers to be used as method names. The anchor $ allows a trailing newline, and both ^ and $ may be bypassed because Ruby regexps are multiline always. The anchors \A and \z are the proper ones to ensure string boundaries. | ||||
* | Let's not dup because value may not always be duplicable. | José Valim | 2011-04-19 | 1 | -1/+1 |
| | |||||
* | Avoid define_method if possible. | José Valim | 2011-04-19 | 1 | -12/+39 |
| | |||||
* | use class_eval with a string when it's possible | Santiago Pastorino | 2011-03-22 | 1 | -2/+8 |
| | |||||
* | Do not in place modify what table_name returns | Santiago Pastorino | 2011-03-22 | 1 | -1/+1 |
| | |||||
* | define_attr_method should be able to define methods that returns nil | Santiago Pastorino | 2011-03-17 | 1 | -2/+2 |
| | |||||
* | remove misleading comment | Aaron Patterson | 2011-03-17 | 1 | -2/+0 |
| | |||||
* | dup strings on return so that in place modifications do not break anything. ↵ | Aaron Patterson | 2011-03-17 | 1 | -1/+1 |
| | | | | I am looking at you "compute_table_name" | ||||
* | define_attr_method correctly defines methods with invalid identifiers | Santiago Pastorino | 2011-03-16 | 1 | -3/+2 |
| | |||||
* | Move ActiveModel::AttributeMethods#attribute_methods_generated? to ↵ | Alexander Uvarov | 2011-03-01 | 1 | -25/+19 |
| | | | | | | | | ActiveRecord, so it's flexible now [#6428 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | module_eval is not needed here | Santiago Pastorino | 2011-02-03 | 1 | -5/+3 |
| | |||||
* | method_defined?, define_method and send accepts a string as argument | Santiago Pastorino | 2011-02-03 | 1 | -6/+6 |
| | |||||
* | allow spaces and other characters in attribute names [#4725 state:resolved] | Caleb Land | 2011-02-03 | 1 | -4/+4 |
| | | | | | | | | * define the dynamically defined methods with 'define_method' instead of def * wrap some string injected method names in quotes Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Deleted extra "a" in method comment | Paul Yoder | 2011-01-27 | 1 | -1/+1 |
| | |||||
* | No need to symbolize these. | José Valim | 2010-12-27 | 1 | -1/+1 |
| | |||||
* | define_attr_method must serialize nil correctly | Aaron Patterson | 2010-12-20 | 1 | -1/+1 |
| | |||||
* | grammar fix | Aditya Sanghi | 2010-11-27 | 1 | -2/+2 |
| | |||||
* | Fix when database column name has some symbolic characters. [#5818 ↵ | Robert Pankowecki | 2010-11-14 | 1 | -5/+5 |
| | | | | state:resolved] | ||||
* | Fewer object allocations | Jeremy Kemper | 2010-09-09 | 1 | -8/+6 |
| | |||||
* | Cheaper attribute_method_matchers | Jeremy Kemper | 2010-09-09 | 1 | -9/+10 |
| | |||||
* | Fix require for class_attribute | Jeremy Kemper | 2010-09-09 | 1 | -1/+1 |
| | |||||
* | Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵ | Santiago Pastorino | 2010-08-14 | 1 | -42/+42 |
| | | | | 's/[ \t]*$//' -i {} \;) | ||||
* | typo in AM | Adam Meehan | 2010-08-09 | 1 | -1/+1 |
| | |||||
* | Fix a bunch of minor spelling mistakes | Evgeniy Dolzhenko | 2010-06-15 | 1 | -1/+1 |
|\ | |||||
| * | Fix a bunch of minor spelling mistakes | Evgeniy Dolzhenko | 2010-06-11 | 1 | -1/+1 |
| | | |||||
* | | Minor changes to active_model/attribute_methods | Rizwan Reza | 2010-06-14 | 1 | -24/+30 |
|/ | |||||
* | typo 'Provivdes' -> 'Provides' | Santiago Pastorino | 2010-05-05 | 1 | -1/+1 |
| | |||||
* | fix stack trace lines on class_eval | Santiago Pastorino | 2010-04-09 | 1 | -2/+2 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge branch 'master' into docrails | wycats | 2010-03-26 | 1 | -3/+15 |
|\ | |||||
| * | cleaning up a bunch of method already defined warnings [#4209 state:resolved] | Aaron Patterson | 2010-03-17 | 1 | -3/+15 |
| | | | | | | | | Signed-off-by: wycats <wycats@gmail.com> | ||||
* | | Merge remote branch 'mainstream/master' | Pratik Naik | 2010-03-12 | 1 | -4/+7 |
|\| | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/base.rb railties/lib/rails/configuration.rb railties/lib/rails/log_subscriber.rb | ||||
| * | Add missing information about attributes method. | José Valim | 2010-02-28 | 1 | -3/+6 |
| | | |||||
| * | Use Object#singleton_class instead of #metaclass. Prefer Ruby's choice. | Jeremy Kemper | 2010-02-25 | 1 | -1/+1 |
| | | |||||
* | | Merge branch 'master' of git://github.com/rails/rails | Mikel Lindsaar | 2010-02-02 | 1 | -0/+1 |
|\| | | | | | | | | | | | | | | | Conflicts: activemodel/README activemodel/lib/active_model/errors.rb activemodel/lib/active_model/serialization.rb railties/guides/source/3_0_release_notes.textile | ||||
| * | Full update on ActiveModel documentation | Mikel Lindsaar | 2010-02-01 | 1 | -20/+120 |
| | | |||||
| * | Ruby 1.9.2: explicit coercion check no longer needed; bug fixed upstream | Jeremy Kemper | 2010-01-01 | 1 | -4/+0 |
| | | |||||
| * | Ruby 1.9.2: explicitly raise NoMethodError for explicit coercion attempts | Jeremy Kemper | 2009-12-06 | 1 | -0/+4 |
| | | |||||
* | | Adding ActiveModel::AttributeMethods documentation | Mikel Lindsaar | 2010-01-17 | 1 | -21/+120 |
|/ | |||||
* | Fixed behavior of attribute_methods_generated? [#3220 state:resolved] | Sam Pohlenz | 2009-10-07 | 1 | -1/+1 |
| | | | | Signed-off-by: Joshua Peek <josh@joshpeek.com> | ||||
* | Don't share attribute matchers between classes [#3216 state:resolved] | Sam Pohlenz | 2009-10-07 | 1 | -1/+4 |
| | | | | | | | Allows separate models that include ActiveModel::AttributeMethods to use different sets of attribute matchers. Signed-off-by: Joshua Peek <josh@joshpeek.com> | ||||
* | Fix warnings in AMo | Joshua Peek | 2009-09-05 | 1 | -1/+1 |
| | |||||
* | AMo overrides alias_attribute and manages aliasing all known attribute ↵ | Joshua Peek | 2009-08-10 | 1 | -0/+10 |
| | | | | method matchers | ||||
* | Centralize attr method name concatenation in AttributeMethodMatch | Joshua Peek | 2009-08-10 | 1 | -11/+22 |
| | |||||
* | Extract generic attribute method generation to AMo | Joshua Peek | 2009-08-04 | 1 | -0/+267 |