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

	Commit message (Collapse)	Author	Age	Files	Lines
...
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26880 from TempestMia/model_schema_doc_strings	Rafael Mendonça França	2016-10-24	1	-44/+103
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix model schema doc strings
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Fix model schema doc strings	Mia Zbikowski	2016-10-24	1	-44/+103
\|/ / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26878 from Erol/fix-guide-typo	Guillermo Iguaran	2016-10-24	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix typo in Upgrading Ruby on Rails Guide [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Fix typo in Upgrading Ruby on Rails Guide [ci skip]	Erol Fornoles	2016-10-24	1	-1/+1
\|/ / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26876 from y-yagi/update_description_of_etag	प्रथमेश Sonpatki	2016-10-24	1	-1/+4
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	update description of Etag [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \|	update description of Etag [ci skip]	yuuji.yaginuma	2016-10-24	1	-1/+4
\|/ / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The document is written with "only generate weak", but it can also be used to strong etag. Also, add missing entory for #24387
* \| \| \| \| \| \| \| \| \| \| \|	Merge release note entry for Date/Time/DateTime extensions	Carlos Antonio da Silva	2016-10-23	1	-4/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The first release note entry mentioned #on_weekday?, but that was only added by the second PR mentioned there. To simplify reading the notes just merge the two, since they are related methods. [ci skip]
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26871 from maclover7/jm-fix-26867	Jon Moss	2016-10-23	1	-1/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Small grammar fix for #26867
\| * \| \| \| \| \| \| \| \| \| \| \|	Small grammar fix for #26867	Jon Moss	2016-10-23	1	-1/+2
\|/ / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip]
* \| / / / / / / / / / /	HSTS subdomains is now true, so let's fix documentation [ci skip] (#26870)	प्रथमेश Sonpatki	2016-10-23	1	-1/+1
\| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- Followup of fda5afeb
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #26867 from bogdanvlviv/opt_loc_docs	Vipul A M	2016-10-23	1	-0/+1
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add info about updating locking column value
\| * \| \| \| \| \| \| \| \| \| \|	Add info about updating locking column value	bogdanvlviv	2016-10-23	1	-0/+1
\|/ / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Follow #26050
* \| \| \| \| \| \| \| \| \| \|	Revert "Made ActiveRecord consistently use ActiveRecord::Type (not"	Sean Griffin	2016-10-23	8	-23/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This reverts commit 671eb742eec77b5c8281ac2a2e3976ef32a6e424. This is not a change we would like moving forward.
* \| \| \| \| \| \| \| \| \| \|	Update mysql2 version in Gemfile.lock	Guillermo Iguaran	2016-10-22	1	-4/+4
\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Update mysql adapter error message when disconnected	Guillermo Iguaran	2016-10-22	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #26863 from claudiob/revert-26826	Rafael França	2016-10-22	1	-1/+6
\|\ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|	Revert #26826 and add documentation
\| * \| \| \| \| \| \| \| \| \|	Revert #26826 and add documentation	claudiob	2016-10-21	1	-1/+6
\|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This reverts commit a01cf703 as explained in the comment to #26826: Realized that this PR caused the following warning in Travis CI: ``` /home/travis/build/rails/rails/activesupport/lib/active_support/dependencies.rb:293: warning: loading in progress, circular require considered harmful - /home/travis/build/rails/rails/activesupport/lib/active_support/core_ext/hash/indifferent_access.rb ``` Indeed, `active_support/core_ext/hash/indifferent_access.rb` needs to require `active_support/hash_with_indifferent_access.rb` in order to access the class `ActiveSupport::HashWithIndifferentAccess`. The other way around, though, is not _strictly_ required, unless someone tries (like I did in the [gist above](https://gist.github.com/claudiob/43cc7fe77ff95951538af2825a71e5ec)) to use `ActiveSupport::HashWithIndifferentAccess` by only requiring `active_support/hash_with_indifferent_access.rb` without first requiring `active_support/core_ext/hash/indifferent_access.rb`. I think the solution to this is to revert this PR and instead change the documentation to explicitly state that developers should not require 'active_support/hash_with_indifferent_access' if all they want is to use `ActiveSupport::HashWithIndifferentAccess` – instead they should require `active_support/core_ext/hash/indifferent_access.rb`.
* \| \| \| \| \| \| \| \| \|	Merge pull request #26790 from iainbeeston/type-tests-in-their-own-files	Rafael França	2016-10-22	13	-168/+266
\|\ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Refactored ActiveModel::Type tests into their own files
\| * \| \| \| \| \| \| \| \| \|	Refactored ActiveModel::Type tests into their own files	Iain Beeston	2016-10-15	13	-168/+266
\| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #26810 from maclover7/jm-fix-26802	Rafael Mendonça França	2016-10-22	2	-0/+38
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Convert ActionController::Parameters to a hash in button_to
\| * \| \| \| \| \| \| \| \| \| \|	Convert ActionController::Parameters to a hash in button_to	Jon Moss	2016-10-17	2	-0/+22
\| \| \|_\|_\|_\|_\|_\|_\|_\|/ / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Before, an error would be raised saying that the method `to_param` was undefined on the instance of `ActionController::Parameters`. Now, we are checking to see if the `params` object being passed to `button_to` responds to the `permitted?` method, and if so, we will call `to_h` on it. If it does not respond to `permitted?`, then the `params` will remain unchanged. [Jon Moss, Rafael Mendonça França]
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #26416 from prathamesh-sonpatki/fix-select-example	Rafael França	2016-10-22	4	-7/+22
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix regression caused due to removal of select method from CollectionAssociation
\| * \| \| \| \| \| \| \| \| \| \|	Fix regression caused due to removal of select method from CollectionAssociation	Prathamesh Sonpatki	2016-10-22	4	-7/+22
\|/ / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	- CollectionAssociation#select was removed in https://github.com/rails/rails/pull/25989 in favor of QueryMethods#select but it caused a regression when passing arguments to select and a block. - This used to work earlier in Rails 4.2 and Rails 5. See gist https://gist.github.com/prathamesh-sonpatki/a7df922273473a77dfbc742a4be4b618. - This commit restores the behavior of Rails 4.2 and Rails 5.0.0 to allow passing arguments and block at the same time but also deprecates it. - Because, these arguments do not have any effect on the output of select when select is used with a block. - Updated documentation to remove the example passing arguments and block at the same time to `CollectionProxy#select`.
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #26843 from denisovlev/strptime_timestamps	Rafael França	2016-10-22	3	-11/+40
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix `ActiveSupport::TimeZone#strptime` cannot parse timestamps (%Q, %s)
\| * \| \| \| \| \| \| \| \| \| \|	Fix `ActiveSupport::TimeZone#strptime` cannot parse timestamps (%Q, %s)	denisovlev	2016-10-21	3	-11/+40
\| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26826 from claudiob/add-require	Claudio B	2016-10-21	1	-0/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add missing require in active_support/hash_with_indifferent_access.rb
\| * \| \| \| \| \| \| \| \| \| \| \|	Add missing require	claudiob	2016-10-19	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26862 from headius/lazy_load_blade_for_tests	Guillermo Iguaran	2016-10-21	1	-1/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Lazy-load blade for actioncable tests; no blade on JRuby.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Lazy-load blade for actioncable tests; no blade on JRuby.	Charles Oliver Nutter	2016-10-21	1	-1/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26860 from headius/soft_ordering_in_yaml_dump	Kasper Timm Hansen	2016-10-22	1	-6/+4
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Soften this test since YAML.dump may produce keys in other orders.
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Soften this test since YAML.dump may produce keys in other orders.	Charles Oliver Nutter	2016-10-21	1	-6/+4
\| \|/ / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Use `on_load` to trigger commandline processing code	Aaron Patterson	2016-10-21	3	-5/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We need to use on_load so that plugins will get the same functionality
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Prevent the test framework from being loaded in production mode	Aaron Patterson	2016-10-21	3	-6/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The test framework should not be autoloaded in production mode. Before this commit, the testing railtie would extend AS::TestCase. This caused AS::TestCase to be preloaded regardless of the environment in which we were running. This commit just moves the code that adds line filtering support in to the test command where we actually execute the test runner. That allows us to maintain the line runner feature but only load the minimal amount of code we need.
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26859 from headius/mask_use_of_rubyvm_on_jruby	Aaron Patterson	2016-10-21	1	-0/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Skip test that depends on RubyVM when it is not available (JRuby).
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Skip test that depends on RubyVM when it is not available (JRuby).	Charles Oliver Nutter	2016-10-21	1	-0/+2
\|/ / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \|	Revert "Undefine assings in IntegrationTest"	Rafael Mendonça França	2016-10-21	2	-10/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This reverts commit 5dde413e1d14c42eb87071db20d075a7b962cb01. Reason: The gem defines it so we don't need to remove
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26819 from lastgabs/add-examples-active-job-test-helper	Rafael França	2016-10-21	1	-0/+24
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add examples of queue_adapter and perform_enqueued jobs to API Docs.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Add examples of queue_adapter and perform_enqueued jobs to API Docs.	Gabi Stefanini	2016-10-21	1	-0/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26050 from bogdanvlviv/optimistic_locking	Aaron Patterson	2016-10-21	5	-20/+157
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Fixed: Optimistic locking does not work correctly
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Added ability update locking_column value	bogdanvlviv	2016-10-21	4	-7/+90
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Fixed: Optimistic locking does not work well with null in the database	bogdanvlviv	2016-10-21	4	-15/+69
\|/ / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26855 from headius/explicit_order_callback_args	Aaron Patterson	2016-10-21	1	-2/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Additional fix for argument-splat ordering differences.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Additional fix for argument-splat ordering differences.	Charles Oliver Nutter	2016-10-21	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	See #26854
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26854 from headius/explicit_order_callback_args	Aaron Patterson	2016-10-21	1	-4/+4
\|\\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Explicitly unpack the expanded args to avoid execution order diff.
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Explicitly unpack the expanded args to avoid execution order diff.	Charles Oliver Nutter	2016-10-21	1	-4/+4
\| \| \|/ / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	In https://bugs.ruby-lang.org/issues/12860 I argue that MRI's execution order here is incorrect. The splatting of the 'c' args should happen before the shift, but it happens after. On JRuby, it behaves the way you would expect, leading to the 'c' args splat still containing the block and producing an error like "cannot convert proc to symbol" when the send attempts to coerce it. This patch makes the unpacking order explicit with a multi-assign, which behaves properly on all implementations I tested.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26853 from zegomesjf/feature/skip_active_record	Guillermo Iguaran	2016-10-21	2	-0/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	don't create db directory when skip active_record
\| * \| \| \| \| \| \| \| \| \| \| \|	don't create db directory when skip active_record	José Gomes Júnior	2016-10-21	2	-0/+2
\|/ / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #26839 from renuo/fix-missing-nsec-transfer	Andrew White	2016-10-21	3	-2/+10
\|\ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|/ / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \|	Fix copy_time_to: Copy nsec instead of usec
\| * \| \| \| \| \| \| \| \| \| \|	Fix copy_time_to: Copy nsec instead of usec	Josua Schmid	2016-10-20	3	-2/+10
\| \|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`copy_time_to` is a helper function for date and time calculations. It's being used by `prev_week`, `next_week` and `prev_weekday` to keep the time fraction when jumping around between days. Previously the nanoseconds part was lost during the operation. This lead to problems in practice if you were using the `end_of_day` calculation. Resulting in the time fraction of `end_of_day` not being the same as next week's `end_of_day`. With this fix `copy_time_to` doesn't forget the `nsec` digits.
* \| \| \| \| \| \| \| \| \| \|	doc, hide non-public methods form the api docs. [ci skip]	Yves Senn	2016-10-20	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This is a follow up to #25681, specifically this comment: https://github.com/rails/rails/pull/25681#issuecomment-238294002 The way the thread local variable is stored is an implementation detail and subject to change. It makes no sense to only generate a reader or writer as you'd have to know where to read from or where it writes to.