PhpStan fix.

baraja-core · Feb 24, 2022 · 7a55c80 · 7a55c80
1 parent d6d6616
commit 7a55c80
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/Repository/OrderRepository.php b/src/Repository/OrderRepository.php
@@ -156,7 +156,6 @@ public function getOrderForCheckPayment(?string $currencyCode = null): array
 				->setParameter('currencyCode', $currencyCode);
 		}
 
-		/** @phpstan-ignore-next-line */
 		return $qb->getQuery()->getResult();
 	}
 

diff --git a/src/Status/OrderStatusManager.php b/src/Status/OrderStatusManager.php
@@ -184,11 +184,15 @@ public function setStatus(OrderInterface $order, OrderStatus|string $status, boo
 			$status->isCreateInvoice()
 			|| $status->getCode() === OrderStatus::STATUS_PAID
 			|| (
-				$status->getCode() === OrderStatus::STATUS_DONE
+				$this->invoiceManager !== null
+				&& $status->getCode() === OrderStatus::STATUS_DONE
 				&& PHP_SAPI !== 'cli'
 				&& $this->invoiceManager->isInvoice($order) === false
 			)
 		) {
+			if ($this->invoiceManager === null) {
+				throw new \LogicException('Invoice manager does not exist, but it is mandatory.');
+			}
 			$attachments[] = $this->invoiceManager->getInvoicePath($this->createInvoice($order));
 		}
-Original file line number
+Diff line change
@@ Expand Up @@
     				->setParameter('currencyCode', $currencyCode);
     		}
-    		/** @phpstan-ignore-next-line */
     		return $qb->getQuery()->getResult();
     	}
@@ Expand Down @@