kitodo / kitodo-publication

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 '';";
     }
 
 }

Braces +8 added lines, -16 removed lines

@@ -19,8 +19,7 @@ 
 use EWW\Dpf\Security\Security;
 use TYPO3\CMS\Extbase\Utility\LocalizationUtility;
 
-class QueryBuilder
-{
+class QueryBuilder {
     /**
      *
      * @var \TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface
@@ -51,8 +50,7 @@ 
      *
      * @return mixed
      */
-    public function getSettings()
-    {
+    public function getSettings() {
         $frameworkConfiguration = $this->configurationManager->getConfiguration(
             \TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface::CONFIGURATION_TYPE_FRAMEWORK
         );
@@ -78,8 +76,7 @@ 
     public function buildQuery(
         $itemsPerPage, $workspaceFilter, $from = 0, $bookmarkIdentifiers = [], $filters = [],
         $excludeFilters = [], $sortField = null, $sortOrder = null, $queryString = null
-    )
-    {
+    ) {
 
         if ($workspaceFilter) {
             $workspaceFilter = [0 => $workspaceFilter];
@@ -478,8 +475,7 @@ 
     }
 
 
-    protected function getSortScriptUniversityCollection($collection)
-    {
+    protected function getSortScriptUniversityCollection($collection) {
         $script  = "for (int i = 0; i < doc['collections'].length; ++i) {";
         $script .= "    if (doc['collections'][i] == '".$collection."') {";
         $script .= "        return '1';";
@@ -490,8 +486,7 @@ 
         return $script;
     }
 
-    protected function getSortScriptHasFiles()
-    {
+    protected function getSortScriptHasFiles() {
         $script = "if (doc['hasFiles'].value == 'true') {";
         $script .= "    return '1';";
         $script .= "}";
@@ -500,8 +495,7 @@ 
         return $script;
     }
 
-    protected function getSortScriptCreatorRole($feUserUid)
-    {
+    protected function getSortScriptCreatorRole($feUserUid) {
         $script = "if (doc['creator'].value == '".$feUserUid."') {";
         $script .= "    return '1';";
         $script .= "}";
@@ -517,8 +511,7 @@ 
     }
 
 
-    protected function getSortScriptState()
-    {
+    protected function getSortScriptState() {
         $sortStates = [];
         foreach (DocumentWorkflow::PLACES as $state) {
             if (array_key_exists($state, DocumentWorkflow::STATE_TO_ALIASSTATE_MAPPING)) {
@@ -535,8 +528,7 @@ 
     }
 
 
-    protected function getSortScriptDoctype()
-    {
+    protected function getSortScriptDoctype() {
         $sortDoctypes = [];
         foreach ($this->documentTypeRepository->findAll() as $documentType) {
             if ($documentType->getName() && $documentType->getDisplayname()) {

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

@@ -35,7 +35,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DocumentTypeRepository
      * @inject
      */
-    protected $documentTypeRepository = null;
+    protected $documentTypeRepository = NULL;
 
     /**
      * security
@@ -43,7 +43,7 @@ 
      * @var \EWW\Dpf\Security\Security
      * @inject
      */
-    protected $security = null;
+    protected $security = NULL;
 
 
     /**
@@ -77,7 +77,7 @@ 
      */
     public function buildQuery(
         $itemsPerPage, $workspaceFilter, $from = 0, $bookmarkIdentifiers = [], $filters = [],
-        $excludeFilters = [], $sortField = null, $sortOrder = null, $queryString = null
+        $excludeFilters = [], $sortField = NULL, $sortOrder = NULL, $queryString = NULL
     )
     {
 
@@ -253,7 +253,7 @@ 
 
             foreach ($filters as $key => $filterValues) {
                 $queryFilterPart = [];
-                if (in_array($key, $validKeys, true)) {
+                if (in_array($key, $validKeys, TRUE)) {
                     if ($key == 'universityCollection') {
                         if ($filterValues && is_array($filterValues)) {
                             if (in_array("true", $filterValues)) {

src/Classes/Services/ElasticSearch/ElasticSearch.php

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

@@ -37,7 +37,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\FrontendUserRepository
      * @inject
      */
-    protected $frontendUserRepository = null;
+    protected $frontendUserRepository = NULL;
 
     protected $client;
 
@@ -140,7 +140,7 @@ 
                 ],
                 'mappings' => [
                     '_source' => [
-                        'enabled' => true
+                        'enabled' => TRUE
                     ],
                     //'dynamic' => 'strict',
                     'properties' => [
@@ -203,11 +203,11 @@ 
                                 ],
                                 'fobId' => [
                                     //'type' => 'keyword'
-                                    'enabled' => false
+                                    'enabled' => FALSE
                                 ],
                                 'index' => [
                                     //'type' => 'integer'
-                                    'enabled' => false
+                                    'enabled' => FALSE
                                 ]
                             ]
                         ],
@@ -261,7 +261,7 @@ 
             if ($document->getCreator()) {
                 $data->creator = $document->getCreator();
             } else {
-                $data->creator = null;
+                $data->creator = NULL;
             }
 
 
@@ -289,9 +289,9 @@ 
 
             $files = $document->getFile();
             if ($files->count() > 0) {
-                $data->hasFiles = true;
+                $data->hasFiles = TRUE;
             } else {
-                $data->hasFiles = false;
+                $data->hasFiles = FALSE;
             }
 
 
@@ -328,11 +328,11 @@ 
 
             $data->source = $document->getSourceDetails();
 
-            $data->universityCollection = false;
+            $data->universityCollection = FALSE;
             if ($data->collections && is_array($data->collections)) {
                 foreach ($data->collections as $collection) {
                     if ($collection == $this->getSettings()['universityCollection']) {
-                        $data->universityCollection = true;
+                        $data->universityCollection = TRUE;
                         break;
                     }
                 }
@@ -342,7 +342,7 @@ 
             if ($embargoDate instanceof \DateTime) {
                 $data->embargoDate = $embargoDate->format("Y-m-d");
             } else {
-                $data->embargoDate = null;
+                $data->embargoDate = NULL;
             }
 
             $data->originalSourceTitle = $internalFormat->getOriginalSourceTitle();
@@ -409,7 +409,7 @@ 
      * @param  array $query search query
      * @return array        result list
      */
-    public function search($query, $type = null)
+    public function search($query, $type = NULL)
     {
         try {
             // define type and index

Braces +9 added lines, -18 removed lines

@@ -23,8 +23,7 @@ 
 use TYPO3\CMS\Core\Log\LogManager;
 use TYPO3\CMS\Extbase\Utility\LocalizationUtility;
 
-class ElasticSearch
-{
+class ElasticSearch {
     /**
      *
      * @var \TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface
@@ -60,8 +59,7 @@ 
     /**
      * elasticsearch client constructor
      */
-    public function __construct()
-    {
+    public function __construct() {
         $objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(ObjectManager::class);
 
         $this->elasticsearchMapper = $objectManager->get(ElasticsearchMapper::class);
@@ -94,8 +92,7 @@ 
      *
      * @return mixed
      */
-    public function getSettings()
-    {
+    public function getSettings() {
         $frameworkConfiguration = $this->configurationManager->getConfiguration(
             \TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface::CONFIGURATION_TYPE_FRAMEWORK
         );
@@ -108,8 +105,7 @@ 
      *
      * @param $indexName
      */
-    protected function initializeIndex($indexName)
-    {
+    protected function initializeIndex($indexName) {
         $paramsIndex = [
             'index' => $indexName,
             'body' => [
@@ -247,8 +243,7 @@ 
      *
      * @param Document $document
      */
-    public function index($document)
-    {
+    public function index($document) {
         try {
             $data = json_decode($this->elasticsearchMapper->getElasticsearchJson($document));
         } catch (\Throwable $throwable) {
@@ -381,8 +376,7 @@ 
      *
      * @param string $identifier
      */
-    public function delete($identifier)
-    {
+    public function delete($identifier) {
         try {
 
             $params = [
@@ -408,8 +402,7 @@ 
     /**
      * @param $identifier
      */
-    public function getDocument($identifier)
-    {
+    public function getDocument($identifier) {
         $params = [
             'index' => $this->indexName,
             'id'    => $identifier
@@ -424,8 +417,7 @@ 
      * @param  array $query search query
      * @return array        result list
      */
-    public function search($query, $type = null)
-    {
+    public function search($query, $type = null) {
         try {
             // define type and index
             if (empty($query['index'])) {
@@ -457,8 +449,7 @@ 
      * Get the results
      * @return mixed
      */
-    public function getResults()
-    {
+    public function getResults() {
         // return results from the last search request
         return $this->results;
     }

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) {

Braces +1 added lines, -2 removed lines

@@ -5,8 +5,7 @@
 use TYPO3\CMS\Extbase\Object\ObjectManager;
 use \Httpful\Request;
 
-class UnpaywallDataService
-{
+class UnpaywallDataService {
     /**
      * clientConfigurationManager
      *

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 
                 {

Braces +1 added lines, -2 removed lines

@@ -5,8 +5,7 @@
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface;
 
-class FisDataService
-{
+class FisDataService {
     /**
      * @var string
      */

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

@@ -35,7 +35,7 @@ 
 
             return $response->body->data->person;
         } catch (\Throwable $e) {
-            return null;
+            return NULL;
         }
     }
 
@@ -47,7 +47,7 @@ 
 
             return $response->body->data->staff;
         } catch (\Throwable $e) {
-            return null;
+            return NULL;
         }
     }
 
@@ -59,7 +59,7 @@ 
 
             return $response->body->data->organisationalUnit;
         } catch (\Throwable $e) {
-            return null;
+            return NULL;
         }
     }
 
@@ -71,7 +71,7 @@ 
 
             return $response->body->data->organisationalUnits;
         } catch (\Throwable $e) {
-            return null;
+            return NULL;
         }
     }
 

src/Classes/Services/FeUser/ZdbDataService.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
     }
 
     public function getDataRequest($zdbId) {
-        $response = Request::get($this->apiUrl . 'resource/' . $zdbId .'/')
+        $response = Request::get($this->apiUrl . 'resource/' . $zdbId . '/')
             ->send();
         
         return $response->body;

Braces +1 added lines, -2 removed lines

@@ -3,8 +3,7 @@
 
 use \Httpful\Request;
 
-class ZdbDataService
-{
+class ZdbDataService {
 
     protected $apiUrl = 'https://www.zeitschriftendatenbank.de/api/hydra/';
 

src/Classes/Services/FeUser/OrcidDataService.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
     public function searchPersonRequest($searchTerm) {
         $response = Request::get($this->apiUrl . '/expanded-search/?q=' . $this->searchTermReplacement($searchTerm))
             ->expectsJson()
-            ->addHeader('Accept','*/*')
+            ->addHeader('Accept', '*/*')
             ->addHeader('Content-Type', 'application/vnd.orcid+json')
             ->send();
 
@@ -30,7 +30,7 @@ 
     public function getPersonData($orcidId) {
         $response = Request::get($this->apiUrl . '/expanded-search/?q=orcid:' . $orcidId)
             ->expectsJson()
-            ->addHeader('Accept','*/*')
+            ->addHeader('Accept', '*/*')
             ->addHeader('Content-Type', 'application/vnd.orcid+json')
             ->send();
 

Braces +1 added lines, -2 removed lines

@@ -3,8 +3,7 @@
 
 use \Httpful\Request;
 
-class OrcidDataService
-{
+class OrcidDataService {
     protected $apiUrl = 'https://pub.orcid.org/v3.0';
 
     protected $params = '&start=0&rows=20';

src/Classes/Services/FeUser/RorDataService.php

Braces +1 added lines, -2 removed lines

@@ -3,8 +3,7 @@
 
 use \Httpful\Request;
 
-class RorDataService
-{
+class RorDataService {
 
     protected $apiUrl = 'https://api.ror.org/organizations';
 

src/Classes/Services/FeUser/GndDataService.php

Braces +1 added lines, -2 removed lines

@@ -3,8 +3,7 @@
 
 use \Httpful\Request;
 
-class GndDataService
-{
+class GndDataService {
 
     protected $apiUrl = 'http://lobid.org/gnd/';
 

src/Classes/Services/ParserGenerator.php

Braces +10 added lines, -20 removed lines

@@ -23,8 +23,7 @@ 
 /**
  * ParserGenerator
  */
-class ParserGenerator
-{
+class ParserGenerator {
     /**
      * clientConfigurationManager
      *
@@ -93,8 +92,7 @@ 
     /**
      * Constructor
      */
-    public function __construct()
-    {
+    public function __construct() {
         $objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(ObjectManager::class);
         $this->clientConfigurationManager = $objectManager->get(ClientConfigurationManager::class);
 
@@ -123,16 +121,14 @@ 
      * returns the mods xml string
      * @return string mods xml
      */
-    public function getXMLData()
-    {
+    public function getXMLData() {
         $xml = $this->xmlData->saveXML();
         $xml = preg_replace("/eww=\"\d-\d-\d\"/", '${1}${2}${3}', $xml);
 
         return $xml;
     }
 
-    public function transformInputXML($xml)
-    {
+    public function transformInputXML($xml) {
         $XSLTransformator = new XSLTransformator();
         return $XSLTransformator->transformInputXML($xml);
     }
@@ -141,8 +137,7 @@ 
      * @param $document
      * @return string The transformed xml
      */
-    public function getTransformedOutputXML($document)
-    {
+    public function getTransformedOutputXML($document) {
         $XSLTransformator = new XSLTransformator();
         return $XSLTransformator->getTransformedOutputXML($document);
     }
@@ -152,8 +147,7 @@ 
      * build mods from form array
      * @param array $array structured form data array
      */
-    public function buildXmlFromForm($array)
-    {
+    public function buildXmlFromForm($array) {
         $this->xmlData = $this->xmlData;
         // Build xml mods from form fields
         // loop each group
@@ -234,8 +228,7 @@ 
      * @param  xpath $xPath xPath expression
      * @return xml
      */
-    public function parseXPath($xPath)
-    {
+    public function parseXPath($xPath) {
 
         $this->parser->generateXmlFromXPath($xPath);
         $xml = $this->parser->getXML();
@@ -243,8 +236,7 @@ 
         return $xml;
     }
 
-    public function parseXPathWrapped($xPath)
-    {
+    public function parseXPathWrapped($xPath) {
         $this->parser->generateXmlFromXPath($xPath);
         $xml = $this->parser->getXML();
 
@@ -259,8 +251,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);
@@ -443,8 +434,7 @@ 
      * sets the file data and generates file xml
      * @param string $value
      */
-    public function setFileData($value = '')
-    {
+    public function setFileData($value = '') {
         $this->files = $value;
         $this->generateFileXML();
     }

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

@@ -38,7 +38,7 @@ 
      *
      * @var \EWW\Dpf\Domain\Repository\DocumentTypeRepository
      */
-    protected $documentTypeRepository = null;
+    protected $documentTypeRepository = NULL;
 
     /**
      * formData
@@ -76,7 +76,7 @@ 
      * xPathXMLGenerator
      * @var object
      */
-    protected $parser = null;
+    protected $parser = NULL;
 
     /**
      * ref id counter
@@ -181,28 +181,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;
                         }
 
                     $this->customXPath($path, $newGroupFlag, $value['value']);
@@ -214,12 +214,12 @@ 
             } else {
                 if (!empty($attributeXPath)) {
                     $path = $mapping . $attributeXPath;
-                    $this->customXPath($path, true, '', true);
+                    $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++;
@@ -259,13 +259,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
@@ -277,7 +277,7 @@ 
                     $path = $explodedXPath[0];
                 }
 
-                $praedicateFlag = true;
+                $praedicateFlag = TRUE;
             } else {
                 $path = $newPath[0];
             }
@@ -314,10 +314,10 @@ 
                 // FIXME: XPATHXmlGenerator XPATH does not generate any namespaces,
                 // which DOMDocument cannot cope with. Actually, namespaces should not be necessary here,
                 // since it is about child elements that are then added to the overall XML.
-                libxml_use_internal_errors(true);
+                libxml_use_internal_errors(TRUE);
                 $docXML = new \DOMDocument();
                 $docXML->loadXML($xml);
-                libxml_use_internal_errors(false);
+                libxml_use_internal_errors(FALSE);
 
                 $domXPath = \EWW\Dpf\Helper\XPath::create($this->xmlData);
 
@@ -328,17 +328,17 @@ 
                     // FIXME: XPATHXmlGenerator XPATH does not generate any namespaces,
                     // which DOMDocument cannot cope with. Actually, namespaces should not be necessary here,
                     // since it is about child elements that are then added to the overall XML.
-                    libxml_use_internal_errors(true);
+                    libxml_use_internal_errors(TRUE);
                     $docXMLNested = new \DOMDocument();
                     $docXMLNested->loadXML($nestedXml);
-                    libxml_use_internal_errors(false);
+                    libxml_use_internal_errors(FALSE);
 
                     $xPath = \EWW\Dpf\Helper\XPath::create($docXML);
 
                     $nodeList = $xPath->query($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);
                 }
@@ -346,7 +346,7 @@ 
                 $domNode = $domXPath->query('/data/' . $path);
                 $node = $docXML->documentElement;
 
-                $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
@@ -395,14 +395,14 @@ 
                     $nodeList = $xPath->query('//' . $path . $match[1]);
                     $node = $nodeList->item(0);
 
-                    $importNode = $doc2->importNode($docXMLNested->documentElement, true);
+                    $importNode = $doc2->importNode($docXMLNested->documentElement, TRUE);
 
                     $node->appendChild($importNode);
                 }
                 $domNode2 = $domXPath2->query('//' . $path)->item(0)->childNodes->item(0);
 
                 // merge xml nodes
-                $nodeToBeAppended = $doc1->importNode($domNode2, true);
+                $nodeToBeAppended = $doc1->importNode($domNode2, TRUE);
                 $domNode->item(0)->appendChild($nodeToBeAppended);
 
                 // add to modsData (merge not required)
@@ -416,7 +416,7 @@ 
                 }
                 //$firstChild = $this->xmlData->childNodes->item(0);
                 $firstItem = $doc1->documentElement->firstChild;
-                $nodeAppendModsData = $this->xmlData->importNode($firstItem, true);
+                $nodeAppendModsData = $this->xmlData->importNode($firstItem, TRUE);
                 $firstChild->appendChild($nodeAppendModsData);
 
                 return $doc1->saveXML();
@@ -428,17 +428,17 @@ 
             // FIXME: XPATHXmlGenerator XPATH does not generate any namespaces,
             // which DOMDocument cannot cope with. Actually, namespaces should not be necessary here,
             // since it is about child elements that are then added to the overall XML.
-            libxml_use_internal_errors(true);
+            libxml_use_internal_errors(TRUE);
             $docXML = new \DOMDocument();
             $docXML->loadXML($xml);
-            libxml_use_internal_errors(false);
+            libxml_use_internal_errors(FALSE);
 
             $domXPath = \EWW\Dpf\Helper\XPath::create($this->xmlData);
             $domNode  = $domXPath->query('/data');
 
             $node = $docXML->documentElement;
 
-            $nodeAppendModsData = $this->xmlData->importNode($node, true);
+            $nodeAppendModsData = $this->xmlData->importNode($node, TRUE);
             $domNode->item($domNode->length - 1)->appendChild($nodeAppendModsData);
 
             return $docXML->saveXML();
@@ -475,14 +475,14 @@ 
         foreach ($this->files as $key => $fileGrp) {
             foreach ($fileGrp as $file) {
 
-                $this->customXPath($fileXpathConfiguration . '/href', true, $file["path"]);
-                $this->customXPath($fileXpathConfiguration . '%mimetype', false, $file["type"]);
-                $this->customXPath($fileXpathConfiguration . '%title', false, $file["title"]);
-                $this->customXPath($fileXpathConfiguration . '%download', false, $file["download"]);
-                $this->customXPath($fileXpathConfiguration . '%archive', false, $file["archive"]);
-                $this->customXPath($fileXpathConfiguration . '%use', false, $file["use"]);
-                $this->customXPath($fileXpathConfiguration . '%id', false, $file["id"]);
-                $this->customXPath($fileXpathConfiguration . '%hasFLocat', false, $file["hasFLocat"]);
+                $this->customXPath($fileXpathConfiguration . '/href', TRUE, $file["path"]);
+                $this->customXPath($fileXpathConfiguration . '%mimetype', FALSE, $file["type"]);
+                $this->customXPath($fileXpathConfiguration . '%title', FALSE, $file["title"]);
+                $this->customXPath($fileXpathConfiguration . '%download', FALSE, $file["download"]);
+                $this->customXPath($fileXpathConfiguration . '%archive', FALSE, $file["archive"]);
+                $this->customXPath($fileXpathConfiguration . '%use', FALSE, $file["use"]);
+                $this->customXPath($fileXpathConfiguration . '%id', FALSE, $file["id"]);
+                $this->customXPath($fileXpathConfiguration . '%hasFLocat', FALSE, $file["hasFLocat"]);
 
             }
         }

Indentation +2 added lines, -2 removed lines

@@ -107,7 +107,7 @@ 
                                 $namespace = explode("=", $value);
                                 $this->namespaceString .= ' xmlns:' . $namespace[0] . '="' . $namespace[1] . '"';
                             }
-         }
+            }
 
         $this->xmlHeader = '<data' . $this->namespaceString . '></data>';
 
@@ -386,7 +386,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 +5 added lines, -5 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
@@ -306,7 +306,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]);
 
@@ -376,7 +376,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]);
                 }