| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
This obsoletes the ruby based implementations.
|
|
|
|
|
|
|
|
|
| |
Ruby-pg's default way to serialize values for transmission to the database is to
call #to_s . This however creates a temporary String object for each value.
Setting a class based type map avoids the allocation of this additional String.
The performance benefit is measurable in microbenchmarks, but not with
the overhead of activerecord. However it's free to use and has no drawback.
|
|
|
|
|
|
|
|
|
|
| |
The type map was introduced in aafee23, but wasn't properly filled.
This mainly adjusts many locations, that expected strings instead of
integers or boolean.
add_pg_decoders is moved after setup of the StatementPool, because
execute_and_clear could potentially make use of it.
|
|\
| |
| | |
Documenting remove_column as a reversible migration method
|
|/ |
|
|\
| |
| |
| | |
Avoid loading user's psqlrc when loading test structure
|
| | |
|
|\ \
| | |
| | | |
[skip ci] Active Record Validations guide fixes
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Remove deprecated `[]=`
- Fix duplicate `errors#add` example. The second code example was
originally `[]=`, replace it with `[] <<`.
- Improve explanations for `errors#add` and `errors#full_messages`
Follow-up to PR #19457 (closed after borking my git history). Apologies
for the duplicate PR.
cc @kaspth
|
|\ \
| |/
|/| |
update `config.assets.digest` default value in guides [ci skip]
|
|/ |
|
|\
| |
| | |
[ci skip] Update 4.2 Release notes with more assert_select information.
|
| |
| |
| |
| |
| | |
Adds documentation for substitutions and why Regexp substitions look different
when assertion fails.
|
|\ \
| |/
|/| |
Update ActiveSupport::Subscriber docs
|
| |
| |
| | |
For consistency purposes with the changes done in https://github.com/rails/rails/pull/12285
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
This reverts commit 5cfa6a8ab997089c3012a82052c8c317b2e095f5, reversing
changes made to bfd5bf8313e6ea0bb2eccb68ee5076bb63f0b2db.
Reason: This broken travis build.
|
|\ \
| |/
|/| |
DRY up STI subclass logic
|
|/
|
|
|
|
|
|
|
| |
the newer method used for discriminating new records did not
use the older and more robust method used for instantiating
existing records, but did have a better post-check to ensure
the sublass was in the hierarchy. so move the descendants check
to find_sti_class, and then simply call find_sti_class from
subclass_from_attributes
|
|\
| |
| |
| |
| |
| | |
pinglamb/fix-referencing-wrong-alias-when-joining-tables-of-has-many-through-association
Fix referencing wrong aliases while joining tables of has many through association
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
association
While joining table of has_many :through association, ActiveRecord will
use the actual table name instead of through-join alias. It results with
a wrong SQL and exception is raised. This only happens when calculation
methods like #count is called.
This issue is affecting Rails 4.1.x and 4.2.x as well.
|
|\ \
| | |
| | | |
PostgreSQL, Add test case for "Infinity" string assignment to float columns
|
| |/
| |
| |
| | |
This is implemented in Type::Float, but not tested, so far.
|
|\ \
| | |
| | | |
[ci skip] Remove unacceptable method name
|
| | | |
|
|\ \ \
| |/ /
|/| | |
Fix incorrect description for `assert_nothing_raised`. [ci skip]
|
|/ / |
|
|\ \
| | |
| | | |
Small doc fix. [CI SKIP]
|
| | | |
|
|\ \ \
| | | |
| | | | |
Make sure Array#to_sentence always returns a String
|
| |/ / |
|
|\ \ \
| |/ /
|/| | |
Pass wrapped class name to Sidekiq for logging purposes
|
|/ /
| |
| |
| |
| |
| |
| | |
Sidekiq logs the name of the job class being performed. Because
ActiveJob wraps the class, this means every job logs as an AJ::JobWrapper
instead of the actual class name.
Will help fix mperham/sidekiq#2248
|
|\ \
| | |
| | | |
Remove alias for `i_suck_and_my_tests_are_order_dependent`.
|
|/ / |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
The status returned in the rack [status, headers, body] array was
a string, which can cause problems with middleware that assumes the
status will be a Fixnum. This likely never surfaced because other
middleware to_i the status returned from downstream apps before
passing it on.
|
|\ \
| | |
| | | |
Add documentation for Duration#to_i for clarification
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | | |
nerdinand/remove-activesupport-integer-example-doc
Remove reference to Numeric#from_now, as it is no longer supported [ci skip]
|
|/ / |
|
|\ \
| | |
| | | |
remove old unavailable link with relevant fix patch 1
|
| | | |
|
|\ \ \
| |/ /
|/| | |
update mysql link that has been replaced
|
|/ / |
|
| |
| |
| |
| |
| |
| | |
This is obviously all very internal, but sometimes you have to look at
it... and when you do, it'll save a lot of confusion if it doesn't lie
about its identity.
|
| |
| |
| |
| |
| |
| |
| | |
The subtype will (quite reasonably) ignore the possibility that it has
`changed_in_place?` by becoming nil.
Fixes #19467
|
| |
| |
| |
| | |
No need to merge hashes when simply setting options does the job.
|
| |
| |
| |
| | |
The expected value comes first. Related to #19465.
|
|\ \
| | |
| | | |
Fix ActiveModel::Errors#delete return value to stay backward compatible
|