$db_prefix is not automatically converted to the DBTNG configuration array $databases, which should be now that we can have per-db prefixes.

CommentFileSizeAuthor
#2 update-inc-db-prefix.patch3.09 KBStevel
update-inc-db-prefix.patch3.15 KBStevel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Needs review » Needs work

Good catch!

The patch looks good, except this part:

+  if (empty($db_prefix)) die('db_prefix is empty :(');
Stevel’s picture

Stevel CreditAttribution: Stevel commented
Status: Needs work » Needs review
FileSize
update-inc-db-prefix.patch3.09 KB

Some leftover debug statement, woops :)

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Needs review » Reviewed & tested by the community

Looks good, thanks.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -D7 upgrade path

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

Issue tags: +D7 upgrade path

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.