| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| |
| |
| | |
Fixed a bug in when using has_many association with :inverse_of option and UUID primary key.
Conflicts:
activerecord/CHANGELOG.md
|
| |
| |
| |
| | |
UUID primary key.
|
|\ \
| | |
| | | |
[ci skip] Add some comment about downcase url encoded string.
|
| | | |
|
|\ \ \
| | | |
| | | | |
[ci skip] Add missing migrate step in generating comment resource section.
|
|/ / / |
|
|\ \ \
| | | |
| | | | |
Fix typo. This test isn't executed even in the postgresql.
|
| |/ / |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This will make easier to hook protected_attributes gem in our code
without making that gem fragile to change in Rails code base.
Closes #12243
|
|\ \ \
| | | |
| | | | |
Removes (redundant?) sentence fragment in intro text
|
| |/ /
| | |
| | | |
Looks like a remnant sentence fragment from the 3.2 guide.
|
|\ \ \
| |/ /
|/| | |
[ci skip] Fix a typo in Engines.md.
|
|/ / |
|
|\ \
| | |
| | | |
Remove unused AV helper fixtures from e10a2531
|
| | |
| | |
| | |
| | |
| | | |
Several fixtures for helpers are removed. They were introduced in
ActionView by @strzalek but never referenced in any test.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Documentation Fix: Remove broken link on rails 3 guide
Closes #12300
|
| | | |
| | | |
| | | |
| | | |
| | | | |
As raised in https://github.com/rails/rails/issues/12300
[ci skip]
|
|\ \ \ \
| | | | |
| | | | | |
Bust the template digest cache key when details are changed
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Since the lookup details will influence which template is resolved, they
need to be included in the cache key -- otherwise two different
templates may erroneously share the same digest value.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Allow attaching to AS::Notifications namespace up front
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Before, you were required to attach *after* adding the methods to the
class, since the attachment process needed the methods to be present.
With this change, any new method will also be attached to the configured
namespace.
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
DRYing up method_missing code; Initializing Codepoint object with default values
|
| | | | | |
|
| | | | | |
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Allow failures in jruby-head
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Run Tests against JRuby master
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Many of the failing tests in rails test suite fail because of missing encoding support, example #11739
And according to recent announcement, lot of encoding issues have been fixed in current master. http://ruby.11.x6.nabble.com/Big-encoding-patches-landed-on-master-td4993719.html
So lets test against JRuby master & incase there are still bugs. it would be easier to report them to JRuby.
|
|\ \ \ \
| | | | |
| | | | | |
Fix incorrect assert_redirected_to failure message
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The previous regex was allowing `_` in the URI scheme, which is not
allowed by RFC 3986. This change brings the regex in line with the RFC.
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In some instances, `assert_redirected_to` assertion was returning an
incorrect and misleading failure message when the assertion failed.
This was due to a disconnect in how the assertion computes the redirect
string for the failure message and how `redirect_to` computes the
string that is actually used for redirection.
I made the `_compute_redirect_to_loaction` method used by `redirect_to`
public and call that from the method `assert_redirect_to` uses to
calculate the URL.
The reveals a new test failure due to the regex used by
`_compute_redirect_to_location` allow `_` in the URL scheme.
|
|\ \ \ \
| |/ / /
|/| | | |
Remove 1.8 compatible code
|
| | | | |
|
|\ \ \ \
| |/ / /
|/| | | |
Calling reset_session inside of a controller with a NullSessionHash raises a nil exception.
|
|/ / /
| | |
| | |
| | | |
Previously it was raising a NilException
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
or even be numbers, so sorting by id doesn't make sense. Please use `sort_by`
and specify the attribute you wish to sort with. For example, change:
Post.all.to_a.sort
to:
Post.all.to_a.sort_by(&:id)
|
| | | |
|
|\ \ \
| | | |
| | | | |
Fix an issue where router can't recognize downcased url encoding path.
|
|/ / / |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
AS does the following inside Time.find_zone! ...
`ActiveSupport::TimeZone[time_zone] || TZInfo::Timezone.get(time_zone)`
and given that the test is stubbing AS::TZ[] we don't need the removed
code.
|
|\ \ \
| | | |
| | | | |
Remove tzinfo dependency from Action Pack
|
|/ / /
| | |
| | |
| | |
| | | |
This gem is used by Active Support but it should not be a dependency of
Action Pack.
|
|\ \ \
| | | |
| | | | |
[ci skip] Improve readability of 4.3's NOTE in migration.md.
|
| | | | |
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Collapse where constraints to the Arel::Nodes::And node
Conflicts:
activerecord/CHANGELOG.md
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In order to remove duplication with joining arel where constraints with
`AND`, all constraints on `build_arel` are collapsed into one head node: `Arel::Nodes::And`
Closes: #11963
|