aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | No need for trailing slash on migration path.Jeremy Kemper2014-04-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Causes a double // in Dir.glob that breaks Ruby 2.2-trunk. Not really a bug, but not relevant to this test either. Originally added in ed21f0c50270139ddb6993acfdaea4586ffd09a3
| * | | reset `@arel` when modifying a Relation in place.Yves Senn2014-04-243-1/+18
| | | | | | | | | | | | | | | | /cc @tenderlove
| * | | Merge pull request #14859 from jefflai2/datetime_typeYves Senn2014-04-242-6/+15
| |\ \ \ | | | | | | | | | | PostgreSQL Timestamps always map to `:datetime`.
| | * | | PostgreSQL Timestamps always map to `:datetime`.Jefferson Lai2014-04-242-6/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The PG Adapter should use `:datetime` consistently instead of mapping mispellings to `:timestamp`. See #14513
| * | | | Merge pull request #14856 from Gaurav2728/gaurav-typo_fixSantiago Pastorino2014-04-241-2/+2
| |\ \ \ \ | | |/ / / | |/| | | [ci skip] Improve formatting and yml
| | * | | [ci skip] Improve formatting and ymlGaurav Sharma2014-04-241-2/+2
| |/ / /
| * | | Merge pull request #14858 from ebouchut/patch-1Robin Dupret2014-04-241-1/+1
| |\ \ \ | | | | | | | | | | Fix a typo in the doc of forty_two AR FinderMethod [ci skip]
| | * | | Fix a typo in the doc of forty_two AR FinderMethodEric Bouchut2014-04-241-1/+1
| |/ / /
| * | | Merge pull request #14843 from ↵Santiago Pastorino2014-04-231-5/+5
| |\ \ \ | | | | | | | | | | | | | | | | | | | | tgxworld/improve_readability_of_contributing_to_rails_guide [ci skip] Improve readability of contributing to rails guide.
| | * | | Improve readability of contributing to rails guide. [ci skip]Guo Xiang Tan2014-04-231-5/+5
| | | | |
| * | | | Merge pull request #14852 from matthewd/fix_build_for_assetsRafael Mendonça França2014-04-231-2/+2
| |\ \ \ \ | | | | | | | | | | | | Fix railties asset tests: precompile the image we're referencing
| | * | | | Precompile the image we're referencing, too.Matthew Draper2014-04-241-2/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | You can't compile a file that references a non-compiled asset's path. .. unless you turn off asset runtime errors. Outside of a test case like this, `config.assets.precompile` would normally retain its default entry, which precompiles all images (and other non-JS/CSS files) that are in `app/assets`.
| * | | | Merge pull request #14842 from lucasmazza/lm-no-touching-callbacksYves Senn2014-04-233-2/+27
| |\ \ \ \ | | | | | | | | | | | | Move `NoTouching` down the inheritance chain on AR::Base
| | * | | | `ActiveRecord::Base.no_touching` no longer triggers callbacks or start empty ↵Lucas Mazza2014-04-233-2/+27
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | transactions. Closes #14841.
| * | | | Merge pull request #14850 from chancancode/fix_json_cookie_jarGuillermo Iguaran2014-04-233-2/+132
| |\ \ \ \ | | | | | | | | | | | | Fixed an issue with migrating legacy json cookies.
| | * | | | Fixed an issue with migrating legacy json cookies.Godfrey Chan2014-04-233-2/+132
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the `VerifyAndUpgradeLegacySignedMessage` assumes all incoming cookies are marshal-encoded. This is not the case when `secret_token` is used in conjunction with the `:json` or `:hybrid` serializer. In those case, when upgrading to use `secret_key_base`, this would cause a `TypeError: incompatible marshal file format` and a 500 error for the user. Fixes #14774. *Godfrey Chan*
| * | | | Merge pull request #14840 from akshay-vishnoi/doc_changesArthur Nogueira Neves2014-04-231-2/+2
| |\ \ \ \ | | |/ / / | |/| | | Correct comment [ci skip]
| | * | | Correct comment [ci skip]Akshay Vishnoi2014-04-231-2/+2
| | | | |
| * | | | Perfer to define methods instead of calling testRafael Mendonça França2014-04-221-4/+4
| | | | | | | | | | | | | | | | | | | | This file is using this pattern already
| * | | | Fix syntax errorRafael Mendonça França2014-04-221-0/+1
| | | | |
| * | | | Add CHANGELOG entry for #14757 [ci skip]Rafael Mendonça França2014-04-221-0/+6
| | | | |
| * | | | Merge pull request #14757 from estsauver/14752Rafael Mendonça França2014-04-225-6/+19
| |\ \ \ \ | | | | | | | | | | | | Fix behavior of select! to be consistent with select #14752
| | * | | | select! renamed to avoid name collision Array#select!Earl J St Sauver2014-04-215-6/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #14752 Select mimics the block interface of arrays, but does not mock the block interface for select!. This change moves the api to be a private method, _select!.
| * | | | | Merge pull request #14573 from habermann24/has_many_through_fixRafael Mendonça França2014-04-225-2/+30
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Properly handle scoping with has_many :through. Fixes #14537. Conflicts: activerecord/CHANGELOG.md activerecord/test/cases/associations/has_many_through_associations_test.rb
| | * | | | | Minor improvement: Use the merge method on the relation instead of ↵Jan Habermann2014-04-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | instance_eval directly.
| | * | | | | Simplify the code in target_scopeJan Habermann2014-04-031-5/+2
| | | | | | |
| | * | | | | Add a CHANGELOG entry for the has_many :through scoping fix (See #14537 for ↵Jan Habermann2014-04-031-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | details).
| | * | | | | Properly handle scoping with has_many :through. Fixes #14537.Jan Habermann2014-04-034-2/+25
| | | | | | |
| * | | | | | Merge pull request #14801 from kuldeepaggarwal/fix-string-inflectionRafael Mendonça França2014-04-222-1/+6
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix inconsistent behavior from String#pluralize
| | * | | | | | Fix inconsistent behavior from String#pluralizeKuldeep Aggarwal2014-04-192-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before: When calling String#pluralize with count=1 then it returned same string, but with count other than 1, returned new string. After: String#pluralize always return a new string. => Prevent mutation of a string inadvertently.
| * | | | | | | Merge pull request #14836 from jonatack/patch-5Yves Senn2014-04-221-8/+8
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix run-on sentences and improve grammar [skip ci]
| | * | | | | | | Fix run-on sentences and improve grammar [skip ci]Jon Atack2014-04-221-8/+8
| |/ / / / / / /
| * | | | | | | Merge pull request #13335 from glorieux/favicon_link_tag_mimetypeRafael Mendonça França2014-04-223-6/+21
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Change favicon_link_tag helper mimetype from image/vnd.microsoft.icon to image/x-icon.
| | * | | | | | | Change favicon_link_tag helper mimetype from image/vnd.microsoft.icon to ↵glorieux2014-04-213-6/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | image/x-icon. Although the official IANA-registered MIME type for ICO files is image/vnd.microsoft.icon, registered in 2003, it was submitted to IANA by a third party and is not recognized by Microsoft products. The MIME type image/x-icon should be used since is the one recognized by the major browsers on the market.
| * | | | | | | | Merge pull request #10764 from mokevnin/replace_class_eval_by_define_methodRafael Mendonça França2014-04-221-35/+29
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | replace class_eval by define_method in abstract_controller/callbacks
| | * | | | | | | | replace class_eval by define_method in abstract_controller/callbackskirill2014-04-201-35/+29
| | | | | | | | | |
| * | | | | | | | | Merge pull request #14835 from zzak/test_form_helper_label_with_block_and_htmlRafael Mendonça França2014-04-222-0/+8
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Add test for using ActionView::Helpers::FormHelper.label with block and html
| | * | | | | | | | | Add test for using ActionView::Helpers::FormHelper.label with block and htmlZachary Scott2014-04-222-0/+8
| |/ / / / / / / / /
| * | / / / / / / / Rearrange deck chairs on the titanic. Organize connection handling test cases.Jeremy Kemper2014-04-214-249/+246
| | |/ / / / / / / | |/| | | | | | |
| * | | | | | | | Merge pull request #14826 from stevegraham/action_controller_renderer_doc_fixArthur Nogueira Neves2014-04-201-2/+2
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | ActionController::Renderers documentation fix
| | * | | | | | | | ActionController::Renderers documentation fixStevie Graham2014-04-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActionController::Renderers::RENDERERS is an instance of Set. Docs incorrectly state that it's a Hash.
| * | | | | | | | | Merge pull request #14825 from arunagw/aa-remove-missleading-commentArun Agrawal2014-04-201-1/+1
| |\ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | | | Removed missleading comment [ci skip]
| | * | | | | | | | Changed miss-leading comment [ci skip]Arun Agrawal2014-04-201-1/+1
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After this 21dbe6f39b57f52967e92716dbd5e2b894e7a64c 2.1.1 :001 > 'business'.classify => "Business" 2.1.1 :004 > 'calculus'.classify => "Calculu" 2.1.1 :005 >
| * | | | | | | | Merge pull request #14823 from kuldeepaggarwal/fix-inflector-exampleArun Agrawal2014-04-201-1/+1
| |\ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | correct the classify example's output.
| | * | | | | | | [ci-skip] correct the classify example's outputKuldeep Aggarwal2014-04-211-1/+1
| |/ / / / / / /
* | | | | | | | Revert "Edge Rails applications doesn't need to use arel master"Aaron Patterson2014-04-231-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 7f62f30f58fe5f2955e3292ab1f01aa2c9a8f0ee.
* | | | | | | | use statement cache for belongs_to relationsAaron Patterson2014-04-223-3/+17
| | | | | | | |
* | | | | | | | extract record fetching to a method for belongs_toAaron Patterson2014-04-221-1/+5
| | | | | | | |
* | | | | | | | Merge branch 'master' into adequaterecordAaron Patterson2014-04-2086-448/+1015
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (74 commits) [ci skip] builtin -> built-in Fix code indentation and improve formatting Grammar fix in Getting Started Guide Make URL escaping more consistent Optimize URI escaping Always escape string passed to url helper. Remove statement assuming coffee shop/public space wifi is inherently insecure Don't rely on Arel master in bug report template [ci skip] wrap methods in backticks [ci skip] "subhash" --> "sub-hash" multibyte_conformance.rb --> multibyte_conformance_test.rb Fix inconsistent behavior from String#first/#last `@destroyed` should always be set to `false` when an object is duped. remove warning `warning: ambiguous first argument; put parentheses or even spaces` :uglify -> :uglifier Regression test for irregular inflection on has_many Singularize association names before camelization Fix spelling and proper nouns Optimize select_value, select_values, select_rows and dry up checking whether to exec with cache for Postgresql adapter Include default rails protect_from_forgery with: :exception ... Conflicts: activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rb
| * | | | | | | Merge pull request #14820 from akshay-vishnoi/doc_changesRichard Schneeman2014-04-206-8/+8
| |\ \ \ \ \ \ \ | | | |_|_|/ / / | | |/| | | | | [ci skip] builtin -> built-in