wp-queue icon indicating copy to clipboard operation
wp-queue copied to clipboard

Matching RedisConnection to ConnectionInterface

Open szepeviktor opened this issue 4 years ago • 0 comments

What do you think about these?

diff --git a/src/WP_Queue/Connections/RedisConnection.php b/src/WP_Queue/Connections/RedisConnection.php
index 346a348..28a72de 100644
--- a/src/WP_Queue/Connections/RedisConnection.php
+++ b/src/WP_Queue/Connections/RedisConnection.php
@@ -16,7 +16,7 @@ class RedisConnection implements ConnectionInterface {
         * @return bool|int
         */
        public function push( Job $job, $delay = 0 ) {
-               //
+               return true;
        }

        /**
@@ -25,7 +25,7 @@ class RedisConnection implements ConnectionInterface {
         * @return bool|Job
         */
        public function pop() {
-               //
+               return true;
        }

        /**
@@ -55,7 +55,7 @@ class RedisConnection implements ConnectionInterface {
         * @return bool
         */
        public function failure( $job, Exception $exception ) {
-               //
+               return true;
        }

        /**
@@ -64,7 +64,7 @@ class RedisConnection implements ConnectionInterface {
         * @return int
         */
        public function jobs() {
-               //
+               return 0;
        }

        /**
@@ -73,7 +73,7 @@ class RedisConnection implements ConnectionInterface {
         * @return int
         */
        public function failed_jobs() {
-               //
+               return 0;
        }

 }

szepeviktor avatar Jul 19 '20 16:07 szepeviktor