aboutsummaryrefslogtreecommitdiffstats
path: root/actionview
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Expand select documentation to tell about the blockRafael Mendonça França2013-09-231-0/+9
| | | | | | | | | | | | [ci skip]
* | | Use ERB in the CHANGELOG [ci skip]Rafael Mendonça França2013-09-231-3/+5
| | |
* | | Ability to pass block to AV#select helperBogdan Gusiev2013-09-234-5/+29
| | | | | | | | | | | | | | | | | | | | | | | | Example: = select(report, "campaign_ids") do - available_campaigns.each do |c| %option{:data => {:tags => c.tags.to_json}, :value => c.id}= c.name
* | | handle `:namespace` form option in collection labelsVasiliy Ermolovich2013-09-224-1/+43
| | |
* | | Merge pull request #12232 from claudiob/delete-unused-helper-fixturesSantiago Pastorino2013-09-204-16/+0
|\ \ \ | | | | | | | | Remove unused AV helper fixtures from e10a2531
| * | | Remove unused AV helper fixtures from e10a2531claudiob2013-09-144-16/+0
| | | | | | | | | | | | | | | | | | | | Several fixtures for helpers are removed. They were introduced in ActionView by @strzalek but never referenced in any test.
* | | | Bust the template digest cache key when details are changedDaniel Schierbeck2013-09-193-2/+39
| | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | There's no need to do thisSantiago Pastorino2013-09-171-4/+1
|/ / / | | | | | | | | | | | | | | | | | | 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.
* | | "generates" applies to "collection radio" so it should be singularWaynn Lue2013-09-121-2/+2
| | | | | | | | | 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).
* | | Merge pull request #12112 from adamniedzielski/fix-form-for-namespace-and-asRafael Mendonça França2013-09-123-1/+21
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | form_for - fix :namespace and :as options clash Conflicts: actionview/CHANGELOG.md
| * | | form_for - fix :namespace and :as options clashAdam Niedzielski2013-09-093-1/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | :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
* | | | Write the right CHANGELOG entry for #11603Rafael Mendonça França2013-09-121-1/+1
| | | | | | | | | | | | | | | | [ci skip]
* | | | Merge pull request #11603 from ↵Rafael Mendonça França2013-09-123-2/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | jetthoughts/join_strings_instead_of_contactination Cleanup: replace String concatenation by joining for excerpt helper
| * | | | Cleanup of excerpt helperPaul Nikitochkin2013-09-063-2/+10
| |/ / / | | | | | | | | | | | | | | | | * replaced String concatenation by joining * separator has default value to '', even it is nil
* | | | Merge pull request #12173 from arunagw/fixes_reverted_by_revert_commitGuillermo Iguaran2013-09-101-1/+1
|\ \ \ \ | | | | | | | | | | Revert fixes
| * | | | Back AV description in gemspecArun Agrawal2013-09-091-1/+1
| |/ / /
* | | | Remove helper fixtures not used in any testclaudiob2013-09-091-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | Remove remaining coupling with AV in MimeRespondsJosé Valim2013-09-091-0/+7
| | | |
* | | | Remove BasicRendering and remove template functionality from AbsC::RenderingJosé Valim2013-09-091-7/+5
|/ / /
* | | Fix failure with minitest 5.0.7Carlos Antonio da Silva2013-09-061-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | Merge pull request #12143 from rajcybage/fixing_typosCarlos Antonio da Silva2013-09-051-1/+1
|\ \ \ | | | | | | | | fix actionview and activemodel test cases typos
| * | | fix actionview and activemodel test cases typosRajarshi Das2013-09-051-1/+1
| | | |
* | | | Fix method name typosVipul A M2013-09-052-2/+2
| | | |
* | | | Fix a few typos. [ci skip]namusyaka2013-09-041-2/+2
| | | |
* | | | Move skeleton methods from AV to AbsCŁukasz Strzałkowski2013-09-031-22/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The methods: * #render_to_body * #render_to_string * #_normalize_render Haven't had anything specyfic to ActionView. This was common code which should belong to AbstractController
* | | | Return to using protected_instance_variables in AVŁukasz Strzałkowski2013-09-021-4/+0
| | | |
* | | | fix the typo [ci skip]Rajarshi Das2013-09-021-1/+1
|/ / / | | | | | | fix the typo newstopics => new topics in action view cache helper
* | | Improve CHANGELOG entry [ci skip]Rafael Mendonça França2013-08-271-4/+2
| | |
* | | Merge pull request #11768 from cloudspace/cache_template_loadingRafael Mendonça França2013-08-273-5/+24
|\ \ \ | | | | | | | | Only cache template digests if !config.cache_template_loading
| * | | Only cache template digests if config.cache_template_loadingJosh Lauer2013-08-063-5/+24
| | |/ | |/| | | | | | | | | | | | | 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
* | | Move abstract's controller tests to AVŁukasz Strzałkowski2013-08-2521-0/+917
| | | | | | | | | | | | The ones that were actually testing AV functionality and should belong in there
* | | Move actionpack's controller testsŁukasz Strzałkowski2013-08-254-12/+12
| | |
* | | Load HTML in ActionView not ActionPackŁukasz Strzałkowski2013-08-251-0/+1
| | | | | | | | | | | | HTML Scanner is part of ActionView and it should be loaded along with it
* | | Move AP's capture tests to AVŁukasz Strzałkowski2013-08-251-0/+81
| | |
* | | Move remaining layouts tests to AVŁukasz Strzałkowski2013-08-2517-0/+288
| | |
* | | Add AP tests to runnerŁukasz Strzałkowski2013-08-251-1/+9
| | |
* | | Cleanup & reorganise rake tasks in AVŁukasz Strzałkowski2013-08-251-16/+13
| | |
* | | Split rendering tests between AP & AVŁukasz Strzałkowski2013-08-251-248/+13
| | | | | | | | | | | | All rendering test remain in AV. Head, ETag etc. test stay in AP
* | | Move render_test to AVŁukasz Strzałkowski2013-08-2592-0/+1722
| | |
* | | Fist stab on basic renderingŁukasz Strzałkowski2013-08-251-2/+0
| | |
* | | Add #rendered_format method to controllersŁukasz Strzałkowski2013-08-251-0/+4
| | |
* | | Add missing requires inside AVŁukasz Strzałkowski2013-08-254-1/+8
| | |
* | | Do not load AV inside APŁukasz Strzałkowski2013-08-252-0/+2
| | | | | | | | | | | | Move that part to AV railtie
* | | Add missing requireŁukasz Strzałkowski2013-08-251-0/+2
| | |
* | | Revert "Move setting content_type to AV"Łukasz Strzałkowski2013-08-252-3/+1
| | | | | | | | | | | | This reverts commit f4d602aff6cec80304b131ecfcc2676d0304f0cc.
* | | Move setting content_type to AVŁukasz Strzałkowski2013-08-252-1/+3
| | |
* | | Use concat to avoid allocating additional arrayŁukasz Strzałkowski2013-08-251-1/+1
| | |
* | | Pass args to render's super methodŁukasz Strzałkowski2013-08-251-1/+2
| | |
* | | Improve AV::Rendering docsŁukasz Strzałkowski2013-08-251-2/+8
| | |
* | | Include AV::Layouts before setting view_pathsŁukasz Strzałkowski2013-08-251-2/+2
| | |