| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
The `native_database_types` only used in `TableDefinition` for look up
the default `:limit` option. But this is duplicated process with
`type_to_sql`. Passing `native_database_types` is not needed.
|
|
|
|
|
|
|
| |
[ci skip]
While `JoinDependency` and `JoinDependency::Aliases` were nodoced, the
inner `Table` class made them appear in the API.
|
|
|
|
| |
[ci skip]
|
|\
| |
| | |
do not override fetch on local cache
|
| |
| |
| |
| |
| |
| | |
fetch is supposed to behave differently, this was a mistake merged in
https://github.com/rails/rails/pull/22194
|
|\ \
| | |
| | | |
Return a sized Enumerator from Hash#transform_values{!}
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
The previous commit changes the state of the class, and while we are
cleaning up the database, I forgot to clean up the class
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
I've added a redundant test for this under the attributes API as well,
as that also causes this bug to manifest through public API (and
demonstrates that calling `reset_column_information` on the child
classes would be insufficient)
Since children of a class should always share a table with their parent,
just reloading the schema from the cache should be sufficient here.
`reload_schema_from_cache` should probably become public and
`# :nodoc:`, but I'd rather avoid the git churn here.
Fixes #22057
|
|\ \ \
| | | |
| | | | |
remove deprecated `:tokenizer` option from guide [ci skip]
|
| | | |
| | | |
| | | |
| | | | |
`:tokenizer` option was deprecated in 1c341eb7cbaeb52435f14c2595e391bfb2e5e38b
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
guides, the required option in generators is no longer available.
[ci skip]
|
| | | |
| | | |
| | | |
| | | | |
option for column
|
|\ \ \ \
| | | | |
| | | | | |
Add comma to to make the information clear
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
It goes expected, then actual.
Only changed this because the file was just touched (please don't submit pull requests :)).
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Suppress warning (warning about assigned but unused variable)
|
|/ / / /
| | | |
| | | |
| | | |
| | | | |
These warings have been appeared from
https://github.com/rails/rails/commit/92bc8cdb0771bf6ffcfb31ef58dba529527b514c
|
|\ \ \ \
| | | | |
| | | | | |
Avoid mutating original response in connection management middleware
|
|/ / / / |
|
|\ \ \ \
| |_|_|/
|/| | | |
Clarify automatic creation/deletion of join models for has_many through
|
| | | |
| | | |
| | | |
| | | | |
[ci-skip]
|
|\ \ \ \
| | | | |
| | | | | |
call local_cache 1 time instead of 3 times
|
| |/ / /
| | | |
| | | |
| | | |
| | | | |
LocalCache read_entry previously called the local_cache method 3 times,
now it only calls it once and keeps the result in a local variable.
|
|\ \ \ \
| |/ / /
|/| | | |
use prepend instead of extending every instance
|
|/ / /
| | |
| | |
| | |
| | | |
extending an instance with a module puts the methods on top of it,
prepend does the same but on the class level, so less work for us and more standard way of doing things
|
|\ \ \
| | | |
| | | | |
cache nil replies from backend cache so misses are fast too
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Update association_basics.md [ci skip]
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Moves the definition of an association to the top of the page. I am just starting to learn Rails, and having this definition at the top instead of in the second section would be really useful.
Updates the Types of Associations section for clarity. Moves the list of associations before the explanation.
Links to wikipedia articles on Primary and Foreign keys.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
javascript => JavaScript
JavaScript is spelled with a capital J and S.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
It was deprecated in https://github.com/rails/rails/pull/19135.
We're now favoring `public_file_server.headers`.
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
kamipo/test_binary_data_is_not_logged_is_for_prepared_statements
`test_binary_data_is_not_logged` is for prepared statements logging
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
kamipo/no_need_mysql_doubles_reset_column_information
No need `MysqlDouble` and `MysqlDouble.reset_column_information`
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| |/ / / / / /
|/| | | | | |
| | | | | | |
| | | | | | | |
jonatack/fix-static-cache-control-deprecation-warning
Fix `static_cache_control` deprecation warning
|
|/ / / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
1. Fix the message by wrapping the value in missing quotes and adding
the missing backtick at the end. Finally, :lipstick: by adding a space
inside the curly braces.
Before:
`config.public_file_server.headers = {'Cache-Control' => public,
max-age=31536000}
Now:
`config.public_file_server.headers = { 'Cache-Control' => 'public,
max-age=31536000' }`
2. Display `static_cache_control` instead of static_cache_control. This
follows what the 2 neighboring methods are doing.
3. Use strip_heredoc to improve the code formatting and readability
like the 2 neighboring methods and wrap to 80 characters.
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Require only necessary concurrent-ruby classes.
|
| | | | | | | |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
docrails is a private repository now to avoid pull requests to it.
|
|/ / / / / / |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This solves:
```
Expected /Completed 200 OK in [\d]ms/ to match "Completed 200 OK in 943ms".
```
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Should use `public_file_server.enabled` instead.
Clarified that static files will be served from the public directory, where it made sense.
Also removed occurrence of the deprecated `static_cache_control`.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Forgot to do it in 748b2f9, when deprecating `serve_static_files`.
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Since `bin/test` would define `Rails.root` before loading AM but `bundle
exec rake` would define it after loading AM, this lead to the following
test failure when using `bin/test`:
```
--- expected
+++ actual
@@ -1 +1 @@
-{:location=>"/var/folders/_r/9kh50y4j0vn7zgg2p65z1lcm0000gn/T/mails"}
+{:location=>"/Users/senny/Projects/rails/actionmailer/tmp/mails"}
```
This patch ensures that `Rails.root` is defined when loading AM to
achieve consistent test results.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Add enabled flag to the public file server settings.
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
The configuration for `config.static_cache_control`, and its replacement
`config.public_file_server.headers` are implemented in Railties.
People would configure this in environment files, which is Railties domain too.
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
As discussed in https://github.com/rails/rails/pull/19135#issuecomment-153385986.
Replaces `serve_static_files` to unify the static options under the `public_file_server` wing.
Deprecates `serve_static_files` accessors, but make them use the newer config internally.
|