use self instead of #read_attribute
Fix for configuring rails guide

`ActiveSupport::Logger` has no concept like auto flushing since it has no userspace buffering.
It's a remnant of `ActiveSupport::BufferedLogger` epoch which was since removed from rails.
Merge pull request #16763 from brainopia/patch-1

Fix for configuring rails guide [ci skip]
Clear schema cache before each test
Merge pull request #16704 from ankit1910/use-existing-method

use self instead of #read_attribute
Use jquery-rails master
Try with web-console master
Remove warning
Break conditional branches into separate methods

This breaks the two branches of the `if reflection.last` and `else`
to clearer see where the two methods can be refactored. Eventually
we hope to remove the need for these separated methods altogether.

Move the first branch outside the loop

This code doesn't need to be in the loop because it it always affects
the last chain. `get_bind_values` and `add_constraints` must match
in this context because `get_bind_values` is the caching of `add_constraints`

Use each_cons to remove need for `chain[i + 1]`

The `chain[i + 1]` is confusing because it's not immediately obvious
what it's trying to achieve. The use of `each_cons` makes it clear
we need to get the `next_reflection`.