Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Add keys/values methods to TestSession | Carlos Antonio da Silva | 2013-01-25 | 1 | -0/+8 | |
| | | | | | | | | Bring back the same API we have with Request::Session. | |||||
* | | Integrate Action Pack with Rack 1.5 | Carlos Antonio da Silva | 2013-01-25 | 2 | -4/+18 | |
| | | | | | | | | | | | | All ActionPack and Railties tests are passing. Closes #8891. [Carlos Antonio da Silva + Santiago Pastorino] | |||||
* | | ActionDispatch::Http::UploadedFile is a permitted scalar [Closes #9051] | Xavier Noria | 2013-01-23 | 1 | -2/+5 | |
|/ | ||||||
* | Only check for unpermmited parameters if | Rafael Mendonça França | 2013-01-22 | 1 | -1/+1 | |
| | | | | action_on_unpermitted_parameters is present | |||||
* | avoid creating an object in every call | Xavier Noria | 2013-01-22 | 1 | -1/+2 | |
| | | | | This was a suggestion of @carlosantoniodasilva, thanks! | |||||
* | Refactor grep call to remove .each | Carlos Antonio da Silva | 2013-01-20 | 1 | -1/+1 | |
| | | | | Grep already yields the matching keys to the given block. | |||||
* | Use 1.9 hash style in docs/comments [ci skip] | Carlos Antonio da Silva | 2013-01-20 | 1 | -3/+3 | |
| | ||||||
* | Removing warning: shadowing outer local variable | Arun Agrawal | 2013-01-20 | 1 | -3/+3 | |
| | ||||||
* | strong parameters filters permitted scalars | Xavier Noria | 2013-01-20 | 1 | -22/+92 | |
| | ||||||
* | Restore and adapt the implementation reverted at | Rafael Mendonça França | 2013-01-19 | 2 | -29/+58 | |
| | | | | | | https://github.com/rails/rails/commit/cc1c3c5be061e7572018f734e5239750ab449e3f Now instead of raise, we log by default in development and test | |||||
* | Added ability to raise or log on unpermitted params. | Thomas Drake-Brockman | 2013-01-20 | 2 | -6/+34 | |
| | ||||||
* | Revert "Merge pull request #8989 from robertomiranda/use-rails-4-find-by" | Guillermo Iguaran | 2013-01-18 | 2 | -3/+3 | |
| | | | | | This reverts commit 637a7d9d357a0f3f725b0548282ca8c5e7d4af4a, reversing changes made to 5937bd02dee112646469848d7fe8a8bfcef5b4c1. | |||||
* | User Rails 4 find_by | robertomiranda | 2013-01-18 | 2 | -3/+3 | |
| | ||||||
* | Remove useless || operation | Carlos Antonio da Silva | 2013-01-17 | 1 | -2/+1 | |
| | ||||||
* | Deprecate direct calls to AC::RecordIdentifier.dom_id and dom_class | Carlos Antonio da Silva | 2013-01-16 | 1 | -5/+17 | |
| | | | | Also add some generic tests to ensure they're properly deprecated. | |||||
* | strong parameters exception handling | Brian Alexander | 2013-01-15 | 1 | -6/+0 | |
| | ||||||
* | Revert "log at debug level what line caused the redirect_to" | Carlos Antonio da Silva | 2013-01-15 | 1 | -1/+0 | |
| | | | | | | | | This reverts commit 3fa00070047b5d019d39e691598ee2890283d052. Reason: This message is usually not accurate and annoying: Redirected by ~/.rbenv/versions/1.9.3-p327-perf/lib/ruby/1.9.1/logger.rb:371:in `add'` | |||||
* | Merge pull request #8876 from senny/extract_performance_tests | Guillermo Iguaran | 2013-01-10 | 1 | -3/+0 | |
|\ | | | | | Extract ActionDispatch::PerformanceTest | |||||
| * | extract PerformanceTest into rails-performance_tests gem | Yves Senn | 2013-01-10 | 1 | -3/+0 | |
| | | ||||||
* | | Merge pull request #8821 from jamis/master | Rafael Mendonça França | 2013-01-10 | 1 | -5/+1 | |
|\ \ | |/ |/| | | | | | | | | | Evaluate view_cache_dependencies at the instance level Conflicts: actionpack/lib/action_controller/caching.rb | |||||
| * | evaluate the dependency blocks at the instance level, not class level | Jamis Buck | 2013-01-08 | 1 | -5/+1 | |
| | | ||||||
* | | Fix typo in deprecation warning | Nathaniel Jones | 2013-01-09 | 1 | -1/+1 | |
| | | ||||||
* | | Fix warning: & interpreted as argument prefix | Carlos Antonio da Silva | 2013-01-08 | 1 | -1/+1 | |
| | | ||||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-01-09 | 4 | -10/+10 | |
|\ \ | |/ |/| | | | | | Conflicts: guides/source/getting_started.md | |||||
| * | prefer american spelling of 'behavior' | Gosha Arinich | 2013-01-07 | 1 | -1/+1 | |
| | | ||||||
| * | HTTP 302 means Found, not Moved | Chase DuBois | 2013-01-05 | 1 | -1/+1 | |
| | | ||||||
| * | PUT => PATCH | Akira Matsuda | 2013-01-03 | 1 | -1/+1 | |
| | | ||||||
| * | find_or_create_by is deprecated in AR 4 | Akira Matsuda | 2013-01-02 | 1 | -3/+3 | |
| | | ||||||
| * | Model.scoped is deprecated in favour of Model.all | Akira Matsuda | 2013-01-02 | 1 | -4/+4 | |
| | | ||||||
* | | view_cache_dependency API | Jamis Buck | 2013-01-08 | 1 | -0/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | A declarative API for specifying dependencies that affect template cache digest computation. In your controller, specify any of said dependencies: view_cache_dependency { "phone" if using_phone? } When the block is evaluated, the resulting value is included in the cache digest calculation, allowing you to generate different digests for effectively the same template. (Mostly useful if you're mucking with template load paths.) | |||||
* | | 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 | |
| |