| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | | |
| | | | | |
| | | | | | |
The intention here is to make the required config copy-able from the console/logs, so add a newline at the end of the message to make that easier. (Otherwise it would be `... raise_in_transactional_callbacks = true (called from...`.)
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
follow up for #17052
|
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | | |
For now, we don't want to take "scoping" calls in to account when
calculating cache keys for relations, so just opt-out.
|
|\ \ \ \ \
| |_|/ / /
|/| | | | |
Improve Journey compliance to RFC 3986
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The scanner in Journey fails to recognize routes that use literals
from the sub-delims section of RFC 3986.
This commit enhance the compatibility of Journey with the RFC by
adding support of authorized delimiters to the scanner.
Fix #17212
|
|\ \ \ \
| | | | |
| | | | | |
Add #key? to ActiveModel::Errors
|
| | | | |
| | | | |
| | | | |
| | | | | |
Mirror Ruby's Hash#key?
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
[ci skip] Clarify `#update_all` doc about values processing
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This clarify the fact that `#update_all` doesn't type-cast passed
values and that these values are written as-is in the SQL DB.
Fix #17242
[ci skip]
|
|\ \ \ \
| | | | |
| | | | | |
Remove duplicate error message "Couldn't find..."
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This commit removes the duplication of the error message:
> Couldn't find #{@klass.name} with [#{arel.where_sql}]
introduced in #15791 by adding a private method `find_nth!` that
deals with all the method like `first!` and `second!`.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
emit an event when we instantiate AR objects so we can see how many
records were instantiated and how long it took
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
It was changed to not set the cookie in get requests at
https://github.com/rails/turbolinks/commit/62cc3db457ad9745ccda047dd43ab84fb3d89707.
Related with dcb05f26.
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Autosave callbacks shouldn't be `after_save` callbacks
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
068f092ced8483e557725542dd919ab7c516e567 registered autosave callbacks
as `after_save` callbacks. This caused the regression described in #17209.
Autosave callbacks should be registered as `after_update` and
`after_create` callbacks, just like before.
This is a partial revert of 068f092ced8483e557725542dd919ab7c516e567.
Fixes #17209.
|
| | | |
| | | |
| | | |
| | | | |
Also add documentation on it
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This reverts commit 997f1575f78bf6a2d12b8665a55807d51fe964df, reversing
changes made to f919d5f5ea75f0e56a57643e5a0472367a98625f.
See https://github.com/rails/rails/pull/17247#issuecomment-58895078
|
|\ \ \ \
| | | | |
| | | | | |
atomic_write rescue also Errno::EACCES
|
| | | | |
| | | | |
| | | | | |
atomic_write rescue also Errno::EACCES on changing file permission. It could be raised with some type of filesystem
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
remove unused tools/profile
We usually use moderns tools like benchmark-ips so we don't need this anymore
|
| | | | | | |
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
fix test_helper for mountable plugin
|
| | |/ / /
| |/| | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Add regression test for router was overwriting PATH_INFO
|
| |/ / / /
| | | | |
| | | | |
| | | | | |
[related #17233]
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Start running railties builds first, since they take the longest finish
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Replace Array#shuffle.first with Array#sample
|
| |/ / / / / |
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
Replace Enumerable#reverse.each with Enumerable#reverse_each
|
|/ / / / / |
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Upgrade to coffee-rails 4.1.0
|
|/ / / / |
|
|\ \ \ \
| |/ / /
|/| | | |
speed up fixtures by not loading all their classes
|
| | | | |
|
| | | | |
|
|\ \ \ \
| |_|/ /
|/| | | |
ActiveJob: allow empty queue names
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | | |
As discussed in #17195, Que uses blank queue names by default for
performance reasons. At the very least, ActiveJob should allow users to
choose "" as their queue name. This commit allows blank queue names, as
well as tests to make sure that nil queue names still use the default.
|
|\ \ \
| | | |
| | | | |
Invalid HTTP method will raise I18n::InvalidLocale if :en is not a valid locale.
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Request#check_method would use to_sentence(locale: :en), which breaks when
I18n.available_locales does not include :en and
I18n.enforce_available_locales is true (default).
Inlined to_sentence functionality to solve this.
|
|\ \ \
| | | |
| | | | |
[ci skip] removing ruby-i18n.org/wiki link that is not available and usi...
|
|/ / /
| | |
| | |
| | | |
“gems” instead of plugins
|
|\ \ \
| | | |
| | | | |
Don't pass retry option to Sidekiq
|
|/ / /
| | |
| | |
| | |
| | | |
* This would override whatever default is configured in Sidekiq
system-wide, and retry is true by default anyway.
|