Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | we don't need to to_s the column | Aaron Patterson | 2013-07-01 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Add missing require | Carlos Antonio da Silva | 2013-07-01 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit cfa35c55bb38b27af305ad21408e181bc3cd739e removed the buffered_logger from AS, which required the logger file. Now we need to add the proper require here. | |||||
* | | | | | the data structure used to store attribute aliases should not be exposed | Aaron Patterson | 2013-07-01 | 2 | -2/+12 | |
| | | | | | ||||||
* | | | | | remove deprecated `ActiveRecord::Base#connection` method. | Yves Senn | 2013-07-01 | 3 | -12/+5 | |
| | | | | | ||||||
* | | | | | Merge pull request #11227 from vipulnsward/form_builder | Yves Senn | 2013-07-01 | 3 | -17/+5 | |
|\ \ \ \ \ | | | | | | | | | | | | | Remove `FormBuilder` deprecation warning about block argument and tests | |||||
| * | | | | | Remove `FormBuilder` deprecation warning about block argument and associated ↵ | Vipul A M | 2013-07-02 | 3 | -17/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | tests | |||||
* | | | | | | remove auto-explain-config deprecation warning | Yves Senn | 2013-07-01 | 2 | -11/+4 | |
| | | | | | | ||||||
* | | | | | | remove deprecated `BufferedLogger`. | Yves Senn | 2013-07-01 | 4 | -44/+4 | |
|/ / / / / | ||||||
* | | | | | Merge pull request #11224 from senny/remove_assert_blank_and_assert_present | Rafael Mendonça França | 2013-07-01 | 3 | -78/+4 | |
|\ \ \ \ \ | | | | | | | | | | | | | remove deprecated `assert_present` and `assert_blank`. | |||||
| * | | | | | remove deprecated `assert_present` and `assert_blank`. | Yves Senn | 2013-07-01 | 3 | -78/+4 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #11226 from senny/remove_deprecated_distinct_option | Rafael Mendonça França | 2013-07-01 | 3 | -15/+4 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | Remove deprecated `:distinct` option from `Relation#count`. | |||||
| * | | | | | Remove deprecated `:distinct` option from `Relation#count`. | Yves Senn | 2013-07-01 | 3 | -15/+4 | |
|/ / / / / | ||||||
* | | | | | Merge pull request #11220 from aditya-kapoor/add-concern | Carlos Antonio da Silva | 2013-07-01 | 1 | -4/+3 | |
|\ \ \ \ \ | | | | | | | | | | | | | Change Rails Migration generator module to use AS::Concern | |||||
| * | | | | | Using ActiveSupport::Concern instead of plain regular Ruby hooks in ↵ | aditya-kapoor | 2013-07-01 | 1 | -4/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | railties/lib/rails/generators/migration.rb file | |||||
* | | | | | | Merge pull request #11221 from neerajdotname/partial_updates_family | Carlos Antonio da Silva | 2013-07-01 | 3 | -25/+5 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Removed deprecated methods partial_updates and family | |||||
| * | | | | | | Removed deprecated methods partial_updates and family | Neeraj Singh | 2013-07-02 | 3 | -25/+5 | |
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Removed deprecated methods `partial_updates`, `partial_updates?` and `partial_updates=` | |||||
* | | | | | | Merge pull request #11213 from neerajdotname/scoped-deprecated | Rafael Mendonça França | 2013-07-01 | 2 | -5/+4 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed deprecated scoped method Conflicts: activerecord/CHANGELOG.md | |||||
| * | | | | | | Removed deprecated method scoped | Neeraj Singh | 2013-07-01 | 2 | -5/+4 | |
| | | | | | | | ||||||
* | | | | | | | Merge pull request #11212 from neerajdotname/default_scopes_q_deprecated | Rafael Mendonça França | 2013-07-01 | 2 | -8/+4 | |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | Removed deprecated default_scopes? | |||||
| * | | | | | | | Removed deprecated method default_scopes? | Neeraj Singh | 2013-07-01 | 2 | -8/+4 | |
| | | | | | | | | ||||||
* | | | | | | | | Merge pull request #11203 from schneems/schneems/plugin_new-plugin | Rafael Mendonça França | 2013-07-01 | 35 | -16/+20 | |
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | s/plugin_new/plugin | |||||
| * | | | | | | | s/plugin_new/plugin | schneems | 2013-06-30 | 35 | -16/+20 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are historical reasons that the `plugin` command was `plugin_new`, now those are no longer applicable, we should remove the naming edge case from the project. This PR is based off of comments from #11176 ATP Railties | |||||
* | | | | | | | | Merge pull request #11207 from vipulnsward/remove_escapes | Carlos Antonio da Silva | 2013-07-01 | 1 | -6/+6 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | Remove redundant escapes from xml serialization test | |||||
| * | | | | | | | | Remove redundant escapes from xml serialization test | Vipul A M | 2013-07-01 | 1 | -6/+6 | |
| | |/ / / / / / | |/| | | | | | | ||||||
* | | | | | | | | Merge pull request #11211 from gsamokovarov/trailing-comma | Guillermo Iguaran | 2013-07-01 | 1 | -10/+9 | |
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | | | Further clean-up of ActiveSupport::Callbacks | |||||
| * | | | | | | | Further clean-up of ActiveSupport::Callbacks | Genadi Samokovarov | 2013-07-01 | 1 | -10/+9 | |
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In #11195 I noticed a trailing comma in the docs, but I decided to further clean it up. What I have done: * Clean up the trailing comma in the docs and some extra whitespace lines. * Used `Array#extract` options to factor the repetitive pattern of `args.last.is_a(Hash) ? ...` * Renamed the local var `config` to `options` in `define_callbacks`, as `options` seems to be the de facto name for the options objects. * Renamed the local var `l` to `line` in `define_callback` (maybe it meant `lambda` in the context) as single `l` may look like `1` in some fonts. | |||||
* | | | | | | | Merge pull request #10099 from wangjohn/railties_each_with_config_2nd_attempt | Santiago Pastorino | 2013-07-01 | 2 | -7/+226 | |
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | Created way of cloning a Rails::Application instance | |||||
| * | | | | | | Allowing multiple rails applications in the same ruby instance. | wangjohn | 2013-06-30 | 2 | -7/+226 | |
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | This change provides the ability to create a new application with a configuration which can be specified. | |||||
* | | | | | | Merge pull request #11195 from yangchenyun/update_doc_for_define_callbacks | Carlos Antonio da Silva | 2013-06-30 | 1 | -13/+14 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | updated AS:Callbacks docs, variable namings | |||||
| * | | | | | | unified the param names across all callbacks manipulation methods | Steven Yang | 2013-07-01 | 1 | -10/+10 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | _ Rename the define_callbacks params to `names` - in order to match the naming conventions for `get_callbacks` and `set_callbacks` at https://github.com/rails/rails/blob/master/activesupport/lib/active_support/callbacks.rb#L736-743 - `define_callbacks` just register names(events), not define the real callback functions. - Rename the `reset_callbacks` params | |||||
| * | | | | | | updated AS:Callbacks doc for terminator option in define_callbacks method | Steven Yang | 2013-06-30 | 1 | -3/+4 | |
| | |/ / / / | |/| | | | | | | | | | | | | | | | | The change is commited at ba552764344bc0a3c25b8576ec11f127ceaa16da | |||||
* | | | | | | Merge pull request #11205 from Gawyn/using-preferred-find_by-syntax-in-guides | Carlos Antonio da Silva | 2013-06-30 | 5 | -10/+10 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Using preferred find_by syntax in guides | |||||
| * | | | | | | Using preferred find_by syntax in guides | Cristian Planas | 2013-07-01 | 5 | -10/+10 | |
| | |/ / / / | |/| | | | | ||||||
* | | | | | | Merge pull request #11204 from Gawyn/find_by-triggers-after_find | Carlos Antonio da Silva | 2013-06-30 | 1 | -0/+1 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Adding find_by in guide to methods that trigger after_find | |||||
| * | | | | | | Adding find_by in guide to methods that trigger after_find | Cristian Planas | 2013-07-01 | 1 | -0/+1 | |
| |/ / / / / | ||||||
* | | | | | | Merge pull request #11192 from charliesome/extra-roflscale | Carlos Antonio da Silva | 2013-06-30 | 1 | -1/+1 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | Store a symbol directly inside DISPATCH_CACHE for additional roflscale | |||||
| * | | | | | store a symbol directly inside DISPATCH_CACHE for extra roflscale | Charlie Somerville | 2013-06-30 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | fix typo in ActiveModel::Error docs [ci skip] | Steven Yang | 2013-06-30 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #11196 from yangchenyun/remove_evals_in_am_callbacks | Carlos Antonio da Silva | 2013-06-30 | 1 | -1/+3 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Remove evals from AM::Validations::Callbacks | |||||
| * | | | | | | remove evals from AM::Validations::Callbacks | Steven Yang | 2013-06-30 | 1 | -1/+3 | |
| | |/ / / / | |/| | | | | | | | | | | | | | | | | follow the same refactor at a63a964a5d1ed02cf0df1b1a33a96ed2a9fa987b | |||||
* | | | | | | Merge pull request #11186 from jetthoughts/synchronize_create_method_body | Yves Senn | 2013-06-30 | 1 | -9/+34 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Synchronize PostController#create code from 5.6 to others sections in Getting Started guide. | |||||
| * | | | | | | Synchronize PostController#create code from 5.6 to others sections. [ci skip] | Paul Nikitochkin | 2013-06-30 | 1 | -9/+34 | |
| | |/ / / / | |/| | | | | | | | | | | | | | | | | Fixed `permit` using for comments. | |||||
* | | | | | | don't shadow `through_scope` method name with local var. | Yves Senn | 2013-06-30 | 1 | -7/+7 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #11180 from robin850/patch-5 | Yves Senn | 2013-06-30 | 1 | -20/+5 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Remove a duplicated section [ci skip] | |||||
| * | | | | | | Remove a duplicated section [ci skip] | Robin Dupret | 2013-06-29 | 1 | -20/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Attachments were previously covered so remove the useless part. Just move a note about multipart headers set when the mail method is triggered to the kept section. | |||||
* | | | | | | | Merge pull request #11189 from ↵ | Yves Senn | 2013-06-30 | 1 | -3/+3 | |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | jetthoughts/destroy_link_description_for_getting_started Added clarification for destroy link_to description in 5.14 | |||||
| * | | | | | | | Added clarification for destroy link_to description in 5.14 [ci skip] | Paul Nikitochkin | 2013-06-30 | 1 | -3/+3 | |
| | |/ / / / / | |/| | | | | | ||||||
* | | | | | | | Merge pull request #11190 from dkaplan88/typo | Yves Senn | 2013-06-30 | 1 | -1/+1 | |
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | Typo in Active Record Migrations Guide | |||||
| * | | | | | | Typo in Active Record Migrations Guide [ci skip] | dkaplan88 | 2013-06-29 | 1 | -1/+1 | |
| |/ / / / / | ||||||
* / / / / / | Clean the code a bit | Santiago Pastorino | 2013-06-29 | 2 | -2/+2 | |
|/ / / / / |