rails.git - Mirror of official rails repo with custom fixes.

	Commit message (Collapse)	Author	Age	Files	Lines
...
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Restore action_cable.js UMD module support. Fixes #28366	Javan Makhmali	2017-03-11	3	-1/+38
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28185 from renchap/system-testing-screenshot-output	Eileen M. Uchitelle	2017-03-13	1	-5/+34
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Dont always display inline screenshots in system testing (#28133)
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Dont always display inline screenshots in system testing (#28133)	Renaud Chaput	2017-03-11	1	-5/+34
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	3 output types are supported: - simple: only display the screenshot path - artifact: display the screenshot in the terminal, using the artifact protocol (supported by some CI) - inline (default): display the screenshot in the terminal, inline (supported by some terminals) You can force the output type by setting the `RAILS_SYSTEM_TESTING_SCREENSHOT` environment variable
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28017 from mtsmfm/suppress-dep-warn	Matthew Draper	2017-03-13	3	-0/+6
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Suppress deprecation warning `implementing to_yaml is deprecated`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Suppress deprecation warning `implementing to_yaml is deprecated`	Fumiaki MATSUSHIMA	2017-02-15	3	-0/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28352 from alpaca-tc/fixes_renderer_with_defaults	Akira Matsuda	2017-03-13	3	-1/+17
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixes ActionController::Rendering#with_defaults
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixes ActionController::Rendering#with_defaults	alpaca-tc	2017-03-12	3	-1/+17
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`env` is undefined.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28371 from kamipo/simplify_count	Andrew White	2017-03-12	1	-7/+4
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply forward `Calculations#count` to `Enumerable#count`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply forward `Calculations#count` to `Enumerable#count`	Ryuta Kamizono	2017-03-10	1	-7/+4
\| \| \|_\|_\|_\|_\|_\|_\|/ / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Follow up of #24203. Since b644964b `ActiveRecord::Relation` includes `Enumerable` so it is enough to call `super` simply.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix weird documentation line	Jon Moss	2017-03-12	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip]
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28372 from kamipo/delegate_as_json	Andrew White	2017-03-12	3	-6/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply delegate `as_json` to `records`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Simply delegate `as_json` to `records`	Ryuta Kamizono	2017-03-10	3	-6/+2
\| \|/ / / / / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Second editing pass on "Active Model Basics" guide	Jon Moss	2017-03-12	1	-16/+18
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Lots of grammar, etc fixes. [ci skip]
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	First editing pass on "Active Model Basics" guide	Jon Moss	2017-03-12	1	-11/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Lots of grammar, etc fixes. [ci skip]
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add link to mentioned API [ci skip] (#28392)	Vipul A M	2017-03-12	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #27152 from tjai89/patch-5	Vipul A M	2017-03-12	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add link to API documentation [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add link to API documentation	Kirill Zhuravlov	2016-11-23	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add a link to http://api.rubyonrails.org/classes/ActionController.html in the beginning of an article.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28390 from bogdanvlviv/fix-typo-in-security-guide	Vipul A M	2017-03-12	1	-2/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix typo in the security guide
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix typo in the security guide	bogdanvlviv	2017-03-12	1	-2/+2
\|/ / / / / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip]
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	AS:Cache:MemoryStore doc fixes [ci skip] (#28389)	Vipul A M	2017-03-12	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #27051 from devonestes/adding-cache-clear-docs	Vipul A M	2017-03-12	1	-0/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add missing documentation for MemoryStore#clear [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add missing documentation for MemoryStore#clear [ci skip]	Devon Estes	2016-11-15	1	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We were missing some form of documentation for this method, so I've gone ahead and added some!
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28378 from y-yagi/use_correct_variable_in_reverse_merge_test	Eileen M. Uchitelle	2017-03-11	1	-8/+4
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Use correct value in `reverse_merge` test
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Use correct value in `reverse_merge` test	yuuji.yaginuma	2017-03-11	1	-8/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28380 from shime/docs-actionpack-api	Andrew White	2017-03-11	1	-4/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[docs] remove unnecessary sentence
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[docs] remove unnecessary sentence	Hrvoje Šimić	2017-03-11	1	-4/+3
\|/ / / / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip]
* / / / / / / / / / / / / / / /	Fix indentation and formatting in CHANGELOGs [ci skip]	Ryuta Kamizono	2017-03-11	2	-19/+19
\|/ / / / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add @takkanm credits in the changelog [ci skip]	Rafael Mendonça França	2017-03-10	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28355 from Edouard-chin/reversemerge-for-strongparameters	Rafael Mendonça França	2017-03-10	3	-0/+48
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Added `reverse_merge`/`reverse_merge!` to AC::Parameters: [Rafael Mendonça França + Mitsutaka Mimura]
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Added `reverse_merge`/`reverse_merge!` to AC::Parameters:	Edouard CHIN	2017-03-09	3	-0/+48
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- This PR adds the `reverse_merge` and `reverse_merge!` method to `ActionController::Parameters` - Fixes #28353
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28373 from godfat/fix-cache-middleware-with-throw	Rafael França	2017-03-10	2	-4/+15
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Make sure local cache cleared even it's throwing:
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Make sure local cache cleared even it's throwing:	Lin Jen-Shin	2017-03-10	2	-4/+15
\| \| \|_\|_\|/ / / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We (GitLab) hit into an issue that somewhere in the middleware chain was throwing `:warden`, which was caught in the wrapping middleware, but `LocalCache::Middleware` was not aware of it. It should look like: ``` ruby result = catch(:warden) do @app.call(env) end ``` Source: https://github.com/hassox/warden/blob/090ed153dbd2f5bf4a1ca672b3018877e21223a4/lib/warden/manager.rb#L35-L37 Using `ensure` could make sure that we would always do the cleanup, and better yet, avoid `rescue Exception` which we all should know that could cause some issues which could be very hard to debug. Please check the discussion thread for more context: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1402#note_25128108
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28370 from kamipo/remove_conditions_params_from_delete_all	Rafael França	2017-03-10	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove conditions parameter from `NullRelation#delete_all`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove conditions parameter from `NullRelation#delete_all`	Ryuta Kamizono	2017-03-10	1	-1/+1
\| \|/ / / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Follow up of e7381d289e4f8751dcec9553dcb4d32153bd922b.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28375 from baerjam/improve-testing-documentaion	Vipul A M	2017-03-11	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Improve readability of testing guide [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Improve readability of testing guide [ci skip]	James Baer	2017-03-10	1	-1/+1
\|/ / / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Small change to improve the readability in section 2.3 of the testing guide.
* \| \| \| \| \| \| \| / / / / / / /	Add missing credit [ci skip]	Ryuta Kamizono	2017-03-10	1	-0/+2
\| \|_\|_\|_\|_\|_\|_\|/ / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28182 from y-yagi/show_correct_rails_commands	Kasper Timm Hansen	2017-03-09	8	-18/+47
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Show correct commands in help
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Show correct commands in help	yuuji.yaginuma	2017-03-03	8	-18/+47
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Currently rails' help shows only namespace. However, the secrets command needs to specify command. Therefore, I fixed the command to display in help.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Document read_encrypted_secrets config.	Kasper Timm Hansen	2017-03-09	1	-0/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Mostly just that it's there. Closes #28193.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28351 from kamipo/deprecate_schema_migrations_table_name	Rafael Mendonça França	2017-03-09	3	-0/+13
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Deprecate `Migrator.schema_migrations_table_name`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Deprecate `Migrator.schema_migrations_table_name`	Ryuta Kamizono	2017-03-09	3	-0/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Since 67fba0cf `SchemaMigration` model was extracted. Use `SchemaMigration.table_name` instead.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28354 from kamipo/fix_select_with_block_and_dirty_target	Eileen M. Uchitelle	2017-03-09	3	-2/+14
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix select with block doesn't return newly built records in has_many association
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix select with block doesn't return newly built records in has_many association	Ryuta Kamizono	2017-03-09	3	-2/+14
\| \| \|_\|/ / / / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `select` in `QueryMethods` is also an enumerable method. Enumerable methods with block should delegate to `records` on `CollectionProxy`, not `scope`. Fixes #28348.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge branch 'refactor-system-test-driver'	eileencodes	2017-03-09	9	-91/+64
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|/ / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Call system test driver per-instance rather than globally	eileencodes	2017-03-09	7	-42/+36
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Previously the system test subclasses would call `driven_by` when the app booted and not again when the test was initialized which resulted in the driver from whichever class was called last to be used in tests. In rails/rails#28144 the `driven_by` method was changed to run `use` on setup and `reset` on teardown. While this was a viable fix this really pointed to the problem that system test `driven_by` was a global setting, rather than a per-class setting. To alieviate this problem calling the driver should be done on an instance level, rather than on the global level. I added an `initialize` method to `SystemTestCase` which will call `use` on the superclass driver. Running the server has been moved to `start_application` so that it only needs to be called once on boot and no options from `driven_by` were being passed to it. This required a largish rewrite of the tests. Each test needs to utilize the subclass so that it can properly test the drivers. `ActionDispatch::SystemTestCase` shouldn't be called directly anymore.
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Refactor system test driver/browser	eileencodes	2017-03-09	6	-56/+35
\|/ / / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Since using a browser is only for selenium it doesn't really make sense to have a separate class for handling it there. This brings a lot of the if/else out of the main SystemTestCase class and into the Driver class so we can abstract away all that extra work.
* / / / / / / / / / / / / / /	force UTF-8 as external encoding in guides generation [ci skip]	Xavier Noria	2017-03-09	1	-3/+16
\|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	See the rationale in the comment found in the patch.
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28340 from k3rni/document-am-default-url-options	Jon Moss	2017-03-08	1	-0/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Document using `default_url_options` in an ActionMailer class.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Document using `default_url_options` in an ActionMailer class.	Krzysztof Zych	2017-03-08	1	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \|