aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | Merge pull request #16072 from sgrif/sg-xml-quotingRafael Mendonça França2014-07-085-5/+56
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | | Don't rely on the sql type to quote XML columns in PG
| * | | | | | | | | | | Don't rely on the sql type to quote XML columns in PGSean Griffin2014-07-065-5/+56
| |/ / / / / / / / / /
* | | | | | | | | | | test i18n against a routed appAaron Patterson2014-07-071-1/+16
| | | | | | | | | | |
* | | | | | | | | | | remove the mounted_helpers respond_to checkAaron Patterson2014-07-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It always responds to mounted_helpers now
* | | | | | | | | | | always test against a routed rack app so there are always url_helpersAaron Patterson2014-07-075-36/+66
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #16078 from tgxworld/retreive_connection_onceRafael Mendonça França2014-07-071-2/+3
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Avoid retrieving connection in ActiveRecord::QueryCache multiple times.
| * | | | | | | | | | | Avoid retrieving connection in ActiveRecord::QueryCache multiple times.Guo Xiang Tan2014-07-071-2/+3
| |/ / / / / / / / / /
* | | | | | | | | | | Merge pull request #16074 from sgrif/sg-remove-text-predicateRafael Mendonça França2014-07-0718-32/+4
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Remove the `text?` predicate from the type objects
| * | | | | | | | | | | Remove the `text?` predicate from the type objectsSean Griffin2014-07-0618-32/+4
| |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was only used for uniqueness validations. The first usage was in conjunction with `limit`. Types which cast to string, but are not considered text cannot have a limit. The second case was only with an explicit `:case_sensitive => true` option given by the user.
* | | | | | | | | | | Merge pull request #16086 from nickpellant/masterEileen M. Uchitelle2014-07-071-0/+14
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add callbacks documentation for upgrading to 4.1.
| * | | | | | | | | | | [ci skip] Improve callback code example for 4.1 upgrade docs.Nick Pellant2014-07-071-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous code example for the new explicit block requirement when setting callbacks was a little confusing. This commit makes the example more obvious.
| * | | | | | | | | | | Add callbacks documentation for upgrading to 4.1.Nick Pellant2014-07-071-0/+14
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is now expected in 4.1 to use an explicit block rather than implicit when setting callbacks through ActiveSupport::Callbacks. This commit highlights this new expectation as part of the upgrading documentation.
* / | | | | | | | | | Update Rails versions [ci skip]Robin Dupret2014-07-075-19/+19
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As discussed in #15304, we need to automate this process but for now, having out-of-date versions is not ideal. Since master targets 4.2.0, let's also update references to the last 4.1 version to 4.2.0. Finally, let's remove mentions to versions when this is not needed. The guides cover the features of the current version anyway. [Juanito Fatas + Robin Dupret]
* | | | | | | | | | Merge pull request #16075 from nickpellant/masterSean Griffin2014-07-061-1/+1
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | | Fix ActiveSupport::Callbacks #set_callback docs.
| * | | | | | | | | Fix ActiveSupport::Callbacks #set_callback docs.Nick Pellant2014-07-071-1/+1
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A minor version breakage due to a rewrite of the callbacks code now requires an explicit block to be passed to #set_callback. This amends the documentation.
* | | | | | | | | Change back occurrences of SQLite(3) to sqlite3 when referring to theZachary Scott2014-07-065-12/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | adapter, fixed from #16057 [ci skip]
* | | | | | | | | Merge pull request #16057 from akshay-vishnoi/doc_changeZachary Scott2014-07-066-9/+9
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | [ci skip] /sqlite/i --> SQLite
| * | | | | | | | | [ci skip] /sqlite/i --> SQLiteAkshay Vishnoi2014-07-066-9/+9
| | | | | | | | | |
* | | | | | | | | | Merge pull request #15958 from aditya-kapoor/active-model-naming-guideRobin Dupret2014-07-061-0/+23
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | [ci skip] add guide for ActiveModel::Naming
| * | | | | | | | | | [ci skip] add guide for ActiveModel::NamingAditya Kapoor2014-07-051-0/+23
| | | | | | | | | | |
* | | | | | | | | | | Generate shallow paths for all children of shallow resources.Seb Jacobs2014-07-063-2/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this commit shallow resources would only generate paths for non-direct children (with a nested depth greater than 1). Take the following routes file. resources :blogs do resources :posts, shallow: true do resources :comments do resources :tags end end end This would generate shallow paths for `tags` nested under `posts`, e.g `/posts/:id/tags/`, however it would not generate shallow paths for `comments` nested under `posts`, e.g `/posts/:id/comments/new`. This commit changes the behaviour of the route mapper so that it generate paths for direct children of shallow resources, for example if you take the previous routes file, this will now generate shallow paths for `comments` nested under `posts`, .e.g `posts/:id/comments/new`. This was the behaviour in Rails `4.0.4` however this was broken in @jcoglan's fix for another routes related issue[1]. This also fixes an issue[2] reported by @smdern. [1] https://github.com/rails/rails/commit/d0e5963 [2] https://github.com/rails/rails/issues/15783
* | | | | | | | | | | Merge pull request #16069 from dylanahsmith/mysql-quote-boolMatthew Draper2014-07-065-5/+49
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | active_record: Type cast booleans and durations for string columns.
| * | | | | | | | | | | active_record: Type cast booleans and durations for string columns.Dylan Thacker-Smith2014-07-065-5/+49
|/ / / / / / / / / / /
* | | | | | | | | | | Merge pull request #16044 from jjb/patch-1Zachary Scott2014-07-051-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] link directly to ActionMailer Previews documentation
| * | | | | | | | | | | link directory to ActionMailer Previews documentationJohn Bachir2014-07-031-1/+1
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #16068 from JuanitoFatas/4-2-changelogsZachary Scott2014-07-051-18/+14
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] Concludes changelog links in the bottom.
| * | | | | | | | | | | | [ci skip] Concludes changelog links in the bottom.Juanito Fatas2014-07-061-18/+14
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This way when we do next release notes, this would be easier to copy and change.
* | | | | | | | | | | | Revert "Merge pull request #16059 from jenncoop/json-serialized-attr"Godfrey Chan2014-07-055-76/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit a03097759bd7103bb9db253e7ba095f011453f75. This needs more work before it would work correctly on master.
* | | | | | | | | | | | Merge pull request #16059 from jenncoop/json-serialized-attrGodfrey Chan2014-07-055-2/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed issue with ActiveRecord serialize object as JSON Conflicts: activerecord/CHANGELOG.md activerecord/lib/active_record/attribute_methods/serialization.rb
* | | | | | | | | | | | Merge pull request #16065 from matthewd/mysql-explicit-non-strictMatthew Draper2014-07-061-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | If our connection is explicitly non-strict, tell MySQL
| * | | | | | | | | | | | If our connection is explicitly non-strict, tell MySQLMatthew Draper2014-07-061-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We default to making the connection strict, but have historically relied on the MySQL default when we want it to be non-strict. On some (recent?) versions of MySQL, new connections default to being strict, so if we've been told 'strict:false', we're obliged to pass that on. This fixes a test failure that we've seen turn up on relatively-new development machines, so we do already have a test covering it.
* | | | | | | | | | | | | 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.