extract setting freight order ids to function

This commit is contained in:
Yisroel Baum 2025-11-16 22:35:14 +02:00
parent 780fb0b769
commit 9cbb481d1b
Signed by: yisroelbaum
GPG key ID: 0FA60884F75520A9

View file

@ -17,7 +17,7 @@ class CreateFreightOrder
CreateFreightOrderRequestDTO $dto, CreateFreightOrderRequestDTO $dto,
): FreightOrder { ): FreightOrder {
$savedFreightOrder = $this->saveFreightOrder($dto); $savedFreightOrder = $this->saveFreightOrder($dto);
$this->updateCarriers( $this->handleCarrierActions(
$dto->carrierIds, $dto->carrierIds,
$savedFreightOrder->getId() $savedFreightOrder->getId()
); );
@ -28,19 +28,26 @@ class CreateFreightOrder
/** /**
* @param int[] $carrierIds * @param int[] $carrierIds
*/ */
private function updateCarriers( private function handleCarrierActions(
array $carrierIds, array $carrierIds,
int $freightOrderId int $freightOrderId
): void { ): void {
foreach ($carrierIds as $carrierId) { foreach ($carrierIds as $carrierId) {
$carrier = $this->carrierRepo->find($carrierId); $this->updateCarrierOrderIds($carrierId, $freightOrderId);
$carrierFreightOrderIds = $carrier->getFreightOrderIds();
$carrierFreightOrderIds[] = $freightOrderId;
$carrier->setFreightOrderIds($carrierFreightOrderIds);
$this->carrierRepo->save($carrier);
} }
} }
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( private function saveFreightOrder(
CreateFreightOrderRequestDTO $dto, CreateFreightOrderRequestDTO $dto,
): FreightOrder { ): FreightOrder {