| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
Fix `bigint?` for Enum columns in MySQL
|
| |
| |
| |
| | |
Follow up to #22896.
|
|\ \
| | |
| | | |
Make `SchemaMigration.drop_table` to one SQL
|
| | |
| | |
| | |
| | |
| | | |
`SchemaMigration.drop_table` is only used in tests.
Simply we can use `drop_table if_exists: true`.
|
|\ \ \
| | | |
| | | | |
There is no need to define test if a connection does not support prim…
|
| | |/
| |/| |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
kamipo/remove_unused_lost_connection_error_messages
Remove unused `LOST_CONNECTION_ERROR_MESSAGES`
|
| | |/
| |/|
| | |
| | |
| | | |
`LOST_CONNECTION_ERROR_MESSAGES` was added by f384582.
But currently unused from anywhere.
|
|\ \ \
| | | |
| | | | |
Explicitly define `columns` method as an interface
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
`ActiveRecord::ConnectionAdapters::SchemaStatements#columns` is defined
here as an interface method here. So changes to raise `NotImplementedError`
same as `tables`, `views` ...etc.
|
|\ \ \
| | | |
| | | | |
remove unused variable from render test
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This removes the following warning.
```
rails/actionpack/test/controller/render_test.rb:278: warning: assigned but unused variable - response
```
|
|\ \ \
| | | |
| | | | |
remove unused require
|
| |/ /
| | |
| | |
| | | |
`with_indifferent_access` had been used in `assigns` method, but has been removed in ca83436.
|
| | | |
|
|\ \ \
| | | |
| | | | |
remove unnecessary gsub for `action_cable_meta_tag`
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | | |
If the specified `skip_action_cable` option, so as not to output the
`action_cable_meta_tag` in template, gsub is unnecessary.
ref: https://github.com/rails/rails/blob/master/railties/lib/rails/generators/rails/app/templates/app/views/layouts/application.html.erb.tt#L6..L8
|
|\ \ \
| |_|/
|/| | |
Refactor `OID::Money.precision`
|
| |/ |
|
|/
|
|
|
|
|
|
|
| |
There are some cases when the `body` in `response_body=` can be set to
nil. One of those cases is in `actionpack-action_caching` which I found
while upgrading it for Rails 5.
It's not possible to run `body.each` on a `nil` body so we have to
return after we run `response.reset_body!`.
|
|\
| |
| | |
Add an after_bundle callback in Rails plugin templates
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Add additional documentation on Headers#[] [ci skip]
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Issue #16519 covers confusion potentially caused by how HTTP
headers, that contain underscores in their names, are retrieved
through `ActionDispatch::Http::Headers#[]`.
This confusion has its origin in how a CGI maps HTTP header names
to variable names. Even though underscores in header names
are rarely encountered, they are valid according to RFC822 [1].
Nonetheless CGI like variable names, as requested by the Rack
specfication, will only contain underscores and therefore the
original header name cannot be recovered after the Rack server passed
on the environemnt hash. Please, see also the disscussion on
StackOverflow [2], which also links to an explaination in the
nginx documentation [3].
[1] http://www.ietf.org/rfc/rfc822.txt
[2] http://stackoverflow.com/questions/22856136/why-underscores-are-forbidden-in-http-header-names
[3] https://www.nginx.com/resources/wiki/start/topics/tutorials/config_pitfalls/#missing-disappearing-http-headers
|
|/ /
| |
| |
| | |
The detach used by close! seems to be making EM very sad on Travis.
|
| |
| |
| |
| |
| | |
It's not strictly necessary, and maybe this will help with the current
test failure.
|
| | |
|
|\ \
| | |
| | | |
remove require logging
|
|/ / |
|
|\ \
| | |
| | | |
EventMachine -> concurrent-ruby, take two
|
| | |
| | |
| | |
| | | |
We may still try to send to it.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Also, address the possibility of the listen thread dying and needing to
be respawned. As a bonus, we now defer construction of the thread until
we are first given something to monitor.
|
| | | |
|
| | |
| | |
| | |
| | | |
They're all at risk of races on the first requests.
|
| | | |
|
|/ / |
|
| |
| |
| | |
These methods are more expensive than the alternatives, and have strange semantics that are likely undesirable.
|
|\ \
| | |
| | | |
Fix broken number_to_currency conversion tests
|
|/ / |
|
|\ \
| | |
| | | |
Remove `limit: 11` as backward-compatibility with Rails 2.0
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Integer limit as a byte size was introduced from Rails 2.1.
`limit: 11` is not a byte size, but take care for backward-compatibility
with Rails 2.0 (a892af6). Integer limit out of range should be allowed
to raise by #6349. I think we should remove this backward-compatibility.
|
|\ \ \
| | | |
| | | | |
Clarify DatabaseStatements#execute docs re: memory usage.
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Change number_to_currency behavior for checking negativity of a number
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
- Instead of using `to_f.phase`, just use `to_f.negative`?.
- This change works same for all cases except when number is "-0.0".
-0.0.to_f.negative? => false
-0.0.to_f.phase? => pi
- So -0.0 will be treated as positive from now onwards.
- So this change reverts changes from https://github.com/rails/rails/pull/6512.
- But it should be acceptable as we could not find any currency which
supports negative zeros.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | | |
Extract options for cache_fragment_name method to allow expires_in on cache method
|