kitodo / kitodo-publication

Classes/Services/ElasticSearch/QueryBuilder.php

Spacing +38 added lines, -38 removed lines

@@ -122,7 +122,7 @@ 
             $query = [
                 'bool' => [
                     'must' => [
-                        'match_all' => (object)[]
+                        'match_all' => (object) []
                     ],
                     'filter' => $queryFilter
                 ]
@@ -164,12 +164,12 @@ 
                             'script' => [
                                 'lang' => 'painless',
                                 'source' =>
-                                    "for (int i = 0; i < doc['collections'].length; ++i) {".
-                                    "    if(doc['collections'][i] =='".
-                                    $this->clientConfigurationManager->getUniversityCollection() ."') {".
-                                    "        return 'true';".
-                                    "    }".
-                                    "}".
+                                    "for (int i = 0; i < doc['collections'].length; ++i) {" .
+                                    "    if(doc['collections'][i] =='" .
+                                    $this->clientConfigurationManager->getUniversityCollection() . "') {" .
+                                    "        return 'true';" .
+                                    "    }" .
+                                    "}" .
                                     "return 'false';"
                             ]
                         ]
@@ -185,23 +185,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';"
                             ]
                         ]
@@ -414,8 +414,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 = "";
@@ -446,11 +446,11 @@ 
             ];
         } else {
             if ($sortField == 'title') {
-                $sortField.= ".keyword";
+                $sortField .= ".keyword";
             }
 
             if ($sortField == 'personsSort') {
-                $sortField.= ".keyword";
+                $sortField .= ".keyword";
             }
 
             $sort = [
@@ -467,7 +467,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 .= "}";
@@ -488,13 +488,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';";
@@ -509,15 +509,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 '';";
     }
 
 
@@ -526,14 +526,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 '';";
     }
 
 }

Classes/Domain/Repository/DocumentRepository.php

Spacing +10 added lines, -10 removed lines

@@ -46,7 +46,7 @@ 
             case Security::ROLE_RESEARCHER:
                 $query->matching(
                     $query->logicalAnd(
-                        array(
+                        array (
                             $query->equals('suggestion', true),
                             $query->equals('creator', $creatorUid)
                         )
@@ -65,7 +65,7 @@ 
     {
         $query = $this->createQuery();
 
-        $constraints = array(
+        $constraints = array (
             $query->logicalNot($query->equals('object_identifier', '')),
             $query->logicalNot($query->equals('object_identifier', null))
         );
@@ -84,7 +84,7 @@ 
 
         $result = $query->execute();
 
-        $objectIdentifiers = array();
+        $objectIdentifiers = array ();
 
         foreach ($result as $document) {
             $objectIdentifiers[$document->getObjectIdentifier()] = $document->getObjectIdentifier();
@@ -104,7 +104,7 @@ 
         $query = $this->createQuery();
         $query->getQuerySettings()->setRespectStoragePage(false);
 
-        $constraints = array();
+        $constraints = array ();
         $constraints[] = $query->equals('process_number', '');
         $constraints[] = $query->equals('process_number', null);
 
@@ -133,7 +133,7 @@ 
         $dateTimeObj = new \DateTime();
         $dateTimeObj->sub(new \DateInterval("PT" . $timeout . "S"));
 
-        $constraints = array();
+        $constraints = array ();
         $constraints[] = $query->lessThan('tstamp', $dateTimeObj->getTimestamp());
 
         $query->matching(
@@ -159,7 +159,7 @@ 
         $dateTimeObj = new \DateTime();
         $dateTimeObj->sub(new \DateInterval("PT" . $timeout . "S"));
 
-        $constraints = array();
+        $constraints = array ();
         $constraints[] = $query->lessThan('tstamp', $dateTimeObj->getTimestamp());
 
         $query->matching(
@@ -181,7 +181,7 @@ 
     {
         $query = $this->createQuery();
 
-        $constraints = array(
+        $constraints = array (
             $query->equals('object_identifier', $objectIdentifier),
             $query->logicalNot($query->equals('temporary', true)),
             $query->logicalNot($query->equals('suggestion', true))
@@ -222,7 +222,7 @@ 
             return null;
         }
 
-        $query->setOrderings(array("tstamp" => \TYPO3\CMS\Extbase\Persistence\QueryInterface::ORDER_DESCENDING));
+        $query->setOrderings(array ("tstamp" => \TYPO3\CMS\Extbase\Persistence\QueryInterface::ORDER_DESCENDING));
         $query->matching($query->logicalAnd($constraints));
 
         return $query->execute()->getFirst();
@@ -309,7 +309,7 @@ 
         $query = $this->createQuery();
         $query->matching(
             $query->logicalAnd(
-                array(
+                array (
                     $query->logicalNot(
                         $query->equals('embargo_date', 0)
                     )
@@ -347,7 +347,7 @@ 
                     $query->equals('suggestion', false)
                 );
 
-        $query->setOrderings(array("tstamp" => \TYPO3\CMS\Extbase\Persistence\QueryInterface::ORDER_DESCENDING));
+        $query->setOrderings(array ("tstamp" => \TYPO3\CMS\Extbase\Persistence\QueryInterface::ORDER_DESCENDING));
         $query->matching($query->logicalOr($constraints));
 
         return $query->execute();