aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge pull request #4574 from ihid/masterJosé Valim2012-01-212-0/+17
|\ \ \ \ | | | | | | | | | | Fixed regression - unable to use a range as choices for form.select.
| * | | | Fix inconsistent ability to use a range as choices for select_tag.Jeremy Walker2012-01-212-0/+17
| | | | |
* | | | | Merge pull request #4588 from carlosantoniodasilva/remove-require-csrfJosé Valim2012-01-212-9/+0
|\ \ \ \ \ | |_|_|/ / |/| | | | Remove not used requires from csrf helper file and test
| * | | | Remove not used requires from csrf helper file and testCarlos Antonio da Silva2012-01-212-9/+0
| |/ / / | | | | | | | | | | | | | | | | These requires were added in a87b92d and the implementation changed in 2cdc1f0, removing the need for them.
* | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-01-211-0/+3
|\ \ \ \ | |/ / / |/| | |
| * | | document `:raise` option support for several helpers [ci skip]Jonathan Roes2012-01-201-0/+3
| | | |
* | | | Merge pull request #4560 from rafaelfranca/av-number_helper-refactorJosé Valim2012-01-201-52/+34
|\ \ \ \ | | | | | | | | | | Refactor NumberHelpers
| * | | | Refactor translations retrievalRafael Mendonça França2012-01-201-18/+15
| | | | |
| * | | | Remove code duplicationRafael Mendonça França2012-01-201-34/+19
| | | | |
* | | | | stop splatting so much. We don't need :star:args everywhereAaron Patterson2012-01-201-7/+6
|/ / / /
* | | | Merge pull request #4556 from brainopia/remove_old_cruftAaron Patterson2012-01-206-11/+10
|\ \ \ \ | | | | | | | | | | Removed unused assigns from ActionView::Template::Error
| * | | | Removed unused assigns from ActionView::Template::Errorbrainopia2012-01-206-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | They existed since initial rails commit by DHH but lost use a long time ago
* | | | | No need to check html_safe? twiceRafael Mendonça França2012-01-202-2/+3
|/ / / /
* | | | Added format to percentageRodrigo Flores2012-01-203-2/+11
| | | |
* | | | Use performed? instead of checking for response_bodyCarlos Antonio da Silva2012-01-193-8/+5
| | | | | | | | | | | | | | | | | | | | | | | | * Check for performed? instead of response_body * Change performed? to return a boolean * Refactor AC::Metal#response_body= to reuse variable
* | | | Do not deprecate performed.José Valim2012-01-193-2/+11
|/ / /
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-01-191-1/+1
|\ \ \
| * | | Fix small typo in mail_to docs in url_helper.rbSimon Baird2012-01-191-1/+1
| | |/ | |/|
* | | Extract method_tagRafael Mendonça França2012-01-192-2/+6
| | |
* | | Refactor button_to helper to use token_tag methodRafael Mendonça França2012-01-194-18/+34
|/ /
* | Remove delegation to ActionController::BaseRafael Mendonça França2012-01-181-1/+0
| |
* | ActionView now has its own loggerRafael Mendonça França2012-01-186-6/+11
| |
* | Fix warning: assigned but unused variable - resultkennyj2012-01-191-1/+1
| |
* | Refactor FormBuilder arguments and default configCarlos Antonio da Silva2012-01-182-13/+10
| | | | | | | | | | | | | | | | | | | | * Do not reopen AV::Base to define default form builder Inside the load hook we are already in AV::Base context. * Do not pass the given block to the form builder The block is evaluated in fields_for context using capture, with the builder as argument. This means we do not need to give the block to the FormBuilder itself.
* | Merge pull request #4509 from rafaelfranca/form_tag-refactorJosé Valim2012-01-172-6/+5
|\ \ | | | | | | Refactor form_for and form_tag
| * | No need to call html_safe since concat two SaffeBuffers always results in a ↵Rafael Mendonça França2012-01-171-1/+1
| | | | | | | | | | | | SafeBuffer
| * | Use block syntax to avoid code duplicationRafael Mendonça França2012-01-171-3/+3
| | |
| * | No need to create a SafeBuffer since form_tag_html always rerturn a SafeBufferRafael Mendonça França2012-01-171-2/+1
| | |
* | | Merge pull request #4510 from carlosantoniodasilva/av-content-tag-forJosé Valim2012-01-172-54/+52
|\ \ \ | | | | | | | | ActionView content_tag_for refactoring and improvements
| * | | Fix errors when running entire suite due to class name collisionCarlos Antonio da Silva2012-01-182-25/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | Cleanup Post model, no need to require fake_modelsCarlos Antonio da Silva2012-01-181-17/+6
| | | |
| * | | 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-182-2/+8
| | | |
| * | | Cleanup persisted setup for modelCarlos Antonio da Silva2012-01-181-7/+7
| | | |
| * | | Mimic AR models yielding when building new records, avoid using tapCarlos Antonio da Silva2012-01-181-6/+8
| | | |
| * | | No need for concat as wellCarlos Antonio da Silva2012-01-181-4/+4
| | | |
| * | | Add test for content tag with prefix and extra html optionsCarlos Antonio da Silva2012-01-181-4/+10
| | | |
| * | | 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
| |/ /
* / / Silent Sprockets logger when config.assets.logger is set to falseGuillermo Iguaran2012-01-171-1/+4
|/ /
* | Merge pull request #4488 from rafaelfranca/av-refactorXavier Noria2012-01-1729-436/+688
|\ \ | | | | | | 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
| | |