aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
* Added unit test to cover changes to RouteSet.url_forRyan McGeary2012-02-071-0/+16
| | | | | | | | | ActionDispatch::Routing::RouteSet.url_for now handles passing params through to ActionDispatch::Http::Url.url_for Conflicts: actionpack/test/controller/base_test.rb
* Fixed force_ssl redirects to include original query paramsRyan McGeary2012-02-071-0/+6
| | | | | | | | | | `ActionController.force_ssl` redirects http URLs to their https equivalent; however, when a URL contains a query string, the resulting redirect lacked the original query string. Conflicts: actionpack/lib/action_controller/metal/force_ssl.rb
* Merge pull request #2490 from gsterndale/x_forwarded_for_orderJosé Valim2012-02-071-4/+4
|\ | | | | The first IP address in the X-Forwarded-For header is the originating IP
| * The first IP address in the X-Forwarded-For header is the originating IPGreg Sterndale2012-02-071-4/+4
| |
* | Merge pull request #2632 from gsterndale/replace_trusted_proxiesJosé Valim2012-02-071-2/+13
|\ \ | | | | | | Trusted proxies are configurable
| * | Trusted proxies is replaced with a Regexp or appended to with a StringGreg Sterndale2012-02-071-2/+13
| |/
* | Merge pull request #4918 from scottwb/fix-single-accept-with-qJosé Valim2012-02-071-0/+6
|\ \ | |/ |/| Fix MIME::Type.parse handling of single media with a q value
| * Failing test case that shows issue #736 should still be open.Scott W. Bradley2012-02-061-0/+6
| |
* | Always use content_tag for audio/video tagRafael Mendonça França2012-02-071-15/+15
|/ | | | Closes #4919
* Fix url_for method's behavior when it is called with :controller option ↵kennyj2012-02-071-0/+33
| | | | | | which starts with "/" from multiple nested controller. Closes #3864
* Revert "Deprecated multi args to http route methods"Jeremy Kemper2012-02-051-2/+1
| | | | | | | | | | Too painful to lose the compact shorthand form! This reverts commit e848c52535fa0f9488cdbdb3f1cedc7c7c02d643. Conflicts: actionpack/lib/action_dispatch/routing/mapper.rb
* Highlight defaults to HTML5 `mark` elementBrian Cardarella2012-02-051-12/+12
|
* Add testcase for non english filename. Related to #2982.kennyj2012-02-052-1/+7
|
* Add *_url helpers to get the full assets URLPrem Sichanugrist2012-02-041-2/+129
| | | | | | | Adds `image_url`, `javascript_url`, `stylesheet_url`, `audio_url`, `video_url`, and `font_url` to assets tag helper. These URL helpers will return the full path to your assets. This is useful when you are going to reference this asset from external host.
* Merge pull request #4869 from sikachu/master-responder-fixJosé Valim2012-02-042-0/+35
|\ | | | | Fix override API response bug in respond_with
| * Fix override API response bug in respond_withPrem Sichanugrist2012-02-032-0/+35
| | | | | | | | | | | | | | | | | | | | Default responder was only using the given respond block when user requested for HTML format, or JSON/XML format with valid resource. This fix the responder so that it will use the given block regardless of the validity of the resource. Note that in this case you'll have to check for object's validity by yourself in the controller. Fixes #4796
* | Fix GH #4873. Allow swapping middleware of same classkennyj2012-02-041-0/+6
|/
* Remove default class to collection_check_boxes andRafael Mendonça França2012-02-021-16/+16
| | | | | | collection_radio_buttons [Carlos Antonio da Silva + Rafael Mendonça França]
* Allow collection radio_buttons/check_boxes to access current text/valueCarlos Antonio da Silva2012-02-021-0/+44
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Add changelog, docs and guides entriesCarlos Antonio da Silva2012-02-021-2/+20
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Allow proc for value/text method in collection_selectCarlos Antonio da Silva2012-02-021-10/+42
| | | | | | And options_from_collection_for_select as well. [Carlos Antonio da Silva + Rafael Mendonça França]
* Create a Builder factory class to use with collection helpers + blockCarlos Antonio da Silva2012-02-021-4/+4
| | | | | | | | This will make it easy for the user to handle how check box/radio and labels should be generated, abstracting any text/value/default html options required to make it work. [Carlos Antonio da Silva + Rafael Mendonça França]
* Remove collection_wrapper* and item_wrapper* optionsCarlos Antonio da Silva2012-02-021-173/+0
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Use @output_buffer variable instead of concat in testsCarlos Antonio da Silva2012-02-021-10/+9
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Implement skipped testRafael Mendonça França2012-02-021-3/+5
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Add collection_radio_buttons and collection_check_boxes to FormBuilderRafael Mendonça França2012-02-021-9/+20
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Change Tag class to Category since there is another Tag classRafael Mendonça França2012-02-021-33/+33
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Add collection_check_boxes helperCarlos Antonio da Silva2012-02-021-1/+227
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Create collection_radio_buttons helperCarlos Antonio da Silva2012-02-021-0/+174
| | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* Replicate :form html5 attribute to hidden field for check_boxCarlos Antonio da Silva2012-02-021-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-fixJosé Valim2012-02-011-0/+4
|\ | | | | check_box with nil as unchecked value should be HTML safe
| * check_box with nil as unchecked value should be HTML safeRafael Mendonça França2012-01-311-0/+4
| | | | | | | | [Carlos Antonio da Silva + Rafael Mendonça França]
* | Move escape_once logic to ERB::Util, where it belongs toCarlos Antonio da Silva2012-02-011-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-helperJosé Valim2012-02-011-8/+8
|\ \ | | | | | | Improvements on some url helper methods and tests
| * | Change OrderedHash with array options to simple hash usageCarlos Antonio da Silva2012-01-251-8/+8
| | |
* | | Remove slash from favicon_link_tag method attributeLucas Caton2012-01-311-1/+1
| |/ |/|
* | Replaced all 'for' loops with Enumerable#eachNathan Broadbent2012-01-301-1/+1
| |
* | tiny refactoringVasiliy Ermolovich2012-01-291-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 Silva2012-01-271-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_paramskennyj2012-01-281-1/+1
| |
* | Do not generate label for attribute when giving nilCarlos Antonio da Silva2012-01-261-0/+4
| |
* | reuse common video/audio tags code and do not modify optionsSergey Nartimov2012-01-261-0/+8
| |
* | do not modify options in image_tagSergey Nartimov2012-01-261-0/+6
| |
* | allow to pass multiple sources to audio/video tags without using arraySergey Nartimov2012-01-251-0/+2
| |
* | Merge pull request #4529 from ask4prasath/refactorVijay Dev2012-01-251-8/+10
|\ \ | |/ |/| Refactored class methods on address render test
| * Refactored class methods on address render testprasath2012-01-191-8/+10
| |
* | Merge pull request #4658 from pwim/unicode-pathsAaron Patterson2012-01-251-0/+20
| | | | | | | | Test for unicode path support
* | Do not mutate given options hash in number helpersCarlos Antonio da Silva2012-01-251-0/+25
| |
* | Make audio_tag able to receive multiple sourcesJanko Marohnić2012-01-251-0/+2
| |
* | Make path_to_video in video_tag apply to each sourceJanko Marohnić2012-01-251-2/+2
| |