kitodo / kitodo-publication

Classes/Domain/Model/MetadataObject.php

Spacing +2 added lines, -2 removed lines

@@ -460,7 +460,7 @@ 
         if ($this->accessRestrictionRoles) {
             return array_map('trim', explode(',', $this->accessRestrictionRoles));
         } else {
-            return array();
+            return array ();
         }
     }
 
@@ -827,7 +827,7 @@ 
     }
 
     protected function isRepeatable() {
-        return !in_array($this->getObjectType(), ['fileDownload','fileArchive','fileLabel'])
+        return !in_array($this->getObjectType(), ['fileDownload', 'fileArchive', 'fileLabel'])
             && $this->getInputField() != self::FILE_UPLOAD;
     }
 }

Classes/Domain/Model/DocumentForm.php

Spacing +1 added lines, -1 removed lines

@@ -219,7 +219,7 @@
 
     public function getFileNames()
     {
-        $fileNames = array();
+        $fileNames = array ();
         foreach ($this->getFiles() as $file) {
             $fileNames[] = $file->getTitle();
         }

Classes/Controller/ApiController.php

Spacing +8 added lines, -8 removed lines

@@ -258,8 +258,8 @@ 
                 $doc->setProcessNumber($processNumber);
             }
 
-            if ($this->documentManager->update($doc, null,true)) {
-                return '{"success": "Document '.$document.' added '.$id.'"}';
+            if ($this->documentManager->update($doc, null, true)) {
+                return '{"success": "Document ' . $document . ' added ' . $id . '"}';
             } else {
                 return '{"failed": Could not update the Document"}';
             }
@@ -298,7 +298,7 @@ 
                     return '{"error": "Invalid data in parameter json."}';
                 }
             } else {
-                if (empty($json) || json_decode($json,true) === []) {
+                if (empty($json) || json_decode($json, true) === []) {
                     return '{"error": "Parameter json can not be empty."}';
                 }
                 if (is_null(json_decode($json))) {
@@ -310,7 +310,7 @@ 
             $doc = $this->documentManager->read($document);
 
             if (!$doc) {
-                return '{"failed": "Document does not exist: '.$document.'"}';
+                return '{"failed": "Document does not exist: ' . $document . '"}';
             }
 
             if ($doc->getState() === DocumentWorkflow::STATE_NEW_NONE) {
@@ -323,7 +323,7 @@ 
             }
 
             if ($linkedDocument) {
-                return '{"failed": "There is already a suggestion for the document: '.$linkedDocument->getUid().'"}';
+                return '{"failed": "There is already a suggestion for the document: ' . $linkedDocument->getUid() . '"}';
             }
 
             $mapper = $this->objectManager->get(\EWW\Dpf\Services\Api\JsonToDocumentMapper::class);
@@ -681,7 +681,7 @@ 
         $missingArguments = [];
         foreach ($parameterNames as $parameterName) {
             if (!$this->request->hasArgument($parameterName)) {
-                $missingArguments[] =  $parameterName;
+                $missingArguments[] = $parameterName;
             }
         }
 
@@ -689,7 +689,7 @@ 
             $this->throwStatus(
                 400,
                 null,
-                '{"error": "Missing parameters: '.implode(", ", $missingArguments).'"}'
+                '{"error": "Missing parameters: ' . implode(", ", $missingArguments) . '"}'
             );
         }
 
@@ -719,7 +719,7 @@ 
             $this->throwStatus(
                 400,
                 null,
-            '{"error": "An invalid action hes been called: '.$actionName.'"}'
+            '{"error": "An invalid action hes been called: ' . $actionName . '"}'
             );
         }
 

Classes/Controller/AbstractDocumentFormController.php

Spacing +2 added lines, -2 removed lines

@@ -192,7 +192,7 @@ 
                 $this->request->setArguments($requestArguments);
             } else {
                 $t = $docForm->getFileNames();
-                $this->redirect('list', 'DocumentForm', null, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
+                $this->redirect('list', 'DocumentForm', null, array ('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
             }
         } else {
             $this->redirectToList("UPLOAD_POST_SIZE_ERROR");
@@ -355,7 +355,7 @@ 
                 $this->request->setArguments($requestArguments);
             } else {
                 $t = $docForm->getFileNames();
-                $this->redirect('list', 'Document', null, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
+                $this->redirect('list', 'Document', null, array ('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
             }
         } else {
             $this->redirectToList("UPLOAD_POST_SIZE_ERROR");

Classes/Services/Transfer/FileId.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function __construct(ObjectStorage $documentFiles)
     {
-        $idList   = array();
+        $idList   = array ();
         $this->id = 0;
 
         if (is_a($documentFiles, ObjectStorage::class)) {

Classes/Services/ParserGenerator.php

Spacing +7 added lines, -7 removed lines

@@ -45,13 +45,13 @@ 
      *
      * @var array
      */
-    protected $formData = array();
+    protected $formData = array ();
 
     /**
      * files from form
      * @var array
      */
-    protected $files = array();
+    protected $files = array ();
 
     /**
      * metsData
@@ -111,7 +111,7 @@ 
 
         $this->xmlHeader = '<data' . $this->namespaceString . '></data>';
 
-        $this->xmlData =  new \DOMDocument();
+        $this->xmlData = new \DOMDocument();
         $this->xmlData->loadXML($this->xmlHeader);
 
         // Parser
@@ -164,7 +164,7 @@ 
             // mods extension
             if ($group['modsExtensionMapping']) {
                 $counter = sprintf("%'03d", $this->counter);
-                $attributeXPath .= '[@ID="'.$fedoraNamespace.'_' . $counter . '"]';
+                $attributeXPath .= '[@ID="' . $fedoraNamespace . '_' . $counter . '"]';
             }
 
             $existsExtensionFlag = false;
@@ -205,7 +205,7 @@ 
                 }
             }
             if (!$existsExtensionFlag && $group['modsExtensionMapping']) {
-                $xPath = $group['modsExtensionMapping'] . $extensionAttribute . '[@' . $group['modsExtensionReference'] . '="'.$fedoraNamespace.'_' . $counter . '"]';
+                $xPath = $group['modsExtensionMapping'] . $extensionAttribute . '[@' . $group['modsExtensionReference'] . '="' . $fedoraNamespace . '_' . $counter . '"]';
                 $xml   = $this->customXPath($xPath, true, '', true);
             }
             if ($group['modsExtensionMapping']) {
@@ -290,7 +290,7 @@ 
                     $nestedXml = $this->parseXPath($nested);
 
                     // object xpath without nested element []
-                    $newPath[1] = str_replace('['.$match[2].']', '', $newPath[1]);
+                    $newPath[1] = str_replace('[' . $match[2] . ']', '', $newPath[1]);
 
                     $xml = $this->parseXPath($newPath[1]);
 
@@ -360,7 +360,7 @@ 
                     $nestedXml = $this->parseXPathWrapped($nested);
 
                     // object xpath without nested element []
-                    $newPath[1] = str_replace('['.$match[2].']', '', $newPath[1]);
+                    $newPath[1] = str_replace('[' . $match[2] . ']', '', $newPath[1]);
 
                     $xml2 = $this->parseXPathWrapped($path . $newPath[1]);
                 }

Classes/Helper/XSLTransformator.php

Spacing +4 added lines, -4 removed lines

@@ -88,7 +88,7 @@ 
             $logger->log(
                 LogLevel::WARNING,
                 "Input XML: No transformation file is present. The given xml data was taken over as it is",
-                array(
+                array (
                     'documentTypeName' => $documentTypeName
                 )
             );
@@ -119,7 +119,7 @@ 
             $filePath = $transformationFile->getFile()->getOriginalResource()->getIdentifier();
             $documentTransformer = new DocumentTransformer();
 
-            if ( !$document->getRemoteState() || $document->getRemoteState() == 'NONE' ) {
+            if (!$document->getRemoteState() || $document->getRemoteState() == 'NONE') {
                 $remoteState = 'ACTIVE';
             } else {
                 $remoteState = $document->getRemoteState();
@@ -138,7 +138,7 @@ 
                 );
             } else {
                 $transformedXml = $documentTransformer->transform(
-                    Environment::getPublicPath(). '/fileadmin' . $filePath, $document->publicXml(), $transformParams
+                    Environment::getPublicPath() . '/fileadmin' . $filePath, $document->publicXml(), $transformParams
                 );
             }
 
@@ -154,7 +154,7 @@ 
             $logger->log(
                 LogLevel::WARNING,
                 "Output XML: No transformation file is present. The generated xml data was taken over as it is",
-                array(
+                array (
                     'documentTypeName' => $documentTypeName
                 )
             );

Classes/Helper/DocumentMapper.php

Spacing +7 added lines, -7 removed lines

@@ -142,7 +142,7 @@ 
                 (
                     $this->getSettings()['deactivatePrimaryFileMandatoryCheck'] ||
                     $document->getDocumentType()->getVirtualType()
-                )? false : true
+                ) ? false : true
             )
         );
 
@@ -159,7 +159,7 @@ 
 
         $internalFormat = new \EWW\Dpf\Helper\InternalFormat($document->getXmlData(), $this->clientPid);
 
-        $excludeGroupAttributes = array();
+        $excludeGroupAttributes = array ();
 
         foreach ($document->getDocumentType()->getMetadataPage() as $metadataPage) {
             $documentFormPage = new \EWW\Dpf\Domain\Model\DocumentFormPage();
@@ -190,7 +190,7 @@ 
                 $xpath = $internalFormat->getXpath();
 
                 // get fixed attributes from xpath configuration
-                $fixedGroupAttributes = array();
+                $fixedGroupAttributes = array ();
 
                 preg_match_all('/[A-Za-z0-9:@\.]+(\[@.*?\])*/', $metadataGroup->getAbsoluteMapping(), $groupMappingPathParts);
                 $groupMappingPathParts = $groupMappingPathParts[0];
@@ -458,7 +458,7 @@ 
 
                 foreach ($group as $groupItem) {
 
-                    $item = array();
+                    $item = array ();
 
                     $uid           = $groupItem->getUid();
                     $metadataGroup = $this->metadataGroupRepository->findByUid($uid);
@@ -483,7 +483,7 @@ 
 
                             $fieldMapping = $metadataObject->getRelativeMapping();
 
-                            $formField = array();
+                            $formField = array ();
 
                             $value = $fieldItem->getValue();
 
@@ -533,11 +533,11 @@ 
                     }
 
                     if (!key_exists('attributes', $item)) {
-                        $item['attributes'] = array();
+                        $item['attributes'] = array ();
                     }
 
                     if (!key_exists('values', $item)) {
-                        $item['values'] = array();
+                        $item['values'] = array ();
                     }
 
                     if ($groupItem->getMandatory() || $defaultValueCount < $fieldValueCount || $defaultValueCount == $fieldCount) {

Classes/Domain/Model/Document.php

Spacing +6 added lines, -6 removed lines

@@ -377,7 +377,7 @@ 
     public function setObjectIdentifier($objectIdentifier)
     {
         // Due to uniqe key uc_object_identifier, which should ignore empty object identifiers.
-        $this->objectIdentifier = empty($objectIdentifier)? null : $objectIdentifier;
+        $this->objectIdentifier = empty($objectIdentifier) ? null : $objectIdentifier;
     }
 
     /**
@@ -712,7 +712,7 @@ 
      */
     public function getCreator()
     {
-        return $this->creator? $this->creator : 0;
+        return $this->creator ? $this->creator : 0;
     }
 
     /**
@@ -867,7 +867,7 @@ 
 
         foreach ($availableProperties as $propertyName) {
             if (\TYPO3\CMS\Extbase\Reflection\ObjectAccess::isPropertySettable($newDocument, $propertyName)
-                && !in_array($propertyName, array('uid','pid', 'file', 'comment', 'linkedUid', 'suggestion', 'creator'))) {
+                && !in_array($propertyName, array ('uid', 'pid', 'file', 'comment', 'linkedUid', 'suggestion', 'creator'))) {
 
                 $propertyValue = \TYPO3\CMS\Extbase\Reflection\ObjectAccess::getProperty($documentToCopy, $propertyName);
                 \TYPO3\CMS\Extbase\Reflection\ObjectAccess::setProperty($newDocument, $propertyName, $propertyValue);
@@ -889,7 +889,7 @@ 
      */
     public function getDocumentIdentifier()
     {
-        return $this->getObjectIdentifier()? $this->getObjectIdentifier() : $this->getUid();
+        return $this->getObjectIdentifier() ? $this->getObjectIdentifier() : $this->getUid();
     }
 
     /**
@@ -922,8 +922,8 @@ 
     public function getPublicationYear()
     {
         $internalFormat = new InternalFormat($this->getXmlData(), $this->getPid());
-        $year =  $internalFormat->getPublishingYear();
-        return $year? $year : "";
+        $year = $internalFormat->getPublishingYear();
+        return $year ? $year : "";
     }
 
     /**