We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -186,7 +186,8 @@ discard block |
||
| 186 | 186 | * |
| 187 | 187 | * @return void |
| 188 | 188 | */ |
| 189 | - protected function configureProxyUrl(&$url) { |
|
| 189 | + protected function configureProxyUrl(&$url) |
|
| 190 | + { |
|
| 190 | 191 | $this->uriBuilder->reset() |
| 191 | 192 | ->setTargetPageUid($GLOBALS['TSFE']->id) |
| 192 | 193 | ->setCreateAbsoluteUri(!empty($this->settings['forceAbsoluteUrl'])) |
@@ -283,7 +284,8 @@ discard block |
||
| 283 | 284 | * |
| 284 | 285 | * @return void |
| 285 | 286 | */ |
| 286 | - protected function setPage() { |
|
| 287 | + protected function setPage() |
|
| 288 | + { |
|
| 287 | 289 | if (!empty($this->requestData['logicalPage'])) { |
| 288 | 290 | $this->requestData['page'] = $this->document->getCurrentDocument()->getPhysicalPage($this->requestData['logicalPage']); |
| 289 | 291 | // The logical page parameter should not appear again |
@@ -300,7 +302,8 @@ discard block |
||
| 300 | 302 | * |
| 301 | 303 | * @return void |
| 302 | 304 | */ |
| 303 | - protected function setDefaultPage() { |
|
| 305 | + protected function setDefaultPage() |
|
| 306 | + { |
|
| 304 | 307 | // Set default values if not set. |
| 305 | 308 | // $this->requestData['page'] may be integer or string (physical structure @ID) |
| 306 | 309 | if ( |
@@ -125,7 +125,7 @@ discard block |
||
| 125 | 125 | foreach ($year['children'] as $month) { |
| 126 | 126 | foreach ($month['children'] as $day) { |
| 127 | 127 | foreach ($day['children'] as $issue) { |
| 128 | - $title = $issue['label'] ?: $issue['orderlabel']; |
|
| 128 | + $title = $issue['label'] ? : $issue['orderlabel']; |
|
| 129 | 129 | if (strtotime($title) !== false) { |
| 130 | 130 | $title = strftime('%x', strtotime($title)); |
| 131 | 131 | } |
@@ -216,8 +216,8 @@ discard block |
||
| 216 | 216 | $this->view->assign('calendarData', $calendarData); |
| 217 | 217 | $this->view->assign('documentId', $this->document->getUid()); |
| 218 | 218 | $this->view->assign('yearLinkTitle', $yearLinkTitle); |
| 219 | - $this->view->assign('parentDocumentId', $this->document->getPartof() ?: $this->document->getCurrentDocument()->tableOfContents[0]['points']); |
|
| 220 | - $this->view->assign('allYearDocTitle', $this->document->getCurrentDocument()->getTitle($this->document->getPartof()) ?: $this->document->getCurrentDocument()->tableOfContents[0]['label']); |
|
| 219 | + $this->view->assign('parentDocumentId', $this->document->getPartof() ? : $this->document->getCurrentDocument()->tableOfContents[0]['points']); |
|
| 220 | + $this->view->assign('allYearDocTitle', $this->document->getCurrentDocument()->getTitle($this->document->getPartof()) ? : $this->document->getCurrentDocument()->tableOfContents[0]['label']); |
|
| 221 | 221 | } |
| 222 | 222 | |
| 223 | 223 | /** |
@@ -253,7 +253,7 @@ discard block |
||
| 253 | 253 | |
| 254 | 254 | if (empty($yearLabel)) { |
| 255 | 255 | // if neither order nor orderlabel is set, use the id... |
| 256 | - $yearLabel = (string)$id; |
|
| 256 | + $yearLabel = (string) $id; |
|
| 257 | 257 | } |
| 258 | 258 | |
| 259 | 259 | $years[] = [ |
@@ -292,7 +292,7 @@ discard block |
||
| 292 | 292 | if (is_int($key)) { |
| 293 | 293 | $yearFilled[] = $yearArray[$key]; |
| 294 | 294 | } else { |
| 295 | - $yearFilled[] = ['title' => $min+$i, 'documentId' => '']; |
|
| 295 | + $yearFilled[] = ['title' => $min + $i, 'documentId' => '']; |
|
| 296 | 296 | } |
| 297 | 297 | } |
| 298 | 298 | $yearArray = $yearFilled; |
@@ -239,7 +239,8 @@ discard block |
||
| 239 | 239 | * |
| 240 | 240 | * @return void |
| 241 | 241 | */ |
| 242 | - private function getAllLogicalUnits() { |
|
| 242 | + private function getAllLogicalUnits() |
|
| 243 | + { |
|
| 243 | 244 | if ( |
| 244 | 245 | !empty($this->requestData['page']) |
| 245 | 246 | && !empty($this->document->getCurrentDocument()->physicalStructure) |
@@ -265,7 +266,8 @@ discard block |
||
| 265 | 266 | * |
| 266 | 267 | * @return string |
| 267 | 268 | */ |
| 268 | - private function getTranslatedType($type) { |
|
| 269 | + private function getTranslatedType($type) |
|
| 270 | + { |
|
| 269 | 271 | return Helper::translate($type, 'tx_dlf_structures', $this->settings['storagePid']); |
| 270 | 272 | } |
| 271 | 273 | |
@@ -285,7 +287,8 @@ discard block |
||
| 285 | 287 | * |
| 286 | 288 | * @return bool |
| 287 | 289 | */ |
| 288 | - private function isMultiElement($type) { |
|
| 290 | + private function isMultiElement($type) |
|
| 291 | + { |
|
| 289 | 292 | return $type === 'multivolume_work' || $type === 'multipart_manuscript'; |
| 290 | 293 | } |
| 291 | 294 | /** |
@@ -297,7 +300,8 @@ discard block |
||
| 297 | 300 | * |
| 298 | 301 | * @return string |
| 299 | 302 | */ |
| 300 | - private function setTitle($entry) { |
|
| 303 | + private function setTitle($entry) |
|
| 304 | + { |
|
| 301 | 305 | if (empty($entry['label']) && empty($entry['orderlabel'])) { |
| 302 | 306 | foreach ($this->settings['titleReplacements'] as $titleReplacement) { |
| 303 | 307 | if ($entry['type'] == $titleReplacement['type']) { |
@@ -327,7 +331,8 @@ discard block |
||
| 327 | 331 | * |
| 328 | 332 | * @return void |
| 329 | 333 | */ |
| 330 | - private function sortMenu(&$menu) { |
|
| 334 | + private function sortMenu(&$menu) |
|
| 335 | + { |
|
| 331 | 336 | if ($menu[0]['type'] == $this->getTranslatedType("newspaper")) { |
| 332 | 337 | $this->sortSubMenu($menu); |
| 333 | 338 | } |
@@ -345,8 +350,10 @@ discard block |
||
| 345 | 350 | * |
| 346 | 351 | * @return void |
| 347 | 352 | */ |
| 348 | - private function sortSubMenu(&$menu) { |
|
| 349 | - usort($menu[0]['_SUB_MENU'], function ($firstElement, $secondElement) { |
|
| 353 | + private function sortSubMenu(&$menu) |
|
| 354 | + { |
|
| 355 | + usort($menu[0]['_SUB_MENU'], function ($firstElement, $secondElement) |
|
| 356 | + { |
|
| 350 | 357 | if (!empty($firstElement['orderlabel'])) { |
| 351 | 358 | return $firstElement['orderlabel'] <=> $secondElement['orderlabel']; |
| 352 | 359 | } |