aboutsummaryrefslogtreecommitdiffstats
path: root/actionview/lib/action_view
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge pull request #10773 from wangjohn/link_and_routing_optionsRafael Mendonça França2013-09-231-2/+3
| | | | | | | | Adding documentation and tests to ``polymorphic_url`` and ``link_to``
* | Fix some edge cases for AV `select` helper with `:selected` optionBogdan Gusiev2013-09-231-1/+2
| |
* | Expand select documentation to tell about the blockRafael Mendonça França2013-09-231-0/+9
| | | | | | | | [ci skip]
* | Ability to pass block to AV#select helperBogdan Gusiev2013-09-232-5/+6
| | | | | | | | | | | | | | | | 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-222-1/+3
| |
* | Bust the template digest cache key when details are changedDaniel Schierbeck2013-09-191-1/+4
| | | | | | | | | | | | 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.
* | Merge pull request #12112 from adamniedzielski/fix-form-for-namespace-and-asRafael Mendonça França2013-09-121-1/+2
|\ \ | | | | | | | | | | | | | | | | | | form_for - fix :namespace and :as options clash Conflicts: actionview/CHANGELOG.md
| * | form_for - fix :namespace and :as options clashAdam Niedzielski2013-09-091-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | :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
* | | Merge pull request #11603 from ↵Rafael Mendonça França2013-09-121-2/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | jetthoughts/join_strings_instead_of_contactination Cleanup: replace String concatenation by joining for excerpt helper
| * | | Cleanup of excerpt helperPaul Nikitochkin2013-09-061-2/+3
| |/ / | | | | | | | | | | | | * replaced String concatenation by joining * separator has default value to '', even it is nil
* | | 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 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
* | Merge pull request #11768 from cloudspace/cache_template_loadingRafael Mendonça França2013-08-271-3/+7
|\ \ | | | | | | Only cache template digests if !config.cache_template_loading
| * | Only cache template digests if config.cache_template_loadingJosh Lauer2013-08-061-3/+7
| | | | | | | | | | | | | | | | | | 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
* | | 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-251-0/+1
| | | | | | | | | | | | 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
| | |
* | | Load AV::Layout to AM::Base in railtiesŁukasz Strzałkowski2013-08-251-0/+6
| | |
* | | Remove view_assigns from AVŁukasz Strzałkowski2013-08-251-7/+0
| | | | | | | | | | | | It wasn't adding anything, just invoking super
* | | Load AV::Layouts dynamicly via railtiesŁukasz Strzałkowski2013-08-251-1/+1
| | |
* | | Code formatting & typo fixesŁukasz Strzałkowski2013-08-251-35/+35
| | |
* | | Remove abstract_controller load hooksŁukasz Strzałkowski2013-08-251-4/+0
| | |
* | | Move protected_instance_variables & view_assigns to AbstractControllerŁukasz Strzałkowski2013-08-251-15/+4
| | |
* | | Remove 'api plugin' from docsŁukasz Strzałkowski2013-08-251-3/+0
| | | | | | | | | | | | AbstractController::Rendering is module which defines thie api
* | | Create AbstractController::Rendering interfaceŁukasz Strzałkowski2013-08-251-17/+4
| | | | | | | | | | | | This interface should be use when implementing renderers.
* | | Revert "Require only path_set && lookup_context instead of whole base"Łukasz Strzałkowski2013-08-251-2/+1
| | | | | | | | | | | | This reverts commit 77bb03df2ca3687b290e9fe77d21007642f0329c.
* | | Hook up AV::Rendering on AV intializationŁukasz Strzałkowski2013-08-251-1/+5
| | |
* | | Move layouts to AVŁukasz Strzałkowski2013-08-252-0/+430
| | |
* | | Move rendering from AP to AVŁukasz Strzałkowski2013-08-251-0/+186
| | |
* | | Require only path_set && lookup_context instead of whole baseŁukasz Strzałkowski2013-08-251-1/+2
| | |
* | | Move view_paths from AP to AVŁukasz Strzałkowski2013-08-251-0/+96
| |/ |/|
* | RDoc tweaksXavier Noria2013-08-071-5/+10
|/
* Removed an unnecessary loop - it kills performance on large texts.Jan Szumiec2013-08-031-4/+6
|
* Add "extname" option to javascript_include_tagNathan Stitt2013-08-031-3/+6
| | | | | | | | | | | | 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"
* Using URI.parser.unescapeArun Agrawal2013-08-031-2/+2
| | | | | | Fixes warning warning: URI.unescape is obsolete
* Make current_page? compare binary stringsRafael Mendonça França2013-08-011-3/+4
|
* Fix `current_page?` when the URL contains escaped charactersRafael Mendonça França2013-08-011-2/+2
| | | | | | | 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.
* Change from `map` => `map!` and `collect!` to save creation of extra array.Vipul A M2013-07-311-2/+2
|