kitodo / kitodo-publication

Classes/Controller/DocumentFormBEController.php

Braces +6 added lines, -12 removed lines

@@ -16,17 +16,14 @@ 
 
 use EWW\Dpf\Services\Transfer\ElasticsearchRepository;
 
-class DocumentFormBEController extends AbstractDocumentFormController
-{
+class DocumentFormBEController extends AbstractDocumentFormController {
 
-    public function __construct()
-    {
+    public function __construct() {
         parent::__construct();
 
     }
 
-    protected function redirectToList($message = null)
-    {
+    protected function redirectToList($message = null) {
         $this->redirect('list', 'Document', null, array('message' => $message));
     }
 
@@ -36,8 +33,7 @@ 
      * @param array $documentData
      * @throws \Exception
      */
-    public function deleteAction($documentData)
-    {
+    public function deleteAction($documentData) {
 
         if (!$GLOBALS['BE_USER']) {
             throw new \Exception('Access denied');
@@ -55,8 +51,7 @@ 
         $this->redirectToList();
     }
 
-    public function editAction(\EWW\Dpf\Domain\Model\DocumentForm $documentForm)
-    {
+    public function editAction(\EWW\Dpf\Domain\Model\DocumentForm $documentForm) {
 
         $document = $this->documentRepository->findByUid($documentForm->getDocumentUid());
         $this->view->assign('document', $document);
@@ -64,8 +59,7 @@ 
     }
 
 
-    public function createAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm)
-    {
+    public function createAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm) {
         parent::createAction($newDocumentForm);
         $this->redirectToList('CREATE_OK');
     }

Classes/Controller/DocumentFormController.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 
     protected function redirectToList($message = null)
     {
-        $this->redirect('list', 'DocumentForm', null, array('message' => $message));
+        $this->redirect('list', 'DocumentForm', null, array ('message' => $message));
     }
 
     /**
@@ -42,7 +42,7 @@ 
 
         parent::createAction($newDocumentForm);
 
-        if (key_exists('afterDocSavedRedirectPage',$this->settings) && $this->settings['afterDocSavedRedirectPage']) {
+        if (key_exists('afterDocSavedRedirectPage', $this->settings) && $this->settings['afterDocSavedRedirectPage']) {
             $uri = $this->uriBuilder
                 ->setTargetPageUid($this->settings['afterDocSavedRedirectPage'])
                 ->build();

Classes/Controller/AbstractDocumentFormController.php

Spacing +9 added lines, -9 removed lines

@@ -87,13 +87,13 @@ 
 
         $documentTypes = $this->documentTypeRepository->findAll();
 
-        $data = array();
-        $docTypes = array();
-        $name = array();
-        $type = array();
+        $data = array ();
+        $docTypes = array ();
+        $name = array ();
+        $type = array ();
 
         foreach ($documentTypes as $docType) {
-            $data[] = array(
+            $data[] = array (
                 "name" => $docType->getDisplayName(),
                 "type" => $docType,
             );
@@ -197,7 +197,7 @@ 
                 $this->request->setArguments($requestArguments);
             } else {
                 $t = $docForm->getNewFileNames();
-                $this->redirect('list', 'DocumentForm', null, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
+                $this->redirect('list', 'DocumentForm', null, array ('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
             }
         } else {
             $this->redirectToList("UPLOAD_POST_SIZE_ERROR");
@@ -253,7 +253,7 @@ 
             $tmpDocument->setSlubInfoData($newDocument->getSlubInfoData());
             $tmpDocument->setDocumentType($newDocument->getDocumentType());
 
-            $this->forward('new', null, null, array('newDocumentForm' => $documentMapper->getDocumentForm($tmpDocument)));
+            $this->forward('new', null, null, array ('newDocumentForm' => $documentMapper->getDocumentForm($tmpDocument)));
         }
 
     }
@@ -309,7 +309,7 @@ 
                 $this->request->setArguments($requestArguments);
             } else {
                 $t = $docForm->getNewFileNames();
-                $this->redirect('list', 'Document', null, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
+                $this->redirect('list', 'Document', null, array ('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
             }
         } else {
             $this->redirectToList("UPLOAD_POST_SIZE_ERROR");
@@ -362,7 +362,7 @@ 
         $elasticsearchRepository->add($updateDocument, $json);
 
         if (array_key_exists('savecontinue', $requestArguments)) {
-            $this->forward('edit', null, null, array('documentForm' => $documentForm));
+            $this->forward('edit', null, null, array ('documentForm' => $documentForm));
         }
 
         $this->redirectToList();

Braces +14 added lines, -28 removed lines

@@ -24,8 +24,7 @@ 
 /**
  * DocumentFormController
  */
-abstract class AbstractDocumentFormController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionController
-{
+abstract class AbstractDocumentFormController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionController {
 
     /**
      * documentRepository
@@ -80,8 +79,7 @@ 
      *
      * @return void
      */
-    public function listAction()
-    {
+    public function listAction() {
 
         $documents = $this->documentRepository->findAll();
 
@@ -130,8 +128,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);
     }
@@ -141,8 +138,7 @@ 
      *
      * @return void
      */
-    public function initializeNewAction()
-    {
+    public function initializeNewAction() {
 
         $requestArguments = $this->request->getArguments();
 
@@ -170,13 +166,11 @@ 
      * @ignorevalidation $newDocumentForm
      * @return void
      */
-    public function newAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm = null)
-    {
+    public function newAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm = null) {
         $this->view->assign('documentForm', $newDocumentForm);
     }
 
-    public function initializeCreateAction()
-    {
+    public function initializeCreateAction() {
 
         $requestArguments = $this->request->getArguments();
 
@@ -210,8 +204,7 @@ 
      * @param \EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm
      * @return void
      */
-    public function createAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm)
-    {
+    public function createAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm) {
 
         $documentMapper = $this->objectManager->get(DocumentMapper::class);
         $newDocument    = $documentMapper->getDocument($newDocumentForm);
@@ -258,8 +251,7 @@ 
 
     }
 
-    public function initializeEditAction()
-    {
+    public function initializeEditAction() {
 
         $requestArguments = $this->request->getArguments();
 
@@ -283,13 +275,11 @@ 
      * @ignorevalidation $documentForm
      * @return void
      */
-    public function editAction(\EWW\Dpf\Domain\Model\DocumentForm $documentForm)
-    {
+    public function editAction(\EWW\Dpf\Domain\Model\DocumentForm $documentForm) {
         $this->view->assign('documentForm', $documentForm);
     }
 
-    public function initializeUpdateAction()
-    {
+    public function initializeUpdateAction() {
         $requestArguments = $this->request->getArguments();
 
         if ($this->request->hasArgument('documentData')) {
@@ -322,8 +312,7 @@ 
      * @param \EWW\Dpf\Domain\Model\DocumentForm $documentForm
      * @return void
      */
-    public function updateAction(\EWW\Dpf\Domain\Model\DocumentForm $documentForm)
-    {
+    public function updateAction(\EWW\Dpf\Domain\Model\DocumentForm $documentForm) {
 
         $requestArguments = $this->request->getArguments();
 
@@ -373,18 +362,15 @@ 
      *
      * @return void
      */
-    public function cancelAction()
-    {
+    public function cancelAction() {
         $this->redirectToList();
     }
 
-    public function initializeAction()
-    {
+    public function initializeAction() {
         parent::initializeAction();
     }
 
-    protected function redirectToList($message = null)
-    {
+    protected function redirectToList($message = null) {
         $this->redirect('list');
     }
 

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

@@ -33,7 +33,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DocumentRepository
      * @inject
      */
-    protected $documentRepository = null;
+    protected $documentRepository = NULL;
 
     /**
      * fileRepository
@@ -41,7 +41,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\FileRepository
      * @inject
      */
-    protected $fileRepository = null;
+    protected $fileRepository = NULL;
 
     /**
      * documentTypeRepository
@@ -49,7 +49,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DocumentTypeRepository
      * @inject
      */
-    protected $documentTypeRepository = null;
+    protected $documentTypeRepository = NULL;
 
     /**
      * metadataGroupRepository
@@ -57,7 +57,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\MetadataGroupRepository
      * @inject
      */
-    protected $metadataGroupRepository = null;
+    protected $metadataGroupRepository = NULL;
 
     /**
      * metadataObjectRepository
@@ -65,7 +65,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\MetadataObjectRepository
      * @inject
      */
-    protected $metadataObjectRepository = null;
+    protected $metadataObjectRepository = NULL;
 
     /**
      * persistence manager
@@ -170,7 +170,7 @@ 
      * @ignorevalidation $newDocumentForm
      * @return void
      */
-    public function newAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm = null)
+    public function newAction(\EWW\Dpf\Domain\Model\DocumentForm $newDocumentForm = NULL)
     {
         $this->view->assign('documentForm', $newDocumentForm);
     }
@@ -193,11 +193,11 @@ 
             $documentType = $this->documentTypeRepository->findByUid($docTypeUid);
             $virtual = $documentType->getVirtual();
 
-            if (!$formDataReader->uploadError() || $virtual === true) {
+            if (!$formDataReader->uploadError() || $virtual === TRUE) {
                 $this->request->setArguments($requestArguments);
             } else {
                 $t = $docForm->getNewFileNames();
-                $this->redirect('list', 'DocumentForm', null, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
+                $this->redirect('list', 'DocumentForm', NULL, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
             }
         } else {
             $this->redirectToList("UPLOAD_POST_SIZE_ERROR");
@@ -253,7 +253,7 @@ 
             $tmpDocument->setSlubInfoData($newDocument->getSlubInfoData());
             $tmpDocument->setDocumentType($newDocument->getDocumentType());
 
-            $this->forward('new', null, null, array('newDocumentForm' => $documentMapper->getDocumentForm($tmpDocument)));
+            $this->forward('new', NULL, NULL, array('newDocumentForm' => $documentMapper->getDocumentForm($tmpDocument)));
         }
 
     }
@@ -305,11 +305,11 @@ 
             $documentType = $this->documentTypeRepository->findByUid($docTypeUid);
             $virtual = $documentType->getVirtual();
 
-            if (!$formDataReader->uploadError() || $virtual === true) {
+            if (!$formDataReader->uploadError() || $virtual === TRUE) {
                 $this->request->setArguments($requestArguments);
             } else {
                 $t = $docForm->getNewFileNames();
-                $this->redirect('list', 'Document', null, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
+                $this->redirect('list', 'Document', NULL, array('message' => 'UPLOAD_MAX_FILESIZE_ERROR', 'errorFiles' => $t));
             }
         } else {
             $this->redirectToList("UPLOAD_POST_SIZE_ERROR");
@@ -332,7 +332,7 @@ 
 
         $objectIdentifier = $updateDocument->getObjectIdentifier();
 
-        $updateDocument->setChanged(true);
+        $updateDocument->setChanged(TRUE);
 
         $this->documentRepository->update($updateDocument);
 
@@ -362,7 +362,7 @@ 
         $elasticsearchRepository->add($updateDocument, $json);
 
         if (array_key_exists('savecontinue', $requestArguments)) {
-            $this->forward('edit', null, null, array('documentForm' => $documentForm));
+            $this->forward('edit', NULL, NULL, array('documentForm' => $documentForm));
         }
 
         $this->redirectToList();
@@ -383,7 +383,7 @@ 
         parent::initializeAction();
     }
 
-    protected function redirectToList($message = null)
+    protected function redirectToList($message = NULL)
     {
         $this->redirect('list');
     }