chore: Cleanup and strict types

Author: keelerm84

README.md

@@ -6,9 +6,9 @@
 
 This library provides a Redis-backed data source for the [LaunchDarkly PHP SDK](https://github.com/launchdarkly/php-server-sdk), replacing the default behavior of querying the LaunchDarkly service endpoints. The underlying Redis client implementation is the [`phpredis`](https://github.com/phpredis/phpredis) extension. If you want to use the Predis package instead, see https://github.com/launchdarkly/php-server-sdk-redis-predis.
 
-The minimum version of the LaunchDarkly PHP SDK for use with this library is 4.0.0. In earlier versions of the SDK, the Redis integrations were bundled in the main SDK package.
+The minimum version of the LaunchDarkly PHP SDK for use with this library is 6.4.0.
 
-The minimum PHP version is 7.3.
+The minimum PHP version is 8.1.
 
 For more information, see [our SDK documentation](https://docs.launchdarkly.com/sdk/features/storing-data).
 
@@ -27,15 +27,13 @@ php composer.phar install launchdarkly/server-sdk-redis-phpredis --save
 3. In your SDK configuration code, configure the Redis integration:
 
 ```php
-    $fr = LaunchDarkly\Integrations\PHPRedis::featureRequester([
-        "prefix" => "my-key-prefix"
-    ]);
-    $config = [ "feature_requester" => $fr ];
+    $fr = LaunchDarkly\Integrations\PHPRedis::featureRequester(
+        $redisClient, ["prefix" => "my-key-prefix"]
+    );
+    $config = ["feature_requester" => $fr];
     $client = new LDClient("sdk_key", $config);
 ```
 
-By default, the store will try to connect to a local Redis instance on port 6379. You may specify an alternate configuration as described in the API documentation for `PHPRedis::featureRequester`. Make sure the `prefix` option corresponds to the key prefix that is being used by the Relay Proxy.
-
 ## About LaunchDarkly
 
 * LaunchDarkly is a continuous delivery platform that provides feature flags as a service and allows developers to iterate quickly and safely. We allow you to easily flag your features and manage them from the LaunchDarkly dashboard.  With LaunchDarkly, you can:

src/LaunchDarkly/Impl/Integrations/PHPRedisFeatureRequester.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace LaunchDarkly\Impl\Integrations;
 
 use LaunchDarkly\Integrations;
@@ -35,7 +37,13 @@ protected function readItemString(string $namespace, string $key): ?string
 
     protected function readItemStringList(string $namespace): ?array
     {
+        /** @var ?array<string, string> */
         $raw = $this->client->hgetall("$this->prefix:$namespace");
-        return $raw ? array_values($raw) : null;
+
+        if ($raw === null) {
+            return null;
+        }
+
+        return array_values($raw);
     }
 }

src/LaunchDarkly/Integrations/PHPRedis.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace LaunchDarkly\Integrations;
 
 use LaunchDarkly\Impl\Integrations;
@@ -21,8 +23,8 @@ class PHPRedis
      * To use this method, you must have installed the `phpredis` extension. After calling this
      * method, store its return value in the `feature_requester` property of your client configuration:
      *
-     *     $fr = LaunchDarkly\Integrations\PHPRedis::featureRequester([ "redis_prefix" => "env1" ]);
-     *     $config = [ "feature_requester" => $fr ];
+     *     $fr = LaunchDarkly\Integrations\PHPRedis::featureRequester(["prefix" => "env1"]);
+     *     $config = ["feature_requester" => $fr];
      *     $client = new LDClient("sdk_key", $config);
      *
      * For more about using LaunchDarkly with databases, see the

tests/Impl/Integrations/PHPRedisBigSegmentsStoreTest.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace LaunchDarkly\Impl\Integrations\Tests\Impl\Integrations;
 
 use Exception;

tests/Impl/Integrations/PHPRedisFeatureRequesterTest.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace LaunchDarkly\Impl\Integrations\Tests\Impl\Integrations;
 
 use LaunchDarkly\Impl\Model\FeatureFlag;