[FSSDK-12610] Fix: PHP 8.4 implicit nullable parameter deprecation warnings (#296)

* [FSSDK-12610] Fix: PHP 8.4 implicit nullable parameter deprecation warnings

Use explicit nullable types (?Type) instead of implicit nullable (Type $param = null)
to resolve deprecation warnings on PHP 8.4.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* [FSSDK-12610] Fix remaining implicit nullable parameters in Validator, VariableTypeUtils, DefaultEventDispatcher, OptimizelyConfigService

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* [FSSDK-12610] Update PHPDoc to reflect explicit nullable parameter types

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Tamara-Barum

src/Optimizely/DecisionService/DecisionService.php

@@ -81,9 +81,9 @@ class DecisionService
      * DecisionService constructor.
      *
      * @param LoggerInterface       $logger
-     * @param UserProfileServiceInterface  $userProfileService
+     * @param ?UserProfileServiceInterface  $userProfileService
      */
-    public function __construct(LoggerInterface $logger, UserProfileServiceInterface $userProfileService = null)
+    public function __construct(LoggerInterface $logger, ?UserProfileServiceInterface $userProfileService = null)
     {
         $this->_logger = $logger;
         $this->_bucketer = new Bucketer($logger);

src/Optimizely/Event/Dispatcher/DefaultEventDispatcher.php

@@ -36,7 +36,7 @@ class DefaultEventDispatcher implements EventDispatcherInterface
      */
     private $httpClient;
 
-    public function __construct(HttpClient $httpClient = null)
+    public function __construct(?HttpClient $httpClient = null)
     {
         $this->httpClient = $httpClient ?: new HttpClient();
     }

src/Optimizely/Optimizely.php

@@ -119,24 +119,24 @@ class Optimizely
      * Optimizely constructor for managing Feature Experimentation PHP projects.
      *
      * @param $datafile string JSON string representing the project.
-     * @param $eventDispatcher EventDispatcherInterface
-     * @param $logger LoggerInterface
-     * @param $errorHandler ErrorHandlerInterface
+     * @param $eventDispatcher ?EventDispatcherInterface
+     * @param $logger ?LoggerInterface
+     * @param $errorHandler ?ErrorHandlerInterface
      * @param $skipJsonValidation boolean representing whether JSON schema validation needs to be performed.
-     * @param $userProfileService UserProfileServiceInterface
-     * @param $configManager ProjectConfigManagerInterface provides ProjectConfig through getConfig method.
-     * @param $notificationCenter NotificationCenter
+     * @param $userProfileService ?UserProfileServiceInterface
+     * @param $configManager ?ProjectConfigManagerInterface provides ProjectConfig through getConfig method.
+     * @param $notificationCenter ?NotificationCenter
      * @param $sdkKey string uniquely identifying the datafile corresponding to project and environment combination. Must provide at least one of datafile or sdkKey.
      */
     public function __construct(
         $datafile,
-        EventDispatcherInterface $eventDispatcher = null,
-        LoggerInterface $logger = null,
-        ErrorHandlerInterface $errorHandler = null,
+        ?EventDispatcherInterface $eventDispatcher = null,
+        ?LoggerInterface $logger = null,
+        ?ErrorHandlerInterface $errorHandler = null,
         $skipJsonValidation = false,
-        UserProfileServiceInterface $userProfileService = null,
-        ProjectConfigManagerInterface $configManager = null,
-        NotificationCenter $notificationCenter = null,
+        ?UserProfileServiceInterface $userProfileService = null,
+        ?ProjectConfigManagerInterface $configManager = null,
+        ?NotificationCenter $notificationCenter = null,
         $sdkKey = null,
         array $defaultDecideOptions = []
     ) {

src/Optimizely/OptimizelyConfig/OptimizelyConfigService.php

@@ -85,7 +85,7 @@ class OptimizelyConfigService
 
     private ProjectConfigInterface $projectConfig;
 
-    public function __construct(ProjectConfigInterface $projectConfig, LoggerInterface $logger = null)
+    public function __construct(ProjectConfigInterface $projectConfig, ?LoggerInterface $logger = null)
     {
         $this->experiments = $projectConfig->getAllExperiments();
         $this->featureFlags = $projectConfig->getFeatureFlags();

src/Optimizely/ProjectConfigManager/HTTPProjectConfigManager.php

@@ -89,9 +89,9 @@ public function __construct(
         $fetchOnInit = true,
         $datafile = null,
         $skipJsonValidation = false,
-        LoggerInterface $logger = null,
-        ErrorHandlerInterface $errorHandler = null,
-        NotificationCenter $notificationCenter = null,
+        ?LoggerInterface $logger = null,
+        ?ErrorHandlerInterface $errorHandler = null,
+        ?NotificationCenter $notificationCenter = null,
         $datafileAccessToken = null
     ) {
         $this->_skipJsonValidation = $skipJsonValidation;

src/Optimizely/Utils/Validator.php

@@ -31,7 +31,7 @@ class Validator
      *
      * @return boolean Representing whether schema is valid or not.
      */
-    public static function validateJsonSchema($datafile, LoggerInterface $logger = null)
+    public static function validateJsonSchema($datafile, ?LoggerInterface $logger = null)
     {
         $data = json_decode($datafile);
 

src/Optimizely/Utils/VariableTypeUtils.php

@@ -24,7 +24,7 @@
 
 class VariableTypeUtils
 {
-    public static function castStringToType($value, $variableType, LoggerInterface $logger = null)
+    public static function castStringToType($value, $variableType, ?LoggerInterface $logger = null)
     {
         if ($variableType == FeatureVariable::STRING_TYPE) {
             return $value;