aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | Always add lambda to scope chain to eliminate branch in eval_scopeeileencodes2014-09-042-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We convert all other scopes to lambda's so it makes sense that we should always returns a lambda on a ThroughReflection as well. This eliminates the need to check if the scope is a Relation.
* | | | | | | | | Fix a warningMatthew Draper2014-09-051-1/+1
| | | | | | | | |
* | | | | | | | | Merge pull request #16797 from ↵Matthew Draper2014-09-041-4/+4
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | eileencodes/refactor-add_constraints-followup-to-16762 Followup to PR #16762
| * | | | | | | | Follup to PR #16762eileencodes2014-09-041-4/+4
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove chain from parameters, it's no longer needed since chain and i are being passed via next_reflection Change name of `reflection` to `owner_reflection` because of shadow variable warning. The last reflection will always be the owner.
* | | | | | | | Merge pull request #16799 from rramsden/fix-typoYves Senn2014-09-041-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | [ci skip] Fix typo in rails guide for routing
| * | | | | | | | [ci skip] Fix typo in rails guide for routingRichard Ramsden2014-09-041-1/+1
| | | | | | | | |
* | | | | | | | | Enums shouldn't ruin people's anniversariesGodfrey Chan2014-09-043-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a few more methods on Module/Class to the dangerous class methods blacklist. (Technically, allocate and new are already protected currently because we happen to redefine them in the current implantation.) Closes #16792
* | | | | | | | | build fix, we have to reset the RACK_ENV as well.Yves Senn2014-09-041-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a follow-up to ded17a4 to get the build passing on Travis. As Travis has `ENV["RACK_ENV"]` set to `test` we need to reset that as well.
* | | | | | | | | get rid of shadowing warning when running tests AR and railtie tests.Yves Senn2014-09-041-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Warning looked like this: ``` /Users/senny/Projects/rails/activerecord/lib/active_record/associations/association_scope.rb:142: warning: shadowing outer local variable - reflection ```
* | | | | | | | | code gardening in ActionController::RenderersXavier Noria2014-09-041-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Renames _handle_render_options to _render_to_body_with_renderer, which is more intention-revealing. * The name of the dynamically generated method for a renderer with key :js was "_render_option_js". That name is too weak. :js is an option if you see the render argument as just a generic options hash, but in the context of renderers that's the renderer key, is what identifies the renderer. Now "_render_with_renderer_js" is generated instead, which is crystal clear. * The name of the dynamically generated method for the renderer was constructed using string literals in a few places. That is now encapsulated in a method. * Since we were on it, also removed a couple of redundant selfs.
* | | | | | | | | Extract data/aria attribute prefixes to a constantRafael Mendonça França2014-09-031-1/+3
| | | | | | | | |
* | | | | | | | | Merge pull request #16796 from ARIATeam/ariaRafael Mendonça França2014-09-033-4/+23
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Add support for ARIA attributes in tags
| * | | | | | | | | Add support for ARIA attributes in tagsPaoMar2014-09-033-4/+23
|/ / / / / / / / /
* | | | | | | | | Merge pull request #16795 from Peeja/methods-are-not-duplicableRafael Mendonça França2014-09-033-1/+16
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Methods are not duplicable.
| * | | | | | | | | Methods are not duplicable.Peter Jaros2014-09-033-1/+16
| | | | | | | | | |
* | | | | | | | | | Merge pull request #16787 from cristianbica/fix-globalid-deserializationMatthew Draper2014-09-041-1/+3
|\ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / |/| | | | | | | | | [Active Job] Try to deserialize with GlobalID only strings and globalids
| * | | | | | | | | [Active Job] Try to deserialize with GlobalID only strings and globalidsCristian Bica2014-09-031-1/+3
| | |_|_|_|_|_|/ / | |/| | | | | | |
* | | | | | | | | Merge pull request #16775 from kaspth/ship-secure-sanitizerRafael Mendonça França2014-09-036-53/+8
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | | Ship with rails-html-sanitizer instead.
| * | | | | | | | Ship with rails-html-sanitizer instead.Kasper Timm Hansen2014-09-036-53/+8
| | | | | | | | |
* | | | | | | | | Merge pull request #16756 from huoxito/dont-mess-railties-default-orderRafael Mendonça França2014-09-032-8/+45
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Dont mess with default order engines load (4.1.6.rc1 regression)
| * | | | | | | | | Dont mess with default order engines loadWashington Luiz2014-09-022-8/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When copying migrations some engines might depend on schema from other engine so we can't blindly reverse all railties collection as that would affect the order they were originally loaded. This patch helps to only apply the order from engines specified in `railties_order`
* | | | | | | | | | extract updating all fixture cache to a method so we can lock laterAaron Patterson2014-09-031-1/+5
| | | | | | | | | |
* | | | | | | | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2014-09-036-17/+247
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | copy edit AS core ext changes [ci skip]Vijay Dev2014-09-031-10/+9
| | | | | | | | | | |
| * | | | | | | | | | Update caching_with_rails.md [ci skip]Harshad Sabne2014-08-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Highlight methods
| * | | | | | | | | | Highlight Class/Module [ci skip]Harshad Sabne2014-08-311-1/+1
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] docs grammarschneems2014-08-261-2/+2
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] proper end to sentenceschneems2014-08-261-1/+1
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] use NOTE instead of Noteschneems2014-08-261-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Fix typo in ActiveJob guide [ci skip]प्रथमेश2014-08-261-1/+1
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] fix grammar in docsschneems2014-08-241-1/+1
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] docs can URL cache invalidationschneems2014-08-241-0/+24
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] docs CDN Cache-Control behaviorschneems2014-08-241-0/+24
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] docs CDN header debuggingschneems2014-08-241-0/+48
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] docs CDN caching behaviorschneems2014-08-241-0/+17
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] docs asset_pipeline CDN set upschneems2014-08-241-1/+63
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] docs asset_pipeline#CDNsschneems2014-08-241-13/+21
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] wrap to 80 charsschneems2014-08-241-1/+2
| | | | | | | | | | |
| * | | | | | | | | | code quotes for classnameMikhail Dieterle2014-08-241-1/+1
| | | | | | | | | | |
| * | | | | | | | | | [ci skip] document truncate_wordsRashmi Yadav2014-08-241-0/+33
| | | | | | | | | | |
| * | | | | | | | | | [ci skip ]Document transform_values && transform_values!Rashmi Yadav2014-08-231-0/+14
| | | | | | | | | | |
| * | | | | | | | | | recount files quantityMikhail Dieterle2014-08-231-1/+1
| | | | | | | | | | |
* | | | | | | | | | | schema loading rake tasks maintain database connection for current env.Yves Senn2014-09-034-0/+36
| |_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Joshua Cody & Yves Senn] Closes #16757. Prior to this patch schema loading rake tasks had the potential to leak a connection to a different database. This had side-effects when rake tasks operating on the current connection (like `db:seed`) were chained.
* | | | | | | | | | Merge pull request #16785 from cristianbica/fix-double-aj-deserialization-errorMatthew Draper2014-09-032-5/+10
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fix for double ActiveJob::DeserializationErorr
| * | | | | | | | | | Fix for double ActiveJob::DeserializationErorrCristian Bica2014-09-032-5/+10
|/ / / / / / / / / /
* | | | | | | | | | Merge pull request #16263 from ↵Yves Senn2014-09-030-0/+0
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | zuhao/refactor_activerecord_serialized_attribute_test reset_column_infomation shouldn't be part of the assertion.
| * | | | | | | | | | reset_column_infomation shouldn't be part of the assertion.Zuhao Wan2014-07-231-1/+1
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #16780 from viditn91/use-correct-operators-in-queryYves Senn2014-09-031-1/+2
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | use correct operator in query based on JSON document [ci skip]
| * | | | | | | | | | | use correct operator in query based on JSON documentviditn912014-09-031-1/+2
| | |_|_|/ / / / / / / | |/| | | | | | | | |
* | | | | | | | | | | Merge pull request #16781 from kamipo/move_column_option_handlingYves Senn2014-09-032-6/+3
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Move column option handling to new_column_definition
generated by cgit v1.2.3 (git 2.25.1) at 2025-02-22 21:28:58 +0000