Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix: forward RedisValue methods hidden in redis types by overloads #2026

Merged
merged 5 commits into from
Mar 5, 2018
Merged

Fix: forward RedisValue methods hidden in redis types by overloads #2026

merged 5 commits into from
Mar 5, 2018

Commits on Jan 9, 2018

  1. Fix RedisHash.remove(). 

    The snipped `m_db.getAsHash("test_hash").remove();` caused the following error at runtime:
object.Exception@../../.dub/packages/vibe-d-0.8.2/vibe-d/redis/vibe/db/redis/redis.d(1370): ERR wrong number of arguments for 'hdel' command
    @Geoffrey-A
    Geoffrey-A committed Jan 9, 2018
    Configuration menu
    Copy the full SHA
    7bce92f View commit details
    Browse the repository at this point in the history

  2. Fix RedisZSet.remove(). 

    The snipped `m_db.getAsZSet("test_zset").remove();` caused the following error at runtime:
object.Exception@../../.dub/packages/vibe-d-0.8.2/vibe-d/redis/vibe/db/redis/redis.d(1370): ERR wrong number of arguments for 'zrem' command
    @Geoffrey-A
    Geoffrey-A committed Jan 9, 2018
    Configuration menu
    Copy the full SHA
    d823f63 View commit details
    Browse the repository at this point in the history

Commits on Jan 16, 2018

  1. Forwarding remove() methods: do not lose information of whether the r… 

    …emoval succeeded or not.
    @Geoffrey-A
    Geoffrey-A committed Jan 16, 2018
    Configuration menu
    Copy the full SHA
    90e5662 View commit details
    Browse the repository at this point in the history

Commits on Jan 19, 2018

  1. Remove unnecessary parenthesis.

    @Geoffrey-A
    Geoffrey-A committed Jan 19, 2018
    Configuration menu
    Copy the full SHA
    d2254fc View commit details
    Browse the repository at this point in the history

  2. Explicitly forward RedisHash.exists() to RedisValue.

    @Geoffrey-A
    Geoffrey-A committed Jan 19, 2018
    Configuration menu
    Copy the full SHA
    d2ca321 View commit details
    Browse the repository at this point in the history