Fix `NoMethodError: undefined method `fields' for nil:NilClass`

This reverts commit ddb7be50, reversing
changes made to ee4a1462.

Reason: It is givin us a lot of timeouts in travis because JRuby 9.0.5.0
needs to be build in every run

convert cable.coffee to cable.js

Deprecate `Module.local_constants`

Add option to error on ignored order or limit

Deprecate :controller and :action path parameters

do not set Action Cable's config when specify `--skip-action-cable` option

Delete CTRL-C message as is duplicates Puma

Publish AS::Executor and AS::Reloader APIs

ActiveRecord #second_to_last tests and finder methods

Merge pull request #22170 from samphilipd/sam/properly_deallocate_prepared_statements_outside_of_transaction

 Correctly deallocate prepared statements if we fail inside a transaction

These should allow external code to run blocks of user code to do
"work", at a similar unit size to a web request, without needing to get
intimate with ActionDipatch.

After Ruby 1.9, we can easily get the constants that have been
defined locally by `Module.constants(false)`.

Also remove duplicated `config.action_cable.url` config.

`OptimizedFileSystemResolver` (which most Rails apps use), but did not
implement the feature on the more generic `PathResolver`, which is often
used in tests etc.

Fixes #23881

Allowing :controller and :action values to be specified via the path
in config/routes.rb has been an underlying cause of a number of issues
in Rails that have resulted in security releases. In light of this it's
better that controllers and actions are explicitly whitelisted rather
than trying to blacklist or sanitize 'bad' values.

Active Job: pare down async adapter for low footprint dev

Updating the dummy app template to have rails_command instead of rake

Remove inconsistency in the Action Cable README

Use one shared worker pool for all queues with 0-#CPU workers rather
than separate pools per queue with 2-10*#CPU workers each.

When the `source_type` option is passed to a has_many through, the
resulting `Reflection` will be an instance of `PolymorphicReflection`
and not `ThroughReflection`, meaning that it will ignore the scopes that
it needs to apply from the through reflections. This adds an additional
delegation call to remedy this. I've been finding the reflection code
completely impenetrable lately, it could use some major love.

Fixes #22726

Run CI for ActionPack on JRuby

Removes potentially quadratic Regexp from ActiveRecord::LogSubscriber…

Honour joining model order in `has_many :through` associations when
eager loading

Drop Action Controller require in ActionDispatch::ExceptionWrapper

Add documentation for #13897 [skip ci]

Fix value of CSS background-color property in Ruby on Rails guide

We only reference the Action Controller error classes by name in
ActionDispatch::ExceptionWrapper, so there is no need to explicitly
require them.

It drops a tiny coupling between Action Dispatch and Action Controller,
so it makes me feel warm inside. We still have a lot of others AC
requires in the AD code base, but here, we can save it.

[ci skip]

Having more readable test descriptions in railties actions_test.rb

Changed description with tests related to rails_command.

Fix typos in Rails guide about Action View Overview