aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_controller
Commit message (Collapse)AuthorAgeFilesLines
* 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
| | |
| * | Remove deprecated logic to render templates starting with /Carlos Antonio da Silva2012-01-171-1/+0
| | | | | | | | | | | | render :template => "/foo/bar"
| * | Remove relative url root setting from ENV varCarlos Antonio da Silva2012-01-171-4/+0
| | | | | | | | | | | | This is already being set by Rails configuration.
| * | Remove rescue_action from compatibility module and testsCarlos Antonio da Silva2012-01-171-4/+0
| | |
| * | Remove other old compatibility constantsCarlos Antonio da Silva2012-01-172-4/+1
| | |
| * | Remove constant already defined in exceptions moduleCarlos Antonio da Silva2012-01-171-3/+0
| | |
| * | Remove old compatibility methods not being usedCarlos Antonio da Silva2012-01-172-6/+2
| | |
* | | Deprecate RAILS_CACHE constant.kennyj2012-01-181-1/+1
|/ /
* | Deprecate ActionController::Integration, ActionController::IntegrationTest, ↵Vishnu Atrai2012-01-153-1/+10
| | | | | | | | ActionController::PerformanceTest,ActionController::AbstractRequest, ActionController::Request, ActionController::AbstractResponse, ActionController::Response and ActionController::Routing
* | refactor response_body= methodVasiliy Ermolovich2012-01-141-7/+1
| | | | | | | | response_to?(:each) returns false for strings in ruby 1.9, so there is no need for these checks
* | Merge pull request #4429 from marcinbunsch/1923-force-ssl-redirect-keep-flashJosé Valim2012-01-121-0/+1
|\ \ | | | | | | When force redirecting to SSL, make sure that the session is kept
| * | When force redirecting to SSL, make sure that the session is kept. As we're ↵Marcin Bunsch and Przemek Dąbek2012-01-121-0/+1
| | | | | | | | | | | | moving from a non-secure to secure environment, it's safe
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-01-131-2/+2
|\ \ \
| * | | Changed 'proc' to 'Proc.new' in ActionController::Caching::Actions descriptionAlejandro Andrés2012-01-121-2/+2
| |/ /
* / / Handle leading spaces in protocol while sanitizingManu2012-01-121-1/+1
|/ /
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-01-112-1/+2
|\ \
| * | Add missing endRafael Mendonça França2012-01-101-0/+1
| | |
| * | Made an example a little more realisticcodesnik2012-01-071-1/+1
| | |
* | | Rails initialization with initialize_on_precompile = false should set assets_dirSantiago Pastorino2012-01-101-1/+4
| | |
* | | AC:TestCase::Behavior#head only accepts parameters as second argumentRafael Mendonça França2012-01-101-1/+1
| | |
* | | Fix ActionController::TestCase::Behavior.headRafael Mendonça França2012-01-101-1/+1
| | |
* | | Merge pull request #4371 from rafaelfranca/test_unit_cleanupAaron Patterson2012-01-091-1/+1
|\ \ \ | | | | | | | | Test unit cleanup
| * | | Whitespaces :scissors:Rafael Mendonça França2012-01-061-1/+1
| | | |
* | | | get rid of using instance_variable_names method from ASSergey Nartimov2012-01-072-6/+6
|/ / / | | | | | | | | | | | | - instance_variables return symbols in 1.9 - there is instance_variable_defined? method
* | | register spec subclasses for people who specAaron Patterson2012-01-061-0/+6
| | |
* | | convert build_message to sprintfAaron Patterson2012-01-061-9/+5
| | |
* | | test response assertionsAaron Patterson2012-01-061-2/+0
| | |
* | | use assert_includes rather than calling includes? specificallyAaron Patterson2012-01-061-2/+2
| | |
* | | Merge pull request #4327 from Karunakar/minorAaron Patterson2012-01-061-1/+1
|\ \ \ | | | | | | | | removed warning because logger.warn differentiate the warings