Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | Implement skipped test | Rafael Mendonça França | 2012-02-02 | 1 | -3/+5 | |
| | | | | | | | | | | | | [Carlos Antonio da Silva + Rafael Mendonça França] | |||||
* | | | Add collection_radio_buttons and collection_check_boxes to FormBuilder | Rafael Mendonça França | 2012-02-02 | 1 | -9/+20 | |
| | | | | | | | | | | | | [Carlos Antonio da Silva + Rafael Mendonça França] | |||||
* | | | Change Tag class to Category since there is another Tag class | Rafael Mendonça França | 2012-02-02 | 1 | -33/+33 | |
| | | | | | | | | | | | | [Carlos Antonio da Silva + Rafael Mendonça França] | |||||
* | | | Add collection_check_boxes helper | Carlos Antonio da Silva | 2012-02-02 | 1 | -1/+227 | |
| | | | | | | | | | | | | [Carlos Antonio da Silva + Rafael Mendonça França] | |||||
* | | | Create collection_radio_buttons helper | Carlos Antonio da Silva | 2012-02-02 | 1 | -0/+174 | |
| | | | | | | | | | | | | [Carlos Antonio da Silva + Rafael Mendonça França] | |||||
* | | | Replicate :form html5 attribute to hidden field for check_box | Carlos Antonio da Silva | 2012-02-02 | 1 | -3/+10 | |
| | | | | | | | | | | | | | | | When the new html5 attribute :form is given to the check_box helper, it should be replicated to the hidden field as well. Closes #4848 | |||||
* | | | Merge pull request #4804 from rafaelfranca/check_box-fix | José Valim | 2012-02-01 | 1 | -0/+4 | |
|\ \ \ | | | | | | | | | check_box with nil as unchecked value should be HTML safe | |||||
| * | | | check_box with nil as unchecked value should be HTML safe | Rafael Mendonça França | 2012-01-31 | 1 | -0/+4 | |
| | | | | | | | | | | | | | | | | [Carlos Antonio da Silva + Rafael Mendonça França] | |||||
* | | | | Move escape_once logic to ERB::Util, where it belongs to | Carlos Antonio da Silva | 2012-02-01 | 1 | -0/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | All the logic is based on the HTML_ESCAPE constant available in ERB::Util, so it seems more logic to have the entire method there and just delegate the helper to use it. | |||||
* | | | | Merge pull request #4672 from carlosantoniodasilva/av-url-helper | José Valim | 2012-02-01 | 1 | -8/+8 | |
|\ \ \ \ | | | | | | | | | | | Improvements on some url helper methods and tests | |||||
| * | | | | Change OrderedHash with array options to simple hash usage | Carlos Antonio da Silva | 2012-01-25 | 1 | -8/+8 | |
| | | | | | ||||||
* | | | | | Remove slash from favicon_link_tag method attribute | Lucas Caton | 2012-01-31 | 1 | -1/+1 | |
| |/ / / |/| | | | ||||||
* | | | | 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 | |
| | | |