Rename Redis class to RedisService (#631) (#632)

.. to avoid class collision with native Redis

Author: mischabraam

cli/ValetPlus/DriverConfigurator.php

@@ -19,20 +19,20 @@ class DriverConfigurator
     protected $files;
     /** @var Site */
     protected $site;
-    /** @var Redis */
+    /** @var RedisService */
     protected $redis;
 
     /**
      * @param CommandLine $cli
      * @param Filesystem $files
      * @param Site $site
-     * @param Redis $redis
+     * @param RedisService $redis
      */
     public function __construct(
         CommandLine $cli,
         Filesystem $files,
         Site $site,
-        Redis $redis
+        RedisService $redis
     ) {
         $this->cli   = $cli;
         $this->files = $files;

cli/ValetPlus/Extended/Status.php

@@ -12,7 +12,7 @@
 use WeProvide\ValetPlus\Mailhog;
 use WeProvide\ValetPlus\Mysql;
 use WeProvide\ValetPlus\Rabbitmq;
-use WeProvide\ValetPlus\Redis;
+use WeProvide\ValetPlus\RedisService;
 use WeProvide\ValetPlus\Varnish;
 
 class Status extends ValetStatus
@@ -23,7 +23,7 @@ class Status extends ValetStatus
     protected $mailhog;
     /** @var Varnish */
     protected $varnish;
-    /** @var Redis */
+    /** @var RedisService */
     protected $redis;
     /** @var Rabbitmq */
     protected $rabbitmq;
@@ -36,7 +36,7 @@ class Status extends ValetStatus
      * @param Mysql $mysql
      * @param Mailhog $mailhog
      * @param Varnish $varnish
-     * @param Redis $redis
+     * @param RedisService $redis
      * @param Rabbitmq $rabbitmq
      */
     public function __construct(
@@ -47,7 +47,7 @@ public function __construct(
         Mysql $mysql,
         Mailhog $mailhog,
         Varnish $varnish,
-        Redis $redis,
+        RedisService $redis,
         Rabbitmq $rabbitmq
     ) {
         parent::__construct($config, $brew, $cli, $files);

cli/ValetPlus/Redis.php cli/ValetPlus/RedisService.phpcli/ValetPlus/Redis.php renamed to cli/ValetPlus/RedisService.php

@@ -6,7 +6,10 @@
 
 use function Valet\info;
 
-class Redis extends AbstractService
+/**
+ * We name this class RedisService to avoid class collision when `shivammathur/extensions/redis@8.1` is installed.
+ */
+class RedisService extends AbstractService
 {
     /** @var string */
     protected const SERVICE_NAME = 'redis';

cli/includes/facades.php

@@ -41,7 +41,7 @@ class Elasticsearch extends ValetPlusFacade
 class Varnish extends ValetPlusFacade
 {
 }
-class Redis extends ValetPlusFacade
+class RedisService extends ValetPlusFacade
 {
 }
 class Rabbitmq extends ValetPlusFacade

cli/valet.php

@@ -120,7 +120,7 @@
                 Mysql::restart();
                 Mailhog::restart();
                 Varnish::restart();
-                Redis::restart();
+                RedisService::restart();
                 Rabbitmq::restart();
                 break;
             case 'mysql':
@@ -136,7 +136,7 @@
 
                 return info('Varnish has been started.');
             case 'redis':
-                Redis::restart();
+                RedisService::restart();
 
                 return info('Redis has been started.');
             case 'rabbitmq':
@@ -158,7 +158,7 @@
                 Mysql::restart();
                 Mailhog::restart();
                 Varnish::restart();
-                Redis::restart();
+                RedisService::restart();
                 Rabbitmq::restart();
                 break;
             case 'mysql':
@@ -174,7 +174,7 @@
 
                 return info('Varnish has been restarted.');
             case 'redis':
-                Redis::restart();
+                RedisService::restart();
 
                 return info('Redis has been restarted.');
             case 'rabbitmq':
@@ -196,7 +196,7 @@
                 Mysql::stop();
                 Mailhog::stop();
                 Varnish::stop();
-                Redis::stop();
+                RedisService::stop();
                 Rabbitmq::stop();
                 break;
             case 'mysql':
@@ -212,7 +212,7 @@
 
                 return info('Varnish has been stopped.');
             case 'redis':
-                Redis::stop();
+                RedisService::stop();
 
                 return info('Redis has been stopped.');
             case 'rabbitmq':
@@ -247,7 +247,7 @@
             info('Removing varnish...');
             Varnish::uninstall();
             info('Removing redis...');
-            Redis::uninstall();
+            RedisService::uninstall();
             info('Removing rabbitmq...');
             Rabbitmq::uninstall();
             info('Removing elasticsearch...');
@@ -348,21 +348,21 @@
 
             switch ($mode) {
                 case 'install':
-                    Redis::install();
+                    RedisService::install();
 
                     break;
                 case 'enable':
                 case 'on':
-                    Redis::enable();
+                    RedisService::enable();
 
                     break;
                 case 'disable':
                 case 'off':
-                    Redis::disable();
+                    RedisService::disable();
 
                     break;
                 case 'uninstall':
-                    Redis::uninstall();
+                    RedisService::uninstall();
 
                     break;
             }