After enabling the module for the first time, the following is shown on the page:

Warning: md5() expects parameter 1 to be string, array given in Redis_Client::getGlobalPrefix() (line 75 of /var/aegir/platforms/panopoly-7.x-1.30/sites/{SITE}/modules/redis/lib/Redis/Client.php).

Is this perhaps a PHP 5.5 incompatibility?

CommentFileSizeAuthor
#4 i2684303-4.patch669 bytesattiks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NWOM created an issue. See original summary.

NWOM’s picture

NWOM CreditAttribution: NWOM commented

I think I may have found the culprit, but can't say for sure. I have an aegir environment with the database being Mysql. However, in addition to the drupal database, I have another database connection which is configured in the local.settings.php. Is it possible that it isn't expecting an array?

As a workaround I changed Line 71 in /lib/Redis/Client.php from:

if (isset($GLOBALS['db_url'])) {

to

if (!isset($GLOBALS['db_url'])) {

pounard’s picture

pounard
Primary language French
CreditAttribution: pounard commented

If I remember correctly, you are not the first experiencing that, it's due to the fact that your global $databases array might not have a 'default' set or is incomplete.

attiks’s picture

attiks CreditAttribution: attiks at Attiks commented
Status: Active » Needs review
FileSize
i2684303-4.patch669 bytes

Ran into the same problem today, patch checks if it is a string

pounard’s picture

pounard
Primary language French
CreditAttribution: pounard commented
Status: Needs review » Fixed

Patch applied and commited, thanks!

There will be a release today.

  • Pierre.R committed c474200 on 7.x-3.x authored by attiks 
    Issue #2684303 by attiks, NWOM: md5() expects parameter 1 to be string,...
NWOM’s picture

NWOM CreditAttribution: NWOM commented

Oh awesome. Thanks guys! I'll have to give the newest dev a shot then.

Status: Fixed » Closed (fixed)

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