Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Replaced all 'for' loops with Enumerable#each | Nathan Broadbent | 2012-01-30 | 1 | -1/+1 |
| | |||||
* | tiny refactoring | Vasiliy Ermolovich | 2012-01-29 | 1 | -3/+3 |
| | | | | | use :default option with translate method instead of condition simplify separator method | ||||
* | Remove not used env[] call in routing_test. | Carlos Antonio da Silva | 2012-01-27 | 1 | -1/+0 |
| | | | The commit 4c321c6d42b6e35f9ead12eb1dccdead03c5abf4 removes the path_params variable assignment, actually the entire line is not used at all. | ||||
* | Fix warning. assigned but unused variable - path_params | kennyj | 2012-01-28 | 1 | -1/+1 |
| | |||||
* | Do not generate label for attribute when giving nil | Carlos Antonio da Silva | 2012-01-26 | 1 | -0/+4 |
| | |||||
* | reuse common video/audio tags code and do not modify options | Sergey Nartimov | 2012-01-26 | 1 | -0/+8 |
| | |||||
* | do not modify options in image_tag | Sergey Nartimov | 2012-01-26 | 1 | -0/+6 |
| | |||||
* | allow to pass multiple sources to audio/video tags without using array | Sergey Nartimov | 2012-01-25 | 1 | -0/+2 |
| | |||||
* | Merge pull request #4529 from ask4prasath/refactor | Vijay Dev | 2012-01-25 | 1 | -8/+10 |
|\ | | | | | Refactored class methods on address render test | ||||
| * | Refactored class methods on address render test | prasath | 2012-01-19 | 1 | -8/+10 |
| | | |||||
* | | Merge pull request #4658 from pwim/unicode-paths | Aaron Patterson | 2012-01-25 | 1 | -0/+20 |
| | | | | | | | | Test for unicode path support | ||||
* | | 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" |