Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | remove depricated Validatior#setup | Kuldeep Aggarwal | 2014-06-10 | 2 | -37/+0 | |
| | | ||||||
* | | Feature requests should be made on the mailing list, not submitted to | Zachary Scott | 2014-06-01 | 1 | -1/+6 | |
| | | | | | | | | the issue tracker. See also #15455 [ci skip] | |||||
* | | [ci skip] Include ActiveModel::Model in a class instead of inheriting | Akshay Vishnoi | 2014-05-30 | 1 | -2/+3 | |
| | | ||||||
* | | Name#model_name doesn't return a String object | ShunsukeAida | 2014-05-30 | 2 | -2/+2 | |
| | | ||||||
* | | Merge pull request #15154 from msgehard/move_password_field | Rafael Mendonça França | 2014-05-20 | 1 | -2/+2 | |
|\ \ | | | | | | | Put attr_reader in with all of the other instance methods | |||||
| * | | Put attr_reader in with all of the other instance methods | Mike Gehard | 2014-05-17 | 1 | -2/+2 | |
| | | | | | | | | | | | | This makes the grouping make a little more sense | |||||
* | | | Let others know why this code is here | Mike Gehard | 2014-05-18 | 1 | -0/+1 | |
|/ / | | | | | | | [ci skip] | |||||
* | | formatting pass through CHANGELOGS. [ci skip] | Yves Senn | 2014-05-16 | 1 | -1/+1 | |
| | | ||||||
* | | Add singular and plural form for some validation messages | Abd ar-Rahman Hamidi | 2014-05-02 | 3 | -6/+28 | |
| | | ||||||
* | | Enhance a bit a few changelog entries [ci skip] | Robin Dupret | 2014-05-02 | 1 | -1/+2 | |
|/ | ||||||
* | Autoload ActiveModel::StrictValidationFailed | hakanensari | 2014-04-16 | 1 | -0/+1 | |
| | | | | | | Currently, if environment doesn’t eager load code, invoking this constant before calling #valid? on a model instance results in a NameError. | |||||
* | add missing parentheses to validates_with documentation [skip ci] | Steve Agalloco | 2014-04-04 | 1 | -1/+1 | |
| | ||||||
* | Fix warning for overshadowing XML variable | Musannif Zahir | 2014-03-28 | 1 | -2/+2 | |
| | ||||||
* | Adds explanation of :base attribute to errors.add | David Underwood | 2014-03-27 | 1 | -0/+7 | |
| | | | [ci skip] | |||||
* | Add CHANGELOG to Active Model too [ci skip] | Rafael Mendonça França | 2014-03-27 | 1 | -0/+6 | |
| | ||||||
* | ActiveRecord/ActiveModel '#validate' alias for 'valid?' | Henrik Nyh | 2014-03-27 | 2 | -0/+13 | |
| | | | | | | | | It's unintuitive to call '#valid?' when you want to run validations but don't care about the return value. The alias in ActiveRecord isn't strictly necessary (the ActiveModel alias is still in effect), but it clarifies. | |||||
* | comment why we are modifying global state. [ci skip] | Yves Senn | 2014-03-11 | 1 | -0/+1 | |
| | ||||||
* | Completely remove potential global state leaks in ActiveModel tests. | Zuhao Wan | 2014-03-11 | 7 | -110/+136 | |
| | | | | ActiveModel tests can now be run in random order. | |||||
* | Merge pull request #14315 from zuhao/activemodel_tests_in_random_order | Yves Senn | 2014-03-09 | 4 | -13/+39 | |
|\ | | | | | Run ActiveModel test suites in random order. | |||||
| * | Run ActiveModel test suites in random order. | Zuhao Wan | 2014-03-10 | 4 | -13/+39 | |
| | | | | | | | | | | | | | | | | | | | | This gets the whole ActiveModel test suites working even if `self.i_suck_and_my_tests_are_order_dependent!` is disabled in `ActiveSupport::TestCase`. Two places are found that potentially leak global state. This patch makes sure states are restored so that none of the changes happen in a single test will be carried over to subsequence tests. | |||||
* | | Fix errors for four of the code samples | Peter Markou | 2014-03-08 | 1 | -30/+49 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The four code samples that fail to run are: - Add attribute magic to objects. Fixed by introducing a Person instance variable. - Tracking value changes. Fixed by replacing `attr_accessor` with `define_attribute_methods`, providing getter and setter methods for `name` and providing the missing `Person#save` method. A call to `Person#save` has to precede the `person.name = 'robert'` assignment, if we want `previous_changes` to include 'bob'. - Adding `errors` interface to objects. Fixed by introducing a Person instance variable, assigning `nil` to its name and calling `Person#validate!`. - Custom validators. Fixed by defining `HasNameValidator` before it is used by `ValidatorPerson`. All the code samples can now be run smoothly. Call Dirty#changes_applied in Person#save, instead of modifying instance vars. | |||||
* | | Tests for indifferent access using full_messages_for. Closes #11916 | Pavel Penkov | 2014-03-08 | 1 | -0/+7 | |
|/ | ||||||
* | Introduce `Rails.gem_version` | Prem Sichanugrist | 2014-03-05 | 2 | -7/+19 | |
| | | | | | | | | | | | | | | | | | | | This method return `Gem::Version.new(Rails.version)`, suggesting a more reliable way to perform version comparison. Example: Rails.version #=> "4.1.2" Rails.gem_version #=> #<Gem::Version "4.1.2"> Rails.version > "4.1.10" #=> false Rails.gem_version > Gem::Version.new("4.1.10") #=> true Gem::Requirement.new("~> 4.1.2") =~ Rails.gem_version #=> true This was originally introduced as `.version` by @charliesome in #8501 but got reverted in #10002 since it was not backward compatible. Also, updating template for `rake update_versions`. | |||||
* | Fix some validators when used on model instance | Eric Hutzelman | 2014-02-26 | 3 | -2/+6 | |
| | | | | | | | | Now that Validator #setup is called from the initializer, we need a reference to the model's class to be passed in to allow the validators to continue functioning when used at the instance level. Closes #14134. | |||||
* | Pass strings to demodulize method | Carlos Antonio da Silva | 2014-02-26 | 1 | -1/+1 | |
| | | | | | Goes along with fea1cdcff4d50d302d8e8532432c3ab107ff816d and 59ec4562a2e70df455b2e44a67c340fa5254e26e. | |||||
* | pass the class name to `tableize` | Aaron Patterson | 2014-02-26 | 1 | -1/+1 | |
| | | | | We should not rely on to_s to return the name of the class | |||||
* | Bump version of bcrypt gem | T.J. Schuck | 2014-02-25 | 1 | -4/+4 | |
| | ||||||
* | Point master changelogs to 4-1-stable branch | Carlos Antonio da Silva | 2014-02-25 | 1 | -72/+1 | |
| | | | | Remove 4-1 related entries from master [ci skip] | |||||
* | update version to 4.2.0.alpha | Yves Senn | 2014-02-23 | 1 | -1/+1 | |
| | ||||||
* | Merge branch '4-1-0-beta2' | Rafael Mendonça França | 2014-02-18 | 1 | -1/+1 | |
|\ | | | | | | | | | | | Conflicts: actionview/CHANGELOG.md activerecord/CHANGELOG.md | |||||
| * | Preparing for 4.1.0.beta2 release | Rafael Mendonça França | 2014-02-18 | 1 | -1/+1 | |
| | | ||||||
* | | add test coverage for activemodel Dirty#reset_changes | Dmitry Polushkin | 2014-02-09 | 1 | -0/+19 | |
| | | ||||||
* | | `#to_param` returns `nil` if `to_key` returns `nil`. Closes #11399. | Yves Senn | 2014-02-04 | 3 | -1/+15 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The documentation of `#to_key` (http://api.rubyonrails.org/classes/ActiveModel/Conversion.html#method-i-to_key) states that it returns `nil` if there are no key attributes. `to_param` needs to be aware of that fact and return `nil` as well. Previously it raised the following exception: ``` 1) Error: ConversionTest#test_to_param_returns_nil_if_to_key_is_nil: NoMethodError: undefined method `join' for nil:NilClass /Users/senny/Projects/rails/activemodel/lib/active_model/conversion.rb:65:in `to_param' /Users/senny/Projects/rails/activemodel/test/cases/conversion_test.rb:34:in `block in <class:ConversionTest>' ``` | |||||
* | | tidy CHANGELOGs [ci skip] | Yves Senn | 2014-01-30 | 1 | -2/+5 | |
| | | ||||||
* | | use the new clear_validators! api everywhere to reset validators in tests | Kuldeep Aggarwal | 2014-01-28 | 1 | -1/+1 | |
| | | ||||||
* | | Fix doc markup of clear_validators! | Carlos Antonio da Silva | 2014-01-27 | 1 | -2/+2 | |
| | | ||||||
* | | Use the new clear_validators! api to reset validators in tests | Carlos Antonio da Silva | 2014-01-27 | 16 | -42/+33 | |
| | | ||||||
* | | Merge tests about multiple validation contexts | Carlos Antonio da Silva | 2014-01-27 | 1 | -10/+6 | |
| | | ||||||
* | | Ability to specify multiple contexts when defining a validation. | Vince Puzzella | 2014-01-27 | 3 | -5/+45 | |
| | | | | | | | | | | | | Example: validates_presence_of :name, on: [:update, :custom_validation_context] | |||||
* | | revises references to :allow_(nil|blank) in some docs [ci skip] [Steven Yang ↵ | Xavier Noria | 2014-01-26 | 10 | -26/+12 | |
| | | | | | | | | | | | | & Xavier Noria] Closes #11247. | |||||
* | | Some minor fixes | Godfrey Chan | 2014-01-24 | 2 | -3/+2 | |
| | | ||||||
* | | Removed old tests | Godfrey Chan | 2014-01-24 | 3 | -250/+121 | |
| | | ||||||
* | | Got all the new tests passing | Godfrey Chan | 2014-01-24 | 1 | -10/+8 | |
| | | ||||||
* | | Restored the ability to clear the password with user.password= nil (see the ↵ | Godfrey Chan | 2014-01-24 | 1 | -1/+3 | |
| | | | | | | | | docs) | |||||
* | | Rewrote the tests for has_secure_password | Godfrey Chan | 2014-01-24 | 3 | -2/+183 | |
| | | ||||||
* | | Update ActiveModel::Errors.has_key? test | Adrien Coquio | 2014-01-22 | 1 | -1/+1 | |
| | | ||||||
* | | Fix ActiveModel::Errors#has_key? return value | Adrien Coquio | 2014-01-22 | 1 | -1/+1 | |
| | | ||||||
* | | Add failing test for ActiveModel::Errors#has_key? method | Adrien Coquio | 2014-01-22 | 1 | -0/+5 | |
| | | | | | | | | From the doc, this method should return false and not nil if there is no errors for this key | |||||
* | | Fix typo | Rafael Mendonça França | 2014-01-21 | 1 | -1/+1 | |
| | | ||||||
* | | Forgot to push this change in the parent commit | Rafael Mendonça França | 2014-01-20 | 1 | -1/+1 | |
| | |