Fatal error: Cannot redeclare class RedisException in [snip]/lib/Redisent/Redisent.php about php-resque HOT 11 CLOSED

I get that too.

from php-resque.

Seems it doesn't like the friendly wrapped Exception handler in lib/Redisent/Redisent.php. If you comment out lines 15 and 16, then it'll run the demos without issue.

from php-resque.

I don't suppose you guys are using php-resque in combination with some sort of framework/library that's already creating its own RedisException class?

I really hate how Redisent hasn't named the class Redisent_Exception - I might look at changing that.

from php-resque.

I have PHPRedis installed: https://github.com/nicolasff/phpredis

from php-resque.

Ah cool - I guess I'll rename the RedisException class to Redisent_Exception or similar then.

from php-resque.

we're using redisent for other stuff so was looking at setting php-resque to use the external lib. A quick look at commits seems to show that he has integrated most of the fixes you have implemented so might do a pull request for an exception rename.

from php-resque.

Any update on this?

from php-resque.

Hello,

There was indeed, this commit should fix it:

1a5564d

The issue was with the autoloader loading it twice.

Hope thats helpful.

On 1 Nov 2011, at 22:18, cj wrote:

Any update on this?

Reply to this email directly or view it on GitHub:
#19 (comment)

Scott Wilcox

@dordotky | [email protected] | http://dor.ky
+44 (0) 7538 842418 | +1 (646) 827-0580

from php-resque.

I don't think this has been fixed yet, as long as /lib/Redisent/Redisent.php defines a RedisException class, it will still conflict with php-resque and break PHP. I'm all for renaming RedisException to RedisentException (or something similar), but at the very least can we wrap the class definition in a class_exists? i.e,

if (! class_exists('RedisException')) {
  class RedisException extends Exception {
  }
}

from php-resque.

Still having the same issue here. I am also using php-redis in addition

from php-resque.

Above changes were merged in, which should fix this. Thanks!

from php-resque.