I had a settings.php where both $db_url and $db_prefix is an array. Results were not nice.

CommentFileSizeAuthor
#4 installdbarrayfix_0.patch1.71 KBchx
installdbarrayfix.patch1.82 KBchx

Comments

chx’s picture

chx commented

Matt asked whether the other values will be validated. No, that's not the goal of this patch. If you are writing DB arrays in your settings.php then you are advanced and you can take of yourself and your setup. But still you need your database loaded and without this patch it won't happen.

drumm’s picture

drumm
he/him
Location NY, US
commented

Is it possible to see the configuration form with these values set?

chx’s picture

chx commented

Nope, it won't display a config form because it can connect to the DB.

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new installdbarrayfix_0.patch1.71 KB

Noone tested besides me but I did and because the changes are so small, I dare to RTBC this. No changes to previous version just rerolled to HEAD.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)