activerecord 6.0.1 → 6.0.3
Sign up to get free protection for your applications and to get access to all the features.
Potentially problematic release.
This version of activerecord might be problematic. Click here for more details.
- checksums.yaml +4 -4
- data/CHANGELOG.md +111 -0
- data/README.rdoc +1 -1
- data/lib/active_record.rb +1 -0
- data/lib/active_record/advisory_lock_base.rb +18 -0
- data/lib/active_record/aggregations.rb +0 -1
- data/lib/active_record/association_relation.rb +4 -11
- data/lib/active_record/associations.rb +1 -1
- data/lib/active_record/associations/alias_tracker.rb +0 -1
- data/lib/active_record/associations/association.rb +5 -9
- data/lib/active_record/associations/builder/has_and_belongs_to_many.rb +0 -2
- data/lib/active_record/associations/collection_association.rb +6 -2
- data/lib/active_record/associations/collection_proxy.rb +1 -2
- data/lib/active_record/associations/has_many_association.rb +0 -1
- data/lib/active_record/associations/join_dependency.rb +9 -0
- data/lib/active_record/associations/preloader.rb +1 -2
- data/lib/active_record/attribute_assignment.rb +0 -1
- data/lib/active_record/attribute_decorators.rb +0 -2
- data/lib/active_record/attribute_methods/before_type_cast.rb +0 -1
- data/lib/active_record/attribute_methods/dirty.rb +2 -2
- data/lib/active_record/attribute_methods/primary_key.rb +0 -2
- data/lib/active_record/attribute_methods/read.rb +0 -1
- data/lib/active_record/attribute_methods/serialization.rb +0 -1
- data/lib/active_record/attribute_methods/time_zone_conversion.rb +0 -2
- data/lib/active_record/attribute_methods/write.rb +0 -1
- data/lib/active_record/attributes.rb +0 -1
- data/lib/active_record/autosave_association.rb +1 -3
- data/lib/active_record/callbacks.rb +1 -2
- data/lib/active_record/coders/yaml_column.rb +0 -1
- data/lib/active_record/connection_adapters/abstract/connection_pool.rb +0 -3
- data/lib/active_record/connection_adapters/abstract/database_statements.rb +17 -15
- data/lib/active_record/connection_adapters/abstract/query_cache.rb +0 -1
- data/lib/active_record/connection_adapters/abstract/schema_creation.rb +1 -2
- data/lib/active_record/connection_adapters/abstract/schema_definitions.rb +27 -27
- data/lib/active_record/connection_adapters/abstract/schema_statements.rb +47 -30
- data/lib/active_record/connection_adapters/abstract/transaction.rb +4 -5
- data/lib/active_record/connection_adapters/abstract_adapter.rb +7 -3
- data/lib/active_record/connection_adapters/abstract_mysql_adapter.rb +12 -28
- data/lib/active_record/connection_adapters/connection_specification.rb +1 -2
- data/lib/active_record/connection_adapters/mysql/column.rb +1 -1
- data/lib/active_record/connection_adapters/mysql/database_statements.rb +5 -11
- data/lib/active_record/connection_adapters/mysql/explain_pretty_printer.rb +0 -1
- data/lib/active_record/connection_adapters/mysql/schema_creation.rb +1 -2
- data/lib/active_record/connection_adapters/mysql/schema_statements.rb +8 -8
- data/lib/active_record/connection_adapters/mysql2_adapter.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/database_statements.rb +6 -2
- data/lib/active_record/connection_adapters/postgresql/oid/array.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/oid/enum.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/oid/hstore.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/oid/legacy_point.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/oid/oid.rb +1 -1
- data/lib/active_record/connection_adapters/postgresql/oid/point.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/oid/range.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/oid/specialized_string.rb +1 -1
- data/lib/active_record/connection_adapters/postgresql/oid/uuid.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/referential_integrity.rb +1 -1
- data/lib/active_record/connection_adapters/postgresql/schema_creation.rb +2 -2
- data/lib/active_record/connection_adapters/postgresql/schema_definitions.rb +1 -1
- data/lib/active_record/connection_adapters/postgresql/schema_dumper.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql/schema_statements.rb +15 -29
- data/lib/active_record/connection_adapters/postgresql/utils.rb +0 -1
- data/lib/active_record/connection_adapters/postgresql_adapter.rb +5 -2
- data/lib/active_record/connection_adapters/sqlite3/database_statements.rb +5 -6
- data/lib/active_record/connection_adapters/sqlite3/quoting.rb +0 -1
- data/lib/active_record/connection_adapters/sqlite3/schema_statements.rb +3 -3
- data/lib/active_record/connection_adapters/sqlite3_adapter.rb +8 -7
- data/lib/active_record/connection_adapters/statement_pool.rb +0 -1
- data/lib/active_record/connection_handling.rb +10 -26
- data/lib/active_record/core.rb +0 -2
- data/lib/active_record/counter_cache.rb +4 -1
- data/lib/active_record/database_configurations/url_config.rb +0 -1
- data/lib/active_record/dynamic_matchers.rb +2 -3
- data/lib/active_record/explain.rb +0 -1
- data/lib/active_record/fixture_set/table_row.rb +0 -1
- data/lib/active_record/fixture_set/table_rows.rb +0 -1
- data/lib/active_record/fixtures.rb +0 -3
- data/lib/active_record/gem_version.rb +1 -1
- data/lib/active_record/inheritance.rb +0 -3
- data/lib/active_record/insert_all.rb +3 -3
- data/lib/active_record/internal_metadata.rb +1 -1
- data/lib/active_record/locking/optimistic.rb +0 -1
- data/lib/active_record/log_subscriber.rb +1 -1
- data/lib/active_record/middleware/database_selector.rb +0 -1
- data/lib/active_record/middleware/database_selector/resolver.rb +1 -2
- data/lib/active_record/migration.rb +4 -4
- data/lib/active_record/migration/command_recorder.rb +6 -18
- data/lib/active_record/migration/compatibility.rb +3 -3
- data/lib/active_record/migration/join_table.rb +0 -1
- data/lib/active_record/model_schema.rb +0 -2
- data/lib/active_record/nested_attributes.rb +0 -2
- data/lib/active_record/no_touching.rb +2 -2
- data/lib/active_record/null_relation.rb +0 -1
- data/lib/active_record/persistence.rb +4 -5
- data/lib/active_record/querying.rb +1 -1
- data/lib/active_record/railtie.rb +1 -1
- data/lib/active_record/railties/collection_cache_association_loading.rb +1 -1
- data/lib/active_record/railties/databases.rake +3 -0
- data/lib/active_record/reflection.rb +8 -8
- data/lib/active_record/relation.rb +12 -1
- data/lib/active_record/relation/batches.rb +0 -1
- data/lib/active_record/relation/calculations.rb +1 -1
- data/lib/active_record/relation/delegation.rb +7 -6
- data/lib/active_record/relation/finder_methods.rb +0 -1
- data/lib/active_record/relation/from_clause.rb +4 -0
- data/lib/active_record/relation/merger.rb +0 -1
- data/lib/active_record/relation/predicate_builder.rb +1 -5
- data/lib/active_record/relation/query_methods.rb +22 -9
- data/lib/active_record/relation/spawn_methods.rb +0 -1
- data/lib/active_record/relation/where_clause.rb +0 -1
- data/lib/active_record/result.rb +0 -1
- data/lib/active_record/schema_migration.rb +1 -1
- data/lib/active_record/scoping.rb +0 -1
- data/lib/active_record/scoping/default.rb +0 -1
- data/lib/active_record/scoping/named.rb +3 -3
- data/lib/active_record/store.rb +1 -1
- data/lib/active_record/suppressor.rb +2 -2
- data/lib/active_record/table_metadata.rb +16 -1
- data/lib/active_record/tasks/mysql_database_tasks.rb +0 -1
- data/lib/active_record/tasks/postgresql_database_tasks.rb +0 -1
- data/lib/active_record/tasks/sqlite_database_tasks.rb +0 -1
- data/lib/active_record/test_fixtures.rb +2 -1
- data/lib/active_record/timestamp.rb +0 -1
- data/lib/active_record/touch_later.rb +1 -2
- data/lib/active_record/transactions.rb +8 -8
- data/lib/active_record/type.rb +0 -1
- data/lib/active_record/type/adapter_specific_registry.rb +2 -5
- data/lib/active_record/type/hash_lookup_type_map.rb +0 -1
- data/lib/active_record/type/serialized.rb +0 -1
- data/lib/active_record/type/type_map.rb +0 -1
- data/lib/active_record/type/unsigned_integer.rb +0 -1
- data/lib/active_record/validations.rb +2 -3
- data/lib/active_record/validations/associated.rb +1 -2
- data/lib/arel.rb +5 -1
- data/lib/arel/predications.rb +5 -6
- data/lib/arel/visitors/depth_first.rb +0 -1
- data/lib/arel/visitors/dot.rb +0 -1
- data/lib/arel/visitors/mssql.rb +0 -1
- data/lib/arel/visitors/oracle.rb +1 -2
- data/lib/arel/visitors/oracle12.rb +0 -1
- data/lib/arel/visitors/postgresql.rb +0 -1
- data/lib/arel/visitors/sqlite.rb +0 -1
- data/lib/arel/visitors/to_sql.rb +0 -1
- data/lib/arel/visitors/visitor.rb +0 -1
- data/lib/arel/visitors/where_sql.rb +0 -1
- data/lib/rails/generators/active_record/application_record/application_record_generator.rb +0 -1
- data/lib/rails/generators/active_record/migration.rb +0 -1
- data/lib/rails/generators/active_record/migration/templates/migration.rb.tt +1 -1
- data/lib/rails/generators/active_record/model/model_generator.rb +0 -1
- metadata +12 -11
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: d8ac16c2194f2d74297c1fdbfd64c286003ce2aea666afd85bfa07ecac816877
|
4
|
+
data.tar.gz: a15e33a862b0985aea1d3b506ce53dd38678c3f47a121dddcd5c90388665788e
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: be693dc74628ffedf85fe5dd569b2bec88e1ccd1b3ed61a347ce63f612747c8e50485d41d46a70fd803f8fbc454572592e59bea246152c0902057ef5c384a235
|
7
|
+
data.tar.gz: 33294c99f5631c31762075cfb88da12fb106794f4a68d28d227fb177d92fe1afd832148331b9ba273f338fa749d1a3679cffaafef464549f79292e2de8b08e08
|
data/CHANGELOG.md
CHANGED
@@ -1,3 +1,114 @@
|
|
1
|
+
## Rails 6.0.3 (May 06, 2020) ##
|
2
|
+
|
3
|
+
* Recommend applications don't use the `database` kwarg in `connected_to`
|
4
|
+
|
5
|
+
The database kwarg in `connected_to` was meant to be used for one-off scripts but is often used in requests. This is really dangerous because it re-establishes a connection every time. It's deprecated in 6.1 and will be removed in 6.2 without replacement. This change soft deprecates it in 6.0 by removing documentation.
|
6
|
+
|
7
|
+
*Eileen M. Uchitelle*
|
8
|
+
|
9
|
+
* Fix support for PostgreSQL 11+ partitioned indexes.
|
10
|
+
|
11
|
+
*Sebastián Palma*
|
12
|
+
|
13
|
+
* Add support for beginless ranges, introduced in Ruby 2.7.
|
14
|
+
|
15
|
+
*Josh Goodall*
|
16
|
+
|
17
|
+
* Fix insert_all with enum values
|
18
|
+
|
19
|
+
Fixes #38716.
|
20
|
+
|
21
|
+
*Joel Blum*
|
22
|
+
|
23
|
+
* Regexp-escape table name for MS SQL
|
24
|
+
|
25
|
+
Add `Regexp.escape` to one method in ActiveRecord, so that table names with regular expression characters in them work as expected. Since MS SQL Server uses "[" and "]" to quote table and column names, and those characters are regular expression characters, methods like `pluck` and `select` fail in certain cases when used with the MS SQL Server adapter.
|
26
|
+
|
27
|
+
*Larry Reid*
|
28
|
+
|
29
|
+
* Store advisory locks on their own named connection.
|
30
|
+
|
31
|
+
Previously advisory locks were taken out against a connection when a migration started. This works fine in single database applications but doesn't work well when migrations need to open new connections which results in the lock getting dropped.
|
32
|
+
|
33
|
+
In order to fix this we are storing the advisory lock on a new connection with the connection specification name `AdisoryLockBase`. The caveat is that we need to maintain at least 2 connections to a database while migrations are running in order to do this.
|
34
|
+
|
35
|
+
*Eileen M. Uchitelle*, *John Crepezzi*
|
36
|
+
|
37
|
+
* Ensure `:reading` connections always raise if a write is attempted.
|
38
|
+
|
39
|
+
Now Rails will raise an `ActiveRecord::ReadOnlyError` if any connection on the reading handler attempts to make a write. If your reading role needs to write you should name the role something other than `:reading`.
|
40
|
+
|
41
|
+
*Eileen M. Uchitelle*
|
42
|
+
|
43
|
+
* Enforce fresh ETag header after a collection's contents change by adding
|
44
|
+
ActiveRecord::Relation#cache_key_with_version. This method will be used by
|
45
|
+
ActionController::ConditionalGet to ensure that when collection cache versioning
|
46
|
+
is enabled, requests using ConditionalGet don't return the same ETag header
|
47
|
+
after a collection is modified. Fixes #38078.
|
48
|
+
|
49
|
+
*Aaron Lipman*
|
50
|
+
|
51
|
+
* A database URL can now contain a querystring value that contains an equal sign. This is needed to support passing PostgresSQL `options`.
|
52
|
+
|
53
|
+
*Joshua Flanagan*
|
54
|
+
|
55
|
+
* Retain explicit selections on the base model after applying `includes` and `joins`.
|
56
|
+
|
57
|
+
Resolves #34889.
|
58
|
+
|
59
|
+
*Patrick Rebsch*
|
60
|
+
|
61
|
+
|
62
|
+
## Rails 6.0.2.2 (March 19, 2020) ##
|
63
|
+
|
64
|
+
* No changes.
|
65
|
+
|
66
|
+
|
67
|
+
## Rails 6.0.2.1 (December 18, 2019) ##
|
68
|
+
|
69
|
+
* No changes.
|
70
|
+
|
71
|
+
|
72
|
+
## Rails 6.0.2 (December 13, 2019) ##
|
73
|
+
|
74
|
+
* Share the same connection pool for primary and replica databases in the
|
75
|
+
transactional tests for the same database.
|
76
|
+
|
77
|
+
*Edouard Chin*
|
78
|
+
|
79
|
+
* Fix the preloader when one record is fetched using `after_initialize`
|
80
|
+
but not the entire collection.
|
81
|
+
|
82
|
+
*Bradley Price*
|
83
|
+
|
84
|
+
* Fix collection callbacks not terminating when `:abort` is thrown.
|
85
|
+
|
86
|
+
*Edouard Chin*, *Ryuta Kamizono*
|
87
|
+
|
88
|
+
* Correctly deprecate `where.not` working as NOR for relations.
|
89
|
+
|
90
|
+
12a9664 deprecated where.not working as NOR, however
|
91
|
+
doing a relation query like `where.not(relation: { ... })`
|
92
|
+
wouldn't be properly deprecated and `where.not` would work as
|
93
|
+
NAND instead.
|
94
|
+
|
95
|
+
*Edouard Chin*
|
96
|
+
|
97
|
+
* Fix `db:migrate` task with multiple databases to restore the connection
|
98
|
+
to the previous database.
|
99
|
+
|
100
|
+
The migrate task iterates and establish a connection over each db
|
101
|
+
resulting in the last one to be used by subsequent rake tasks.
|
102
|
+
We should reestablish a connection to the connection that was
|
103
|
+
established before the migrate tasks was run
|
104
|
+
|
105
|
+
*Edouard Chin*
|
106
|
+
|
107
|
+
* Fix multi-threaded issue for `AcceptanceValidator`.
|
108
|
+
|
109
|
+
*Ryuta Kamizono*
|
110
|
+
|
111
|
+
|
1
112
|
## Rails 6.0.1 (November 5, 2019) ##
|
2
113
|
|
3
114
|
* Common Table Expressions are allowed on read-only connections.
|
data/README.rdoc
CHANGED
@@ -216,4 +216,4 @@ Bug reports for the Ruby on Rails project can be filed here:
|
|
216
216
|
|
217
217
|
Feature requests should be discussed on the rails-core mailing list here:
|
218
218
|
|
219
|
-
* https://
|
219
|
+
* https://discuss.rubyonrails.org/c/rubyonrails-core
|
data/lib/active_record.rb
CHANGED
@@ -0,0 +1,18 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module ActiveRecord
|
4
|
+
# This class is used to create a connection that we can use for advisory
|
5
|
+
# locks. This will take out a "global" lock that can't be accidentally
|
6
|
+
# removed if a new connection is established during a migration.
|
7
|
+
class AdvisoryLockBase < ActiveRecord::Base # :nodoc:
|
8
|
+
self.abstract_class = true
|
9
|
+
|
10
|
+
self.connection_specification_name = "AdvisoryLockBase"
|
11
|
+
|
12
|
+
class << self
|
13
|
+
def _internal?
|
14
|
+
true
|
15
|
+
end
|
16
|
+
end
|
17
|
+
end
|
18
|
+
end
|
@@ -2,7 +2,7 @@
|
|
2
2
|
|
3
3
|
module ActiveRecord
|
4
4
|
class AssociationRelation < Relation
|
5
|
-
def initialize(klass, association)
|
5
|
+
def initialize(klass, association, **)
|
6
6
|
super(klass)
|
7
7
|
@association = association
|
8
8
|
end
|
@@ -17,28 +17,21 @@ module ActiveRecord
|
|
17
17
|
|
18
18
|
def build(attributes = nil, &block)
|
19
19
|
block = _deprecated_scope_block("new", &block)
|
20
|
-
@association.
|
21
|
-
@association.build(attributes, &block)
|
22
|
-
end
|
20
|
+
scoping { @association.build(attributes, &block) }
|
23
21
|
end
|
24
22
|
alias new build
|
25
23
|
|
26
24
|
def create(attributes = nil, &block)
|
27
25
|
block = _deprecated_scope_block("create", &block)
|
28
|
-
@association.
|
29
|
-
@association.create(attributes, &block)
|
30
|
-
end
|
26
|
+
scoping { @association.create(attributes, &block) }
|
31
27
|
end
|
32
28
|
|
33
29
|
def create!(attributes = nil, &block)
|
34
30
|
block = _deprecated_scope_block("create!", &block)
|
35
|
-
@association.
|
36
|
-
@association.create!(attributes, &block)
|
37
|
-
end
|
31
|
+
scoping { @association.create!(attributes, &block) }
|
38
32
|
end
|
39
33
|
|
40
34
|
private
|
41
|
-
|
42
35
|
def exec_queries
|
43
36
|
super do |record|
|
44
37
|
@association.set_inverse_instance_from_queries(record)
|
@@ -1857,7 +1857,7 @@ module ActiveRecord
|
|
1857
1857
|
hm_options[k] = options[k] if options.key? k
|
1858
1858
|
end
|
1859
1859
|
|
1860
|
-
has_many name, scope, hm_options, &extension
|
1860
|
+
has_many name, scope, **hm_options, &extension
|
1861
1861
|
_reflections[name.to_s].parent_reflection = habtm_reflection
|
1862
1862
|
end
|
1863
1863
|
end
|
@@ -43,7 +43,6 @@ module ActiveRecord
|
|
43
43
|
reflection.check_validity!
|
44
44
|
|
45
45
|
@owner, @reflection = owner, reflection
|
46
|
-
@_scope = nil
|
47
46
|
|
48
47
|
reset
|
49
48
|
reset_scope
|
@@ -96,7 +95,11 @@ module ActiveRecord
|
|
96
95
|
end
|
97
96
|
|
98
97
|
def scope
|
99
|
-
|
98
|
+
if (scope = klass.current_scope) && scope.try(:proxy_association) == self
|
99
|
+
scope.spawn
|
100
|
+
else
|
101
|
+
target_scope.merge!(association_scope)
|
102
|
+
end
|
100
103
|
end
|
101
104
|
|
102
105
|
def reset_scope
|
@@ -196,13 +199,6 @@ module ActiveRecord
|
|
196
199
|
_create_record(attributes, true, &block)
|
197
200
|
end
|
198
201
|
|
199
|
-
def scoping(relation, &block)
|
200
|
-
@_scope = relation
|
201
|
-
relation.scoping(&block)
|
202
|
-
ensure
|
203
|
-
@_scope = nil
|
204
|
-
end
|
205
|
-
|
206
202
|
private
|
207
203
|
def find_target
|
208
204
|
scope = self.scope
|
@@ -46,7 +46,6 @@ module ActiveRecord::Associations::Builder # :nodoc:
|
|
46
46
|
end
|
47
47
|
|
48
48
|
private
|
49
|
-
|
50
49
|
def self.suppress_composite_primary_key(pk)
|
51
50
|
pk unless pk.is_a?(Array)
|
52
51
|
end
|
@@ -73,7 +72,6 @@ module ActiveRecord::Associations::Builder # :nodoc:
|
|
73
72
|
end
|
74
73
|
|
75
74
|
private
|
76
|
-
|
77
75
|
def middle_options(join_model)
|
78
76
|
middle_options = {}
|
79
77
|
middle_options[:class_name] = "#{lhs_model.name}::#{join_model.name}"
|
@@ -378,7 +378,9 @@ module ActiveRecord
|
|
378
378
|
end
|
379
379
|
|
380
380
|
def remove_records(existing_records, records, method)
|
381
|
-
|
381
|
+
catch(:abort) do
|
382
|
+
records.each { |record| callback(:before_remove, record) }
|
383
|
+
end || return
|
382
384
|
|
383
385
|
delete_records(existing_records, method) if existing_records.any?
|
384
386
|
@target -= records
|
@@ -434,7 +436,9 @@ module ActiveRecord
|
|
434
436
|
end
|
435
437
|
|
436
438
|
def replace_on_target(record, index, skip_callbacks)
|
437
|
-
|
439
|
+
catch(:abort) do
|
440
|
+
callback(:before_add, record)
|
441
|
+
end || return unless skip_callbacks
|
438
442
|
|
439
443
|
set_inverse_instance(record)
|
440
444
|
|
@@ -27,7 +27,7 @@ module ActiveRecord
|
|
27
27
|
# is computed directly through SQL and does not trigger by itself the
|
28
28
|
# instantiation of the actual post records.
|
29
29
|
class CollectionProxy < Relation
|
30
|
-
def initialize(klass, association) #:nodoc:
|
30
|
+
def initialize(klass, association, **) #:nodoc:
|
31
31
|
@association = association
|
32
32
|
super klass
|
33
33
|
|
@@ -1101,7 +1101,6 @@ module ActiveRecord
|
|
1101
1101
|
delegate(*delegate_methods, to: :scope)
|
1102
1102
|
|
1103
1103
|
private
|
1104
|
-
|
1105
1104
|
def find_nth_with_limit(index, limit)
|
1106
1105
|
load_target if find_from_target?
|
1107
1106
|
super
|
@@ -105,7 +105,9 @@ module ActiveRecord
|
|
105
105
|
|
106
106
|
model_cache = Hash.new { |h, klass| h[klass] = {} }
|
107
107
|
parents = model_cache[join_root]
|
108
|
+
|
108
109
|
column_aliases = aliases.column_aliases join_root
|
110
|
+
column_aliases += explicit_selections(column_aliases, result_set)
|
109
111
|
|
110
112
|
message_bus = ActiveSupport::Notifications.instrumenter
|
111
113
|
|
@@ -135,6 +137,13 @@ module ActiveRecord
|
|
135
137
|
private
|
136
138
|
attr_reader :alias_tracker
|
137
139
|
|
140
|
+
def explicit_selections(root_column_aliases, result_set)
|
141
|
+
root_names = root_column_aliases.map(&:name).to_set
|
142
|
+
result_set.columns
|
143
|
+
.reject { |n| root_names.include?(n) || n =~ /\At\d+_r\d+\z/ }
|
144
|
+
.map { |n| Aliases::Column.new(n, n) }
|
145
|
+
end
|
146
|
+
|
138
147
|
def aliases
|
139
148
|
@aliases ||= Aliases.new join_root.each_with_index.map { |join_part, i|
|
140
149
|
columns = join_part.column_names.each_with_index.map { |column_name, j|
|
@@ -95,7 +95,6 @@ module ActiveRecord
|
|
95
95
|
end
|
96
96
|
|
97
97
|
private
|
98
|
-
|
99
98
|
# Loads all the given data into +records+ for the +association+.
|
100
99
|
def preloaders_on(association, records, scope, polymorphic_parent = false)
|
101
100
|
case association
|
@@ -185,7 +184,7 @@ module ActiveRecord
|
|
185
184
|
# and attach it to a relation. The class returned implements a `run` method
|
186
185
|
# that accepts a preloader.
|
187
186
|
def preloader_for(reflection, owners)
|
188
|
-
if owners.
|
187
|
+
if owners.all? { |o| o.association(reflection.name).loaded? }
|
189
188
|
return AlreadyLoaded
|
190
189
|
end
|
191
190
|
reflection.check_preloadable!
|
@@ -46,7 +46,6 @@ module ActiveRecord
|
|
46
46
|
end
|
47
47
|
|
48
48
|
private
|
49
|
-
|
50
49
|
def load_schema!
|
51
50
|
super
|
52
51
|
attribute_types.each do |name, type|
|
@@ -75,7 +74,6 @@ module ActiveRecord
|
|
75
74
|
end
|
76
75
|
|
77
76
|
private
|
78
|
-
|
79
77
|
def decorators_for(name, type)
|
80
78
|
matching(name, type).map(&:last)
|
81
79
|
end
|
@@ -49,7 +49,7 @@ module ActiveRecord
|
|
49
49
|
# +to+ When passed, this method will return false unless the value was
|
50
50
|
# changed to the given value
|
51
51
|
def saved_change_to_attribute?(attr_name, **options)
|
52
|
-
mutations_before_last_save.changed?(attr_name.to_s, options)
|
52
|
+
mutations_before_last_save.changed?(attr_name.to_s, **options)
|
53
53
|
end
|
54
54
|
|
55
55
|
# Returns the change to an attribute during the last save. If the
|
@@ -99,7 +99,7 @@ module ActiveRecord
|
|
99
99
|
# +to+ When passed, this method will return false unless the value will be
|
100
100
|
# changed to the given value
|
101
101
|
def will_save_change_to_attribute?(attr_name, **options)
|
102
|
-
mutations_from_database.changed?(attr_name.to_s, options)
|
102
|
+
mutations_from_database.changed?(attr_name.to_s, **options)
|
103
103
|
end
|
104
104
|
|
105
105
|
# Returns the change to an attribute that will be persisted during the
|
@@ -45,7 +45,6 @@ module ActiveRecord
|
|
45
45
|
end
|
46
46
|
|
47
47
|
private
|
48
|
-
|
49
48
|
def attribute_method?(attr_name)
|
50
49
|
attr_name == "id" || super
|
51
50
|
end
|
@@ -120,7 +119,6 @@ module ActiveRecord
|
|
120
119
|
end
|
121
120
|
|
122
121
|
private
|
123
|
-
|
124
122
|
def suppress_composite_primary_key(pk)
|
125
123
|
return pk unless pk.is_a?(Array)
|
126
124
|
|