kitodo / kitodo-publication

src/Configuration/TCA/Overrides/fe_users.php

Spacing +6 added lines, -6 removed lines

@@ -86,28 +86,28 @@
             ),
         )
     ),
-    'fis_pers_id' => array(
+    'fis_pers_id' => array (
         'exclude' => 0,
         'label'   => 'LLL:EXT:dpf/Resources/Private/Language/locallang_db.xlf:tx_dpf_domain_model_feuser.fis_pers_id',
-        'config'  => array(
+        'config'  => array (
             'type' => 'input',
             'size' => '30',
             'eval' => 'trim',
         ),
     ),
-    'orga_name' => array(
+    'orga_name' => array (
         'exclude' => 0,
         'label'   => 'LLL:EXT:dpf/Resources/Private/Language/locallang_db.xlf:tx_dpf_domain_model_feuser.orga_name',
-        'config'  => array(
+        'config'  => array (
             'type' => 'input',
             'size' => '30',
             'eval' => 'trim',
         ),
     ),
-    'api_token' => array(
+    'api_token' => array (
         'exclude' => 0,
         'label'   => 'LLL:EXT:dpf/Resources/Private/Language/locallang_db.xlf:tx_dpf_domain_model_feuser.api_token',
-        'config'  => array(
+        'config'  => array (
             'type' => 'input',
             'size' => '30',
             'eval' => 'trim',

src/Classes/Configuration/ClientConfigurationManager.php

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
      *
      * @var array
      */
-    protected $settings = array();
+    protected $settings = array ();
 
 
     /**
@@ -58,7 +58,7 @@ 
      *
      * @var array
      */
-    protected $extensionConfiguration = array();
+    protected $extensionConfiguration = array ();
 
     public function __construct()
     {
@@ -66,7 +66,7 @@ 
         $clientRepository = $objectManager->get(ClientRepository::class);
 
         if (TYPO3_MODE === 'BE') {
-            $selectedPageId = (int)\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('id');
+            $selectedPageId = (int) \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('id');
             if ($selectedPageId) {
                 $this->client = $clientRepository->findAll()->current();
 

src/Classes/Helper/XPath.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(ObjectManager::class);
         $clientConfigurationManager = $objectManager->get(ClientConfigurationManager::class);
 
-        $namespaceConfiguration = explode(";",$clientConfigurationManager->getNamespaces());
+        $namespaceConfiguration = explode(";", $clientConfigurationManager->getNamespaces());
 
         $xpath = new \DOMXPath($dom);
 

src/Classes/Helper/Mods.php

Spacing +3 added lines, -3 removed lines

@@ -135,8 +135,8 @@ 
 
             $identifierNode = $xpath->query('mods:nameIdentifier[@type="FOBID"]', $personNode);
 
-            $affiliationNodes  = $xpath->query('mods:affiliation', $personNode);
-            $affiliationIdentifierNodes  = $xpath->query(
+            $affiliationNodes = $xpath->query('mods:affiliation', $personNode);
+            $affiliationIdentifierNodes = $xpath->query(
                 'mods:nameIdentifier[@type="ScopusAuthorID"][@typeURI="http://www.scopus.com/authid"]',
                 $personNode
             );
@@ -282,7 +282,7 @@ 
 
     public function getOriginalSourceTitle()
     {
-        $node= $this->getModsXpath()->query('/mods:mods/mods:relatedItem[@type="original"]/mods:titleInfo/mods:title');
+        $node = $this->getModsXpath()->query('/mods:mods/mods:relatedItem[@type="original"]/mods:titleInfo/mods:title');
         if ($node->length > 0) {
             return $node->item(0)->nodeValue;
         }

src/Classes/Helper/InternalFormat.php

Spacing +16 added lines, -16 removed lines

@@ -86,7 +86,7 @@ 
     public function setRepositoryState($state)
     {
         $stateXpath = $this->clientConfigurationManager->getStateXpath();
-        $this->setValue($stateXpath,$state);
+        $this->setValue($stateXpath, $state);
     }
 
     public function getProcessNumber()
@@ -314,7 +314,7 @@ 
         $xpath = $this->getXpath();
         $data = [];
         $sourceDetailsXpaths = $this->clientConfigurationManager->getSourceDetailsXpaths();
-        $sourceDetailsXpathList = explode(";", trim($sourceDetailsXpaths," ;"));
+        $sourceDetailsXpathList = explode(";", trim($sourceDetailsXpaths, " ;"));
         $dataNodes = [];
 
         foreach ($sourceDetailsXpathList as $sourceDetailsXpathItem) {
@@ -347,7 +347,7 @@ 
     {
         $xpath = $this->getXpath();
         $personXpath = $this->clientConfigurationManager->getPersonXpath();
-        $fisIdentifierXpath =  $this->clientConfigurationManager->getPersonFisIdentifierXpath();
+        $fisIdentifierXpath = $this->clientConfigurationManager->getPersonFisIdentifierXpath();
         $personNodes = $xpath->query(self::rootNode . $personXpath);
         $identifiers = [];
         foreach ($personNodes as $key => $node) {
@@ -379,9 +379,9 @@ 
         $xpath = $this->getXpath();
         $notesNodes = $xpath->query(self::rootNode . $notesXpath);
 
-        $notes = array();
+        $notes = array ();
 
-        for ($i=0; $i < $notesNodes->length; $i++)
+        for ($i = 0; $i < $notesNodes->length; $i++)
         {
             $notes[] = $notesNodes->item($i)->nodeValue;
         }
@@ -395,7 +395,7 @@ 
 
         $parserGenerator = new ParserGenerator();
         $parserGenerator->setXml($this->xml->saveXML());
-        $parserGenerator->customXPath($notesXpath,true, $noteContent);
+        $parserGenerator->customXPath($notesXpath, true, $noteContent);
         $this->xml = new \DOMDocument();
         $this->xml->loadXML($parserGenerator->getXMLData());
     }
@@ -422,9 +422,9 @@ 
         $familyXpath = $this->clientConfigurationManager->getPersonFamilyXpath();
         $givenXpath = $this->clientConfigurationManager->getPersonGivenXpath();
         $roleXpath = $this->clientConfigurationManager->getPersonRoleXpath();
-        $fisIdentifierXpath =  $this->clientConfigurationManager->getPersonFisIdentifierXpath();
-        $affiliationXpath =  $this->clientConfigurationManager->getPersonAffiliationXpath();
-        $affiliationIdentifierXpath =  $this->clientConfigurationManager->getPersonAffiliationIdentifierXpath();
+        $fisIdentifierXpath = $this->clientConfigurationManager->getPersonFisIdentifierXpath();
+        $affiliationXpath = $this->clientConfigurationManager->getPersonAffiliationXpath();
+        $affiliationIdentifierXpath = $this->clientConfigurationManager->getPersonAffiliationIdentifierXpath();
 
         $xpath = $this->getXpath();
         $personNodes = $xpath->query(self::rootNode . $personXpath);
@@ -504,9 +504,9 @@ 
      */
     public function getValidation()
     {
-        $validationXpath =  $this->clientConfigurationManager->getValidationXpath();
+        $validationXpath = $this->clientConfigurationManager->getValidationXpath();
         $validation = $this->getValue($validationXpath);
-        return (strtolower($validation) === 'true')? true : false;
+        return (strtolower($validation) === 'true') ? true : false;
     }
 
     /**
@@ -514,8 +514,8 @@ 
      */
     public function setValidation($validated)
     {
-        $validationXpath =  $this->clientConfigurationManager->getValidationXpath();
-        $this->setValue($validationXpath, ($validated? 'true' : 'false'));
+        $validationXpath = $this->clientConfigurationManager->getValidationXpath();
+        $this->setValue($validationXpath, ($validated ? 'true' : 'false'));
     }
 
     /**
@@ -523,7 +523,7 @@ 
      */
     public function setFisId($fisId)
     {
-        $fisIdXpath =  $this->clientConfigurationManager->getFisIdXpath();
+        $fisIdXpath = $this->clientConfigurationManager->getFisIdXpath();
         $this->setValue($fisIdXpath, $fisId);
     }
 
@@ -548,10 +548,10 @@ 
         $nodes = $xpath->query(self::rootNode . $xpathString);
         if ($nodes->length > 0) {
             $nodes->item(0)->nodeValue = $value;
-        } elseif(!empty($value)) {
+        } elseif (!empty($value)) {
             $parserGenerator = new ParserGenerator();
             $parserGenerator->setXml($this->xml->saveXML());
-            $parserGenerator->customXPath($xpathString,true, $value);
+            $parserGenerator->customXPath($xpathString, true, $value);
             $this->xml = new \DOMDocument();
             $this->xml->loadXML($parserGenerator->getXMLData());
         }

src/Classes/Services/MetsExporter.php

Spacing +7 added lines, -7 removed lines

@@ -44,13 +44,13 @@ 
      *
      * @var array
      */
-    protected $formData = array();
+    protected $formData = array ();
 
     /**
      * files from form
      * @var array
      */
-    protected $files = array();
+    protected $files = array ();
 
     /**
      * metsData
@@ -105,7 +105,7 @@ 
 
         $this->documentTypeRepository = $objectManager->get(DocumentTypeRepository::class);
 
-        $namespaceConfiguration = explode(";",$this->clientConfigurationManager->getNamespaces());
+        $namespaceConfiguration = explode(";", $this->clientConfigurationManager->getNamespaces());
 
 
         foreach ($namespaceConfiguration as $key => $value) {
@@ -115,7 +115,7 @@ 
 
         $this->xmlHeader = '<data' . $this->namespaceString . '></data>';
 
-        $this->xmlData =  new \DOMDocument();
+        $this->xmlData = new \DOMDocument();
         $this->xmlData->loadXML($this->xmlHeader);
 
         // Parser
@@ -206,7 +206,7 @@ 
             $values     = $group['values'];
             $attributes = $group['attributes'];
 
-            $attributeXPath     = '';
+            $attributeXPath = '';
             foreach ($attributes as $attribute) {
                 $attributeXPath .= '[' . $attribute['mapping'] . '="' . $attribute['value'] . '"]';
             }
@@ -314,7 +314,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]);
 
@@ -373,7 +373,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]);
                 }

src/Classes/Services/ElasticSearch/QueryBuilder.php

Spacing +37 added lines, -37 removed lines

@@ -138,7 +138,7 @@ 
             $query = [
                 'bool' => [
                     'must' => [
-                        'match_all' => (object)[]
+                        'match_all' => (object) []
                     ],
                     'filter' => $queryFilter
                 ]
@@ -180,11 +180,11 @@ 
                             'script' => [
                                 'lang' => 'painless',
                                 'source' =>
-                                    "for (int i = 0; i < doc['collections'].length; ++i) {".
-                                    "    if(doc['collections'][i] =='".$this->getSettings()['universityCollection']."') {".
-                                    "        return 'true';".
-                                    "    }".
-                                    "}".
+                                    "for (int i = 0; i < doc['collections'].length; ++i) {" .
+                                    "    if(doc['collections'][i] =='" . $this->getSettings()['universityCollection'] . "') {" .
+                                    "        return 'true';" .
+                                    "    }" .
+                                    "}" .
                                     "return 'false';"
                             ]
                         ]
@@ -200,23 +200,23 @@ 
                             'script' => [
                                 'lang' => 'painless',
                                 'source' =>
-                                    "if (".
-                                    "    doc['creator'].size() > 0 &&".
-                                    "    doc['creator'].value == '".$this->security->getUser()->getUid()."') {".
-                                    "    return 'self';".
-                                    "}".
-                                    "if (".
-                                    "    doc['creatorRole'].size() > 0 &&".
-                                    "    doc['creatorRole'].value == '".Security::ROLE_LIBRARIAN."'".
-                                    ") {".
-                                    "    return 'librarian';".
-                                    "}".
-                                    "if (".
-                                    "    doc['creatorRole'].size() > 0 &&".
-                                    "    doc['creatorRole'].value == '".Security::ROLE_RESEARCHER."'".
-                                    ") {".
-                                    "    return 'user';".
-                                    "}".
+                                    "if (" .
+                                    "    doc['creator'].size() > 0 &&" .
+                                    "    doc['creator'].value == '" . $this->security->getUser()->getUid() . "') {" .
+                                    "    return 'self';" .
+                                    "}" .
+                                    "if (" .
+                                    "    doc['creatorRole'].size() > 0 &&" .
+                                    "    doc['creatorRole'].value == '" . Security::ROLE_LIBRARIAN . "'" .
+                                    ") {" .
+                                    "    return 'librarian';" .
+                                    "}" .
+                                    "if (" .
+                                    "    doc['creatorRole'].size() > 0 &&" .
+                                    "    doc['creatorRole'].value == '" . Security::ROLE_RESEARCHER . "'" .
+                                    ") {" .
+                                    "    return 'user';" .
+                                    "}" .
                                     "return 'unknown';"
                             ]
                         ]
@@ -429,8 +429,8 @@ 
      */
     protected function buildSortQueryPart($sortField, $sortOrder) {
 
-        $sortField = ($sortField)? $sortField : 'title';
-        $sortOrder = ($sortOrder)? $sortOrder : 'asc';
+        $sortField = ($sortField) ? $sortField : 'title';
+        $sortOrder = ($sortOrder) ? $sortOrder : 'asc';
 
         // Build the sorting part.
         $script = "";
@@ -460,11 +460,11 @@ 
             ];
         } else {
             if ($sortField == 'title') {
-                $sortField.= ".keyword";
+                $sortField .= ".keyword";
             }
 
             if ($sortField == 'personsSort') {
-                $sortField.= ".keyword";
+                $sortField .= ".keyword";
             }
 
             $sort = [
@@ -481,7 +481,7 @@ 
     protected function getSortScriptUniversityCollection($collection)
     {
         $script  = "for (int i = 0; i < doc['collections'].length; ++i) {";
-        $script .= "    if (doc['collections'][i] == '".$collection."') {";
+        $script .= "    if (doc['collections'][i] == '" . $collection . "') {";
         $script .= "        return '1';";
         $script .= "    }";
         $script .= "}";
@@ -502,13 +502,13 @@ 
 
     protected function getSortScriptCreatorRole($feUserUid)
     {
-        $script = "if (doc['creator'].value == '".$feUserUid."') {";
+        $script = "if (doc['creator'].value == '" . $feUserUid . "') {";
         $script .= "    return '1';";
         $script .= "}";
-        $script .= "if (doc['creatorRole'].value == '".Security::ROLE_LIBRARIAN."') {";
+        $script .= "if (doc['creatorRole'].value == '" . Security::ROLE_LIBRARIAN . "') {";
         $script .= "return '2';";
         $script .= "}";
-        $script .= "if (doc['creatorRole'].value == '".Security::ROLE_RESEARCHER."') {";
+        $script .= "if (doc['creatorRole'].value == '" . Security::ROLE_RESEARCHER . "') {";
         $script .= "    return '3';";
         $script .= "}";
         $script .= "return '4';";
@@ -523,15 +523,15 @@ 
         foreach (DocumentWorkflow::PLACES as $state) {
             if (array_key_exists($state, DocumentWorkflow::STATE_TO_ALIASSTATE_MAPPING)) {
                 $aliasState = DocumentWorkflow::STATE_TO_ALIASSTATE_MAPPING[$state];
-                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:manager.documentList.state.'.$aliasState;
+                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:manager.documentList.state.' . $aliasState;
                 $stateName = LocalizationUtility::translate($key, 'dpf');
-                $sortStates[] = "if (doc['state'].value == '".$state."') return '".$stateName."';";
+                $sortStates[] = "if (doc['state'].value == '" . $state . "') return '" . $stateName . "';";
             }
         }
 
         $sortStates = implode(" ", $sortStates);
 
-        return $sortStates." return '';";
+        return $sortStates . " return '';";
     }
 
 
@@ -540,14 +540,14 @@ 
         $sortDoctypes = [];
         foreach ($this->documentTypeRepository->findAll() as $documentType) {
             if ($documentType->getName() && $documentType->getDisplayname()) {
-                $sortDoctypes[] = "if (doc['doctype'].value == '".$documentType->getName()."')"
-                    ." return '".$documentType->getDisplayname()."';";
+                $sortDoctypes[] = "if (doc['doctype'].value == '" . $documentType->getName() . "')"
+                    ." return '" . $documentType->getDisplayname() . "';";
             }
         }
 
         $sortDoctypes = implode(" ", $sortDoctypes);
 
-        return $sortDoctypes." return '';";
+        return $sortDoctypes . " return '';";
     }
 
 }

src/Classes/Services/FeUser/UnpaywallDataService.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 
     public function searchRequest($searchTerm) {
         $mail = $this->clientConfigurationManager->getSetting("adminEmail");
-        $response = Request::get($this->apiUrl . '' . $searchTerm .'?email=' . $mail)
+        $response = Request::get($this->apiUrl . '' . $searchTerm . '?email=' . $mail)
             ->send();
         if ($response->body->HTTP_status_code == 404) {
             return ['entries' => ''];
@@ -37,7 +37,7 @@ 
 
     public function getDataRequest($id) {
         $mail = $this->clientConfigurationManager->getSetting("adminEmail");
-        $response = Request::get($this->apiUrl . '' . $id .'?email=' . $mail)
+        $response = Request::get($this->apiUrl . '' . $id . '?email=' . $mail)
             ->send();
 
         if ($response->body->HTTP_status_code == 404) {

src/Classes/Services/FeUser/FisDataService.php

Spacing +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
 
     protected function getPersonRequestBody($id) {
         $graphQl = '{
-            person (id:"'.$id.'") 
+            person (id:"'.$id . '") 
             {
                 _createDate
                 _createUser
@@ -72,7 +72,7 @@ 
 
     protected function getSearchPersonBody($searchTerm) {
         $graphQl = '{   
-            staff(filter: {fullName: "'.$searchTerm.'"}, pageSize: 100) 
+            staff(filter: {fullName: "'.$searchTerm . '"}, pageSize: 100) 
             {     
                 entries 
                 {
@@ -97,7 +97,7 @@ 
 
     protected function getOrgaRequestBody($id) {
         $graphQl = '{   
-            organisationalUnit(id:'.$id.') {
+            organisationalUnit(id:'.$id . ') {
                 titleDe
                 id
             }
@@ -108,7 +108,7 @@ 
 
     protected function getSearchOrgaBody($searchTerm) {
         $graphQl = '{   
-            organisationalUnits(filter: {text: "'.$searchTerm.'"}) 
+            organisationalUnits(filter: {text: "'.$searchTerm . '"}) 
             {     
                 entries 
                 {