LaravelDebugbar.php
if (is_a($client, 'Illuminate\Redis\Connections\PredisConnection', false)) {
$client = $client->client();
}
$storage = new RedisStorage($client);
When configuring redis.client = 'phpredis'
Argument 1 passed to DebugBar\Storage\RedisStorage::__construct() must be an instance of Predis\Client, instance of Illuminate\Redis\Connections\PhpRedisConnection given
@barryvdh Can you help me to repair it ?
Can you make a PR that checks for the type of Redis?
RedisStorage (maximebf/debugbar) need a instance of Predis\Client
If use phpredis of PHP extension , I can not fix it in laravel-debugbar
Ah I see. Can't you just use the filesystem?
We could otherwise make a Storage implementation based on phpredis, or adapt the other one.
In a production environment We use the phpredis
Laravel5.4 already supports predis / phpredis, if maximebf/debugbar is compatible with both ways. That's better :)
Ok can you make a PR there? (And also, don't use this in production)
So, what is the status for now?
@barryvdh now, Is there any intention to be compatible with phpredis ?
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If this issue is still present on the latest version of this library on supported Laravel versions, please let us know by replying to this issue so we can investigate further.
Thank you for your contribution! Apologies for any delayed response on our side.
Most helpful comment
So, what is the status for now?