Problem/Motivation

Trying to update the module using drupal 10.1 in drush updb step:

> [notice] Update started: simplenews_update_8203
> [error] Call to undefined method Drupal\Core\Config\ConfigManager::findConfigEntityDependents()
> [error] Update failed: simplenews_update_8203
[error] Update aborted by: simplenews_update_8203
[error] Finished performing updates.

Steps to reproduce

Proposed resolution

Replace with findConfigEntityDependencies as said in https://www.drupal.org/node/3225357

doing so clears the error

CommentFileSizeAuthor
#7 update-3387265-7.patch764 bytesaleix
#2 update-3387265-2.patch763 bytesaleix

Comments

aleix created an issue. See original summary.

aleix’s picture

StatusFileSize
new763 bytes
adamps’s picture

You are right.

However that update hook is from 2019, for updating from 8.x-1.x to 8.x-2.x. The 1.x version doesn't support Drupal 9, and it became unsupported along with Drupal 8, nearly 2 years ago.😃

The recommended/supported sequence is to upgrade from simplenews 8.x-1.x to 8.x-2.x then afterwards upgrade from Drupal 8 to Drupal 9.

If we commit this patch, then we would likely break sites that follow the supported/recommended sequence.

aleix’s picture

Ups, that's true Adam. but I hit this issue when upgrading from 7 to 10 version... In this case I cannot run drush updb .

adamps’s picture

Version: 3.x-dev » 4.x-dev
Status: Active » Needs review

OK then I agree we should fix it.

adamps’s picture

Status: Needs review » Needs work

The patch fails to apply

aleix’s picture

Status: Needs work » Needs review
StatusFileSize
new764 bytes

I am sorry, it was for 3.x (the version I am using...) , uploading now for 4.x.

  • AdamPS committed 959cae0f on 4.x authored by aleix
    Issue #3387265 by aleix, AdamPS: Call to undefined method Drupal\Core\...
adamps’s picture

Status: Needs review » Fixed

Great thanks

Status: Fixed » Closed (fixed)

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