CLEANUP: Use DI for ContainerRepository

codappix · Apr 9, 2024 · d4414a5 · d4414a5
1 parent d444a04
commit d4414a5
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 15 deletions.
diff --git a/Classes/DataProcessing/ResponsiveImagesProcessor.php b/Classes/DataProcessing/ResponsiveImagesProcessor.php
@@ -24,6 +24,7 @@
  */
 
 use Codappix\ResponsiveImages\Configuration\ConfigurationManager;
+use Codappix\ResponsiveImages\Domain\Repository\ContainerRepository;
 use Codappix\ResponsiveImages\Sizes\Breakpoint;
 use Codappix\ResponsiveImages\Sizes\Rootline;
 use TYPO3\CMS\Core\Resource\FileInterface;
@@ -33,8 +34,6 @@
 
 final class ResponsiveImagesProcessor implements DataProcessorInterface
 {
-    private readonly ConfigurationManager $configurationManager;
-
     /**
      * @var FileInterface[]
      */
@@ -44,9 +43,10 @@ final class ResponsiveImagesProcessor implements DataProcessorInterface
 
     private array $contentElementSizes = [];
 
-    public function __construct()
-    {
-        $this->configurationManager = GeneralUtility::makeInstance(ConfigurationManager::class);
+    public function __construct(
+        private ConfigurationManager $configurationManager,
+        private ContainerRepository $containerRepository
+    ) {
     }
 
     public function process(
@@ -76,7 +76,7 @@ public function process(
             return $processedData;
         }
 
-        $this->contentElementSizes = (new Rootline($processedData['data'], $fieldName))->getFinalSizes();
+        $this->contentElementSizes = (new Rootline($this->containerRepository, $processedData['data'], $fieldName))->getFinalSizes();
         $this->calculateFileDimensions();
 
         $targetFieldName = (string) $cObj->stdWrapValue(

diff --git a/Classes/Domain/Repository/ContainerRepository.php b/Classes/Domain/Repository/ContainerRepository.php
@@ -5,21 +5,22 @@
 namespace Codappix\ResponsiveImages\Domain\Repository;
 
 use TYPO3\CMS\Core\Database\Connection;
-use TYPO3\CMS\Core\Database\ConnectionPool;
-use TYPO3\CMS\Core\Database\Query\QueryBuilder;
 use TYPO3\CMS\Core\Error\Exception;
-use TYPO3\CMS\Core\Utility\GeneralUtility;
 
 class ContainerRepository
 {
+    public function __construct(
+        private Connection $connection
+    ) {
+    }
+
     /**
      * @throws \Doctrine\DBAL\Exception
      * @throws Exception
      */
     public function findByIdentifier(int $identifier): array
     {
-        /** @var QueryBuilder $queryBuilder */
-        $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('tt_content');
+        $queryBuilder = $this->connection->createQueryBuilder();
         $rawData = $queryBuilder
             ->select('*')
             ->from('tt_content')

diff --git a/Classes/Sizes/Rootline.php b/Classes/Sizes/Rootline.php
@@ -36,20 +36,17 @@ final class Rootline
 
     private BackendLayout $backendLayout;
 
-    private ContainerRepository $containerRepository;
-
     private array $finalSizes = [];
 
     private string $fieldName;
 
     private string $backendLayoutIdentifier;
 
     public function __construct(
+        private ContainerRepository $containerRepository,
         array $data,
         string $fieldName
     ) {
-        $this->containerRepository = new ContainerRepository();
-
         $this->determineBackendLayout();
         $this->fieldName = $fieldName;
         $this->contentElement = $this->determineContentElement(null, $data);

diff --git a/Configuration/Services.yaml b/Configuration/Services.yaml
@@ -8,6 +8,14 @@ services:
     Codappix\ResponsiveImages\:
         resource: '../Classes/*'
 
+    dbconnection.tt_content:
+        class: 'TYPO3\CMS\Core\Database\Connection'
+        factory:
+            - '@TYPO3\CMS\Core\Database\ConnectionPool'
+            - 'getConnectionForTable'
+        arguments:
+            - 'tt_content'
+
     extbaseSettings.ResponsiveImages:
         class: 'array'
         factory:
@@ -18,7 +26,18 @@ services:
             $extensionName: 'ResponsiveImages'
             $pluginName: ''
 
+    Codappix\ResponsiveImages\DataProcessing\ResponsiveImagesProcessor:
+        public: true
+        arguments:
+            $configurationManager: '@Codappix\ResponsiveImages\Configuration\ConfigurationManager'
+            $containerRepository: '@Codappix\ResponsiveImages\Domain\Repository\ContainerRepository'
+
     Codappix\ResponsiveImages\Configuration\ConfigurationManager:
         public: true
         arguments:
             $settings: '@extbaseSettings.ResponsiveImages'
+
+    Codappix\ResponsiveImages\Domain\Repository\ContainerRepository:
+        public: true
+        arguments:
+            - '@dbconnection.tt_content'