aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_controller
Commit message (Collapse)AuthorAgeFilesLines
* 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
* Use anonymous? that works on both Ruby 1.8 and 1.9.José Valim2011-05-172-5/+7
|
* add more robust test for wrapping params with anonymous classDavid Chelimsky2011-05-172-3/+6
|
* Add fix for error when an anonymous controller subclasses ApplicationControllerAndy Lindeman2011-05-171-1/+4
|
* Make ParamsWrapper calling newly introduced `Model.attribute_names` instead ↵Prem Sichanugrist2011-05-151-3/+3
| | | | of `.column_names`
* Do not try to call `column_names` on the abstract class.Prem Sichanugrist2011-05-151-1/+1
| | | Normally the table for abstract class won't be existed, so we should not trying to call `#column_names` on it.
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-05-141-0/+21
|\ | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/date_helper.rb railties/lib/rails/generators/rails/app/templates/config/initializers/wrap_parameters.rb.tt
| * minor edits after going through what's new in docrailsXavier Noria2011-05-141-4/+7
| |
| * added an example for AC::UrlFor usage to make usage simpler.Nick Sutterer2011-05-121-1/+13
| |
| * added docs for AbC::UrlFor and AC::UrlFor.Nick Sutterer2011-05-121-0/+6
| |
* | Get around weird missing constant error caused by AS instead of simply ↵José Valim2011-05-111-5/+13
| | | | | | | | raising NameError, closes #477.
* | Warn if we cannot verify CSRF token authenticityJosé Valim2011-05-091-1/+4
| |
* | Merge pull request #280 from jballanc/frozen-string-strip-tagsJosé Valim2011-05-071-1/+1
|\ \ | | | | | | Stripping tags from a frozen string
| * | Fix for stripping tags from frozen strings.Joshua Ballanco2011-04-141-1/+1
| | | | | | | | | | | | This returns behavior under Ruby 1.9 to match Ruby 1.8.
* | | Use .ref instead of .to_sym.José Valim2011-05-072-2/+4
| | |
* | | Revert to old semantics, use available_action? instead of action_method?.José Valim2011-05-061-9/+7
| | |
* | | More updates to ivars list.José Valim2011-05-061-2/+4
| | |
* | | Move variables to underscore format, update protected instance variables list.José Valim2011-05-061-7/+2
| | |
* | | Ensure params wrapper settings are not inherited and calculated each time.José Valim2011-05-061-1/+1
| | |
* | | Update respond_with docs to deal better with collections.José Valim2011-05-051-1/+6
| |/ |/|
* | More performance optimizations.José Valim2011-05-031-13/+17
| |
* | Improve performance for filtered parameters and add tests.José Valim2011-05-033-5/+7
| |
* | Instrumentation should have callbacks.José Valim2011-05-031-4/+4
| |
* | Move most processing to load time for performance and improve test suite.José Valim2011-05-031-25/+49
| |
* | Update CHANGELOG.José Valim2011-05-031-1/+1
| |
* | Fix ParamsWrapper docs errorsSebastian Martinez2011-05-021-2/+2
| |
* | Merge pull request #359 from sikachu/params_wrapper.David Heinemeier Hansson2011-05-022-0/+198
|\ \ | | | | | | Add `ActionController::ParamsWrapper` to wrap JSON parameters into a nested parameters.
| * | Add `ActionController::ParamsWrapper` to wrap parameters into a nested hashPrem Sichanugrist2011-05-032-0/+198
| | | | | | | | | This will allow us to do a rootless JSON/XML request to server.
* | | made a copy-edit pass on the streaming RDocXavier Noria2011-05-021-33/+35
|/ /
* | Streaming docs.José Valim2011-05-011-1/+201
| |
* | Introduce view renderer.José Valim2011-05-011-1/+1
| |
* | Merge branch 'master' of github.com:lifo/docrailsDan Pickett2011-04-291-0/+2
|\ \
| * | document HTML::Selector's :has(string) pseudo classmisfo2011-04-231-0/+2
| | |