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

	Commit message (Collapse)	Author	Age	Files	Lines
...
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove unnecessary `doc` directory deletion	yuuji.yaginuma	2017-08-01	2	-2/+0
\| \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Since 553b695, `doc` directory is not created in application.
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Update working_with_javascript_in_rails.md	Guillermo Iguaran	2017-08-01	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Path parameters should default to UTF8	eileencodes	2017-08-01	5	-14/+44
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This commit changes the behavior such the path_params now default to UTF8 just like regular parameters. This also changes the behavior such that if a path parameter contains invalid UTF8 it returns a 400 bad request. Previously the behavior was to encode the path params as binary but that's not the same as query params. So this commit makes path params behave the same as query params. It's important to test with a path that's encoded as binary because that's how paths are encoded from the socket. The test that was altered was changed to make the behavior for bad encoding the same as query params. We want to treat path params the same as query params. The params in the test are invalid UTF8 so they should return a bad request. Fixes #29669 Eileen M. Uchitelle, Aaron Patterson, & Tsukuru Tanimichi
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30029 from yhirano55/fix_rails_command_comments	Sean Griffin	2017-08-01	1	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Fix rails_command comments
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Fix rails_command comments	Yoshiyuki Hirano	2017-08-01	1	-3/+3
\| \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ / \| \|/\| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30021 from roberts1000/fix-rubocop-issues	Rafael França	2017-08-01	2	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix rubocop issues
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix rubocop style issues in yarn and spring.rb templates"	roberts1000	2017-07-31	2	-3/+3
\| \| \|/ / / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30024 from kamipo/remove_redundant_unless_branch	Sean Griffin	2017-07-31	1	-2/+0
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove redundant `unless current_adapter?(:OracleAdapter)`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove redundant `unless current_adapter?(:OracleAdapter)`	Ryuta Kamizono	2017-07-28	1	-2/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Talk about bytes not characters	Rafael Mendonça França	2017-07-31	1	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Closes #30012
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add backticks around method [ci skip]	Ryuta Kamizono	2017-08-01	1	-1/+1
\| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	And make reference to `Relation`.
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30017 from bslobodin/fix-typo-in-assert-changes	Rafael França	2017-07-31	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Fix typo in assert_changes error message
\| * \| \| \| \| \| \| \| \| \| \| \| \|	fix typo in assert_changes error message	Boris Slobodin	2017-07-31	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30016 from ↵	Guillermo Iguaran	2017-07-31	5	-0/+25
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	albertoalmagro/set-ruby-version-in-gemfile-and-ruby-version-by-default Set Ruby version in Gemfile and .ruby-version by default
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Set Ruby version in Gemfile and .ruby-version by default	Alberto Almagro	2017-07-31	5	-0/+25
\|/ / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29950 from MaxLap/avoid_or_clause_duplicates	Sean Griffin	2017-07-31	3	-6/+66
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	Avoid duplicate clauses when using #or
\| * \| \| \| \| \| \| \| \| \| \| \|	Edits following the reviews	Maxime Lapointe	2017-07-28	2	-27/+43
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Avoid duplicate clauses when using #or	Maxime Lapointe	2017-07-25	3	-9/+53
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Condenses the clauses that are common to both sides of the OR and put them outside, before the OR This fix the current behavior where the number of conditions is exponential based on the number of times #or is used.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30001 from y-yagi/fix_test_directory	Kasper Timm Hansen	2017-07-30	3	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix test directory to correct path
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Fix test directory to correct path	yuuji.yaginuma	2017-07-30	3	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Together, fix to the following lint violation. ``` rails/actionview/test/ujs/public/test/data-confirm.js 303:11 error Strings must use singlequote quotes rails/actionview/test/ujs/public/test/data-remote.js 414:32 error Extra semicolon semi ✖ 2 problems (2 errors, 0 warnings) ```
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29951 from georgeclaghorn/action-view-test-flow	Kasper Timm Hansen	2017-07-30	2	-0/+10
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix testing helpers that use Action View's capturing helpers (e.g. content_for)
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix testing helpers that use Action View's capturing helpers (e.g. content_for)	George Claghorn	2017-07-25	2	-0/+10
\| \| \|/ / / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix github user output.	Kasper Timm Hansen	2017-07-30	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ Orhan Toy & Kasper Timm Hansen ]
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29963 from y-yagi/fix_ruby_warnings	Kasper Timm Hansen	2017-07-30	1	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix `warning: method redefined;`
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix `warning: method redefined;`	yuuji.yaginuma	2017-07-28	1	-3/+3
\| \|/ / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This fixes the following warning: ``` /tmp/d20170727-7039-kmdtb1/app/app/models/user.rb:5: warning: method redefined; discarding old model_name rails/activemodel/lib/active_model/naming.rb:222: warning: previous definition of model_name was here ```
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30002 from y-yagi/change_http_postgresql_links_to_https	Vipul A M	2017-07-30	8	-32/+32
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Change http postgresql.org links to https [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Change http postgresql.org links to https [ci skip]	yuuji.yaginuma	2017-07-30	8	-32/+32
\|/ / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It seems that it accepts only HTTPS connections. Ref: https://github.com/postgres/postgres/commit/7f77cbd996855a06fb742ea11adbe55c42b48fe2
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29933 from kirs/frozen-actionpack	Matthew Draper	2017-07-29	281	-3/+563
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	Use frozen string literal in actionpack/
\| * \| \| \| \| \| \| \| \| \| \| \|	Use frozen string literal in actionpack/	Kir Shatrov	2017-07-29	281	-3/+563
\|/ / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29946 from kamipo/passing_arel_to_where_is_boundable	Sean Griffin	2017-07-28	1	-4/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Building `where_clause` in `UniquenessValidator` is no longer needed
\| * \| \| \| \| \| \| \| \| \| \| \|	Building `where_clause` in `UniquenessValidator` is no longer needed	Ryuta Kamizono	2017-07-26	1	-4/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Building `where_clause` manually was introduced at #26073 to include both `comparison` and `binds` in `where_clause`. Since 213796f, `comparison` includes `binds`, so it is enough to use `where` simply.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29979 from kamipo/use_build_bind_attribute	Sean Griffin	2017-07-28	4	-8/+7
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Use `predicate_builder.build_bind_attribute` wherever possible
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Use `predicate_builder.build_bind_attribute` wherever possible	Ryuta Kamizono	2017-07-28	4	-8/+7
\| \| \|_\|/ / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	For less duplicated code.
* \| \| \| \| \| \| \| \| \| \| \| \|	Add missing support for modulo operations on durations	Sayan Chakraborty	2017-07-28	3	-14/+131
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Rails 5.1 introduce an `ActiveSupport::Duration::Scalar` class as a wrapper around a numeric value as a way of ensuring a duration was the outcome of an expression. However the implementation was missing support for modulo operations. This commit adds support for those operations and should result in a duration being returned from expressions involving them. Fixes #29603 and #29743.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29982 from ydakuka/patch-1	Robin Dupret	2017-07-28	1	-3/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] update routing guide
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Squashed commit of the following:	ydakuka	2017-07-28	1	-3/+3
\|/ / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	commit 4a3d295f3011e771cddead80de7497ca15d15c13 Author: Yauheni Dakuka <yauheni.dakuka@gmail.com> Date: Fri Jul 28 14:31:35 2017 +0300 Update routing.md commit 620a4ce47288e3ef6504290c78f931214968e7e3 Author: Yauheni Dakuka <yauheni.dakuka@gmail.com> Date: Fri Jul 28 14:19:29 2017 +0300 [ci skip] update routing guide
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29980 from ydakuka/patch-1	Robin Dupret	2017-07-28	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] update routing guide
\| * \| \| \| \| \| \| \| \| \| \| \|	[ci skip] update routing guide	Yauheni Dakuka	2017-07-28	1	-1/+1
\|/ / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \|	Eager load the association classes on boot	Rafael Mendonça França	2017-07-27	1	-7/+7
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We were expecting those classes to be loaded when an association is defined but they are not. If you add a debugger on the first line of any of those classes and try to define the corresponding association you will see that the classes are not loaded. Fixes #26273
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29971 from rails/fix-duration-division	Andrew White	2017-07-27	3	-15/+28
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix division where a duration is the denominator
\| * \| \| \| \| \| \| \| \| \| \| \|	Fix division where a duration is the denominator	Andrew White	2017-07-27	3	-15/+28
\| \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	PR #29163 introduced a change in behavior when a duration was the denominator in a calculation - this was incorrect as dividing by a duration should always return a `Numeric`. The behavior of previous versions of Rails has been restored. Fixes #29592.
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #28695 from koic/integer_type_has_precision_option_in_oracle	Akira Matsuda	2017-07-27	1	-1/+5
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|	Fix an AR test of schema dump when using Oracle
\| * \| \| \| \| \| \| \| \| \| \|	Fix an AR test of schema dump when using Oracle	Koichi ITO	2017-04-07	1	-1/+5
\| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29959 from kaanklky/master	Rafael França	2017-07-26	1	-0/+4
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix search input's type & placeholder conflict in Routing Error page
\| * \| \| \| \| \| \| \| \| \| \| \|	Fix search input's type & placeholder conflict in Routing Error page	Kaan Kölköy	2017-07-26	1	-0/+4
\| \| \|_\|/ / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \|
* / \| \| \| \| \| \| \| \| \| \|	Use `_relation_for_itself` wherever possible	Sean Griffin	2017-07-26	1	-3/+2
\|/ / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Anywhere that we're doing `self.class.unscoped.where(primary_key => id)` is somewhere that someone may want to extend. Even if this method isn't public API yet, this will make it easier for us to eventually expose an API around this. Plus, duplicated code makes me sad.
* \| \| \| \| \| \| \| \| \| \|	Use exact Rails version when verifying.	Kasper Timm Hansen	2017-07-25	1	-1/+6
\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Require arel master in the benchmark template	Sean Griffin	2017-07-25	1	-0/+1
\| \|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Pointing at Rails master generally requires pointing at Arel master
* \| \| \| \| \| \| \| \| \|	Merge pull request #29941 from kamipo/remove_single_element_array_preprocess	Sean Griffin	2017-07-25	2	-2/+2
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove single element array preprocess
\| * \| \| \| \| \| \| \| \| \|	Remove single element array preprocess	Ryuta Kamizono	2017-07-26	2	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Since 213796f, array predicate handler supports making binds, so the preprocess is no longer needed.