aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_controller/metal/mime_responds.rb
Commit message (Collapse)AuthorAgeFilesLines
...
* Revert "Used Yield instead of block.call" -- this causes all of ↵David Heinemeier Hansson2013-11-141-2/+2
| | | | | | atom_feed_helper_test.rb to fail with "SystemStackError: stack level too deep". This reverts commit d3a1ce1cdc60d593de1682c5f4e3230c8db9a0fd.
* Merge pull request #12889 from kuldeepaggarwal/speed_upsRafael Mendonça França2013-11-141-2/+2
|\ | | | | Used Yield instead of block.call
| * Used Yield instead of block.callKuldeep Aggarwal2013-11-151-2/+2
| |
* | Remove order attribute from collectorLukasz Strzalkowski2013-11-101-4/+3
|/ | | | Ruby 1.8 legacy. Since 1.9 hash preserves insertion order. No need for additional array to achieve this
* Fixing repond_with working directly on the options hashBlueHotDog2013-10-091-0/+1
| | | | | | | | This fixes an issue where the respond_with worked directly with the given options hash, so that if a user relied on it after calling respond_with, the hash wouldn't be the same. Fixes #12029
* Remove remaining coupling with AV in MimeRespondsJosé Valim2013-09-091-3/+1
|
* fix respond_to without blocks not working if one of the blocks is allgrosser2013-02-241-1/+1
|
* Missing or unneeded require extract_optionsAkira Matsuda2013-02-011-0/+1
|
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-01-091-4/+4
|\ | | | | | | | | Conflicts: guides/source/getting_started.md
| * PUT => PATCHAkira Matsuda2013-01-031-1/+1
| |
| * find_or_create_by is deprecated in AR 4Akira Matsuda2013-01-021-3/+3
| |
* | Reduce number of Strings a bitAkira Matsuda2013-01-071-1/+1
|/
* Multiple changes to 1,9 hash syntaxAvnerCohen2012-10-271-12/+12
|
* removes usage of Object#in? from the code base (the method remains defined ↵Xavier Noria2012-08-061-2/+2
| | | | | | | | | | | | | | | | | | | by Active Support) Selecting which key extensions to include in active_support/rails made apparent the systematic usage of Object#in? in the code base. After some discussion in https://github.com/rails/rails/commit/5ea6b0df9a36d033f21b52049426257a4637028d we decided to remove it and use plain Ruby, which seems enough for this particular idiom. In this commit the refactor has been made case by case. Sometimes include? is the natural alternative, others a simple || is the way you actually spell the condition in your head, others a case statement seems more appropriate. I have chosen the one I liked the most in each case.
* load active_support/core_ext/class/attribute in active_support/railsXavier Noria2012-08-021-1/+0
|
* load active_support/core_ext/object/inclusion in active_support/railsXavier Noria2012-08-021-1/+0
|
* copy-edits 9674d2cXavier Noria2012-07-211-2/+2
|
* Clarification to doc of ActionController::MimeResponse.respond_toShigeya Suzuki2012-07-191-1/+2
| | | | | - #respond_to's documentation refer to .respond_to, but it was written as just <respond_to>. Added class name for clarification.
* Removing ==Examples and last blank lines of docs from actionpackFrancesco Rodriguez2012-05-151-7/+0
|
* remove .new from raise ActionController::UnknownFormatSteven Soroka2012-05-061-1/+1
|
* Raise a rescuable exception when Rails doesn't know what to do with the ↵Steven Soroka2012-05-061-2/+1
| | | | format, rather than responding with a head :not_acceptable (406)
* Remove non-obligatory params in AC respond_to examplesAlexey Vakhov2012-04-031-2/+2
|
* respond_with description: changed 'response' to 'format'Mark Thomson2012-03-181-1/+1
|
* Revised comments for respond_withMark Thomson2012-03-181-17/+18
|
* Merge remote-tracking branch 'origin/master'Mark Thomson2012-03-171-2/+6
|\
| * Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-03-131-1/+24
| |\
| * | Updated description of #retrieve_collector_from_mimesMark Thomson2012-03-131-2/+6
| | |
* | | Revised description for responds_withMark Thomson2012-03-171-19/+115
| |/ |/|
* | Added documentation for the ActionController::MimeResponds::Collector class.Mark Thomson2012-03-131-1/+24
|/
* Set the rendered_format on respond_to.José Valim2012-03-071-0/+1
|
* Avoid ImplicitRender just call render directlySantiago Pastorino2012-03-061-3/+1
|
* Always passing a respond block from to responderPrem Sichanugrist2012-03-051-10/+2
| | | | | | | We should let the responder to decide what to do with the given overridden response block, and not short circuit it. Fixes #5280
* remove usages of AS::OrderedHashVishnu Atrai2012-03-031-1/+1
|
* format lookup for partials is derived from the format in which the template ↵Santiago Pastorino2012-02-221-1/+1
| | | | | | is being rendered Closes #5025 part 2
* Clean up a bit default_response handling and cache format negotiation.José Valim2012-02-041-17/+16
|
* Fix override API response bug in respond_withPrem Sichanugrist2012-02-031-8/+23
| | | | | | | | | | 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 incorrect code exampleVijay Dev2011-11-071-1/+1
|
* Support symbol and string actions in AC#respond_to optionsAlexey Vakhov2011-10-111-3/+3
|
* Add missing dependency.José Valim2011-04-181-0/+2
|
* removes the RJS template handlerXavier Noria2011-04-131-4/+4
|
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-04-121-2/+2
|\
| * removed :nodoc: from ActionController::MimeRespondsJo Liss2011-04-101-2/+2
| | | | | | | | | | So we don't miss out on the nice documentation of the respond_to and respond_with instance methods. Also added :nodoc: to protected method.
* | Using Object#in? and Object#either? in various placesPrem Sichanugrist2011-04-111-2/+3
|/ | | | There're a lot of places in Rails source code which make a lot of sense to switching to Object#in? or Object#either? instead of using [].include?.
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-04-031-0/+3
|\
| * Explain in the method doc that you need to call respond_to at the class ↵Amaia Castro2011-03-311-0/+3
| | | | | | | | level in order to use respond_with.
* | pass respond_with options to controller render when using a template for api ↵Josh Kalderimis2011-03-311-2/+2
|/ | | | | | navigation Signed-off-by: José Valim <jose.valim@gmail.com>
* Fix respond_with example code so it makes senseWill2010-12-191-2/+2
|
* Correct deprecated AR usage in ActionController::MimeResponds documentationWill2010-12-191-4/+4
|
* implicit self hereSantiago Pastorino2010-11-251-1/+1
|
* move the setting up of the mime collector into the collector on initJosh Kalderimis2010-11-251-3/+3
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>