aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch
Commit message (Collapse)AuthorAgeFilesLines
* Fix example in flash middlewareAlexey Vakhov2012-04-171-1/+1
|
* Fix comment in AD http responseAlexey Vakhov2012-04-131-1/+1
| | | | response in integration test accessible without @ via attr_reader
* Revert "Merge pull request #5750 from ahoward/master"Santiago Pastorino2012-04-081-10/+1
| | | | | | | This reverts commit 174cf8b22064c3999dfa685014d5b31ac993bb54, reversing changes made to 7ecd6a731bd60665bc6de94095137f0b2c4ada2a. The reverted commit improved the performance in the wrong place, now we have added this 6ddbd1844a6fd6aca2992f5f75c9f605cf89808f improvement.
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-04-071-1/+1
|\
| * Fix example action dispatch in mime typeAlexey Vakhov2012-04-071-1/+1
| |
* | documents the contract for the argument of AD::Routing::RouteSet#url_forXavier Noria2012-04-051-0/+1
| |
* | Make url_for work when option is nil and default_url_options hasRafael Mendonça França2012-04-051-1/+1
| | | | | | | | stringified keys
* | Fix url_for when options is nilRafael Mendonça França2012-04-051-1/+1
|/ | | | | RouteSet#url_for was modifying the options hash that belong to the caller
* url_for: avoid a nil check. unroll a loop. handle most common case 1st.ara.t.howard2012-04-051-18/+13
|
* carefully and quickly merge url_for optionsara.t.howard2012-04-041-1/+17
|
* Merge pull request #5716 from jurriaan/tr-gsub-cleanupSantiago Pastorino2012-04-042-3/+3
|\ | | | | Updated/changed unneeded tr/gsubs
| * Updated/changed useless tr/gsubsJurriaan Pruis2012-04-032-3/+3
| |
* | No need to extend ActiveSupport::Extend hereOscar Del Ben2012-04-041-2/+0
|/
* Merge pull request #5694 from hardipe/masterSantiago Pastorino2012-04-011-4/+0
|\ | | | | Tidy up cookies.rb a bit
| * Useless/Confusing method definition removedhardi2012-04-011-4/+0
| |
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-04-011-1/+1
|\ \ | |/ |/|
| * :success includes the whole 200 range, not just 200.Erich Menge2012-04-011-1/+1
| | | | | | | | | | # File lib/rack/response.rb, line 114 114: def successful?; @status >= 200 && @status < 300; end
* | Remove not needed requiresSantiago Pastorino2012-03-311-1/+0
| |
* | Remove the leading \n added by textarea on assert_selectSantiago Pastorino2012-03-301-0/+1
| |
* | adds delegetion for eof? to AD::Http::UploadedFileJens Fahnenbruck2012-03-271-1/+1
|/ | | | | if you want to read the file you may need to ask if there is something to read from
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-03-271-0/+19
|\
| * add missing do [ci skip]Vijay Dev2012-03-271-1/+1
| |
| * added shallow_prefix option description in documantationLukasz Sarnacki2012-03-271-0/+19
| | | | | | | | It is in response for confusion in issue #5301
* | Return the same session data object when setting session idCarlos Antonio da Silva2012-03-261-1/+2
| |
* | Allow a defining custom member field on resourcesJamie Macey2012-03-251-4/+5
| | | | | | | | | | | | | | | | | | | | By default, resources routes are created with :resource/:id. A model defining to_param can make prettier urls by using something more readable than an integer ID, but since the route picks it up as :id you wind up with awkward User.find_by_username(params[:id]) calls. By overriding the key to be used in @request.params you can be more obvious in your intent.
* | Revert "Return an actual boolean from xml_http_request?"Xavier Noria2012-03-251-1/+1
|/ | | | | | | | | | | | | | Reason: This commit changes code that was committed some year and a half ago. The original code is an ordinary predicate that delegates straight to a boolean operator with no further unnecessaru adorments, as clearly explained in #5329. This change also may confuse users who may now believe they can rely now on singletons, while predicates in Rails rely on standard Ruby semantics for boolean values and guarantee no singletons whatsover. This reverts commit 6349791d5aaf652631b29f1371701fc2985fda83.
* Return an actual boolean from xml_http_request?Tim Pope2012-03-231-1/+1
|
* Fix secure cookies when there are more than one space before the secureRafael Mendonça França2012-03-191-1/+1
| | | | keyword
* Remove exclude option from ActionDispatch::SSLRafael Mendonça França2012-03-191-7/+0
|
* Whitespaces :scissors:Rafael Mendonça França2012-03-171-1/+1
|
* Some refactoring and update ActionDispatch::SSL code to use the Rack 1.4.xRafael Mendonça França2012-03-171-23/+13
|
* Rack::SSL -> ActionDispatch::SSLRafael Mendonça França2012-03-171-0/+87
|
* Add missing requireSantiago Pastorino2012-03-171-0/+1
|
* Remove unused requireSantiago Pastorino2012-03-171-1/+0
|
* strip null bytes from Location header as wellBrian Lopez2012-03-151-1/+1
| | | | add tests for stripping \r\n chars since that's already happening
* Merge pull request #5337 from mreinsch/static_invalid_byte_sequenceAaron Patterson2012-03-151-0/+1
|\ | | | | fix ArgumentError being raised in case of invalid byte sequences
| * fix ArgumentError being raised in case of invalid byte sequencesMichael Reinsch2012-03-081-0/+1
| |
* | Add ActionController::HTTPSantiago Pastorino2012-03-141-2/+7
|/ | | | | | More info http://edgeguides.rubyonrails.org/api_app.html [Carlos Antonio da Silva & Santiago Pastorino]
* Refactor AD::UploadedFile, and raise sooner if tempfile is not presentCarlos Antonio da Silva2012-03-061-8/+6
|
* Added parsing of arbitrary media type parameters.Steve Klabnik2012-03-051-3/+3
| | | | | | Based on #4918. Related to #4127.
* Route root helper shortcutBrian Cardarella2012-03-031-0/+5
| | | | Allow the root route helper to accept just a string
* Turn off verbose mode of rack-cache, we still have X-Rack-Cache to check ↵Santiago Pastorino2012-03-031-1/+1
| | | | | | that info Closes #5245
* don't pass unnecessary argumentSergey Nartimov2012-03-021-2/+2
|
* Optimize url helpers.Sergey Nartimov + José Valim2012-03-022-11/+13
|
* Optimize path helpers.José Valim2012-03-024-13/+67
|
* Tap, tap goes the middlewareJeremy Kemper2012-02-291-4/+1
|
* Merge pull request #5219 from ↵Aaron Patterson2012-02-292-4/+5
|\ | | | | | | | | mattfawcett/2781-fix-querystring-route-constraint-for-testing Fix for #2781
| * Fix the assert_recognizes test method so that it works when there areMatt Fawcett2012-02-242-4/+5
| | | | | | | | constraints on the querystring. Issue #2781
* | reduced localhost check to two method callsAaron Patterson2012-02-281-2/+3
| |
* | remove unnecessary memcache equire in ActionDispatch::Session::CacheStoreBrian Durand2012-02-281-1/+0
| |