aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_controller
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Updated description of #retrieve_collector_from_mimesMark Thomson2012-03-131-2/+6
| | | | | |
| * | | | | Merge pull request #5375 from nertzy/assert_template_with_regexpAaron Patterson2012-03-121-2/+7
| |\ \ \ \ \ | | | | | | | | | | | | | | assert_template matches against Regexp
| | * | | | | assert_template matches against RegexpGrant Hutchins2012-03-101-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows for more strict template assertions, while maintaining backward compatibility. For example, if you use assert_template("foo/bar") and "foo/bar/baz" was rendered, the test passes. But if you use assert_template(%r{\Afoo/bar\Z}), you will catch that a different template was rendered. Also, if you passed an unsupported argument to assert_template() in the past, it would silently succeed. Now it raises an ArgumentError.
| * | | | | | Merge pull request #5326 from lest/patch-2Michael Koziarski2012-03-101-2/+18
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | configure how unverified request will be handled
| | * | | | | configure how unverified request will be handledSergey Nartimov2012-03-091-2/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | can be configured using `:with` option in `protect_from_forgery` method or `request_forgery_protection_method` config option possible values: - :reset_session (default) - :exception new applications are generated with: protect_from_forgery :with => :exception
* | | | | | | Revised description for responds_withMark Thomson2012-03-171-19/+115
| |_|/ / / / |/| | | | |
* | | | | | Added documentation for the ActionController::MimeResponds::Collector class.Mark Thomson2012-03-131-1/+24
|/ / / / /
* | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-03-091-0/+10
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | copy edits [ci skip]Vijay Dev2012-03-071-2/+2
| | | | |
| * | | | Add some docs for ActionController::Base.without_modulesCarlos Antonio da Silva2012-03-061-0/+10
| | | | |
* | | | | Set the rendered_format on respond_to.José Valim2012-03-071-0/+1
| | | | |
* | | | | Remove unused ActionController::SessionManagementSantiago Pastorino2012-03-062-10/+0
| | | | |
* | | | | Avoid ImplicitRender just call render directlySantiago Pastorino2012-03-061-3/+1
| | | | |
* | | | | Clean up module docs [ci skip]Vijay Dev2012-03-071-21/+21
| |/ / / |/| | | | | | | | | | | Removed some useless docstrings and no-doc'ed some.
* | | | Always passing a respond block from to responderPrem Sichanugrist2012-03-052-12/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We should let the responder to decide what to do with the given overridden response block, and not short circuit it. Fixes #5280
* | | | force response body to be read in assert_templateSergey Nartimov2012-03-051-0/+4
| | | |
* | | | remove usages of AS::OrderedHashVishnu Atrai2012-03-031-1/+1
|/ / /
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-03-031-1/+1
|\ \ \
| * | | Removed max-stale from the setting the cache-headers in the response as ↵Anuj Dutta2012-03-021-1/+1
| | | | | | | | | | | | | | | | max-stale is a cache request header.
* | | | Can't cache url_options on a controller levelPiotr Sarnacki2012-03-021-11/+12
| | | | | | | | | | | | | | | | | | | | It fails if routes from to railties are called in one context, for example: blog.posts_path and main_app.users_path
* | | | Optimize path helpers.José Valim2012-03-021-16/+14
| | | |
* | | | fix api doc [ci skip]Vijay Dev2012-03-011-1/+1
| | | |
* | | | adds a nodoc [ci skip]Vijay Dev2012-03-011-1/+1
|/ / / | | | | | | | | | We have an empty api doc page otherwise :)
* | | Document the :host option for force_sslAndrew White2012-02-231-0/+1
| | |
* | | Update documentation for force_ssl - closes #5023.Andrew White2012-02-231-1/+14
| | |
* | | SSL should not be disabled by default in any environment.Pat Allan2012-02-231-1/+1
| | |
* | | Merge pull request #5130 from dlee/revised_patch_verbXavier Noria2012-02-223-9/+15
|\ \ \ | | | | | | | | Add config.default_method_for_update to support PATCH
| * | | Add config.default_method_for_update to support PATCHDavid Lee2012-02-223-9/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PATCH is the correct HTML verb to map to the #update action. The semantics for PATCH allows for partial updates, whereas PUT requires a complete replacement. Changes: * adds config.default_method_for_update you can set to :patch * optionally use PATCH instead of PUT in resource routes and forms * adds the #patch verb to routes to detect PATCH requests * adds #patch? to Request * changes documentation and comments to indicate support for PATCH This change maintains complete backwards compatibility by keeping :put as the default for config.default_method_for_update.
* | | | format lookup for partials is derived from the format in which the template ↵Santiago Pastorino2012-02-222-2/+2
|/ / / | | | | | | | | | | | | | | | is being rendered Closes #5025 part 2
* | | search private / protected methods in trunk rubyAaron Patterson2012-02-201-1/+1
| | |
* | | Merge pull request #3479 from arvida/ensure-date-header-on-expires-inJosé Valim2012-02-181-0/+3
|\ \ \ | | | | | | | | Ensure Date header on expires_in
| * | | Ensuring a HTTP Date header when setting Cache-Control's max-age with ↵arvida2011-10-311-0/+3
| | | | | | | | | | | | | | | | #expires_in
* | | | let expires_in accept a must_revalidate flagXavier Noria2012-02-171-2/+6
| | | |
* | | | Rack body respond to each and not to joinSantiago Pastorino2012-02-141-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes undef `to_str' for Rack::Chunked::Body when using caches_action + streaming on an action Closes #5027
* | | | Fixed force_ssl redirects to include original query paramsRyan McGeary2012-02-071-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `ActionController.force_ssl` redirects http URLs to their https equivalent; however, when a URL contains a query string, the resulting redirect lacked the original query string. Conflicts: actionpack/lib/action_controller/metal/force_ssl.rb
* | | | Clean up a bit default_response handling and cache format negotiation.José Valim2012-02-042-19/+17
| | | |
* | | | 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
* | | | Merge pull request #4784 from semaperepelitsa/constantizeAaron Patterson2012-02-011-1/+1
|\ \ \ \ | | | | | | | | | | A little constantize update
| * | | | Dont do underscore and camelize, only camelize is fineSemyon Perepelitsa2012-01-311-1/+1
| | | | |
* | | | | Updates documentaton to indicate that :expires_in can be used with all cache ↵dev2012-01-261-1/+2
|/ / / / | | | | | | | | | | | | stores.
* | | | example bracket errorDamian Le Nouaille2012-01-231-1/+1
| | | |
* | | | Use performed? instead of checking for response_bodyCarlos Antonio da Silva2012-01-192-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | * Check for performed? instead of response_body * Change performed? to return a boolean * Refactor AC::Metal#response_body= to reuse variable
* | | | Do not deprecate performed.José Valim2012-01-191-0/+4
| | | |
* | | | Merge pull request #4497 from carlosantoniodasilva/action-controller-refactorJosé Valim2012-01-175-64/+17
|\ \ \ \ | | | | | | | | | | Action controller refactor - remove Compatibility module
| * | | | Move protected instance variables definition, kill compatibility moduleCarlos Antonio da Silva2012-01-172-15/+5
| | | | |
| * | | | Remove deprecated default_charset= from AC::BaseCarlos Antonio da Silva2012-01-171-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | This should be set globally as a configuration, using `config.action_dispatch.default_charset` instead
| * | | | Move render_to_body logic to return a spaced string to AC::RenderingCarlos Antonio da Silva2012-01-173-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This seems to be required only when calling render :partial with an empty collection from a controller. This call happens to return no content, letting the response body empty, which means to Rails that it should go on and try to find a template to render based on the current action name, thus failing hard. Although tests keep all green, we need to check a better way to fix this.
| * | | | Remove method missing handling when action is not found, use action missing ↵Carlos Antonio da Silva2012-01-171-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instead Do not create a method_missing method to handle not found actions, use the action_missing method provided by Rails instead.
| * | | | Refactor render nothing/text => nil logic, and move to right placeCarlos Antonio da Silva2012-01-171-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | Options :nothing and :text => nil should be handled by ActionController::Rendering instead.
| * | | | Move render :nothing and :text => nil options to AC::RenderingCarlos Antonio da Silva2012-01-171-10/+0
| | | | |