diff --git a/src/FreightOrder/UseCases/CreateFreightOrder.php b/src/FreightOrder/UseCases/CreateFreightOrder.php index ed87f7d..2a46f35 100644 --- a/src/FreightOrder/UseCases/CreateFreightOrder.php +++ b/src/FreightOrder/UseCases/CreateFreightOrder.php @@ -17,7 +17,7 @@ class CreateFreightOrder CreateFreightOrderRequestDTO $dto, ): FreightOrder { $savedFreightOrder = $this->saveFreightOrder($dto); - $this->updateCarriers( + $this->handleCarrierActions( $dto->carrierIds, $savedFreightOrder->getId() ); @@ -28,19 +28,26 @@ class CreateFreightOrder /** * @param int[] $carrierIds */ - private function updateCarriers( + private function handleCarrierActions( array $carrierIds, int $freightOrderId ): void { foreach ($carrierIds as $carrierId) { - $carrier = $this->carrierRepo->find($carrierId); - $carrierFreightOrderIds = $carrier->getFreightOrderIds(); - $carrierFreightOrderIds[] = $freightOrderId; - $carrier->setFreightOrderIds($carrierFreightOrderIds); - $this->carrierRepo->save($carrier); + $this->updateCarrierOrderIds($carrierId, $freightOrderId); } } + private function updateCarrierOrderIds( + int $carrierId, + int $freightOrderId + ): void { + $carrier = $this->carrierRepo->find($carrierId); + $carrierFreightOrderIds = $carrier->getFreightOrderIds(); + $carrierFreightOrderIds[] = $freightOrderId; + $carrier->setFreightOrderIds($carrierFreightOrderIds); + $this->carrierRepo->save($carrier); + } + private function saveFreightOrder( CreateFreightOrderRequestDTO $dto, ): FreightOrder {