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; |