aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_view
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-01-093-6/+6
|\ | | | | | | | | Conflicts: guides/source/getting_started.md
| * Revert "TODO typo fix"Akira Matsuda2013-01-041-1/+1
| | | | | | | | | | | | This reverts commit 1a59a6dfdca217e31a52779d92aa56b67c6689cb. I guess it's not a typo: https://github.com/jorlhuda/exceptron
| * TODO typo fixGosha Arinich2013-01-041-1/+1
| |
| * PUT => PATCHAkira Matsuda2013-01-032-5/+5
| |
| * s/ERb/ERB/Akira Matsuda2013-01-021-1/+1
| |
* | view_cache_dependency APIJamis Buck2013-01-082-8/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | A declarative API for specifying dependencies that affect template cache digest computation. In your controller, specify any of said dependencies: view_cache_dependency { "phone" if using_phone? } When the block is evaluated, the resulting value is included in the cache digest calculation, allowing you to generate different digests for effectively the same template. (Mostly useful if you're mucking with template load paths.)
* | set 'alt' attribute for image_submit_tagNihad Abbasov2013-01-081-7/+7
| |
* | Do not generate local vars for partials without object or collectionCarlos Antonio da Silva2013-01-081-1/+1
| | | | | | | | | | | | | | | | | | Previously rendering a partial without giving :object or :collection would generate a local variable with the partial name by default. This was noticed due to warnings in Ruby 2.0 of not used variables, which turned out to be the generation of not used variables inside partials that do not contain objects related to them.
* | These are already required through AS/railsAkira Matsuda2013-01-062-2/+0
| | | | | | | | | | | | | | * core_ext/object/blank * concern * core_ext/class/attribute * deprecation
* | Merge pull request #8777 from goshakkk/delegate-classAndrew White2013-01-061-1/+1
|\ \ | | | | | | Delegate to :class rather than 'self.class'
| * | delegate to :class rather than 'self.class'Gosha Arinich2013-01-061-1/+1
| | |
* | | Refactor the logic that checks whether or not to emit the hidden id fieldCarlos Antonio da Silva2013-01-061-7/+6
| | | | | | | | | | | | | | | By checking for object.persisted? first, we avoid the hash lookups for new objects.
* | | Move the hidden :id field logic to where it belongs toCarlos Antonio da Silva2013-01-061-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | When dealing with nested forms, Rails automatically generates a hidden field with the id value of the current object being generated by fields_for. This logic was inside the method that's available from the template object, but we just need it when really dealing with nested attributes, so moving the code to here makes more sense.
* | | Eliminate the usage of parent_builder option from form_forCarlos Antonio da Silva2013-01-061-19/+15
| | | | | | | | | | | | Just use it internally from fields_for until we come up with a better solution.
* | | Do not call fields_for from form_for, to avoid instantiating two buildersCarlos Antonio da Silva2013-01-061-5/+5
| | |
* | | Remove unnecessary begin..rescue..end, use only rescueAkira Matsuda2013-01-061-8/+6
|/ /
* | Fix documentation to content_tag_for [ci skip]Rafael Mendonça França2013-01-051-2/+2
| |
* | Merge pull request #8705 from amparo-luna/change_update_attributes_to_updateGuillermo Iguaran2013-01-033-6/+6
|\ \ | | | | | | Rename update_attributes method to update
| * | Change docs to use update instead of update_attributesAmparo Luna + Guillermo Iguaran2013-01-033-6/+6
| | |
* | | Fix typo on form_tag_helper.rbPaulo Luis Franchini Casaretto2013-01-031-1/+1
| | | | | | | | | | | | The inline documentantion had a typo that was somewhat confusing.
* | | take into account multipart when calculating tag attributesAaron Patterson2013-01-021-4/+7
| | |
* | | ask the form builder for form tag attributesAaron Patterson2013-01-021-2/+3
| | |
* | | do not mutate the options hash (when we do not need to)Aaron Patterson2013-01-021-2/+2
|/ /
* / Make content_tag_for work without blockRafael Mendonça França2013-01-021-1/+5
|/ | | | This is version of #8640 for master
* formatted_code_for should be privateGuillermo Iguaran2012-12-311-12/+12
|
* Add style to AV::Template::Error exception pageGuillermo Iguaran2012-12-311-4/+12
|
* Inherit from MiniTest::Unit::TestCase instead of MiniTest::SpecRafael Mendonça França2012-12-311-3/+0
|
* Extract the component rendering of collection_radio_buttons andRafael Mendonça França2012-12-262-2/+14
| | | | | | | | | | | | | | | | | | | | | | collection_check_boxes to a private method. It will make easier to extend. Example: class CollectionRadioButtons < ActionView::Helpers::Tags::CollectionRadioButtons include CollectionExtensions def render wrap_rendered_collection(super, @options) end private def render_component(builder) builder.radio_button + builder.label(:class => "collection_radio_buttons") end end
* Fix collection_radio_buttons with the option `:checked` with value ofRafael Mendonça França2012-12-261-1/+2
| | | | `false`
* fetch highlighter option only when neededNihad Abbasov2012-12-241-2/+2
|
* Digestor explicit dependency should not contain trailing whitespaceBrian Alexander2012-12-211-1/+1
| | | | test for rails/rails#8586
* Remove obfuscation support from mail_to helperNick Reed2012-12-181-54/+12
| | | | | | Removes support for :encode, :replace_at, and :replace_dot options from the mail_to helper. Support for these options has been extracted to the 'actionview-encoded_mail_to' gem.
* handle protocol option in stylesheet_link_tag and javascript_include_tagVasiliy Ermolovich2012-12-181-2/+6
| | | | closes #8388
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-12-151-0/+412
|\ | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/form_helper.rb railties/lib/rails/info_controller.rb
| * repeating documentation for form helpersThiago Pinto2012-12-091-18/+406
| |
* | Merge pull request #8510 from thedarkone/thread_safety_improvementsAaron Patterson2012-12-144-48/+41
|\ \ | | | | | | Thread safety improvements
| * | Replace some global Hash usages with the new thread safe cache.thedarkone2012-12-144-48/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary of the changes: * Add thread_safe gem. * Use thread safe cache for digestor caching. * Replace manual synchronization with ThreadSafe::Cache in Relation::Delegation. * Replace @attribute_method_matchers_cache Hash with ThreadSafe::Cache. * Use TS::Cache to avoid the synchronisation overhead on listener retrieval. * Replace synchronisation with TS::Cache usage. * Use a preallocated array for performance/memory reasons. * Update the controllers cache to the new AS::Dependencies::ClassCache API. The original @controllers cache no longer makes much sense after @tenderlove's changes in 7b6bfe84f3 and f345e2380c. * Use TS::Cache in the connection pool to avoid locking overhead. * Use TS::Cache in ConnectionHandler.
* | | Improve the documentation of cache_if and cache_unlessRafael Mendonça França2012-12-141-4/+7
| | |
* | | Merge pull request #8497 from acapilleri/conditional_cacheRafael Mendonça França2012-12-141-12/+24
|\ \ \ | | | | | | | | | | | | Removed :if / :unless conditions to fragment cache in favour of *cache_i...
| * | | Removed :if and :unless from fragment cache option in favour ofAngelo capilleri2012-12-141-12/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cache_if(condition, option, &block) and cache_unless(condition, option, &block). In the PR #8371 was introduced conditional options :if and :unless in the cache method. Example: <%= cache @model, if: some_condition(@model) do %> ... <%end%> This is a good feature but *cache_if* and and *cache_unless* are more concise and close to the standard of rails view helpers (ex: link_to_if and link_to_unless). Example: <%= cache_if condition, @model do %> ... <%end%>
* | | | Add missing require.Rafael Mendonça França2012-12-132-0/+3
|/ / / | | | | | | | | | | | | I was trying to use those files without Rails and that require was missing.
* / / Remove deprecation message from Action PackCarlos Antonio da Silva2012-12-111-1/+1
|/ / | | | | | | | | Related to the deprecation of Time.utc_time in favor of Time.utc, in 48583f8bf74d1cefefea3cd6591bd546a9eaff6c.
* / Revert "API reader should look elsewhere for helper instructions"Vijay Dev2012-12-091-24/+0
|/ | | | | | | | | This reverts commit ba2d867b2961b5ef72aad353c2d647b04654cbe3. Reason: https://github.com/lifo/docrails/commit/ba2d867b2961b5ef72aad353c2d647b04654cbe3#commitcomment-2276670 [ci skip]
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-12-082-2/+40
|\
| * API reader should look elsewhere for helper instructionsThiago Pinto2012-12-061-0/+24
| |
| * complementary options guidelines for f.file_field and file_field_tagThiago Pinto2012-12-062-0/+8
| |
| * adding example for f.file_inputThiago Pinto2012-12-061-0/+3
| |
| * adding example for f.file_inputThiago Pinto2012-12-061-0/+3
| |
| * correct bad jquery syntaxThiago Pinto2012-12-061-2/+2
| |
* | Allow fragment cache to accept :if and :unless optionsFabrizio Regini2012-12-051-1/+13
|/ | | | [Stephen Ausman + Fabrizio Regini]