groove/Part-DB-server
1
0
Fork 0
mirror of https://github.com/Part-DB/Part-DB-server.git synced 2026-03-01 12:59:36 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

Fix 304 response body, parse_url safety, and location/stock consistency

Browse source 
- Use empty Response instead of JsonResponse(null) for 304 Not Modified
  to avoid sending "null" as response body
- Guard parse_url() result with is_string() since it can return false
  for malformed URLs
- Move storage location tracking inside the availability check so
  expired and unknown-quantity lots don't contribute locations
This commit is contained in:
Sebastian Almberg 2026-02-08 10:46:53 +01:00
parent 6422fa62d1
commit 5a19a56a45
2 changed files with 9 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/Controller/KiCadApiController.php
View file

@ -97,7 +97,7 @@ class KiCadApiController extends AbstractController
$etag = '"' . md5(json_encode($data)) . '"';
if ($request->headers->get('If-None-Match') === $etag) {
return new JsonResponse(null, Response::HTTP_NOT_MODIFIED);
return new Response('', Response::HTTP_NOT_MODIFIED);
}
$response = new JsonResponse($data);

15
src/Services/EDA/KiCadHelper.php
View file

@ -294,15 +294,16 @@ class KiCadHelper
}
}
//Add stock quantity and storage locations
//Add stock quantity and storage locations (only count non-expired lots with known quantity)
$totalStock = 0;
$locations = [];
foreach ($part->getPartLots() as $lot) {
if (!$lot->isInstockUnknown() && $lot->isExpired() !== true) {
$isAvailable = !$lot->isInstockUnknown() && $lot->isExpired() !== true;
if ($isAvailable) {
$totalStock += $lot->getAmount();
}
if ($lot->getAmount() > 0 && $lot->getStorageLocation() !== null) {
$locations[] = $lot->getStorageLocation()->getName();
if ($lot->getAmount() > 0 && $lot->getStorageLocation() !== null) {
$locations[] = $lot->getStorageLocation()->getName();
}
}
}
$result['fields']['Stock'] = $this->createField($totalStock);
@ -443,8 +444,8 @@ class KiCadHelper
if ($firstPdf === null) {
$extension = $attachment->getExtension();
if ($extension === null && $attachment->hasExternal()) {
$urlPath = parse_url($attachment->getExternalPath(), PHP_URL_PATH) ?? '';
$extension = strtolower(pathinfo($urlPath, PATHINFO_EXTENSION));
$urlPath = parse_url($attachment->getExternalPath(), PHP_URL_PATH);
$extension = is_string($urlPath) ? strtolower(pathinfo($urlPath, PATHINFO_EXTENSION)) : null;
}
if ($extension === 'pdf') {
$firstPdf = $attachment;