aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Fix mysql/mysql2 failing with FK constraint errorsGodfrey Chan2014-07-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | One of the author fixture we have ("david") references an author address by ID. Since we disable FK checks when inserting fixtures, this is all fine until we try to update it, at which point MySQL would complain about the missing row referenced by the `author_address_id`. [Godfrey Chan, Matthew Draper]
* | | | | | | Re-enable foriegn key tests on MySQLGodfrey Chan2014-07-052-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e84799d, e31104c and e6ca8e2
* | | | | | | Merge pull request #16035 from sgrif/sg-range-typesRafael Mendonça França2014-07-056-54/+40
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Use the type object for quoting PG Ranges
| * | | | | | | Use the type object for quoting PG RangesSean Griffin2014-07-056-54/+40
| | | | | | | |
* | | | | | | | Only reference time specific methods in the time section [ci skip]Sean Griffin2014-07-051-2/+2
| | | | | | | |
* | | | | | | | Remove *all* references to `Time.yesterday` [ci skip]Sean Griffin2014-07-051-1/+1
| | | | | | | |
* | | | | | | | `Time.today` -> `Date.today` in guides. [ci skip]Sean Griffin2014-07-051-1/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed mentions of non-existent methods, also added a mention for `Date.current`, which is analagous to `Time.current`. Closes #16061.
* | | | | | | Add release notes entry for #15819 [ci skip]Sean Griffin2014-07-041-0/+5
| | | | | | |
* | | | | | | CHANGELOG and release notes entry for #16056Sean Griffin2014-07-042-0/+9
| | | | | | |
* | | | | | | Merge pull request #16056 from sgrif/sg-required-associationsDavid Heinemeier Hansson2014-07-044-1/+105
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add a `required` option to singular associations
| * | | | | | | Add a `required` option to singular associationsSean Griffin2014-07-044-1/+105
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In addition to defining the association, a `required` association will also have its presence validated. Before: ```ruby belongs_to :account validates_presence_of :account ``` After: ```ruby belongs_to :account, required: true ``` This helps to draw a distinction between types of validations, since validations on associations are generally for data integrity purposes, and aren't usually set through form inputs.
* | | | | | | Merge pull request #16054 from rails/revert-16003-sg-refactor-sqlite3-stringsMatthew Draper2014-07-051-37/+13
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Revert "Use a type object for type casting behavior on SQLite3"
| * | | | | | Revert "Use a type object for type casting behavior on SQLite3"Matthew Draper2014-07-051-37/+13
|/ / / / / /
* | | | | | Don't fail if unicode.org isn't talking to usMatthew Draper2014-07-051-1/+3
| | | | | |
* | | | | | Fix typoRafael Mendonça França2014-07-041-1/+1
| | | | | |
* | | | | | Merge pull request #16013 from tgxworld/remove_symbolized_path_parametersRafael Mendonça França2014-07-045-10/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove symbolized_path_parameters.
| * | | | | | Remove symbolized_path_parameters.Guo Xiang Tan2014-07-025-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This pull request is a continuation of https://github.com/rails/rails/commit/925bd975 and https://github.com/rails/rails/commit/8d8ebe3d.
* | | | | | | Merge pull request #16003 from sgrif/sg-refactor-sqlite3-stringsMatthew Draper2014-07-051-13/+37
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Use a type object for type casting behavior on SQLite3
| * | | | | | | Use a type object for type casting behavior on SQLite3Sean Griffin2014-07-011-13/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note: I'm not sure we actually need to be logging when this happens. This code would be a fair bit cleaner if we didn't need to log it.
* | | | | | | | Merge pull request #16028 from ↵Rafael Mendonça França2014-07-043-1/+16
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cade/fix_counter_cache_count_with_association_selects Add `:all` argument to `count` in `reset_counters` Conflicts: activerecord/CHANGELOG.md
| * | | | | | | | Add `:all` argument to `count` in `reset_counters`Cade Truitt2014-07-023-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this fix, if an association had a scope with a `select`, calls to `reset_counters` would generate invalid SQL and throw: ActiveRecord::StatementInvalid: [$DB_ADAPTER]: wrong number of arguments to function COUNT() References #10710, #13648
* | | | | | | | | Merge pull request #16011 from xjlu/token_and_optionsRafael Mendonça França2014-07-042-3/+23
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Improve token_and_options regex and test
| * | | | | | | | | Improve token_and_options regex and testXinjiang Lu2014-07-012-3/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add a test case to test the regex for the helper method raw_params
* | | | | | | | | | Merge pull request #16040 from seuros/patch-1Matthew Draper2014-07-051-0/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Add arel to the controller template [ci skip]
| * | | | | | | | | | Add arel to the controller templateAbdelkader Boudih2014-07-031-0/+1
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #14005 from dmitry/test_coverage_for_actionmailerRafael Mendonça França2014-07-046-38/+16
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add test coverage for the action mailer
| * | | | | | | | | | | add test coverage for the action mailerDmitry Polushkin2014-07-046-38/+16
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #16042 from akshay-vishnoi/doc_changeMatthew Draper2014-07-053-5/+5
|\ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | [ci skip] /javascript/ -> JavaScript - cover whole app
| * | | | | | | | | | | [ci skip] /javascript/ -> JavaScript - cover whole appAkshay Vishnoi2014-07-043-5/+5
| | |/ / / / / / / / / | |/| | | | | | | | |
* | | | | | | | | | | Merge pull request #16049 from matthewd/find-via-reloadMatthew Draper2014-07-053-0/+19
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | After find-via-reload, the record is not new
| * | | | | | | | | | | After find-via-reload, the record is not newMatthew Draper2014-07-053-0/+19
| | |/ / / / / / / / / | |/| | | | | | | | |
* | | | | | | | | | | Merge pull request #16046 from grosser/grosser/cleangenRafael Mendonça França2014-07-041-2/+1
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Do not load rails without loading bundler
| * | | | | | | | | | | remove bundler magicgrosser2014-07-031-2/+1
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #16037 from sgrif/sg-money-quotingRafael Mendonça França2014-07-043-11/+28
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | | Remove unneccessary special case for money in quoting
| * | | | | | | | | | | Remove unneccessary special case for money in quotingSean Griffin2014-07-033-11/+28
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #16048 from arunagw/aa-added-notice-index-pageRafael Mendonça França2014-07-043-1/+11
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / / |/| | | | | | | | | | | Display notice in index.html pages in scaffolded generated views
| * | | | | | | | | | | Display notice in index.html pages in scaffolded generated viewsArun Agrawal2014-07-043-1/+11
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As we are setting notice in destroy action we should display that For more information see https://github.com/rails/rails/pull/14044 And https://github.com/rails/jbuilder/pull/191 closes #14044
* | | / / / / / / / / :fire: these are lambdas now [ci skip]Godfrey Chan2014-07-031-9/+2
| |_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | This has changed since around 2b1500d6
* | | | | | | | | | Merge pull request #16038 from tbpgr/fix_guide_active_record_validationsRafael Mendonça França2014-07-031-1/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Fix guide. sample code syntax error.[ci skip]
| * | | | | | | | | | Fix guide. sample code syntax error.[ci skip]tbpgr2014-07-041-1/+1
| |/ / / / / / / / /
* | | | | | | | | | Merge pull request #16036 from sgrif/sg-datetime-infinityRafael Mendonça França2014-07-033-3/+46
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Do not rely on the column type when quoting infinity
| * | | | | | | | | | Do not rely on the column type when quoting infinitySean Griffin2014-07-033-3/+46
| |/ / / / / / / / /
* | | | | | | | | | Merge pull request #16034 from sgrif/sg-hstore-type-objectRafael Mendonça França2014-07-034-66/+39
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | | Use the type object for type casting HStore columns
| * | | | | | | | | Use the type object for type casting HStore columnsSean Griffin2014-07-034-66/+39
|/ / / / / / / / /
* | | | | | | | | Merge pull request #15941 from variousred/docrails-changesVijay Dev2014-07-031-8/+10
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | [ci skip] Changes some wording to better foreshadow what is coming up later
| * | | | | | | | [ci skip] Changes some wording to better foreshadow what is coming up later.Michael Mitchell2014-07-021-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should prevent some anxiety readers may experience when the doc goes into something different than what it just explained. Also clarifies a statement about priority in engines and application environments [ci skip] fixes double `the` [ci skip] more clarifications [ci skip] changes from tics to quotes
* | | | | | | | | Merge pull request #16029 from cade/fix_typo_in_commentVijay Dev2014-07-031-1/+1
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | | Fix typo in comment
| * | | | | | | | Fix typo in commentCade Truitt2014-07-021-1/+1
|/ / / / / / / /
* | | | | | | | Merge pull request #16026 from lucasmazza/lm-respond-to-jsonp-fixRafael Mendonça França2014-07-023-1/+24
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Change the JSON renderer to enforce the 'JS' Content Type
| * | | | | | | | Change the JSON renderer to enforce the 'JS' Content TypeLucas Mazza2014-07-023-1/+24
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The controller can set the response format as 'JSON' before the renderer code be evaluated, so we must replace it when necessary. Fixes #15081
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-20 00:24:48 +0000