Problem/Motivation

It was pointed out in #3240971: Support install profiles/distributions that we failed hard if \Drupal\automatic_updates\Updater::getCorePackageName fails.

We should at least create a validator that call this to ensure there is no exception.

Proposed resolution

For general instructions see the parent issue: #3230045: [Meta] Create validation listeners

The subscriber should subscribe to events `\Drupal\automatic_updates\AutomaticUpdatesEvents::READINESS_CHECK` and `\Drupal\automatic_updates\AutomaticUpdatesEvents::PRE_START`
it should add a Error validation result because this will fail an update

Issue fork automatic_updates-3241105

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

tedbow created an issue. See original summary.

tedbow opened merge request !68

phenaproxima made their first commit to this issue’s fork.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs review
tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Needs review » Reviewed & tested by the community

RTBC!

  • phenaproxima committed 9480391 on 8.x-2.x authored by tedbow 
    Issue #3241105 by phenaproxima, tedbow: Ensure core package is validated...
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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