Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The patch #22336-134: Move all core Drupal files under a /core folder to improve usability and upgrades leads to the following error during the installation:
'Installing: failed to set database information (database name, user, password were wrong)',
I grepped through project_issue_file_review 6.x-3.x but could find such string...
Comments
Comment #1
scor CreditAttribution: scor commentedmy bad, I was grepping the wrong branch, it comes from ./review/drupal/pifr_drupal.client.inc in 6.x-2.x and 6.x-3.x. What's the relevant version used on d.o?
Comment #2
scor CreditAttribution: scor commentedinstall.php is moved to core/install.php could explain why this is not working...
Comment #3
quicksketchThanks for pushing on these issues scor. I'm commenting here rather than in #22336: Move all core Drupal files under a /core folder to improve usability and upgrades since this specifically relates to testbot.
Perhaps we could include in the patch a few basic redirects (both to assist testbot and existing users)? Maybe in the patch we can set up basic 301 redirect for cron.php, install.php and update.php? Then testbot wouldn't have to work around two different locations until the move is permanently a part of core.
Comment #4
scor CreditAttribution: scor commentedyes, I was thinking of the same @quicksketch, sounds like a good interim solution.
Comment #5
jthorson CreditAttribution: jthorson commentedBased on the /core discussion, can we assume that this was taken care of through #1120030: Support moving of core files to core/ directory?
Comment #6
rfayI think we can call it fixed, although this is a general error that happens when Drupal cannot successfully install.