mirror/Part-DB.Part-DB-server
1
0
Fork 1
mirror of https://github.com/Part-DB/Part-DB-server.git synced 2025-06-22 18:03:37 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Only try to retrieve the targetLot from DB if the parameter is existing

Browse source 
This fixes an excpetion occuring during withdrawal of parts, when moving is disabled for the lot.
This commit is contained in:
Jan Böhmer 2023-04-16 01:22:58 +02:00
parent 0dcdd252f5
commit 37fb895d67
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/Controller/PartController.php
View file

@ -350,8 +350,10 @@ class PartController extends AbstractController
if($partLot->getPart() !== $part) { if($partLot->getPart() !== $part) {
throw new \RuntimeException("The origin partlot does not belong to the part!"); throw new \RuntimeException("The origin partlot does not belong to the part!");
} }
//Try to determine the target lot (used for move actions)
$targetLot = $em->find(PartLot::class, $request->request->get('target_id')); //Try to determine the target lot (used for move actions), if the parameter is existing
$targetId = $request->request->get('target_id', null);
$targetLot = $targetId ? $em->find(PartLot::class, $targetId) : null;
if ($targetLot && $targetLot->getPart() !== $part) { if ($targetLot && $targetLot->getPart() !== $part) {
throw new \RuntimeException("The target partlot does not belong to the part!"); throw new \RuntimeException("The target partlot does not belong to the part!");
} }