diff --git a/src/Controller/AdminPages/BaseAdminController.php b/src/Controller/AdminPages/BaseAdminController.php index 5fc8c5a8..d5e0e4ee 100644 --- a/src/Controller/AdminPages/BaseAdminController.php +++ b/src/Controller/AdminPages/BaseAdminController.php @@ -81,7 +81,6 @@ abstract class BaseAdminController extends AbstractController protected $passwordEncoder; protected $translator; - protected $attachmentHelper; protected $attachmentSubmitHandler; protected $commentHelper; @@ -90,7 +89,7 @@ abstract class BaseAdminController extends AbstractController protected $dataTableFactory; public function __construct(TranslatorInterface $translator, UserPasswordEncoderInterface $passwordEncoder, - AttachmentManager $attachmentHelper, AttachmentSubmitHandler $attachmentSubmitHandler, + AttachmentSubmitHandler $attachmentSubmitHandler, EventCommentHelper $commentHelper, HistoryHelper $historyHelper, TimeTravel $timeTravel, DataTableFactory $dataTableFactory) { @@ -108,7 +107,6 @@ abstract class BaseAdminController extends AbstractController $this->translator = $translator; $this->passwordEncoder = $passwordEncoder; - $this->attachmentHelper = $attachmentHelper; $this->attachmentSubmitHandler = $attachmentSubmitHandler; $this->commentHelper = $commentHelper; $this->historyHelper = $historyHelper; @@ -206,7 +204,6 @@ abstract class BaseAdminController extends AbstractController return $this->render($this->twig_template, [ 'entity' => $entity, 'form' => $form->createView(), - 'attachment_helper' => $this->attachmentHelper, 'route_base' => $this->route_base, 'datatable' => $table, 'timeTravel' => $timeTravel_timestamp, @@ -322,7 +319,6 @@ abstract class BaseAdminController extends AbstractController 'form' => $form->createView(), 'import_form' => $import_form->createView(), 'mass_creation_form' => $mass_creation_form->createView(), - 'attachment_helper' => $this->attachmentHelper, 'route_base' => $this->route_base, ]); } diff --git a/src/Controller/PartController.php b/src/Controller/PartController.php index 1c6389cf..d5495927 100644 --- a/src/Controller/PartController.php +++ b/src/Controller/PartController.php @@ -76,15 +76,13 @@ use Symfony\Contracts\Translation\TranslatorInterface; */ class PartController extends AbstractController { - protected $attachmentManager; protected $pricedetailHelper; protected $partPreviewGenerator; protected $commentHelper; - public function __construct(AttachmentManager $attachmentManager, PricedetailHelper $pricedetailHelper, + public function __construct(PricedetailHelper $pricedetailHelper, PartPreviewGenerator $partPreviewGenerator, EventCommentHelper $commentHelper) { - $this->attachmentManager = $attachmentManager; $this->pricedetailHelper = $pricedetailHelper; $this->partPreviewGenerator = $partPreviewGenerator; $this->commentHelper = $commentHelper; @@ -136,7 +134,6 @@ class PartController extends AbstractController [ 'part' => $part, 'datatable' => $table, - 'attachment_helper' => $this->attachmentManager, 'pricedetail_helper' => $this->pricedetailHelper, 'pictures' => $this->partPreviewGenerator->getPreviewAttachments($part), 'timeTravel' => $timeTravel_timestamp, @@ -194,7 +191,6 @@ class PartController extends AbstractController [ 'part' => $part, 'form' => $form->createView(), - 'attachment_helper' => $this->attachmentManager, ]); } @@ -231,7 +227,7 @@ class PartController extends AbstractController * @return Response */ public function new(Request $request, EntityManagerInterface $em, TranslatorInterface $translator, - AttachmentManager $attachmentHelper, AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null): Response + AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null): Response { if (null === $part) { $new_part = new Part(); @@ -318,7 +314,6 @@ class PartController extends AbstractController [ 'part' => $new_part, 'form' => $form->createView(), - 'attachment_helper' => $attachmentHelper, ]); } } diff --git a/templates/AdminPages/_attachments.html.twig b/templates/AdminPages/_attachments.html.twig index 7b7ce7ba..f0105b32 100644 --- a/templates/AdminPages/_attachments.html.twig +++ b/templates/AdminPages/_attachments.html.twig @@ -22,7 +22,7 @@ {# @var attach \App\Entity\Attachments\Attachment #} - {% if attachment_helper.fileExisting(attach) %} + {% if attachment_manager.fileExisting(attach) %} {% if not attach.external %}

@@ -31,7 +31,7 @@
- {{ attachment_helper.humanFileSize(attach) }} + {{ attachment_manager.humanFileSize(attach) }}
{% else %} diff --git a/templates/Parts/edit/_attachments.html.twig b/templates/Parts/edit/_attachments.html.twig index b625d524..6b21a1e8 100644 --- a/templates/Parts/edit/_attachments.html.twig +++ b/templates/Parts/edit/_attachments.html.twig @@ -19,7 +19,7 @@ {% set attach = attachment.vars.value %} - {% if attachment_helper.fileExisting(attach) %} + {% if attachment_manager.fileExisting(attach) %} {% if not attach.external %}

@@ -28,7 +28,7 @@
- {{ attachment_helper.humanFileSize(attach) }} + {{ attachment_manager.humanFileSize(attach) }}
{% else %}