Merge pull request #60 from tuyennn/feature/issue-59-php84

[fix] PHP 8.4 compatibiltiy

Author: tuyennn

Model/ResourceModel/Inventory.php

@@ -53,7 +53,7 @@ public function __construct(
         Manager $moduleManager,
         StockRegistryInterface $stockRegistry,
         Context $context,
-        string $connectionName = null
+        ?string $connectionName = null
     ) {
         parent::__construct($context, $connectionName);
         $this->moduleManager = $moduleManager;

Plugin/Model/ResourceModel/Product/CollectionPlugin.php

@@ -102,7 +102,7 @@ private function applyOutOfStockAtLastOrders(Collection $collection)
     public function beforeAddOrder(
         Collection $subject,
         $attribute,
-        string $dir = Select::SQL_DESC
+        $dir = Select::SQL_DESC
     ): array {
         if (!$subject->getFlag('is_processing')) {
             $result = $this->beforeSetOrder($subject, $attribute, $dir);
@@ -124,7 +124,7 @@ public function aroundAddAttributeToSort(
         Collection $collection,
         callable $proceed,
         string $attribute,
-        string $dir = Collection::SORT_ORDER_ASC
+        $dir = Collection::SORT_ORDER_ASC
     ): Collection {
         if (!$collection->getFlag(sprintf('sorted_by_%s_attribute', $attribute))) {
             $collection->setFlag(sprintf('sorted_by_%s_attribute', $attribute), true);