aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix calling quote column name in interpolated stringCarlos Antonio da Silva2012-12-211-1/+1
|
* Merge pull request #8267 from marcandre/reversible_drop_table_etcAaron Patterson2012-12-2115-307/+841
|\ | | | | Reversible commands
| * Update Migration and 4.0 Release Guides, Changelogs [#8267]Marc-Andre Lafortune2012-12-214-36/+223
| |
| * Make execute, change_column and remove_columns methods actually irreversibleMarc-Andre Lafortune2012-12-212-2/+9
| | | | | | | | [#8267]
| * Make change_table reversible when possible [#8267]Marc-Andre Lafortune2012-12-212-1/+25
| |
| * Migration generators use `change` even for destructive methods [#8267]Marc-Andre Lafortune2012-12-212-47/+16
| |
| * Factorize methods that are easily reversible [#8267]Marc-Andre Lafortune2012-12-212-42/+28
| |
| * Make remove_index reversible [#8267]Marc-Andre Lafortune2012-12-212-7/+34
| |
| * Differentiate between remove_column and remove_columns. Make remove_column ↵Marc-Andre Lafortune2012-12-216-19/+44
| | | | | | | | | | | | reversible. [#8267]
| * Make drop_table reversible [#8267]Marc-Andre Lafortune2012-12-214-17/+41
| |
| * Add drop_join_table [#8267]Marc-Andre Lafortune2012-12-214-7/+69
| |
| * Add Migration#reversible for reversible data operations [#8267]Marc-Andre Lafortune2012-12-213-1/+85
| |
| * Allow revert of whole migration [#8267]Marc-Andre Lafortune2012-12-212-3/+102
| |
| * Extract exec_migration [#8267]Marc-Andre Lafortune2012-12-211-11/+16
| |
| * Allow reverting of migration commands with Migration#revert [#8267]Marc-Andre Lafortune2012-12-214-90/+155
| |
| * Split ridiculously long lineMarc-Andre Lafortune2012-12-211-1/+4
| |
| * Simple replacement of variable name for consistencyMarc-Andre Lafortune2012-12-211-32/+32
| |
| * Simplify change_table and avoid duplicated logicMarc-Andre Lafortune2012-12-212-40/+7
| |
* | Merge pull request #8584 from garysweaver/security_guide_updateGuillermo Iguaran2012-12-211-3/+3
|\ \ | | | | | | Updated security guide to reference secret_key_base instead of secret_token
| * | Updated security guide to reference secret_key_base instead of secret_token, ↵Gary S. Weaver2012-12-211-3/+3
|/ / | | | | | | with a little information about the change from CookieStore to EncryptedCookieStore.
* | Merge pull request #7376 from ↵Rafael Mendonça França2012-12-214-15/+100
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | dmitriy-kiriyenko/fix-double-callback-in-same-statement Prevent callback from being set twice. Conflicts: activesupport/CHANGELOG.md
| * | Prevent callback from being set twice.Dmitriy Kiriyenko2012-12-204-15/+100
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When you add one callack in two separate `set_callback` calls - it is only called once. When you do it in one `set_callback` call - it is called twice. This violates the principle of least astonishment for me. Duplicating callback is usually an error. There is a correct and obvious way to do anything without this "feature". If you want to do before_save :clear_balance, :calculate_tax, :clear_balance or whatever, you should better do before_save :carefully_calculate_tax def carefully_calculate_tax clear_balance calculate_tax clear_balance end And this even opens gates for some advanced refactorings, unlike the first approach. My assumptions are: - Principle of least astonishment is violated, when callbacks are either prevented from duplication, or not. - Duplicating callbacks is usually an error. When it is intentional - it's a smell of a bad design and can be approached without abusing this "feature". My suggestion is: do not allow duplicating callbacks in one callback call, like it is not allowed in separate callbacks call.
* | | Remove CHANGELOG entry for ActiveModel::Errors#add_on_presentRafael Mendonça França2012-12-211-5/+0
| | |
* | | Use :present as key for the absence validatior messageRafael Mendonça França2012-12-212-2/+2
| | |
* | | Remove ActiveModel::Errors#add_on_present method.Rafael Mendonça França2012-12-213-16/+3
| | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge pull request #8581 from garysweaver/security_guide_updateGuillermo Iguaran2012-12-211-7/+11
|\ \ \ | | | | | | | | Update security guide
| * | | Updated security guide with information about secret_token.rb and to suggest ↵Gary S. Weaver2012-12-211-7/+11
| | |/ | |/| | | | | | | securing sensitive files like database.yml and secret_token.rb
* | | BufferedLogger is deprecated too.Rafael Mendonça França2012-12-211-1/+1
| | |
* | | Fix broken test for postgresqlRafael Mendonça França2012-12-211-1/+1
|/ / | | | | | | | | | | For some reason postgresql doesn't pass an integer value to load. cc @tenderlove
* | Revert "Make sure that ActiveSupport::Logger includes the Logger extensions ↵David Heinemeier Hansson2012-12-214-21/+30
| | | | | | | | | | | | from core_ext/logger" (some confusion over deprecation) This reverts commit d00f568a83a5159ed93618b1081bd17858536d1c.
* | Make sure that ActiveSupport::Logger includes the Logger extensions from ↵David Heinemeier Hansson2012-12-214-30/+21
| | | | | | | | core_ext/logger
* | Add ActiveSupport::Logger#silence that works the same as the old ↵David Heinemeier Hansson2012-12-215-20/+45
| | | | | | | | Logger#silence extension
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-12-214-23/+45
|\ \
| * | copy edits in assets guide [ci skip]Vijay Dev2012-12-211-23/+26
| | |
| * | Revert "Fix incorrect adjustment 4c41e87e3ae548c44810b66437b2f0f6e73b2106"Vijay Dev2012-12-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e1f8ec59f2cc83f052b15233147aa2d6d8114a4d. Reason: seems bad styling [ci skip]
| * | Fix incorrect adjustment 4c41e87e3ae548c44810b66437b2f0f6e73b2106kei2012-12-201-1/+1
| | |
| * | Fix documentation stylekei2012-12-201-1/+1
| | |
| * | Remove Reference to Ruby 1.8.7John Kelly2012-12-201-2/+1
| | |
| * | Explain controller specific assets more thoroughlyDamian Galarza2012-12-171-17/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | The current section on controller specific assets does not really explain how the default application.css and application.js files generated by rails will work with controller specific assets and is a bit ambiguous. We should remind users that they will be included into their application by default but that they have the option to include them only where needed if they want and how this works with precompiling assets. [ci_skip]
| * | Add script_name option description and example for #url_for optionsIvan Vanyak2012-12-171-0/+5
| | |
* | | Serialized attribute can be serialized in an integer columnRafael Mendonça França2012-12-215-3/+47
| | | | | | | | | | | | Fix #8575
* | | Wrap table creation in a transaction.Rafael Mendonça França2012-12-211-6/+8
| | | | | | | | | | | | This will make the tests pass when the intrange datatype is not present
* | | Fix doc pointing to non existent logger class [ci skip]Carlos Antonio da Silva2012-12-211-1/+1
| | |
* | | Fix indentation issuing warningCarlos Antonio da Silva2012-12-201-1/+1
| | |
* | | Merge pull request #8560 from u16suzu/masterRafael Mendonça França2012-12-201-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix document for String#humanize Conflicts: activesupport/lib/active_support/core_ext/string/inflections.rb
| * | | Fix: documentation for String#humanizeYuichiro Suzuki2012-12-201-2/+2
| | | |
* | | | Merge pull request #8567 from frodsan/update_coding_conventions_journeyRafael Mendonça França2012-12-2017-438/+440
|\ \ \ \ | | | | | | | | | | update AD::Journey to follow Rails coding conventions
| * | | | update AD::Journey to follow Rails coding conventionsFrancesco Rodriguez2012-12-2017-438/+440
| | | | |
* | | | | Fix CHANGELOG entry about scope changeGuillermo Iguaran2012-12-201-1/+1
|/ / / / | | | | | | | | I missed attribution on this :flushed:
* | | | Fix Action Mailer changelog indenation to match other changelogsCarlos Antonio da Silva2012-12-194-27/+29
| | | | | | | | | | | | | | | | Also some minor improvements to other changelogs. [ci skip]
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-06 23:37:07 +0000