| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
Fix `attributes_before_type_cast` for serialized attributes.
|
|/
|
|
| |
Public method `attributes_before_type_cast` used to return internal AR structure (ActiveRecord::AttributeMethods::Serialization::Attribute), patch fixes this. Now behaves like `read_attribute_before_type_cast` and returns unserialised values.
|
|\
| |
| | |
Fix #6951. Use query cache/uncache, when using not only database.yml but also DATABASE_URL.
|
|/
|
|
| |
also DATABASE_URL.
|
|
|
|
|
|
|
|
|
|
|
|
| |
`Mailer.foo(*args).deliver`."
This reverts commit 7e0cf563639bc7508da381b1b8321c7a89be1aa8.
Conflicts:
actionmailer/CHANGELOG.md
See discussion at
https://github.com/rails/rails/commit/7e0cf563639bc7508da381b1b8321c7a89be1aa8#commitcomment-2075489
|
|
|
|
|
| |
This reverts commit 7b290ad13c294005db062e746f8b79c8a192e4fa.
debugger gem is now working ok with p286
|
|\
| |
| | |
Allow include_root_in_json to be inheritable by using the class_attribute created by Active Model serialization module.
|
| | |
|
|\|
| |
| |
| |
| | |
gazay/make_caller_attribute_in_deprecation_optional
Make callstack attribute optional in AS:Deprecation
|
|/ |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
This can occur if the user is using :integer columns to store boolean
values. Now we are handling the boolean values but it still raises if
the value can't type cast to integer and is not a boolean. See #7509.
Fixes #8067.
Conflicts:
activerecord/CHANGELOG.md
|
|\
| |
| | |
Add test for code change introduced in this commit f20032f
|
|/ |
|
| |
|
|\
| |
| | |
Provide a call stack for deprecation warnings where needed.
|
| |
| |
| |
| | |
It's sometimes hard to quickly find where deprecated call was performed, especially in case of migrating between Rails versions. So this is an attempt to improve the call stack part of the warning message by providing caller explicitly.
|
|/ |
|
| |
|
|
|
|
|
|
| |
Should not eager_load app/assets
Conflicts:
railties/CHANGELOG.md
|
|\
| |
| | |
Atomic.rb assumes it may chown/chmod a file but doesn't handle the EPERM error.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously this code just assumed it is capable of changing the file
ownership, both user and group. This will fail in a lot of scenario's
unless:
* The process is run as a superuser (root);
* The owning user and group are already set to the user and group we're
trying to chown to;
* The user chown'ing only changes the group to another group it is a
member of.
If either of those conditions are not met the filesystem will simply
deny the operation throwing an error.
It is also not always possible to do a chmod, there might be a SELinux
policy or another limitation preventing the user to change the file
mode. To this end the chmod call has also been added to the rescue
block.
I've also added a little comment above the chmod command that doing a
chmod on a file which has an ACL set will cause the ACL to be
recalculated / modified.
|
|
|
|
| |
Checking for the constant doesn't work
|
|\
| |
| | |
SQLite3Adapter#type_cast should not mutate arguments
|
|/ |
|
|\
| |
| | |
Added ActiveRecord::Relation#none! method
|
| | |
|
|/ |
|
|\
| |
| | |
Unbreak update_column/update_columns for the primary key attribute.
|
|/
|
|
| |
Didn't work before because it updated the model-in-memory first, so the DB query couldn't find the record.
|
|\
| |
| | |
Replace flow-control exception with explicit test.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
It was noticed while profiling 'assets:precompile' in JRuby that
exception creation was consuming a large portion of time, and
some of that was due to File.atomic_write.
Testing first with File.exists? eliminates the need for an exception
which should be a perfomrance improvement on both JRuby and MRI.
In this case, the stat() isn't even extra overhead, since it is always
called.
|
|\ \
| | |
| | | |
Do not render views when mail() isn't called. (NullMail refactoring)
|
| |/ |
|
| |
| |
| |
| | |
Removing JSON for other ruby
|
| | |
|
|\ \
| | |
| | |
| | |
| | | |
senny/6378_create_table_raises_when_defining_pk_column
create_table raises an ArgumentError when the primary key is redefined.
|
| | | |
|
|/ / |
|
|\ \
| | |
| | | |
AR::AttributeMethods#[] raises AM::AttributeMissingError for missing attributes
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This fixes the following behaviour:
class Person < ActiveRecord::Base
belongs_to :company
end
# Before:
person = Person.select('id').first
person[:name] # => nil
person.name # => ActiveModel::MissingAttributeError: missing_attribute: name
person[:company_id] # => nil
person.company # => nil
# After:
person = Person.select('id').first
person[:name] # => ActiveModel::MissingAttributeError: missing_attribute: name
person.name # => ActiveModel::MissingAttributeError: missing_attribute: name
person[:company_id] # => ActiveModel::MissingAttributeError: missing_attribute: company_id
person.company # => ActiveModel::MissingAttributeError: missing_attribute: company_id
Fixes #5433.
|
|\ \
| | |
| | | |
Don't log on pending migration check
|
|/ /
| |
| |
| |
| | |
Conversation from: #6665 cc/ @rafaelfranca
|
| | |
|
| | |
|
|\ \
| |/
|/| |
fix typo in button_to helper example [ci skip]
|
|/ |
|
| |
|
|\
| |
| | |
Use the MySQL varbinary type when appropriate in migrations
|
| | |
|