aboutsummaryrefslogtreecommitdiffstats
path: root/activemodel/test
Commit message (Collapse)AuthorAgeFilesLines
* validates method should not change options argumentdreamfall2012-01-171-0/+6
|
* Fix ActiveModel::Errors#dupPaweł Kondzior2012-01-161-0/+8
| | | | | | | | | Since ActiveModel::Errors instance keeps all error messages as hash we should duplicate this object as well. Previously ActiveModel::Errors was a subclass of ActiveSupport::OrderedHash, which results in different behavior on `dup`, this may result in regression for people relying on it.
* Add ActiveModel::Errors#delete, which was not available after move to use ↵Piotr Sarnacki2012-01-161-0/+7
| | | | delegation
* Fix test class name that should read WithoutCarlos Antonio da Silva2012-01-161-2/+2
|
* Fix stack level too deep when model does not have attributes method.Piotr Sarnacki2012-01-151-0/+8
| | | | | | | | | | | Without that patch when using ActiveModel::AttributeMethods in a class that does not respond to `attributes` method, stack level too deep error will be raised on non existing method. While documentation is clear that you need to define `attributes` method in order to use AttributeMethods module, `stack level too deep` is rather obscure and hard to debug, therefore we should try to not break `method_missing` if someone forgets about defining `attributes`.
* Merge pull request #4360 from rafaelfranca/patch-1Aaron Patterson2012-01-061-2/+2
|\ | | | | Remove more references to Test::Unit
| * Use ActiveModel::TestCase instead of Test::Unit:TestCaseRafael Mendonça França2012-01-061-2/+2
| |
* | require minitest rather than test/unitAaron Patterson2012-01-061-1/+1
|/
* Support configuration on ActiveRecord::Model.Jon Leighton2011-12-281-0/+154
| | | | | | | | | | | | | | | The problem: We need to be able to specify configuration in a way that can be inherited to models that include ActiveRecord::Model. So it is no longer sufficient to put 'top level' config on ActiveRecord::Base, but we do want configuration specified on ActiveRecord::Base and descendants to continue to work. So we need something like class_attribute that can be defined on a module but that is inherited when ActiveRecord::Model is included. The solution: added ActiveModel::Configuration module which provides a config_attribute macro. It's a bit specific hence I am not putting this in Active Support or making it a 'public API' at present.
* remove deprecated define_attr_method from ActiveModel::AttributeMethodsSergey Nartimov2011-12-241-31/+0
|
* Changed the require path for loggerKarunakar (Ruby)2011-12-211-1/+1
|
* removed deprecated loggersKarunakar (Ruby)2011-12-211-1/+1
|
* added :other_than => :!= option to numericality validatorJakub Kuźma2011-12-211-0/+7
|
* Remove dead code from AMo.José Valim2011-12-201-47/+37
|
* Fixed test "ArgumentError: wrong number of arguments(1 for 0)"Arun Agrawal2011-12-171-1/+1
|
* Fixed bug when error message is an empty string.Antonio Roberto2011-12-161-0/+6
|
* Improve cache on route_key lookup.José Valim2011-12-081-0/+8
|
* ActiveModel::Name#i18n_key: Fix doc and add testsMarc-Andre Lafortune2011-12-051-0/+20
|
* Namespaced attribute lookup now works as 'model/association.attribute'.José Valim2011-12-051-5/+5
|
* Merge pull request #3859 from kuroda/human_attribute_nameJosé Valim2011-12-051-0/+10
|\ | | | | Fix human_attribute_name to handle names with dots
| * Fix human_attribute_name to handle names with dotsTsutomu Kuroda2011-12-051-0/+10
| | | | | | | | | | | | | | | | Nested I18n namespace lookup under activerecord.models is deprecated now (c19bd4f). But when a model uses accepts_nested_attributes_for, its Errors object can have an attribute name with "addresses.street" style. In this case, the dots should be substituted with slashes so that we can provide the translation under the "activemodel.attributes.person.addresses/street" key.
* | Revert "Implement ArraySerializer and move old serialization API to a new ↵José Valim2011-11-303-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | namespace." This reverts commit 8896b4fdc8a543157cdf4dfc378607ebf6c10ab0. Conflicts: activemodel/lib/active_model.rb activemodel/lib/active_model/serializable.rb activemodel/lib/active_model/serializer.rb activemodel/test/cases/serializer_test.rb
* | fix method redefined warning in activemodellest2011-11-302-1/+5
| |
* | Deprecated `define_attr_method` in `ActiveModel::AttributeMethods`Jon Leighton2011-11-291-5/+18
| | | | | | | | | | This only existed to support methods like `set_table_name` in Active Record, which are themselves being deprecated.
* | Warning removed unused variableArun Agrawal2011-11-261-1/+1
| |
* | Revert the serializers API as other alternatives are now also under discussionJosé Valim2011-11-251-432/+0
| |
* | Merge branch 'serializers'José Valim2011-11-254-5/+437
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | This implements the ActiveModel::Serializer object. Includes code, tests, generators and guides. From José and Yehuda with love. Conflicts: railties/CHANGELOG.md
| * | Rename UserSerializer to DefaultUserSerializer in tests.José Valim2011-11-231-15/+13
| | |
| * | Rely solely on active_model_serializer and remove the fancy constant lookup.José Valim2011-11-231-34/+3
| | |
| * | Implement ArraySerializer and move old serialization API to a new namespace.José Valim2011-11-234-8/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following constants were renamed: ActiveModel::Serialization => ActiveModel::Serializable ActiveModel::Serializers::JSON => ActiveModel::Serializable::JSON ActiveModel::Serializers::Xml => ActiveModel::Serializable::XML The main motivation for such a change is that `ActiveModel::Serializers::JSON` was not actually a serializer, but a module that when included allows the target to be serializable to JSON. With such changes, we were able to clean up the namespace to add true serializers as the ArraySerializer.
| * | Merge branch 'master' into serializersJosé Valim2011-11-237-40/+128
| |\ \
| * | | Remove 1.9 Hash syntax - tests passing on 1.8.7Jose and Yehuda2011-10-151-3/+3
| | | |
| * | | Add initial support for embed APIJose and Yehuda2011-10-151-4/+82
| | | |
| * | | Add support for the root attributeJose and Yehuda2011-10-151-14/+62
| | | |
| * | | Fix nil has_one associationJose and Yehuda2011-10-151-0/+18
| | | |
| * | | Add association_idsJose and Yehuda2011-10-151-1/+60
| | | |
| * | | Add support for overriding associations, mostly used for authorizationJose and Yehuda2011-10-151-0/+26
| | | |
| * | | Add support for implicit serializersJose and Yehuda2011-10-151-2/+33
| | | |
| * | | Added has_one and has_manyJose and Yehuda2011-10-151-5/+76
| | | |
| * | | Don't require serializable_hash to take options.Jose and Yehuda2011-10-151-2/+2
| | | |
| * | | Initial implementation of ActiveModel::SerializerJose and Yehuda2011-10-151-0/+75
| | | |
* | | | make ActiveModel::Name fail gracefully with anonymous classesJakub Kuźma2011-11-241-0/+13
| |/ / |/| |
* | | Rely on a public contract between railties instead of accessing railtie ↵José Valim2011-11-232-10/+36
| | | | | | | | | | | | methods directly.
* | | Update variable's name in the test case to reflect the class we're testingPrem Sichanugrist2011-11-183-24/+24
| | |
* | | Revert "Merge pull request #2378 from cesario/remove_warnings_activemodel"Jon Leighton2011-11-051-1/+0
| | | | | | | | | | | | | | | | | | | | | This reverts commit 6aaae3de277b572f37e09f16ae12737c3c87dfb7, reversing changes made to fdbc4e5f4e5746ebf558485348c841b33f038fda. Reason: build failure.
* | | Merge pull request #2378 from cesario/remove_warnings_activemodelVijay Dev2011-11-051-0/+1
|\ \ \ | | | | | | | | Remove warnings by calling remove_method
| * | | Remove warnings by calling remove_methodFranck Verrot2011-07-311-0/+1
| | | |
* | | | New #added? method on ActiveModel::ErrorsMartin Svalin2011-10-191-0/+43
| | | | | | | | | | | | 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 setupVijay Dev2011-10-171-6/+4
| | | |
* | | | ActiveModel::Errors#generate_message without i18n_scope, and more test cases ↵Martin Svalin2011-10-171-0/+21
| |/ / |/| | | | | | | | for #add