Follow up for #1951068-59: install.php error reporting is broken

Problem/Motivation

People empty their database when they want to reinstall, but do not copy the default settings.php over the existing one. The error messages that then occur do not point in a helpful direction.

Proposed resolution

enhance install.php error reporting for the case of an existing settings.php pointing to an empty database

Remaining tasks

TBD

User interface changes

No UI changes.

API changes

No API changes anticipated.

Comments

seanr’s picture

seanr
he/him
commented

In my personal opinion, the behavior in this case should simply be to proceed with the installation from the step after the database settings. Maybe a confirmation to proceed, but you should be able to continue the installation from that step.

seanr’s picture

seanr
he/him
commented

And it looks like there's already an issue for it: #1963760: Make sure install.php gives accurate information if Drupal is already installed

Should we mark this one as a duplicate?

greg.1.anderson’s picture

greg.1.anderson commented
Issue summary: View changes

This is a duplicate of the issue mentioned in #2. It would also be nice if Drupal 8 redirected to core/install.php automatically if the database settings were available, but the database was empty.

c.f. https://github.com/pantheon-systems/drops-8/issues/3

greg.1.anderson’s picture

greg.1.anderson commented
Status: Active » Closed (duplicate)

Closing as duplicate of #1963760: Make sure install.php gives accurate information if Drupal is already installed