aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_controller
Commit message (Collapse)AuthorAgeFilesLines
* ActionController::Redirecting : fix docs typodharmatech2011-08-201-1/+1
|
* Tags with invalid names should also be stripped in order to preventAaron Patterson2011-08-161-1/+1
| | | | XSS attacks. Thanks Sascha Depold for the report.
* Use lazy load hooks to set parameter wrapping configuration. This means that ↵Jon Leighton2011-08-161-2/+0
| | | | it doesn't force Action Controller / Active Record to load, but it doesn't fail if they have already loaded. Thanks @josevalim for the hint.
* Don't refer to ActionController::Base in the wrap_parameters initializer - ↵Jon Leighton2011-08-161-0/+2
| | | | use config object instead. Cuts about 15% off the load time. (#734)
* Methods like status and location are interfering with redirect_to [Closes #2511]Franck Verrot2011-08-141-3/+3
|
* Merge branch 'master' of github.com:lifo/docrailsXavier Noria2011-08-131-1/+1
|\ | | | | | | | | | | | | | | | | | | | | Conflicts: RELEASING_RAILS.rdoc actionpack/lib/sprockets/railtie.rb actionpack/test/template/sprockets_helper_test.rb activerecord/test/cases/calculations_test.rb railties/guides/source/3_1_release_notes.textile railties/guides/source/active_resource_basics.textile railties/guides/source/command_line.textile
| * Typo fixOge Nnadi2011-08-131-1/+1
| |
| * Fix the issue where default_url_options is being cached on test cases. ↵thoefer2011-08-132-0/+6
| | | | | | | | | | | | Closes #1872. Closes #2031. Signed-off-by: José Valim <jose.valim@gmail.com>
* | x_sendfile_header now defaults to nil and production.rb env file doesn'tSantiago Pastorino2011-08-071-1/+1
| | | | | | | | | | | | | | set a particular value for it. This allows servers to set it through X-Sendfile-Type, read https://github.com/rack/rack/blob/master/lib/rack/sendfile.rb for more info. Anyways you can force this value in your production.rb
* | Fix the issue where default_url_options is being cached on test cases. ↵thoefer2011-08-012-0/+6
|/ | | | | | Closes #1872. Closes #2031. Signed-off-by: José Valim <jose.valim@gmail.com>
* Changed a few instances of of words in the API docs written in British ↵Oemuer Oezkir2011-07-241-1/+1
| | | | | | English to American English(according to Weber)
* Ensure that status codes are logged properlyJesse Storimer2011-07-181-4/+4
| | | | | | Needed to move AC::Metal::Instrumentation before AM::Metal::Rescue so that status codes rendered from rescue_from blocks are logged properly.
* Merge pull request #2009 from vatrai/autoload-todo-newJosé Valim2011-07-112-0/+2
|\ | | | | TODO fix explicitly loading exceptations, autoload removed
| * TODO fix explicitly loading exceptations, autoload removedVishnu Atrai2011-07-112-0/+2
| |
* | Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-07-093-4/+4
|\ \ | |/ |/|
| * find(:all) => allAkira Matsuda2011-07-083-4/+4
| |
* | Remove stream at the class level.José Valim2011-07-061-38/+5
|/ | | | | | This is because only template rendering works with streaming. Setting it at the class level was also changing the behavior of JSON and XML responses, closes #1337.
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-07-052-11/+13
|\
| * document handle_unverified_request methodVijay Dev2011-07-021-0/+2
| |
| * update doc about resetting the session in case of authenticity token mismatchVijay Dev2011-07-011-6/+5
| |
| * clarify which url would be used to cache a page when options are not ↵Matt Jankowski2011-06-261-2/+2
| | | | | | | | provided to #cache_page to determine that url
| * elaborate on how web server configuration to check for static files on disk ↵Matt Jankowski2011-06-261-3/+4
| | | | | | | | enables page caching to work without hitting the app server stack
* | Make sure respond_with with :js tries to render a template in all casesJosé Valim2011-06-301-0/+5
| |
* | [IMPORTANT] Make "sprockets/railtie" require explicit.José Valim2011-06-291-1/+0
| | | | | | | | | | | | | | | | | | | | This makes "sprockets/railtie" explicit. This means that sprockets will be loaded when you require "rails/all". If you are not using requiring "rails/all", you need to manually load it with all other framework railties. In order to be complete, this commit also adds --skip-sprockets to the rails generator.
* | Make send_file guess content-type from file extension, if type wasn't ↵Esad Hajdarevic2011-06-281-2/+13
| | | | | | | | supplied (Issue #1847). Update tests & documentation.
* | make the logs a little simpler to view, put the render message inline with ↵James Cox2011-06-241-4/+5
|/ | | | other events, pull processing to char[0] and add a new line to the completed. looks more like a block now, which is useful for serial actions like test/dev
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-06-231-7/+9
|\
| * more detail on how the flow between redirect and show works, and minor grammarMatt Jankowski2011-06-211-7/+9
| |
* | Fixes an issue where cache sweepers with only after filters would have no ↵Jeroen Jacobs2011-06-211-0/+1
| | | | | | | | | | | | controller object It would raise undefined method controller_name for nil
* | Call super here to allow SessionHash initialize new stuff for usSantiago Pastorino2011-06-191-1/+1
|/
* Tweak linebreak in ActionController::Redirecting docChristine Yen2011-06-151-2/+2
|
* Improve documentation around status code argument of redirect_toChristine Yen2011-06-151-1/+2
|
* remove warning: assigned but unused variableSantiago Pastorino2011-06-082-2/+1
|
* Added missing assert to test checking form with namespaced isolated model ↵Piotr Sarnacki2011-06-071-1/+1
| | | | and fixed helpers to work correctly in such case
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-06-063-6/+5
|\ | | | | | | | | Conflicts: activerecord/RUNNING_UNIT_TESTS
| * Remove trailing white-spacesGuillermo Iguaran2011-06-051-1/+0
| |
| * some grammatical correctionsVijay Dev2011-06-041-3/+3
| |
| * Removing trailing white-spacesGuillermo Iguaran2011-05-311-2/+2
| |
* | Add backward compatibility for testing cookiesAndrew White2011-06-051-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit restores the ability to assign cookies for testing via @request.env['HTTP_COOKIE'] and @request.cookies, e.g: @request.env['HTTP_COOKIE'] = 'user_name=david' get :index assert_equal 'david', cookies[:user_name] and @request.cookies[:user_name] = 'david' get :index assert_equal 'david', cookies[:user_name] Assigning via cookies[] is the preferred method and will take precedence over the other two methods. This is so that cookies set in controller actions have precedence and are carried over between calls to get, post, etc.
* | Refactor ActionController::TestCase cookiesAndrew White2011-06-041-13/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Assigning cookies for test cases should now use cookies[], e.g: cookies[:email] = 'user@example.com' get :index assert_equal 'user@example.com', cookies[:email] To clear the cookies, use clear, e.g: cookies.clear get :index assert_nil cookies[:email] We now no longer write out HTTP_COOKIE and the cookie jar is persistent between requests so if you need to manipulate the environment for your test you need to do it before the cookie jar is created.
* | Merge pull request #1203 from dchelimsky/stringify-parameter-values-in-testsJosé Valim2011-05-281-0/+19
|\ \ | | | | | | Stringify param values in controller tests.
| * | use to_param (and change method to name accordingly)David Chelimsky2011-05-221-8/+8
| | | | | | | | | | | | | | | - exclude Rack::Test::UploadedFile to pass existing tests. Are there any other types we're missing?
| * | Stringify param values in controller tests.David Chelimsky2011-05-221-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | This reduces false positives that come from using ints in params in tests, which do not get converted to strings in the tests. In implementations going through rack, they do get converted to strings. - David Chelimsky and Sam Umbach
* | | Merge pull request #1324 from dmathieu/no_error_on_invalid_formatJosé Valim2011-05-261-1/+1
|\ \ \ | |_|/ |/| | Don't raise an exception if the format isn't recognized
| * | don't raise an exception if the format isn't recognizeddmathieu2011-05-261-1/+1
| | | | | | | | | | | | Fixed while traveling to heuruko
* | | Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-05-256-14/+14
|\ \ \ | |/ / |/| | | | | | | | | | | Conflicts: actionmailer/lib/action_mailer/base.rb activesupport/lib/active_support/core_ext/kernel/requires.rb
| * | fix minor spelling mistakes in commentsDmytrii Nagirniak2011-05-231-3/+3
| | |
| * | Remove extra white spaces on ActionPack docs.Sebastian Martinez2011-05-235-11/+11
| | |
* | | Replace references to ActiveSupport::SecureRandom with just SecureRandom, ↵Jon Leighton2011-05-232-2/+2
| |/ |/| | | | | and require 'securerandom' from the stdlib when active support is required.
* | renamed the wrap_parameters :only and :except options to :include and ↵Josh Kalderimis2011-05-191-17/+17
|/ | | | :exclude to make it consistent with controller filters