Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | fixed a bug with the json serialization when the class setting is set to not ↵ | Matt Aimonetti | 2011-09-22 | 1 | -0/+10 | |
| | | | | | | | | | | | | 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 | 1 | -0/+17 | |
| | | | ||||||
* | | | Generate attribute method unless it's already in the module. | Jon Leighton | 2011-09-14 | 1 | -0/+23 | |
| | | | | | | | | | | | | | | | | | | | | | 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 | 1 | -0/+15 | |
| | | | | | | | | | | | | This can be overloaded by implementors if necessary. | |||||
* | | | Let Ruby deal with method visibility. | Jon Leighton | 2011-09-13 | 1 | -0/+36 | |
| | | | | | | | | | | | | | | | | | | 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 | 1 | -6/+11 | |
| | | | ||||||
* | | | Use an empty AttributeMethodMatcher by default. | Jon Leighton | 2011-09-13 | 1 | -0/+17 | |
| | | | | | | | | | | | | | | | | | | 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 | 1 | -2/+18 | |
| |/ |/| | ||||||
* | | add has_key? to ActiveModel::Errors | Damien Mathieu | 2011-09-06 | 1 | -0/+6 | |
| | | ||||||
* | | Merge pull request #2563 from bogdan/internal_validation | José Valim | 2011-08-25 | 1 | -0/+33 | |
|\ \ | | | | | | | Implemented strict validation concept | |||||
| * | | Implemented strict validation concept | Bogdan Gusiev | 2011-08-17 | 1 | -0/+33 | |
| | | | | | | | | | | | | | | | | | | | | | In order to deliver debug information to dev team instead of display error message to end user Implemented strict validation concept that suppose to define validation that always raise exception when fails | |||||
* | | | added more tests for only-include and except-include options in serialization | Anand | 2011-08-23 | 1 | -1/+14 | |
|/ / | ||||||
* | | Allow ActiveRecord observers to be disabled. | Myron Marston | 2011-08-12 | 1 | -0/+12 | |
| | | | | | | We have to use Observer#update rather than Observer#send since the enabled state is checked in #update before forwarding the method call on. | |||||
* | | Merge pull request #2385 from bogdan/test_default_sanitizer2 | José Valim | 2011-08-01 | 1 | -1/+9 | |
|\ \ | | | | | | | MassAssignmentProtection: consider 'id' insensetive in StrictSanitizer | |||||
| * | | MassAssignmentProtection: consider 'id' insensetive in StrictSanitizer | Bogdan Gusiev | 2011-07-28 | 1 | -1/+9 | |
| | | | | | | | | | | | | | | | | | | In order to use StrictSanitizer in test mode Consider :id as not sensetive attribute that can be filtered from mass assignement without exception. | |||||
* | | | Rename new method to_path to to_partial_path to avoid conflicts with ↵ | José Valim | 2011-08-01 | 2 | -7/+7 | |
| |/ |/| | | | | | File#to_path and similar. | |||||
* | | Merge pull request #2034 from Casecommons/to_path | José Valim | 2011-07-28 | 2 | -5/+20 | |
|\ \ | | | | | | | Allow ActiveModel-compatible instances to define their own partial paths | |||||
| * | | Let ActiveModel instances define partial paths. | Grant Hutchins & Peter Jaros | 2011-07-25 | 2 | -5/+20 | |
| |/ | | | | | | | | | | | Deprecate ActiveModel::Name#partial_path. Now you should call #to_path directly on ActiveModel instances. | |||||
* / | Handle the empty array correctly. | thedarkone | 2011-07-27 | 1 | -0/+1 | |
|/ | ||||||
* | ActiveModel support for the :include serialization option | John Firebaugh | 2011-07-17 | 2 | -8/+117 | |
| | | | | | | | | | | | | | | | | | | | | This commit moves support for the :include serialization option for serializing associated objects out of ActiveRecord in into ActiveModel. The following methods support the :include option: * serializable_hash * to_json * to_xml Instances must respond to methods named by the values of the :includes array (or keys of the :includes hash). If an association method returns an object that is_a?(Enumerable) (which AR has_many associations do), it is assumed to be a collection association, and its elements must respond to :serializable_hash. Otherwise it must respond to :serializable_hash itself. While here, fix #858, XmlSerializer should not singularize already singular association names. | |||||
* | Add a test | John Firebaugh | 2011-07-17 | 1 | -0/+5 | |
| | ||||||
* | serializable_hash(:only => [], :methods => [:foo]) should work | John Firebaugh | 2011-07-17 | 1 | -0/+10 | |
| | ||||||
* | Enable passing root: false to #to_json | Joe Fiorini | 2011-07-01 | 1 | -1/+52 | |
| | ||||||
* | changing an attribute multiple times retains the correct original value | Ian Stewart | 2011-06-28 | 1 | -0/+9 | |
| | ||||||
* | Allow to specify mass-assignment roles as array | Alexander Uvarov | 2011-06-23 | 1 | -0/+14 | |
| | ||||||
* | Merge pull request #1654 from ganeshkumar/test_cases | Santiago Pastorino | 2011-06-13 | 1 | -1/+0 | |
|\ | | | | | removed duplicate code in tests | |||||
| * | removed duplicate code | ganesh | 2011-06-11 | 1 | -1/+0 | |
| | | ||||||
* | | Get rid of the alternate namespace lookup. | José Valim | 2011-06-11 | 1 | -10/+0 | |
|/ | ||||||
* | Brought back alternative convention for namespaced models in i18n. | thoefer | 2011-06-08 | 1 | -0/+10 | |
| | ||||||
* | Transform the symbol into a constant lookup. | José Valim | 2011-05-31 | 1 | -4/+4 | |
| | ||||||
* | ActiveModel::MassAssignmentSecurity.mass_assignment_sanitizer method | Bogdan Gusiev | 2011-05-30 | 1 | -5/+13 | |
| | | | | | In order to specify your own sanitize method Implemented .mass_assignment_sanitizer configuration option | |||||
* | MassAssignmentSecurity: add ability to specify your own sanitizer | Bogdan Gusiev | 2011-05-26 | 4 | -25/+25 | |
| | | | | | | Added an ability to specify your own behavior on mass assingment protection, controlled by option: ActiveModel::MassAssignmentSecurity.mass_assignment_sanitizer | |||||
* | No more need rubygems here. | Arun Agrawal | 2011-05-22 | 1 | -1/+0 | |
| | ||||||
* | Specify the name to be used for ActiveModel::Name fixes #1168 | Yves Senn | 2011-05-20 | 1 | -1/+38 | |
| | | | | This patch allows to specify the name of your models independent of the class name. | |||||
* | Attributes with :string type should not be given the type passed in model ↵ | Jim Herzberg | 2011-05-18 | 1 | -1/+7 | |
| | | | | serialization options. Closes #1058 | |||||
* | Follow code conventions on some tests | Sebastian Martinez | 2011-05-10 | 4 | -29/+29 | |
| | ||||||
* | Methods that return nil should not be considered YAML | Joshua Wehner | 2011-05-10 | 1 | -0/+4 | |
| | ||||||
* | renamed mass-assignment scopes to roles, updated code, tests, docs and ↵ | Josh Kalderimis | 2011-05-08 | 1 | -2/+2 | |
| | | | | security guide | |||||
* | :if should not fire on validations when not in context with :on | Aditya Sanghi | 2011-04-29 | 1 | -0/+14 | |
| | ||||||
* | Add additional tests for AM::ObserverArray that I had missed yesterday. | Myron Marston | 2011-04-28 | 1 | -1/+50 | |
| | ||||||
* | Fix bug with AM::Observer disablement. | Myron Marston | 2011-04-28 | 1 | -0/+10 | |
| | | | | | Now that we propagate the enabling/disabling to descendants, we no longer have to check the disabled_observer Set on each superclass of the model class. This was causing a bug when disabling all observers at a superclass level and then enabling an individual observer at a subclass level. Plus the logic is simpler now :). | |||||
* | Revert "Revert "Handle enabling/disabling observers at different levels of ↵ | Myron Marston | 2011-04-28 | 1 | -0/+39 | |
| | | | | | | | | | the class hierarchy."" This reverts commit 2a25c5818b03d7d6cd63aad180bff23479dbd861. I'm going to add another commit that keeps the same behavior of fixes the problems of leaking memory in development. | |||||
* | Revert "Handle enabling/disabling observers at different levels of the class ↵ | José Valim | 2011-04-28 | 1 | -39/+0 | |
| | | | | | | | | | | hierarchy." This reverts commit ad62f1928768bd2676958a4a08512bad342fe469 because the current subclasses implementation leaks memory in development. Instead of keeping an array of subclasses, the better solution is to ask the parent if an observer is being disabled or not. | |||||
* | Handle enabling/disabling observers at different levels of the class hierarchy. | Myron Marston | 2011-04-28 | 1 | -0/+39 | |
| | | | | Last call wins. | |||||
* | Allow observers to be enabled and disabled. | Myron Marston | 2011-04-28 | 2 | -0/+127 | |
| | | | | This is useful in situations like model unit tests and the occasional rake task to backfill old data. | |||||
* | AM mass assignment security attr_accessible and attr_protected now allow for ↵ | Josh Kalderimis | 2011-04-24 | 2 | -11/+39 | |
| | | | | | | | | scopes using :as => scope eg. attr_accessible :name attr_accessible :name, :admin, :as => :admin | |||||
* | bcrypt will encrypt anything, so validate_presence_of would not catch nil / ↵ | Aaron Patterson | 2011-04-14 | 1 | -0/+12 | |
| | | | | blank passwords. Thank you to Aleksander Kamil Modzelewski for reporting this | |||||
* | No need to rescue here. Invoking the wrong method in an object can happen ↵ | José Valim | 2011-04-11 | 3 | -34/+0 | |
| | | | | anywhere if you are not careful. In other words, test your shit. | |||||
* | Revert "Add :use_include option to allow user to explicitly use ↵ | José Valim | 2011-04-11 | 2 | -19/+0 | |
| | | | | | | | | `Range#include?` method in Ruby 1.9" Use :with => range.to_a instead. This reverts commit f6540211b5b9133c9f93c11655a04d613c237e67. |