aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_controller
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-11-0316-79/+79
|\ | | | | | | | | | | | | Conflicts: actionpack/lib/action_controller/metal/mime_responds.rb activerecord/lib/active_record/attribute_methods.rb guides/source/working_with_javascript_in_rails.md
| * Revert "dont encourage AC::Parameters#permit_all_parameters usage [ci skip]"Vijay Dev2012-11-031-1/+21
| | | | | | | | | | | | | | | | This reverts commit cd17ee5598411728747011566abf6779166be9d3. Reason: Let's note that this is discouraged, not remove the docs [ci skip]
| * dont encourage AC::Parameters#permit_all_parameters usage [ci skip]Francesco Rodriguez2012-10-281-21/+1
| |
| * Multiple changes to 1,9 hash syntaxAvnerCohen2012-10-2715-77/+77
| |
| * fix StrongParameters example [ci skip]Francesco Rodriguez2012-10-211-1/+1
| |
* | Allow #permit to take its list of permitted parameters as an arrayDavid Heinemeier Hansson2012-10-311-1/+1
| |
* | Make caller attribute in deprecation methods optionalAlexey Gaziev2012-10-301-2/+2
| |
* | Provide a call stack for deprecation warnings where needed.Nikita Afanasenko2012-10-292-5/+5
| | | | | | | | It's sometimes hard to quickly find where deprecated call was performed, especially in case of migrating between Rails versions. So this is an attempt to improve the call stack part of the warning message by providing caller explicitly.
* | cleanup, remove trailing whitespace within actionpackYves Senn2012-10-271-1/+1
| |
* | remain backwards compatible, exception can be constructed with no argumentsAaron Patterson2012-10-251-1/+3
| |
* | give access to the original exception raised in case of a parse errorAaron Patterson2012-10-251-0/+7
|/
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-10-211-17/+42
|\ | | | | | | | | | | Conflicts: activesupport/lib/active_support/core_ext/hash/slice.rb guides/source/active_support_core_extensions.md
| * add a nested attributes example into Strong Parameters documentation [ci skip]Francesco Rodriguez2012-10-181-6/+31
| |
| * remove some non-breaking spaces [ci skip]Francesco Rodriguez2012-10-181-13/+13
| |
* | There is already a Set of non-hidden action_names lying around.thedarkone2012-10-181-8/+2
| |
* | Permit string and float values in the multiparameter attributesRafael Mendonça França2012-10-181-1/+1
| |
* | :fire: Rails asset id supportJoshua Peek2012-10-151-1/+0
| |
* | warning removed: shadowing outer local variable - valueArun Agrawal2012-10-121-2/+2
| |
* | update AC::Parameters#permit documentation [ci skip]Francesco Rodriguez2012-10-111-0/+22
| | | | | | | | bdd105d changes the behaviour of AC::Parameters#permit.
* | When executing permit with just a key that points to a hash, DO NOT allow ↵Santiago Pastorino2012-10-121-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | all the hash params.require(:person).permit(:projects_attributes) was returning => {"projects_attributes"=>{"0"=>{"name"=>"Project 1"}}} When should return => {} You should be doing ... params.require(:person).permit(projects_attributes: :name) to get just the projects attributes you want to allow
* | refactor `ActionView::TestCase` internals to track rendered localsYves Senn2012-10-111-9/+4
| | | | | | | | | | this refactoring extracts the semi complex data structure of rendered locals per view into into a separate class
* | recognizes when a partial was rendered twice. Closes #3675Yves Senn2012-10-111-4/+10
| |
* | moving to new hash syntax, for discussion before I take the time on full foldersAvnerCohen2012-10-091-9/+9
|/
* can't pass :locals to #assert_template without a view test case. Closes #3415Yves Senn2012-10-061-3/+7
| | | | | | | | | | | the documentation on #assert_template states that the :locals option is only available in view test cases: # In a view test case, you can also assert that specific locals are passed # to partials: I added a warning when it's passed in an inapropriate context to prevent a NoMethodError.
* We don't need the each call since grep take a block and yield each matchRafael Mendonça França2012-10-061-1/+1
| | | | | | to the block. Thanks @tenderlove
* Allow parameter filters to match multi-parameter attributesRafael Mendonça França2012-10-051-0/+1
| | | | | | | This will make easier to permit date/time attributes generated by helpers like date_select. [Sven Schwyn + Rafael Mendonça França]
* Merge pull request #7847 from bloudermilk/recursive_permitDavid Heinemeier Hansson2012-10-041-0/+5
|\ | | | | ActionController::Parameters#permit! is recursive
| * ActionController::Parameters#permit! is recursiveBrendan Loudermilk2012-10-041-0/+5
| |
* | Do not set removed optionRafael Mendonça França2012-10-041-1/+0
|/ | | | | Sine page caching was remove from the core we don't have the page_cache_directory anymore
* rename page_cache_extension option to default_static_extensionFrancesco Rodriguez2012-10-031-10/+12
|
* extract AP Page and Action caching to actionpack-deprecated_caching gemFrancesco Rodriguez2012-10-033-399/+14
|
* Merge pull request #7708 from bdurand/optimize_log_subscribersRafael Mendonça França2012-10-011-6/+11
|\ | | | | Optimize log subscribers to check if the log level is sufficient
| * Optimize log subscribers to check if the log level is sufficient before ↵Brian Durand2012-09-301-6/+11
| | | | | | | | performing an operations.
* | prefix TemplateAssertions ivars (#7459)Yves Senn2012-10-011-20/+20
| |
* | Whitespaces :scissors: [ci skip]Rafael Mendonça França2012-09-301-1/+1
| |
* | `assert_template` no more passing with what ever string that matches.Hugo Roque2012-09-291-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Given Im rendering an template `/layout/hello.html.erb`, assert_template was passing with any string that matches. This behavior allowed false passing like: assert_template "layout" assert_template "out/hello" Now the passing possibilities are: assert_template "layout/hello" assert_template "hello" fixing assert_template bug when template matches expected, but not ends with Cherry Pick Merge: Fixes issue #3849 assert_template false positive taking redundant test off prevening incorrect assert_template when rendering with repeated names in path updating CHANGELOG with bugfix: assert_template false passing
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-09-287-117/+137
|\ \ | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/asset_tag_helper.rb
| * | update AC::ConditionalGet documentation [ci skip]Francesco Rodriguez2012-09-221-27/+31
| | |
| * | update AC::Caching documentation [ci skip]Francesco Rodriguez2012-09-225-87/+103
| | |
| * | remove AC::Parameters reference [ci skip]Francesco Rodriguez2012-09-211-1/+1
| | |
| * | fix typos in AC::StrongParameters documentation [ci skip]Francesco Rodriguez2012-09-211-2/+2
| | |
* | | Is not needed to wrap name inside an array hereSantiago Pastorino2012-09-271-1/+1
| | |
* | | Change key not found to param not foundSantiago Pastorino2012-09-271-6/+6
| | |
* | | Also includes ConstantLookup dependency for controller and mailer testsAndy Lindeman2012-09-261-0/+1
| | |
* | | Allow strings in the controller test describe blocksMike Moore2012-09-241-0/+1
| | | | | | | | | | | | | | | Allow controller tests using the spec DSL to match strings. Add test coverage for the register_spec_type calls.
* | | Support controller tests using spec DSL - fixes #7743Mike Moore2012-09-241-3/+5
|/ / | | | | | | Add tests for controller tests using the minitest spec DSL.
* | fix typos and improve AC::StrongParameters documentation [ci skip]Francesco Rodriguez2012-09-201-6/+8
| |
* | update AC::StrongParameters documentationFrancesco Rodriguez2012-09-191-8/+53
| |
* | define permitted? method instead of use an alias to fix rdocFrancesco Rodriguez2012-09-191-2/+12
| |
* | update AC::Parameters documentation [ci skip]Francesco Rodriguez2012-09-191-1/+149
| |