| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
| |
| |
| | |
There is no reason `attributes=` doesn't take `assign_attributes`.
|
| |
| |
| |
| | |
This was added in 9bfa13b, but it is never used from the beginning.
|
|\ \
| | |
| | | |
Add separate test to ensure that `delegate` with `:private` option returns correct value
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
correct value
Remove extra comments `# Asking for private method` in activesupport/test/core_ext/module_test.rb
Improve docs of using `delegate` with `:private`
Update changelog of #31944
|
|\ \
| | |
| | | |
Fix a typo in activerecord/CHANGELOG.md
|
|/ /
| |
| |
| | |
columsn ->
columns
|
|\ \
| | |
| | | |
Add ActiveModel::Attributes#attributes
|
| | |
| | |
| | |
| | |
| | |
| | | |
This starts to fix #31832.
ActiveModel::Attributes includes ActiveModel::AttributeMethods,
which requires an `#attributes` method that returns a hash with string keys.
|
|\ \ \
| | | |
| | | | |
Let select render default selected option for required field
|
| | |/
| |/| |
|
|\ \ \
| | | |
| | | | |
Fix `new_framework_defaults_6_0.rb`
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
`default_enforce_utf8` belongs to `config.action_view`
Update info about `:skip_enforcing_utf8` since we can change default
behavior via `config.action_controller.default_enforce_utf8`
Related to #32125
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Expand AR::Base.abstract_class documentation
[ci skip]
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The previous documentation is somewhat unclear about the use case for an
abstract ActiveRecord class.
This clears it up by highlighting the following points:
- table_name is not derived from the abstract class' name
- type is not derived on direct descendants of the abstract class
- validations, not abstract_class, should be used to specify whether
the parent model can be instantiated or not
|
|\ \ \ \
| |_|/ /
|/| | | |
Use `with_partial_writes` helper in `activerecord/test/cases/dirty_test.rb`
|
|/ / /
| | |
| | |
| | | |
Related to 948b931925febac3c965ab13470065ced68f7b53
|
|\ \ \
| | | |
| | | | |
Uses the absolute path for system test screenshots
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Why:
* When getting an error that generates a screenshot it would be helpful
to be able to ctrl+click it to quickly open it in the browser, which
does not work with relative paths
This change addresses the need by:
* Changing `image_path` to disregard the relative path and use the
absolute one instead
|
|\ \ \ \
| | | | |
| | | | | |
Move rake routes task to rails command
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
After a discussion with matthewd. It was mentioned that rake tasks need
to be moved to rails command.
See: https://github.com/rails/rails/issues/32117
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Don't enforce UTF-8 by default
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
With the disabling of TLS 1.0 by most major websites, continuing to run
IE8 or lower becomes increasingly difficult so default to not enforcing
UTF-8 encoding as it's not relevant to other browsers.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | | |
Use column alias of primary_key in limited_ids_for
|
| | |_|/ / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Prevent `ActiveRecord::FinderMethods#limited_ids_for` from using correct primary
key values even if `ORDER BY` columns include other table's primary key.
Fixes #28364.
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | |
| | | | | | |
Don't accidentally lose includes in serialization
|
| | | | | | |
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | | |
Active Record distinct & order #count regression
|
| | | | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
marceloperini/marceloperini/add-nodoc-to-ActionView__Helpers__FormBuilder#emitted_hidden_id
Add :nodoc: to ActionView::Helpers::FormBuilder#emitted_hidden_id?
|
| | | | | | | | |
|
|\ \ \ \ \ \ \ \
| |/ / / / / / /
|/| | | | | | |
| | | | | | | |
| | | | | | | | |
tgxworld/raise_error_when_advisory_lock_is_not_releases
Raise an error if advisory lock in migrator was not released.
|
| | | | | | | | |
|
|\ \ \ \ \ \ \ \
| |_|_|_|/ / / /
|/| | | | | | | |
[ci skip]Change requring order of files in doc
|
|/ / / / / / /
| | | | | | |
| | | | | | |
| | | | | | | |
This change was made at 4a835aa3236eedb135ccf8b59ed3c03e040b8b01
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Support i18n key at translation of value in submit tag
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
module_name/class_name
Currently submit_tag value translation does not support i18n key style
locale key.
It confuses me a bit because many other components support i18n key
style locale key.
I added i18n key style locale key support to submit tag.
|
|\ \ \ \ \ \ \ \
| |/ / / / / / /
|/| | | | | | | |
Correct `to` option's value of the route in the Bound Parameters sect…
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
routing guide.
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | | |
Remove extra changelog added by #31944
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Previously it was removed by #32106 since it was backported to `5-2-stable`.
[ci skip]
|
|/ / / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Prevent older versions of Chrome from appending a Content-Type header containing the Blob type, rendering the request invalid if we intend not to provide a Content-Type. This behavior was observed in Chrome 58.
|
|\ \ \ \ \ \ \ \
| |_|_|_|_|/ / /
|/| | | | | | | |
delegate to, with private: true option
|
|/ / / / / / / |
|
| | | | | | | |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Actually `reflection.klass` should be valid AR model unless
`polymorphic?`. Previously it worked limitedly by ignoring `NameError`
even if `reflection.klass` is invalid, and our isolated testing depends
on the limited working.
Probably we should also check the klass validity in `check_validity!`
properly. Until that time, I restored the error suppression for now.
Closes #32113.
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
See 948b931925febac3c965ab13470065ced68f7b53 for context
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Currently the place where we limit what gets sent to the database is in
the implementation for `partial_writes`. We should also be restricting
it to column names when partial writes are turned off.
Note that we're using `&` instead of just defaulting to
`self.class.column_names`, as the instance version of `attribute_names`
does not include attributes which are uninitialized (were not included
in the select clause)
|
|/ / / / / /
| | | | | |
| | | | | |
| | | | | | |
Fix customizing the download Content-Type for a directly-uploaded blob via a signed URL. See e8286ee.
|
| |/ / / /
|/| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Since #31895, build of 2.5 and AR combination failed.
https://travis-ci.org/rails/rails/jobs/346064349#L1638
It seems to be the reason that model is not loading properly, so I added
require. But I'm not sure if this is correct
|