Version 2.13 as well as 2.x from git (release is missing, so can't select here):

Got this one:
PHP Fatal error: Cannot use object of type Predis\\Response\\Status as array in .../sites/all/modules/redis/lib/Redis/Lock/Backend/Predis.php on line 55

CommentFileSizeAuthor
#9 redis-2507997.patch533 byteskfritsche
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pounard’s picture

Status: Active » Postponed (maintainer needs more info)

Thank you for reporting. Which Predis version are you using ?

I'll try to fix this ASAP, but I need to test with the same version as you are.

geek-merlin’s picture

version sais 1.0.2-dev, file timstamps 2015-01-02,
so this is current dev from https://github.com/nrk/predis

pounard’s picture

glass.dimly’s picture

I'm getting this too, v1.0.2 (2015-xx-xx) of Predis library, 7.x-2.15 version of this module.

glass.dimly’s picture

Status: Postponed (maintainer needs more info) » Active
glass.dimly’s picture

Switching to PhpRedis as a backend is a workaround that fixed this problem for me.

glass.dimly’s picture

I also just pulled down the latest from the predis repo here and replicated the issue: https://github.com/nrk/predis

I replicated this issue by saving a view and I think clearing the cache also triggered this.

It appears to have quuiiite a few changes, even though the version number is unchanged. That is, the version number is the same as my comment #4 but the code is different.

        modified:   predis/.gitignore
	modified:   predis/.travis.yml
	modified:   predis/CHANGELOG.md
	modified:   predis/README.md
	modified:   predis/src/Command/Processor/KeyPrefixProcessor.php
	modified:   predis/src/Connection/AbstractConnection.php
	modified:   predis/src/Connection/Factory.php
	modified:   predis/src/Connection/Parameters.php
	modified:   predis/src/Connection/PhpiredisSocketConnection.php
	modified:   predis/src/Connection/PhpiredisStreamConnection.php
	modified:   predis/src/Connection/StreamConnection.php
	modified:   predis/src/Profile/RedisVersion260.php
	modified:   predis/src/Profile/RedisVersion280.php
	modified:   predis/src/Profile/RedisVersion300.php
	modified:   predis/src/Replication/ReplicationStrategy.php
	modified:   predis/tests/PHPUnit/PredisTestCase.php
	modified:   predis/tests/Predis/Command/Processor/KeyPrefixProcessorTest.php
	modified:   predis/tests/Predis/Connection/CompositeStreamConnectionTest.php
	modified:   predis/tests/Predis/Connection/FactoryTest.php
	modified:   predis/tests/Predis/Connection/ParametersTest.php
	modified:   predis/tests/Predis/Connection/PhpiredisSocketConnectionTest.php
	modified:   predis/tests/Predis/Connection/PhpiredisStreamConnectionTest.php
	modified:   predis/tests/Predis/Connection/StreamConnectionTest.php
	modified:   predis/tests/Predis/Connection/WebdisConnectionTest.php
	modified:   predis/tests/Predis/Profile/RedisUnstableTest.php
	modified:   predis/tests/Predis/Profile/RedisVersion260Test.php
	modified:   predis/tests/Predis/Profile/RedisVersion280Test.php
	modified:   predis/tests/Predis/Profile/RedisVersion300Test.php
	modified:   predis/tests/Predis/Replication/ReplicationStrategyTest.php
pounard’s picture

Thank you very much for the time you spent into this, I hope I'll be able to proceed to some debugging myself soon.

kfritsche’s picture

FileSize
533 bytes

Patch attached for Predis 1.0.3.

Works for us here, but didn't tested downwards compatibility with older Predis versions.

Drupa1ish’s picture

Status: Active » Reviewed & tested by the community

#9 works for me. Thanks!

pounard’s picture

I did not forget you, I still don't get the chance to have the sufficient time to work on Redis' module maintenance, will do ASAP. This bug and associated fix needs to be run through the whole unit tests to ensure we don't forget anywhere it could fail.

Does this happen only when you are using the lock backend and not using the cache backend ?

Drupa1ish’s picture

@pounard Yes, it occurs only with $conf['lock_inc'] = 'sites/all/modules/redis/redis.lock.inc';

With default drupal lock works just fine.