diff options
author | Lisa Ugray <lisa.ugray@shopify.com> | 2017-07-24 14:17:56 -0400 |
---|---|---|
committer | Lisa Ugray <lisa.ugray@shopify.com> | 2017-07-24 21:22:48 -0400 |
commit | 75ccdfed8d43d79f6590653212ecea7124759439 (patch) | |
tree | 3f0917b8813a7f6e05dd732306a759911d3ce4d2 /activerecord/CHANGELOG.md | |
parent | 9c35bf2a6a27431c6aa283db781c19f61c5155be (diff) | |
download | rails-75ccdfed8d43d79f6590653212ecea7124759439.tar.gz rails-75ccdfed8d43d79f6590653212ecea7124759439.tar.bz2 rails-75ccdfed8d43d79f6590653212ecea7124759439.zip |
Stop creating ApplicationRecord on model generation
When generating models, we created ApplicationRecord in the default
location if no file existed there. That was annoying for people who
moved it to somewhere else in the autoload path. At this point, the
vast majority of apps should have either run the upgrade script or
generated a model since upgrading. For those that haven't the error
message after generating a new model should be helpful:
NameError: uninitialized constant ApplicationRecord
To ease friction in that case, this also adds a generator for
ApplicationRecord.
Diffstat (limited to 'activerecord/CHANGELOG.md')
-rw-r--r-- | activerecord/CHANGELOG.md | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/activerecord/CHANGELOG.md b/activerecord/CHANGELOG.md index afc459ef68..a656c767c0 100644 --- a/activerecord/CHANGELOG.md +++ b/activerecord/CHANGELOG.md @@ -1,3 +1,8 @@ +* ApplicationRecord is no longer generated when generating models. If you + need to generate it, it can be created with `rails g application_record`. + + *Lisa Ugray* + * Fix `COUNT(DISTINCT ...)` with `ORDER BY` and `LIMIT` to keep the existing select list. *Ryuta Kamizono* |