aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge pull request #5032 from carlosantoniodasilva/collection-helpers-objectJosé Valim2012-02-141-0/+26
|\ \ | | | | | | Allow collection helpers with block to access current object in the collection
| * | Allow collection helpers with block to access current object in the collectionCarlos Antonio da Silva2012-02-131-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This gives a lot more flexibility to the user, for instance to generate a collection of check boxes and labels, allowing to add custom classes or data-* attributes to the label/check_box using another object attribute. This basically mimics options_for_select functionality that accepts a third option for each item to generate html attributes for each option.
* | | Merge pull request #3305 from cjolly/request_filtered_envJeremy Kemper2012-02-131-0/+33
|\ \ \ | |/ / |/| | Exclude rack.request.form_vars from request.filtered_env
| * | Exclude rack.request.form_vars from request.filtered_envChad Jolly2012-01-181-0/+33
| | |
* | | remove unwanted require for string encodingVishnu Atrai2012-02-131-1/+0
| | |
* | | Merge pull request #5016 from exviva/form_helper_date_fieldsJosé Valim2012-02-122-1/+32
|\ \ \ | | | | | | | | Add HTML5 input[type="date"] helper
| * | | Add HTML5 input[type="date"] helperOlek Janiszewski2012-02-122-1/+32
| | | |
* | | | Removing old Controller testThiago Pradi2012-02-123-39/+0
|/ / /
* | | Cleaning Route generation testsThiago Pradi2012-02-122-58/+2
| | |
* | | say goodbye to #with_test_routes :axe:Xavier Noria2012-02-101-1166/+947
| | |
* | | fixes a regression introduced by 532cd4, and a bogus test in AP the ↵Xavier Noria2012-02-101-6/+6
| | | | | | | | | | | | regression uncovered
* | | Fix GH #4720. Routing problem with nested namespace and already camelized ↵kennyj2012-02-101-0/+26
| | | | | | | | | | | | controller option.
* | | Merge pull request #4957 from ganeshkumar/order_hash_changeXavier Noria2012-02-091-3/+2
|\ \ \ | | | | | | | | replacing the orderhash with hash for ruby-1.9
| * | | replacing the orderhash with hash for ruby-1.9ganesh2012-02-091-3/+2
| | | |
* | | | removed unnecessary codeganesh2012-02-091-1/+0
|/ / /
* | | Fix failing request testCarlos Antonio da Silva2012-02-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Latest changes in remote ip handling conflicted with each other in tests. Related: dd09811fa6214a130fdc2de1d4c00b4337cb15f9 6a720226aad2adffcbd2422d40db772719579e2f
* | | 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