aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
* Fix errors when running entire suite due to class name collisionCarlos Antonio da Silva2012-01-181-2/+2
| | | | | | | | | The Post class is created everywhere in the test suite, and due to that when applying the Array() logic to refactor content_tag_for, some other change to the Post class was breaking record tag tests. The solution is to rename the class to not collide with others already defined in the test suite.
* Refactor content tag to not detect options Hash alwaysCarlos Antonio da Silva2012-01-181-7/+6
| | | | | Only check for options and prefix arguments order once when running content_tag_for with a collection.
* Do not mutate options hashCarlos Antonio da Silva2012-01-181-2/+2
|
* Remove capture.Tim Cooper2012-01-181-1/+1
|
* Always treat the object passed to content_tag_for as an array.Tim Cooper2012-01-181-7/+3
|
* Merge pull request #4488 from rafaelfranca/av-refactorXavier Noria2012-01-1728-436/+684
|\ | | | | ActionView::Helpers::FormHelper refactoring
| * Extract input_checked? to a moduleRafael Mendonça França2012-01-174-9/+24
| |
| * Do not need of ActionView::Helpers scope since we are insideRafael Mendonça França2012-01-173-21/+21
| | | | | | | | ActionView::Helpers
| * Remove unused codeRafael Mendonça França2012-01-171-39/+0
| |
| * Do not need each here anymoreRafael Mendonça França2012-01-171-3/+1
| |
| * Extract DatetimeSelectRafael Mendonça França2012-01-174-54/+11
| |
| * Extract TimeSelectRafael Mendonça França2012-01-175-7/+16
| |
| * Extract DateSelectRafael Mendonça França2012-01-174-6/+63
| |
| * CleanupRafael Mendonça França2012-01-171-72/+1
| |
| * Extract TimeZoneSelectRafael Mendonça França2012-01-173-34/+22
| |
| * Extract GroupedCollectionSelectRafael Mendonça França2012-01-173-24/+43
| |
| * Extract CollectionSelectRafael Mendonça França2012-01-175-51/+64
| |
| * Extract SelectRafael Mendonça França2012-01-173-18/+59
| |
| * Remove unused codeRafael Mendonça França2012-01-171-31/+0
| |
| * Extract RangeFieldRafael Mendonça França2012-01-173-11/+10
| |
| * Extract NumberFieldRafael Mendonça França2012-01-173-1/+21
| |
| * Extract EmailFieldRafael Mendonça França2012-01-173-1/+10
| |
| * Extract UrlFieldRafael Mendonça França2012-01-173-1/+10
| |
| * Extract TelFieldRafael Mendonça França2012-01-173-1/+10
| |
| * Extract SearchFieldRafael Mendonça França2012-01-173-14/+26
| |
| * Remove code duplication between CheckBox ans RadioButtonRafael Mendonça França2012-01-173-19/+14
| |
| * Extract RadioButtonRafael Mendonça França2012-01-175-25/+41
| |
| * Extract CheckBoxRafael Mendonça França2012-01-173-40/+62
| |
| * Extract TextAreaRafael Mendonça França2012-01-173-13/+22
| |
| * Extract FileFieldRafael Mendonça França2012-01-174-3/+16
| |
| * Extract HiddenFieldRafael Mendonça França2012-01-173-1/+14
| |
| * Extract PasswordFieldRafael Mendonça França2012-01-174-5/+24
| |
| * Rename classesRafael Mendonça França2012-01-175-8/+8
| |
| * Extract duplicated code to a classRafael Mendonça França2012-01-174-186/+116
| |
| * Extract TextFieldTagRafael Mendonça França2012-01-173-2/+117
| |
| * Extract LabelTagRafael Mendonça França2012-01-173-54/+156
| |
* | Merge pull request #4497 from carlosantoniodasilva/action-controller-refactorJosé Valim2012-01-178-67/+20
|\ \ | | | | | | Action controller refactor - remove Compatibility module
| * | Bring back rendering templates that start with / in nested structuresCarlos Antonio da Silva2012-01-171-0/+1
| | |
| * | Move protected instance variables definition, kill compatibility moduleCarlos Antonio da Silva2012-01-172-15/+5
| | |
| * | Remove deprecated default_charset= from AC::BaseCarlos Antonio da Silva2012-01-172-5/+2
| | | | | | | | | | | | | | | This should be set globally as a configuration, using `config.action_dispatch.default_charset` instead
| * | Move render_to_body logic to return a spaced string to AC::RenderingCarlos Antonio da Silva2012-01-174-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This seems to be required only when calling render :partial with an empty collection from a controller. This call happens to return no content, letting the response body empty, which means to Rails that it should go on and try to find a template to render based on the current action name, thus failing hard. Although tests keep all green, we need to check a better way to fix this.
| * | Remove method missing handling when action is not found, use action missing ↵Carlos Antonio da Silva2012-01-171-8/+0
| | | | | | | | | | | | | | | | | | | | | instead Do not create a method_missing method to handle not found actions, use the action_missing method provided by Rails instead.
| * | Refactor render nothing/text => nil logic, and move to right placeCarlos Antonio da Silva2012-01-172-3/+4
| | | | | | | | | | | | | | | Options :nothing and :text => nil should be handled by ActionController::Rendering instead.
| * | Move render :nothing and :text => nil options to AC::RenderingCarlos Antonio da Silva2012-01-172-10/+3
| | |
| * | Remove deprecated logic to render templates starting with /Carlos Antonio da Silva2012-01-171-1/+0
| | | | | | | | | | | | render :template => "/foo/bar"
| * | Remove relative url root setting from ENV varCarlos Antonio da Silva2012-01-171-4/+0
| | | | | | | | | | | | This is already being set by Rails configuration.
| * | Remove rescue_action from compatibility module and testsCarlos Antonio da Silva2012-01-171-4/+0
| | |
| * | Remove other old compatibility constantsCarlos Antonio da Silva2012-01-172-4/+1
| | |
| * | Remove constant already defined in exceptions moduleCarlos Antonio da Silva2012-01-171-3/+0
| | |
| * | Remove old compatibility methods not being usedCarlos Antonio da Silva2012-01-172-6/+2
| |/