We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -212,7 +212,7 @@ discard block |
||
| 212 | 212 | // Add filter query for date search |
| 213 | 213 | if (!empty($this->searchParams['dateFrom']) && !empty($this->searchParams['dateTo'])) { |
| 214 | 214 | // combine dateFrom and dateTo into range search |
| 215 | - $params['filterquery'][]['query'] = '{!join from=' . $fields['uid'] . ' to=' . $fields['uid'] . '}'. $fields['date'] . ':[' . $this->searchParams['dateFrom'] . ' TO ' . $this->searchParams['dateTo'] . ']'; |
|
| 215 | + $params['filterquery'][]['query'] = '{!join from=' . $fields['uid'] . ' to=' . $fields['uid'] . '}' . $fields['date'] . ':[' . $this->searchParams['dateFrom'] . ' TO ' . $this->searchParams['dateTo'] . ']'; |
|
| 216 | 216 | } |
| 217 | 217 | |
| 218 | 218 | // Add filter query for faceting. |
@@ -243,16 +243,16 @@ discard block |
||
| 243 | 243 | $virtualCollectionsQueryString = ''; |
| 244 | 244 | foreach ($this->collection as $collectionEntry) { |
| 245 | 245 | // check for virtual collections query string |
| 246 | - if($collectionEntry->getIndexSearch()) { |
|
| 247 | - $virtualCollectionsQueryString .= empty($virtualCollectionsQueryString) ? '(' . $collectionEntry->getIndexSearch() . ')' : ' OR ('. $collectionEntry->getIndexSearch() . ')' ; |
|
| 246 | + if ($collectionEntry->getIndexSearch()) { |
|
| 247 | + $virtualCollectionsQueryString .= empty($virtualCollectionsQueryString) ? '(' . $collectionEntry->getIndexSearch() . ')' : ' OR (' . $collectionEntry->getIndexSearch() . ')'; |
|
| 248 | 248 | } else { |
| 249 | 249 | $collectionsQueryString .= empty($collectionsQueryString) ? '"' . $collectionEntry->getIndexName() . '"' : ' OR "' . $collectionEntry->getIndexName() . '"'; |
| 250 | 250 | } |
| 251 | 251 | } |
| 252 | 252 | |
| 253 | 253 | // distinguish between simple collection browsing and actual searching within the collection(s) |
| 254 | - if(!empty($collectionsQueryString)) { |
|
| 255 | - if(empty($query)) { |
|
| 254 | + if (!empty($collectionsQueryString)) { |
|
| 255 | + if (empty($query)) { |
|
| 256 | 256 | $collectionsQueryString = '(collection_faceting:(' . $collectionsQueryString . ') AND toplevel:true AND partof:0)'; |
| 257 | 257 | } else { |
| 258 | 258 | $collectionsQueryString = '(collection_faceting:(' . $collectionsQueryString . '))'; |
@@ -260,7 +260,7 @@ discard block |
||
| 260 | 260 | } |
| 261 | 261 | |
| 262 | 262 | // virtual collections might query documents that are neither toplevel:true nor partof:0 and need to be searched separatly |
| 263 | - if(!empty($virtualCollectionsQueryString)) { |
|
| 263 | + if (!empty($virtualCollectionsQueryString)) { |
|
| 264 | 264 | $virtualCollectionsQueryString = '(' . $virtualCollectionsQueryString . ')'; |
| 265 | 265 | } |
| 266 | 266 | |
@@ -337,8 +337,8 @@ discard block |
||
| 337 | 337 | } |
| 338 | 338 | if ($documents[$doc['uid']]) { |
| 339 | 339 | // translate language code if applicable |
| 340 | - if($doc['metadata']['language']) { |
|
| 341 | - foreach($doc['metadata']['language'] as $indexName => $language) { |
|
| 340 | + if ($doc['metadata']['language']) { |
|
| 341 | + foreach ($doc['metadata']['language'] as $indexName => $language) { |
|
| 342 | 342 | $doc['metadata']['language'][$indexName] = Helper::getLanguageName($doc['metadata']['language'][$indexName]); |
| 343 | 343 | } |
| 344 | 344 | } |
@@ -438,8 +438,8 @@ discard block |
||
| 438 | 438 | |
| 439 | 439 | foreach ($result['documents'] as $doc) { |
| 440 | 440 | // translate language code if applicable |
| 441 | - if($doc['metadata']['language']) { |
|
| 442 | - foreach($doc['metadata']['language'] as $indexName => $language) { |
|
| 441 | + if ($doc['metadata']['language']) { |
|
| 442 | + foreach ($doc['metadata']['language'] as $indexName => $language) { |
|
| 443 | 443 | $doc['metadata']['language'][$indexName] = Helper::getLanguageName($doc['metadata']['language'][$indexName]); |
| 444 | 444 | } |
| 445 | 445 | } |
@@ -337,7 +337,7 @@ discard block |
||
| 337 | 337 | $solrDoc->setField('title', $metadata['title'][0], self::$fields['fieldboost']['title']); |
| 338 | 338 | $solrDoc->setField('volume', $metadata['volume'][0], self::$fields['fieldboost']['volume']); |
| 339 | 339 | // verify date formatting |
| 340 | - if(strtotime($metadata['date'][0])) { |
|
| 340 | + if (strtotime($metadata['date'][0])) { |
|
| 341 | 341 | // do not alter dates YYYY or YYYY-MM or YYYY-MM-DD |
| 342 | 342 | if ( |
| 343 | 343 | preg_match("/^[\d]{4}$/", $metadata['date'][0]) |
@@ -346,7 +346,7 @@ discard block |
||
| 346 | 346 | ) { |
| 347 | 347 | $solrDoc->setField('date', $metadata['date'][0]); |
| 348 | 348 | // change date YYYYMMDD to YYYY-MM-DD |
| 349 | - } elseif (preg_match("/^[\d]{8}$/", $metadata['date'][0])){ |
|
| 349 | + } elseif (preg_match("/^[\d]{8}$/", $metadata['date'][0])) { |
|
| 350 | 350 | $solrDoc->setField('date', date("Y-m-d", strtotime($metadata['date'][0]))); |
| 351 | 351 | // convert any datetime to proper ISO extended datetime format and timezone for SOLR |
| 352 | 352 | } else { |
@@ -484,7 +484,7 @@ discard block |
||
| 484 | 484 | !empty($data) |
| 485 | 485 | && substr($index_name, -8) == '_sorting' |
| 486 | 486 | ) { |
| 487 | - $solrDoc->setField($index_name , $doc->metadataArray[$doc->toplevelId][$index_name]); |
|
| 487 | + $solrDoc->setField($index_name, $doc->metadataArray[$doc->toplevelId][$index_name]); |
|
| 488 | 488 | } |
| 489 | 489 | } |
| 490 | 490 | } |
@@ -127,7 +127,7 @@ discard block |
||
| 127 | 127 | foreach ($year['children'] as $month) { |
| 128 | 128 | foreach ($month['children'] as $day) { |
| 129 | 129 | foreach ($day['children'] as $issue) { |
| 130 | - $title = $issue['label'] ?: $issue['orderlabel']; |
|
| 130 | + $title = $issue['label'] ? : $issue['orderlabel']; |
|
| 131 | 131 | if (strtotime($title) !== false) { |
| 132 | 132 | $title = strftime('%x', strtotime($title)); |
| 133 | 133 | } |
@@ -218,8 +218,8 @@ discard block |
||
| 218 | 218 | $this->view->assign('calendarData', $calendarData); |
| 219 | 219 | $this->view->assign('documentId', $this->document->getUid()); |
| 220 | 220 | $this->view->assign('yearLinkTitle', $yearLinkTitle); |
| 221 | - $this->view->assign('parentDocumentId', $this->document->getPartof() ?: $this->document->getDoc()->tableOfContents[0]['points']); |
|
| 222 | - $this->view->assign('allYearDocTitle', $this->document->getDoc()->getTitle($this->document->getPartof()) ?: $this->document->getDoc()->tableOfContents[0]['label']); |
|
| 221 | + $this->view->assign('parentDocumentId', $this->document->getPartof() ? : $this->document->getDoc()->tableOfContents[0]['points']); |
|
| 222 | + $this->view->assign('allYearDocTitle', $this->document->getDoc()->getTitle($this->document->getPartof()) ? : $this->document->getDoc()->tableOfContents[0]['label']); |
|
| 223 | 223 | } |
| 224 | 224 | |
| 225 | 225 | /** |
@@ -255,7 +255,7 @@ discard block |
||
| 255 | 255 | |
| 256 | 256 | if (empty($yearLabel)) { |
| 257 | 257 | // if neither order nor orderlabel is set, use the id... |
| 258 | - $yearLabel = (string)$id; |
|
| 258 | + $yearLabel = (string) $id; |
|
| 259 | 259 | } |
| 260 | 260 | |
| 261 | 261 | $years[] = [ |
@@ -294,7 +294,7 @@ discard block |
||
| 294 | 294 | if (is_int($key)) { |
| 295 | 295 | $yearFilled[] = $yearArray[$key]; |
| 296 | 296 | } else { |
| 297 | - $yearFilled[] = ['title' => $min+$i, 'documentId' => '']; |
|
| 297 | + $yearFilled[] = ['title' => $min + $i, 'documentId' => '']; |
|
| 298 | 298 | } |
| 299 | 299 | } |
| 300 | 300 | $yearArray = $yearFilled; |