| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
AddXXXToYYY/RemoveXXXFromYYY migrations are produced with references
statements, for instance
rails g migration AddReferencesToProducts user:references
supplier:references{polymorphic}
will generate the migration with:
add_reference :products, :user, index: true
add_reference :products, :supplier, polymorphic: true, index: true
|
|\ |
|
| | |
|
| |
| |
| |
| |
| |
| | |
This reverts commit c47a698d5d497340d4e349257522212173865838.
Reason: Let's revert pending further discussions
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
Introduced in 75b340d1a4bcf2f1233fb65a15ff6b8059e2230e
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
A test was failing due to the way that Relation#inspect causes
association proxies to ignore unsaved records added to the association.
This is fixed by simply calling to_a and letting to_a figure out how to
get the records (which, in the case of associations, takes into account
new records).
I think it is acceptable to do this rather than limiting the query at
the database level:
* It's what we've done in all released Rails versions up to this point
* The goal of the limit is to not flood the console with output - this
is the problem we're targeting, rather than the actual loading of the
records from the database
* You probably want to do something with those records later anyway,
otherwise you wouldn't have built a relation for them.
|
|\ \
| | |
| | | |
Allow to register database tasks from different adapters
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
relation
|
| | | |
|
|\ \ \
| | | |
| | | | |
Engine table name prefix generator fix
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Added *instance_writer: false* to stored/serialized attributes.
|
| |/ / / |
|
|/ / /
| | |
| | |
| | | |
While it's interesting to have the results array, it can make a console or a webpage freeze if there are a lot of them.
So this limits the number of records displayed in #inspect to 10 and tells how much were effectively found.
|
|\ \ \
| | | |
| | | | |
Disable query cache for lock queries
|
| | | |
| | | |
| | | |
| | | | |
Fixes #867
|
| | | |
| | | |
| | | |
| | | |
| | | | |
See 07314e64fd62fb8e6165c8c539420160da9437e9.
Also fix some tabs in AR Changelog.
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The reason for removing the previous implementation of `#inspect` was
that it hid from you that you were dealing with a `Relation` rather than
an `Array`.
But it is still useful to be able to see the records, particularly if you're
writing something like the following in tests:
assert_equal [foo], Post.where(:bar)
If the assertion fails, you want to see what records were actually
loaded.
So this implementation makes it clear that you've got a `Relation`, but
also shows your records.
|
|\ \ \
| | | |
| | | | |
Make ArgumentError message more consistent in nested attributes
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This patch fixes quoting for ActiveSupport::Duration instances:
# before
>> ActiveRecord::Base.connection.quote 30.minutes
=> "'--- 1800\n...\n'"
# after
>> ActiveRecord::Base.connection.quote 30.minutes
=> "1800"
Also, adds a test for type casting ActiveSupport::Duration instances.
Related to #1119.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Mocha is already required by AS::TestCase, so remove the duplicate
requires.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Examples:
add_reference :products, :supplier, polymorphic: true, index: true
remove_reference :products, :user
`add_belongs_to` and `remove_belongs_to` are
acceptable.
|
| | | | |
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | | |
Also fix some wrong formatting.
Related discussion:
https://github.com/rails/rails/commit/ab72040b74f742b6676b2d2a5dd029bfdca25a7a#commitcomment-1525256
|
| | |
| | |
| | |
| | | |
connection.tables returns an array of strings
|
| | | |
|
|\ \ \
| | | |
| | | | |
Refactor migration test_helper
|
| | | |
| | | |
| | | |
| | | | |
and create_join_table_test
|
| | | | |
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Add :default and :null options to column_exists? method
Examples:
column_exists?(:testings, :taggable_id, :integer, null: false)
column_exists?(:testings, :taggable_type, :string, default: 'Photo')
|
| |/ /
| | |
| | |
| | | |
Also fix failures in check options for nil
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Conflicts:
activemodel/lib/active_model/errors.rb
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
option [ci skip]
|
| | | | |
|