Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Revert "unpermitted params" exception -- it's just not going to work. See ↵ | David Heinemeier Hansson | 2013-01-08 | 2 | -59/+11 |
| | | | | the discussion on https://github.com/rails/strong_parameters/pull/75. | ||||
* | Never treat action or controller as unpermitted params | David Heinemeier Hansson | 2013-01-08 | 1 | -6/+15 |
| | |||||
* | improve StrongParameters documentation [ci skip] | Francesco Rodriguez | 2013-01-07 | 1 | -8/+7 |
| | |||||
* | Reduce number of Strings a bit | Akira Matsuda | 2013-01-07 | 1 | -1/+1 |
| | |||||
* | Namespace HashWithIndifferentAccess | Akira Matsuda | 2013-01-07 | 1 | -1/+1 |
| | |||||
* | These are already required through AS/rails | Akira Matsuda | 2013-01-06 | 3 | -4/+0 |
| | | | | | | | * core_ext/object/blank * concern * core_ext/class/attribute * deprecation | ||||
* | Wrong copy and paste :bomb: | Rafael Mendonça França | 2013-01-05 | 1 | -1/+1 |
| | | | | [ci skip] | ||||
* | Add documentation to raise_on_unpermitted_parameters option | Rafael Mendonça França | 2013-01-05 | 1 | -2/+17 |
| | | | | [ci skip] | ||||
* | Rename the configuration to raise_on_unpermitted_parameters | Rafael Mendonça França | 2013-01-05 | 2 | -11/+13 |
| | | | | Also changed the exception to UnpermittedParameters | ||||
* | Ensure that raise_on_unexpected_params configuration will work | Rafael Mendonça França | 2013-01-05 | 1 | -9/+7 |
| | |||||
* | Allow developers to enable raising of exception when unexpected params are ↵ | Thomas Drake-Brockman | 2013-01-05 | 2 | -6/+31 |
| | | | | provided. | ||||
* | Change docs to use update instead of update_attributes | Amparo Luna + Guillermo Iguaran | 2013-01-03 | 1 | -2/+2 |
| | |||||
* | Inherit from MiniTest::Unit::TestCase instead of MiniTest::Spec | Rafael Mendonça França | 2012-12-31 | 1 | -7/+0 |
| | |||||
* | charset should not be appended for `head` responses | Yves Senn | 2012-12-31 | 1 | -0/+1 |
| | | | | | | | 1) Failure: test_head_created_with_image_png_content_type(RenderTest) [test/controller/render_test.rb:1238]: Expected: "image/png" Actual: "image/png; charset=utf-8" | ||||
* | "warning: ambiguous first argument; put parentheses or even spaces" | Akira Matsuda | 2012-12-24 | 1 | -1/+1 |
| | |||||
* | Refactoring the token_and_options method to fix bugs | Kurtis Rainbolt-Greene | 2012-12-15 | 1 | -11/+27 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding a test for the equal trun bug Adding a test for the after equal trunc bug Adding a test for the slash bug Adding a test for the slash quote bug Adding a helper method for creating a sample request object with token Writing a method to create params array from raw params Writing a method to rewrite param values in the params Writing a method to get the token params from an authorization value Refactoring the token_and_options method to fix bugs Removing unnessecary test A constant for this shared regex seemed appropriate Wanting to split up this logic Adding small documentation pieces | ||||
* | Add missing require. | Rafael Mendonça França | 2012-12-13 | 1 | -0/+1 |
| | | | | | I was trying to use those files without Rails and that require was missing. | ||||
* | Bring back helpers_path attr accessor | Carlos Antonio da Silva | 2012-12-13 | 1 | -1/+1 |
| | | | | | This is apparently used by the railtie to setup the app helpers paths correctly between initializers. I'll need to check it further. | ||||
* | Refactor helpers code in Action Pack a bit | Carlos Antonio da Silva | 2012-12-13 | 2 | -7/+5 |
| | | | | | | | | * Avoid calling class_eval when not needed * Remove helpers_path attr accessor, it's defined as a class attribute a few lines later * Avoid creating extra arrays when finding helpers, use flat_map and sort! * Remove not required refer variable when redirecting :back | ||||
* | Remove not used variable warnings from AM and AC | Carlos Antonio da Silva | 2012-12-11 | 1 | -1/+1 |
| | |||||
* | use `_action` instead of `_filter` callbacks | Francesco Rodriguez | 2012-12-07 | 1 | -6/+6 |
| | |||||
* | update documentation and code to use _action callbacks | Francesco Rodriguez | 2012-12-07 | 2 | -7/+7 |
| | |||||
* | Merge pull request #8404 from freegenie/filter_redirects | Andrew White | 2012-12-05 | 1 | -1/+1 |
|\ | |||||
| * | Adding filter capability to ActionController logs | Fabrizio Regini | 2012-12-05 | 1 | -1/+1 |
| | | |||||
* | | Replace comments' non-breaking spaces with spaces | claudiob | 2012-12-04 | 2 | -3/+3 |
|/ | | | | | | | | | | Sometimes, on Mac OS X, programmers accidentally press Option+Space rather than just Space and don’t see the difference. The problem is that Option+Space writes a non-breaking space (0XA0) rather than a normal space (0x20). This commit removes all the non-breaking spaces inadvertently introduced in the comments of the code. | ||||
* | Fix a typo in ActionController::Caching | Jean Boussier | 2012-12-03 | 1 | -2/+1 |
| | |||||
* | Cleans and removes useless 'Examples' tag [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -2/+0 |
| | |||||
* | Revert "Remove trailing whitespaces" | Florent Guilleux | 2012-12-01 | 1 | -1/+1 |
| | | | | This reverts commit 90c887fa7d0c454b7533e208daefc342dea4d5f3. | ||||
* | Remove trailing whitespaces | Florent Guilleux | 2012-12-01 | 1 | -1/+1 |
| | |||||
* | hash filters should be accessed with symbols or strings | Francesco Rodriguez | 2012-11-30 | 1 | -3/+5 |
| | |||||
* | Remove observers and sweepers | Rafael Mendonça França | 2012-11-28 | 2 | -121/+2 |
| | | | | | | | | They was extracted from a plugin. See https://github.com/rails/rails-observers [Rafael Mendonça França + Steve Klabnik] | ||||
* | Fix Strong Parameters docs. | Uģis Ozols | 2012-11-27 | 1 | -2/+1 |
| | | | | | It's only possible to inherit from ActiveRecord::Base and not include it. | ||||
* | Remove warning of circular require | Rafael Mendonça França | 2012-11-27 | 1 | -1/+1 |
| | | | | | | | | | This file was using mime_types before load the Mime::Type class. When trying to register first Mime::Type it load mime_type that loads mime_types in the end. Requiring mime_type ensure that we have the class definition and the mime types | ||||
* | Accept symbols as #send_data :disposition value | Elia Schito | 2012-11-27 | 1 | -0/+1 |
| | |||||
* | assert_template: validating option keys | Roberto Soares | 2012-11-25 | 1 | -0/+2 |
| | |||||
* | `assert_template` fails with empty string. | Roberto Soares | 2012-11-23 | 1 | -8/+4 |
| | |||||
* | `assert_template` fails with empty string when a template has been rendered | Roberto Soares | 2012-11-23 | 1 | -4/+8 |
| | | | | | | | | For instance, it prevents false positive in this case: file = nil get :index assert_template("#{file}") | ||||
* | Merge pull request #8183 from jcoglan/objectless_sessions | Jon Leighton | 2012-11-21 | 1 | -1/+2 |
|\ | | | | | Store FlashHashes in the session as plain hashes | ||||
| * | Store FlashHashes in the session as plain hashes rather than custom objects ↵ | James Coglan | 2012-11-09 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | with unstable class names and instance variables. Refactor FlashHash to take values for its ivars in the constructor, to pretty up FlashHash.from_session_value. Remove stale comment on FlashHash: it is no longer Marshaled in the session so we can change its implementation. Remove blank lines I introduced in controller/test_case.rb. Unit tests for FlashHash#to_session_value. Put in a compatibility layer to accept FlashHash serializations from Rails 3.0+. Test that Rails 3.2 session flashes are correctly converted to the new format. Remove code path for processing Rails 3.0 FlashHashes since they can no longer deserialize. Fix session['flash'] deletion condition: it will never be empty?, it will either be nil or a hash with 'discard' and 'flashes' keys. | ||||
* | | Merge pull request #8112 from rails/encrypted_cookies | Santiago Pastorino | 2012-11-15 | 2 | -7/+7 |
|\ \ | | | | | | | Encrypted cookies | ||||
| * | | Allow users to change the default salt if they want, shouldn't be necessary | Santiago Pastorino | 2012-11-03 | 1 | -2/+3 |
| | | | |||||
| * | | Use derived keys everywhere, http_authentication was missing it | Santiago Pastorino | 2012-11-03 | 1 | -3/+2 |
| | | | |||||
| * | | Sign cookies using key deriver | Santiago Pastorino | 2012-11-03 | 1 | -4/+4 |
| |/ | |||||
* | | lazily calculate name in the options object | Aaron Patterson | 2012-11-13 | 1 | -14/+18 |
| | | |||||
* | | move include calculation to include method on the options object | Aaron Patterson | 2012-11-13 | 1 | -9/+24 |
| | | |||||
* | | model name is lazily evaluated in the options object | Aaron Patterson | 2012-11-13 | 1 | -27/+37 |
| | | |||||
* | | move the controller class to the options object | Aaron Patterson | 2012-11-13 | 1 | -9/+12 |
| | | |||||
* | | attribute_names will always return a list, so just use any? | Aaron Patterson | 2012-11-13 | 1 | -1/+1 |
| | | |||||
* | | wrap up hash conversion in the constructor | Aaron Patterson | 2012-11-13 | 1 | -5/+5 |
| | | |||||
* | | use the options object in the wrapper defaults | Aaron Patterson | 2012-11-13 | 1 | -13/+12 |
| | |