We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -438,7 +438,7 @@ |
||
| 438 | 438 | return ''; |
| 439 | 439 | } |
| 440 | 440 | |
| 441 | - $makeCacheKey = function ($pid, $uid) { |
|
| 441 | + $makeCacheKey = function($pid, $uid) { |
|
| 442 | 442 | return $pid . '.' . $uid; |
| 443 | 443 | }; |
| 444 | 444 | |
@@ -34,7 +34,7 @@ |
||
| 34 | 34 | |
| 35 | 35 | $view->assign( |
| 36 | 36 | 'configObject', |
| 37 | - [ 'wrap' => 'all.wrap = <article class="shlb-metadata-text-item metadata-title">|</article> |
|
| 37 | + ['wrap' => 'all.wrap = <article class="shlb-metadata-text-item metadata-title">|</article> |
|
| 38 | 38 | key.wrap = <label>|</label> |
| 39 | 39 | value.required = 1 |
| 40 | 40 | value.wrap = <li>|</li>' |
@@ -331,7 +331,7 @@ discard block |
||
| 331 | 331 | $pages = []; |
| 332 | 332 | $pagesSect = []; |
| 333 | 333 | $aRange = []; |
| 334 | - $nRange = 5; // ToDo: should be made configurable |
|
| 334 | + $nRange = 5; // ToDo: should be made configurable |
|
| 335 | 335 | |
| 336 | 336 | // lower limit of the range |
| 337 | 337 | $nBottom = $currentPageNumber - $nRange; |
@@ -346,7 +346,7 @@ discard block |
||
| 346 | 346 | |
| 347 | 347 | // check whether the first screen page is > 1, if yes then points must be added |
| 348 | 348 | if ($aRange[0] > 1) { |
| 349 | - array_push($pagesSect, ['label' => '...','startRecordNumber' => '...']); |
|
| 349 | + array_push($pagesSect, ['label' => '...', 'startRecordNumber' => '...']); |
|
| 350 | 350 | }; |
| 351 | 351 | $lastStartRecordNumberGrid = 0; // due to validity outside the loop |
| 352 | 352 | foreach (range($firstPage, $lastPage) as $i) { |
@@ -389,7 +389,7 @@ discard block |
||
| 389 | 389 | |
| 390 | 390 | // Check if screen page is in range |
| 391 | 391 | if (in_array($i, $aRange)) { |
| 392 | - array_push($pagesSect, ['label' => $i,'startRecordNumber' => $startRecordNumber]); |
|
| 392 | + array_push($pagesSect, ['label' => $i, 'startRecordNumber' => $startRecordNumber]); |
|
| 393 | 393 | }; |
| 394 | 394 | }; |
| 395 | 395 | }; |
@@ -334,7 +334,7 @@ |
||
| 334 | 334 | * |
| 335 | 335 | * @return void |
| 336 | 336 | */ |
| 337 | - private function addCollections(Document &$document, array $collections): void |
|
| 337 | + private function addCollections(Document & $document, array $collections): void |
|
| 338 | 338 | { |
| 339 | 339 | foreach ($collections as $collection) { |
| 340 | 340 | $documentCollection = $this->collectionRepository->findOneByIndexName($collection); |
@@ -82,7 +82,7 @@ discard block |
||
| 82 | 82 | // extract collection(s) from collection parameter |
| 83 | 83 | $collections = []; |
| 84 | 84 | if (array_key_exists('collection', $this->searchParams)) { |
| 85 | - foreach(explode(',', $this->searchParams['collection']) as $collectionEntry) { |
|
| 85 | + foreach (explode(',', $this->searchParams['collection']) as $collectionEntry) { |
|
| 86 | 86 | $collections[] = $this->collectionRepository->findByUid((int) $collectionEntry); |
| 87 | 87 | } |
| 88 | 88 | } |
@@ -113,7 +113,7 @@ discard block |
||
| 113 | 113 | $simplePagination = new SimplePagination($solrPaginator); |
| 114 | 114 | |
| 115 | 115 | $pagination = $this->buildSimplePagination($simplePagination, $solrPaginator); |
| 116 | - $this->view->assignMultiple([ 'pagination' => $pagination, 'paginator' => $solrPaginator ]); |
|
| 116 | + $this->view->assignMultiple(['pagination' => $pagination, 'paginator' => $solrPaginator]); |
|
| 117 | 117 | } |
| 118 | 118 | |
| 119 | 119 | $this->view->assign('viewData', $this->viewData); |
@@ -347,7 +347,7 @@ discard block |
||
| 347 | 347 | } |
| 348 | 348 | } |
| 349 | 349 | } |
| 350 | - return $label ?: $orderLabel; |
|
| 350 | + return $label ? : $orderLabel; |
|
| 351 | 351 | } |
| 352 | 352 | |
| 353 | 353 | /** |
@@ -382,7 +382,7 @@ discard block |
||
| 382 | 382 | { |
| 383 | 383 | usort( |
| 384 | 384 | $menu[0]['_SUB_MENU'], |
| 385 | - function ($firstElement, $secondElement) { |
|
| 385 | + function($firstElement, $secondElement) { |
|
| 386 | 386 | if (!empty($firstElement['orderlabel'])) { |
| 387 | 387 | return $firstElement['orderlabel'] <=> $secondElement['orderlabel']; |
| 388 | 388 | } |
@@ -122,7 +122,7 @@ discard block |
||
| 122 | 122 | } |
| 123 | 123 | |
| 124 | 124 | if (isset($listRequestData['searchParameter']) && is_array($listRequestData['searchParameter'])) { |
| 125 | - $this->searchParams = array_merge($this->searchParams ?: [], $listRequestData['searchParameter']); |
|
| 125 | + $this->searchParams = array_merge($this->searchParams ? : [], $listRequestData['searchParameter']); |
|
| 126 | 126 | $listViewSearch = true; |
| 127 | 127 | $GLOBALS['TSFE']->fe_user->setKey('ses', 'search', $this->searchParams); |
| 128 | 128 | } |
@@ -190,7 +190,7 @@ discard block |
||
| 190 | 190 | $simplePagination = new SimplePagination($solrPaginator); |
| 191 | 191 | |
| 192 | 192 | $pagination = $this->buildSimplePagination($simplePagination, $solrPaginator); |
| 193 | - $this->view->assignMultiple([ 'pagination' => $pagination, 'paginator' => $solrPaginator ]); |
|
| 193 | + $this->view->assignMultiple(['pagination' => $pagination, 'paginator' => $solrPaginator]); |
|
| 194 | 194 | } |
| 195 | 195 | |
| 196 | 196 | $this->view->assign('documents', !empty($solrResults) ? $solrResults : []); |
@@ -413,7 +413,7 @@ discard block |
||
| 413 | 413 | // Add filter query for date search |
| 414 | 414 | if (!empty($this->searchParams['dateFrom']) && !empty($this->searchParams['dateTo'])) { |
| 415 | 415 | // combine dateFrom and dateTo into range search |
| 416 | - $params['filterquery'][]['query'] = '{!join from=' . $fields['uid'] . ' to=' . $fields['uid'] . '}'. $fields['date'] . ':[' . $this->searchParams['dateFrom'] . ' TO ' . $this->searchParams['dateTo'] . ']'; |
|
| 416 | + $params['filterquery'][]['query'] = '{!join from=' . $fields['uid'] . ' to=' . $fields['uid'] . '}' . $fields['date'] . ':[' . $this->searchParams['dateFrom'] . ' TO ' . $this->searchParams['dateTo'] . ']'; |
|
| 417 | 417 | } |
| 418 | 418 | |
| 419 | 419 | // Add filter query for faceting. |
@@ -531,8 +531,8 @@ discard block |
||
| 531 | 531 | if ($this->searchParams['fulltext'] == '1') { |
| 532 | 532 | $searchResult['snippet'] = $doc['snippet']; |
| 533 | 533 | $searchResult['highlight'] = $doc['highlight']; |
| 534 | - $searchResult['highlight_word'] = preg_replace('/^;|;$/', '', // remove ; at beginning or end |
|
| 535 | - preg_replace('/;+/', ';', // replace any multiple of ; with a single ; |
|
| 534 | + $searchResult['highlight_word'] = preg_replace('/^;|;$/', '', // remove ; at beginning or end |
|
| 535 | + preg_replace('/;+/', ';', // replace any multiple of ; with a single ; |
|
| 536 | 536 | preg_replace('/[{~\d*}{\s+}{^=*\d+.*\d*}`~!@#$%\^&*()_|+-=?;:\'",.<>\{\}\[\]\\\]/', ';', $this->searchParams['query']))); // replace search operators and special characters with ; |
| 537 | 537 | } |
| 538 | 538 | $documents[$doc['uid']]['searchResults'][] = $searchResult; |
@@ -860,7 +860,7 @@ discard block |
||
| 860 | 860 | private function translateLanguageCode(&$doc): void |
| 861 | 861 | { |
| 862 | 862 | if (array_key_exists('language', $doc['metadata'])) { |
| 863 | - foreach($doc['metadata']['language'] as $indexName => $language) { |
|
| 863 | + foreach ($doc['metadata']['language'] as $indexName => $language) { |
|
| 864 | 864 | $doc['metadata']['language'][$indexName] = Helper::getLanguageName($language); |
| 865 | 865 | } |
| 866 | 866 | } |
@@ -354,7 +354,7 @@ |
||
| 354 | 354 | $solrDoc->setField('title', $metadata['title'][0]); |
| 355 | 355 | $solrDoc->setField('volume', $metadata['volume'][0]); |
| 356 | 356 | // verify date formatting |
| 357 | - if(strtotime($metadata['date'][0])) { |
|
| 357 | + if (strtotime($metadata['date'][0])) { |
|
| 358 | 358 | $solrDoc->setField('date', self::getFormattedDate($metadata['date'][0])); |
| 359 | 359 | } |
| 360 | 360 | $solrDoc->setField('record_id', $metadata['record_id'][0]); |