| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|/ /
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| | |
I accidentally pushed this change to docrails before realizing that it's technically a code change, so I'm opening this pull request (and rolling back my change to docrails).
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
form_for - fix :namespace and :as options clash
Conflicts:
actionview/CHANGELOG.md
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
:as option should not overwrite :namespace option when
generating html id attribute of the form element. id should be prefixed
by specified namespace even if :as option is present
Add test case showing the issue and code fixing it
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | | |
jetthoughts/join_strings_instead_of_contactination
Cleanup: replace String concatenation by joining for excerpt helper
|
| |/ /
| | |
| | |
| | |
| | | |
* replaced String concatenation by joining
* separator has default value to '', even it is nil
|
|/ /
| |
| |
| |
| |
| | |
The fixture for module AbcHelper defines three functions bare_a,
bare_b and bare_c, but only bare_a is used in the code that tests
helper functions.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Require minitest/mock in test where stub method is needed.
Apparently this was being "imported" by minitest pride plugin, which
previously required "minitest/autorun", that required "minitest/mock",
making the method available by chance.
It has been changed in minitest:
https://github.com/seattlerb/minitest/commit/595ce955c0d89575726d10fc18d2afd8f334bcbe
So we need to make sure we require what we really need where necessary.
|
|\ \
| | |
| | | |
fix actionview and activemodel test cases typos
|
| | | |
|
|/ / |
|
|\ \
| | |
| | | |
Only cache template digests if !config.cache_template_loading
|
| | |
| | |
| | |
| | |
| | |
| | | |
since ActionView::Resolver.caching is set to the same value as config.cache_template_loading
only cache template digests if config.cache_template_loading is not falsy
fixes issues #10752 and #10791
|
| | |
| | |
| | |
| | | |
The ones that were actually testing AV functionality and should belong in there
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
All rendering test remain in AV. Head, ETag etc. test stay in AP
|
| | | |
|
| | |
| | |
| | |
| | | |
Move that part to AV railtie
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| |
| | |
MRI reimplemented Date in C so it doesn't hit this division anymore
while JRuby still uses the old stdlib implementation of Date so
it will always hit this.
With this change the actionview date_helper_test.rb tests should pass on JRuby.
|
|\ \
| |/
|/| |
[Jruby] Make all tests in ActionView::TemplateDigestorTest green
|
| |
| |
| |
| |
| |
| |
| |
| | |
Missing tmp directory will cause the test to fail on JRuby. will commit
adds tmp directory & 22 TemplateDigestorTests which were failing before
are now green.
Further Discussion on this can be seen at #11743
|
|/
|
|
|
|
|
|
|
|
|
|
| |
ActionView::Helpers.asset_path is where the logic for
javascript_include_tag resides. It takes an extname option for
specifying the extension or false to not append it. This exposes that
option to javascript_include_tag.
Without the option files that didn't end with ".js" would get the
extension appended to them. This broke JST templates and other file
types that should be interpreted as JavaScript but who's file extension
isn't ".js"
|
| |
|
|
|
|
|
|
|
| |
In some cases webservers like nginx send the escaped characters
lowercased to the Rails application. The current_page? helper was
comparing the escaped strings that are different since Ruby escapes the
URL using uppercased characters.
|
| |
|
| |
|
| |
|
|
|
|
| |
:content_type option. Closes #11393.
|
|\
| |
| | |
Fix actionview link_to with block and url_hash
|
| |
| |
| |
| |
| |
| | |
Use link_to with block and url_hash, expect block as name.
But ignore block and use url_hash as name.
3-2-stable passes this test. 4-0-stable and master fail this.
|
| |
| |
| |
| |
| |
| |
| | |
When rendering recursive partial Action View is trying to generate the
view digest infinitly causing a stack level error.
Fixes #11340
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Control to output a hidden input tag with name `utf8` without monkey
patching
Before:
form_tag
# => '<form>..<input name="utf8" type="hidden" value="✓" />..</form>'
After:
form_tag
# => '<form>..<input name="utf8" type="hidden" value="✓" />..</form>'
form_tag({}, { :enforce_utf8 => false })
# => '<form>....</form>'
|
|
|
|
| |
Same test exists above 2 or 3 lines.
|
|
|
|
| |
tests
|
| |
|
|
|
|
| |
because it was deprecated.
|
|
|
|
| |
Now if somebody by mistake will remove malformed files test will raise error.
|
| |
|
| |
|
| |
|
| |
|