Fatal error: Cannot redeclare class RedisException in [snip]/lib/Redisent/Redisent.php

[ssx]

When downloading and running the demo's against the latest version of redis, I get the above titled exception.

Any ideas?

[Bandit]

I get that too.

[ssx]

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.

[chrisboulton]

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.

[Bandit]

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

[chrisboulton]

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

[eby]

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.

[cj]

Any update on this?

[ssx]

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 | scott@dor.ky | http://dor.ky
+44 (0) 7538 842418 | +1 (646) 827-0580

[stevencorona]

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 {
  }
}

[ghost]

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

[chrisboulton]

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