kitodo / kitodo-publication

Classes/Services/Document/EditingLockService.php

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
 
     public function getLockedDocumentIdentifiersByUserUid($userUid)
     {
-        $identifiers = array();
+        $identifiers = array ();
 
         $locks = $this->editingLockRepository->findByEditorUid($userUid);
         /** @var  \EWW\Dpf\Domain\Model\EditingLock $lock */

Classes/Domain/Model/FrontendUser.php

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
     public function getUserRole()
     {
         // Get frontend user groups of the client.
-        $clientFrontendGroups = array();
+        $clientFrontendGroups = array ();
         foreach ($this->frontendUserGroupRepository->findAll() as $clientGroup) {
             if ($clientGroup->getKitodoRole()) {
                 $clientFrontendGroups[$clientGroup->getUid()] = $clientGroup;
@@ -46,7 +46,7 @@ 
         }
 
         // Get frontend user groups of the user.
-        $frontendUserGroups = array();
+        $frontendUserGroups = array ();
         foreach ($this->getUsergroup() as $userGroup) {
             // Because getUsergroup() does not return objects of the class
             // \EWW\Dpf\Domain\Model\FrontendUserRepository
@@ -55,7 +55,7 @@ 
         }
 
         // Get the roles the user has in the current client.
-        $roles = array();
+        $roles = array ();
         foreach ($frontendUserGroups as $uid => $group) {
             if (array_key_exists($uid, $clientFrontendGroups)) {
                 $roles[$uid] = $group->getKitodoRole();

Classes/Controller/AbstractSearchController.php

Spacing +19 added lines, -19 removed lines

@@ -54,7 +54,7 @@ 
 
         $searchString = $this->escapeQuery(trim($searchString));
 
-        $query['body']['query']['bool']['should'][0]['query_string']['query']                       = $searchString;
+        $query['body']['query']['bool']['should'][0]['query_string']['query'] = $searchString;
         //$query['body']['query']['bool']['should'][1]['has_child']['query']['query_string']['query'] = $searchString;
         $query['body']['query']['bool']['minimum_should_match'] = "1"; // 1
 
@@ -71,21 +71,21 @@ 
      * build array for elasticsearch
      * @return array Elasticsearch query array
      */
-    public function extendedSearch($searchArray = array())
+    public function extendedSearch($searchArray = array ())
     {
 
-        $query  = array();
-        $filter = array();
+        $query  = array ();
+        $filter = array ();
         foreach ($searchArray as $key => $qry) {
             $qry = trim($qry);
 
             if (!empty($qry) && in_array($key, self::$matches)) {
 
-                $query['body']['query']['bool']['must'][] = array('match' => array($key => $qry));
+                $query['body']['query']['bool']['must'][] = array ('match' => array ($key => $qry));
 
             } elseif (!empty($qry) && in_array($key, self::$terms)) {
 
-                $query['body']['query']['bool']['must'][] = array('term' => array($key => $qry));
+                $query['body']['query']['bool']['must'][] = array ('term' => array ($key => $qry));
 
             } elseif (!empty($qry) && $key == 'from') {
 
@@ -104,7 +104,7 @@ 
 
         if (isset($filter['gte']) || isset($filter['lte'])) {
 
-            $query['body']['query']['bool']['must'][] = array('range' => array('distribution_date' => $filter));
+            $query['body']['query']['bool']['must'][] = array ('range' => array ('distribution_date' => $filter));
 
         }
 
@@ -122,19 +122,19 @@ 
     public function resultsFilter($query, $showDeleted = false)
     {
 
-        $queryFilter = array();
+        $queryFilter = array ();
 
         // Frontend only
         $searchResultsFilter = $this->settings['searchResultsFilter'];
-        if(!empty($searchResultsFilter)) {
+        if (!empty($searchResultsFilter)) {
 
             // add doctypes
-            if($searchResultsFilter['doctype']) {
+            if ($searchResultsFilter['doctype']) {
 
                 $uids = GeneralUtility::trimExplode(',', $searchResultsFilter['doctype']);
                 $documentTypeRepository = $this->documentTypeRepository;
-                $documentTypes = array();
-                foreach($uids as $uid) {
+                $documentTypes = array ();
+                foreach ($uids as $uid) {
                     $documentType = $documentTypeRepository->findByUid($uid);
                     $documentTypes[] = $documentType->getName();
                 };
@@ -142,7 +142,7 @@ 
             }
 
             // add date filter
-            $dateFilter = array();
+            $dateFilter = array ();
             if ($searchResultsFilter['from']) {
 
                 $from     = date('d.m.Y', $searchResultsFilter['from']);
@@ -154,7 +154,7 @@ 
 
             if ($searchResultsFilter['till']) {
 
-                $till          = date('d.m.Y', $searchResultsFilter['till']);
+                $till = date('d.m.Y', $searchResultsFilter['till']);
                 $dateTime = $this->convertFormDate($till, true);
                 $dateFilter['lte'] = $dateTime->format('Y-m-d');
                 unset($searchResultsFilter['till']);
@@ -163,14 +163,14 @@ 
 
             if (isset($dateFilter['gte']) || isset($dateFilter['lte'])) {
 
-                $queryFilter['body']['query']['bool']['must'][] = array('range' => array('distribution_date' => $dateFilter));
+                $queryFilter['body']['query']['bool']['must'][] = array ('range' => array ('distribution_date' => $dateFilter));
 
             }
 
             foreach ($searchResultsFilter as $key => $qry) {
 
-                if(!empty($qry)) {
-                    $queryFilter['body']['query']['bool']['must'][] = array('match' => array($key => $qry));
+                if (!empty($qry)) {
+                    $queryFilter['body']['query']['bool']['must'][] = array ('match' => array ($key => $qry));
                 }
 
             }
@@ -178,7 +178,7 @@ 
         }
 
         // document must be active
-        if($showDeleted == false) {
+        if ($showDeleted == false) {
 
             //  $queryFilter['body']['query']['bool']['must'][]['term']['STATE'] = 'A';
 
@@ -236,7 +236,7 @@ 
         $luceneReservedCharacters = preg_quote('+-&|!(){}[]^"~?:\\');
         $string                   = preg_replace_callback(
             '/([' . $luceneReservedCharacters . '])/',
-            function ($matches) {
+            function($matches) {
                 return '\\' . $matches[0];
             },
             $string

class.ext_update.php

Spacing +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
 
     public function access() {
         $registry = GeneralUtility::makeInstance(Registry::class);
-        $version = $registry->get('tx_dpf','updatescript-'.self::VERSION);
+        $version = $registry->get('tx_dpf', 'updatescript-' . self::VERSION);
 
         // If the version has already been registered in the table sys_register the updatscript will be blocked.
         if ($version) {
@@ -43,7 +43,7 @@ 
     public function main() {
         // This script registers itself into the sys_registry table to prevent a re-run with the same version number.
         $registry = GeneralUtility::makeInstance(Registry::class);
-        $version = $registry->get('tx_dpf','updatescript-'.self::VERSION);
+        $version = $registry->get('tx_dpf', 'updatescript-' . self::VERSION);
         if ($version) {
             return FALSE;
         } else {
@@ -55,10 +55,10 @@ 
                 (new UpdateVirtualType)->execute();
                 //$GLOBALS['TYPO3_DB']->sql_query("update tx_dpf_domain_model_document set creator = owner");
             } catch (\Throwable $throwable) {
-                return "Error while updating the extension: ".($throwable->getMessage());
+                return "Error while updating the extension: " . ($throwable->getMessage());
             }
 
-            $registry->set('tx_dpf','updatescript-'.self::VERSION,TRUE);
+            $registry->set('tx_dpf', 'updatescript-' . self::VERSION, TRUE);
         }
 
         return "The extension has been successfully updated.";
@@ -123,7 +123,7 @@ 
             foreach ($repository->crossClientFindAll() as $record) {
                 if ($record['backend_only']) {
                     $recordObject = $repository->findByUid($record['uid']);
-                    $recordObject->setAccessRestrictionRoles(array(Security::ROLE_LIBRARIAN, Security::ROLE_RESEARCHER));
+                    $recordObject->setAccessRestrictionRoles(array (Security::ROLE_LIBRARIAN, Security::ROLE_RESEARCHER));
                     $repository->update($recordObject);
                 }
             }

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',

Classes/ViewHelpers/ShowStatusViewHelper.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
         }
 
         return \TYPO3\CMS\Extbase\Utility\LocalizationUtility::translate(
-            "manager.documentList.state.".$aliasState,
+            "manager.documentList.state." . $aliasState,
             'dpf',
             $arguments = null
         );

Classes/Controller/DocumentController.php

Spacing +8 added lines, -8 removed lines

@@ -402,7 +402,7 @@ 
                             // check if new document form has more field items as the existing form
                             if ($valueFieldCounter < $fieldCounter && !$checkFieldsForAdding) {
                                 // field added
-                                for ($i = count($valueField); $i < $fieldCounter;$i++) {
+                                for ($i = count($valueField); $i < $fieldCounter; $i++) {
                                     $returnArray['added'][] = $docForm2->getItems()[$keyPage][$keyRepeatPage]->getItems()[$keyGroup][$keyRepeatGroup]->getItems()[$keyField][$i];
 
                                 }
@@ -414,7 +414,7 @@ 
                     if ($valueGroupCounter < count($docForm2->getItems()[$keyPage][$keyRepeatPage]->getItems()[$keyGroup])) {
                         // group added
                         $counter = count($docForm2->getItems()[$keyPage][$keyRepeatPage]->getItems()[$keyGroup]);
-                        for ($i = $valueGroupCounter; $i < $counter;$i++) {
+                        for ($i = $valueGroupCounter; $i < $counter; $i++) {
                             $returnArray['added'][] = $docForm2->getItems()[$keyPage][$keyRepeatPage]->getItems()[$keyGroup][$i];
                         }
                     }
@@ -663,7 +663,7 @@ 
 
         $newDocument->setState(DocumentWorkflow::STATE_NEW_NONE);
 
-        $copyTitle = LocalizationUtility::translate("manager.workspace.title.copy", "dpf").$document->getTitle();
+        $copyTitle = LocalizationUtility::translate("manager.workspace.title.copy", "dpf") . $document->getTitle();
 
         $newDocument->setTitle($copyTitle);
 
@@ -804,7 +804,7 @@ 
         }
 
         $this->editingLockService->lock(
-            ($document->getObjectIdentifier()? $document->getObjectIdentifier() : $document->getUid()),
+            ($document->getObjectIdentifier() ? $document->getObjectIdentifier() : $document->getUid()),
             $this->security->getUser()->getUid()
         );
 
@@ -871,7 +871,7 @@ 
         $documentForm = $documentMapper->getDocumentForm($document);
 
         $this->view->assign('suggestMod', true);
-        $this->forward('edit','DocumentFormBackoffice',NULL, ['documentForm' => $documentForm, 'suggestMod' => true]);
+        $this->forward('edit', 'DocumentFormBackoffice', NULL, ['documentForm' => $documentForm, 'suggestMod' => true]);
     }
 
 
@@ -1008,7 +1008,7 @@ 
 
             if ($this->documentManager->update($document, $workflowTransition)) {
 
-                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:'.$messageKeyPart.'.success';
+                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:' . $messageKeyPart . '.success';
                 $this->flashMessage($document, $key, AbstractMessage::OK);
 
                 if ($this->security->getUserRole() === Security::ROLE_LIBRARIAN) {
@@ -1037,7 +1037,7 @@ 
 
                 $this->redirectToDocumentList();
             } else {
-                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:'.$messageKeyPart.'.failure';
+                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:' . $messageKeyPart . '.failure';
                 $this->flashMessage($document, $key, AbstractMessage::ERROR);
                 $this->redirect('showDetails', 'Document', NULL, ['document' => $document]);
             }
@@ -1048,7 +1048,7 @@ 
             if ($exception instanceof DPFExceptionInterface) {
                 $key = $exception->messageLanguageKey();
             } else {
-                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:'.$messageKeyPart.'.failure';
+                $key = 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:' . $messageKeyPart . '.failure';
             }
             $this->flashMessage($document, $key, AbstractMessage::ERROR);
             $this->redirectToDocumentList();

Classes/Controller/DocumentFormBackofficeController.php

Spacing +11 added lines, -11 removed lines

@@ -92,7 +92,7 @@ 
     }
 
     public function arrayRecursiveDiff($aArray1, $aArray2) {
-        $aReturn = array();
+        $aReturn = array ();
 
         foreach ($aArray1 as $mKey => $mValue) {
             if (array_key_exists($mKey, $aArray2)) {
@@ -146,13 +146,13 @@ 
                 $message = LocalizationUtility::translate(
                     'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_edit.accessDenied',
                     'dpf',
-                    array($document->getTitle())
+                    array ($document->getTitle())
                 );
             } else {
                 $message = LocalizationUtility::translate(
                     'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_edit.failureBlocked',
                     'dpf',
-                    array($document->getTitle())
+                    array ($document->getTitle())
                 );
             }
 
@@ -165,7 +165,7 @@ 
         $this->view->assign('suggestMod', $suggestMod);
 
         $this->editingLockService->lock(
-            ($document->getObjectIdentifier()? $document->getObjectIdentifier() : $document->getUid()),
+            ($document->getObjectIdentifier() ? $document->getObjectIdentifier() : $document->getUid()),
             $this->security->getUser()->getUid()
         );
 
@@ -286,7 +286,7 @@ 
 
         } catch (\Throwable $t) {
             $severity = \TYPO3\CMS\Core\Messaging\AbstractMessage::ERROR;
-            $this->addFlashMessage("Failed", '', $severity,false);
+            $this->addFlashMessage("Failed", '', $severity, false);
         }
 
         $this->redirectToDocumentList();
@@ -348,7 +348,7 @@ 
                 $message = LocalizationUtility::translate(
                     'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_update.accessDenied',
                     'dpf',
-                    array($document->getTitle())
+                    array ($document->getTitle())
                 );
                 $this->addFlashMessage($message, '', AbstractMessage::ERROR);
 
@@ -376,7 +376,7 @@ 
             $workflowTransition = null;
 
             // Convert the temporary copy into a local working copy if needed.
-            if ( $updateDocument->isTemporaryCopy() && $saveMode == 'saveWorkingCopy') {
+            if ($updateDocument->isTemporaryCopy() && $saveMode == 'saveWorkingCopy') {
                 $saveWorkingCopy = true;
                 $updateDocument->setTemporary(false);
                 $workflowTransition = DocumentWorkflow::TRANSITION_IN_PROGRESS;
@@ -428,7 +428,7 @@ 
                 $message = LocalizationUtility::translate(
                     'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_update.success',
                     'dpf',
-                    array($updateDocument->getTitle())
+                    array ($updateDocument->getTitle())
                 );
                 $this->addFlashMessage($message, '', AbstractMessage::OK);
 
@@ -482,7 +482,7 @@ 
                 $message = LocalizationUtility::translate(
                     'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_update.failure',
                     'dpf',
-                    array($updateDocument->getTitle())
+                    array ($updateDocument->getTitle())
                 );
                 $this->addFlashMessage($message, '', AbstractMessage::ERROR);
             }
@@ -516,7 +516,7 @@ 
             $exceptionMsg[] = LocalizationUtility::translate(
                 'LLL:EXT:dpf/Resources/Private/Language/locallang.xlf:document_update.failure',
                 'dpf',
-                array($updateDocument->getTitle())
+                array ($updateDocument->getTitle())
             );
 
             $exceptionMsg[] = LocalizationUtility::translate($key, 'dpf');
@@ -645,7 +645,7 @@ 
         if ($redirectUri) {
             $this->redirectToUri($redirectUri);
         } else {
-            $this->redirect($action, $controller, null, array('message' => $message));
+            $this->redirect($action, $controller, null, array ('message' => $message));
         }
     }
 }

Classes/Controller/UserController.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
                 $this->addFlashMessage(
                     LocalizationUtility::translate("manager.locallang.user.settings.message.invalidFisId", "dpf"),
                     '',
-                    $severity,false
+                    $severity, false
                 );
             }
         }
@@ -61,7 +61,7 @@ 
         $this->addFlashMessage(
             LocalizationUtility::translate("manager.locallang.user.settings.message.successfullySaved", "dpf"),
             '',
-            $severity,false
+            $severity, false
         );
 
         $this->forward('settings');