Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 |