$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

Comments

damien tournoud’s picture

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 commented
Status: Needs work » Needs review
StatusFileSize
new update-inc-db-prefix.patch3.09 KB

Some leftover debug statement, woops :)

damien tournoud’s picture

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 🇨🇦
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.