kaliop-uk / ezfindsearchenginebundle

Core/Persistence/eZFind/Content/Search/Common/Gateway/CriterionHandler/LocationId.php

Doc Comments -1 removed lines

@@ -15,7 +15,6 @@
 
     /**
      * @param CriteriaConverter $converter
-     * @param SelectQuery $query
      * @param Criterion $criterion
      * @return string|string[]
      */

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
 
 class LocationId extends CriterionHandler
 {
-    public function accept( Criterion $criterion )
+    public function accept(Criterion $criterion)
     {
         return $criterion instanceof Criterion\LocationId;
     }
@@ -19,7 +19,7 @@ 
      * @param Criterion $criterion
      * @return string|string[]
      */
-    public function handle( CriteriaConverter $converter, Criterion $criterion )
+    public function handle(CriteriaConverter $converter, Criterion $criterion)
     {
         $result = [];
 

Core/Persistence/eZFind/Content/Search/Common/Gateway/CriterionHandler/ObjectStateId.php

Doc Comments -1 removed lines

@@ -15,7 +15,6 @@
 
     /**
      * @param CriteriaConverter $converter
-     * @param SelectQuery $query
      * @param Criterion $criterion
      * @return string|string[]
      *

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
 
 class ObjectStateId extends CriterionHandler
 {
-    public function accept( Criterion $criterion )
+    public function accept(Criterion $criterion)
     {
         return $criterion instanceof Criterion\ObjectStateId;
     }
@@ -21,7 +21,7 @@ 
      *
      * @todo verify if handling of multiple states via OR is correct
      */
-    public function handle( CriteriaConverter $converter, Criterion $criterion )
+    public function handle(CriteriaConverter $converter, Criterion $criterion)
     {
         $result = [];
 

Core/Repository/SearchService.php

Doc Comments +6 added lines

@@ -345,6 +345,9 @@ 
         return $searchParameters;
     }
 
+    /**
+     * @param string $paramName
+     */
     protected function extractLegacyParameter($paramName, Query $query)
     {
         switch ($paramName) {
@@ -422,6 +425,9 @@ 
         return $facets;
     }
 
+    /**
+     * @param Query\SortClause[] $sortClauses
+     */
     protected function extractSort($sortClauses)
     {
         $result = [];

Spacing +11 added lines, -11 removed lines

@@ -127,7 +127,7 @@ 
 
             // trick to access data from a protected member of ezfSearchResultInfo
             // @see http://blag.kazeno.net/development/access-private-protected-properties
-            $propGetter = Closure::bind(function($prop){return $this->$prop;}, $extras, $extras);
+            $propGetter = Closure::bind(function($prop) {return $this->$prop; }, $extras, $extras);
             $resultArray = $propGetter('ResultArray');
 
             if (isset($resultArray['response']['maxScore'])) {
@@ -137,7 +137,7 @@ 
             // optimize: remove from SearchExtras 'response' to save memory using the 'Closure::bind' hack
             /// @todo (!important) make the cutover limit configurable
             if ($result['SearchCount'] > 100) {
-                $resultsCleaner = Closure::bind(function(){unset($this->ResultArray['response']['docs']);}, $extras, $extras);
+                $resultsCleaner = Closure::bind(function() {unset($this->ResultArray['response']['docs']); }, $extras, $extras);
                 $resultsCleaner();
             }
         }
@@ -238,7 +238,7 @@ 
 
         /** @var array $searchResult */
         $searchResult = $this->getLegacyKernel()->runCallback(
-            function () use ($searchParameters) {
+            function() use ($searchParameters) {
                 return \eZFunctionHandler::execute($this->ezFindModule, $this->ezFindFunction, $searchParameters);
             },
             false
@@ -497,7 +497,7 @@ 
 
             // trick to access data from a protected member of ezfSearchResultInfo
             // @see http://blag.kazeno.net/development/access-private-protected-properties
-            $propGetter = Closure::bind(function($prop){return $this->$prop;}, $extras, $extras);
+            $propGetter = Closure::bind(function($prop) {return $this->$prop; }, $extras, $extras);
             $resultArray = $propGetter('ResultArray');
             $searchResults = $resultArray['response']['docs'];
         }
@@ -509,16 +509,16 @@ 
         $results = array();
 
         foreach ($searchResults as $index => $result) {
-            switch($returnType) {
+            switch ($returnType) {
 
                 case KaliopQuery::RETURN_CONTENTS:
                     try {
                         $results[$index] = new SearchHit(
                             [
                                 'valueObject' => $this->contentService->loadContent($result['id']),
-                                'score' => isset($result['score'])? $result['score'] : null,
-                                'highlight' => isset($result['highlight'])? $result['highlight'] : null,
-                                'elevated' => isset($result['elevated'])? $result['elevated'] : null,
+                                'score' => isset($result['score']) ? $result['score'] : null,
+                                'highlight' => isset($result['highlight']) ? $result['highlight'] : null,
+                                'elevated' => isset($result['elevated']) ? $result['elevated'] : null,
                                 /// @todo decide what is the correct value for 'index': guid, installation_id/guid ?
                                 //'index' => isset($result['guid'])? $result['guid'] : null,
                             ]
@@ -559,9 +559,9 @@ 
                     $results[$index] = new SearchHit(
                         [
                             'valueObject' => $result,
-                            'score' => isset($result['score'])? $result['score'] : null,
-                            'highlight' => isset($result['highlight'])? $result['highlight'] : null,
-                            'elevated' => isset($result['elevated'])? $result['elevated'] : null,
+                            'score' => isset($result['score']) ? $result['score'] : null,
+                            'highlight' => isset($result['highlight']) ? $result['highlight'] : null,
+                            'elevated' => isset($result['elevated']) ? $result['elevated'] : null,
                             /// @todo decide what is the correct value for 'index': guid, installation_id/guid ?
                             //'index' => isset($result['guid'])? $result['guid'] : null,
                         ]

DependencyInjection/KaliopEzFindSearchEngineExtension.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 
     protected function loadInternal(array $mergedConfig, ContainerBuilder $container)
     {
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('services.yml');
 
         if (self::$loadTestConfig != false) {

Command/TestCommand.php

Spacing +4 added lines, -4 removed lines

@@ -28,11 +28,11 @@ 
         $legacyKernelClosure = $this->getContainer()->get('ezpublish_legacy.kernel');
         $legacyKernel = $legacyKernelClosure();
         $legacyKernel->runCallback(
-            function ()  {
+            function() {
                 $moduleRepositories = \eZModule::activeModuleRepositories(true);
                 $moduleRepositories[] = 'extension/ezfind/modules';
                 //var_dump($moduleRepositories);
-                \eZModule::setGlobalPathList( $moduleRepositories );
+                \eZModule::setGlobalPathList($moduleRepositories);
             },
             false
         );
@@ -82,12 +82,12 @@ 
         echo "count: "; var_dump(count($results->searchHits));
         echo "totalCount: "; var_dump($results->totalCount);
         echo "time (ms): "; var_dump($results->time);
-        echo "maxScore: ";  var_dump($results->maxScore);
+        echo "maxScore: "; var_dump($results->maxScore);
         echo "q: "; var_dump($results->searchExtras->attribute('responseHeader')['params']['q']);
         echo "fq: "; var_dump($results->searchExtras->attribute('responseHeader')['params']['fq']);
         echo "sort: "; var_dump($results->searchExtras->attribute('responseHeader')['params']['sort']);
 
-        foreach($results->searchHits as $i => $searchHit) {
+        foreach ($results->searchHits as $i => $searchHit) {
             /** @var Content $content */
             $content = $searchHit->valueObject;
             $score = $searchHit->score;

Core/Persistence/eZFind/Content/Search/Common/Gateway/FacetHandler/DateRange.php

Spacing +8 added lines, -8 removed lines

@@ -88,29 +88,29 @@
      */
     protected function convertDateInterval(DateInterval $dateInterval)
     {
-        $sign = $dateInterval->invert? '-' : '+';
+        $sign = $dateInterval->invert ? '-' : '+';
         if ($dateInterval->days) {
-            return $sign . $dateInterval->days . 'DAY' . (($dateInterval->y > 1)? 'S' : '');
+            return $sign . $dateInterval->days . 'DAY' . (($dateInterval->y > 1) ? 'S' : '');
         }
 
         $syntax = '';
         if ($dateInterval->y) {
-            $syntax .= $sign . $dateInterval->y . 'YEAR' . (($dateInterval->y > 1)? 'S' : '');
+            $syntax .= $sign . $dateInterval->y . 'YEAR' . (($dateInterval->y > 1) ? 'S' : '');
         }
         if ($dateInterval->m) {
-            $syntax .= $sign . $dateInterval->m . 'MONTH' . (($dateInterval->m > 1)? 'S' : '');
+            $syntax .= $sign . $dateInterval->m . 'MONTH' . (($dateInterval->m > 1) ? 'S' : '');
         }
         if ($dateInterval->d) {
-            $syntax .= $sign . $dateInterval->d . 'DAY' . (($dateInterval->d > 1)? 'S' : '');
+            $syntax .= $sign . $dateInterval->d . 'DAY' . (($dateInterval->d > 1) ? 'S' : '');
         }
         if ($dateInterval->h) {
-            $syntax .= $sign . $dateInterval->h . 'HOUR' . (($dateInterval->h > 1)? 'S' : '');
+            $syntax .= $sign . $dateInterval->h . 'HOUR' . (($dateInterval->h > 1) ? 'S' : '');
         }
         if ($dateInterval->i) {
-            $syntax .= $sign . $dateInterval->i . 'MINUTE' . (($dateInterval->i > 1)? 'S' : '');
+            $syntax .= $sign . $dateInterval->i . 'MINUTE' . (($dateInterval->i > 1) ? 'S' : '');
         }
         if ($dateInterval->s) {
-            $syntax .= $sign . $dateInterval->s . 'SECOND' . (($dateInterval->s > 1)? 'S' : '');
+            $syntax .= $sign . $dateInterval->s . 'SECOND' . (($dateInterval->s > 1) ? 'S' : '');
         }
 
         return $syntax;

Core/Persistence/eZFind/Content/Search/Common/Gateway/FacetHandler/Field.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     {
         $facetKey = $this->getFacetKey($facetBuilder);
         $fieldNames = [];
-        foreach ((array) $facetBuilder->fieldPaths as $fieldPath) {
+        foreach ((array)$facetBuilder->fieldPaths as $fieldPath) {
             $fieldNames[] = \eZSolr::getFieldName($fieldPath, false, 'facet');
         }
 

Core/Persistence/eZFind/Content/Search/Common/Gateway/CriterionHandler/LocationPriority.php

Spacing +7 added lines, -7 removed lines

@@ -8,29 +8,29 @@
 
 class LocationPriority extends CriterionHandler
 {
-    public function accept( Criterion $criterion )
+    public function accept(Criterion $criterion)
     {
         return $criterion instanceof Criterion\LocationPriority;
     }
 
-    public function handle( CriteriaConverter $converter, Criterion $criterion )
+    public function handle(CriteriaConverter $converter, Criterion $criterion)
     {
-        switch ( $criterion->operator )
+        switch ($criterion->operator)
         {
             case Criterion\Operator::BETWEEN:
                 return 'priority:[' . $this->escapeValue($criterion->value[0]) . ' TO ' . $this->escapeValue($criterion->value[1]) . ']';
 
             case Criterion\Operator::GT:
-                return 'priority:{' . $this->escapeValue(reset( $criterion->value )) . ' TO *]';
+                return 'priority:{' . $this->escapeValue(reset($criterion->value)) . ' TO *]';
 
             case Criterion\Operator::GTE:
-                return 'priority:[' . $this->escapeValue(reset( $criterion->value )) . ' TO *]';
+                return 'priority:[' . $this->escapeValue(reset($criterion->value)) . ' TO *]';
 
             case Criterion\Operator::LT:
-                return 'priority:[* TO ' . $this->escapeValue(reset( $criterion->value )) . '}';
+                return 'priority:[* TO ' . $this->escapeValue(reset($criterion->value)) . '}';
 
             case Criterion\Operator::LTE:
-                return 'priority:[* TO ' . $this->escapeValue(reset( $criterion->value )) . ']';
+                return 'priority:[* TO ' . $this->escapeValue(reset($criterion->value)) . ']';
 
         }
     }

Core/Persistence/eZFind/Content/Search/Common/Gateway/CriterionHandler/FieldBase.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     protected function determineFieldName($fieldName)
     {
         $fieldName = $this->getLegacyKernel()->runCallback(
-            function () use ($fieldName) {
+            function() use ($fieldName) {
                 return \eZSolr::getFieldName($fieldName, false, 'facet');
             },
             false