This require makes the dependency even more clear.
In particular we are eager loading the session
store but that does not work if AR is used
outside Rails, this patch is preliminary work
in fixing #7160.

This reverts commit 14fc8b34.

Reason: we need to discuss a better path from this removal.

Conflicts:
	activerecord/lib/active_record/reflection.rb
	activerecord/test/cases/base_test.rb
	activerecord/test/models/developer.rb

to address ORA-01400 errors with Oracle enhanced adapter.

The original commit 3c0bf043 requires :json_data_empty attribute
has empty string OR null, then setting `:default => ""` is enough.

It has been moved to active_record_deprecated_finders.

Use #to_a instead.

It doesn't serve much purpose now that ActiveRecord::Base.all returns a
Relation.

The code is moved to active_record_deprecated_finders.

Previously it returned an Array.

If you want an array, call e.g. `Post.to_a` rather than `Post.all`. This
is more explicit.

In most cases this should not break existing code, since
Relations use method_missing to delegate unknown methods to #to_a
anyway.

`:rails_env` tasks is not needed in all the tasks that depends of
`load_config`, only in the tasks that uses `Rails.env`.

Since `:rails_env` task set the `Rails.env` to be "development" if it is
not set we don't need the `||` statements too

Fix #7175.

Conflicts:
	activerecord/lib/active_record/railties/databases.rake

to address ORA-01400 errors with Oracle enhanced adapter.

Issue #4856 had been fixed and tested with
the attribute `:null => false, :default => ""`.
Now `:null => false` attribute is not necessary to test this issue.

Moved logic from class_of_active_record_descendant(class) to the
base_class method. This method was confusing because it required
an argument, but that argument was 'self'.

Moved base_class tests to inheritance_test.rb and added some test
coverage for some untested cases.

Fixed markdown code indenting

Related to #7164

Looks like the last one; thanks!

to make store works all database adapters.

Closes #1190

Issue #6115 has been fixed and tested with
the attribute `:null => false, :default => ""`
However `:null => false` attribute is not necessary to test this issue,
which causes many ORA-01400 errors with Oracle enhanced adapter.

This allows us to mark the parent object as invalid if all associated objects
in a presence validated association are marked for destruction.

See: https://github.com/rails/rails/issues/6812

interpolation is no longer a thing separate from "normal" assoc
conditions.

now everything is converted to the new style, this is not needed

This code is broken (it should say association_scope.uniq_value rather
than options[:uniq]) but the tests still pass.

I think it is designed to uniq-ify associations using finder_sql.
However, I am about to remove that anyway.

This fixes active_record_deprecated_finders.

Lets stick with a symbol in join table name, it was removed in
993e1643 and broke the build.

Commit 3dbedd28 added NOT NULL constraints to timestamps.
Commit fcef7289 started to revert this, but was incomplete.
With this commit, 3dbedd28 should be fully reverted and
timestamps will no longer default to NOT NULL.