aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
* Split asset url helpers into seperate AssetUrlHelperJoshua Peek2012-10-125-419/+416
|
* 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
* Allow for deep directory path for view templates.Andy Shipman2012-10-111-1/+1
|
* refactor `ActionView::TestCase` internals to track rendered localsYves Senn2012-10-112-20/+33
| | | | | 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-112-9/+18
|
* Merge pull request #7890 from AvnerCohen/new_hash_syntaxVijay Dev2012-10-101-9/+9
|\ | | | | Moving to new hash syntax
| * moving to new hash syntax, for discussion before I take the time on full foldersAvnerCohen2012-10-091-9/+9
| |
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-10-112-2/+2
|\ \ | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/persistence.rb railties/lib/rails/generators/rails/resource_route/resource_route_generator.rb
| * | Closed unclosed <tt>, working on a complete solution for docrails, but ↵AvnerCohen2012-10-081-1/+1
| | | | | | | | | | | | testing push process first
| * | Space between "every" and "time"schneems2012-10-021-1/+1
| | |
* | | Remove expansion configJoshua Peek2012-10-101-2/+0
| | |
* | | Remove old asset tag concatenationJoshua Peek2012-10-104-378/+16
| |/ |/| | | | | Use sprockets, jammit, or some other asset bundler
* | Fix nodoc commentRafael Mendonça França2012-10-071-16/+15
| |
* | More Ruby 1.9 hash syntax.Rafael Mendonça França2012-10-061-52/+47
| |
* | We don't need to check blank? here.Rafael Mendonça França2012-10-061-1/+1
| | | | | | | | | | | | | | | | Also the blank? check introduced a bug. $ rails generate model Foo blank:boolean form_for(Foo.new(:blank => true)) => ArgumentError, "First argument in form cannot contain nil or be empty"
* | Update some code styles.Rafael Mendonça França2012-10-061-50/+48
| | | | | | | | | | * Uses the Ruby 1.9 hash syntax * Avoid escaping " inside string using the %{} syntax
* | Accept :remote as symbol in link_to optionsRiley Lynch2012-10-061-2/+5
| | | | | | | | | | Accept either :remote or 'remote' in both the html_options and (url_)options hash arguments to link_to.
* | 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]
* | Revert "Use flat_map { } instead of map {}.flatten"Santiago Pastorino2012-10-051-2/+2
| | | | | | | | | | | | | | | | | | | | | | This reverts commit abf8de85519141496a6773310964ec03f6106f3f. We should take a deeper look to those cases flat_map doesn't do deep flattening. irb(main):002:0> [[[1,3], [1,2]]].map{|i| i}.flatten => [1, 3, 1, 2] irb(main):003:0> [[[1,3], [1,2]]].flat_map{|i| i} => [[1, 3], [1, 2]]
* | Use flat_map { } instead of map {}.flattenSantiago Pastorino2012-10-051-2/+2
| |
* | config.action_dispatch.rack_cache should set explicitly to enable Rack::CacheGuillermo Iguaran2012-10-041-6/+1
| |
* | 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
* | Merge pull request #7833 from frodsan/extract_ap_pages_actions_cachingRafael Mendonça França2012-10-044-400/+17
|\ \ | | | | | | Extract AP Page and Action caching from Rails
| * | rename page_cache_extension option to default_static_extensionFrancesco Rodriguez2012-10-032-11/+13
| | |
| * | extract AP Page and Action caching to actionpack-deprecated_caching gemFrancesco Rodriguez2012-10-033-399/+14
| | |
* | | Add nodoc to now public fragment_name_with_digestRyan Garver2012-10-041-0/+1
| | |
* | | Merge branch 'master' into feature/public-fragment_name_with_digestRyan Garver2012-10-0214-166/+202
|\| |
| * | Failsafe exception returns text/plain.Steve Klabnik2012-10-011-3/+3
| | | | | | | | | | | | | | | | | | | | | It's best to just return text/plain when something has gone terribly wrong. Fixes #5660.
| * | Merge pull request #7708 from bdurand/optimize_log_subscribersRafael Mendonça França2012-10-012-8/+16
| |\ \ | | | | | | | | 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-302-8/+16
| | | | | | | | | | | | | | | | performing an operations.
| * | | prefix TemplateAssertions ivars (#7459)Yves Senn2012-10-012-23/+23
| | | |
| * | | resource and resources do no longer modify passed optionsYves Senn2012-10-011-2/+2
| | | | | | | | | | | | | | | | this is a patch for #7777.
| * | | 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-288-119/+139
| |\ \ | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/asset_tag_helper.rb
| | * | update image_tag output in examples to actualNihad Abbasov2012-09-251-4/+4
| | | | | | | | | | | | [ci-skip]
| | * | 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
| | | |
| * | | Fix tests broken by adding a new instance variable to view test casesJeremy Kemper2012-09-271-0/+1
| | | |
| * | | 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
| | | |
* | | | Move the CacheHelper#fragment_name_with_digest to be public so custom ↵Ryan Garver2012-09-271-11/+11
|/ / / | | | | | | | | | fragment caching can benefit from it.
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-05 06:13:35 +0000