Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Raise a better error if anyone wants to create your own ↵ | Rafael Mendonça França | 2012-01-23 | 1 | -0/+8 |
| | | | | ActionView::Helpers::Tags::Base child and do not implement the render method | ||||
* | Raise exception when date.order includes invalid elements | Indrek Juhkam | 2012-01-22 | 1 | -0/+8 |
| | |||||
* | Merge pull request #4574 from ihid/master | José Valim | 2012-01-21 | 1 | -0/+16 |
|\ | | | | | 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 | 1 | -0/+16 |
| | | |||||
* | | Remove not used requires from csrf helper file and test | Carlos Antonio da Silva | 2012-01-21 | 1 | -7/+0 |
|/ | | | | | These requires were added in a87b92d and the implementation changed in 2cdc1f0, removing the need for them. | ||||
* | Removed unused assigns from ActionView::Template::Error | brainopia | 2012-01-20 | 4 | -6/+6 |
| | | | | | They existed since initial rails commit by DHH but lost use a long time ago | ||||
* | Added format to percentage | Rodrigo Flores | 2012-01-20 | 1 | -0/+1 |
| | |||||
* | Use performed? instead of checking for response_body | Carlos Antonio da Silva | 2012-01-19 | 1 | -3/+0 |
| | | | | | | * 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 | 1 | -0/+6 |
| | |||||
* | Refactor button_to helper to use token_tag method | Rafael Mendonça França | 2012-01-19 | 2 | -5/+24 |
| | |||||
* | ActionView now has its own logger | Rafael Mendonça França | 2012-01-18 | 2 | -4/+3 |
| | |||||
* | 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 | 1 | -3/+3 |
| | | | | | | | | | | * 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. | ||||
* | Fix errors when running entire suite due to class name collision | Carlos Antonio da Silva | 2012-01-18 | 1 | -23/+23 |
| | | | | | | | | | 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 |
| | |||||
* | Do not mutate options hash | Carlos Antonio da Silva | 2012-01-18 | 1 | -0/+6 |
| | |||||
* | 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 |
| | |||||
* | Merge pull request #4488 from rafaelfranca/av-refactor | Xavier Noria | 2012-01-17 | 1 | -0/+4 |
|\ | | | | | ActionView::Helpers::FormHelper refactoring | ||||
| * | Adding one test case for label with block | Rafael Mendonça França | 2012-01-17 | 1 | -0/+4 |
| | | |||||
* | | Bring back rendering templates that start with / in nested structures | Carlos Antonio da Silva | 2012-01-17 | 1 | -0/+22 |
| | | |||||
* | | Remove deprecated default_charset= from AC::Base | Carlos Antonio da Silva | 2012-01-17 | 1 | -4/+4 |
| | | | | | | | | | | 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::Rendering | Carlos Antonio da Silva | 2012-01-17 | 1 | -2/+0 |
| | | | | | | | | | | | | | | | | | | | | | | 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 Silva | 2012-01-17 | 1 | -45/+0 |
| | | | | | | | | | | | | | | instead Do not create a method_missing method to handle not found actions, use the action_missing method provided by Rails instead. | ||||
* | | Rename test class and fix tests to keep consistency | Carlos Antonio da Silva | 2012-01-17 | 1 | -10/+10 |
| | | | | | | | | Based on 50d23bc2bd3653b3c66e480c22ae97c5f7fd7f62. | ||||
* | | Remove deprecated logic to render templates starting with / | Carlos Antonio da Silva | 2012-01-17 | 1 | -22/+0 |
| | | | | | | | | render :template => "/foo/bar" | ||||
* | | Remove rescue_action from compatibility module and tests | Carlos Antonio da Silva | 2012-01-17 | 22 | -83/+7 |
| | | |||||
* | | Remove other old compatibility constants | Carlos Antonio da Silva | 2012-01-17 | 4 | -11/+11 |
| | | |||||
* | | Remove old compatibility methods not being used | Carlos Antonio da Silva | 2012-01-17 | 2 | -9/+0 |
|/ | |||||
* | Stub ip method of alternative request class | Sam Pohlenz | 2012-01-14 | 1 | -0/+4 |
| | |||||
* | Use a BodyProxy instead of including a Module that responds to close. | Santiago Pastorino | 2012-01-13 | 1 | -1/+12 |
| | | | | Closes #4441 if Active Record is disabled assets are delivered correctly | ||||
* | removed unnecessary test_pass test from form_tag_helper_test.rb | Karunakar (Ruby) | 2012-01-13 | 1 | -4/+0 |
| | |||||
* | Merge pull request #4429 from marcinbunsch/1923-force-ssl-redirect-keep-flash | José Valim | 2012-01-12 | 1 | -1/+37 |
|\ | | | | | When force redirecting to SSL, make sure that the session is kept | ||||
| * | When force redirecting to SSL, make sure that the session is kept. As we're ↵ | Marcin Bunsch and Przemek Dąbek | 2012-01-12 | 1 | -1/+37 |
| | | | | | | | | moving from a non-secure to secure environment, it's safe | ||||
* | | Handle leading spaces in protocol while sanitizing | Manu | 2012-01-12 | 1 | -1/+8 |
|/ | |||||
* | AC:TestCase::Behavior#head only accepts parameters as second argument | Rafael Mendonça França | 2012-01-10 | 1 | -0/+4 |
| | |||||
* | Whitespaces :scissors: | Rafael Mendonça França | 2012-01-10 | 1 | -4/+4 |
| | |||||
* | Fix ActionController::TestCase::Behavior.head | Rafael Mendonça França | 2012-01-10 | 1 | -0/+5 |
| | |||||
* | Merge pull request #4371 from rafaelfranca/test_unit_cleanup | Aaron Patterson | 2012-01-09 | 1 | -3/+5 |
|\ | | | | | Test unit cleanup | ||||
| * | Test constant is not defined anymore | Rafael Mendonça França | 2012-01-06 | 1 | -3/+5 |
| | | |||||
* | | instance_variables method returns symbols in 1.9 ruby | Sergey Nartimov | 2012-01-07 | 1 | -1/+1 |
| | | |||||
* | | get rid of using instance_variable_names method from AS | Sergey Nartimov | 2012-01-07 | 1 | -1/+1 |
| | | | | | | | | | | - instance_variables return symbols in 1.9 - there is instance_variable_defined? method | ||||
* | | Merge pull request #4369 from lest/remove-warnings | Piotr Sarnacki | 2012-01-07 | 1 | -2/+2 |
|\ \ | | | | | | | use stub instead of redefining method to suppress warnings | ||||
| * | | use stub instead of redefining method to suppress warnings | Sergey Nartimov | 2012-01-07 | 1 | -2/+2 |
| | | | |||||
* | | | add benchmark helper that works in erb | Sergey Nartimov | 2012-01-07 | 1 | -0/+24 |
| |/ |/| | |||||
* | | require minitest rather than test/unit | Aaron Patterson | 2012-01-06 | 2 | -2/+2 |
|/ | |||||
* | test response assertions | Aaron Patterson | 2012-01-06 | 1 | -0/+55 |
| | |||||
* | don't set the hidden checkbox value if it's nil | Damien Mathieu | 2012-01-06 | 1 | -0/+8 |
| |