Closed (works as designed)
Project:
Drupal core
Version:
9.4.x-dev
Component:
request processing system
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
1 Sep 2015 at 10:31 UTC
Updated:
27 Nov 2022 at 23:03 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
wim leersComment #4
wim leersThe first hunk in
FormBuildershouldn't have been included; I badly merged the two patches.Comment #6
wim leersComment #8
effulgentsia commentedThis fixes RedirectTest.
Comment #10
wim leersThis fixes the rest, and improves #8.
Comment #12
wim leersHurray! Green!
Opened #2561775: Forms without $form['#action'] set get their action automatically generated based on current path + query args: cacheability metadata is missing to commit the fix developed in this issue. And #2559011: Ensure form tokens are marked max-age=0 was committed in the mean time.
This testing issue has served its purpose.
Comment #13
wim leersNow checking if #2429617-347: Make D8 2x as fast: Dynamic Page Cache: context-dependent page caching (for *all* users!) + #2504139-99: Blocks containing a form include the form action in the cache, so they always submit to the first URL the form was viewed at are green.
Comment #14
wim leersForgot one hunk. That fixes one fail. But the remaining 4 fails in
SearchCommentTestare because we went with a different solution than in #1–#12. Will debug those tomorrow unless someone beats me to it.Comment #17
wim leersComment #19
wim leersComment #20
wim leersComment #25
wim leersThis issue has again served its purpose: #2429617-348: Make D8 2x as fast: Dynamic Page Cache: context-dependent page caching (for *all* users!).
Comment #37
quietone commentedThis was opened to help test and issue that is now closed. Time to close this one.