refactor
This commit is contained in:
parent
652c23e330
commit
326dd0dfa9
1 changed files with 15 additions and 15 deletions
|
|
@ -16,29 +16,29 @@ class CreateFreightOrder
|
||||||
public function execute(
|
public function execute(
|
||||||
CreateFreightOrderRequestDTO $dto,
|
CreateFreightOrderRequestDTO $dto,
|
||||||
): FreightOrder {
|
): FreightOrder {
|
||||||
$savedFreightOrder = $this->constructFreightOrder($dto);
|
$savedFreightOrder = $this->saveFreightOrder($dto);
|
||||||
foreach ($dto->carrierIds as $carrierId) {
|
$this->updateCarriers(
|
||||||
$this->updateCarrier(
|
$dto->carrierIds,
|
||||||
$carrierId,
|
$savedFreightOrder->getId()
|
||||||
$savedFreightOrder->getId()
|
);
|
||||||
);
|
|
||||||
}
|
|
||||||
|
|
||||||
return $savedFreightOrder;
|
return $savedFreightOrder;
|
||||||
}
|
}
|
||||||
|
|
||||||
private function updateCarrier(
|
private function updateCarriers(
|
||||||
int $carrierId,
|
array $carrierIds,
|
||||||
int $freightOrderId
|
int $freightOrderId
|
||||||
): void {
|
): void {
|
||||||
$carrier = $this->carrierRepo->find($carrierId);
|
foreach ($carrierIds as $carrierId) {
|
||||||
$carrierFreightOrderIds = $carrier->getFreightOrderIds();
|
$carrier = $this->carrierRepo->find($carrierId);
|
||||||
$carrierFreightOrderIds[] = $freightOrderId;
|
$carrierFreightOrderIds = $carrier->getFreightOrderIds();
|
||||||
$carrier->setFreightOrderIds($carrierFreightOrderIds);
|
$carrierFreightOrderIds[] = $freightOrderId;
|
||||||
$this->carrierRepo->save($carrier);
|
$carrier->setFreightOrderIds($carrierFreightOrderIds);
|
||||||
|
$this->carrierRepo->save($carrier);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private function constructFreightOrder(
|
private function saveFreightOrder(
|
||||||
CreateFreightOrderRequestDTO $dto,
|
CreateFreightOrderRequestDTO $dto,
|
||||||
): FreightOrder {
|
): FreightOrder {
|
||||||
return $this->freightOrderRepo->save(
|
return $this->freightOrderRepo->save(
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue