Remove conditional, since results are always an instance of `ActiveRecord::Result`
Rails.logger should have level specified by config.log_level. 

Fix bug when log level of Rails.logger (which was set via config.logger) does not match the    config.log_level.
fix tests
update guide to reflect default HMAC SHA1 in MessageVerifier used in SignedCookieStore [ci skip]
remove language about configuring digest method [ci skip]
Minor optimization and code cleanup in query_methods.

- Use symbols rather than strings where possible to avoid extra object construction
- Use destructive methods where possible to avoid extra object construction
- Use array union rather than concat followed by uniq
- Use shorthand block syntax where possible
- Use consistent multiline block styles, method names, method parenteses style, and spacing
use flat_map
Updated the guide to say that the localisation file for validation messages is in activemodel, not activerecord
Avoid calling define_method with non-english chars in InflectorTest

When we call define_method with non-english chars like ¿por qué? it
errors out on JRuby as of 1.7.4 & would leave out the following error

invalid byte sequence in US-ASCII

To work around this issue, I have switched to define_test method call
define method with fixed string & the index of the hash. the index was
added because otherwise, ruby will raise method redefined warning.

As far as I can see there are no side-effect of this change for
other implementations. For readbility I have added a message to
asssert_equal informing for which word/phase the test has passed.

Before this Change:
JRuby:
Tests terminated suddenly with an error. no reported of Failues
or errors
MRI:
All Green.

After this Change,
JRuby:
the `ActiveSupport` TestsSuite gracefully fails with report at the end which test failed & why.
MRI:
All Green(no change)
Using URI.parser.unescape

Fixes warning

warning: URI.unescape is obsolete
Merge pull request #11735 from arunagw/av-warning-removed-master

Using URI.parser.unescape
Add "extname" option to javascript_include_tag

ActionView::Helpers.asset_path is where the logic for
javascript_include_tag resides.  It takes an extname option for
specifying the extension or false to not append it.  This exposes that
option to javascript_include_tag.

Without the option files that didn't end with ".js" would get the
extension appended to them.  This broke JST templates and other file
types that should be interpreted as JavaScript but who's file extension
isn't ".js"
[Fixes #11734]Correct Association Basics Guide on has_and_belongs_to_many[ci skip]

Done the following changes:
 1. Replace uniq call in example with autosave as uniq is not supported
 on has_and_belongs_to_many
 2. Add :readonly option to list of supported call as per
 [EdgeApi](http://edgeapi.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#method-i-has_and_belongs_to_many)
Merge pull request #11736 from gaurish/avoid-multibyte-method-define

[JRuby] Avoid calling define_method with non-english chars in InflectorTest
Changing the def self.table_name to self.table_name in the column_name.rb
fix for postgresql failing test in local
used flat_map instead of map.flatten in /guides/rails_guides/helpers.rb
Merge pull request #11730 from neerajdotname/use_flat_map

use flat_map
Merge pull request #11741 from Karunakar/failing_test_in_local

fix for postgresql failing test in local
Merge pull request #11707 from nathanstitt/extname_option

javascript_include_tag "extname" option for use by asset_path