| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
Rewrite to avoid 'we'/'you', add missing period, and keep lines at 80 chars. Cheers :)
Improve readability with help from @senny
[ci skip]
|
|\
| |
| | |
Refactored AssociationScope#get_bind_values
|
| |
| |
| |
| | |
Added #join_id_for(owner) to reflection to avoid accessing the source_macro
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Now that delete_all with destroy or delete_all dependency behave
the same we no longer need this conditional. This means we can remove
the new delete_all_with_dependency method I added and go straight
to delete_records from delete_all, passing :all and the dependent
directly.
|
|/
|
|
|
| |
This makes table_name_suffix work the same as table_name_prefix when
using namespaced models. Pretty much the same as 67d1cec.
|
|
|
|
|
|
|
|
| |
This is a regression 4.0 -> 4.1 fix.
In 4.1.0 Relation#join is delegated to Arel#SelectManager.
In 4.0 series it is delegated to Array#join
This patch puts back the behaviour of 4.0
|
|
|
|
| |
The case where the value is present is more common
|
|\
| |
| |
| |
| |
| |
| | |
Fix exception when logging SQL w/ nil binary value.
Conflicts:
activerecord/CHANGELOG.md
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
column_for will raise in case column is not found for the given table,
so there is no need to handle that here.
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
make `db:structure:load` listed with `rake -T`
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
This is a follow up to #14139.
/cc @carlosantoniodasilva
|
|\ \ \
| |/ /
|/| |
| | | |
Fixed custom validation context bug for child associations
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
associations were not being saved.
Fixes #13854.
[Eric Chahin, Aaron Nelson, & Kevin Casey]
|
| |/
|/|
| |
| |
| | |
bind parameters we not being propogated to simple subquery calculation
calls. This fixes it
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Symbolize variables of mysql/mysql2 connection configuration
Conflicts:
activerecord/CHANGELOG.md
|
| |/
| |
| |
| |
| |
| |
| | |
For mysql2/mysql adapters, `sql_mode` variable name set in `database.yml`
as string, was ignored and `sql_mode` was set to use strict mode.
Fixes #14895
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
SQLite3: Always close statements.
Conflicts:
activerecord/CHANGELOG.md
|
| | |
| | |
| | |
| | |
| | |
| | | |
SQLite3 adapter must make sure to close statements after queries.
Fixes: #13631
|
| |/
|/| |
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
eileencodes/fix-delete-all-with-nil-dependency-to-not-produce-in-statement
Fix delete all with nil (:nullify) dependency to not produce in statement
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Nullify (or nil dependency) was doing the same thing delete_all
was doing in issue #14546, creating a large IN statement if
the association was loaded. Loaded and not loaded associations
should behave the same. The IN statement is also not great because
it's inefficient.
|
|\ \ \
| |/ /
|/| | |
[ci skip] updating active_record/associations to demonstrate where conflict with eager loading.
|
| | |
| | |
| | |
| | | |
with eager loading.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
After reviewing this code I realized the conditional that was
there previously was basically saying if the dependency is not
delete all. This is a better, cleaner, and clearer way to write
this method.
|
| | |
| | |
| | |
| | |
| | | |
Now that we have a new method delete_all_with_dependency this
coupled conditional is no longer needed.
|
| | |
| | |
| | |
| | |
| | |
| | | |
The delete method is very coupled with delete all even
though only a portion of the conditionals apply. Decoupling
this will make the code easier to understand and manipulate.
|
|\ \ \
| | | |
| | | | |
Fix to use custom join table in habtm
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
When used a custom join_table name on a habtm, rails was not saving it
on Reflections. This causes a problem when rails loads fixtures, because
it uses the reflections to set database with fixtures.
|
|\| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* master: (28 commits)
move AR length validation tests into separate test-case.
No need for trailing slash on migration path.
reset `@arel` when modifying a Relation in place.
PostgreSQL Timestamps always map to `:datetime`.
[ci skip] Improve formatting and yml
Fix a typo in the doc of forty_two AR FinderMethod
Improve readability of contributing to rails guide. [ci skip]
Precompile the image we're referencing, too.
`ActiveRecord::Base.no_touching` no longer triggers callbacks or start empty transactions.
Fixed an issue with migrating legacy json cookies.
Correct comment [ci skip]
Perfer to define methods instead of calling test
Fix syntax error
Add CHANGELOG entry for #14757 [ci skip]
Fix run-on sentences and improve grammar [skip ci]
Add test for using ActionView::Helpers::FormHelper.label with block and html
select! renamed to avoid name collision Array#select!
Rearrange deck chairs on the titanic. Organize connection handling test cases.
Change favicon_link_tag helper mimetype from image/vnd.microsoft.icon to image/x-icon.
ActionController::Renderers documentation fix
...
|
| | | |
| | | |
| | | |
| | | | |
/cc @tenderlove
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The PG Adapter should use `:datetime` consistently instead of mapping
mispellings to `:timestamp`.
See #14513
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
transactions.
Closes #14841.
|
| |\ \ \
| | | | |
| | | | | |
Fix behavior of select! to be consistent with select #14752
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
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!.
|
| |\ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Properly handle scoping with has_many :through. Fixes #14537.
Conflicts:
activerecord/CHANGELOG.md
activerecord/test/cases/associations/has_many_through_associations_test.rb
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
instance_eval directly.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|