Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Do not mutate given options hash in number helpers | Carlos Antonio da Silva | 2012-01-25 | 1 | -0/+25 |
| | |||||
* | Make audio_tag able to receive multiple sources | Janko Marohnić | 2012-01-25 | 1 | -0/+2 |
| | |||||
* | Make path_to_video in video_tag apply to each source | Janko Marohnić | 2012-01-25 | 1 | -2/+2 |
| | |||||
* | Remove unused variables to avoid warnings | Rafael Mendonça França | 2012-01-24 | 1 | -3/+0 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge pull request #4641 from rafaelfranca/date_select-fix-3-2 | José Valim | 2012-01-24 | 1 | -1/+38 |
| | | | | date_select fixes [3-2-stable] | ||||
* | Merge pull request #3775 from karevn/master | Aaron Patterson | 2012-01-24 | 1 | -1/+14 |
|\ | | | | | Please pull my changes - they fix a rare problem with tests framework | ||||
| * | Fix: when using subdomains and constraints, request params were not passed ↵ | karevn | 2011-11-28 | 1 | -1/+14 |
| | | | | | | | | to constraints callback | ||||
* | | Added custom regexps to ASTs that have literal nodes on either side of | Aaron Patterson | 2012-01-23 | 1 | -0/+62 |
| | | | | | | | | symbol nodes. Fixes #4585 | ||||
* | | moved the `get` testing method to a module | Aaron Patterson | 2012-01-23 | 2 | -11/+17 |
| | | |||||
* | | Deprecated multi args to http route methods | Aaron Patterson | 2012-01-23 | 1 | -1/+2 |
| | | |||||
* | | ActionDispatch::Cookies::CookieJar#deleted? predicate method. | Paul Annesley | 2012-01-24 | 1 | -0/+11 |
| | | | | | | | | | | Necessary in controller tests to determine if the CookieJar will delete the given cookie. | ||||
* | | 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 |
| | |