When redis module is installed, but not configured through settings.php (We have redis on our production box, but I don't have it on a local machine), it spits out the following:

Notice: Undefined index: redis_client_interface in require_once() (line 43 of /Users/tom/workspace/abc/profiles/abc/modules/contrib/redis/redis.autoload.inc). Backtrace:
require_once('/Users/tom/workspace/abc/profiles/abc/modules/contrib/redis/redis.autoload.inc') redis.module:13
include_once('/Users/tom/workspace/abc/profiles/abc/modules/contrib/redis/redis.module') bootstrap.inc:1128
drupal_load('module', 'redis') module.inc:24
module_load_all() common.inc:5029
_drupal_bootstrap_full() bootstrap.inc:2201
drupal_bootstrap(7) index.php:20

CommentFileSizeAuthor
#1 1887544-redis-undefined-warning.patch450 bytesmrfelton

Comments

mrfelton’s picture

mrfelton commented
Status: Active » Needs review
StatusFileSize
new 1887544-redis-undefined-warning.patch450 bytes
pounard’s picture

pounard
Primary language French
commented

Thanks for reporting and thanks for the patch, I'll commit this as soon as I can.

Note that this variable is mandatory, without the Redis module can't guess which backend to use. I'd recommend that you add it anyway in your settings.php file, especially if you're using it for early loaded caches (for example, the page cache or the bootstrap cache that will contain variables).

pounard’s picture

pounard
Primary language French
commented
Status: Needs review » Fixed

Pushed and created the new 7.x-2.0-beta3 release.

Thanks.

Status: Fixed » Closed (fixed)

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

  • Commit da1aba7 on 7.x-2.x, 7.x-2.x-sharding, 7.x-2.x-path by Pierre.R: 
    #1887544 - by mrfelton - Avoid some PHP notices when not using nor...