Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Allow #permit to take its list of permitted parameters as an array | David Heinemeier Hansson | 2012-10-31 | 1 | -1/+1 |
| | |||||
* | Make caller attribute in deprecation methods optional | Alexey Gaziev | 2012-10-30 | 1 | -2/+2 |
| | |||||
* | Provide a call stack for deprecation warnings where needed. | Nikita Afanasenko | 2012-10-29 | 2 | -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 actionpack | Yves Senn | 2012-10-27 | 1 | -1/+1 |
| | |||||
* | remain backwards compatible, exception can be constructed with no arguments | Aaron Patterson | 2012-10-25 | 1 | -1/+3 |
| | |||||
* | give access to the original exception raised in case of a parse error | Aaron Patterson | 2012-10-25 | 1 | -0/+7 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-10-21 | 1 | -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 Rodriguez | 2012-10-18 | 1 | -6/+31 |
| | | |||||
| * | remove some non-breaking spaces [ci skip] | Francesco Rodriguez | 2012-10-18 | 1 | -13/+13 |
| | | |||||
* | | There is already a Set of non-hidden action_names lying around. | thedarkone | 2012-10-18 | 1 | -8/+2 |
| | | |||||
* | | Permit string and float values in the multiparameter attributes | Rafael Mendonça França | 2012-10-18 | 1 | -1/+1 |
| | | |||||
* | | :fire: Rails asset id support | Joshua Peek | 2012-10-15 | 1 | -1/+0 |
| | | |||||
* | | warning removed: shadowing outer local variable - value | Arun Agrawal | 2012-10-12 | 1 | -2/+2 |
| | | |||||
* | | update AC::Parameters#permit documentation [ci skip] | Francesco Rodriguez | 2012-10-11 | 1 | -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 Pastorino | 2012-10-12 | 1 | -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 locals | Yves Senn | 2012-10-11 | 1 | -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 #3675 | Yves Senn | 2012-10-11 | 1 | -4/+10 |
| | | |||||
* | | moving to new hash syntax, for discussion before I take the time on full folders | AvnerCohen | 2012-10-09 | 1 | -9/+9 |
|/ | |||||
* | can't pass :locals to #assert_template without a view test case. Closes #3415 | Yves Senn | 2012-10-06 | 1 | -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 match | Rafael Mendonça França | 2012-10-06 | 1 | -1/+1 |
| | | | | | | to the block. Thanks @tenderlove | ||||
* | Allow parameter filters to match multi-parameter attributes | Rafael Mendonça França | 2012-10-05 | 1 | -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_permit | David Heinemeier Hansson | 2012-10-04 | 1 | -0/+5 |
|\ | | | | | ActionController::Parameters#permit! is recursive | ||||
| * | ActionController::Parameters#permit! is recursive | Brendan Loudermilk | 2012-10-04 | 1 | -0/+5 |
| | | |||||
* | | Do not set removed option | Rafael Mendonça França | 2012-10-04 | 1 | -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_extension | Francesco Rodriguez | 2012-10-03 | 1 | -10/+12 |
| | |||||
* | extract AP Page and Action caching to actionpack-deprecated_caching gem | Francesco Rodriguez | 2012-10-03 | 3 | -399/+14 |
| | |||||
* | Merge pull request #7708 from bdurand/optimize_log_subscribers | Rafael Mendonça França | 2012-10-01 | 1 | -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 Durand | 2012-09-30 | 1 | -6/+11 |
| | | | | | | | | performing an operations. | ||||
* | | prefix TemplateAssertions ivars (#7459) | Yves Senn | 2012-10-01 | 1 | -20/+20 |
| | | |||||
* | | Whitespaces :scissors: [ci skip] | Rafael Mendonça França | 2012-09-30 | 1 | -1/+1 |
| | | |||||
* | | `assert_template` no more passing with what ever string that matches. | Hugo Roque | 2012-09-29 | 1 | -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/docrails | Vijay Dev | 2012-09-28 | 7 | -117/+137 |
|\ \ | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/asset_tag_helper.rb | ||||
| * | | update AC::ConditionalGet documentation [ci skip] | Francesco Rodriguez | 2012-09-22 | 1 | -27/+31 |
| | | | |||||
| * | | update AC::Caching documentation [ci skip] | Francesco Rodriguez | 2012-09-22 | 5 | -87/+103 |
| | | | |||||
| * | | remove AC::Parameters reference [ci skip] | Francesco Rodriguez | 2012-09-21 | 1 | -1/+1 |
| | | | |||||
| * | | fix typos in AC::StrongParameters documentation [ci skip] | Francesco Rodriguez | 2012-09-21 | 1 | -2/+2 |
| | | | |||||
* | | | Is not needed to wrap name inside an array here | Santiago Pastorino | 2012-09-27 | 1 | -1/+1 |
| | | | |||||
* | | | Change key not found to param not found | Santiago Pastorino | 2012-09-27 | 1 | -6/+6 |
| | | | |||||
* | | | Also includes ConstantLookup dependency for controller and mailer tests | Andy Lindeman | 2012-09-26 | 1 | -0/+1 |
| | | | |||||
* | | | Allow strings in the controller test describe blocks | Mike Moore | 2012-09-24 | 1 | -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 #7743 | Mike Moore | 2012-09-24 | 1 | -3/+5 |
|/ / | | | | | | | Add tests for controller tests using the minitest spec DSL. | ||||
* | | fix typos and improve AC::StrongParameters documentation [ci skip] | Francesco Rodriguez | 2012-09-20 | 1 | -6/+8 |
| | | |||||
* | | update AC::StrongParameters documentation | Francesco Rodriguez | 2012-09-19 | 1 | -8/+53 |
| | | |||||
* | | define permitted? method instead of use an alias to fix rdoc | Francesco Rodriguez | 2012-09-19 | 1 | -2/+12 |
| | | |||||
* | | update AC::Parameters documentation [ci skip] | Francesco Rodriguez | 2012-09-19 | 1 | -1/+149 |
| | | |||||
* | | update AC::ParameterMissing documentation [ci skip] | Francesco Rodriguez | 2012-09-19 | 1 | -2/+9 |
| | | |||||
* | | update AC::ParamsWrapper documentation [ci skip] | Francesco Rodriguez | 2012-09-19 | 1 | -3/+4 |
|/ | |||||
* | Set AC::Parameters.permit_all_parameters explicitly to false when ↵ | Guillermo Iguaran | 2012-09-19 | 1 | -1/+1 |
| | | | | config.action_controller.permit_all_parameters is not present | ||||
* | fix shadowing outer local variable warning | Sergey Nartimov | 2012-09-19 | 1 | -3/+3 |
| | |||||
* | Merge pull request #7251 from rails/integrate-strong_parameters | David Heinemeier Hansson | 2012-09-18 | 4 | -7/+132 |
|\ | | | | | Integrate strong_parameters in Rails 4 |