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.
Problem/Motivation
9.4.0 was released June 15th and we're moving forward with 9.4.5 scheduled for a production release in september, so far it works nicely with php 8.1.
patch is ready, why not go straight to 9.4.0 from 9.2.x ?
Proposed resolution
see patch
Remaining tasks
review latest patch and review plan
Should we reroll this also and jam it in? #3236799: Canada.ca flyout menu doesn't work?
User interface changes
Some subtle improvements in claro.
API changes
I don't expect much of any
Data model changes
handled by update.php or drush updb
Comment | File | Size | Author |
---|---|---|---|
#42 | D947-3284468-42.patch | 4.57 KB | smulvih2 |
#41 | D946-3284468-41.patch | 4.57 KB | joseph.olstad |
Comments
Comment #2
joseph.olstadComment #3
smulvih2Nice, thanks for the patch!!! I will start testing this in a sandbox shortly.
Comment #4
joseph.olstadI put some more runs on 9.4.0-beta1 , so far so good.
Less patches required, more issues resolved.
Comment #5
sylus CreditAttribution: sylus commentedComment #6
sylus CreditAttribution: sylus commentedComment #7
joseph.olstadComment #8
joseph.olstadrerolled
Comment #9
joseph.olstadI suggest a 4.4.x branch soon so that we can start putting it through it's paces but as you wish.
it will need a very minor reroll again once 4.3.x is updated to last nights build https://www.drupal.org/project/drupal/releases/9.3.16
Comment #10
joseph.olstadComment #11
joseph.olstadComment #12
joseph.olstad9.4.0 broke this patch:
#1356276-680: Allow profiles to define a base/parent profile
needs a reroll
Comment #13
joseph.olstadrerolled
Comment #14
joseph.olstad9.4.1 was released, here's a reroll.
Comment #15
smulvih2WxT 4.3.x-dev is now on core 9.3.16, updating patch.
I upgraded a sandbox site on the WxT 4.3.x-dev branch using the following in my local composer.json. Changes to drupal/core patches work well, thanks @joseph!!!
Comment #16
joseph.olstad** edit **
comment was not relevant to the 9.4.x upgrade, moving it to another issue.
** end edit **
Comment #17
joseph.olstadWondering on a timeline for the 4.4.x branch? How soon can we expect it?
Just wondering if we should be overriding 4.3.x for our project or wait until 4.4.x is released?
Comment #18
joseph.olstadassuming 4.3.1 is tagged and released soon with D9.3.19, updating the patch here to upgrade from 9.3.19 to 9.4.3
Comment #19
joseph.olstadpatch #18 still applies cleanly against 4.3.2 or current HEAD of 4.3.xreviewing
Comment #20
joseph.olstadre-reviewing this now
Comment #21
joseph.olstadNew patch.
Comment #22
joseph.olstadComment #23
joseph.olstadpatch 20 (should have been labelled 21 sorry) applies cleanly to head and also doing
rm composer.lock; composer install;
it installs cleanly no patch conflicts.The one patch that was removed was fixed 6 months ago in 9.4.x so therefore it is no longer needed.
#3184667: Node form layout looks awkward on wide screens since #3158854
Less and less need for patches, this is good!
Comment #24
smulvih2Great work @joseph, thanks for keeping this updated!
Comment #25
joseph.olstadD9.4.5 version reroll
interdiff is basically one line, instead of 9.4.3 , 9.4.5
Comment #26
joseph.olstad@sylus, can we expect a 9.4.x release soon? just wondering if I should override our composer.json now or wait for a 4.4.x WxT release?
I am being asked this question. D9.4.x is the recommended core .
Comment #27
joseph.olstadlook at adding this patch to the 9.4.x release
#3107212-13: views no results behavior on field level is not translateable
Comment #28
joseph.olstadanother one to look at (possible reroll for this patch if necessary)
#3236799: Canada.ca flyout menu doesn't work?
consider adding this one also
#3150294-109: New translations for moderated nodes are not created in the initial workflow state
Comment #29
joseph.olstadOk new patch. This is for wxt similar to the above patches except I added two core patches mentioned in comment #27 and #29:
Here's the interdiff.
I've basically done something similar in overrides, overridden 4.3.0 with D9.4.5 instead of 9.3.16, with a similar list of patches, so far it's good.
Using PHP 8.1 btw, applied patches for wxt_library and bootstrap for that one, not yet adventurous enough to try wxt 4.3.2, going with core upgrade first.
Comment #30
joseph.olstadComment #31
joseph.olstadComment #32
joseph.olstadD9.4 with PHP 8.1 needs quite a few contrib updates also
simple_sitemap needs the 4.x branch instead of 3.x and/or the patch from this issue:
#3299827: php8.1 compatibility - Passing null to parameter #1 ($string) of type string is deprecated
webform 6.1.x needs this patch #3279930: Deprecated function: strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in webform_requirements()
OR upgrade webform from 6.1x to 6.2.x
to name a few
Comment #33
joseph.olstadoverrode core using wxt 4.3.0 and testing 9.4.5 using php 8.1 , so far only one deprecated warning left to resolve and I'm not sure where it's comming from but there were many fixed by patching a couple contrib modules , I'll update the patch again for this soon.
Comment #34
joseph.olstadwith PHP 8.0 patch 29 should do the job, for PHP 8.1 I'll provide another patch.
Comment #35
joseph.olstadpatches for webform 6.2.x recommended with 9.4.5 and PHP 8.1
Comment #36
joseph.olstadPatch 29 works well with PHP 8.0, D9.4.5
Comment #37
joseph.olstadI recommend people go with PHP 8.0 and 9.4.5, this will allow us to move forward in a timely manner.
PHP 8.0 works nicely no fuss. 9.4.5 also works well. Is there anything else needed before commit?
Patch 29 should be easy enough to apply.
Comment #38
smulvih2@joseph I use the drupalwxt/docker-scaffold setup (9.3.x branch) for all of my projects and that's currently on PHP 8.0.23. I haven't tested core 9.4.x in a while, will test it again this week with the suggested patches you posted above and report back.
Comment #39
joseph.olstadjust as a FYI, I haven't upgraded any sites to 4.3.3 yet because we need 9.4.x and we've overridden 4.3.0 to install D9.4.5
Unfortunately I haven't had time to test 4.3.3. We're basically waiting for 9.4.x to be integrated before upgrading.
I was hoping to be upgraded to the latest wxt however due to the core lagging behind we took matters into our own hands and overrode it. Due to the base project child project setup we have I'd prefer to postpone upgrading our wxt until it contains 9.4.x.
Comment #40
joseph.olstadpatch needs a slight adjustment, D9.4.6 was released
it's a bug fix release, highly recommend using 9.4.6 instead of 9.4.5, I looked at the release notes and it fixes annoyances that I ran into when upgrading to 9.4.5
so upgrading to 9.4.6 from 9.3.21 should be painless.
Comment #41
joseph.olstadnew patch
no interdiff is needed, just changed 9.4.5 to 9.4.6
Comment #42
smulvih2Update patch to bring core to 9.4.7 based on this security advisory.
Comment #43
joseph.olstadThe latest twig/twig requires Drupal 9.4.7 to work properly. I haven't tested 9.3.22 yet though but it's broken on 9.3.21.
#3312270: Twig LoaderError after upgrading Twig to 2.15.3
#3230673-28: [WxT5] compatibility with Layout Builder.
Comment #44
joseph.olstadFYI: 9.4.8 was released.
Comment #45
sylus CreditAttribution: sylus commentedComment #46
sylus CreditAttribution: sylus commentedThank you so much for your work on this and making it easy for me to just update the composer.json.
Once I updated the build system for this new branch it all worked!
https://github.com/drupalwxt/wxt/actions/runs/3216099857/jobs/5257658367
Committed and attributed!
Comment #47
joseph.olstad@sylus thanks for jamming this in and @smulvih2 for testing it out several times.