8000 [Cache] \Redis::mget() can return bool, Redis6Proxy::mget() doesn't account for that · Issue #52668 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content
[Cache] \Redis::mget() can return bool, Redis6Proxy::mget() doesn't account for that #52668
Closed
@dkarlovi

Description

@dkarlovi

Symfony version(s) affected

6.3.5

Description

Redis::mget can return false when no keys are provided. phpredis/phpredis#1810

This is annoying because the signature says mget(): \Redis|array, but it's actually mget(): \Redis|array|false.

Redis6Proxy::mget has the correct signature, but it doesn't handle the possible false.

How to reproduce

Set up PHPRedis v6 and do Redis6Proxy::mget([])

Possible Solution

Return [] if false.

Additional Context

IMO this is a bug in the module, but it's a bug since 2020.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0