Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | provide a more sementicthe local variables name for ↵ | Steven Yang | 2013-06-29 | 1 | -8/+8 | |
| | | | | | | | ActiveModel::Validations::Clusivity#include? method the original name `exclusion` is a bit confusing when using with the method `inclusion_method` rename it to a more logic neutral name. | |||||
* | Use Range#cover? for Numeric ranges (tests via endpoints) and use ↵ | Charles Bergeron | 2013-05-27 | 2 | -4/+5 | |
| | | | | | | Range#include? for non-numeric ranges added changelog message | |||||
* | deprecate Validator#setup (to get rid of a respond_to call). validators do ↵ | Nick Sutterer | 2013-05-23 | 3 | -3/+12 | |
| | | | | their setup in their constructor now. | |||||
* | deprecating string based terminators | Aaron Patterson | 2013-05-14 | 1 | -1/+4 | |
| | ||||||
* | Convert ActiveModel to 1.9 hash syntax. | Patrick Robertson | 2013-05-01 | 9 | -15/+15 | |
| | | | | | I also attempted to fix other styleguide violations such as { a: :b } over {a: :b} and foo(b: 'bar') over foo( b: 'bar' ). | |||||
* | `validates_confirmation_of` does not override writer methods. | Yves Senn | 2013-03-04 | 1 | -1/+5 | |
| | ||||||
* | Reduce number of Strings a bit | Akira Matsuda | 2013-01-07 | 2 | -3/+3 | |
| | ||||||
* | Revert the change at ActiveModel::Errors#add_on_blank and fix in the | Rafael Mendonça França | 2012-12-26 | 1 | -2/+2 | |
| | | | | | | | | | | | right place. The EachValidator#validate already handle :allow_blank and :allow_nil, correctly. Closes #8622. Fix #8621. | |||||
* | Use :present as key for the absence validatior message | Rafael Mendonça França | 2012-12-21 | 1 | -1/+1 | |
| | ||||||
* | Remove ActiveModel::Errors#add_on_present method. | Rafael Mendonça França | 2012-12-21 | 1 | -2/+2 | |
| | | | | | | | | We don't need to define a new method in ActiveMode::Errors for each validatior. See https://github.com/rails/rails/commit/d72a07f1d1478db9daed847eadb35bfd840674f6#commitcomment-2325333 | |||||
* | Add `ActiveModel::Validations::AbsenceValidator`, a validator to check the ↵ | Roberto Vasquez Angel | 2012-12-15 | 1 | -0/+31 | |
| | | | | | | absence of attributes. Add `ActiveModel::Errors#add_on_present` method. Adds error messages to present attributes. | |||||
* | Merge pull request #7282 from xHire/validates_length_of_fix | Rafael Mendonça França | 2012-11-26 | 1 | -2/+14 | |
|\ | | | | | | | | | | | | | Length validation handles correctly nil. Fix #7180 Conflicts: activemodel/CHANGELOG.md | |||||
| * | Length validation handles correctly nil. Fix #7180 | Michal Zima | 2012-11-26 | 1 | -2/+14 | |
| | | | | | | | | When nil or empty string are not allowed, they are not valid. | |||||
* | | cleanup, removed dispensable `require` statements from `ActiveModel` | Yves Senn | 2012-11-25 | 1 | -2/+0 | |
| | | ||||||
* | | cleanup, remove broken whitespace | Yves Senn | 2012-11-25 | 1 | -2/+2 | |
|/ | ||||||
* | Set hash value instead of using merge! | Carlos Antonio da Silva | 2012-11-04 | 1 | -1/+1 | |
| | ||||||
* | minor edits and remove mixed titles in AM::Validations docs [ci skip] | Francesco Rodriguez | 2012-10-25 | 9 | -21/+13 | |
| | ||||||
* | minor edits in AM documentation [ci skip] | Francesco Rodriguez | 2012-10-21 | 8 | -12/+12 | |
| | ||||||
* | Switch to using include? on validation callbacks | John Foley | 2012-09-23 | 1 | -2/+2 | |
| | ||||||
* | Fix collisions with before and after validation callbacks. | John Foley | 2012-09-23 | 1 | -2/+6 | |
| | | | | | | This commit allows a user to do something like: before_validation :do_stuff, :on => [ :create, :update ] after_validation :do_more, :on => [ :create, :update ] | |||||
* | remove '.rb' from require call | Nihad Abbasov | 2012-09-12 | 1 | -1/+1 | |
| | ||||||
* | Added forgotten :message option to ActiveModel validates documentation | Anatoly Makarevich | 2012-09-07 | 1 | -3/+3 | |
| | ||||||
* | Accept a symbol for `:in` option on inclusion and exclusion validators | Gabriel Sobrinho | 2012-08-24 | 3 | -5/+14 | |
| | ||||||
* | AM::Validation#validates: ability to pass custom exception to `:strict` option | Bogdan Gusiev | 2012-08-06 | 1 | -2/+5 | |
| | ||||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-08-04 | 3 | -65/+130 | |
|\ | | | | | | | | | | | Conflicts: activemodel/lib/active_model/secure_password.rb activerecord/lib/active_record/associations/collection_proxy.rb | |||||
| * | fix #after_validation example [ci skip] | Francesco Rodriguez | 2012-07-30 | 1 | -1/+1 | |
| | | ||||||
| * | update ActiveModel::Validations::Callbacks documentation [ci skip] | Francesco Rodriguez | 2012-07-29 | 1 | -18/+69 | |
| | | ||||||
| * | update #validates and #validates! documentation [ci skip] | Francesco Rodriguez | 2012-07-29 | 1 | -31/+43 | |
| | | ||||||
| * | update ActiveModel::Valdations#validates_with docs [ci skip] | Francesco Rodriguez | 2012-07-29 | 1 | -16/+18 | |
| | | ||||||
* | | Merge pull request #7253 from acapilleri/lenght_validator_errors_options | Rafael Mendonça França | 2012-08-03 | 1 | -2/+2 | |
|\ \ | | | | | | | calculate errors_options one time in validate_each | |||||
| * | | calculate errors_options one time in validate_each | Angelo Capilleri | 2012-08-03 | 1 | -2/+2 | |
| |/ | | | | | | | | | | | errors_options is calculated for every *CHECKS* but it could be calculated one time because options and RESERVED_OPTIONS not change during this cycle | |||||
* / | load active_support/core_ext/object/blank in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 | |
|/ | ||||||
* | Update documentation and change the range method name to delimiter. | Rafael Mendonça França | 2012-07-20 | 3 | -5/+7 | |
| | ||||||
* | Don't pass `:within` option to the i18n | Rafael Mendonça França | 2012-07-20 | 2 | -2/+2 | |
| | ||||||
* | `validates_inclusion_of` and `validates_exclusion_of` now accept | Rafael Mendonça França | 2012-07-20 | 1 | -3/+6 | |
| | | | | | | `:within` option as alias of `:in` as documented. Fix #7118 | |||||
* | AM::Validations: remove documentation duplicates | Bogdan Gusiev | 2012-07-10 | 9 | -119/+52 | |
| | ||||||
* | add :nodoc: to internal implementations of AM::Validator [ci skip] | Francesco Rodriguez | 2012-07-06 | 1 | -1/+1 | |
| | ||||||
* | fix syntax of AM::Validations::HelperMethods examples [ci skip] | Francesco Rodriguez | 2012-07-05 | 4 | -6/+6 | |
| | ||||||
* | update ActiveModel::Validations::HelperMethods documentation [ci skip] | Francesco Rodriguez | 2012-07-05 | 8 | -130/+141 | |
| | ||||||
* | Use default + merge! instead of reverse_merge | Carlos Antonio da Silva | 2012-06-26 | 1 | -1/+1 | |
| | | | | | Since we're not directly requiring the reverse_merge extension, we can avoid another hash creation by using default_hash.merge! instead. | |||||
* | Set hash value instead of merge a single key, and use flatten! if possible | Carlos Antonio da Silva | 2012-06-26 | 1 | -2/+4 | |
| | | | | | | | | There's no need to create two extra hashes with options.merge(another_hash), with the goal of setting only one value, so lets just set it. Also refactor validates_each to use _merge_attributes, like other validates_* helpers do. | |||||
* | add :nodoc: to internal implementations [ci skip] | Francesco Rodriguez | 2012-06-25 | 9 | -16/+16 | |
| | ||||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-06-14 | 2 | -4/+3 | |
|\ | ||||||
| * | Range#cover? is not implemented in AS now | Akira Matsuda | 2012-06-12 | 2 | -4/+3 | |
| | | ||||||
* | | prevent users from unknowingly using bad regexps that can compromise ↵ | MrBrdo | 2012-06-14 | 1 | -3/+20 | |
|/ | | | | security (http://homakov.blogspot.co.uk/2012/05/saferweb-injects-in-various-ruby.html) | |||||
* | Don't enable validations when passing false hash values to ActiveModel.validates | Steve Purcell | 2012-05-28 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Passing a falsey option value for a validator currently causes that validator to be enabled, just like "true": ActiveModel.validates :foo, :presence => false This is rather counterintuitive, and makes it inconvenient to wrap `validates` in methods which may conditionally enable different validators. As an example, one is currently forced to write: def has_slug(source_field, options={:unique => true}) slugger = Proc.new { |r| r[:slug] = self.class.sluggify(r[source_field]) if r[:slug].blank? } before_validation slugger validations = { :presence => true, :slug => true } if options[:unique] validations[:uniqueness] = true end validates :slug, validations end because the following reasonable-looking alternative fails to work as expected: def has_slug(source_field, options={:unique => true}) slugger = Proc.new { |r| r[:slug] = self.class.sluggify(r[source_field]) if r[:slug].blank? } before_validation slugger validates :slug, :presence => true, :slug => true, :uniqueness => options[:unique] end (This commit includes a test, and all activemodel and activerecord tests pass as before.) | |||||
* | fixing removed empty lines and examples | Francesco Rodriguez | 2012-05-16 | 8 | -6/+11 | |
| | ||||||
* | fixing validates_with docs | Francesco Rodriguez | 2012-05-16 | 1 | -6/+4 | |
| | ||||||
* | better format and fixes to validators docs | Francesco Rodriguez | 2012-05-16 | 8 | -104/+154 | |
| | ||||||
* | better example format in validates_exclusion_of docs | Francesco Rodriguez | 2012-05-16 | 1 | -2/+2 | |
| |