Laravel-debugbar: not support phpredis in Laravel 5.4

Created on 14 Feb 2017  路  10Comments  路  Source: barryvdh/laravel-debugbar

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
stale

Most helpful comment

So, what is the status for now?

All 10 comments

@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.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

lucasdcrk picture lucasdcrk  路  3Comments

MRZ2017 picture MRZ2017  路  3Comments

coderdiaz picture coderdiaz  路  4Comments

hookover picture hookover  路  4Comments

Megachill picture Megachill  路  3Comments