Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | Merge pull request #4574 from ihid/master | José Valim | 2012-01-21 | 2 | -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 Walker | 2012-01-21 | 2 | -0/+17 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #4588 from carlosantoniodasilva/remove-require-csrf | José Valim | 2012-01-21 | 2 | -9/+0 | |
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | Remove not used requires from csrf helper file and test | |||||
| * | | | | | Remove not used requires from csrf helper file and test | Carlos Antonio da Silva | 2012-01-21 | 2 | -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/docrails | Vijay Dev | 2012-01-21 | 1 | -0/+3 | |
|\ \ \ \ \ | |/ / / / |/| | | | | ||||||
| * | | | | document `:raise` option support for several helpers [ci skip] | Jonathan Roes | 2012-01-20 | 1 | -0/+3 | |
| | | | | | ||||||
* | | | | | Merge pull request #4560 from rafaelfranca/av-number_helper-refactor | José Valim | 2012-01-20 | 1 | -52/+34 | |
|\ \ \ \ \ | | | | | | | | | | | | | Refactor NumberHelpers | |||||
| * | | | | | Refactor translations retrieval | Rafael Mendonça França | 2012-01-20 | 1 | -18/+15 | |
| | | | | | | ||||||
| * | | | | | Remove code duplication | Rafael Mendonça França | 2012-01-20 | 1 | -34/+19 | |
| | | | | | | ||||||
* | | | | | | stop splatting so much. We don't need :star:args everywhere | Aaron Patterson | 2012-01-20 | 1 | -7/+6 | |
|/ / / / / | ||||||
* | | | | | Merge pull request #4556 from brainopia/remove_old_cruft | Aaron Patterson | 2012-01-20 | 6 | -11/+10 | |
|\ \ \ \ \ | | | | | | | | | | | | | Removed unused assigns from ActionView::Template::Error | |||||
| * | | | | | Removed unused assigns from ActionView::Template::Error | brainopia | 2012-01-20 | 6 | -11/+10 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They existed since initial rails commit by DHH but lost use a long time ago | |||||
* | | | | | | No need to check html_safe? twice | Rafael Mendonça França | 2012-01-20 | 2 | -2/+3 | |
|/ / / / / | ||||||
* | | | | | Added format to percentage | Rodrigo Flores | 2012-01-20 | 3 | -2/+11 | |
| | | | | | ||||||
* | | | | | Use performed? instead of checking for response_body | Carlos Antonio da Silva | 2012-01-19 | 3 | -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é Valim | 2012-01-19 | 3 | -2/+11 | |
|/ / / / | ||||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-01-19 | 1 | -1/+1 | |
|\ \ \ \ | ||||||
| * | | | | Fix small typo in mail_to docs in url_helper.rb | Simon Baird | 2012-01-19 | 1 | -1/+1 | |
| | |/ / | |/| | | ||||||
* | | | | Extract method_tag | Rafael Mendonça França | 2012-01-19 | 2 | -2/+6 | |
| | | | | ||||||
* | | | | Refactor button_to helper to use token_tag method | Rafael Mendonça França | 2012-01-19 | 4 | -18/+34 | |
|/ / / | ||||||
* | / | Remove delegation to ActionController::Base | Rafael Mendonça França | 2012-01-18 | 1 | -1/+0 | |
| |/ |/| | ||||||
* | | ActionView now has its own logger | Rafael Mendonça França | 2012-01-18 | 6 | -6/+11 | |
| | | ||||||
* | | Fix warning: assigned but unused variable - result | kennyj | 2012-01-19 | 1 | -1/+1 | |
| | | ||||||
* | | Refactor FormBuilder arguments and default config | Carlos Antonio da Silva | 2012-01-18 | 2 | -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-refactor | José Valim | 2012-01-17 | 2 | -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ça | 2012-01-17 | 1 | -1/+1 | |
| | | | | | | | | | | | | SafeBuffer | |||||
| * | | Use block syntax to avoid code duplication | Rafael Mendonça França | 2012-01-17 | 1 | -3/+3 | |
| | | | ||||||
| * | | No need to create a SafeBuffer since form_tag_html always rerturn a SafeBuffer | Rafael Mendonça França | 2012-01-17 | 1 | -2/+1 | |
| | | | ||||||
* | | | Merge pull request #4510 from carlosantoniodasilva/av-content-tag-for | José Valim | 2012-01-17 | 2 | -54/+52 | |
|\ \ \ | | | | | | | | | ActionView content_tag_for refactoring and improvements | |||||
| * | | | Fix errors when running entire suite due to class name collision | Carlos Antonio da Silva | 2012-01-18 | 2 | -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_models | Carlos Antonio da Silva | 2012-01-18 | 1 | -17/+6 | |
| | | | | ||||||
| * | | | Refactor content tag to not detect options Hash always | Carlos Antonio da Silva | 2012-01-18 | 1 | -7/+6 | |
| | | | | | | | | | | | | | | | | | | | | Only check for options and prefix arguments order once when running content_tag_for with a collection. | |||||
| * | | | Do not mutate options hash | Carlos Antonio da Silva | 2012-01-18 | 2 | -2/+8 | |
| | | | | ||||||
| * | | | Cleanup persisted setup for model | Carlos Antonio da Silva | 2012-01-18 | 1 | -7/+7 | |
| | | | | ||||||
| * | | | Mimic AR models yielding when building new records, avoid using tap | Carlos Antonio da Silva | 2012-01-18 | 1 | -6/+8 | |
| | | | | ||||||
| * | | | No need for concat as well | Carlos Antonio da Silva | 2012-01-18 | 1 | -4/+4 | |
| | | | | ||||||
| * | | | Add test for content tag with prefix and extra html options | Carlos Antonio da Silva | 2012-01-18 | 1 | -4/+10 | |
| | | | | ||||||
| * | | | Remove capture. | Tim Cooper | 2012-01-18 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Always treat the object passed to content_tag_for as an array. | Tim Cooper | 2012-01-18 | 1 | -7/+3 | |
| |/ / | ||||||
* / / | Silent Sprockets logger when config.assets.logger is set to false | Guillermo Iguaran | 2012-01-17 | 1 | -1/+4 | |
|/ / | ||||||
* | | Merge pull request #4488 from rafaelfranca/av-refactor | Xavier Noria | 2012-01-17 | 29 | -436/+688 | |
|\ \ | | | | | | | ActionView::Helpers::FormHelper refactoring | |||||
| * | | Extract input_checked? to a module | Rafael Mendonça França | 2012-01-17 | 4 | -9/+24 | |
| | | | ||||||
| * | | Do not need of ActionView::Helpers scope since we are inside | Rafael Mendonça França | 2012-01-17 | 3 | -21/+21 | |
| | | | | | | | | | | | | ActionView::Helpers | |||||
| * | | Remove unused code | Rafael Mendonça França | 2012-01-17 | 1 | -39/+0 | |
| | | | ||||||
| * | | Do not need each here anymore | Rafael Mendonça França | 2012-01-17 | 1 | -3/+1 | |
| | | | ||||||
| * | | Extract DatetimeSelect | Rafael Mendonça França | 2012-01-17 | 4 | -54/+11 | |
| | | | ||||||
| * | | Extract TimeSelect | Rafael Mendonça França | 2012-01-17 | 5 | -7/+16 | |
| | | | ||||||
| * | | Extract DateSelect | Rafael Mendonça França | 2012-01-17 | 4 | -6/+63 | |
| | | | ||||||
| * | | Cleanup | Rafael Mendonça França | 2012-01-17 | 1 | -72/+1 | |
| | | | ||||||
| * | | Extract TimeZoneSelect | Rafael Mendonça França | 2012-01-17 | 3 | -34/+22 | |
| | | |