Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I'm going to look at the errors related to theme initialization. Hopefully this gets me started down the path of fixing all of this test fails.
Comment | File | Size | Author |
---|---|---|---|
#2 | 1545672_2_kernel.patch | 102.06 KB | cosmicdreams |
Comments
Comment #1
cosmicdreams CreditAttribution: cosmicdreams commentedI was able to fully fix the errors here by doing the following:
BOOM, no exceptions and 5 passed tests.
Comment #2
cosmicdreams CreditAttribution: cosmicdreams commentedI've probably done something silly. Starting with the current kernel branch, I applied the patch from #1183208: Remove variable_get('clean_url') and switch to index.php/path pattern for dirty URL support
Then, I did a find and replace for all $_GET['q'] and replaced it with current_path().
In testing the tests that this issue targets, I found that all of them passed. But that got me to wonder, what else would pass?
So I'm submitting this patch for the testbot to test. Hopefully there will be less that 750 failed tests and less than 16k exceptions.
Comment #3
cosmicdreams CreditAttribution: cosmicdreams commentedOh, the test bot isn't going to test this...I'll post it to the kernel issue.
Comment #4
cosmicdreams CreditAttribution: cosmicdreams commentedIn the recent #1463656: Add a Drupal kernel; leverage HttpFoundation and HttpKernel patch this issue is resolved.