aboutsummaryrefslogtreecommitdiffstats
path: root/activemodel/lib
Commit message (Collapse)AuthorAgeFilesLines
* after_validation should be called irrespective of the result of validation.Neeraj Singh2010-08-201-1/+1
| | | | | | | | I confirmed that this is the behavior on 2.3.x . [5419 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Merge remote branch 'docrails/master'Xavier Noria2010-08-181-2/+2
|\
| * you rarely want ^ or $ in validations, use \A when you mean \AXavier Noria2010-08-181-2/+2
| |
| * Missing object for comparison in ActiveModel::EachValidator example code.Trey Bean2010-08-171-1/+1
| |
| * Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-1417-222/+222
| | | | | | | | 's/[ \t]*$//' -i {} \;)
| * no callbacks should be created for empty array [#5289 state:resolved]Subba Rao Pasupuleti2010-08-141-3/+6
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
| * tidy up validations length code [#5297 state:resolved]Subba Rao Pasupuleti2010-08-141-6/+2
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
| * Add missing ActiveModel::Validations requireGreg Campbell2010-08-141-0/+1
| | | | | | | | | | | | | | | | | | | | | | [#5311 state: resolved] ActiveModel::Validations uses Hash#except, but does not require it from ActiveSupport. (This wasn't showing up in the tests, because it was required in the helper, and was also required in ActiveModel::Serialization). Signed-off-by: José Valim <jose.valim@gmail.com>
* | to_key should return all exists keys (if any exists), regardless if the ↵José Valim2010-08-152-3/+7
| | | | | | | | object is persisted or not. If you need it to reflect persistance, you should use to_param.
* | Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-1417-222/+222
| | | | | | | | 's/[ \t]*$//' -i {} \;)
* | no callbacks should be created for empty array [#5289 state:resolved]Subba Rao Pasupuleti2010-08-121-3/+6
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | tidy up validations length code [#5297 state:resolved]Subba Rao Pasupuleti2010-08-121-6/+2
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | Add missing ActiveModel::Validations requireGreg Campbell2010-08-121-0/+1
|/ | | | | | | | | | | [#5311 state: resolved] ActiveModel::Validations uses Hash#except, but does not require it from ActiveSupport. (This wasn't showing up in the tests, because it was required in the helper, and was also required in ActiveModel::Serialization). Signed-off-by: José Valim <jose.valim@gmail.com>
* Merge remote branch 'docrails/master'Xavier Noria2010-08-121-3/+3
|\
| * ActiveModel::Base doesn't exist, include_root_in_json is a class_attribute ↵Santiago Pastorino2010-08-101-3/+3
| | | | | | | | for the classes which includes this module
* | fixes a typo reported by rymaiXavier Noria2010-08-101-1/+1
|/
* typo in AMAdam Meehan2010-08-091-1/+1
|
* edit pass to AMo::Serializers::JSONXavier Noria2010-08-061-9/+9
|
* Merge remote branch 'docrails/master'Xavier Noria2010-08-052-2/+4
|\
| * adding documentation to ActiveSupport::Concern ht:strictly typed for an ↵Neeraj Singh2010-08-041-0/+2
| | | | | | | | | | | | awesome example some minor documentation changes
| * fixing documentationNeeraj Singh2010-08-031-2/+2
| |
* | Tidy up error.rb codeSubba Rao Pasupuleti2010-08-031-5/+1
| | | | | | | | | | | | [#5288 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | Freeze options so we raise an error when people modify it in place.José Valim2010-08-032-3/+5
| |
* | validates_length_of should not change the options hash in place. [#5283 ↵José Valim2010-08-031-4/+7
| | | | | | | | state:resolved]
* | Remove or fix non-working examples and add a few tests to Dirty [#5185 ↵Tore Darell2010-08-031-7/+2
|/ | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* ActiveModel::Errors json serialization to work as Rails 3b4 [#5254 ↵Alex Le2010-08-021-2/+2
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Merge remote branch 'rails/master'Xavier Noria2010-07-307-31/+30
|\
| * edit pass to apply API guideline wrt the use of "# =>" in example codeXavier Noria2010-07-306-30/+29
| |
| * Prep for RCDavid Heinemeier Hansson2010-07-261-1/+1
| |
* | expanding on the will_change! method in documentationNeeraj Singh2010-07-291-1/+4
|/
* replacing around with for in the comments for callbacksNeeraj Singh2010-07-221-1/+1
|
* Moved a few methods from RecordIdentifier to ActiveModel::NamingPiotr Sarnacki2010-07-211-0/+29
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* adding proper markup to commentNeeraj Singh2010-07-161-2/+2
|
* Add missing require to use SetSantiago Pastorino2010-07-111-1/+2
|
* Improve a bit the code in latest commits.José Valim2010-07-081-1/+2
|
* Removing method before redefining it. It was causing warnings. [#5072 ↵Łukasz Strzałkowski2010-07-081-0/+1
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Change documentation for ActiveModel::MassAssignmentSecurity a bit and make ↵José Valim2010-07-083-47/+54
| | | | debug always be called since some people may overwrite warn! to add extra behavior even if logger is not available.
* mass_assignment_security moved from AR to AMo, and minor test cleanupJosh Kalderimis2010-07-084-0/+216
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Add a missing require to allow the usage of Array#to_xmlSantiago Pastorino2010-07-031-0/+1
|
* Move constantize from conversions to inflections.Santiago Pastorino2010-06-261-1/+0
| | | | | | | | This removes ActiveModel dependency on TZInfo. [#4979 state:committed] Signed-off-by: José Valim <jose.valim@gmail.com>
* Work around the fact the JSON gem was overwriting to_json implementation for ↵José Valim2010-06-261-10/+7
| | | | | | | all Ruby core classes. This is required because the JSON gem is incompatible with Rails behavior and was not allowing ActiveModel::Errors to be serialized. So we need to ensure Rails implementation is the one triggered. [#4890 state:resolved]
* Regression with how base errors messages are added to a model. Works ↵Josh Kalderimis2010-06-232-2/+2
| | | | | | correctly for both string error messages and symbol translated messages. Signed-off-by: José Valim <jose.valim@gmail.com>
* Refactor previous commit a bit [#4057 state:resolved]José Valim2010-06-217-16/+16
|
* Make ActiveModel::Errors#add_on_blank and #add_on_empty accept an options ↵Jeroen van Dijk2010-06-219-32/+69
| | | | | | | | | | | | | | | | hash and make various Validators pass their (filtered) options. This makes it possible to pass additional options through Validators to message generation. E.g. plugin authors want to add validates_presence_of :foo, :format => "some format". Also, cleanup the :default vs :message options confusion in ActiveModel validation message generation. Also, deprecate ActiveModel::Errors#add_on_blank(attributes, custom_message) in favor of ActiveModel::Errors#add_on_blank(attributes, options). Also, refactoring of ActiveModel and ActiveRecord Validation tests. Test are a lot more DRY now. Better test coverage as well now. The first four points were reapplied from an older patch of Sven Fuchs which didn't apply cleanly anymore and was not complete yet. Signed-off-by: José Valim <jose.valim@gmail.com>
* Merge remote branch 'rails/master'Xavier Noria2010-06-202-2/+66
|\ | | | | | | | | Conflicts: actionpack/lib/abstract_controller/base.rb
| * ActiveModel::Validations::Callbacks should not be required by default.José Valim2010-06-192-13/+23
| |
| * moving before_validation and after_validation functionality from ↵Neeraj Singh2010-06-192-12/+66
| | | | | | | | | | | | | | | | ActiveRecord to ActiveModel [#4653 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Changes call backs to callbacks.Rizwan Reza2010-06-152-9/+9
| |
* | Merge branch 'master' of github.com:lifo/docrailsRizwan Reza2010-06-151-2/+2
|\ \
| * | Active Model lint - fix minor typosBryan Woods2010-06-151-2/+2
| | |