Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 |
| | |||||
* | Ensure details are frozen after @details_keys lookup. The implementation ↵ | José Valim | 2010-03-27 | 1 | -2/+1 |
| | | | | waits to freeze until the last required moment, to avoid duping hashes. | ||||
* | Improve performance of the rendering stack by freezing formats as a sign ↵ | José Valim | 2010-03-19 | 1 | -0/+1 |
| | | | | that they shouldn't be further modified. | ||||
* | Finish cleaning up rendering stack from views and move assigns evaluation to ↵ | José Valim | 2010-03-12 | 1 | -0/+1 |
| | | | | controller (so plugins and/or controllers can overwrite just one method). | ||||
* | Convert to class_attribute | Jeremy Kemper | 2010-02-01 | 1 | -9/+12 |
| | |||||
* | Added AbstractController::Collector. | José Valim and Mikel Lindsaar | 2010-01-23 | 1 | -26/+3 |
| | |||||
* | Raise an error if respond_with is invoked and no format is declared. | José Valim | 2010-01-10 | 1 | -3/+6 |
| | |||||
* | Unify class_inheritable_accessor and extlib_inheritable_accessor and allow ↵ | José Valim | 2009-10-17 | 1 | -6/+3 |
| | | | | responder to be set in the class level. | ||||
* | Ensure that blocks are also handled inside the responder. | José Valim | 2009-08-29 | 1 | -18/+16 |
| |