Problem/Motivation

When running thunder_update_8112() some other update functions are called. Unfortunately the updater does not check if the module of the requested function is enabled. If (i.e.) you don't have thunder_article enabled, the update will fail.

Proposed resolution

Run updates for enabled modules only.

Remaining tasks

Create pull request.

Comments

stBorchert created an issue. See original summary.

stborchert’s picture

Status: Active » Needs review
stborchert’s picture

mtodor’s picture

Copy from GitHub PR

@stBorchert Thank you for reporting problem.

I'm not sure this is correct way to solve this problem. Ignoring some parts of configuration updates could lead to incorrect behaviour of introduced functionality or made changes. Currently we have extensive logging of failed configuration updates and with this change they are silently ignored.

Also we want to have one module where all configuration updates and update hooks will be collected. That means thunder_update_8112() hook has wrong approach.

I can't propose solution like this for following problem, it should not ever happen that update hook is executed and required configuration update definition files are not available for it. I would rather move all configuration update files into profile, where update hook is.

mtodor’s picture

@stBorchert

I have made other PR for this problem: https://github.com/BurdaMagazinOrg/thunder-distribution/pull/525
Can you please take a look and test?

As I have explained the problem is not in Updater functionality, but the wrong approach in update hook, where update hook relies on un-available resources.

mtodor’s picture

mtodor’s picture

Crediting for review.

  • mtodor authored 39aa5b4 on 8.x-2.x
    Issue #2937285 by stBorchert, mtodor, chr.fritsch: thunder_update_8112...
mtodor’s picture

Status: Needs review » Fixed

  • mtodor authored 39aa5b4 on 8.x-3.x
    Issue #2937285 by stBorchert, mtodor, chr.fritsch: thunder_update_8112...

Status: Fixed » Closed (fixed)

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