Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fixing repond_with working directly on the options hash | BlueHotDog | 2013-10-09 | 1 | -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 MimeResponds | José Valim | 2013-09-09 | 1 | -3/+1 |
| | |||||
* | fix respond_to without blocks not working if one of the blocks is all | grosser | 2013-02-24 | 1 | -1/+1 |
| | |||||
* | Missing or unneeded require extract_options | Akira Matsuda | 2013-02-01 | 1 | -0/+1 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-01-09 | 1 | -4/+4 |
|\ | | | | | | | | | Conflicts: guides/source/getting_started.md | ||||
| * | PUT => PATCH | Akira Matsuda | 2013-01-03 | 1 | -1/+1 |
| | | |||||
| * | find_or_create_by is deprecated in AR 4 | Akira Matsuda | 2013-01-02 | 1 | -3/+3 |
| | | |||||
* | | Reduce number of Strings a bit | Akira Matsuda | 2013-01-07 | 1 | -1/+1 |
|/ | |||||
* | Multiple changes to 1,9 hash syntax | AvnerCohen | 2012-10-27 | 1 | -12/+12 |
| | |||||
* | removes usage of Object#in? from the code base (the method remains defined ↵ | Xavier Noria | 2012-08-06 | 1 | -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/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | load active_support/core_ext/object/inclusion in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | copy-edits 9674d2c | Xavier Noria | 2012-07-21 | 1 | -2/+2 |
| | |||||
* | Clarification to doc of ActionController::MimeResponse.respond_to | Shigeya Suzuki | 2012-07-19 | 1 | -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 actionpack | Francesco Rodriguez | 2012-05-15 | 1 | -7/+0 |
| | |||||
* | remove .new from raise ActionController::UnknownFormat | Steven Soroka | 2012-05-06 | 1 | -1/+1 |
| | |||||
* | Raise a rescuable exception when Rails doesn't know what to do with the ↵ | Steven Soroka | 2012-05-06 | 1 | -2/+1 |
| | | | | format, rather than responding with a head :not_acceptable (406) | ||||
* | Remove non-obligatory params in AC respond_to examples | Alexey Vakhov | 2012-04-03 | 1 | -2/+2 |
| | |||||
* | respond_with description: changed 'response' to 'format' | Mark Thomson | 2012-03-18 | 1 | -1/+1 |
| | |||||
* | Revised comments for respond_with | Mark Thomson | 2012-03-18 | 1 | -17/+18 |
| | |||||
* | Merge remote-tracking branch 'origin/master' | Mark Thomson | 2012-03-17 | 1 | -2/+6 |
|\ | |||||
| * | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-03-13 | 1 | -1/+24 |
| |\ | |||||
| * | | Updated description of #retrieve_collector_from_mimes | Mark Thomson | 2012-03-13 | 1 | -2/+6 |
| | | | |||||
* | | | Revised description for responds_with | Mark Thomson | 2012-03-17 | 1 | -19/+115 |
| |/ |/| | |||||
* | | Added documentation for the ActionController::MimeResponds::Collector class. | Mark Thomson | 2012-03-13 | 1 | -1/+24 |
|/ | |||||
* | Set the rendered_format on respond_to. | José Valim | 2012-03-07 | 1 | -0/+1 |
| | |||||
* | Avoid ImplicitRender just call render directly | Santiago Pastorino | 2012-03-06 | 1 | -3/+1 |
| | |||||
* | Always passing a respond block from to responder | Prem Sichanugrist | 2012-03-05 | 1 | -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::OrderedHash | Vishnu Atrai | 2012-03-03 | 1 | -1/+1 |
| | |||||
* | format lookup for partials is derived from the format in which the template ↵ | Santiago Pastorino | 2012-02-22 | 1 | -1/+1 |
| | | | | | | is being rendered Closes #5025 part 2 | ||||
* | Clean up a bit default_response handling and cache format negotiation. | José Valim | 2012-02-04 | 1 | -17/+16 |
| | |||||
* | Fix override API response bug in respond_with | Prem Sichanugrist | 2012-02-03 | 1 | -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 example | Vijay Dev | 2011-11-07 | 1 | -1/+1 |
| | |||||
* | Support symbol and string actions in AC#respond_to options | Alexey Vakhov | 2011-10-11 | 1 | -3/+3 |
| | |||||
* | Add missing dependency. | José Valim | 2011-04-18 | 1 | -0/+2 |
| | |||||
* | removes the RJS template handler | Xavier Noria | 2011-04-13 | 1 | -4/+4 |
| | |||||
* | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-04-12 | 1 | -2/+2 |
|\ | |||||
| * | removed :nodoc: from ActionController::MimeResponds | Jo Liss | 2011-04-10 | 1 | -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 places | Prem Sichanugrist | 2011-04-11 | 1 | -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/docrails | Xavier Noria | 2011-04-03 | 1 | -0/+3 |
|\ | |||||
| * | Explain in the method doc that you need to call respond_to at the class ↵ | Amaia Castro | 2011-03-31 | 1 | -0/+3 |
| | | | | | | | | level in order to use respond_with. | ||||
* | | pass respond_with options to controller render when using a template for api ↵ | Josh Kalderimis | 2011-03-31 | 1 | -2/+2 |
|/ | | | | | | navigation Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Fix respond_with example code so it makes sense | Will | 2010-12-19 | 1 | -2/+2 |
| | |||||
* | Correct deprecated AR usage in ActionController::MimeResponds documentation | Will | 2010-12-19 | 1 | -4/+4 |
| | |||||
* | implicit self here | Santiago Pastorino | 2010-11-25 | 1 | -1/+1 |
| | |||||
* | move the setting up of the mime collector into the collector on init | Josh Kalderimis | 2010-11-25 | 1 | -3/+3 |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | There's no need for ternary op here | Santiago Pastorino | 2010-11-22 | 1 | -1/+1 |
| | |||||
* | Correctly handle the case of an API response that returns a hash by treating ↵ | Chris Eppstein | 2010-11-06 | 1 | -1/+1 |
| | | | | a single hash argument as the resource instead of as options. | ||||
* | doc: cleanup respond_to documentation | Wincent Colaiuta | 2010-07-26 | 1 | -9/+12 |
| | | | | Signed-off-by: Wincent Colaiuta <win@wincent.com> | ||||
* | Fixed many references to the old config/environment.rb and Rails::Initializer | Benjamin Quorning | 2010-07-13 | 1 | -1/+1 |
| |