Problem/Motivation

Had some talks with @bnjmnm and @phenaproxima about this problem space

In #3284945: Install endpoints that leverage Package Manager + core APIs Project Browser is also firing the PreCreateEvent to check if the system meets all its requirements, but if another subscriber assumes a create is about to happen this could be a problem. In AutoUpdates we are using ReadinessCheckEvent but that is a AutoUpdates class and Project Browser just requires package_manager

In \Drupal\automatic_updates\Form\UpdateReady \Drupal\automatic_updates_extensions\Form\UpdateReady we call our StagedDatabaseUpdateValidator to warn the user about database updates. StagedDatabaseUpdateValidator is validator because it stops Cron updates

In #3293427: Display all projects that will be updated in Extensions confirmation form we will be displaying other dependencies that have been updated

Project browser also will need to display the messages about DB updates and new projects modules that have been updated or added.

Right now we have to build the logic into the form. But we could just fire StatusCheckEvent and display any warning. The forms themselve won't have to have any particular of what things to check.

Proposed resolution

Create StatusCheckEvent that will show warning and errors to the user.

It would be the module developers responsibility to show the warnings and error and to not allow the user to continue if there are errors.

then do #3304417: Deprecate ReadinessCheckEvent in favor of StatusCheckEvent in package manager

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:

Comments

tedbow created an issue. See original summary.

tedbow’s picture

Status: Active » Needs review

I pushed up a very rough start. Setting to Needs Review to get some feedback on the general idea.

tedbow’s picture

I removed all changes unrelated to package_manager

tedbow’s picture

tedbow’s picture

tedbow’s picture

tedbow’s picture

Status: Needs review » Reviewed & tested by the community

all issues resolved

tedbow credited bnjmnm.

tedbow’s picture

  • tedbow committed 963df7b on 8.x-2.x
    Issue #3304367 by tedbow, bnjmnm, phenaproxima, TravisCarden: Add...
tedbow’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

tedbow’s picture

Issue tags: +core-mvp