Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | 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 | |||||
| * | | | | | Exception handling for controllers using ActionController::Live | Sean Griffin | 2013-03-18 | 2 | -1/+103 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Any exceptions that occured at the view or controller level for a controller using ActionController::Live would cause the server to either hang with an open socket indefinitely, or immediately crash (depending on whether the server was launched with rails s or directly). Changed the behavior of exceptions to act the same as streaming templates for html requests, and allow for an on_error callback if needed. | |||||
* | | | | | | Merge pull request #10122 from wangjohn/adding_documentation_to_form_builder | Rafael Mendonça França | 2013-04-08 | 1 | -2/+55 | |
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | Added documentation to the FormBuilder class Closes #10115 | |||||
| * | | | | | Added documentation to the FormBuilder class, should help | wangjohn | 2013-04-06 | 1 | -2/+55 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | clarify issue #10115. Also made the field_helpers an explicit list of methods. | |||||
* | | | | | | Merge pull request #10130 from Agis-/patch-2 | Carlos Antonio da Silva | 2013-04-08 | 1 | -3/+3 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Mark unused block parameters. | |||||
| * | | | | | | Remove unused block parameters | Agis Anastasopoulos | 2013-04-07 | 1 | -3/+3 | |
| | |_|/ / / | |/| | | | | ||||||
* | | | | | | Merge pull request #10131 from Agis-/metal_helper | Carlos Antonio da Silva | 2013-04-08 | 1 | -1/+0 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | Remove unecessary variable call since `sort!` always returns the sorted array | |||||
| * | | | | | Remove unecessary variable call, `#sort!` always returns an array | Agis Anastasopoulos | 2013-04-07 | 1 | -1/+0 | |
|/ / / / / | ||||||
* | | | | | Minor AR changelog improvements [ci skip] | Carlos Antonio da Silva | 2013-04-07 | 1 | -4/+3 | |
| | | | | | ||||||
* | | | | | Avoid iterating over records hash when not necessary | Carlos Antonio da Silva | 2013-04-07 | 1 | -2/+6 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the reflection scope is not flagged with distinct value, there is no need to iterate over the records, so we avoid that by doing the check before iterating rather than inside the iteration block. | |||||
* | | | | | Merge pull request #10127 from vipulnsward/each_to_each_value | Carlos Antonio da Silva | 2013-04-07 | 2 | -3/+3 | |
|\ \ \ \ \ | | | | | | | | | | | | | each to each_value; remove unused vars | |||||
| * | | | | | each to each_value; remove unused vars | Vipul A M | 2013-04-07 | 2 | -3/+3 | |
|/ / / / / | ||||||
* | | | | | Merge pull request #10123 from ↵ | Carlos Antonio da Silva | 2013-04-06 | 1 | -3/+3 | |
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | wangjohn/added_bang_to_raise_nested_attributes_method Added a bang to the raise_nested_attributes_record_not_found method | |||||
| * | | | | Added a bang to the end of +raise_nested_attributes_record_not_found+ | wangjohn | 2013-04-06 | 1 | -3/+3 | |
|/ / / / | | | | | | | | | | | | | method to signify an exception possibly being raised. | |||||
* | | | | Merge pull request #10101 from wangjohn/refactoring_routing_mapper | Andrew White | 2013-04-06 | 1 | -12/+13 | |
|\ \ \ \ | | | | | | | | | | | Refactoring some reused code into a method (inside of the routes mapper) | |||||
| * | | | | Refactoring some reused code into a method (inside of the routes | wangjohn | 2013-04-05 | 1 | -12/+13 | |
| | | | | | | | | | | | | | | | | | | | | mapper) and adding a constant for all the possible scopes. | |||||
* | | | | | Forgot to lock the uglifier version in our Gemfile | Rafael Mendonça França | 2013-04-06 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | Related with 575838795019f7402992e3bfa83a0c5048cf143d | |||||
* | | | | | Merge pull request #10121 from Agis-/patch-2 | Rafael Mendonça França | 2013-04-06 | 1 | -1/+1 | |
|\ \ \ \ \ | | | | | | | | | | | | | Fix markdown formatting | |||||
| * | | | | | Fix markdown formatting | Agis Anastasopoulos | 2013-04-06 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #10120 from vipulnsward/change_merge_on_new_hash | Rafael Mendonça França | 2013-04-06 | 1 | -3/+3 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | change some more merge to merge! on new hashes | |||||
| * | | | | | change some more merge to merge! on new hashes | Vipul A M | 2013-04-06 | 1 | -3/+3 | |
| | | | | | | ||||||
* | | | | | | sprockets 2.9.x doesn't work with uglifier 2.0 yet. | Rafael Mendonça França | 2013-04-06 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See https://travis-ci.org/rails/rails/jobs/6110543#L1285 for the failure cc @josh | |||||
* | | | | | | Merge pull request #10113 from Agis-/default_scopes_predicate | Rafael Mendonça França | 2013-04-06 | 6 | -6/+35 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | Deprecate ActiveRecord#Base.default_scopes? Closes #10107 | |||||
| * | | | | | Deprecate ActiveRecord#Base.default_scopes? | Agis- | 2013-04-06 | 2 | -1/+15 | |
| | | | | | | | | | | | | | | | | | | | | | | | | See #10107. | |||||
| * | | | | | Add option to Class#class_attribute for skipping the query method | Agis- | 2013-04-06 | 4 | -5/+20 | |
|/ / / / / | ||||||
* | | | | | Bring back ActiveRecord::Base#connection_handler removed by mistake on b37399ab | Rafael Mendonça França | 2013-04-06 | 1 | -0/+4 | |
| | | | | | ||||||
* | | | | | Merge pull request #10118 from vipulnsward/remove_unused_var_task | Carlos Antonio da Silva | 2013-04-06 | 2 | -2/+1 | |
|\ \ \ \ \ | | | | | | | | | | | | | remove unused variables | |||||
| * | | | | | remove unused variables | Vipul A M | 2013-04-06 | 2 | -2/+1 | |
| | | | | | | ||||||
* | | | | | | `fast_xs` support has been removed. Use 'String#encode(xml: :attr)`. | Aaron Patterson | 2013-04-05 | 3 | -19/+2 | |
|/ / / / / | ||||||
* | | | | | updating the guide | Aaron Patterson | 2013-04-05 | 1 | -5/+5 | |
| | | | | | ||||||
* | | | | | add integration test for shorthand rake tests | Aaron Patterson | 2013-04-05 | 1 | -0/+7 | |
| | | | | | ||||||
* | | | | | rake test app/model/foo.rb and rake test models/foo works | Aaron Patterson | 2013-04-05 | 2 | -3/+47 | |
| | | | | | ||||||
* | | | | | extract test info from the command line and set up the test task | Aaron Patterson | 2013-04-05 | 2 | -5/+28 | |
| | | | | | ||||||
* | | | | | add a class for splitting up rake commands | Aaron Patterson | 2013-04-05 | 2 | -0/+70 | |
| | | | | | ||||||
* | | | | | just use ruby to run tests | Aaron Patterson | 2013-04-05 | 1 | -4/+4 | |
| | | | | |