Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | renderers typo fix [ci skip] | ota42y | 2016-12-27 | 1 | -1/+1 |
| | |||||
* | Remove obsolete comment about class-level respond_to | Andrew White | 2016-10-01 | 1 | -2/+0 |
| | | | | | | The class-level respond_to was extracted in ee77770 to responders gem [ci skip] | ||||
* | applies remaining conventions across the project | Xavier Noria | 2016-08-06 | 1 | -1/+0 |
| | |||||
* | applies new string literal convention in actionpack/lib | Xavier Noria | 2016-08-06 | 1 | -1/+1 |
| | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
* | Fix some typos in comments. | Joe Rafaniello | 2016-05-04 | 1 | -1/+1 |
| | | | | [ci skip] | ||||
* | [ci skip] Fix typo Renderering --> Rendering | Akshay Vishnoi | 2016-01-09 | 1 | -1/+1 |
| | |||||
* | Group related methods together | Benjamin Fleischer | 2016-01-03 | 1 | -67/+67 |
| | |||||
* | Add ActionController:Renderers test | Benjamin Fleischer | 2016-01-03 | 1 | -0/+9 |
| | | | | To complement actionpack/test/controller/metal/renderers_test.rb | ||||
* | Test ActionController::Renderers::use_renderers | Benjamin Fleischer | 2015-12-31 | 1 | -0/+34 |
| | |||||
* | Use `Mime[:foo]` instead of `Mime::Type[:FOO]` for back compat | Jeremy Daer | 2015-10-06 | 1 | -7/+7 |
| | | | | | | | | | | | | | | | | | Rails 4.x and earlier didn't support `Mime::Type[:FOO]`, so libraries that support multiple Rails versions would've had to feature-detect whether to use `Mime::Type[:FOO]` or `Mime::FOO`. `Mime[:foo]` has been around for ages to look up registered MIME types by symbol / extension, though, so libraries and plugins can safely switch to that without breaking backward- or forward-compatibility. Note: `Mime::ALL` isn't a real MIME type and isn't registered for lookup by type or extension, so it's not available as `Mime[:all]`. We use it internally as a wildcard for `respond_to` negotiation. If you use this internal constant, continue to reference it with `Mime::ALL`. Ref. efc6dd550ee49e7e443f9d72785caa0f240def53 | ||||
* | stop calling deprecated methods | Aaron Patterson | 2015-09-21 | 1 | -7/+7 |
| | | | | | We should be asking the mime type method for the mime objects rather than via const lookup | ||||
* | added description instead of remove usage [ci skip] | Julio Lopez | 2015-07-11 | 1 | -1/+1 |
| | |||||
* | Removed documentation that still mentioned using respond_with in place | Robert Evans | 2014-11-05 | 1 | -2/+1 |
| | | | | | of respond_to. respond_with was moved into the responders gem and deprecated inside rails, so there is no need to mention it within rails itself. | ||||
* | code gardening in ActionController::Renderers | Xavier Noria | 2014-09-04 | 1 | -7/+12 |
| | | | | | | | | | | | | | | | | | | | * Renames _handle_render_options to _render_to_body_with_renderer, which is more intention-revealing. * The name of the dynamically generated method for a renderer with key :js was "_render_option_js". That name is too weak. :js is an option if you see the render argument as just a generic options hash, but in the context of renderers that's the renderer key, is what identifies the renderer. Now "_render_with_renderer_js" is generated instead, which is crystal clear. * The name of the dynamically generated method for the renderer was constructed using string literals in a few places. That is now encapsulated in a method. * Since we were on it, also removed a couple of redundant selfs. | ||||
* | Address CVE-2014-4671 (JSONP Flash exploit) | Greg Campbell | 2014-07-09 | 1 | -1/+1 |
| | | | | | | Adds a comment before JSONP callbacks. See http://miki.it/blog/2014/7/8/abusing-jsonp-with-rosetta-flash/ for more details on the exploit in question. | ||||
* | Change the JSON renderer to enforce the 'JS' Content Type | Lucas Mazza | 2014-07-02 | 1 | -1/+4 |
| | | | | | | | The controller can set the response format as 'JSON' before the renderer code be evaluated, so we must replace it when necessary. Fixes #15081 | ||||
* | Fix docs for ActionController::Renderers.add | Guillermo Iguaran | 2014-05-26 | 1 | -1/+1 |
| | |||||
* | Add ActionController::Renderers.remove. | Zuhao Wan | 2014-05-20 | 1 | -0/+16 |
| | |||||
* | ActionController::Renderers documentation fix | Stevie Graham | 2014-04-20 | 1 | -2/+2 |
| | | | | | ActionController::Renderers::RENDERERS is an instance of Set. Docs incorrectly state that it's a Hash. | ||||
* | Improve font of some code in API documentation [ci skip] | Chun-wei Kuo | 2013-12-26 | 1 | -1/+1 |
| | | | | | * Add "<tt>" or "+" to improve font of some code and filenames in API documentation * Does not contain wording changes | ||||
* | Do not use instance variables if they are not reused elsewhere | Carlos Antonio da Silva | 2013-09-06 | 1 | -2/+1 |
| | |||||
* | Fail informatively in #respond_with when no appropriate #api_behavior ↵ | Ben Woosley | 2013-08-17 | 1 | -0/+7 |
| | | | | | | | | | | | | renderer is available. Currently if a user calls #respond_with(csvable), but has not csv renderer available, Responder will just run through the default render behavior twice, raising ActionView::MissingTemplate both times. This changes ActionController::Metal::Responder#api_behavior to check in advance whether there is a renderer available, and raise ActionController::MissingRenderer if not. | ||||
* | Multiple changes to 1,9 hash syntax | AvnerCohen | 2012-10-27 | 1 | -3/+3 |
| | |||||
* | 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/blank in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | Removing ==Examples and last blank lines of docs from actionpack | Francesco Rodriguez | 2012-05-15 | 1 | -1/+0 |
| | |||||
* | Avoid calling content type multiple times | José Valim | 2012-04-29 | 1 | -4/+6 |
| | |||||
* | Merge remote-tracking branch 'upstream/master' | Olli Jokinen | 2011-12-01 | 1 | -11/+9 |
|\ | |||||
| * | Refactor to make renderers a Set | Jose and Yehuda | 2011-10-15 | 1 | -11/+9 |
| | | |||||
* | | Change mimetype to Mime::JS if JSONP is used | Olli Jokinen | 2011-07-28 | 1 | -1/+4 |
|/ | |||||
* | Slightly reorganize rendering stack. | José Valim | 2011-04-18 | 1 | -3/+3 |
| | |||||
* | removes support for render :update | Xavier Noria | 2011-04-13 | 1 | -8/+1 |
| | |||||
* | add some docs for ActionController::Renderers | Gabriel Horner | 2011-02-10 | 1 | -0/+37 |
| | |||||
* | If a user wants json output then try best to render json output. In such ↵ | Neeraj Singh | 2010-11-24 | 1 | -1/+1 |
| | | | | | | | | cases prefer kind_of(String) over respond_to?(to_str) [#5841 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Remove unused constant | Santiago Pastorino | 2010-11-22 | 1 | -2/+0 |
| | |||||
* | Remove unneeded metaprogramming and method generation in favor of direct ↵ | Santiago Pastorino | 2010-11-22 | 1 | -24/+10 |
| | | | | definition | ||||
* | renderer calls object.to_json when rendering :json => object [#5655 ↵ | Diego Carrion | 2010-09-27 | 1 | -1/+1 |
| | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Always downstream given options in :json, :xml and :js renderers and add ↵ | José Valim | 2010-04-22 | 1 | -2/+2 |
| | | | | tests for it. | ||||
* | It should be possible to render :json with options | wycats | 2010-04-20 | 1 | -1/+1 |
| | |||||
* | adds missing requires for Object#blank? and Object#present? | Xavier Noria | 2010-03-28 | 1 | -0/+1 |
| | |||||
* | All tests pass without memoizing view_context | Carlhuda | 2010-03-18 | 1 | -1/+1 |
| | |||||
* | Finish cleaning up rendering stack from views and move assigns evaluation to ↵ | José Valim | 2010-03-12 | 1 | -1/+2 |
| | | | | controller (so plugins and/or controllers can overwrite just one method). | ||||
* | Remove the renderer option from the hash. | José Valim | 2010-02-24 | 1 | -1/+1 |
| | |||||
* | Convert to class_attribute | Jeremy Kemper | 2010-02-01 | 1 | -3/+7 |
| | |||||
* | Rename RenderOptions to Renderers | David Heinemeier Hansson | 2009-12-20 | 1 | -0/+91 |