Erikmitk / kitodo-presentation

Classes/Controller/PageViewController.php

Braces +4 added lines, -8 removed lines

@@ -30,8 +30,7 @@ 
  *
  * @access public
  */
-class PageViewController extends AbstractController
-{
+class PageViewController extends AbstractController {
     /**
      * @access protected
      * @var array Holds the controls to add to the map
@@ -256,8 +255,7 @@ 
      * @param $page
      * @return false|int|mixed|string|null
      */
-    public function convertMeasureOrPage($document, $measure = null, $page = null)
-    {
+    public function convertMeasureOrPage($document, $measure = null, $page = null) {
         $return = null;
         $measure2Page = array_column($document->musicalStructure, 'page');
         if ($measure) {
@@ -273,8 +271,7 @@ 
      * Action to add multiple mets sources (multi page view)
      * @return void
      */
-    public function addDocumentAction(FormAddDocument $formAddDocument)
-    {
+    public function addDocumentAction(FormAddDocument $formAddDocument) {
         if (GeneralUtility::isValidUrl($formAddDocument->getLocation())) {
             $nextMultipleSourceKey = 0;
             if ($this->requestData['multipleSource']) {
@@ -367,8 +364,7 @@ 
      *
      * @return array URL and MIME type of fulltext file
      */
-    protected function getScore(int $page, MetsDocument $specificDoc = null)
-    {
+    protected function getScore(int $page, MetsDocument $specificDoc = null) {
         $score = [];
         $loc = '';
         if ($specificDoc) {

Upper-Lower-Casing +12 added lines, -12 removed lines

@@ -174,7 +174,7 @@ 
             // convert either page or measure if requestData exists
             if ($this->requestData['docPage'][$i] && empty($this->requestData['docMeasure'][$i])) {
                 // convert document page information to measure count information
-                $this->requestData['docMeasure'][$i] = $this->convertMeasureOrPage($document, null, $this->requestData['docPage'][$i]);
+                $this->requestData['docMeasure'][$i] = $this->convertMeasureOrPage($document, NULL, $this->requestData['docPage'][$i]);
 
             } elseif ((empty($this->requestData['docPage'][$i]) || $this->requestData['docPage'][$i] === 1) && $this->requestData['docMeasure'][$i]) {
                 $this->requestData['docPage'][$i] = $this->convertMeasureOrPage($document, $this->requestData['docMeasure'][$i]);
@@ -256,9 +256,9 @@ 
      * @param $page
      * @return false|int|mixed|string|null
      */
-    public function convertMeasureOrPage($document, $measure = null, $page = null)
+    public function convertMeasureOrPage($document, $measure = NULL, $page = NULL)
     {
-        $return = null;
+        $return = NULL;
         $measure2Page = array_column($document->musicalStructure, 'page');
         if ($measure) {
             $return = $measure2Page[$measure];
@@ -303,7 +303,7 @@ 
      * @param int|null $docNumber
      * @return array
      */
-    protected function getMeasures(int $page, MetsDocument $specificDoc = null, $docNumber = null): array
+    protected function getMeasures(int $page, MetsDocument $specificDoc = NULL, $docNumber = NULL): array
     {
         if ($specificDoc) {
             $doc = $specificDoc;
@@ -367,7 +367,7 @@ 
      *
      * @return array URL and MIME type of fulltext file
      */
-    protected function getScore(int $page, MetsDocument $specificDoc = null)
+    protected function getScore(int $page, MetsDocument $specificDoc = NULL)
     {
         $score = [];
         $loc = '';
@@ -397,7 +397,7 @@ 
                     // Configure @action URL for form.
                     $uri = $this->uriBuilder->reset()
                         ->setTargetPageUid($this->pageUid)
-                        ->setCreateAbsoluteUri(!empty($this->settings['forceAbsoluteUrl']) ? true : false)
+                        ->setCreateAbsoluteUri(!empty($this->settings['forceAbsoluteUrl']) ? TRUE : FALSE)
                         ->setArguments(
                             [
                                 'eID' => 'tx_dlf_pageview_proxy',
@@ -466,7 +466,7 @@ 
             $jsViewer = 'tx_dlf_viewer = [];';
             $i = 0;
             foreach ($this->documentArray as $document) {
-                if ($document !== null) {
+                if ($document !== NULL) {
                     $docPage = $this->requestData['docPage'][$i];
                     $docImage = [];
                     $docFulltext = [];
@@ -480,7 +480,7 @@ 
                             $docPage = $measure2Page[$this->requestData['docMeasure'][$i]];
                         }
                     }
-                    if ($docPage == null) {
+                    if ($docPage == NULL) {
                         $docPage = 1;
                     }
                     $docImage[0] = $this->getImage($docPage, $document);
@@ -567,7 +567,7 @@ 
             if ($iiif instanceof ManifestInterface) {
                 $canvas = $iiif->getContainedResourceById($canvasId);
                 /* @var $canvas \Ubl\Iiif\Presentation\Common\Model\Resources\CanvasInterface */
-                if ($canvas != null && !empty($canvas->getPossibleTextAnnotationContainers(Motivation::PAINTING))) {
+                if ($canvas != NULL && !empty($canvas->getPossibleTextAnnotationContainers(Motivation::PAINTING))) {
                     $annotationContainers = [];
                     /*
                      *  TODO Analyzing the annotations on the server side requires loading the annotation lists / pages
@@ -577,11 +577,11 @@ 
                      *  On the other hand, server connections are potentially better than client connections. Downloading annotation lists
                      */
                     foreach ($canvas->getPossibleTextAnnotationContainers(Motivation::PAINTING) as $annotationContainer) {
-                        if (($textAnnotations = $annotationContainer->getTextAnnotations(Motivation::PAINTING)) != null) {
+                        if (($textAnnotations = $annotationContainer->getTextAnnotations(Motivation::PAINTING)) != NULL) {
                             foreach ($textAnnotations as $annotation) {
                                 if (
                                     $annotation->getBody()->getFormat() == 'text/plain'
-                                    && $annotation->getBody()->getChars() != null
+                                    && $annotation->getBody()->getChars() != NULL
                                 ) {
                                     $annotationListData = [];
                                     $annotationListData['uri'] = $annotationContainer->getId();
@@ -618,7 +618,7 @@ 
      *
      * @return array URL and MIME type of image file
      */
-    protected function getImage(int $page, MetsDocument $specificDoc = null): array
+    protected function getImage(int $page, MetsDocument $specificDoc = NULL): array
     {
         $image = [];
         // Get @USE value of METS fileGrp.

Spacing +20 added lines, -20 removed lines

@@ -181,18 +181,18 @@ 
             }
 
             $navigationArray[$i]['next'] = [
-                'tx_dlf[docPage][' . $i . ']' =>
+                'tx_dlf[docPage]['.$i.']' =>
                     MathUtility::forceIntegerInRange((int) $this->requestData['docPage'][$i] + 1, 1, $document->numPages, 1)
             ];
             $navigationArray[$i]['prev'] = [
-                'tx_dlf[docPage][' . $i . ']' =>
+                'tx_dlf[docPage]['.$i.']' =>
                     MathUtility::forceIntegerInRange((int) $this->requestData['docPage'][$i] - 1, 1, $document->numPages, 1)
             ];
 
             $navigateAllPageNext = array_merge(
                 $navigateAllPageNext,
                 [
-                    'tx_dlf[docPage][' . $i . ']' =>
+                    'tx_dlf[docPage]['.$i.']' =>
                         MathUtility::forceIntegerInRange((int) $this->requestData['docPage'][$i] + 1, 1, $document->numPages, 1)
                 ]
             );
@@ -200,7 +200,7 @@ 
             $navigateAllPagePrev = array_merge(
                 $navigateAllPagePrev,
                 [
-                    'tx_dlf[docPage][' . $i . ']' =>
+                    'tx_dlf[docPage]['.$i.']' =>
                         MathUtility::forceIntegerInRange((int) $this->requestData['docPage'][$i] - 1, 1, $document->numPages, 1)
                 ]
             );
@@ -208,7 +208,7 @@ 
             $navigateAllMeasureNext = array_merge(
                 $navigateAllMeasureNext,
                 [
-                    'tx_dlf[docMeasure][' . $i . ']' =>
+                    'tx_dlf[docMeasure]['.$i.']' =>
                         MathUtility::forceIntegerInRange((int) $this->requestData['docMeasure'][$i] + 1, 1, $document->numMeasures, 1)
                 ]
             );
@@ -216,19 +216,19 @@ 
             $navigateAllMeasurePrev = array_merge(
                 $navigateAllMeasurePrev,
                 [
-                    'tx_dlf[docMeasure][' . $i . ']' =>
+                    'tx_dlf[docMeasure]['.$i.']' =>
                         MathUtility::forceIntegerInRange((int) $this->requestData['docMeasure'][$i] - 1, 1, $document->numMeasures, 1)
                 ]
             );
 
             if ($document->numMeasures > 0) {
                 $navigationMeasureArray[$i]['next'] = [
-                    'tx_dlf[docMeasure][' . $i . ']' =>
+                    'tx_dlf[docMeasure]['.$i.']' =>
                         MathUtility::forceIntegerInRange((int) $this->requestData['docMeasure'][$i] + 1, 1, $document->numMeasures, 1)
                 ];
 
                 $navigationMeasureArray[$i]['prev'] = [
-                    'tx_dlf[docMeasure][' . $i . ']' =>
+                    'tx_dlf[docMeasure]['.$i.']' =>
                         MathUtility::forceIntegerInRange((int) $this->requestData['docMeasure'][$i] - 1, 1, $document->numMeasures, 1)
                 ];
             }
@@ -282,7 +282,7 @@ 
             }
             $params = array_merge(
                 ['tx_dlf' => $this->requestData],
-                ['tx_dlf[multipleSource][' . $nextMultipleSourceKey . ']' => $formAddDocument->getLocation()],
+                ['tx_dlf[multipleSource]['.$nextMultipleSourceKey.']' => $formAddDocument->getLocation()],
                 ['tx_dlf[multiview]' => 1]
             );
             $uriBuilder = $this->uriBuilder;
@@ -329,7 +329,7 @@ 
                             // build link for each measure
                             $params = [
                                 'tx_dlf' => $this->requestData,
-                                'tx_dlf[docMeasure][' . $docNumber . ']' => $i
+                                'tx_dlf[docMeasure]['.$docNumber.']' => $i
                             ];
                         } else {
                             // build link for each measure
@@ -413,7 +413,7 @@ 
         }
 
         if (empty($score)) {
-            $this->logger->notice('No score file found for page "' . $page . '" in fileGrps "' . $this->settings['fileGrpScore'] . '"');
+            $this->logger->notice('No score file found for page "'.$page.'" in fileGrps "'.$this->settings['fileGrpScore'].'"');
         }
         return $score;
     }
@@ -444,11 +444,11 @@ 
                 $fulltext['mimetype'] = $file['mimeType'];
                 break;
             } else {
-                $this->logger->notice('No full-text file found for page "' . $page . '" in fileGrp "' . $fileGrpFulltext . '"');
+                $this->logger->notice('No full-text file found for page "'.$page.'" in fileGrp "'.$fileGrpFulltext.'"');
             }
         }
         if (empty($fulltext)) {
-            $this->logger->notice('No full-text file found for page "' . $page . '" in fileGrps "' . $this->extConf['files']['fileGrpFulltext'] . '"');
+            $this->logger->notice('No full-text file found for page "'.$page.'" in fileGrps "'.$this->extConf['files']['fileGrpFulltext'].'"');
         }
         return $fulltext;
     }
@@ -508,7 +508,7 @@ 
                         'measureIdLinks' => $docMeasures['measureLinks']
                     ];
 
-                    $jsViewer .= 'tx_dlf_viewer[' . $i . '] = new dlfViewer(' . json_encode($viewer) . ');
+                    $jsViewer .= 'tx_dlf_viewer['.$i.'] = new dlfViewer('.json_encode($viewer).');
                             ';
                     $i++;
                 }
@@ -517,8 +517,8 @@ 
             // Viewer configuration.
             $viewerConfiguration = '$(document).ready(function() {
                     if (dlfUtils.exists(dlfViewer)) {
-                        ' . $jsViewer . '
-                        viewerCount = ' . ($i - 1) . ';
+                        ' . $jsViewer.'
+                        viewerCount = ' . ($i - 1).';
                     }
                 });';
         } else {
@@ -548,7 +548,7 @@ 
             // Viewer configuration.
             $viewerConfiguration = '$(document).ready(function() {
                     if (dlfUtils.exists(dlfViewer)) {
-                        tx_dlf_viewer = new dlfViewer(' . json_encode($viewer) . ');
+                        tx_dlf_viewer = new dlfViewer(' . json_encode($viewer).');
                     }
                 });';
         }
@@ -644,7 +644,7 @@ 
                     }
                     break;
                 } else {
-                    $this->logger->notice('No image file found for page "' . $page . '" in fileGrp "' . $fileGrpImages . '"');
+                    $this->logger->notice('No image file found for page "'.$page.'" in fileGrp "'.$fileGrpImages.'"');
                 }
 
             } else {
@@ -664,12 +664,12 @@ 
                     }
                     break;
                 } else {
-                    $this->logger->notice('No image file found for page "' . $page . '" in fileGrp "' . $fileGrpImages . '"');
+                    $this->logger->notice('No image file found for page "'.$page.'" in fileGrp "'.$fileGrpImages.'"');
                 }
             }
         }
         if (empty($image)) {
-            $this->logger->warning('No image file found for page "' . $page . '" in fileGrps "' . $this->extConf['files']['fileGrpImages'] . '"');
+            $this->logger->warning('No image file found for page "'.$page.'" in fileGrps "'.$this->extConf['files']['fileGrpImages'].'"');
         }
         return $image;
     }

Classes/Controller/AnnotationController.php

Braces +2 added lines, -4 removed lines

@@ -21,15 +21,13 @@
  * @subpackage dlf
  * @access public
  */
-class AnnotationController extends AbstractController
-{
+class AnnotationController extends AbstractController {
     /**
      * The main method of the plugin
      *
      * @return void
      */
-    public function mainAction()
-    {
+    public function mainAction() {
         $this->loadDocument();
 
         if (

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
         $this->loadDocument();
 
         if (
-            $this->document === null
-            || $this->document->getCurrentDocument() === null
+            $this->document === NULL
+            || $this->document->getCurrentDocument() === NULL
         ) {
             // Quit without doing anything if required variables are not set.
             return;

Classes/ViewHelpers/IsArrayViewHelper.php

Braces +3 added lines, -6 removed lines

@@ -18,10 +18,8 @@ 
 /**
  * Checks if the given subject is an array.
  */
-class IsArrayViewHelper extends AbstractViewHelper
-{
-    public function initializeArguments()
-    {
+class IsArrayViewHelper extends AbstractViewHelper {
+    public function initializeArguments() {
         parent::initializeArguments();
         $this->registerArgument('subject', 'string', 'The subject');
     }
@@ -29,8 +27,7 @@ 
     /**
      * @return bool
      */
-    public static function renderStatic(array $arguments, \Closure $renderChildrenClosure, RenderingContextInterface $renderingContext)
-    {
+    public static function renderStatic(array $arguments, \Closure $renderChildrenClosure, RenderingContextInterface $renderingContext) {
         $subject = $arguments['subject'];
         if ($subject === null) {
             $subject = $renderChildrenClosure();

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     public static function renderStatic(array $arguments, \Closure $renderChildrenClosure, RenderingContextInterface $renderingContext)
     {
         $subject = $arguments['subject'];
-        if ($subject === null) {
+        if ($subject === NULL) {
             $subject = $renderChildrenClosure();
         }
 

Classes/ViewHelpers/IsInArrayViewHelper.php

Braces +3 added lines, -6 removed lines

@@ -13,10 +13,8 @@ 
 
 use TYPO3Fluid\Fluid\Core\ViewHelper\AbstractViewHelper;
 
-class IsInArrayViewHelper extends AbstractViewHelper
-{
-    public function initializeArguments()
-    {
+class IsInArrayViewHelper extends AbstractViewHelper {
+    public function initializeArguments() {
         parent::initializeArguments();
         $this->registerArgument('needle', 'mixed', 'The searched value', true);
         $this->registerArgument('haystack', 'array', 'The array', true);
@@ -27,8 +25,7 @@ 
      *
      * @return bool
      */
-    public function render()
-    {
+    public function render() {
         $needle = $this->arguments['needle'];
         $haystack = $this->arguments['haystack'];
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -18,8 +18,8 @@
     public function initializeArguments()
     {
         parent::initializeArguments();
-        $this->registerArgument('needle', 'mixed', 'The searched value', true);
-        $this->registerArgument('haystack', 'array', 'The array', true);
+        $this->registerArgument('needle', 'mixed', 'The searched value', TRUE);
+        $this->registerArgument('haystack', 'array', 'The array', TRUE);
     }
 
     /**

Classes/ViewHelpers/ImplodeViewHelper.php

Braces +3 added lines, -6 removed lines

@@ -13,10 +13,8 @@ 
 
 use TYPO3Fluid\Fluid\Core\ViewHelper\AbstractViewHelper;
 
-class ImplodeViewHelper extends AbstractViewHelper
-{
-    public function initializeArguments()
-    {
+class ImplodeViewHelper extends AbstractViewHelper {
+    public function initializeArguments() {
         parent::initializeArguments();
         $this->registerArgument('value', 'array', 'The array to be imploded', true);
         $this->registerArgument('delimiter', 'string', 'The delimiter ', true);
@@ -27,8 +25,7 @@ 
      *
      * @return string
      */
-    public function render()
-    {
+    public function render() {
         $value = $this->arguments['value'];
         $delimiter = $this->arguments['delimiter'];
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -18,8 +18,8 @@
     public function initializeArguments()
     {
         parent::initializeArguments();
-        $this->registerArgument('value', 'array', 'The array to be imploded', true);
-        $this->registerArgument('delimiter', 'string', 'The delimiter ', true);
+        $this->registerArgument('value', 'array', 'The array to be imploded', TRUE);
+        $this->registerArgument('delimiter', 'string', 'The delimiter ', TRUE);
     }
 
     /**

Classes/Common/AnnotationRequest.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
      */
     public function __construct($apiUrl)
     {
-        $this->apiUrl =  trim($apiUrl, "/ ");
+        $this->apiUrl = trim($apiUrl, "/ ");
     }
 
 
@@ -59,7 +59,7 @@ 
     {
         $annotations = [];
 
-        $annotationData = $this->requestAnnotions($this->apiUrl . '?target=' . urlencode($id . '/*'));
+        $annotationData = $this->requestAnnotions($this->apiUrl.'?target='.urlencode($id.'/*'));
 
         if (array_key_exists('first', $annotationData)) {
             $annotationPageData = $annotationData['first'];

Braces +3 added lines, -6 removed lines

@@ -12,8 +12,7 @@ 
  * LICENSE.txt file that was distributed with this source code.
  */
 
-class AnnotationRequest
-{
+class AnnotationRequest {
     /**
      * @var string
      */
@@ -22,8 +21,7 @@ 
     /**
      * @param string $apiUrl The url of the annotation server api.
      */
-    public function __construct($apiUrl)
-    {
+    public function __construct($apiUrl) {
         $this->apiUrl =  trim($apiUrl, "/ ");
     }
 
@@ -55,8 +53,7 @@ 
      * @param string $id Document id (purl)
      * @return array
      */
-    public function getAll($id)
-    {
+    public function getAll($id) {
         $annotations = [];
 
         $annotationData = $this->requestAnnotions($this->apiUrl . '?target=' . urlencode($id . '/*'));

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         $jsonld = Helper::getUrl($url);
 
         if ($jsonld) {
-            $annotationData = json_decode($jsonld, true);
+            $annotationData = json_decode($jsonld, TRUE);
 
             if ($annotationData) {
                 return $annotationData;

Classes/Common/MetsDocument.php

Spacing +35 added lines, -35 removed lines

@@ -230,7 +230,7 @@ 
     {
         $file = $this->getFileInfo($id);
         if ($file['mimeType'] === 'application/vnd.kitodo.iiif') {
-            $file['location'] = (strrpos($file['location'], 'info.json') === strlen($file['location']) - 9) ? $file['location'] : (strrpos($file['location'], '/') === strlen($file['location']) ? $file['location'] . 'info.json' : $file['location'] . '/info.json');
+            $file['location'] = (strrpos($file['location'], 'info.json') === strlen($file['location']) - 9) ? $file['location'] : (strrpos($file['location'], '/') === strlen($file['location']) ? $file['location'].'info.json' : $file['location'].'/info.json');
             $conf = GeneralUtility::makeInstance(ExtensionConfiguration::class)->get(self::$extKey, 'iiif');
             IiifHelper::setUrlReader(IiifUrlReader::getInstance());
             IiifHelper::setMaxThumbnailHeight($conf['thumbnailHeight']);
@@ -240,9 +240,9 @@ 
                 return $service->getImageUrl();
             }
         } elseif ($file['mimeType'] === 'application/vnd.netfpx') {
-            $baseURL = $file['location'] . (strpos($file['location'], '?') === false ? '?' : '');
+            $baseURL = $file['location'].(strpos($file['location'], '?') === false ? '?' : '');
             // TODO CVT is an optional IIP server capability; in theory, capabilities should be determined in the object request with '&obj=IIP-server'
-            return $baseURL . '&CVT=jpeg';
+            return $baseURL.'&CVT=jpeg';
         }
         return $file['location'];
     }
@@ -271,14 +271,14 @@ 
      */
     public function getFileLocation(string $id): string
     {
-        $location = $this->mets->xpath('./mets:fileSec/mets:fileGrp/mets:file[@ID="' . $id . '"]/mets:FLocat[@LOCTYPE="URL"]');
+        $location = $this->mets->xpath('./mets:fileSec/mets:fileGrp/mets:file[@ID="'.$id.'"]/mets:FLocat[@LOCTYPE="URL"]');
         if (
             !empty($id)
             && !empty($location)
         ) {
             return (string) $location[0]->attributes('http://www.w3.org/1999/xlink')->href;
         } else {
-            $this->logger->warning('There is no file node with @ID "' . $id . '"');
+            $this->logger->warning('There is no file node with @ID "'.$id.'"');
             return '';
         }
     }
@@ -290,9 +290,9 @@ 
     {
         $mets = $this->mets
             ->xpath(
-                './mets:structMap[@TYPE="PHYSICAL"]' .
-                '//mets:div[@ID="' .  $pageId .  '"]' .
-                '/mets:fptr[@FILEID="' .  $fileId .  '"]' .
+                './mets:structMap[@TYPE="PHYSICAL"]'.
+                '//mets:div[@ID="'.$pageId.'"]'.
+                '/mets:fptr[@FILEID="'.$fileId.'"]'.
                 '/mets:area/@BEGIN'
             );
         return empty($mets) ? '' : $mets[0]->__toString();
@@ -304,14 +304,14 @@ 
      */
     public function getFileMimeType(string $id): string
     {
-        $mimetype = $this->mets->xpath('./mets:fileSec/mets:fileGrp/mets:file[@ID="' . $id . '"]/@MIMETYPE');
+        $mimetype = $this->mets->xpath('./mets:fileSec/mets:fileGrp/mets:file[@ID="'.$id.'"]/@MIMETYPE');
         if (
             !empty($id)
             && !empty($mimetype)
         ) {
             return (string) $mimetype[0];
         } else {
-            $this->logger->warning('There is no file node with @ID "' . $id . '" or no MIME type specified');
+            $this->logger->warning('There is no file node with @ID "'.$id.'" or no MIME type specified');
             return '';
         }
     }
@@ -331,7 +331,7 @@ 
             return $this->logicalUnits[$id];
         } elseif (!empty($id)) {
             // Get specified logical unit.
-            $divs = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="' . $id . '"]');
+            $divs = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="'.$id.'"]');
         } else {
             // Get all logical units at top level.
             $divs = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]/mets:div');
@@ -546,7 +546,7 @@ 
         $cPid = max($cPid, 0);
         if ($cPid == 0 && ($this->cPid || $this->pid)) {
             // Retain current PID.
-            $cPid = $this->cPid ?: $this->pid;
+            $cPid = $this->cPid ? : $this->pid;
         }
         return $cPid;
     }
@@ -613,7 +613,7 @@ 
         if (isset($this->fileInfos[$id]) || in_array('dmdSec', $metadataSections)) {
             return $metadata;
         } else {
-            $this->logger->warning('No supported descriptive metadata found for logical structure with @ID "' . $id . '"');
+            $this->logger->warning('No supported descriptive metadata found for logical structure with @ID "'.$id.'"');
             return [];
         }
     }
@@ -692,7 +692,7 @@ 
         if (!empty($this->logicalUnits[$id])) {
             return [$this->logicalUnits[$id]['type']];
         } else {
-            $struct = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="' . $id . '"]/@TYPE');
+            $struct = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="'.$id.'"]/@TYPE');
             if (!empty($struct)) {
                 return [(string) $struct[0]];
             }
@@ -849,17 +849,17 @@ 
             if ($resArray['format'] > 0 && !empty($resArray['xpath_sorting'])) {
                 $values = $domXPath->evaluate($resArray['xpath_sorting'], $domNode);
                 if ($values instanceof DOMNodeList && $values->length > 0) {
-                    $metadata[$indexName . '_sorting'][0] = trim((string) $values->item(0)->nodeValue);
+                    $metadata[$indexName.'_sorting'][0] = trim((string) $values->item(0)->nodeValue);
                 } elseif (!($values instanceof DOMNodeList)) {
-                    $metadata[$indexName . '_sorting'][0] = trim((string) $values);
+                    $metadata[$indexName.'_sorting'][0] = trim((string) $values);
                 }
             }
-            if (empty($metadata[$indexName . '_sorting'][0])) {
+            if (empty($metadata[$indexName.'_sorting'][0])) {
                 if (is_array($currentMetadata)) {
                     $sortingValue = implode(',', array_column($currentMetadata, 0));
-                    $metadata[$indexName . '_sorting'][0] = $sortingValue;
+                    $metadata[$indexName.'_sorting'][0] = $sortingValue;
                 } else {
-                    $metadata[$indexName . '_sorting'][0] = $currentMetadata;
+                    $metadata[$indexName.'_sorting'][0] = $currentMetadata;
                 }
             }
         }
@@ -920,11 +920,11 @@ 
                         return true;
                     }
                 } else {
-                    $this->logger->warning('Invalid class/method "' . $class . '->extractMetadata()" for metadata format "' . $this->mdSec[$dmdId]['type'] . '"');
+                    $this->logger->warning('Invalid class/method "'.$class.'->extractMetadata()" for metadata format "'.$this->mdSec[$dmdId]['type'].'"');
                 }
             }
         } else {
-            $this->logger->notice('Unsupported metadata format "' . $this->mdSec[$dmdId]['type'] . '" in ' . $mdSectionType . ' with @ID "' . $dmdId . '"');
+            $this->logger->notice('Unsupported metadata format "'.$this->mdSec[$dmdId]['type'].'" in '.$mdSectionType.' with @ID "'.$dmdId.'"');
         }
         return false;
     }
@@ -1077,7 +1077,7 @@ 
             $dmdIds = $this->logicalUnits[$id]['dmdId'] ?? '';
             $admIds = $this->logicalUnits[$id]['admId'] ?? '';
         } else {
-            $mdSec = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="' . $id . '"]')[0];
+            $mdSec = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="'.$id.'"]')[0];
             if ($mdSec) {
                 $dmdIds = (string) $mdSec->attributes()->DMDID;
                 $admIds = (string) $mdSec->attributes()->ADMID;
@@ -1107,7 +1107,7 @@ 
 
         return array_filter(
             $allMdIds,
-            function ($element) {
+            function($element) {
                 return !empty($element);
             }
         );
@@ -1133,7 +1133,7 @@ 
      */
     public function getStructureDepth(string $logId)
     {
-        $ancestors = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="' . $logId . '"]/ancestor::*');
+        $ancestors = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="'.$logId.'"]/ancestor::*');
         if (!empty($ancestors)) {
             return count($ancestors);
         } else {
@@ -1157,9 +1157,9 @@ 
             $this->registerNamespaces($this->mets);
         } else {
             if (!empty($location)) {
-                $this->logger->error('No METS part found in document with location "' . $location . '".');
+                $this->logger->error('No METS part found in document with location "'.$location.'".');
             } elseif (!empty($this->recordId)) {
-                $this->logger->error('No METS part found in document with recordId "' . $this->recordId . '".');
+                $this->logger->error('No METS part found in document with recordId "'.$this->recordId.'".');
             } else {
                 $this->logger->error('No METS part found in current document.');
             }
@@ -1180,7 +1180,7 @@ 
                 return true;
             }
         }
-        $this->logger->error('Could not load XML file from "' . $location . '"');
+        $this->logger->error('Could not load XML file from "'.$location.'"');
         return false;
     }
 
@@ -1303,10 +1303,10 @@ 
 
         if (!empty($mdType) && !empty($this->formats[(string) $mdType[0]])) {
             $type = (string) $mdType[0];
-            $xml = $element->xpath('./mets:mdWrap[@MDTYPE="' . $type . '"]/mets:xmlData/' . strtolower($type) . ':' . $this->formats[$type]['rootElement']);
+            $xml = $element->xpath('./mets:mdWrap[@MDTYPE="'.$type.'"]/mets:xmlData/'.strtolower($type).':'.$this->formats[$type]['rootElement']);
         } elseif (!empty($otherMdType) && !empty($this->formats[(string) $otherMdType[0]])) {
             $type = (string) $otherMdType[0];
-            $xml = $element->xpath('./mets:mdWrap[@MDTYPE="OTHER"][@OTHERMDTYPE="' . $type . '"]/mets:xmlData/' . strtolower($type) . ':' . $this->formats[$type]['rootElement']);
+            $xml = $element->xpath('./mets:mdWrap[@MDTYPE="OTHER"][@OTHERMDTYPE="'.$type.'"]/mets:xmlData/'.strtolower($type).':'.$this->formats[$type]['rootElement']);
         }
 
         if (empty($xml)) {
@@ -1513,7 +1513,7 @@ 
             // Get track info wtih begin end extent time for later assignment with musical
             if ((string) $elementNode['TYPE'] === 'track') {
                 foreach ($elementNode->children('http://www.loc.gov/METS/')->fptr as $fptr) {
-                    if (isset($fptr->area) &&  ((string) $fptr->area->attributes()->BETYPE === 'TIME')) {
+                    if (isset($fptr->area) && ((string) $fptr->area->attributes()->BETYPE === 'TIME')) {
                         // Check if file has valid @USE attribute.
                         if (!empty($fileUse[(string) $fptr->area->attributes()->FILEID])) {
                             $this->physicalStructureInfo[$elements[(int) $elementNode['ORDER']]]['tracks'][$fileUse[(string) $fptr->area->attributes()->FILEID]] = [
@@ -1567,9 +1567,9 @@ 
             || $forceReload
         ) {
             // Retain current PID.
-            $cPid = $this->cPid ?: $this->pid;
+            $cPid = $this->cPid ? : $this->pid;
             if (!$cPid) {
-                $this->logger->error('Invalid PID ' . $cPid . ' for structure definitions');
+                $this->logger->error('Invalid PID '.$cPid.' for structure definitions');
                 $this->thumbnailLoaded = true;
                 return $this->thumbnail;
             }
@@ -1606,7 +1606,7 @@ 
                 if (!empty($resArray['thumbnail'])) {
                     $strctType = Helper::getIndexNameFromUid($resArray['thumbnail'], 'tx_dlf_structures', $cPid);
                     // Check if this document has a structure element of the desired type.
-                    $strctIds = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@TYPE="' . $strctType . '"]/@ID');
+                    $strctIds = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@TYPE="'.$strctType.'"]/@ID');
                     if (!empty($strctIds)) {
                         $strctId = (string) $strctIds[0];
                     }
@@ -1629,7 +1629,7 @@ 
                     }
                 }
             } else {
-                $this->logger->error('No structure of type "' . $metadata['type'][0] . '" found in database');
+                $this->logger->error('No structure of type "'.$metadata['type'][0].'" found in database');
             }
             $this->thumbnailLoaded = true;
         }
@@ -1675,7 +1675,7 @@ 
     {
         if (empty($this->parentHref)) {
             // Get the closest ancestor of the current document which has a MPTR child.
-            $parentMptr = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="' . $this->toplevelId . '"]/ancestor::mets:div[./mets:mptr][1]/mets:mptr');
+            $parentMptr = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="'.$this->toplevelId.'"]/ancestor::mets:div[./mets:mptr][1]/mets:mptr');
             if (!empty($parentMptr)) {
                 $this->parentHref = (string) $parentMptr[0]->attributes('http://www.w3.org/1999/xlink')->href;
             }

Braces +8 added lines, -16 removed lines

@@ -72,8 +72,7 @@ 
  * @property-read SimpleXMLElement $mets this holds the XML file's METS part as SimpleXMLElement object
  * @property-read string $parentHref URL of the parent document (determined via mptr element), or empty string if none is available
  */
-final class MetsDocument extends AbstractDocument
-{
+final class MetsDocument extends AbstractDocument {
     /**
      * @access protected
      * @var string[] Subsections / tags that may occur within `<mets:amdSec>`
@@ -286,8 +285,7 @@ 
     /**
      * This gets the measure beginning of a page
      */
-    public function getPageBeginning($pageId, $fileId)
-    {
+    public function getPageBeginning($pageId, $fileId) {
         $mets = $this->mets
             ->xpath(
                 './mets:structMap[@TYPE="PHYSICAL"]' .
@@ -482,8 +480,7 @@ 
      *
      * @return ?string thumbnail or null if not found
      */
-    private function getThumbnail(string $id = '')
-    {
+    private function getThumbnail(string $id = '') {
         // Load plugin configuration.
         $extConf = GeneralUtility::makeInstance(ExtensionConfiguration::class)->get(self::$extKey, 'files');
         $fileGrpsThumb = GeneralUtility::trimExplode(',', $extConf['fileGrpThumbs']);
@@ -624,8 +621,7 @@ 
      * @param DOMNode $parentNode
      * @return array|false
      */
-    private function getSubentries($allSubentries, string $parentIndex, DOMNode $parentNode)
-    {
+    private function getSubentries($allSubentries, string $parentIndex, DOMNode $parentNode) {
         $domXPath = new DOMXPath($parentNode->ownerDocument);
         $this->registerNamespaces($domXPath);
         $theseSubentries = [];
@@ -655,8 +651,7 @@ 
      * @param $subentry
      * @return array
      */
-    private function getSubentryValue($values, $subentry)
-    {
+    private function getSubentryValue($values, $subentry) {
         $theseSubentries = [];
         if (
             ($values instanceof DOMNodeList
@@ -906,8 +901,7 @@ 
      *
      * @return bool true if extraction successful, false otherwise
      */
-    private function extractMetadataIfTypeSupported(string $dmdId, string $mdSectionType, array &$metadata)
-    {
+    private function extractMetadataIfTypeSupported(string $dmdId, string $mdSectionType, array &$metadata) {
         // Is this metadata format supported?
         if (!empty($this->formats[$this->mdSec[$dmdId]['type']])) {
             if (!empty($this->formats[$this->mdSec[$dmdId]['type']]['class'])) {
@@ -939,8 +933,7 @@ 
      *
      * @return array additional metadata data queried from database
      */
-    private function getAdditionalMetadataFromDatabase(int $cPid, string $dmdId)
-    {
+    private function getAdditionalMetadataFromDatabase(int $cPid, string $dmdId) {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
             ->getQueryBuilderForTable('tx_dlf_metadata');
         // Get hidden records, too.
@@ -1131,8 +1124,7 @@ 
     /**
      * @see AbstractDocument::getStructureDepth()
      */
-    public function getStructureDepth(string $logId)
-    {
+    public function getStructureDepth(string $logId) {
         $ancestors = $this->mets->xpath('./mets:structMap[@TYPE="LOGICAL"]//mets:div[@ID="' . $logId . '"]/ancestor::*');
         if (!empty($ancestors)) {
             return count($ancestors);

Upper-Lower-Casing +44 added lines, -44 removed lines

@@ -109,7 +109,7 @@ 
      *
      * @see MetsDocument::$mdSec
      */
-    protected bool $mdSecLoaded = false;
+    protected bool $mdSecLoaded = FALSE;
 
     /**
      * @access protected
@@ -131,7 +131,7 @@ 
      *
      * @see $fileGrps
      */
-    protected bool $fileGrpsLoaded = false;
+    protected bool $fileGrpsLoaded = FALSE;
 
     /**
      * @access protected
@@ -174,7 +174,7 @@ 
      * @var bool
      * @access protected
      */
-    protected bool $musicalStructureLoaded = false;
+    protected bool $musicalStructureLoaded = FALSE;
 
     /**
      * The holds the total number of measures
@@ -240,7 +240,7 @@ 
                 return $service->getImageUrl();
             }
         } elseif ($file['mimeType'] === 'application/vnd.netfpx') {
-            $baseURL = $file['location'] . (strpos($file['location'], '?') === false ? '?' : '');
+            $baseURL = $file['location'] . (strpos($file['location'], '?') === FALSE ? '?' : '');
             // TODO CVT is an optional IIP server capability; in theory, capabilities should be determined in the object request with '&obj=IIP-server'
             return $baseURL . '&CVT=jpeg';
         }
@@ -263,7 +263,7 @@ 
             $this->fileInfos[$id]['mimeType'] = $this->getFileMimeType($id);
         }
 
-        return $this->fileInfos[$id] ?? null;
+        return $this->fileInfos[$id] ?? NULL;
     }
 
     /**
@@ -319,7 +319,7 @@ 
     /**
      * @see AbstractDocument::getLogicalStructure()
      */
-    public function getLogicalStructure(string $id, bool $recursive = false): array
+    public function getLogicalStructure(string $id, bool $recursive = FALSE): array
     {
         $details = [];
         // Is the requested logical unit already loaded?
@@ -360,7 +360,7 @@ 
      *
      * @return array Array of the element's id, label, type and physical page indexes/mptr link
      */
-    protected function getLogicalStructureInfo(SimpleXMLElement $structure, bool $recursive = false): array
+    protected function getLogicalStructureInfo(SimpleXMLElement $structure, bool $recursive = FALSE): array
     {
         $attributes = $structure->attributes();
 
@@ -378,7 +378,7 @@ 
             'pagination' => '',
             'type' => isset($attributes['TYPE']) ? (string) $attributes['TYPE'] : '',
             'description' => '',
-            'thumbnailId' => null,
+            'thumbnailId' => NULL,
             'files' => [],
         ];
 
@@ -413,7 +413,7 @@ 
             $details['children'] = [];
             foreach ($structure->children('http://www.loc.gov/METS/')->div as $child) {
                 // Repeat for all children.
-                $details['children'][] = $this->getLogicalStructureInfo($child, true);
+                $details['children'][] = $this->getLogicalStructureInfo($child, TRUE);
             }
         }
         return $details;
@@ -459,7 +459,7 @@ 
             && array_key_exists($details['id'], $this->smLinks['l2p'])
         ) {
             // Link logical structure to the first corresponding physical page/track.
-            $details['points'] = max((int) array_search($this->smLinks['l2p'][$details['id']][0], $this->physicalStructure, true), 1);
+            $details['points'] = max((int) array_search($this->smLinks['l2p'][$details['id']][0], $this->physicalStructure, TRUE), 1);
             $details['thumbnailId'] = $this->getThumbnail();
             // Get page/track number of the first page/track related to this structure element.
             $details['pagination'] = $this->physicalStructureInfo[$this->smLinks['l2p'][$details['id']][0]]['orderlabel'];
@@ -468,7 +468,7 @@ 
             $details['points'] = 1;
             $details['thumbnailId'] = $this->getThumbnail();
         }
-        if ($details['thumbnailId'] === null) {
+        if ($details['thumbnailId'] === NULL) {
             unset($details['thumbnailId']);
         }
     }
@@ -488,14 +488,14 @@ 
         $extConf = GeneralUtility::makeInstance(ExtensionConfiguration::class)->get(self::$extKey, 'files');
         $fileGrpsThumb = GeneralUtility::trimExplode(',', $extConf['fileGrpThumbs']);
 
-        $thumbnail = null;
+        $thumbnail = NULL;
 
         while ($fileGrpThumb = array_shift($fileGrpsThumb)) {
             if (empty($id)) {
-                $thumbnail = $this->physicalStructureInfo[$this->physicalStructure[1]]['files'][$fileGrpThumb] ?? null;
+                $thumbnail = $this->physicalStructureInfo[$this->physicalStructure[1]]['files'][$fileGrpThumb] ?? NULL;
             } else {
-                $parentId = $this->smLinks['l2p'][$id][0] ?? null;
-                $thumbnail = $this->physicalStructureInfo[$parentId]['files'][$fileGrpThumb] ?? null;
+                $parentId = $this->smLinks['l2p'][$id][0] ?? NULL;
+                $thumbnail = $this->physicalStructureInfo[$parentId]['files'][$fileGrpThumb] ?? NULL;
             }
 
             if (!empty($thumbnail)) {
@@ -645,7 +645,7 @@ 
             }
         }
         if (empty($theseSubentries)) {
-            return false;
+            return FALSE;
         }
         return $theseSubentries;
     }
@@ -716,13 +716,13 @@ 
     private function extractAndProcessMetadata(string $dmdId, string $mdSectionType, array &$metadata, int $cPid, array $metadataSections): bool
     {
         if ($this->hasMetadataSection($metadataSections, $mdSectionType, 'dmdSec')) {
-            return true;
+            return TRUE;
         }
 
         $metadataExtracted = $this->extractMetadataIfTypeSupported($dmdId, $mdSectionType, $metadata);
 
         if (!$metadataExtracted) {
-            return false;
+            return FALSE;
         }
 
         $additionalMetadata = $this->getAdditionalMetadataFromDatabase($cPid, $dmdId);
@@ -733,7 +733,7 @@ 
 
         $this->processAdditionalMetadata($additionalMetadata, $domXPath, $domNode, $metadata);
 
-        return true;
+        return TRUE;
     }
 
     /**
@@ -917,7 +917,7 @@ 
                     $obj = GeneralUtility::makeInstance($class);
                     if ($obj instanceof MetadataInterface) {
                         $obj->extractMetadata($this->mdSec[$dmdId]['xml'], $metadata, GeneralUtility::makeInstance(ExtensionConfiguration::class)->get(self::$extKey, 'general')['useExternalApisForMetadata']);
-                        return true;
+                        return TRUE;
                     }
                 } else {
                     $this->logger->warning('Invalid class/method "' . $class . '->extractMetadata()" for metadata format "' . $this->mdSec[$dmdId]['type'] . '"');
@@ -926,7 +926,7 @@ 
         } else {
             $this->logger->notice('Unsupported metadata format "' . $this->mdSec[$dmdId]['type'] . '" in ' . $mdSectionType . ' with @ID "' . $dmdId . '"');
         }
-        return false;
+        return FALSE;
     }
 
     /**
@@ -1172,16 +1172,16 @@ 
     protected function loadLocation(string $location): bool
     {
         $fileResource = Helper::getUrl($location);
-        if ($fileResource !== false) {
+        if ($fileResource !== FALSE) {
             $xml = Helper::getXmlFileAsString($fileResource);
             // Set some basic properties.
-            if ($xml !== false) {
+            if ($xml !== FALSE) {
                 $this->xml = $xml;
-                return true;
+                return TRUE;
             }
         }
         $this->logger->error('Could not load XML file from "' . $location . '"');
-        return false;
+        return FALSE;
     }
 
     /**
@@ -1203,9 +1203,9 @@ 
 
         if ($preloadedDocument instanceof SimpleXMLElement) {
             $this->xml = $preloadedDocument;
-            return true;
+            return TRUE;
         }
-        return false;
+        return FALSE;
     }
 
     /**
@@ -1231,7 +1231,7 @@ 
             foreach ($this->mets->xpath('./mets:dmdSec') as $dmdSecTag) {
                 $dmdSec = $this->processMdSec($dmdSecTag);
 
-                if ($dmdSec !== null) {
+                if ($dmdSec !== NULL) {
                     $this->mdSec[$dmdSec['id']] = $dmdSec;
                     $this->dmdSec[$dmdSec['id']] = $dmdSec;
                 }
@@ -1248,7 +1248,7 @@ 
                     // TODO: Should we check that the format may occur within this type (e.g., to ignore VIDEOMD within rightsMD)?
                     $mdSec = $this->processMdSec($mdSecTag);
 
-                    if ($mdSec !== null) {
+                    if ($mdSec !== NULL) {
                         $this->mdSec[$mdSec['id']] = $mdSec;
 
                         $childIds[] = $mdSec['id'];
@@ -1261,7 +1261,7 @@ 
                 }
             }
 
-            $this->mdSecLoaded = true;
+            $this->mdSecLoaded = TRUE;
         }
         return $this->mdSec;
     }
@@ -1292,7 +1292,7 @@ 
     {
         $mdId = (string) $element->attributes()->ID;
         if (empty($mdId)) {
-            return null;
+            return NULL;
         }
 
         $this->registerNamespaces($element);
@@ -1310,7 +1310,7 @@ 
         }
 
         if (empty($xml)) {
-            return null;
+            return NULL;
         }
 
         $this->registerNamespaces($xml[0]);
@@ -1376,9 +1376,9 @@ 
                 !empty($extConf['fileGrpFulltext'])
                 && array_intersect(GeneralUtility::trimExplode(',', $extConf['fileGrpFulltext']), $this->fileGrps) !== []
             ) {
-                $this->hasFulltext = true;
+                $this->hasFulltext = TRUE;
             }
-            $this->fileGrpsLoaded = true;
+            $this->fileGrpsLoaded = TRUE;
         }
         return $this->fileGrps;
     }
@@ -1439,7 +1439,7 @@ 
 
                 $this->physicalStructure = $this->getPhysicalElements($elementNodes, $fileUse);
             }
-            $this->physicalStructureLoaded = true;
+            $this->physicalStructureLoaded = TRUE;
 
         }
 
@@ -1552,7 +1552,7 @@ 
                     $this->smLinks['p2l'][(string) $smLink->attributes('http://www.w3.org/1999/xlink')->to][] = (string) $smLink->attributes('http://www.w3.org/1999/xlink')->from;
                 }
             }
-            $this->smLinksLoaded = true;
+            $this->smLinksLoaded = TRUE;
         }
         return $this->smLinks;
     }
@@ -1560,7 +1560,7 @@ 
     /**
      * @see AbstractDocument::magicGetThumbnail()
      */
-    protected function magicGetThumbnail(bool $forceReload = false): string
+    protected function magicGetThumbnail(bool $forceReload = FALSE): string
     {
         if (
             !$this->thumbnailLoaded
@@ -1570,14 +1570,14 @@ 
             $cPid = $this->cPid ?: $this->pid;
             if (!$cPid) {
                 $this->logger->error('Invalid PID ' . $cPid . ' for structure definitions');
-                $this->thumbnailLoaded = true;
+                $this->thumbnailLoaded = TRUE;
                 return $this->thumbnail;
             }
             // Load extension configuration.
             $extConf = GeneralUtility::makeInstance(ExtensionConfiguration::class)->get(self::$extKey, 'files');
             if (empty($extConf['fileGrpThumbs'])) {
                 $this->logger->warning('No fileGrp for thumbnails specified');
-                $this->thumbnailLoaded = true;
+                $this->thumbnailLoaded = TRUE;
                 return $this->thumbnail;
             }
             $strctId = $this->magicGetToplevelId();
@@ -1631,7 +1631,7 @@ 
             } else {
                 $this->logger->error('No structure of type "' . $metadata['type'][0] . '" found in database');
             }
-            $this->thumbnailLoaded = true;
+            $this->thumbnailLoaded = TRUE;
         }
         return $this->thumbnail;
     }
@@ -1709,8 +1709,8 @@ 
     public function __toString(): string
     {
         $xml = new DOMDocument('1.0', 'utf-8');
-        $xml->appendChild($xml->importNode(dom_import_simplexml($this->mets), true));
-        $xml->formatOutput = true;
+        $xml->appendChild($xml->importNode(dom_import_simplexml($this->mets), TRUE));
+        $xml->formatOutput = TRUE;
         return $xml->saveXML();
     }
 
@@ -1725,7 +1725,7 @@ 
     public function __wakeup(): void
     {
         $xml = Helper::getXmlFileAsString($this->asXML);
-        if ($xml !== false) {
+        if ($xml !== FALSE) {
             $this->asXML = '';
             $this->xml = $xml;
             // Rebuild the unserializable properties.
@@ -1848,7 +1848,7 @@ 
                 }
 
             }
-            $this->musicalStructureLoaded = true;
+            $this->musicalStructureLoaded = TRUE;
         }
 
         return $this->musicalStructure;

Classes/Common/DocumentAnnotation.php

Indentation +3 added lines, -3 removed lines

@@ -130,11 +130,11 @@
                             );
                         }
                     } elseif ($annotationTarget->getObjectId()) {
-                         $objectTargetPages = [];
+                            $objectTargetPages = [];
                         foreach ($this->document->getCurrentDocument()->physicalStructureInfo as $physInfo) {
-                             $order = $physInfo['order'];
+                                $order = $physInfo['order'];
                             if ($order) {
-                                 $objectTargetPages[] = $order;
+                                    $objectTargetPages[] = $order;
                             }
                         }
                         if ($objectTargetPages) {

Spacing +11 added lines, -11 removed lines

@@ -108,8 +108,8 @@ 
                                     ];
                                 } else {
                                     $this->logger->warning(
-                                        ' No target pages found! Annotation: "' . $annotation->getId() . '", '
-                                        . 'Target: "' . $annotationTarget->getUrl() . '"'
+                                        ' No target pages found! Annotation: "'.$annotation->getId().'", '
+                                        . 'Target: "'.$annotationTarget->getUrl().'"'
                                     );
                                 }
                             }
@@ -125,8 +125,8 @@ 
                             ];
                         } else {
                             $this->logger->warning(
-                                ' No target pages found! Annotation: "' . $annotation->getId() . '", '
-                                . 'Target: "' . $annotationTarget->getUrl() . '"'
+                                ' No target pages found! Annotation: "'.$annotation->getId().'", '
+                                . 'Target: "'.$annotationTarget->getUrl().'"'
                             );
                         }
                     } elseif ($annotationTarget->getObjectId()) {
@@ -145,14 +145,14 @@ 
                         }
                     } else {
                         $this->logger->warning(
-                            ' No target pages found! Annotation: "' . $annotation->getId() . '", '
-                            . 'Target: "' . $annotationTarget->getUrl() . '"'
+                            ' No target pages found! Annotation: "'.$annotation->getId().'", '
+                            . 'Target: "'.$annotationTarget->getUrl().'"'
                         );
                     }
                 } else {
                     $this->logger->warning(
-                        'Invalid target! Annotation: "' . $annotation->getId() . '", '
-                        . 'Target: "' . $annotationTarget->getUrl() . '"'
+                        'Invalid target! Annotation: "'.$annotation->getId().'", '
+                        . 'Target: "'.$annotationTarget->getUrl().'"'
                     );
                 }
             }
@@ -267,10 +267,10 @@ 
                 $intervalTo = \DateTime::createFromFormat('U.u', $to);
             }
             foreach ($tracks as $index => $track) {
-                $begin = new DateTime("1970-01-01 " . $track['begin']);
-                $extent = new DateTime("1970-01-01 " . $track['extent']);
+                $begin = new DateTime("1970-01-01 ".$track['begin']);
+                $extent = new DateTime("1970-01-01 ".$track['extent']);
                 $diff = (new DateTime("1970-01-01 00:00:00"))->diff($extent);
-                $end = (new DateTime("1970-01-01 " . $track['begin']))->add($diff);
+                $end = (new DateTime("1970-01-01 ".$track['begin']))->add($diff);
                 if (
                     !(
                         $intervalFrom < $end && (

Braces +11 added lines, -22 removed lines

@@ -28,8 +28,7 @@ 
  *
  * @access public
  */
-class DocumentAnnotation
-{
+class DocumentAnnotation {
     /**
      * @var null|DocumentAnnotation
      */
@@ -55,8 +54,7 @@ 
      * @param array $annotationData
      * @param Document $document
      */
-    private function __construct($annotationData, $document)
-    {
+    private function __construct($annotationData, $document) {
         $this->annotationData = $annotationData;
         $this->document = $document;
         $this->logger = GeneralUtility::makeInstance(LogManager::class)->getLogger(static::class);
@@ -67,8 +65,7 @@ 
      *
      * @return Annotation[]|array
      */
-    public function getAnnotations()
-    {
+    public function getAnnotations() {
         if (empty($this->annotationData)) {
             return [];
         }
@@ -168,8 +165,7 @@ 
      * @param string $logicalId
      * @return array
      */
-    protected function getPagesByLogicalId($logicalId)
-    {
+    protected function getPagesByLogicalId($logicalId) {
         $pages = [];
         if (
             array_key_exists('l2p', $this->document->getCurrentDocument()->smLinks) &&
@@ -193,8 +189,7 @@ 
      * @param string $physicalId
      * @return array
      */
-    protected function getPagesByPhysicalId($physicalId)
-    {
+    protected function getPagesByPhysicalId($physicalId) {
         $pages = [];
         foreach ($this->document->getCurrentDocument()->physicalStructureInfo as $physicalInfo) {
             $order = $physicalInfo['order'];
@@ -217,8 +212,7 @@ 
      * @param string $fileId
      * @return array
      */
-    protected function getPagesByFileId($fileId)
-    {
+    protected function getPagesByFileId($fileId) {
         $pages = [];
         foreach ($this->document->getCurrentDocument()->physicalStructureInfo as $physicalInfo) {
             if (
@@ -243,8 +237,7 @@ 
      * @param string $range
      * @return array
      */
-    protected function getAudioPagesByFileId($fileId, $range = null)
-    {
+    protected function getAudioPagesByFileId($fileId, $range = null) {
         $tracks = [];
         foreach ($this->document->getCurrentDocument()->physicalStructureInfo as $physicalInfo) {
             if (array_key_exists('tracks', $physicalInfo) && is_array($physicalInfo['tracks'])) {
@@ -299,8 +292,7 @@ 
      * @param string $range
      * @return array
      */
-    protected function getMeasurePagesByFileId($fileId, $range = null)
-    {
+    protected function getMeasurePagesByFileId($fileId, $range = null) {
         // Get all measures referencing the fileid
         $measures = [];
         // Get the related page numbers
@@ -355,8 +347,7 @@ 
      *
      * @return array
      */
-    public function getVerovioRelevantAnnotations()
-    {
+    public function getVerovioRelevantAnnotations() {
         $annotations = [];
         /** @var Annotation $annotation */
         foreach ($this->getAnnotations() as $annotation) {
@@ -373,8 +364,7 @@ 
      * @param Document $document
      * @return array
      */
-    protected static function loadData($document)
-    {
+    protected static function loadData($document) {
         $annotationData = [];
         $conf = GeneralUtility::makeInstance(ExtensionConfiguration::class)->get('dlf');
         $apiBaseUrl = $conf['annotationServerUrl'];
@@ -393,8 +383,7 @@ 
      * @return DocumentAnnotation|null
      *
      */
-    public static function getInstance($document)
-    {
+    public static function getInstance($document) {
         if (self::$instance == null) {
             $annotationData = self::loadData($document);
             self::$instance = new DocumentAnnotation($annotationData, $document);

Upper-Lower-Casing +7 added lines, -7 removed lines

@@ -90,7 +90,7 @@ 
                                     $targetPages[] = [
                                         'target' => $annotationTarget,
                                         'pages' => $meiTargetPages,
-                                        'verovioRelevant' => true
+                                        'verovioRelevant' => TRUE
                                     ];
                                 } elseif (
                                     $audioTargetPages = $this->getAudioPagesByFileId(
@@ -243,7 +243,7 @@ 
      * @param string $range
      * @return array
      */
-    protected function getAudioPagesByFileId($fileId, $range = null)
+    protected function getAudioPagesByFileId($fileId, $range = NULL)
     {
         $tracks = [];
         foreach ($this->document->getCurrentDocument()->physicalStructureInfo as $physicalInfo) {
@@ -261,7 +261,7 @@ 
             $from = sprintf('%02.6f', (empty($from) ? "0" : $from));
             $intervalFrom = \DateTime::createFromFormat('U.u', $from);
             if (empty($to)) {
-                $intervalTo = null;
+                $intervalTo = NULL;
             } else {
                 $to = sprintf('%02.6f', $to);
                 $intervalTo = \DateTime::createFromFormat('U.u', $to);
@@ -274,7 +274,7 @@ 
                 if (
                     !(
                         $intervalFrom < $end && (
-                            $intervalTo === null || $intervalTo > $begin
+                            $intervalTo === NULL || $intervalTo > $begin
                         )
                     )
                 ) {
@@ -285,7 +285,7 @@ 
         // Get the related page numbers
         $trackPages = [];
         foreach ($tracks as $track) {
-            if ($track['order'] !== null) {
+            if ($track['order'] !== NULL) {
                 $trackPages[] = $track['order'];
             }
         }
@@ -299,7 +299,7 @@ 
      * @param string $range
      * @return array
      */
-    protected function getMeasurePagesByFileId($fileId, $range = null)
+    protected function getMeasurePagesByFileId($fileId, $range = NULL)
     {
         // Get all measures referencing the fileid
         $measures = [];
@@ -395,7 +395,7 @@ 
      */
     public static function getInstance($document)
     {
-        if (self::$instance == null) {
+        if (self::$instance == NULL) {
             $annotationData = self::loadData($document);
             self::$instance = new DocumentAnnotation($annotationData, $document);
         }

Configuration/TCA/tx_dlf_metadatasubentries.php

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -87,7 +87,7 @@ 
             'label' => 'LLL:EXT:dlf/Resources/Private/Language/locallang_labels.xlf:tx_dlf_metadata.wrap',
             'config' => [
                 'behaviour' => [
-                    'allowLanguageSynchronization' => true
+                    'allowLanguageSynchronization' => TRUE
                 ],
                 'type' => 'text',
                 'cols' => 48,
@@ -95,8 +95,8 @@ 
                 'wrap' => 'off',
                 'eval' => 'trim',
                 'default' => "key.wrap = <strong>|:</strong>\nvalue.required = 1\nvalue.wrap = <span>|</span>\nall.wrap = <li>|</li>",
-                'fixedFont' => true,
-                'enableTabulator' => true
+                'fixedFont' => TRUE,
+                'enableTabulator' => TRUE
             ],
         ],
     ],