kitodo / kitodo-publication

Classes/Domain/Model/ProcessNumber.php

Spacing +1 added lines, -1 removed lines

@@ -106,6 +106,6 @@
     }
 
     public function getProcessNumberString() {
-        return strtoupper($this->getOwnerId()).'-'. $this->getYear().'-'.$this->getCounter();
+        return strtoupper($this->getOwnerId()) . '-' . $this->getYear() . '-' . $this->getCounter();
     }
 }
\ No newline at end of file

Braces +6 added lines, -12 removed lines

@@ -47,8 +47,7 @@ 
      *
      * @return string $ownerId
      */
-    public function getOwnerId()
-    {
+    public function getOwnerId() {
         return $this->ownerId;
     }
 
@@ -58,8 +57,7 @@ 
      * @param string $ownerId
      * @return void
      */
-    public function setOwnerId($ownerId)
-    {
+    public function setOwnerId($ownerId) {
         $this->ownerId = $ownerId;
     }
 
@@ -68,8 +66,7 @@ 
      *
      * @return int $year
      */
-    public function getYear()
-    {
+    public function getYear() {
         return $this->year;
     }
 
@@ -79,8 +76,7 @@ 
      * @param int $year
      * @return void
      */
-    public function setYear($year)
-    {
+    public function setYear($year) {
         $this->year = $year;
     }
 
@@ -89,8 +85,7 @@ 
      *
      * @return int $counter
      */
-    public function getCounter()
-    {
+    public function getCounter() {
         return $this->counter;
     }
 
@@ -100,8 +95,7 @@ 
      * @param int $counter
      * @return void
      */
-    public function setCounter($counter)
-    {
+    public function setCounter($counter) {
         $this->counter = $counter;
     }
 

Classes/Plugins/DownloadTool/DownloadTool.php

Indentation -1 removed lines

@@ -107,7 +107,6 @@
 
     /**
      * Get PDF document list
-
      * @return array of attachments
      */
     protected function getAttachments()

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

@@ -46,7 +46,7 @@ 
 
         if (is_array($dpfTSconfig['settings.'])) {
 
-            \TYPO3\CMS\Core\Utility\ArrayUtility::mergeRecursiveWithOverrule($dpfTSconfig['settings.'], $this->conf, true, false);
+            \TYPO3\CMS\Core\Utility\ArrayUtility::mergeRecursiveWithOverrule($dpfTSconfig['settings.'], $this->conf, TRUE, FALSE);
             $this->conf = $dpfTSconfig['settings.'];
 
         }
@@ -54,7 +54,7 @@ 
         // Load current document.
         $this->loadDocument();
 
-        if ($this->doc === null || empty($this->conf['fileGrpDownload'])) {
+        if ($this->doc === NULL || empty($this->conf['fileGrpDownload'])) {
 
             // Quit without doing anything if required variables are not set.
             return $content;
@@ -87,7 +87,7 @@ 
                     'useCacheHash'     => 0,
                     'parameter'        => $this->conf['apiPid'] . ' - piwik_download',
                     'additionalParams' => '&tx_dpf[qid]=' . $this->doc->recordId . '&tx_dpf[action]=attachment' . '&tx_dpf[attachment]=' . $file['ID'],
-                    'forceAbsoluteUrl' => true,
+                    'forceAbsoluteUrl' => TRUE,
                 );
 
                 $title = $file['LABEL'] ? $file['LABEL'] : $file['ID'];
@@ -101,7 +101,7 @@ 
 
         }
 
-        return $this->cObj->substituteSubpart($this->template, '###DOWNLOADS###', $content, true);
+        return $this->cObj->substituteSubpart($this->template, '###DOWNLOADS###', $content, TRUE);
 
     }
 

Spacing +6 added lines, -6 removed lines

@@ -87,7 +87,7 @@ 
 
             foreach ($attachments as $id => $file) {
 
-                $conf = array(
+                $conf = array (
                     'useCacheHash'     => 0,
                     'parameter'        => $this->conf['apiPid'] . ' - piwik_download',
                     'additionalParams' => '&tx_dpf[qid]=' . $this->doc->recordId . '&tx_dpf[action]=attachment' . '&tx_dpf[attachment]=' . $file['ID'],
@@ -103,11 +103,11 @@ 
                     $markerArray['###FILE###'] = $this->cObj->typoLink($title, $conf);
 
                     // Create a-tag with VG-Wort Redirect
-                } elseif(!empty($vgwort)) {
+                } elseif (!empty($vgwort)) {
 
                     $qucosaUrl = urlencode($this->cObj->typoLink_URL($conf));
 
-                    $confVgwort = array(
+                    $confVgwort = array (
                         'useCacheHash'     => 0,
                         'parameter'        => $vgwort . $qucosaUrl . ' - piwik_download',
                     );
@@ -141,13 +141,13 @@ 
 
         if (!is_array($files)) {
 
-            return array();
+            return array ();
 
         }
 
         foreach ($files as $key => $file) {
 
-            $singleFile = array();
+            $singleFile = array ();
 
             foreach ($file->attributes('mext', 1) as $attribute => $value) {
 
@@ -182,7 +182,7 @@ 
         $xPath = '//slub:info/slub:vgwortOpenKey';
         $vgwortOpenKey = $this->doc->mets->xpath($xPath)[0];
 
-        if (!empty($vgwortOpenKey) or $vgwortOpenKey != FALSE ) {
+        if (!empty($vgwortOpenKey) or $vgwortOpenKey != FALSE) {
 
             if (GeneralUtility::getIndpEnv('TYPO3_SSL')) {
 

Braces +4 added lines, -8 removed lines

@@ -24,8 +24,7 @@ 
  * @subpackage    tx_dpf
  * @access    public
  */
-class DownloadTool extends \tx_dlf_plugin
-{
+class DownloadTool extends \tx_dlf_plugin {
 
     /**
      * The main method of the PlugIn
@@ -37,8 +36,7 @@ 
      *
      * @return    string        The content that is displayed on the website
      */
-    public function main($content, $conf)
-    {
+    public function main($content, $conf) {
 
         $this->init($conf);
 
@@ -131,8 +129,7 @@ 
 
      * @return array of attachments
      */
-    protected function getAttachments()
-    {
+    protected function getAttachments() {
 
         // Get pdf documents
         $xPath = 'mets:fileSec/mets:fileGrp[@USE="' . $this->conf['fileGrpDownload'] . '"]/mets:file';
@@ -174,8 +171,7 @@ 
         return $attachments;
     }
 
-    protected function getVGWortUrl()
-    {
+    protected function getVGWortUrl() {
 
         // Get VG-Wort-OpenKey for document
         $this->doc->mets->registerXPathNamespace("slub", 'http://slub-dresden.de/');

Classes/Services/ProcessNumber/ProcessNumberGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             $datetime = new \DateTime();
             $currentYear = $datetime->format('y');
 
-            $processNumber = $processNumberRepository->getHighestProcessNumberByOwnerIdAndYear(strtolower($ownerId),$currentYear);
+            $processNumber = $processNumberRepository->getHighestProcessNumberByOwnerIdAndYear(strtolower($ownerId), $currentYear);
 
             if ($processNumber) {
                 $counter = $processNumber->getCounter() + 1;

Braces +1 added lines, -2 removed lines

@@ -20,8 +20,7 @@
 use EWW\Dpf\Domain\Model\ProcessNumber;
 use EWW\Dpf\Domain\Repository\ClientRepository;
 
-class ProcessNumberGenerator
-{
+class ProcessNumberGenerator {
     public function getProcessNumber($ownerId = NULL) {
         $objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(ObjectManager::class);
         $processNumberRepository = $objectManager->get(ProcessNumberRepository::class);

Classes/Services/Transfer/FedoraRepository.php

Indentation +5 added lines, -5 removed lines

@@ -35,11 +35,11 @@
     protected $documentTransferLogRepository;
 
     /**
-    * clientConfigurationManager
-    *
-    * @var \EWW\Dpf\Configuration\ClientConfigurationManager
-    * @inject
-    */
+     * clientConfigurationManager
+     *
+     * @var \EWW\Dpf\Configuration\ClientConfigurationManager
+     * @inject
+     */
     protected $clientConfigurationManager;
 
     /**

Braces +9 added lines, -18 removed lines

@@ -23,8 +23,7 @@ 
 use \EWW\Dpf\Services\Logger\TransferLogger;
 use \Httpful\Request;
 
-class FedoraRepository implements Repository
-{
+class FedoraRepository implements Repository {
 
     /**
      * documentTransferLogRepository
@@ -61,8 +60,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return string
      */
-    public function ingest($document, $metsXml)
-    {
+    public function ingest($document, $metsXml) {
 
         try {
             $response = Request::post($this->clientConfigurationManager->getSwordHost() . "/sword/" . $this->getSWORDCollection())
@@ -96,8 +94,7 @@ 
      * @param string $metsXml
      * @return string
      */
-    public function update($document, $metsXml)
-    {
+    public function update($document, $metsXml) {
 
         $remoteId = $document->getObjectIdentifier();
 
@@ -129,8 +126,7 @@ 
      * @param string $remoteId
      * @return string
      */
-    public function retrieve($remoteId)
-    {
+    public function retrieve($remoteId) {
 
         try {
             $response = Request::get($this->clientConfigurationManager->getFedoraHost() . "/fedora/objects/" . $remoteId . "/methods/qucosa:SDef/getMETSDissemination")
@@ -159,8 +155,7 @@ 
      * @param string $remoteId
      * @return string
      */
-    public function getNextDocumentId()
-    {
+    public function getNextDocumentId() {
 
         try {
             $response = Request::get($this->clientConfigurationManager->getFedoraHost() . "/fedora/management/getNextPID?numPIDs=1&namespace=qucosa&xml=true")
@@ -191,8 +186,7 @@ 
      * @param $state
      * @return boolean
      */
-    public function delete($document, $state)
-    {
+    public function delete($document, $state) {
 
         $remoteId = $document->getObjectIdentifier();
 
@@ -225,8 +219,7 @@ 
      * @param  \Httpful\Response $response
      * @return string
      */
-    protected function getRemoteDocumentId($response)
-    {
+    protected function getRemoteDocumentId($response) {
 
         // Get repository ID and write into document
         $responseDom = new \DOMDocument();
@@ -243,8 +236,7 @@ 
         return null;
     }
 
-    protected function getOwnerId()
-    {
+    protected function getOwnerId() {
         $ownerId = $this->clientConfigurationManager->getOwnerId();
         if (empty($ownerId)) {
             throw new \Exception('Owner id can not be empty or null!');
@@ -253,8 +245,7 @@ 
         return $ownerId;
     }
 
-    protected function getSWORDCollection()
-    {
+    protected function getSWORDCollection() {
         return $this->clientConfigurationManager->getSwordCollectionNamespace();
     }
 

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

@@ -78,15 +78,15 @@ 
             if (!$response->hasErrors() && $response->code == 201) {
                 return $this->getRemoteDocumentId($response);
             } else {
-                TransferLogger::Log('INGEST', $document->getUid(), null, $response);
+                TransferLogger::Log('INGEST', $document->getUid(), NULL, $response);
             }
         } catch (Exception $exception) {
             // curl error handling,
             // but extbase already catches all exceptions
-            return null;
+            return NULL;
         }
 
-        return null;
+        return NULL;
     }
 
     /**
@@ -119,7 +119,7 @@ 
         } catch (Exception $exception) {
             // curl error handling,
             // but extbase already catches all exceptions
-            return null;
+            return NULL;
         }
     }
 
@@ -142,15 +142,15 @@ 
             if (!$response->hasErrors() && $response->code == 200) {
                 return $response->__toString();
             } else {
-                TransferLogger::Log('RETRIEVE', null, $remoteId, $response);
+                TransferLogger::Log('RETRIEVE', NULL, $remoteId, $response);
             }
         } catch (Exception $exception) {
             // curl error handling,
             // but extbase already catches all exceptions
-            return null;
+            return NULL;
         }
 
-        return null;
+        return NULL;
     }
 
     /**
@@ -173,15 +173,15 @@ 
             if (!$response->hasErrors() && $response->code == 200) {
                 return $response->__toString();
             } else {
-                TransferLogger::Log('GET_NEXT_DOCUMENT_ID', null, $remoteId, $response);
+                TransferLogger::Log('GET_NEXT_DOCUMENT_ID', NULL, $remoteId, $response);
             }
         } catch (\Exception $exception) {
             // curl error handling,
             // but extbase already catches all exceptions
-            return null;
+            return NULL;
         }
 
-        return null;
+        return NULL;
     }
 
     /**
@@ -206,17 +206,17 @@ 
 
             // if transfer successful
             if (!$response->hasErrors() && $response->code == 204) {
-                return true;
+                return TRUE;
             } else {
                 TransferLogger::Log('DELETE', $document->getUid(), $remoteId, $response);
             }
         } catch (Exception $exception) {
             // curl error handling,
             // but extbase already catches all exceptions
-            return false;
+            return FALSE;
         }
 
-        return false;
+        return FALSE;
     }
 
     /**
@@ -240,7 +240,7 @@ 
             return $objectIdentifier;
         }
 
-        return null;
+        return NULL;
     }
 
     protected function getOwnerId()

Classes/Services/MetsExporter.php

Indentation +1 added lines, -1 removed lines

@@ -439,7 +439,7 @@
 
                 $domXPath2 = \EWW\Dpf\Helper\XPath::create($doc2);
 
-                  // second part nested xpath
+                    // second part nested xpath
                 if ($match[2] && $secondMatch[2]) {
                     // import node from nested
                     $docXMLNested = new \DOMDocument();

Spacing +4 added lines, -4 removed lines

@@ -24,13 +24,13 @@ 
      *
      * @var array
      */
-    protected $formData = array();
+    protected $formData = array ();
 
     /**
      * files from form
      * @var array
      */
-    protected $files = array();
+    protected $files = array ();
 
     /**
      * metsData
@@ -368,7 +368,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]);
 
@@ -429,7 +429,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]);
 
                     $xml2 = $this->parseXPath($path . $newPath[1]);
                 }

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

@@ -72,13 +72,13 @@ 
     /**
      * simpleXMLElement
      */
-    protected $sxe = null;
+    protected $sxe = NULL;
 
     /**
      * xPathXMLGenerator
      * @var object
      */
-    protected $parser = null;
+    protected $parser = NULL;
 
     /**
      * ref id counter
@@ -173,26 +173,26 @@ 
         $xmlData = $modsWrap->firstChild->firstChild->firstChild->firstChild;
 
         // import mods into mets
-        $nodeAppendModsData = $modsWrap->importNode($this->modsData->firstChild, true);
+        $nodeAppendModsData = $modsWrap->importNode($this->modsData->firstChild, TRUE);
         $xmlData->appendChild($nodeAppendModsData);
 
         // add SLUB data
-        $nodeAppendModsData = $modsWrap->importNode($this->buildMetsSlub()->firstChild, true);
+        $nodeAppendModsData = $modsWrap->importNode($this->buildMetsSlub()->firstChild, TRUE);
         $modsWrap->firstChild->appendChild($nodeAppendModsData);
 
         if ($fileSection) {
             // add filesection
-            $nodeAppendModsData = $modsWrap->importNode($fileSection->firstChild->firstChild, true);
+            $nodeAppendModsData = $modsWrap->importNode($fileSection->firstChild->firstChild, TRUE);
             $modsWrap->firstChild->appendChild($nodeAppendModsData);
         }
 
         if ($structureMap) {
             // add structure map
-            $nodeAppendModsData = $modsWrap->importNode($structureMap->firstChild->firstChild, true);
+            $nodeAppendModsData = $modsWrap->importNode($structureMap->firstChild->firstChild, TRUE);
             $modsWrap->firstChild->appendChild($nodeAppendModsData);
         }
 
-        $modsWrap->formatOutput = true;
+        $modsWrap->formatOutput = TRUE;
         $modsWrap->encoding     = 'UTF-8';
 
         $this->metsData = $modsWrap;
@@ -254,28 +254,28 @@ 
                 $attributeXPath .= '[@ID="QUCOSA_' . $counter . '"]';
             }
 
-            $existsExtensionFlag = false;
+            $existsExtensionFlag = FALSE;
             $i                   = 0;
             // loop each object
             if (!empty($values)) {
                 foreach ($values as $value) {
 
                     if ($value['modsExtension']) {
-                        $existsExtensionFlag = true;
+                        $existsExtensionFlag = TRUE;
                         // mods extension
                         $counter            = sprintf("%'03d", $this->counter);
                         $referenceAttribute = $extensionAttribute . '[@' . $group['modsExtensionReference'] . '="QUCOSA_' . $counter . '"]';
 
                         $path = $group['modsExtensionMapping'] . $referenceAttribute . '%/' . $value['mapping'];
 
-                        $xml = $this->customXPath($path, false, $value['value']);
+                        $xml = $this->customXPath($path, FALSE, $value['value']);
                     } else {
                         $path = $mapping . $attributeXPath . '%/' . $value['mapping'];
 
                         if ($i == 0) {
-                            $newGroupFlag = true;
+                            $newGroupFlag = TRUE;
                         } else {
-                            $newGroupFlag = false;
+                            $newGroupFlag = FALSE;
                         }
 
                         $xml = $this->customXPath($path, $newGroupFlag, $value['value']);
@@ -287,12 +287,12 @@ 
             } else {
                 if (!empty($attributeXPath)) {
                     $path = $mapping . $attributeXPath;
-                    $xml  = $this->customXPath($path, true, '', true);
+                    $xml  = $this->customXPath($path, TRUE, '', TRUE);
                 }
             }
             if (!$existsExtensionFlag && $group['modsExtensionMapping']) {
                 $xPath = $group['modsExtensionMapping'] . $extensionAttribute . '[@' . $group['modsExtensionReference'] . '="QUCOSA_' . $counter . '"]';
-                $xml   = $this->customXPath($xPath, true, '', true);
+                $xml   = $this->customXPath($xPath, TRUE, '', TRUE);
             }
             if ($group['modsExtensionMapping']) {
                 $this->counter++;
@@ -322,13 +322,13 @@ 
      * @param  string $value form value
      * @return xml    created xml
      */
-    public function customXPath($xPath, $newGroupFlag = false, $value = '', $attributeOnly = false)
+    public function customXPath($xPath, $newGroupFlag = FALSE, $value = '', $attributeOnly = FALSE)
     {
         if (!$attributeOnly) {
             // Explode xPath
             $newPath = explode('%', $xPath);
 
-            $praedicateFlag = false;
+            $praedicateFlag = FALSE;
             $explodedXPath  = explode('[', $newPath[0]);
             if (count($explodedXPath) > 1) {
                 // praedicate is given
@@ -340,7 +340,7 @@ 
                     $path = $explodedXPath[0];
                 }
 
-                $praedicateFlag = true;
+                $praedicateFlag = TRUE;
             } else {
                 $path = $newPath[0];
             }
@@ -390,7 +390,7 @@ 
                     $nodeList = $xPath->query('/mods:mods' . $match[1]);
                     $node = $nodeList->item(0);
 
-                    $importNode = $docXML->importNode($docXMLNested->getElementsByTagName("mods")->item(0)->firstChild, true);
+                    $importNode = $docXML->importNode($docXMLNested->getElementsByTagName("mods")->item(0)->firstChild, TRUE);
 
                     $node->appendChild($importNode);
                 }
@@ -401,7 +401,7 @@ 
 
                 $node = $domNodeList->item(0)->firstChild;
 
-                $nodeAppendModsData = $this->xmlData->importNode($node, true);
+                $nodeAppendModsData = $this->xmlData->importNode($node, TRUE);
                 $domNode->item($domNode->length - 1)->appendChild($nodeAppendModsData);
             } else {
                 // first xpath doesn't exist
@@ -449,7 +449,7 @@ 
                     $nodeList = $xPath->query('/mods:mods/' . $path . $match[1]);
                     $node = $nodeList->item(0);
 
-                    $importNode = $doc2->importNode($docXMLNested->getElementsByTagName("mods")->item(0)->firstChild, true);
+                    $importNode = $doc2->importNode($docXMLNested->getElementsByTagName("mods")->item(0)->firstChild, TRUE);
 
                     $node->appendChild($importNode);
                 }
@@ -457,7 +457,7 @@ 
                 $domNode2 = $domXPath2->query('/mods:mods/' . $path)->item(0)->childNodes->item(0);
 
                 // merge xml nodes
-                $nodeToBeAppended = $doc1->importNode($domNode2, true);
+                $nodeToBeAppended = $doc1->importNode($domNode2, TRUE);
 
                 $domNode->item(0)->appendChild($nodeToBeAppended);
 
@@ -466,7 +466,7 @@ 
                 $firstChild = $this->xmlData->firstChild;
                 $firstItem  = $doc1->getElementsByTagName('mods')->item(0)->firstChild;
 
-                $nodeAppendModsData = $this->xmlData->importNode($firstItem, true);
+                $nodeAppendModsData = $this->xmlData->importNode($firstItem, TRUE);
                 $firstChild->appendChild($nodeAppendModsData);
 
                 return $doc1->saveXML();
@@ -485,7 +485,7 @@ 
 
             $node = $domNodeList->item(0)->firstChild;
 
-            $nodeAppendModsData = $this->xmlData->importNode($node, true);
+            $nodeAppendModsData = $this->xmlData->importNode($node, TRUE);
             $domNode->item($domNode->length - 1)->appendChild($nodeAppendModsData);
 
             return $docXML->saveXML();
@@ -494,13 +494,13 @@ 
         return $this->xmlData->saveXML();
     }
 
-    public function customXPathSlub($xPath, $newGroupFlag = false, $value = '', $attributeOnly = false)
+    public function customXPathSlub($xPath, $newGroupFlag = FALSE, $value = '', $attributeOnly = FALSE)
     {
         if (!$attributeOnly) {
             // Explode xPath
             $newPath = explode('%', $xPath);
 
-            $praedicateFlag = false;
+            $praedicateFlag = FALSE;
             $explodedXPath  = explode('[', $newPath[0]);
             if (count($explodedXPath) > 1) {
                 // praedicate is given
@@ -512,7 +512,7 @@ 
                     $path = $explodedXPath[0];
                 }
 
-                $praedicateFlag = true;
+                $praedicateFlag = TRUE;
             } else {
                 $path = $newPath[0];
             }
@@ -539,7 +539,7 @@ 
 
                 $node = $domNodeList->item(0)->firstChild;
 
-                $nodeAppendModsData = $this->xmlData->importNode($node, true);
+                $nodeAppendModsData = $this->xmlData->importNode($node, TRUE);
                 $domNode->item($domNode->length - 1)->appendChild($nodeAppendModsData);
             } else {
                 // first xpath doesn't exist
@@ -568,7 +568,7 @@ 
                 $domNode2 = $domXPath2->query('/slub:info/' . $path)->item(0)->childNodes->item(0);
 
                 // merge xml nodes
-                $nodeToBeAppended = $doc1->importNode($domNode2, true);
+                $nodeToBeAppended = $doc1->importNode($domNode2, TRUE);
 
                 $domNode->item(0)->appendChild($nodeToBeAppended);
 
@@ -578,7 +578,7 @@ 
                 $firstChild = $this->xmlData->firstChild;
                 $firstItem  = $doc1->getElementsByTagName('info')->item(0)->firstChild;
 
-                $nodeAppendModsData = $this->xmlData->importNode($firstItem, true);
+                $nodeAppendModsData = $this->xmlData->importNode($firstItem, TRUE);
                 $firstChild->appendChild($nodeAppendModsData);
 
                 return $doc1->saveXML();
@@ -597,7 +597,7 @@ 
 
             $node = $domNodeList->item(0)->firstChild;
 
-            $nodeAppendModsData = $this->xmlData->importNode($node, true);
+            $nodeAppendModsData = $this->xmlData->importNode($node, TRUE);
             $domNode->item($domNode->length - 1)->appendChild($nodeAppendModsData);
 
             return $docXML->saveXML();
@@ -852,7 +852,7 @@ 
         $second = $this->slubData;
 
         foreach ($second->childNodes as $node) {
-            $importNode = $domDocument->importNode($node, true);
+            $importNode = $domDocument->importNode($node, TRUE);
             $domWrapElement->appendChild($importNode);
         }
 
@@ -909,15 +909,15 @@ 
 
                             $path = $group['modsExtensionMapping'] . $referenceAttribute . '%/' . $value['mapping'];
 
-                            $xml = $this->customXPathSlub($path, false, $value['value']);
+                            $xml = $this->customXPathSlub($path, FALSE, $value['value']);
                         } else {
                             $path = $mapping . $attributeXPath . '%/' . $value['mapping'];
                             // print_r($path);print_r("\n");
 
                             if ($i == 0) {
-                                $newGroupFlag = true;
+                                $newGroupFlag = TRUE;
                             } else {
-                                $newGroupFlag = false;
+                                $newGroupFlag = FALSE;
                             }
 
                             $xml = $this->customXPathSlub($path, $newGroupFlag, $value['value']);
@@ -930,7 +930,7 @@ 
                 } else {
                     if (!empty($attributeXPath)) {
                         $path = $mapping . $attributeXPath;
-                        $xml  = $this->customXPathSlub($path, true, '', true);
+                        $xml  = $this->customXPathSlub($path, TRUE, '', TRUE);
                     }
                 }
 

Braces +24 added lines, -48 removed lines

@@ -17,8 +17,7 @@ 
 /**
  * MetsExporter
  */
-class MetsExporter
-{
+class MetsExporter {
     /**
      * formData
      *
@@ -96,8 +95,7 @@ 
     /**
      * Constructor
      */
-    public function __construct()
-    {
+    public function __construct() {
         // mets data beginning
         $this->metsHeader = '<mets:mets xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                             xmlns:mets="http://www.loc.gov/METS/" xmlns:xlink="http://www.w3.org/1999/xlink"
@@ -138,8 +136,7 @@ 
      * returns the mets xml string
      * @return string mets xml
      */
-    public function getMetsData()
-    {
+    public function getMetsData() {
         $xml = $this->metsData->saveXML();
 
         $xml = preg_replace("/eww=\"\d-\d-\d\"/", '${1}${2}${3}', $xml);
@@ -152,8 +149,7 @@ 
      * returns the mods xml string
      * @return string mods xml
      */
-    public function getModsData()
-    {
+    public function getModsData() {
         return $this->modsData->saveXML();
     }
 
@@ -161,8 +157,7 @@ 
      * Build mets data structure
      * @return string mets xml
      */
-    public function buildMets()
-    {
+    public function buildMets() {
         // get mods domDocument
         $modsWrap = $this->buildModsWrap();
         // get mets filesection
@@ -203,8 +198,7 @@ 
      * @param  xml $xml xml data which should be wrapped with mods
      * @return xml wrapped xml
      */
-    public function wrapMods($xml)
-    {
+    public function wrapMods($xml) {
         $newXML = $this->modsHeader;
 
         $newXML = str_replace("</mods:mods>", $xml . "</mods:mods>", $newXML);
@@ -212,8 +206,7 @@ 
         return $newXML;
     }
 
-    public function wrapSlub($xml)
-    {
+    public function wrapSlub($xml) {
         $newXML = $this->slubHeader;
 
         $newXML = str_replace("</slub:info>", $xml . "</slub:info>", $newXML);
@@ -225,8 +218,7 @@ 
      * build mods from form array
      * @param array $array structured form data array
      */
-    public function buildModsFromForm($array)
-    {
+    public function buildModsFromForm($array) {
         $this->xmlData = $this->modsData;
         // Build xml mods from form fields
         // loop each group
@@ -308,8 +300,7 @@ 
      * @param  xpath $xPath xPath expression
      * @return xml
      */
-    public function parseXPath($xPath)
-    {
+    public function parseXPath($xPath) {
 
         $this->parser->generateXmlFromXPath($xPath);
         $xml = $this->parser->getXML();
@@ -323,8 +314,7 @@ 
      * @param  string $value form value
      * @return xml    created xml
      */
-    public function customXPath($xPath, $newGroupFlag = false, $value = '', $attributeOnly = false)
-    {
+    public function customXPath($xPath, $newGroupFlag = false, $value = '', $attributeOnly = false) {
         if (!$attributeOnly) {
             // Explode xPath
             $newPath = explode('%', $xPath);
@@ -495,8 +485,7 @@ 
         return $this->xmlData->saveXML();
     }
 
-    public function customXPathSlub($xPath, $newGroupFlag = false, $value = '', $attributeOnly = false)
-    {
+    public function customXPathSlub($xPath, $newGroupFlag = false, $value = '', $attributeOnly = false) {
         if (!$attributeOnly) {
             // Explode xPath
             $newPath = explode('%', $xPath);
@@ -610,8 +599,7 @@ 
      * Builds the xml wrapping part for mods
      * @return xml
      */
-    public function buildModsWrap()
-    {
+    public function buildModsWrap() {
         // Build wrap for mod
 
         $domDocument = new \DOMDocument();
@@ -649,8 +637,7 @@ 
      * Builds xml amdSection
      * @return xml
      */
-    public function buildAmdSection()
-    {
+    public function buildAmdSection() {
         // Build xml amd:sec
 
         $domDocument = new \DOMDocument();
@@ -661,8 +648,7 @@ 
         $amdSec = $domDocument->createElement('mets:amdSec');
     }
 
-    public function setMods($value = '')
-    {
+    public function setMods($value = '') {
         $domDocument = new \DOMDocument();
         if (is_null(@$domDocument->loadXML($value))) {
             throw new \Exception("Couldn't load MODS data");
@@ -670,13 +656,11 @@ 
         $this->modsData = $domDocument;
     }
 
-    public function setFileData($value = '')
-    {
+    public function setFileData($value = '') {
         $this->files = $value;
     }
 
-    public function loopFiles($array, $domElement, $domDocument)
-    {
+    public function loopFiles($array, $domElement, $domDocument) {
         $i = 0;
         // set xml for uploded files
         foreach ($array as $key => $value) {
@@ -721,8 +705,7 @@ 
      * Builds the xml fileSection part if files are uploaded
      * @return xml
      */
-    public function buildFileSection()
-    {
+    public function buildFileSection() {
         if (empty($this->files['original']) && empty($this->files['download'])) {
             return;
         }
@@ -769,8 +752,7 @@ 
      * Builds the xml structMap part if files are uploaded
      * @return xml
      */
-    public function buildStructureMap()
-    {
+    public function buildStructureMap() {
         // Build xml Mets:structMap
 
         $domDocument = new \DOMDocument();
@@ -809,8 +791,7 @@ 
         return $domDocument;
     }
 
-    public function setSlubInfo($value = '')
-    {
+    public function setSlubInfo($value = '') {
         // build DOMDocument with slub xml
         $domDocument = new \DOMDocument();
         $domDocument->loadXML($value);
@@ -822,8 +803,7 @@ 
      * @param  Array $array Array with slub information
      * @return xml        xml slubInfo
      */
-    public function buildMetsSlub()
-    {
+    public function buildMetsSlub() {
         $domDocument = new \DOMDocument();
         $domDocument->loadXML('<mets:amdSec ID="AMD_000" xmlns:mets="http://www.loc.gov/METS/"></mets:amdSec>');
         $domWrapElement = $domDocument->firstChild;
@@ -864,8 +844,7 @@ 
      * returns the mods xml string
      * @return string mods xml
      */
-    public function getSlubData()
-    {
+    public function getSlubData() {
         return $this->slubData->saveXML();
     }
 
@@ -873,8 +852,7 @@ 
      *
      * @param string $slubInfoData
      */
-    public function buildSlubInfoFromForm($slubInfoData, $documentType, $processNumber)
-    {
+    public function buildSlubInfoFromForm($slubInfoData, $documentType, $processNumber) {
         $this->xmlData = $this->slubData;
         if (is_array($slubInfoData['metadata'])) {
             foreach ($slubInfoData['metadata'] as $key => $group) {
@@ -957,8 +935,7 @@ 
      *
      * @return string slub info xml
      */
-    public function getSlubInfoData()
-    {
+    public function getSlubInfoData() {
         return $this->slubData->saveXML();
     }
 
@@ -967,8 +944,7 @@ 
      * @param string $objId
      * @return void
      */
-    public function setObjId($objId)
-    {
+    public function setObjId($objId) {
         $this->objId = $objId;
     }
 }

class.ext_update.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
             $clients = $clientRepository->findAllByPid($pid);
             if ($clients) {
                 if (count($clients) != 1) {
-                    throw new \Exception('Invalid number of client records for pid: '.$pid);
+                    throw new \Exception('Invalid number of client records for pid: ' . $pid);
                 }
             }
         }

Braces +3 added lines, -1 removed lines

@@ -32,7 +32,9 @@
 
         $documents = $documentRepository->findDocumentsWithoutProcessNumber();
 
-        if (count($documents) == 0) return;
+        if (count($documents) == 0) {
+            return;
+        }
 
         foreach ($documents as $document) {
             $pid = $document->getPid();

Classes/Controller/DocumentTypeController.php

Braces +8 added lines, -16 removed lines

@@ -17,8 +17,7 @@ 
 /**
  * DocumentTypeController
  */
-class DocumentTypeController extends \EWW\Dpf\Controller\AbstractController
-{
+class DocumentTypeController extends \EWW\Dpf\Controller\AbstractController {
 
     /**
      * documentTypeRepository
@@ -33,8 +32,7 @@ 
      *
      * @return void
      */
-    public function listAction()
-    {
+    public function listAction() {
         $documentTypes = $this->documentTypeRepository->findAll();
         $this->view->assign('documentTypes', $documentTypes);
     }
@@ -45,8 +43,7 @@ 
      * @param \EWW\Dpf\Domain\Model\DocumentType $documentType
      * @return void
      */
-    public function showAction(\EWW\Dpf\Domain\Model\DocumentType $documentType)
-    {
+    public function showAction(\EWW\Dpf\Domain\Model\DocumentType $documentType) {
         $this->view->assign('documentType', $documentType);
     }
 
@@ -57,8 +54,7 @@ 
      * @ignorevalidation $newDocumentType
      * @return void
      */
-    public function newAction(\EWW\Dpf\Domain\Model\DocumentType $newDocumentType = null)
-    {
+    public function newAction(\EWW\Dpf\Domain\Model\DocumentType $newDocumentType = null) {
         $this->view->assign('newDocumentType', $newDocumentType);
     }
 
@@ -68,8 +64,7 @@ 
      * @param \EWW\Dpf\Domain\Model\DocumentType $newDocumentType
      * @return void
      */
-    public function createAction(\EWW\Dpf\Domain\Model\DocumentType $newDocumentType)
-    {
+    public function createAction(\EWW\Dpf\Domain\Model\DocumentType $newDocumentType) {
         $this->addFlashMessage('The object was created. Please be aware that this action is publicly accessible unless you implement an access check. See <a href="http://wiki.typo3.org/T3Doc/Extension_Builder/Using_the_Extension_Builder#1._Model_the_domain" target="_blank">Wiki</a>', '', \TYPO3\CMS\Core\Messaging\AbstractMessage::ERROR);
         $this->documentTypeRepository->add($newDocumentType);
         $this->redirect('list');
@@ -82,8 +77,7 @@ 
      * @ignorevalidation $documentType
      * @return void
      */
-    public function editAction(\EWW\Dpf\Domain\Model\DocumentType $documentType)
-    {
+    public function editAction(\EWW\Dpf\Domain\Model\DocumentType $documentType) {
         $this->view->assign('documentType', $documentType);
     }
 
@@ -93,8 +87,7 @@ 
      * @param \EWW\Dpf\Domain\Model\DocumentType $documentType
      * @return void
      */
-    public function updateAction(\EWW\Dpf\Domain\Model\DocumentType $documentType)
-    {
+    public function updateAction(\EWW\Dpf\Domain\Model\DocumentType $documentType) {
         $this->addFlashMessage('The object was updated. Please be aware that this action is publicly accessible unless you implement an access check. See <a href="http://wiki.typo3.org/T3Doc/Extension_Builder/Using_the_Extension_Builder#1._Model_the_domain" target="_blank">Wiki</a>', '', \TYPO3\CMS\Core\Messaging\AbstractMessage::ERROR);
         $this->documentTypeRepository->update($documentType);
         $this->redirect('list');
@@ -106,8 +99,7 @@ 
      * @param \EWW\Dpf\Domain\Model\DocumentType $documentType
      * @return void
      */
-    public function deleteAction(\EWW\Dpf\Domain\Model\DocumentType $documentType)
-    {
+    public function deleteAction(\EWW\Dpf\Domain\Model\DocumentType $documentType) {
         $this->addFlashMessage('The object was deleted. Please be aware that this action is publicly accessible unless you implement an access check. See <a href="http://wiki.typo3.org/T3Doc/Extension_Builder/Using_the_Extension_Builder#1._Model_the_domain" target="_blank">Wiki</a>', '', \TYPO3\CMS\Core\Messaging\AbstractMessage::ERROR);
         $this->documentTypeRepository->remove($documentType);
         $this->redirect('list');

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

@@ -26,7 +26,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DocumentTypeRepository
      * @inject
      */
-    protected $documentTypeRepository = null;
+    protected $documentTypeRepository = NULL;
 
     /**
      * action list
@@ -57,7 +57,7 @@ 
      * @ignorevalidation $newDocumentType
      * @return void
      */
-    public function newAction(\EWW\Dpf\Domain\Model\DocumentType $newDocumentType = null)
+    public function newAction(\EWW\Dpf\Domain\Model\DocumentType $newDocumentType = NULL)
     {
         $this->view->assign('newDocumentType', $newDocumentType);
     }

Classes/Controller/AjaxDocumentFormController.php

Spacing +1 added lines, -1 removed lines

@@ -170,7 +170,7 @@
         }
 
         return json_encode(
-            array(
+            array (
                 'qucosaId' => $qucosaId,
                 'value'    => $urn,
             )

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

@@ -26,7 +26,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\MetadataGroupRepository
      * @inject
      */
-    protected $metadataGroupRepository = null;
+    protected $metadataGroupRepository = NULL;
 
     /**
      * metadataObjectRepository
@@ -34,7 +34,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\MetadataObjectRepository
      * @inject
      */
-    protected $metadataObjectRepository = null;
+    protected $metadataObjectRepository = NULL;
 
     /**
      *

Braces +7 added lines, -14 removed lines

@@ -21,8 +21,7 @@ 
 /**
  * DocumentFormController
  */
-class AjaxDocumentFormController extends \EWW\Dpf\Controller\AbstractController
-{
+class AjaxDocumentFormController extends \EWW\Dpf\Controller\AbstractController {
 
     /**
      * metadataGroupRepository
@@ -47,8 +46,7 @@ 
      * @param integer $groupIndex
      * @return void
      */
-    public function groupAction($pageUid, $groupUid, $groupIndex)
-    {
+    public function groupAction($pageUid, $groupUid, $groupIndex) {
 
         $group = $this->metadataGroupRepository->findByUid($groupUid);
 
@@ -91,8 +89,7 @@ 
      * @param integer $fieldIndex
      * @return void
      */
-    public function fieldAction($pageUid, $groupUid, $groupIndex, $fieldUid, $fieldIndex)
-    {
+    public function fieldAction($pageUid, $groupUid, $groupIndex, $fieldUid, $fieldIndex) {
 
         $field = $this->metadataObjectRepository->findByUid($fieldUid);
 
@@ -123,8 +120,7 @@ 
      *
      * @return void
      */
-    public function primaryUploadAction($groupIndex)
-    {
+    public function primaryUploadAction($groupIndex) {
     }
 
     /**
@@ -132,8 +128,7 @@ 
      * @param integer $groupIndex
      * @return void
      */
-    public function secondaryUploadAction($groupIndex)
-    {
+    public function secondaryUploadAction($groupIndex) {
         $this->view->assign('groupIndex', $groupIndex);
         //$this->view->assign('displayName','Sekundärdatei');
     }
@@ -144,8 +139,7 @@ 
      * @param integer $isPrimary
      * @return void
      */
-    public function deleteFileAction($fileUid, $isPrimary = 0)
-    {
+    public function deleteFileAction($fileUid, $isPrimary = 0) {
         $this->view->assign('fileUid', $fileUid);
         $this->view->assign('isPrimary', $isPrimary);
     }
@@ -155,8 +149,7 @@ 
      * @param string $qucosaId
      * @return string
      */
-    public function fillOutAction($qucosaId)
-    {
+    public function fillOutAction($qucosaId) {
 
         $urnService = $this->objectManager->get(Urn::class);
 

Classes/Controller/DocumentController.php

Spacing +2 added lines, -2 removed lines

@@ -160,7 +160,7 @@ 
 
         $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_discard.success';
 
-        $args = array();
+        $args = array ();
 
         $message = \TYPO3\CMS\Extbase\Utility\LocalizationUtility::translate($key, 'dpf', $args);
         $message = empty($message) ? "" : $message;
@@ -179,7 +179,7 @@ 
     public function duplicateAction(\EWW\Dpf\Domain\Model\Document $document)
     {
 
-        $args = array();
+        $args = array ();
 
         $key     = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_duplicate.success';
         $message = \TYPO3\CMS\Extbase\Utility\LocalizationUtility::translate($key, 'dpf', $args);

Braces +24 added lines, -48 removed lines

@@ -26,8 +26,7 @@ 
 /**
  * DocumentController
  */
-class DocumentController extends \EWW\Dpf\Controller\AbstractController
-{
+class DocumentController extends \EWW\Dpf\Controller\AbstractController {
 
     /**
      * documentRepository
@@ -50,8 +49,7 @@ 
      *
      * @return void
      */
-    public function listAction()
-    {
+    public function listAction() {
         $documents = $this->documentRepository->findAll();
 
         if ($this->request->hasArgument('message')) {
@@ -65,14 +63,12 @@ 
         $this->view->assign('documents', $documents);
     }
 
-    public function listNewAction()
-    {
+    public function listNewAction() {
         $documents = $this->documentRepository->getNewDocuments();
         $this->view->assign('documents', $documents);
     }
 
-    public function listEditAction()
-    {
+    public function listEditAction() {
         $documents = $this->documentRepository->getInProgressDocuments();
         $this->view->assign('documents', $documents);
     }
@@ -83,8 +79,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function showAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function showAction(\EWW\Dpf\Domain\Model\Document $document) {
 
         $this->view->assign('document', $document);
     }
@@ -96,8 +91,7 @@ 
      * @ignorevalidation $newDocument
      * @return void
      */
-    public function newAction(\EWW\Dpf\Domain\Model\Document $newDocument = null)
-    {
+    public function newAction(\EWW\Dpf\Domain\Model\Document $newDocument = null) {
 
         $this->view->assign('newDocument', $newDocument);
     }
@@ -108,8 +102,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $newDocument
      * @return void
      */
-    public function createAction(\EWW\Dpf\Domain\Model\Document $newDocument)
-    {
+    public function createAction(\EWW\Dpf\Domain\Model\Document $newDocument) {
 
         $this->addFlashMessage('The object was created. Please be aware that this action is publicly accessible unless you implement an access check. See <a href="http://wiki.typo3.org/T3Doc/Extension_Builder/Using_the_Extension_Builder#1._Model_the_domain" target="_blank">Wiki</a>', '', \TYPO3\CMS\Core\Messaging\AbstractMessage::ERROR);
         $this->documentRepository->add($newDocument);
@@ -123,8 +116,7 @@ 
      * @ignorevalidation $document
      * @return void
      */
-    public function editAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function editAction(\EWW\Dpf\Domain\Model\Document $document) {
         $this->view->assign('document', $document);
     }
 
@@ -134,8 +126,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function updateAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function updateAction(\EWW\Dpf\Domain\Model\Document $document) {
         $this->addFlashMessage('The object was updated. Please be aware that this action is publicly accessible unless you implement an access check. See <a href="http://wiki.typo3.org/T3Doc/Extension_Builder/Using_the_Extension_Builder#1._Model_the_domain" target="_blank">Wiki</a>', '', \TYPO3\CMS\Core\Messaging\AbstractMessage::ERROR);
         $this->documentRepository->update($document);
         $this->redirect('list');
@@ -147,8 +138,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function discardConfirmAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function discardConfirmAction(\EWW\Dpf\Domain\Model\Document $document) {
         $this->view->assign('document', $document);
     }
 
@@ -158,8 +148,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function discardAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function discardAction(\EWW\Dpf\Domain\Model\Document $document) {
         // remove document from local index
         $elasticsearchRepository = $this->objectManager->get(ElasticsearchRepository::class);
         // send document to index
@@ -185,8 +174,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function duplicateAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function duplicateAction(\EWW\Dpf\Domain\Model\Document $document) {
 
         $args = array();
 
@@ -236,8 +224,7 @@ 
      * @param string $releaseType
      * @return void
      */
-    public function releaseConfirmAction(\EWW\Dpf\Domain\Model\Document $document, $releaseType)
-    {
+    public function releaseConfirmAction(\EWW\Dpf\Domain\Model\Document $document, $releaseType) {
         $this->view->assign('releaseType', $releaseType);
         $this->view->assign('document', $document);
     }
@@ -248,8 +235,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function releaseAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function releaseAction(\EWW\Dpf\Domain\Model\Document $document) {
 
         // generate URN if needed
         $qucosaId = $document->getObjectIdentifier();
@@ -317,8 +303,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function restoreConfirmAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function restoreConfirmAction(\EWW\Dpf\Domain\Model\Document $document) {
         $this->view->assign('document', $document);
     }
 
@@ -328,8 +313,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function restoreAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function restoreAction(\EWW\Dpf\Domain\Model\Document $document) {
 
         $documentTransferManager = $this->objectManager->get(DocumentTransferManager::class);
         $remoteRepository        = $this->objectManager->get(FedoraRepository::class);
@@ -354,8 +338,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function deleteConfirmAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function deleteConfirmAction(\EWW\Dpf\Domain\Model\Document $document) {
         $this->view->assign('document', $document);
     }
 
@@ -365,8 +348,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function deleteAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function deleteAction(\EWW\Dpf\Domain\Model\Document $document) {
 
         $documentTransferManager = $this->objectManager->get(DocumentTransferManager::class);
         $remoteRepository        = $this->objectManager->get(FedoraRepository::class);
@@ -391,8 +373,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function activateConfirmAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function activateConfirmAction(\EWW\Dpf\Domain\Model\Document $document) {
         $this->view->assign('document', $document);
     }
 
@@ -402,8 +383,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function activateAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function activateAction(\EWW\Dpf\Domain\Model\Document $document) {
 
         $documentTransferManager = $this->objectManager->get(DocumentTransferManager::class);
         $remoteRepository        = $this->objectManager->get(FedoraRepository::class);
@@ -428,8 +408,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function inactivateConfirmAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function inactivateConfirmAction(\EWW\Dpf\Domain\Model\Document $document) {
         $this->view->assign('document', $document);
     }
 
@@ -439,8 +418,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return void
      */
-    public function inactivateAction(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function inactivateAction(\EWW\Dpf\Domain\Model\Document $document) {
 
         $documentTransferManager = $this->objectManager->get(DocumentTransferManager::class);
         $remoteRepository        = $this->objectManager->get(FedoraRepository::class);
@@ -459,8 +437,7 @@ 
         $this->redirect('list');
     }
 
-    protected function getStoragePID()
-    {
+    protected function getStoragePID() {
         return $this->settings['persistence']['classes']['EWW\Dpf\Domain\Model\Document']['newRecordStoragePid'];
     }
 
@@ -470,8 +447,7 @@ 
      * @param string $key
      * @param string $severity
      */
-    protected function flashMessage(\EWW\Dpf\Domain\Model\Document $document, $key, $severity)
-    {
+    protected function flashMessage(\EWW\Dpf\Domain\Model\Document $document, $key, $severity) {
 
         // Show success or failure of the action in a flash message
         $args[] = $document->getTitle();

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

@@ -35,7 +35,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DocumentRepository
      * @inject
      */
-    protected $documentRepository = null;
+    protected $documentRepository = NULL;
 
     /**
      * persistence manager
@@ -96,7 +96,7 @@ 
      * @ignorevalidation $newDocument
      * @return void
      */
-    public function newAction(\EWW\Dpf\Domain\Model\Document $newDocument = null)
+    public function newAction(\EWW\Dpf\Domain\Model\Document $newDocument = NULL)
     {
         $this->view->assign('newDocument', $newDocument);
     }
@@ -483,7 +483,7 @@ 
             $message,
             '',
             $severity,
-            true
+            TRUE
         );
 
     }