Completed
Pull Request — master (#874)
by
unknown
31:19 queued 16:35
created
Classes/IndexQueue/Queue.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -856,7 +856,7 @@
 block discarded – undo
856 856
      * Marks an item as failed and causes the indexer to skip the item in the
857 857
      * next run.
858 858
      *
859
-     * @param int|Item $item Either the item's Index Queue
859
+     * @param Item $item Either the item's Index Queue
860 860
      *      uid or the complete item
861 861
      * @param string $errorMessage Error message
862 862
      */
Please login to merge, or discard this patch.
Classes/Plugin/PluginBase.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -438,7 +438,7 @@
 block discarded – undo
438 438
      * This method executes the requested commands and applies the changes to
439 439
      * the template.
440 440
      *
441
-     * @param $actionResult
441
+     * @param string $actionResult
442 442
      * @return string Rendered plugin content
443 443
      */
444 444
     abstract protected function render($actionResult);
Please login to merge, or discard this patch.
Classes/Plugin/Results/NoResultsCommand.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@
 block discarded – undo
80 80
     /**
81 81
      * Constructs label markers.
82 82
      *
83
-     * @return array Array of label markers.
83
+     * @return string Array of label markers.
84 84
      */
85 85
     protected function getLabelMarkers()
86 86
     {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
         $markers = $this->getLabelMarkers();
65 65
 
66 66
         if ($this->parentPlugin->typoScriptConfiguration->getSearchSpellcheckingSearchUsingSpellCheckerSuggestion()) {
67
-            $suggestionResults =  $this->getSuggestionResults();
67
+            $suggestionResults = $this->getSuggestionResults();
68 68
             $markers['suggestion_results'] = $suggestionResults;
69 69
             $markers['has_suggestion_results'] = trim($suggestionResults) !== '';
70 70
         }
Please login to merge, or discard this patch.
Classes/System/Configuration/TypoScriptConfiguration.php 3 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -544,7 +544,7 @@  discard block
 block discarded – undo
544 544
      *
545 545
      * @param string $configurationName
546 546
      * @param string $defaultIfEmpty
547
-     * @return mixed
547
+     * @return string
548 548
      */
549 549
     public function getIndexQueueInitializerClassByConfigurationName($configurationName, $defaultIfEmpty = 'ApacheSolrForTypo3\\Solr\\IndexQueue\\Initializer\\Record')
550 550
     {
@@ -1237,7 +1237,7 @@  discard block
 block discarded – undo
1237 1237
      * plugin.tx_solr.search.query.allowEmptyQuery
1238 1238
      *
1239 1239
      * @param string $defaultIfEmpty
1240
-     * @return string
1240
+     * @return boolean
1241 1241
      */
1242 1242
     public function getSearchQueryAllowEmptyQuery($defaultIfEmpty = '')
1243 1243
     {
@@ -1299,7 +1299,7 @@  discard block
 block discarded – undo
1299 1299
      *
1300 1300
      * plugin.tx_solr.search.query.returnFields
1301 1301
      *
1302
-     * @param array $defaultIfEmpty
1302
+     * @param string[] $defaultIfEmpty
1303 1303
      * @return array
1304 1304
      */
1305 1305
     public function getSearchQueryReturnFieldsAsArray($defaultIfEmpty = array())
Please login to merge, or discard this patch.
Indentation   -3 removed lines patch added patch discarded remove patch
@@ -767,7 +767,6 @@  discard block
 block discarded – undo
767 767
      * Returns if a log message should be written when a page was indexed.
768 768
      *
769 769
      * plugin.tx_solr.logging.indexing.pageIndexed
770
-
771 770
      * @param bool $defaultIfEmpty
772 771
      * @return bool
773 772
      */
@@ -781,7 +780,6 @@  discard block
 block discarded – undo
781 780
      * Returns if a log message should be written when the TYPO3 search markers are missing in the page.
782 781
      *
783 782
      * plugin.tx_solr.logging.indexing.missingTypo3SearchMarkers
784
-
785 783
      * @param bool $defaultIfEmpty
786 784
      * @return bool
787 785
      */
@@ -809,7 +807,6 @@  discard block
 block discarded – undo
809 807
      * Indicates if the debug mode is enabled or not.
810 808
      *
811 809
      * plugin.tx_solr.enableDebugMode
812
-
813 810
      * @param bool $defaultIfEmpty
814 811
      * @return bool
815 812
      */
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
     {
407 407
         $monitoredTables = [];
408 408
 
409
-        $indexingConfigurations =  $this->getEnabledIndexQueueConfigurationNames();
409
+        $indexingConfigurations = $this->getEnabledIndexQueueConfigurationNames();
410 410
         foreach ($indexingConfigurations as $indexingConfigurationName) {
411 411
             $monitoredTable = $this->getIndexQueueTableNameOrFallbackToConfigurationName($indexingConfigurationName);
412 412
             $monitoredTables[] = $monitoredTable;
@@ -532,13 +532,13 @@  discard block
 block discarded – undo
532 532
     public function getInitialPagesAdditionalWhereClause($defaultIfEmpty = ' AND 1=1')
533 533
     {
534 534
         $path = 'plugin.tx_solr.index.queue.pages' . '.initialPagesAdditionalWhereClause';
535
-        $initialPagesAdditionalWhereClause =  $this->getValueByPathOrDefaultValue($path, '');
535
+        $initialPagesAdditionalWhereClause = $this->getValueByPathOrDefaultValue($path, '');
536 536
 
537 537
         if (trim($initialPagesAdditionalWhereClause) === '') {
538 538
             return $defaultIfEmpty;
539 539
         }
540 540
 
541
-        return ' AND ' .  $initialPagesAdditionalWhereClause;
541
+        return ' AND ' . $initialPagesAdditionalWhereClause;
542 542
     }
543 543
 
544 544
     /**
Please login to merge, or discard this patch.
Classes/Backend/SolrModule/AbstractModuleController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -190,12 +190,12 @@
 block discarded – undo
190 190
     protected function getRequestArgumentOrDefaultValue($argumentKey, $default)
191 191
     {
192 192
         // no request -> return default
193
-        if (! isset($this->request)) {
193
+        if (!isset($this->request)) {
194 194
             return $default;
195 195
         }
196 196
 
197 197
         // argument not present -> return default value
198
-        if (! $this->request->hasArgument($argumentKey)) {
198
+        if (!$this->request->hasArgument($argumentKey)) {
199 199
             return $default;
200 200
         }
201 201
 
Please login to merge, or discard this patch.
Classes/Access/Rootline.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -221,7 +221,7 @@
 block discarded – undo
221 221
     public static function cleanGroupArray(array $groups)
222 222
     {
223 223
         $groups = array_unique($groups); // removes duplicates
224
-        sort($groups, SORT_NUMERIC);     // sort
224
+        sort($groups, SORT_NUMERIC); // sort
225 225
 
226 226
         return $groups;
227 227
     }
Please login to merge, or discard this patch.
Classes/Task/IndexQueueWorkerTask.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@
 block discarded – undo
145 145
             $message .= ' Failures: ' . $failedItemsCount;
146 146
         }
147 147
 
148
-        $message .=  ' / Using webroot: ' . htmlspecialchars($this->getWebRoot());
148
+        $message .= ' / Using webroot: ' . htmlspecialchars($this->getWebRoot());
149 149
 
150 150
         return $message;
151 151
     }
Please login to merge, or discard this patch.
Classes/Domain/Search/FrequentSearches/FrequentSearchesService.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
      * @param TypoScriptFrontendController $tsfe
70 70
      * @param DatabaseConnection $database
71 71
      */
72
-    public function __construct(TypoScriptConfiguration $typoscriptConfiguration, AbstractFrontend $cache,  TypoScriptFrontendController $tsfe, DatabaseConnection $database)
72
+    public function __construct(TypoScriptConfiguration $typoscriptConfiguration, AbstractFrontend $cache, TypoScriptFrontendController $tsfe, DatabaseConnection $database)
73 73
     {
74 74
         $this->configuration = $typoscriptConfiguration;
75 75
         $this->cache = $cache;
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
             $checkLanguageWhere = '';
133 133
         }
134 134
 
135
-        $frequentSearchConfiguration['select.']['ADD_WHERE'] =  $checkRootPidWhere .
135
+        $frequentSearchConfiguration['select.']['ADD_WHERE'] = $checkRootPidWhere .
136 136
                                                                 $checkLanguageWhere . ' ' .
137 137
                                                                 $frequentSearchConfiguration['select.']['ADD_WHERE'];
138 138
 
@@ -167,10 +167,10 @@  discard block
 block discarded – undo
167 167
         $identifier = 'frequentSearchesTags';
168 168
 
169 169
         if ($frequentSearchConfiguration['select.']['checkRootPageId']) {
170
-            $identifier .= '_RP' . (int) $this->tsfe->tmpl->rootLine[0]['uid'];
170
+            $identifier .= '_RP' . (int)$this->tsfe->tmpl->rootLine[0]['uid'];
171 171
         }
172 172
         if ($frequentSearchConfiguration['select.']['checkLanguage']) {
173
-            $identifier .= '_L' . (int) $this->tsfe->sys_language_uid;
173
+            $identifier .= '_L' . (int)$this->tsfe->sys_language_uid;
174 174
         }
175 175
 
176 176
         $identifier .= '_' . md5(serialize($frequentSearchConfiguration));
Please login to merge, or discard this patch.
Classes/Domain/Search/LastSearches/LastSearchesService.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
      */
99 99
     public function addToLastSearches($keywords)
100 100
     {
101
-        $mode   = $this->configuration->getSearchLastSearchesMode();
101
+        $mode = $this->configuration->getSearchLastSearchesMode();
102 102
         switch ($mode) {
103 103
             case 'user':
104 104
                 $this->storeKeywordsToSession($keywords);
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
     protected function getNextSequenceId()
228 228
     {
229 229
         $nextSequenceId = 0;
230
-        $numberOfLastSearchesToLog = (int) $this->configuration->getSearchLastSearchesLimit();
230
+        $numberOfLastSearchesToLog = (int)$this->configuration->getSearchLastSearchesLimit();
231 231
 
232 232
         $row = $this->database->exec_SELECTgetRows(
233 233
             '(sequence_id + 1) % ' . $numberOfLastSearchesToLog . ' as next_sequence_id',
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@
 block discarded – undo
134 134
     /**
135 135
      * Gets the last searched keywords from the database
136 136
      *
137
-     * @param int|bool $limit
137
+     * @param integer $limit
138 138
      * @return array An array containing the last searches of the current user
139 139
      */
140 140
     protected function getLastSearchesFromDatabase($limit = false)
Please login to merge, or discard this patch.