Merge pull request #162 from xav-developer/fix/increment-forever

Fix : increment and forever

Author: butschster

CHANGELOG.md

@@ -18,6 +18,8 @@ The format is based on [Keep a Changelog][keepachangelog] and this project adher
 - Tasks were stuck in case of an error, the "release" method did not return them to the queue.
 - The "calculateBackoff" method incorrectly took the index "$job->attempts()"
 - The "withHeader" method of the "\Spiral\RoadRunner\Jobs\Task\WritableHeadersInterface" interface expects the type "string|iterable", "int" is passed
+- Fixed the `$ttl` in the `increment` method
+- The `forever` method passed `$seconds` as 0, `Spiral\RoadRunner\KeyValue` summed `timestamp + 0`, and `forever` was not `forever`
 
 [#147]:https://github.com/roadrunner-php/laravel-bridge/issues/147
 

src/Cache/RoadRunnerStore.php

@@ -60,11 +60,12 @@ public function putMany(array $values, $seconds)
 
     public function increment($key, $value = 1)
     {
-        $data = $this->get($this->prefix . $key);
+        $data = $this->get($key);
 
         return tap(((int) $data) + $value, function ($newValue) use ($key): void {
             $ttl = $this->storage->getTtl($this->prefix . $key);
-            $this->storage->set($key, $newValue, $ttl->diff(new \DateTimeImmutable()));
+
+            $this->put($key, $newValue, ($ttl ? $ttl->diff(new \DateTimeImmutable()) : null));
         });
     }
 
@@ -75,7 +76,7 @@ public function decrement($key, $value = 1)
 
     public function forever($key, $value)
     {
-        return $this->put($key, $value, 0);
+        return $this->put($key, $value, null);
     }
 
     public function forget($key)