INT-286: Add support for Magento 2.4.9 and PHP 8.4

Add support for Magento 2.4.9 and PHP 8.4

Author: Rayn93

.github/workflows/ci.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php: ['8.3']
+        php: ['8.4']
         deps: ['stable']
 
     steps:

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 ## Unreleased
+### Add
+- Add support for Magento v2.4.9
+- Add support for PHP 8.4
+
 ### Update
 - Update WebComponents library to version 5.1.7
 - Add template to suggest element

composer.json

@@ -10,7 +10,7 @@
     }
   ],
   "require": {
-    "php": "~8.1.0||~8.2.0||~8.3.0",
+    "php": "~8.1.0||~8.2.0||~8.3.0||~8.4.0",
     "omikron/factfinder-communication-sdk": "^0.9.9",
     "magento/framework": "^103.0.7",
     "magento/module-catalog": "^104.0.7",
@@ -25,7 +25,7 @@
   "require-dev": {
     "pdepend/pdepend": "^2.16",
     "phpmd/phpmd" : "^2.15",
-    "phpunit/phpunit": "~9.5.0",
+    "phpunit/phpunit": "^9.6",
     "magento/magento-coding-standard": "*",
     "friendsofphp/php-cs-fixer": "^3.58"
   },

src/Console/Command/Export.php

@@ -56,7 +56,7 @@ protected function configure(): void
     /**
      * {@inheritdoc}
      */
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->state->setAreaCode('frontend');
         $storeIds = $this->getStoreIds((int) $input->getOption('store'));

src/Exception/ExportPreviewValidationException.php

@@ -8,7 +8,7 @@
 
 class ExportPreviewValidationException extends \Exception
 {
-    public function __construct($message = '', $code = 0, Throwable $previous = null)
+    public function __construct($message = '', $code = 0, ?Throwable $previous = null)
     {
         parent::__construct(sprintf('Product will not be exported. Reason: %s', $message), $code, $previous);
     }

src/Exception/ResponseException.php

@@ -6,7 +6,7 @@
 
 class ResponseException extends \RuntimeException
 {
-    public function __construct(string $message = '', int $code = 0, \Throwable $previous = null) // phpcs:ignore
+    public function __construct(string $message = '', int $code = 0, ?\Throwable $previous = null) // phpcs:ignore
     {
         parent::__construct($message ?: 'Response body was empty', $code, $previous);
     }

src/Model/Adminhtml/System/Config/Backend/Feed/Frequency.php

@@ -28,8 +28,8 @@ public function __construct(
         ScopeConfigInterface $config,
         TypeListInterface $cacheTypeList,
         WriterInterface $configWriter,
-        AbstractResource $resource = null,
-        AbstractDb $resourceCollection = null,
+        ?AbstractResource $resource = null,
+        ?AbstractDb $resourceCollection = null,
         array $data = []
     ) {
         parent::__construct($context, $registry, $config, $cacheTypeList, $resource, $resourceCollection, $data);

src/Model/Api/CredentialsFactory.php

@@ -19,7 +19,7 @@ public function __construct(ObjectManagerInterface $objectManager, AuthConfig $a
         $this->authConfig    = $authConfig;
     }
 
-    public function create(array $authData = null)
+    public function create(?array $authData = null)
     {
         return $this->objectManager->create(Credentials::class, $authData ??
               [

src/Model/Config/CmsConfig.php

@@ -17,7 +17,7 @@ public function __construct(ScopeConfigInterface $scopeConfig)
         $this->scopeConfig = $scopeConfig;
     }
 
-    public function getCmsBlacklist(int $scopeCode = null): array
+    public function getCmsBlacklist(?int $scopeCode = null): array
     {
         $pages = (string) $this->scopeConfig->getValue(self::PATH_DISABLE_CMS_PAGES, 'store', $scopeCode);
         return array_filter(explode(',', $pages));

src/Model/Config/CommunicationConfig.php

@@ -31,7 +31,7 @@ public function __construct(ScopeConfigInterface $scopeConfig, UrlInterface $url
         $this->urlBuilder  = $urlBuilder;
     }
 
-    public function getChannel(int $scopeId = null): string
+    public function getChannel(?int $scopeId = null): string
     {
         return (string) $this->scopeConfig->getValue(self::PATH_CHANNEL, ScopeInterface::SCOPE_STORES, $scopeId);
     }
@@ -41,12 +41,12 @@ public function getAddress(): string
         return (string) $this->scopeConfig->getValue(self::PATH_ADDRESS, ScopeInterface::SCOPE_STORES);
     }
 
-    public function isChannelEnabled(int $scopeId = null): bool
+    public function isChannelEnabled(?int $scopeId = null): bool
     {
         return $this->scopeConfig->isSetFlag(self::PATH_IS_ENABLED, ScopeInterface::SCOPE_STORES, $scopeId);
     }
 
-    public function isPushImportEnabled(int $scopeId = null): bool
+    public function isPushImportEnabled(?int $scopeId = null): bool
     {
         return $this->scopeConfig->isSetFlag(self::PATH_DATA_TRANSFER_IMPORT, ScopeInterface::SCOPE_STORES, $scopeId);
     }