| Comment | File | Size | Author |
|---|---|---|---|
| #20 | interdiff.txt | 850 bytes | wim leers |
| #20 | smartcache-2560959-20.patch | 74.95 KB | wim leers |
| #19 | interdiff.txt | 1004 bytes | wim leers |
| #19 | smartcache-2560959-19.patch | 74.15 KB | wim leers |
| #17 | interdiff.txt | 681 bytes | wim leers |
| Comment | File | Size | Author |
|---|---|---|---|
| #20 | interdiff.txt | 850 bytes | wim leers |
| #20 | smartcache-2560959-20.patch | 74.95 KB | wim leers |
| #19 | interdiff.txt | 1004 bytes | wim leers |
| #19 | smartcache-2560959-19.patch | 74.15 KB | wim leers |
| #17 | interdiff.txt | 681 bytes | wim leers |
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.