fix(phpstan): adjust method argument typings for better code readability

Updated method argument typings from `Type $var = null` to `null | Type $var = null` across
multiple files to improve readability and understanding of the code. This change aligns with modern
coding standards and the principle of explicit over implicit.
Affected files:
- `AbstractShippingEntry.php`
- `AbstractShippingType.php`
- `ShippingInterface.php`
- `ShippingTypeInterface.php`
- `Factory.php`
- `ShippingRule.php`
- `Shipping.php`

PHP9 + PHP8.3 null deprecated

src/QUI/ERP/Shipping/Api/AbstractShippingEntry.php

@@ -92,7 +92,7 @@ abstract public function getTitle($Locale = null): string;
      * @param null|QUI\Locale $Locale
      * @return string
      */
-    abstract public function getDescription(QUI\Locale $Locale = null): string;
+    abstract public function getDescription(null | QUI\Locale $Locale = null): string;
 
     /**
      * @return string
@@ -146,7 +146,7 @@ public function isVisible(): bool
      * @return string
      */
     public function getInvoiceInformationText(
-        QUI\ERP\Accounting\Invoice\Invoice|QUI\ERP\Accounting\Invoice\InvoiceTemporary|QUI\ERP\Accounting\Invoice\InvoiceView $Invoice
+        QUI\ERP\Accounting\Invoice\Invoice | QUI\ERP\Accounting\Invoice\InvoiceTemporary | QUI\ERP\Accounting\Invoice\InvoiceView $Invoice
     ): string {
         return '';
     }

src/QUI/ERP/Shipping/Api/AbstractShippingType.php

@@ -30,7 +30,7 @@ public function getType(): string
      * @param QUI\Locale|null $Locale
      * @return array
      */
-    public function toArray(QUI\Locale $Locale = null): array
+    public function toArray(null | QUI\Locale $Locale = null): array
     {
         if ($Locale === null) {
             $Locale = QUI::getLocale();
@@ -46,7 +46,7 @@ public function toArray(QUI\Locale $Locale = null): array
      * @param QUI\Locale|null $Locale
      * @return string
      */
-    abstract public function getTitle(QUI\Locale $Locale = null): string;
+    abstract public function getTitle(null | QUI\Locale $Locale = null): string;
 
     /**
      * @return string

src/QUI/ERP/Shipping/Api/ShippingInterface.php

@@ -18,7 +18,7 @@ interface ShippingInterface
     /**
      * @return int|string
      */
-    public function getId(): int|string;
+    public function getId(): int | string;
 
     /**
      * @param null|QUI\Locale $Locale
@@ -48,7 +48,7 @@ public function getShippingType(): ShippingTypeInterface;
      *
      * @return float|int
      */
-    public function getPrice(): float|int;
+    public function getPrice(): float | int;
 
     /**
      * Return the price display

src/QUI/ERP/Shipping/Api/ShippingTypeInterface.php

@@ -22,7 +22,7 @@ public function getType(): string;
      * @param null|QUI\Locale $Locale
      * @return string
      */
-    public function getTitle(QUI\Locale $Locale = null): string;
+    public function getTitle(null | QUI\Locale $Locale = null): string;
 
     /**
      * @return string
@@ -35,5 +35,5 @@ public function getIcon(): string;
      * @param QUI\Locale|null $Locale
      * @return array
      */
-    public function toArray(QUI\Locale $Locale = null): array;
+    public function toArray(null | QUI\Locale $Locale = null): array;
 }

src/QUI/ERP/Shipping/Rules/Factory.php

@@ -70,6 +70,7 @@ public function createChild(array $data = []): QUI\CRUD\Child
     {
         // filter
         $allowed = array_flip([
+            'active',
             'title',
             'workingTitle',
             'date_from',

src/QUI/ERP/Shipping/Rules/ShippingRule.php

@@ -191,7 +191,7 @@ public function toArray(): array
      * @param Locale|null $Locale
      * @return string
      */
-    public function getTitle(QUI\Locale $Locale = null): string
+    public function getTitle(null | QUI\Locale $Locale = null): string
     {
         if ($Locale === null) {
             $Locale = QUI::getLocale();
@@ -337,7 +337,7 @@ public function canUsedBy(QUI\Interfaces\Users\User $User): bool
      * @param Address|QUI\Users\Address|null $Address
      * @return bool
      */
-    public function canUsedWithAddress(QUI\ERP\Address|QUI\Users\Address $Address = null): bool
+    public function canUsedWithAddress(null | QUI\ERP\Address | QUI\Users\Address $Address = null): bool
     {
         if (!$Address) {
             return false;
@@ -365,7 +365,7 @@ public function canUsedWithAddress(QUI\ERP\Address|QUI\Users\Address $Address =
      * @param QUI\ERP\ErpEntityInterface|null $ErpEntity
      * @return bool
      */
-    public function canUsedIn(QUI\ERP\ErpEntityInterface $ErpEntity = null): bool
+    public function canUsedIn(null | QUI\ERP\ErpEntityInterface $ErpEntity = null): bool
     {
         if (!$this->isValid()) {
             Debug::addLog("{$this->getTitle()} :: is not valid");
@@ -819,7 +819,7 @@ public function getDiscountType(): int
      *
      * @return bool|array
      */
-    public function getUnitTerms(): bool|array
+    public function getUnitTerms(): bool | array
     {
         $unitTerms = $this->getAttribute('unit_terms');
 

src/QUI/ERP/Shipping/Shipping.php

@@ -239,7 +239,7 @@ public function getShippingType(string $shippingType): Api\ShippingTypeInterface
      *
      * @throws Exception
      */
-    public function getShippingEntry(int|string $shippingId): Types\ShippingEntry
+    public function getShippingEntry(int | string $shippingId): Types\ShippingEntry
     {
         try {
             return Factory::getInstance()->getChild($shippingId);
@@ -277,8 +277,10 @@ public function getShippingList(array $queryParams = []): array
      * @param QUI\ERP\ErpEntityInterface|null $Entity - optional
      * @return QUI\ERP\Shipping\Types\ShippingEntry[]
      */
-    public function getUserShipping(User $User = null, QUI\ERP\ErpEntityInterface $Entity = null): array
-    {
+    public function getUserShipping(
+        null | User $User = null,
+        null | QUI\ERP\ErpEntityInterface $Entity = null
+    ): array {
         if ($User === null) {
             $User = QUI::getUserBySession();
         }
@@ -302,8 +304,8 @@ public function getUserShipping(User $User = null, QUI\ERP\ErpEntityInterface $E
      * @param QUI\ERP\ErpEntityInterface $Entity
      * @return PriceFactorInterface|ErpPriceFactor|null
      */
-    public function getShippingPriceFactor(QUI\ERP\ErpEntityInterface $Entity): ErpPriceFactor|PriceFactorInterface|null
-    {
+    public function getShippingPriceFactor(QUI\ERP\ErpEntityInterface $Entity
+    ): ErpPriceFactor | PriceFactorInterface | null {
         $PriceFactors = $Entity->getArticles()->getPriceFactors();
 
         foreach ($PriceFactors as $PriceFactor) {
@@ -321,7 +323,7 @@ public function getShippingPriceFactor(QUI\ERP\ErpEntityInterface $Entity): ErpP
      *
      * @deprecated use getShippingPriceFactor
      */
-    public function getShippingPriceFactorByOrder(AbstractOrder $Order): ErpPriceFactor|PriceFactorInterface|null
+    public function getShippingPriceFactorByOrder(AbstractOrder $Order): ErpPriceFactor | PriceFactorInterface | null
     {
         QUI\System\Log::addNotice(
             'Shipping->getShippingPriceFactorByOrder() is deprecated, use getShippingPriceFactor'
@@ -421,7 +423,7 @@ public function getHost(): string
      */
     public function getShippingByObject(
         QUI\ERP\ErpEntityInterface $Entity
-    ): Types\ShippingEntry|Types\ShippingUnique|null {
+    ): Types\ShippingEntry | Types\ShippingUnique | null {
         $Shipping = null;
         $Delivery = $Entity->getDeliveryAddress();
 
@@ -440,7 +442,7 @@ public function getShippingByObject(
      * @param $orderId
      * @return ShippingEntry|ShippingUnique|null
      */
-    public function getShippingByOrderId($orderId): ShippingEntry|ShippingUnique|null
+    public function getShippingByOrderId($orderId): ShippingEntry | ShippingUnique | null
     {
         try {
             $Order = QUI\ERP\Order\Handler::getInstance()->getOrderById($orderId);
@@ -551,7 +553,7 @@ public function getVat(QUI\ERP\ErpEntityInterface $ErpEntity): mixed
     public function sendStatusChangeNotification(
         ErpEntityInterface $ErpEntity,
         int $statusId,
-        string $message = null
+        null | string $message = null
     ): void {
         $Customer = $ErpEntity->getCustomer();
         $customerEmail = $Customer->getAttribute('email');