| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Merge pull request #9246 from Noemj/update_prepared_statements | Aaron Patterson | 2013-02-24 | 2 | -3/+26 |
| |\ | | | | | Changed update to use prepared statements | ||||
| | * | Changed update_record to use prepared statements. | Noemj | 2013-02-11 | 2 | -3/+26 |
| | | | |||||
| * | | Merge pull request #9320 from wangjohn/touch_handling_on_new_instances | Guillermo Iguaran | 2013-02-24 | 3 | -0/+20 |
| |\ \ | | | | | | | Raising an ActiveRecordError when one tries to use touch on a new record object | ||||
| | * | | Raising an ActiveRecordError when one tries to use .touch(name) on a new | wangjohn | 2013-02-25 | 3 | -0/+20 |
| |/ / | | | | | | | | | object that has not yet been persisted. This behavior follows the precedent set by update_columns. | ||||
| * | | Revert "Simplify code in AS json encoder" | Guillermo Iguaran | 2013-02-25 | 1 | -8/+15 |
| | | | | | | | | | | | | | This reverts commit 046e27a7338f2961c10e9d133e0e2229b51c2ba8. Check https://github.com/rails/rails/pull/8815#issuecomment-14026122 for details. | ||||
| * | | Merge pull request #8815 from bogdan/simplified-to-json-args | Steve Klabnik | 2013-02-24 | 1 | -15/+8 |
| |\ \ | | | | | | | AS JSON encoder: remove monkey patch of Array | ||||
| | * | | Simplify code in AS json encoder | Bogdan Gusiev | 2013-01-06 | 1 | -15/+8 |
| | | | | |||||
| * | | | Improve AR changelog [ci skip] | Carlos Antonio da Silva | 2013-02-24 | 1 | -2/+2 |
| | | | | |||||
| * | | | Add more information to the CHANGELOG entry [ci skip] | Rafael Mendonça França | 2013-02-24 | 1 | -0/+12 |
| | | | | |||||
| * | | | Merge pull request #9403 from stopdropandrew/TimeWithZone#to_r | Rafael Mendonça França | 2013-02-24 | 3 | -0/+19 |
| |\ \ \ | | | | | | | | | | | | | | | | | Added `ActiveSupport::TimeWithZone#to_r` for `Time#at` compatibility. Closes #9404 | ||||
| | * | | | Added `ActiveSupport::TimeWithZone#to_r` for `Time#at` compatibility. | stopdropandrew | 2013-02-24 | 3 | -0/+19 |
| | | | | | |||||
| * | | | | Remove warning | Rafael Mendonça França | 2013-02-24 | 1 | -1/+1 |
| | | | | | |||||
| * | | | | Make explicit the method will raise an exception | Rafael Mendonça França | 2013-02-24 | 1 | -10/+10 |
| |/ / / | | | | | | | | | | Taking the wise advice of @carlosantoniodasilva | ||||
| * | | | did a pass over the AS changelog [ci skip] | Xavier Noria | 2013-02-24 | 1 | -55/+61 |
| | | | | |||||
| * | | | Merge pull request #9391 from senny/8663_broken_hmt_ordering_with_includes | Rafael Mendonça França | 2013-02-24 | 3 | -2/+13 |
| |\ \ \ | | | | | | | | | don't apply invalid ordering when preloading hmt associations. | ||||
| | * | | | don't apply invalid ordering when preloading hmt associations. | Yves Senn | 2013-02-24 | 3 | -2/+13 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | closes #8663. When preloading a hmt association there two possible scenarios: 1.) preload with 2 queries: first hm association, then hmt with id IN () 2.) preload with join: hmt association is loaded with a join on the hm association The bug was happening in scenario 1.) with a normal order clause on the hmt association. The ordering was also applied when loading the hm association, which resulted in the error. This patch only applies the ordering the the hm-relation if we are performing a join (2). Otherwise the order will only appear in the second query (1). | ||||
| * | | | | Merge pull request #9402 from senny/9356_update_guides | Rafael Mendonça França | 2013-02-24 | 1 | -9/+7 |
| |\ \ \ \ | | | | | | | | | | | update the example for `after_commit` in the guides. | ||||
| | * | | | | update the example for `after_commit` in the guides. | Yves Senn | 2013-02-24 | 1 | -9/+7 |
| | |/ / / | | | | | | | | | | | | | This is a follow up to #9356. | ||||
| * | | | | did a pass over the AM changelog [ci skip] | Xavier Noria | 2013-02-24 | 1 | -15/+35 |
| | | | | | |||||
| * | | | | Merge pull request #9401 from sikachu/master-remove-builder-option | Rafael Mendonça França | 2013-02-24 | 12 | -183/+7 |
| |\ \ \ \ | |/ / / |/| | | | Remove `--builder` option from `rails` command | ||||
| | * | | | Remove `--builder` option from `rails` command | Prem Sichanugrist | 2013-02-24 | 12 | -183/+7 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ability to use a custom builder by passing `--builder` (or `-b`) has been removed. Consider using application template instead. See this guide for more detail: http://guides.rubyonrails.org/rails_application_templates.html | ||||
| * | | | | Merge pull request #9400 from senny/remove_auto_explain_threshold_in_seconds | Xavier Noria | 2013-02-24 | 14 | -223/+39 |
| |\ \ \ \ | | | | | | | | | | | remove config.auto_explain_threshold_in_seconds | ||||
| | * | | | | remove AR auto-explain (config.auto_explain_threshold_in_seconds) | Yves Senn | 2013-02-24 | 14 | -223/+39 |
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We discussed that the auto explain feature is rarely used. This PR removes only the automatic explain. You can still display the explain output for any given relation using `ActiveRecord::Relation#explain`. As a side-effect this should also fix the connection problem during asset compilation (#9385). The auto explain initializer in the `ActiveRecord::Railtie` forced a connection. | ||||
| * | | | | Use sprockets-rails 2.0.0.rc3 in new apps | Guillermo Iguaran | 2013-02-24 | 2 | -2/+2 |
| | | | | | |||||
| * | | | | Merge pull request #9399 from neerajdotname/7832-dependent-option | Rafael Mendonça França | 2013-02-24 | 1 | -2/+2 |
| |\ \ \ \ | | | | | | | | | | | Updated doc to clarify about update_all and delete_all | ||||
| | * | | | | Updated doc to clarify about update_all and delete_all | Neeraj Singh | 2013-02-24 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | delete_all will call callbacks if depdent option is specified as :destroy. This behavior is not very clear from the documetation and hence issue #7832 was raised. closes #7832 | ||||
| * | | | | | Mention the extraction of Active Resource in the upgrading guide | Rafael Mendonça França | 2013-02-24 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | | | | | | | [ci skip] | ||||
| * | | | | | Mention the removal of attr_accessible and attr_protected in the upgrading guide | Rafael Mendonça França | 2013-02-24 | 1 | -0/+2 |
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Point to the Protected Attributes gem to those who want a smoothly upgrade path before to change to use Strong Parameters. [ci skip] | ||||
| * | | | | Skip connection url test when the machine is using socket configuration. | Rafael Mendonça França | 2013-02-24 | 1 | -0/+3 |
| | | | | | | | | | | | | | | | | | The connection url parssing don't accept the socket option | ||||
| * | | | | Do not type cast all the database url values. | Rafael Mendonça França | 2013-02-24 | 8 | -56/+32 |
| |/ / / | | | | | | | | | | | | | | | | We should only type cast when we need to use. Related to 4b005fb371c2e7af80df7da63be94509b1db038c | ||||
| * | | | Merge pull request #9398 from senny/revert_precompile_config_move | Guillermo Iguaran | 2013-02-24 | 3 | -10/+6 |
| |\ \ \ | | | | | | | | | Revert "move config.assets.precompile into config/application.rb" | ||||
| | * | | | Revert "move config.assets.precompile into config/application.rb" | Yves Senn | 2013-02-24 | 3 | -10/+6 |
| |/ / / | | | | | | | | | | | | | | | | | | | This reverts commit 304f4d4c25ccabdbf97d37dd7a92a54d0b63a9c9. As it turns out everything should behave as it did and we don't need to move this configuration option. | ||||
| * | | | fix respond_to without blocks not working if one of the blocks is all | grosser | 2013-02-24 | 4 | -1/+25 |
| | | | | |||||
| * | | | Merge pull request #9356 from senny/988_multiple_actions_for_after_commit | Steve Klabnik | 2013-02-24 | 3 | -14/+60 |
| |\ \ \ | | | | | | | | | multiple actions for :on option with `after_commit` and `after_rollback` | ||||
| | * | | | multiple actions for :on option with `after_commit` and `after_rollback` | Yves Senn | 2013-02-21 | 3 | -14/+60 |
| | | | | | | | | | | | | | | | | | Closes #988. | ||||
| * | | | | Merge pull request #9397 from senny/9201_duplicate_associations_with_natural_key | Rafael Mendonça França | 2013-02-24 | 2 | -0/+12 |
| |\ \ \ \ | | | | | | | | | | | test case to prevent duplicated associations with custom PK. | ||||
| | * | | | | test case to prevent duplicated associations with custom PK. | Yves Senn | 2013-02-24 | 2 | -0/+12 |
| |/ / / / | | | | | | | | | | | | | closes #9201 | ||||
| * | | | | Merge pull request #9395 from yahonda/oracle_index_length | Carlos Antonio da Silva | 2013-02-23 | 1 | -2/+10 |
| |\ \ \ \ | | | | | | | | | | | Address failure test_rename_column_with_multi_column_index with Oracle | ||||
| | * | | | | Oracle enhanced adapter shortens its name if it is longer than 30 bytes | Yasuo Honda | 2013-02-24 | 1 | -2/+10 |
| |/ / / / | | | | | | | | | | | | | because of Oracle database index length spec. | ||||
| * | | | | Merge pull request #9394 from senny/actually_rename_the_test_case | Rafael Mendonça França | 2013-02-23 | 1 | -1/+1 |
| |\ \ \ \ | | | | | | | | | | | also rename the test-case class inside columns_test.rb | ||||
| | * | | | | also rename the test-case class inside columns_test.rb | Yves Senn | 2013-02-24 | 1 | -1/+1 |
| |/ / / / | |||||
| * | | | | Revert "prevent minitest from printing a --seed run option" | Xavier Noria | 2013-02-24 | 1 | -12/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reason: on a second thought, minitest prints a seed as run option regardless of the test order, and it actually calls srand, so albeit it might be misleading I believe, it is the way it is and should be left that way. This reverts commit c15862ae0cb876d745609170f0f90a9bb9b5e0ae. | ||||
| * | | | | prevent minitest from printing a --seed run option | Xavier Noria | 2013-02-24 | 1 | -2/+12 |
| | | | | | | | | | | | | | | | | | See the first FIXME comment in the patch for the rationale. | ||||
| * | | | | registers the encrypted session store in the AP changelog | Xavier Noria | 2013-02-23 | 1 | -0/+4 |
| | | | | | |||||
| * | | | | Merge pull request #9393 from rubys/master | Guillermo Iguaran | 2013-02-23 | 1 | -1/+1 |
| |\ \ \ \ | | | | | | | | | | | guides/rails_guides moved up and out of the railties directory | ||||
| | * | | | | guides/rails_guides moved up and out of the railties directory | Sam Ruby | 2013-02-23 | 1 | -1/+1 |
| | | | | | | |||||
| * | | | | | adds a missing LIMIT 1 in #take docs | Xavier Noria | 2013-02-23 | 1 | -1/+1 |
| |/ / / / | |||||
| * | | | | Merge pull request #9389 from senny/forwardport_9388 | Carlos Antonio da Silva | 2013-02-23 | 1 | -1/+15 |
| |\ \ \ \ | | | | | | | | | | | Forwardport #9388 rename/remove column preserves custom PK. | ||||
| | * | | | | rename_column_test.rb -> columns_test.rb to reveal intent. | Yves Senn | 2013-02-23 | 1 | -0/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I renamed the test to better communicate it's intention. Since it also tests: - add_column - remove_column - change_column There is no reason to call it rename_column_test. | ||||
| | * | | | | forwardport #9388 rename/remove column preserves custom PK. | Yves Senn | 2013-02-23 | 1 | -1/+15 |
| |/ / / / | |||||
 |
index : rails.git | |
| Mirror of official rails repo with custom fixes. | Harald Eilertsen |
| aboutsummaryrefslogtreecommitdiffstats |