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

scor’s picture

scor CreditAttribution: scor commented

my 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?

scor’s picture

scor CreditAttribution: scor commented

install.php is moved to core/install.php could explain why this is not working...

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented

Thanks 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.

scor’s picture

scor CreditAttribution: scor commented

yes, I was thinking of the same @quicksketch, sounds like a good interim solution.

jthorson’s picture

jthorson CreditAttribution: jthorson commented
Version: » 6.x-2.x-dev

Based on the /core discussion, can we assume that this was taken care of through #1120030: Support moving of core files to core/ directory?

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented
Status: Active » Fixed

I think we can call it fixed, although this is a general error that happens when Drupal cannot successfully install.

Status: Fixed » Closed (fixed)

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