Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Revert "Merge pull request #10158 from steveklabnik/issue_10125" | Rafael Mendonça França | 2013-04-10 | 2 | -19/+2 |
| | | | | | | | | This reverts commit fa3ef8e82ab2f96cf15ef9bc885b2468fad77621, reversing changes made to e0af93dd3a5eeee2e2a67b05f34afb66cc80c00b. Reason: Routes, Active Record and the rendering stack should not depend on the default locale | ||||
* | code review of 22e5ab3 | Xavier Noria | 2013-04-10 | 1 | -8/+7 |
| | |||||
* | Merge pull request #10157 from prathamesh-sonpatki/app_loader | Xavier Noria | 2013-04-10 | 2 | -19/+14 |
|\ | | | | | Searching for rails executable correctly | ||||
| * | Searching for rails executable correctly | Prathamesh Sonpatki | 2013-04-10 | 2 | -19/+14 |
|/ | | | | | | | | | | * Current logic of finding Rails executable in parent directory is not returning full path of executable if it is found in one of the parent directories * To compensate for this, we have to call exec_app_rails recursively until the executable is found or we cant do 'chdir' anymore * This solution finds the correct executable path from parent directory(s) recursively | ||||
* | Merge pull request #10165 from wangjohn/public_send_for_instance | Rafael Mendonça França | 2013-04-10 | 1 | -1/+1 |
|\ | | | | | Using public send instead of send for the PerThreadRegistry module. | ||||
| * | Using public send instead of send for the PerThreadRegistry module. | wangjohn | 2013-04-10 | 1 | -1/+1 |
|/ | | | | Prevents you from accidentally calling a protected method. | ||||
* | Merge pull request #10162 from choudhuryanupam/fix_activesupport_test | Carlos Antonio da Silva | 2013-04-10 | 1 | -2/+0 |
|\ | | | | | Removed unused setup | ||||
| * | Removed unused setup | Anupam Choudhury | 2013-04-10 | 1 | -2/+0 |
|/ | |||||
* | Merge pull request #10160 from choudhuryanupam/fix_typo_n_grammar | Xavier Noria | 2013-04-10 | 1 | -1/+1 |
|\ | | | | | Fixed typo | ||||
| * | Fixed typo | Anupam Choudhury | 2013-04-10 | 1 | -1/+1 |
|/ | |||||
* | Return nil for Mime::NullType#ref | Andrew White | 2013-04-10 | 2 | -1/+5 |
| | |||||
* | Reverts rendering behavior when format is unknown | Grzegorz Świrski | 2013-04-10 | 2 | -2/+6 |
| | | | | | | | | | If a request has unknown format (eg. /foo.bar), the renderer fallbacks to default format. This patch reverts Rails 3.2 behavior after c2267db commit. Fixes issue #9654. | ||||
* | Add failing test case for #9654 | Andrew White | 2013-04-10 | 1 | -0/+45 |
| | |||||
* | Merge pull request #10158 from steveklabnik/issue_10125 | Steve Klabnik | 2013-04-09 | 2 | -2/+19 |
|\ | | | | | Fix inflector to respect default locale. | ||||
| * | Fix inflector to respect default locale. | Nick Cox | 2013-04-10 | 2 | -2/+19 |
| | | | | | | | | | | | | | | | | The inflector was made aware of locales in 7db0b073fec6bc3e6f213b58c76e7f43fcc2ab97, but it defaulted to :en. That should actually be our default locale instead. Fixes #10125 | ||||
* | | Merge pull request #10156 from wangjohn/grouping_thread_locals | Jeremy Kemper | 2013-04-09 | 9 | -19/+92 |
|\ \ | |/ |/| | Grouping thread locals in ActiveRecord | ||||
| * | Changed the ScopeRegistry and the InstrumentationRegistry to use the | wangjohn | 2013-04-09 | 3 | -13/+12 |
| | | | | | | | | PerThreadRegistry module. | ||||
| * | Created a runtime registry for thread local variables in active record. | wangjohn | 2013-04-09 | 5 | -6/+39 |
| | | |||||
| * | Creating a module so that per thread registries can be easily created as | wangjohn | 2013-04-09 | 1 | -0/+41 |
|/ | | | | thread local variables. | ||||
* | Explain how to upgrade bin/ for Rails 4 | Jeremy Kemper | 2013-04-09 | 1 | -8/+29 |
| | |||||
* | Merge pull request #10154 from killthekitten/fix_template_error_header | Carlos Antonio da Silva | 2013-04-09 | 1 | -1/+1 |
|\ | | | | | Use camelize instead of capitalize on template error screen | ||||
| * | Use camelize instead of capitalize on error screen | Nikolay Shebanov | 2013-04-10 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #10103 from ↵ | Steve Klabnik | 2013-04-09 | 2 | -118/+238 |
|\ \ | |/ |/| | | | | | senny/get_action_mailer_guide_ready_for_prime_time Get Action Mailer guide ready for prime time [ci skip] | ||||
| * | get the Action Mailer guide ready. [ci skip] | Yves Senn | 2013-04-09 | 2 | -118/+238 |
| | | |||||
* | | copy-edit pass in AM changelog [ci skip] | Xavier Noria | 2013-04-09 | 1 | -9/+11 |
| | | |||||
* | | Merge pull request #9843 from indirect/rails_bin | Rafael Mendonça França | 2013-04-09 | 1 | -10/+20 |
|\ \ | | | | | | | Rails 4 prints help for "rails new" when running "rails console" | ||||
| * | | fix binstub typo | Andre Arko | 2013-03-30 | 1 | -1/+1 |
| | | | |||||
| * | | need to delete bin, not binstub, update wording | Andre Arko | 2013-03-27 | 1 | -4/+4 |
| | | | |||||
| * | | script/rails could never be from bundler | Andre Arko | 2013-03-27 | 1 | -1/+1 |
| | | | |||||
| * | | rails commands even if bin/rails is a gem stub | Andre Arko | 2013-03-20 | 1 | -10/+20 |
| | | | |||||
* | | | Fix typo | Rafael Mendonça França | 2013-04-09 | 1 | -2/+2 |
| | | | |||||
* | | | Merge pull request #10153 from choudhuryanupam/master | Rafael Mendonça França | 2013-04-09 | 1 | -4/+0 |
|\ \ \ | | | | | | | | | Removed unused setup | ||||
| * | | | Removed unused setup | Anupam Choudhury | 2013-04-09 | 1 | -4/+0 |
|/ / / | |||||
* | | | Merge pull request #10142 from wangjohn/grouping_thread_locals | Rafael Mendonça França | 2013-04-09 | 2 | -9/+16 |
|\ \ \ | | | | | | | | | Delegating the value getter and setters in the ScopeRegistry | ||||
| * | | | Delegating the value getter and setters in the ScopeRegistry to the | wangjohn | 2013-04-08 | 2 | -9/+16 |
| | | | | | | | | | | | | | | | | current ScopeRegister object. | ||||
* | | | | Merge pull request #10151 from neerajdotname/ar-to-klass | Rafael Mendonça França | 2013-04-09 | 4 | -12/+12 |
|\ \ \ \ | | | | | | | | | | | changed variable name active_record => base_klass | ||||
| * | | | | changed variable name active_record => base_klass | Neeraj Singh | 2013-04-09 | 4 | -12/+12 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Current code stores the klass name in active_record and this is used throughout. While reviewing the code time and again I had the mental picture of active_record being an instance of a klass. However here the actual klass is being stored in @active_record. Secondly at two different places while referring to @active_record the comment refers to it as base klass. All this points to active_record being not the best variable name. So I thought it is better to replace active_record with base_klass. This change is confined to JoinDependency, JoinBase, JoinPart and JoinAssociation - all joining related work. | ||||
* | | | | | Merge pull request #10138 from hbakhtiyor/test-dirs-environment | Rafael Mendonça França | 2013-04-09 | 1 | -2/+3 |
|\ \ \ \ \ | |/ / / / |/| | | | | Add TEST_DIRS enviroment for constrain test directories in rakefile | ||||
| * | | | | Add TEST_DIRS enviroment for constrain test directories in rakefile | Abd ar-Rahman Hamidi | 2013-04-08 | 1 | -2/+3 |
| | | | | | |||||
* | | | | | Merge pull request #10148 from vipulnsward/traces_change | Carlos Antonio da Silva | 2013-04-09 | 1 | -6/+4 |
|\ \ \ \ \ | | | | | | | | | | | | | change array of array to hash | ||||
| * | | | | | change array of array to hash | Vipul A M | 2013-04-09 | 1 | -6/+4 |
| | | | | | | |||||
* | | | | | | Merge pull request #10147 from vipulnsward/fix_AP_live_warning | Carlos Antonio da Silva | 2013-04-08 | 2 | -1/+2 |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | fix AP warning; remove unused variable | ||||
| * | | | | | fix AP warning; remove unused variable | Vipul A M | 2013-04-09 | 2 | -1/+2 |
|/ / / / / | |||||
* | | | | | Merge pull request #10145 from wangjohn/instrumentation_registry_creation | Rafael Mendonça França | 2013-04-08 | 1 | -1/+27 |
|\ \ \ \ \ | | | | | | | | | | | | | Consolidating thread locals in AS::Notifications | ||||
| * | | | | | Creating an object to consolidate thread locals which hold the | wangjohn | 2013-04-08 | 1 | -1/+27 |
|/ / / / / | | | | | | | | | | | | | | | | instrumenters for the AS::Notifications module. | ||||
* | | | | | Merge pull request #10143 from Agis-/mark_unused_vars | Rafael Mendonça França | 2013-04-08 | 9 | -13/+13 |
|\ \ \ \ \ | |_|/ / / |/| | | | | Mark unused variables and make some style fixes | ||||
| * | | | | Mark unused variables and make some style fixes | Agis Anastasopoulos | 2013-04-08 | 9 | -13/+13 |
|/ / / / | | | | | | | | | | | | | It'd be a nice convention to mark the unused variables like this, now that Ruby 2 will issue no warnings for such vars being unused. | ||||
* | | | | Merge pull request #10135 from wangjohn/grouping_thread_locals | Rafael Mendonça França | 2013-04-08 | 3 | -6/+57 |
|\ \ \ \ | | | | | | | | | | | Grouping thread locals in the ActiveRecord scopes | ||||
| * | | | | Grouping thread locals in the ActiveRecord scopes so that the | wangjohn | 2013-04-08 | 3 | -6/+57 |
|/ / / / | | | | | | | | | | | | | | | | | current_scope and ignore_default_scope locals are brought together under a registry object. | ||||
* | | | | Merge pull request #9604 from sgrif/live_streaming_exceptions | Rafael Mendonça França | 2013-04-08 | 2 | -1/+103 |
|\ \ \ \ | | | | | | | | | | | Exceptions raised when using ActionController::Live cause server crash |