Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | | make ActiveModel::Name fail gracefully with anonymous classes | Jakub Kuźma | 2011-11-24 | 2 | -0/+16 | |
| |/ / / / |/| | | | | ||||||
* | | | | | Rely on a public contract between railties instead of accessing railtie ↵ | José Valim | 2011-11-23 | 3 | -21/+49 | |
| |/ / / |/| | | | | | | | | | | | methods directly. | |||||
* | | | | Update variable's name in the test case to reflect the class we're testing | Prem Sichanugrist | 2011-11-18 | 3 | -24/+24 | |
| | | | | ||||||
* | | | | Speed up attribute invocation by checking if both name and calls are compilable. | José Valim | 2011-11-14 | 1 | -27/+32 | |
| | | | | ||||||
* | | | | use any? instead of !empty? | Oscar Del Ben | 2011-11-14 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Fix a tiny typo in custom validators documentation | Jean Boussier | 2011-11-10 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | replaced remove and define method calls to :redefine | Anand | 2011-11-07 | 1 | -2/+1 | |
| | | | | ||||||
* | | | | Revert "Merge pull request #2378 from cesario/remove_warnings_activemodel" | Jon Leighton | 2011-11-05 | 1 | -1/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 6aaae3de277b572f37e09f16ae12737c3c87dfb7, reversing changes made to fdbc4e5f4e5746ebf558485348c841b33f038fda. Reason: build failure. | |||||
* | | | | Merge pull request #2378 from cesario/remove_warnings_activemodel | Vijay Dev | 2011-11-05 | 1 | -0/+1 | |
|\ \ \ \ | | | | | | | | | | | Remove warnings by calling remove_method | |||||
| * | | | | Remove warnings by calling remove_method | Franck Verrot | 2011-07-31 | 1 | -0/+1 | |
| | | | | | ||||||
* | | | | | Synchronize the gemspecs since CHANGELOG has been renamed to CHANGELOG.md | Franck Verrot | 2011-11-04 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Convert CHANGELOGs to Markdown format. | Jon Leighton | 2011-11-04 | 2 | -109/+109 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reasons: * Markdown reads well as plain text, but can also be formatted. * It will make it easier for people to read on the web as Github formats the Markdown nicely. * It will encourage a level of consistency when people are writing CHANGELOG entries. The script used to perform the conversion is at https://gist.github.com/1339263 | |||||
* | | | | | correct documentation on initialize method to accept a single parameter | bradrobertson | 2011-10-26 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-10-20 | 1 | -3/+2 | |
|\ \ \ \ \ | ||||||
| * | | | | | Removed mention of deprecated ActiveModel::Errors#on | Martin Svalin | 2011-10-19 | 1 | -3/+2 | |
| | |/ / / | |/| | | | ||||||
* | | | | | New #added? method on ActiveModel::Errors | Martin Svalin | 2011-10-19 | 3 | -7/+68 | |
| | | | | | | | | | | | | | | | The #added? method makes it possible to check if a specific error has been added, using the same parameters as for #add. | |||||
* | | | | | use variables from test setup | Vijay Dev | 2011-10-17 | 1 | -6/+4 | |
| | | | | | ||||||
* | | | | | fix a typo and slightly reword has_secure_password comment | Vijay Dev | 2011-10-17 | 1 | -2/+2 | |
| | | | | | ||||||
* | | | | | ActiveModel::Errors#generate_message without i18n_scope, and more test cases ↵ | Martin Svalin | 2011-10-17 | 2 | -4/+29 | |
|/ / / / | | | | | | | | | | | | | for #add | |||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-10-14 | 3 | -6/+6 | |
|\ \ \ \ | ||||||
| * | | | | Use .add instead of << to add errors | Evgeniy Dolzhenko | 2011-10-11 | 3 | -6/+6 | |
| | | | | | ||||||
* | | | | | Fixed serialization issues with multiple includes with options | zhengjia | 2011-10-11 | 2 | -1/+9 | |
|/ / / / | ||||||
* | | | | One of the activemodel naming tests wasn't asserting anything. | Uģis Ozols | 2011-10-07 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-10-05 | 2 | -5/+5 | |
|\ \ \ \ | ||||||
| * | | | | Change one more place for JSON string to Hash | Robert Ross | 2011-10-04 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Fix as_json doc to say what it actually returns. | Robert Ross | 2011-10-04 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Class name specified in example is Person, not AttributePerson. | Uģis Ozols | 2011-09-29 | 1 | -3/+3 | |
| | | | | | ||||||
* | | | | | Provide read_attribute_for_serialization as the API to serialize attributes. | José Valim | 2011-09-30 | 2 | -5/+26 | |
| | | | | | ||||||
* | | | | | Fixing `as_json` method for ActiveRecord models. | Nicolás Hock Isaza | 2011-09-29 | 1 | -0/+10 | |
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When you've got an AR Model and you override the `as_json` method, you should be able to add default options to the renderer, like this: class User < ActiveRecord::Base def as_json(options = {}) super(options.merge(:except => [:password_digest])) end end This was not possible before this commit. See the added test case. | |||||
* | | | | Make serializable_hash take attr values directly from attributes hash. | Jon Leighton | 2011-09-26 | 2 | -1/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, it would use send() to get the attribute. In Active Resource, this would rely on hitting method missing. If a method with the same name was defined further up the ancestor chain, that method would wrongly be called. This change fixes test_to_xml_with_private_method_name_as_attribute in activeresource/test/cases/base_test.rb, which was broken after 51bef9d8fb0b4da7a104425ab8545e9331387743, because that change made to_xml use serializable_hash. | |||||
* | | | | Refactor AMo as_json. | José Valim | 2011-09-23 | 1 | -9/+5 | |
| | | | | ||||||
* | | | | fixed a bug with the json serialization when the class setting is set to not ↵ | Matt Aimonetti | 2011-09-22 | 2 | -12/+20 | |
| | | | | | | | | | | | | | | | | include the root, but an instance is serialized with the root option passed as true | |||||
* | | | | to_xml should also rely on serializable hash. | José Valim | 2011-09-18 | 2 | -31/+32 | |
| | | | | ||||||
* | | | | update 3.1 release date in changelogs | Vijay Dev | 2011-09-14 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Generate attribute method unless it's already in the module. | Jon Leighton | 2011-09-14 | 2 | -1/+24 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | There's no harm in generating a method name that's already defined on the host class, since we're generating the attribute methods in a module that gets included. In fact, this is desirable as it allows the host class to call super. | |||||
* | | | | Add an attribute_missing method to ActiveModel::AttributeMethods. | Jon Leighton | 2011-09-13 | 2 | -8/+26 | |
| | | | | | | | | | | | | | | | | This can be overloaded by implementors if necessary. | |||||
* | | | | Avoid double super call in some cases. | Jon Leighton | 2011-09-13 | 1 | -5/+4 | |
| | | | | | | | | | | | | | | | | | | | | If super was false earlier, it is still going to be false, so we don't need to call it again at the end of the method. | |||||
* | | | | Let Ruby deal with method visibility. | Jon Leighton | 2011-09-13 | 2 | -13/+47 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Check respond_to_without_attributes? in method_missing. If there is any method that responds (even private), let super handle it and raise NoMethodError if necessary. | |||||
* | | | | Add deprecation for doing `attribute_method_suffix ''` | Jon Leighton | 2011-09-13 | 2 | -6/+22 | |
| | | | | ||||||
* | | | | refactoring | Jon Leighton | 2011-09-13 | 1 | -19/+11 | |
| | | | | ||||||
* | | | | Use an empty AttributeMethodMatcher by default. | Jon Leighton | 2011-09-13 | 2 | -2/+26 | |
| | | | | | | | | | | | | | | | | | | | | | | | | This means that attribute methods which don't exist will get generated when define_attribute_methods is called, so we don't have to use hacks like `attribute_method_suffix ''`. | |||||
* | | | | Added test for obj.errors.as_json | Lawrence Pit | 2011-09-09 | 1 | -0/+11 | |
| | | | | ||||||
* | | | | Add ability to get an individual full error message + test for full_messages. | Lawrence Pit | 2011-09-09 | 2 | -16/+35 | |
| |/ / |/| | | ||||||
* | | | add has_key? to ActiveModel::Errors | Damien Mathieu | 2011-09-06 | 2 | -0/+7 | |
| | | | ||||||
* | | | Add comments about bcrypt-ruby gem to SecurePassword | Guillermo Iguaran | 2011-09-03 | 1 | -0/+6 | |
| | | | ||||||
* | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-09-02 | 1 | -1/+1 | |
|\ \ \ | ||||||
| * | | | Fix typo in ActiveModel::Dirty comment. define_attribute_methods is a class ↵ | Josh Nesbitt | 2011-09-02 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | method, not attribute. | |||||
* | | | | Remove hard dependency on bcrypt. | Guillermo Iguaran | 2011-09-02 | 2 | -3/+3 | |
|/ / / | ||||||
* | | | Merge pull request #2563 from bogdan/internal_validation | José Valim | 2011-08-25 | 13 | -6/+77 | |
|\ \ \ | | | | | | | | | Implemented strict validation concept | |||||
| * | | | Typo fix | Bogdan Gusiev | 2011-08-25 | 1 | -1/+1 | |
| | | | |