Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | Merge pull request #6063 from marcandre/observer_extra_args | Aaron Patterson | 2012-04-30 | 1 | -9/+17 | |
|\ \ \ \ | | | | | | | | | | | Allow extra arguments for Observers | |||||
| * | | | | Allow extra arguments for Observers | Marc-Andre Lafortune | 2012-04-30 | 1 | -9/+17 | |
| | | | | | ||||||
* | | | | | Merge pull request #6071 from marcandre/observer_redef | Jeremy Kemper | 2012-04-29 | 1 | -1/+1 | |
|\ \ \ \ \ | | | | | | | | | | | | | Fix Observer by acting on singleton class. Fixes #3505. | |||||
| * | | | | | Fix Observer by acting on singleton class [#3505] | Marc-Andre Lafortune | 2012-04-29 | 1 | -1/+1 | |
| |/ / / / | | | | | | | | | | | | | | | | Also [issue #1034] [pull #6068] | |||||
* | | | | | Merge pull request #5942 from ↵ | Aaron Patterson | 2012-04-29 | 2 | -2/+3 | |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | bcardarella/confirmation_error_message_on_confirmation_attribute confirmation validation error attribute | |||||
| * | | | | | Support i18n attributes for confirmation | Brian Cardarella | 2012-04-24 | 1 | -1/+2 | |
| | | | | | | ||||||
| * | | | | | confirmation validation error attribute | Brian Cardarella | 2012-04-23 | 2 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will render the error message on :#{attribute}_confirmation instead of on attribute itself. When rendering confirmation errors inline on the form with form builders such as SimpleForm and Formtastic it is confusing to the ender user to see the confirmation error message on the attribute element. Instead it makes more sense to have this validation error render on the confirmation field instead. The i18n message has been updated for the confirmation validator error message to include the original attribute name. | |||||
* | | | | | | Generate appropriate error more judiciously | Marc-Andre Lafortune | 2012-04-29 | 1 | -2/+3 | |
| | | | | | | ||||||
* | | | | | | Fix error message: | Marc-Andre Lafortune | 2012-04-28 | 1 | -3/+3 | |
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | - can pass the class, not the instance - "instance method" is confusing, use "method :instance" instead | |||||
* | | | | | Refactor human attribute name | Carlos Antonio da Silva | 2012-04-28 | 2 | -7/+7 | |
| | | | | | ||||||
* | | | | | Do not modify options hash in human_attribute_name, remove reverse_merge | Carlos Antonio da Silva | 2012-04-28 | 1 | -3/+2 | |
| | | | | | ||||||
* | | | | | Merge pull request #6025 from carlosantoniodasilva/amo-errors-refactor | José Valim | 2012-04-28 | 1 | -7/+7 | |
|\ \ \ \ \ | | | | | | | | | | | | | Refactor AMo::Errors | |||||
| * | | | | | Refactor AMo::Errors | Carlos Antonio da Silva | 2012-04-27 | 1 | -7/+7 | |
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | * Use merge!, remove reverse_merge * Use case statement instead of is_a? checks | |||||
* / | | | | Return earlier if attribute already changed in *_will_change! methods | Carlos Antonio da Silva | 2012-04-27 | 1 | -1/+3 | |
|/ / / / | | | | | | | | | | | | | Avoid obtaining the value and duplicating it when possible. | |||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-28 | 3 | -6/+19 | |
|\ \ \ \ | ||||||
| * | | | | Use <tt>Foo::Bar</tt> instead of +Foo::Bar+ | Mark Rushakof | 2012-04-27 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The latter doesn't render as code in HTML output. Regex used in Rubymine to locate the latter form: (\+)(:*\w+:(?::|\w)+)(\+) | |||||
| * | | | | minor docs improvements | Oscar Del Ben | 2012-04-26 | 2 | -4/+4 | |
| | | | | | ||||||
| * | | | | Add documentation for validate options | Oscar Del Ben | 2012-04-26 | 1 | -0/+13 | |
| | | | | | ||||||
| * | | | | Enhance validations documentation | Oscar Del Ben | 2012-04-26 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Merge pull request #5878 from oscardelben/refactor_observed_class | Vijay Dev | 2012-04-25 | 1 | -5/+2 | |
|\ \ \ \ \ | |/ / / / |/| | | | | Refactor Observer.observed_class | |||||
| * | | | | Refactor Observer.observed_class | Oscar Del Ben | 2012-04-25 | 1 | -5/+2 | |
| | | | | | ||||||
* | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-25 | 2 | -6/+13 | |
|\ \ \ \ \ | ||||||
| * | | | | | fix code syntax | Oscar Del Ben | 2012-04-25 | 1 | -2/+2 | |
| | | | | | | ||||||
| * | | | | | Add include documentation to serialization | Oscar Del Ben | 2012-04-25 | 1 | -1/+6 | |
| | | | | | | ||||||
| * | | | | | improve serialization doc | Oscar Del Ben | 2012-04-25 | 1 | -4/+5 | |
| | | | | | | ||||||
| * | | | | | Correct documentation of SecurePassword | Oscar Del Ben | 2012-04-24 | 1 | -1/+2 | |
| | |/ / / | |/| | | | ||||||
* | | | | | Minor refactor of serializable hash | Oscar Del Ben | 2012-04-25 | 1 | -2/+1 | |
| |/ / / |/| | | | ||||||
* | | | | Merge pull request #5841 from oscardelben/rename_count_observers | José Valim | 2012-04-24 | 1 | -1/+8 | |
|\ \ \ \ | | | | | | | | | | | Rename Observing#count_observers to Observing#observers_count | |||||
| * | | | | Rename Observing#count_observers to Observing#observers_count | Oscar Del Ben | 2012-04-14 | 1 | -1/+8 | |
| | | | | | ||||||
* | | | | | Merge pull request #5966 from oscardelben/fix_secure_password_setter | José Valim | 2012-04-24 | 1 | -1/+1 | |
|\ \ \ \ \ | | | | | | | | | | | | | Fix secure_password setter | |||||
| * | | | | | Fix secure_password setter | Oscar Del Ben | 2012-04-24 | 1 | -1/+1 | |
| | |/ / / | |/| | | | ||||||
* / | | | | Refactor SecurePassword#authenticate | Oscar Del Ben | 2012-04-24 | 1 | -5/+1 | |
|/ / / / | ||||||
* | | | | Improve observers documentation | Oscar Del Ben | 2012-04-18 | 1 | -0/+2 | |
| | | | | ||||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-18 | 3 | -11/+12 | |
|\ \ \ \ | ||||||
| * | | | | Fix confusing example in ActiveModel::Errors | Thibaut Courouble | 2012-04-17 | 1 | -2/+2 | |
| | | | | | ||||||
| * | | | | fix ActiveModel::Validations::Callbacks doc | Jan Xie | 2012-04-15 | 1 | -1/+2 | |
| | | | | | ||||||
| * | | | | fix bad format [ci skip] | Michael de Silva | 2012-04-14 | 1 | -7/+7 | |
| | | | | | ||||||
| * | | | | Improve example in initializer | Oscar Del Ben | 2012-04-14 | 1 | -1/+1 | |
| |/ / / | ||||||
* / / / | Rename notify_observers argument *arg to *args to make it more clear | Oscar Del Ben | 2012-04-14 | 1 | -2/+2 | |
|/ / / | | | | | | | | | | that the method accepts multiple arguments | |||||
* | | | Merge pull request #5736 from oscardelben/configuration_errors_reporting | Jon Leighton | 2012-04-11 | 1 | -3/+3 | |
|\ \ \ | | | | | | | | | Add better errors reporting for ActiveModel::Configuration | |||||
| * | | | Add better errors reporting for ActiveModel::Configuration | Oscar Del Ben | 2012-04-04 | 1 | -3/+3 | |
| |/ / | ||||||
* / / | Updated/changed useless tr/gsubs | Jurriaan Pruis | 2012-04-03 | 1 | -1/+1 | |
|/ / | ||||||
* / | Don't include deprecation in AM::Naming anymore | Grant Hutchins | 2012-03-31 | 1 | -1/+0 | |
|/ | ||||||
* | Merge pull request #4904 from ask4prasath/refactor_dirty_module_with_helper | José Valim | 2012-03-28 | 1 | -1/+2 | |
|\ | | | | | Changing active model dirty module helper method to more appropriate met... | |||||
| * | Changing active model dirty module helper method to more appropriate method ↵ | Prasath Venkatraman | 2012-02-06 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | with helper #4903 Changing active model dirty module helper method to more appropriate method with helper with Active Support Helper #4903 Changing active model dirty module helper method to more appropriate method with helper with Active Support Helper typo fixed #4903 | |||||
* | | Remove code duplication in InclusionValidator and ExclusionValidator. | Rafael Mendonça França | 2012-03-28 | 3 | -44/+39 | |
| | | | | | | | | | | | | | | | | Orginal code by @jamescook Closes #1352 [Rafael Mendonça França + James Cook] | |||||
* | | Allow ActiveModel::Name to duck type | Piotr Sarnacki | 2012-03-28 | 1 | -4/+4 | |
| | | ||||||
* | | no need to freeze things all the time | Aaron Patterson | 2012-03-26 | 1 | -8/+7 | |
| | | ||||||
* | | ActiveModel::Name does not inherit from string | Lukasz Sarnacki | 2012-03-26 | 2 | -12/+17 | |
| | | ||||||
* | | Merge pull request #5374 from ↵ | José Valim | 2012-03-23 | 1 | -4/+1 | |
|\ \ | | | | | | | | | | | | | nertzy/remove_deprecated_partial_path_from_active_model_naming Remove ActiveModel::Naming#partial_path |