[TASK] Sync with EXT:solr addons :: fix old linter issues

b13 · Aug 15, 2023 · b15df83 · b15df83
1 parent d96e4eb
commit b15df83
Show file tree

Hide file tree

Showing 8 changed files with 13 additions and 13 deletions.
diff --git a/Classes/System/Solr/SolrConnection.php b/Classes/System/Solr/SolrConnection.php
@@ -45,7 +45,7 @@ class SolrConnection
 {
     protected ?SolrAdminService $adminService = null;
 
-    protected ?SolrReadService $readService= null;
+    protected ?SolrReadService $readService = null;
 
     protected ?SolrWriteService $writeService = null;
 

diff --git a/Classes/System/Util/SiteUtility.php b/Classes/System/Util/SiteUtility.php
@@ -150,9 +150,9 @@ public static function getAllSolrConnectionConfigurations(
      */
     protected static function getConnectionPropertyOrFallback(
         CoreSite $typo3Site,
-        string   $property,
-        int      $languageId,
-        string   $scope,
+        string $property,
+        int $languageId,
+        string $scope,
     ): string|int|bool|null {
         if ($scope === 'write' && !self::isWriteConnectionEnabled($typo3Site, $languageId)) {
             $scope = 'read';

diff --git a/Tests/Integration/Controller/SearchControllerTest.php b/Tests/Integration/Controller/SearchControllerTest.php
@@ -936,8 +936,8 @@ public function canSeeTheParsedQueryWhenABackendUserIsLoggedIn()
     public function frontendWillRenderErrorMessageIfSolrNotAvailableDataProvider(): array
     {
         return [
-            ['action' => 'results', 'getArguments' =>['q' => '*']],
-            ['action' => 'detail', 'getArguments' =>['id' => 1]],
+            ['action' => 'results', 'getArguments' => ['q' => '*']],
+            ['action' => 'detail', 'getArguments' => ['id' => 1]],
         ];
     }
 

diff --git a/Tests/Integration/Domain/Search/ResultSet/SearchResultSetServiceTest.php b/Tests/Integration/Domain/Search/ResultSet/SearchResultSetServiceTest.php
@@ -82,7 +82,7 @@ public function canGetVariants(): void
 
         $typoScriptConfiguration = Util::getSolrConfiguration();
         $typoScriptConfiguration->mergeSolrConfiguration([
-           'search.' =>[
+           'search.' => [
                'variants' => 1,
                'variants.' => [
                    'variantField' => 'pid',
@@ -130,7 +130,7 @@ public function canGetCaseSensitiveVariants(): void
 
         $typoScriptConfiguration = Util::getSolrConfiguration();
         $typoScriptConfiguration->mergeSolrConfiguration([
-            'search.' =>[
+            'search.' => [
                 'variants' => 1,
                 'variants.' => [
                     'variantField' => 'author',
@@ -196,7 +196,7 @@ public function canGetZeroResultsWithVariantsOnEmptyIndex(): void
 
         $typoScriptConfiguration = Util::getSolrConfiguration();
         $typoScriptConfiguration->mergeSolrConfiguration([
-            'search.' =>[
+            'search.' => [
                 'variants' => 1,
                 'variants.' => [
                     'variantField' => 'pid',

diff --git a/Tests/Integration/Domain/Search/StatisticsRepository/StatisticsRepositoryTest.php b/Tests/Integration/Domain/Search/StatisticsRepository/StatisticsRepositoryTest.php
@@ -139,7 +139,7 @@ public function canSaveStatisticsRecord()
     protected static function getDaysSinceTimestamp(int $timestamp): int
     {
         $secondsUntilNow = time() - $timestamp;
-        $days = floor($secondsUntilNow / (60*60*24));
+        $days = floor($secondsUntilNow / (60 * 60 * 24));
         return (int)$days;
     }
 }
diff --git a/Tests/Integration/System/Configuration/TypoScriptConfigurationTest.php b/Tests/Integration/System/Configuration/TypoScriptConfigurationTest.php
@@ -45,7 +45,7 @@ public function testCanUsePlainValuesFromConfiguration()
         $configuration = [
             'plugin.' => [
                 'tx_solr.' => [
-                    'search.' =>[
+                    'search.' => [
                         'sorting' => 1,
                     ],
                 ],

diff --git a/Tests/Unit/Domain/Search/ResultSet/SearchResultSetServiceTest.php b/Tests/Unit/Domain/Search/ResultSet/SearchResultSetServiceTest.php
@@ -156,7 +156,7 @@ public function canCreateGroups(): void
         $parserRegistry = GeneralUtility::makeInstance(ResultParserRegistry::class, $typoScriptConfiguration);
         $parserRegistry->registerParser(GroupedResultParser::class, 300);
 
-        $queryMock= $this->createMock(Query::class);
+        $queryMock = $this->createMock(Query::class);
         $queryMock->expects(self::once())->method('getComponent')->willReturn($this->createMock(Grouping::class));
         $queryBuilderMock = $this->createMock(QueryBuilder::class);
         $queryBuilderMock->expects(self::once())->method('buildSearchQuery')->willReturn($queryMock);

diff --git a/Tests/Unit/Domain/Site/SiteRepositoryTest.php b/Tests/Unit/Domain/Site/SiteRepositoryTest.php
@@ -143,7 +143,7 @@ public function canGetAllLanguages(): void
         );
 
         $siteOne = $this->siteRepository->getFirstAvailableSite();
-        $connections =$siteOne->getAllSolrConnectionConfigurations();
+        $connections = $siteOne->getAllSolrConnectionConfigurations();
         self::assertEquals([0, 2, 5], array_keys($connections), 'Could not get languages for site');
     }