mirror/Part-DB.Part-DB-server
1
0
Fork 1
mirror of https://github.com/Part-DB/Part-DB-server.git synced 2025-07-07 17:04:30 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

If a newly created attachment is a picture, then assign it as master picture if no master picture was set yet.

Browse source
This commit is contained in:
Jan Böhmer 2019-10-03 15:31:55 +02:00
parent 2872e334ac
commit 07dcbc0464
1 changed files with 15 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

18
src/Services/AttachmentHelper.php
View file

@ -33,6 +33,7 @@ namespace App\Services;
use App\Entity\Attachments\Attachment; use App\Entity\Attachments\Attachment;
use App\Entity\Attachments\AttachmentContainingDBElement;
use App\Entity\Attachments\AttachmentTypeAttachment; use App\Entity\Attachments\AttachmentTypeAttachment;
use App\Entity\Attachments\CategoryAttachment; use App\Entity\Attachments\CategoryAttachment;
use App\Entity\Attachments\CurrencyAttachment; use App\Entity\Attachments\CurrencyAttachment;
@ -173,7 +174,7 @@ class AttachmentHelper
MeasurementUnitAttachment::class => 'measurement_unit', StorelocationAttachment::class => 'storelocation', MeasurementUnitAttachment::class => 'measurement_unit', StorelocationAttachment::class => 'storelocation',
SupplierAttachment::class => 'supplier', UserAttachment::class => 'user']; SupplierAttachment::class => 'supplier', UserAttachment::class => 'user'];
$path = $this->base_path . DIRECTORY_SEPARATOR . $mapping[get_class($attachment)] . DIRECTORY_SEPARATOR . $attachment->getElement()->getID(); $path = $this->pathResolver->getMediaPath() . DIRECTORY_SEPARATOR . $mapping[get_class($attachment)] . DIRECTORY_SEPARATOR . $attachment->getElement()->getID();
return $path; return $path;
} }
@ -181,9 +182,11 @@ class AttachmentHelper
* Moves the given uploaded file to a permanent place and saves it into the attachment * Moves the given uploaded file to a permanent place and saves it into the attachment
* @param Attachment $attachment The attachment in which the file should be saved * @param Attachment $attachment The attachment in which the file should be saved
* @param UploadedFile|null $file The file which was uploaded * @param UploadedFile|null $file The file which was uploaded
* @param bool $become_preview_if_empty If this is true, the uploaded attachment can become the preview picture
* if the of the element, if no was set already.
* @return Attachment The attachment with the new filepath * @return Attachment The attachment with the new filepath
*/ */
public function upload(Attachment $attachment, ?UploadedFile $file) : Attachment public function upload(Attachment $attachment, ?UploadedFile $file, bool $become_preview_if_empty = true) : Attachment
{ {
//If file is null, do nothing (helpful, so we dont have to check if the file was reuploaded in controller) //If file is null, do nothing (helpful, so we dont have to check if the file was reuploaded in controller)
if (!$file) { if (!$file) {
@ -194,7 +197,7 @@ class AttachmentHelper
//Sanatize filename //Sanatize filename
$originalFilename = pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME); $originalFilename = pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME);
$newFilename = $attachment->getName() . '-' . uniqid('', false) . '.' . $file->guessExtension(); $newFilename = $attachment->getName() . '-' . uniqid('', false) . '.' . $file->getClientOriginalExtension();
//Move our temporay attachment to its final location //Move our temporay attachment to its final location
$file_path = $file->move($folder, $newFilename)->getRealPath(); $file_path = $file->move($folder, $newFilename)->getRealPath();
@ -207,6 +210,15 @@ class AttachmentHelper
//And save original filename //And save original filename
$attachment->setFilename($file->getClientOriginalName()); $attachment->setFilename($file->getClientOriginalName());
//Check if we should assign this to master picture
//this is only possible if the attachment is new (not yet persisted to DB)
if ($become_preview_if_empty && $attachment->getID() === null && $attachment->isPicture()) {
$element = $attachment->getElement();
if ($element instanceof AttachmentContainingDBElement && $element->getMasterPictureAttachment() === null) {
$element->setMasterPictureAttachment($attachment);
}
}
return $attachment; return $attachment;
} }