@@ -8,17 +8,17 @@ discard block |
||
| 8 | 8 | |
| 9 | 9 | class Field extends FieldBase |
| 10 | 10 | { |
| 11 | - public function accept( Criterion $criterion ) |
|
| 11 | + public function accept(Criterion $criterion) |
|
| 12 | 12 | { |
| 13 | 13 | return $criterion instanceof Criterion\Field; |
| 14 | 14 | } |
| 15 | 15 | |
| 16 | 16 | /// @todo should we apply specific conversions based on field type? |
| 17 | - public function handle( CriteriaConverter $converter, Criterion $criterion ) |
|
| 17 | + public function handle(CriteriaConverter $converter, Criterion $criterion) |
|
| 18 | 18 | { |
| 19 | 19 | $field = $this->determineFieldName($criterion->target); |
| 20 | 20 | |
| 21 | - switch ( $criterion->operator ) |
|
| 21 | + switch ($criterion->operator) |
|
| 22 | 22 | { |
| 23 | 23 | case Criterion\Operator::IN: |
| 24 | 24 | $values = array_map(array($this, 'escapeValue'), $criterion->value); |
@@ -52,7 +52,7 @@ discard block |
||
| 52 | 52 | * - replace %, _ chars |
| 53 | 53 | * - escape regexp chars |
| 54 | 54 | */ |
| 55 | - protected function like2regexp($string, $delimiter='/') |
|
| 55 | + protected function like2regexp($string, $delimiter = '/') |
|
| 56 | 56 | { |
| 57 | 57 | $string = preg_quote($string, $delimiter); |
| 58 | 58 | $string = str_replace(array('%', '_'), array('.*', '.'), $string); |
@@ -9,16 +9,16 @@ |
||
| 9 | 9 | |
| 10 | 10 | class DateMetadata extends CriterionHandler |
| 11 | 11 | { |
| 12 | - public function accept( Criterion $criterion ) |
|
| 12 | + public function accept(Criterion $criterion) |
|
| 13 | 13 | { |
| 14 | 14 | return $criterion instanceof Criterion\DateMetadata; |
| 15 | 15 | } |
| 16 | 16 | |
| 17 | - public function handle( CriteriaConverter $converter, Criterion $criterion ) |
|
| 17 | + public function handle(CriteriaConverter $converter, Criterion $criterion) |
|
| 18 | 18 | { |
| 19 | 19 | $field = $criterion->target == Criterion\DateMetadata::MODIFIED ? 'modified' : 'published'; |
| 20 | 20 | |
| 21 | - switch ( $criterion->operator ) |
|
| 21 | + switch ($criterion->operator) |
|
| 22 | 22 | { |
| 23 | 23 | case Criterion\Operator::BETWEEN: |
| 24 | 24 | return $field . ':[' . $this->formatDate($criterion->value[0]) . ' TO ' . $this->formatDate($criterion->value[1]) . ']'; |
@@ -9,12 +9,12 @@ |
||
| 9 | 9 | |
| 10 | 10 | class SolrRaw extends CriterionHandler |
| 11 | 11 | { |
| 12 | - public function accept( Criterion $criterion ) |
|
| 12 | + public function accept(Criterion $criterion) |
|
| 13 | 13 | { |
| 14 | 14 | return $criterion instanceof KCriterion\SolrRaw; |
| 15 | 15 | } |
| 16 | 16 | |
| 17 | - public function handle( CriteriaConverter $converter, Criterion $criterion ) |
|
| 17 | + public function handle(CriteriaConverter $converter, Criterion $criterion) |
|
| 18 | 18 | { |
| 19 | 19 | return $criterion->value; |
| 20 | 20 | } |
@@ -20,7 +20,7 @@ |
||
| 20 | 20 | return array_merge( |
| 21 | 21 | array('OR'), |
| 22 | 22 | array_map( |
| 23 | - function ($value) use ($converter) { |
|
| 23 | + function($value) use ($converter) { |
|
| 24 | 24 | return $converter->handle($value); |
| 25 | 25 | }, |
| 26 | 26 | $criterion->criteria |
@@ -8,7 +8,7 @@ discard block |
||
| 8 | 8 | |
| 9 | 9 | class LocationId extends CriterionHandler |
| 10 | 10 | { |
| 11 | - public function accept( Criterion $criterion ) |
|
| 11 | + public function accept(Criterion $criterion) |
|
| 12 | 12 | { |
| 13 | 13 | return $criterion instanceof Criterion\LocationId; |
| 14 | 14 | } |
@@ -19,7 +19,7 @@ discard block |
||
| 19 | 19 | * @param Criterion $criterion |
| 20 | 20 | * @return string|string[] |
| 21 | 21 | */ |
| 22 | - public function handle( CriteriaConverter $converter, Criterion $criterion ) |
|
| 22 | + public function handle(CriteriaConverter $converter, Criterion $criterion) |
|
| 23 | 23 | { |
| 24 | 24 | $result = []; |
| 25 | 25 | |
@@ -8,12 +8,12 @@ |
||
| 8 | 8 | |
| 9 | 9 | class ContentId extends CriterionHandler |
| 10 | 10 | { |
| 11 | - public function accept( Criterion $criterion ) |
|
| 11 | + public function accept(Criterion $criterion) |
|
| 12 | 12 | { |
| 13 | 13 | return $criterion instanceof Criterion\ContentId; |
| 14 | 14 | } |
| 15 | 15 | |
| 16 | - public function handle( CriteriaConverter $converter, Criterion $criterion ) |
|
| 16 | + public function handle(CriteriaConverter $converter, Criterion $criterion) |
|
| 17 | 17 | { |
| 18 | 18 | $result = []; |
| 19 | 19 | |
@@ -65,7 +65,7 @@ |
||
| 65 | 65 | $queryBuilderClass = $this->queryBuilderClass; |
| 66 | 66 | /** @var \ezfeZPSolrQueryBuilder $queryBuilder */ |
| 67 | 67 | $queryBuilder = new $queryBuilderClass(null); |
| 68 | - $filterCreator = \Closure::bind(function($filter){return $this->getParamFilterQuery(array('Filter' => $filter));}, $queryBuilder, $queryBuilder); |
|
| 68 | + $filterCreator = \Closure::bind(function($filter) {return $this->getParamFilterQuery(array('Filter' => $filter)); }, $queryBuilder, $queryBuilder); |
|
| 69 | 69 | return $filterCreator($filter); |
| 70 | 70 | } |
| 71 | 71 | |
@@ -94,7 +94,7 @@ |
||
| 94 | 94 | protected function getSolrFieldName($baseName) |
| 95 | 95 | { |
| 96 | 96 | return $this->getLegacyKernel()->runCallback( |
| 97 | - function () use ($baseName) { |
|
| 97 | + function() use ($baseName) { |
|
| 98 | 98 | return \eZSolr::getFieldName($baseName, false, 'facet'); |
| 99 | 99 | }, |
| 100 | 100 | false |
@@ -127,7 +127,7 @@ discard block |
||
| 127 | 127 | |
| 128 | 128 | // trick to access data from a protected member of ezfSearchResultInfo |
| 129 | 129 | // @see http://blag.kazeno.net/development/access-private-protected-properties |
| 130 | - $propGetter = Closure::bind(function($prop){return $this->$prop;}, $extras, $extras); |
|
| 130 | + $propGetter = Closure::bind(function($prop) {return $this->$prop; }, $extras, $extras); |
|
| 131 | 131 | $resultArray = $propGetter('ResultArray'); |
| 132 | 132 | |
| 133 | 133 | if (isset($resultArray['response']['maxScore'])) { |
@@ -137,7 +137,7 @@ discard block |
||
| 137 | 137 | // optimize: remove from SearchExtras 'response' to save memory using the 'Closure::bind' hack |
| 138 | 138 | /// @todo (!important) make the cutover limit configurable |
| 139 | 139 | if ($result['SearchCount'] > 100) { |
| 140 | - $resultsCleaner = Closure::bind(function(){unset($this->ResultArray['response']['docs']);}, $extras, $extras); |
|
| 140 | + $resultsCleaner = Closure::bind(function() {unset($this->ResultArray['response']['docs']); }, $extras, $extras); |
|
| 141 | 141 | $resultsCleaner(); |
| 142 | 142 | } |
| 143 | 143 | } |
@@ -238,7 +238,7 @@ discard block |
||
| 238 | 238 | |
| 239 | 239 | /** @var array $searchResult */ |
| 240 | 240 | $searchResult = $this->getLegacyKernel()->runCallback( |
| 241 | - function () use ($searchParameters) { |
|
| 241 | + function() use ($searchParameters) { |
|
| 242 | 242 | return \eZFunctionHandler::execute($this->ezFindModule, $this->ezFindFunction, $searchParameters); |
| 243 | 243 | }, |
| 244 | 244 | false |
@@ -497,7 +497,7 @@ discard block |
||
| 497 | 497 | |
| 498 | 498 | // trick to access data from a protected member of ezfSearchResultInfo |
| 499 | 499 | // @see http://blag.kazeno.net/development/access-private-protected-properties |
| 500 | - $propGetter = Closure::bind(function($prop){return $this->$prop;}, $extras, $extras); |
|
| 500 | + $propGetter = Closure::bind(function($prop) {return $this->$prop; }, $extras, $extras); |
|
| 501 | 501 | $resultArray = $propGetter('ResultArray'); |
| 502 | 502 | $searchResults = $resultArray['response']['docs']; |
| 503 | 503 | } |
@@ -509,16 +509,16 @@ discard block |
||
| 509 | 509 | $results = array(); |
| 510 | 510 | |
| 511 | 511 | foreach ($searchResults as $index => $result) { |
| 512 | - switch($returnType) { |
|
| 512 | + switch ($returnType) { |
|
| 513 | 513 | |
| 514 | 514 | case KaliopQuery::RETURN_CONTENTS: |
| 515 | 515 | try { |
| 516 | 516 | $results[$index] = new SearchHit( |
| 517 | 517 | [ |
| 518 | 518 | 'valueObject' => $this->contentService->loadContent($result['id']), |
| 519 | - 'score' => isset($result['score'])? $result['score'] : null, |
|
| 520 | - 'highlight' => isset($result['highlight'])? $result['highlight'] : null, |
|
| 521 | - 'elevated' => isset($result['elevated'])? $result['elevated'] : null, |
|
| 519 | + 'score' => isset($result['score']) ? $result['score'] : null, |
|
| 520 | + 'highlight' => isset($result['highlight']) ? $result['highlight'] : null, |
|
| 521 | + 'elevated' => isset($result['elevated']) ? $result['elevated'] : null, |
|
| 522 | 522 | /// @todo decide what is the correct value for 'index': guid, installation_id/guid ? |
| 523 | 523 | //'index' => isset($result['guid'])? $result['guid'] : null, |
| 524 | 524 | ] |
@@ -559,9 +559,9 @@ discard block |
||
| 559 | 559 | $results[$index] = new SearchHit( |
| 560 | 560 | [ |
| 561 | 561 | 'valueObject' => $result, |
| 562 | - 'score' => isset($result['score'])? $result['score'] : null, |
|
| 563 | - 'highlight' => isset($result['highlight'])? $result['highlight'] : null, |
|
| 564 | - 'elevated' => isset($result['elevated'])? $result['elevated'] : null, |
|
| 562 | + 'score' => isset($result['score']) ? $result['score'] : null, |
|
| 563 | + 'highlight' => isset($result['highlight']) ? $result['highlight'] : null, |
|
| 564 | + 'elevated' => isset($result['elevated']) ? $result['elevated'] : null, |
|
| 565 | 565 | /// @todo decide what is the correct value for 'index': guid, installation_id/guid ? |
| 566 | 566 | //'index' => isset($result['guid'])? $result['guid'] : null, |
| 567 | 567 | ] |