- 30 9月, 2017 8 次提交
-
-
由 Ryuta Kamizono 提交于
Follow up of #30728.
-
由 Ryuta Kamizono 提交于
Don't use Active Support where it is not needed.
-
由 Ryuta Kamizono 提交于
Fix "warning: method redefined; discarding old test_scaffold_plural_names"
-
由 yuuji.yaginuma 提交于
This code has been changed with https://github.com/rails/rails/pull/30735/files#diff-8e5f6b33c191ad6dec07f3288345a13fL47. However, `active_support/time` is not load automatically, so if use Action Pack alone, `days` method can not use and an error occurs. In this case, I think that there is no problem by specifying a value with Integer.
-
由 yuuji.yaginuma 提交于
Since warning was issued due to duplication of test names, fix the test name to a meaningful name. This fixes following warnings: ``` railties/test/generators/named_base_test.rb:141: warning: method redefined; discarding old test_scaffold_plural_names railties/test/generators/named_base_test.rb:62: warning: previous definition of test_scaffold_plural_names was here ```
-
由 Ryuta Kamizono 提交于
-
由 Eileen M. Uchitelle 提交于
Fix formatting in ActionDispatch::SSL middleware docs
-
- 29 9月, 2017 6 次提交
-
-
由 Vipul A M 提交于
Fix Typo in Testing Guide [ci skip]
-
由 Conrad Beach 提交于
-
由 George Claghorn 提交于
-
由 Kasper Timm Hansen 提交于
-
由 Kasper Timm Hansen 提交于
Fixes for use_authenticated_cookie_encryption
-
由 Kasper Timm Hansen 提交于
See the changelog entry. Remove `secrets.secret_token` from the bug report templates, since we don't accept bug reports for Rails versions that don't support a `secret_key_base`. [ claudiob & Kasper Timm Hansen ]
-
- 28 9月, 2017 6 次提交
-
-
由 Ryuta Kamizono 提交于
Set version in activestorage/package.json in proper format.
-
由 George Claghorn 提交于
Add assert_enqueued_email_with to ActionMailer::TestHelper
-
由 Mikkel Malmberg 提交于
-
由 bogdanvlviv 提交于
`5.2.0.alpha` => `5.2.0-alpha` System versioning isn't compliant with npm.
-
由 T.J. Schuck 提交于
Before: https://monosnap.com/file/J6xewF0tYpm6dC9nSTe82ddsHAOcM5.png After: https://monosnap.com/file/0tCYicLXNqRHAEMDb81u0aLb3gH9Wf.png [ci skip]
-
由 Aaron Patterson 提交于
activerecord: Remove a redundant mutation tracker
-
- 27 9月, 2017 19 次提交
-
-
由 Aaron Patterson 提交于
Nested scaffold should be fine
-
由 Michael Coyne 提交于
Use CBC encryption is this configuration value is set to false
-
由 Javan Makhmali 提交于
Align `bin/update`'s Yarn usage with `bin/setup`
-
由 Ryuta Kamizono 提交于
[ci skip]
-
由 yalab 提交于
-
由 Ryuta Kamizono 提交于
Ensure `HashWithIndifferentAccess#transform_keys` to return `HashWithIndifferentAccess`
-
由 yuuji.yaginuma 提交于
Currently, `#transform_values`, `#select` and `#reject` return instance of `HashWithIndifferentAccess`. But `#transform_keys` returns instance of Hash. This behavior is a bit confusing. I think that `HashWithIndifferentAccess#transform_keys` should also return instance of `HashWithIndifferentAccess` as well as other methods.
-
由 Ryuta Kamizono 提交于
Since #29301, `arel_attribute` respects a custom table name.
-
由 Sean Griffin 提交于
Use `ActiveRecord::ConnectionAdapters::Quoting.quote` to quote bounds for a PostgreSQL range
-
由 Thomas Cannon 提交于
PostgreSQL 9.1+ introduced range types, and Rails added support for using this datatype in ActiveRecord. However, the serialization of `PostgreSQL::OID::Range` was incomplete, because it did not properly quote the bounds that make up the range. A clear example of this is a `tsrange`. Normally, ActiveRecord quotes Date/Time objects to include the milliseconds. However, the way `PostgreSQL::OID::Range` serialized its bounds, the milliseconds were dropped. This meant that the value was incomplete and not equal to the submitted value. An example of normal timestamps vs. a `tsrange`. Note how the bounds for the range do not include their milliseconds (they were present in the ruby Range): UPDATE "iterations" SET "updated_at" = $1, "range" = $2 WHERE "iterations"."id" = $3 [["updated_at", "2017-09-23 17:07:01.304864"], ["range", "[2017-09-23 00:00:00 UTC,2017-09-23 23:59:59 UTC]"], ["id", 1234]] `PostgreSQL::OID::Range` serialized the range by interpolating a string for the range, which works for most cases, but does not work for timestamps: def serialize(value) if value.is_a?(::Range) from = type_cast_single_for_database(value.begin) to = type_cast_single_for_database(value.end) "[#{from},#{to}#{value.exclude_end? ? ')' : ']'}" else super end end (byebug) from = type_cast_single_for_database(value.begin) 2010-01-01 13:30:00 UTC (byebug) to = type_cast_single_for_database(value.end) 2011-02-02 19:30:00 UTC (byebug) "[#{from},#{to}#{value.exclude_end? ? ')' : ']'}" "[2010-01-01 13:30:00 UTC,2011-02-02 19:30:00 UTC)" @sgrif (the original implementer for Postgres Range support) provided some feedback about where the quoting should occur: Yeah, quoting at all is definitely wrong here. I'm not sure what I was thinking in 02579b56, but what this is doing is definitely in the wrong place. It should probably just be returning a range of subtype.serialize(value.begin) and subtype.serialize(value.end), and letting the adapter handle the rest. `Postgres::OID::Range` now returns a `Range` object, and `ActiveRecord::ConnectionAdapters::PostgreSQL::Quoting` can now encode and quote a `Range`: def encode_range(range) "[#{type_cast(range.first)},#{type_cast(range.last)}#{range.exclude_end? ? ')' : ']'}" end ... encode_range(range) #=> "['2010-01-01 13:30:00.670277','2011-02-02 19:30:00.745125')" This commit includes tests to make sure the milliseconds are preserved in `tsrange` and `tstzrange` columns
-
由 Ryuta Kamizono 提交于
It is also used in `BlockTestCase`.
-
由 Ryuta Kamizono 提交于
`abstract_unit.rb` in actionview was copied from actionpack in the commit eb23754e. But some part is never used for actionview's tests.
-
由 Ryuta Kamizono 提交于
These are no longer used since 66736c8e.
-
由 Ryuta Kamizono 提交于
Change :github git source for bug report templates
-
由 Ryuta Kamizono 提交于
Because the reflection doesn't have `foreign_type` unless the association is a polymorphic association.
-
由 bogdanvlviv 提交于
:github source uses `git://` url by default, `https://` is recommended. See http://bundler.io/v1.15/guides/git.html#security We do the same in our `Gemfile` and templates.
-
由 Javan Makhmali 提交于
Add a space to RefineryCMS in Engine guide [ci skip]
-
由 Sean Griffin 提交于
I do not want to set the expectation that any enumerable object should behave this way, but this case in particular comes up frequently enough that I'm caving on this one. Fixes #30684.
-
由 Sean Griffin 提交于
PERF: Partially recover some performance when preloading.
-
- 26 9月, 2017 1 次提交
-
-
由 Matthew Draper 提交于
Fix typo in 'find_in_batches' example
-