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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joseph.olstad created an issue. See original summary.

joseph.olstad’s picture

smulvih2’s picture

Nice, thanks for the patch!!! I will start testing this in a sandbox shortly.

joseph.olstad’s picture

I put some more runs on 9.4.0-beta1 , so far so good.

Less patches required, more issues resolved.

sylus’s picture

Title: Update to D9 core 9.4.0 » Update to D9 core 9.4.x
sylus’s picture

Version: 4.2.x-dev » 4.3.x-dev
joseph.olstad’s picture

Assigned: Unassigned » joseph.olstad
Status: Needs review » Active
joseph.olstad’s picture

FileSize
4.3 KB

rerolled

joseph.olstad’s picture

I 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

joseph.olstad’s picture

Status: Active » Needs review
joseph.olstad’s picture

Assigned: joseph.olstad » Unassigned
joseph.olstad’s picture

Status: Needs review » Needs work
joseph.olstad’s picture

Status: Needs work » Needs review
FileSize
4.3 KB

rerolled

joseph.olstad’s picture

FileSize
440 bytes
4.3 KB

9.4.1 was released, here's a reroll.

smulvih2’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
3.9 KB

WxT 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!!!

"require": {
  "cweagans/composer-patches": "^1.6.0",
  "drupal/core-composer-scaffold": "*",
  "drupalwxt/wxt": "4.3.x-dev",
  "drush/drush": ">=9.7",
  "oomphinc/composer-installers-extender": "^1.1 || ^2",
  "drupal/core": "9.4.1 as 9.3.16"
},

"patches": {
  "drupal/core": {
    "Enter drupal/core patch #850600 description here": "https://www.drupal.org/files/issues/2022-06-08/D94x-postgresql_ssl_dsn-850600-23.patch",
    "Enter drupal/core patch #1356276 description here": "https://www.drupal.org/files/issues/2022-06-17/1356276-686.patch",
    "2921627 - WCAG required markers on forms": "https://www.drupal.org/files/issues/2022-03-11/2921627-75.patch",
    "3107212 - allow translation of views field empty text": "https://www.drupal.org/files/issues/2021-12-22/drupal-core-9.2.x-li-role-separator-tag-linkGenerator-3236799-11.patch"
  }
},

"patches-ignore": {
  "drupalwxt/wxt": {
    "drupal/core": {
    "Enter drupal/core patch #850600 description here": "https://www.drupal.org/files/issues/postgresql_ssl_dsn-850600-3.patch",
    "Enter drupal/core patch #1356276 description here": "https://www.drupal.org/files/issues/2021-11-18/1356276-675.patch",
    "2921627 - WCAG required markers on forms": "https://www.drupal.org/files/issues/2021-10-13/2921627-71.patch",
    "Enter drupal/core patch #3164210 description here": "https://www.drupal.org/files/issues/2021-05-17/3164210-11.patch",
    "Enter drupal/core patch #3184667 description here": "https://www.drupal.org/files/issues/2021-08-20/3184667-38.patch",
    "Enter drupal/core patch #3236799 description here": "https://www.drupal.org/files/issues/2021-12-22/drupal-core-9.2.x-li-role-separator-tag-linkGenerator-3236799-11.patch",
    "2632132 - Fix seven install": "https://www.drupal.org/files/issues/2019-10-11/Move_Seven_block_config_into_the_Seven_theme-2632132-42.patch",
    "3107212 - allow translation of views field empty text": "https://www.drupal.org/files/issues/2022-03-09/d9-3-x--3107212-13.patch"
  }
},
joseph.olstad’s picture

** edit **
comment was not relevant to the 9.4.x upgrade, moving it to another issue.
** end edit **

joseph.olstad’s picture

Wondering 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?

joseph.olstad’s picture

assuming 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

joseph.olstad’s picture

patch #18 still applies cleanly against 4.3.2 or current HEAD of 4.3.x
reviewing

joseph.olstad’s picture

Assigned: Unassigned » joseph.olstad

re-reviewing this now

joseph.olstad’s picture

New patch.

joseph.olstad’s picture

Status: Reviewed & tested by the community » Needs review
joseph.olstad’s picture

patch 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!

smulvih2’s picture

Great work @joseph, thanks for keeping this updated!

joseph.olstad’s picture

FileSize
4.13 KB

D9.4.5 version reroll

interdiff is basically one line, instead of 9.4.3 , 9.4.5

joseph.olstad’s picture

@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 .

joseph.olstad’s picture

joseph.olstad’s picture

another 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

joseph.olstad’s picture

FileSize
4.57 KB

Ok 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.

diff --git a/composer.json b/composer.json
index 8e084ea..5b440cc 100644
--- a/composer.json
+++ b/composer.json
@@ -208,8 +208,12 @@
                 "https://www.drupal.org/files/issues/2019-01-18/2938129-pagetitle-11.patch",
                 "Enter drupal/core patch #3017710 description here":
                 "https://www.drupal.org/files/issues/2018-12-05/3017710-20.patch",
+                "3016131 - Improved exception message developer experience":
+                "https://www.drupal.org/files/issues/2020-05-26/drupal-3016131-23-useless-exception.patch",
+                "3107212 - Views no results behavior on field level translation fix.":
+                "https://www.drupal.org/files/issues/2022-03-09/d9-4-x--3107212-13.patch",
                 "3150294 - New translations for moderated nodes are not created in the initial workflow state":
-                "https://www.drupal.org/files/issues/2022-01-05/3150294-95.patch",
+                "https://www.drupal.org/files/issues/2022-05-11/3150294-109.patch",
                 "3177182 - Book title in breadcrumb not translated":
                 "https://www.drupal.org/files/issues/2020-10-20/book-breadcrumb_title_xlate-3177182-3.patch",
                 "Enter drupal/core patch #3173808 description here":

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.

joseph.olstad’s picture

Issue summary: View changes
joseph.olstad’s picture

Issue summary: View changes
joseph.olstad’s picture

D9.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

joseph.olstad’s picture

overrode 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.

joseph.olstad’s picture

with PHP 8.0 patch 29 should do the job, for PHP 8.1 I'll provide another patch.

joseph.olstad’s picture

patches for webform 6.2.x recommended with 9.4.5 and PHP 8.1

3279930 - PHP 8.1
Source: https://www.drupal.org/files/issues/2022-05-11/webform-strpos-php81-webform-requirements-3279930-2.patch

3301970 - PHP 8.1 deprecation
Source: https://www.drupal.org/files/issues/2022-08-05/3301970_2.patch

fix 404 not found date-calendar.png
Source: https://www.drupal.org/files/issues/2022-08-11/webform-3303194-02.patch

Enter drupal/webform patch #3205860 description here
Source: https://www.drupal.org/files/issues/2021-03-26/webform_clientside_validation-3205860-2.patch
joseph.olstad’s picture

Patch 29 works well with PHP 8.0, D9.4.5

joseph.olstad’s picture

I 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.

smulvih2’s picture

@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.

joseph.olstad’s picture

just 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.

joseph.olstad’s picture

patch 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.

joseph.olstad’s picture

FileSize
4.57 KB

new patch

no interdiff is needed, just changed 9.4.5 to 9.4.6

smulvih2’s picture

FileSize
4.57 KB

Update patch to bring core to 9.4.7 based on this security advisory.

joseph.olstad’s picture

The 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.

joseph.olstad’s picture

Assigned: joseph.olstad » Unassigned

FYI: 9.4.8 was released.

sylus’s picture

Version: 4.3.x-dev » 4.4.x-dev
sylus’s picture

Status: Needs review » Fixed

Thank 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!

joseph.olstad’s picture

@sylus thanks for jamming this in and @smulvih2 for testing it out several times.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.