kitodo / kitodo-publication

Classes/Domain/Model/DocumentForm.php

Braces +29 added lines, -58 removed lines

@@ -20,8 +20,7 @@ 
  * The TYPO3 project - inspiring people to share!
  */
 
-class DocumentForm extends AbstractFormElement
-{
+class DocumentForm extends AbstractFormElement {
 
     /**
      * @var string CSRF token for this form
@@ -103,8 +102,7 @@ 
      *
      * @param string $csrfToken
      */
-    public function generateCsrfToken()
-    {
+    public function generateCsrfToken() {
         $formProtection = FormProtectionFactory::get();
         $this->csrfToken = $formProtection->generateToken('DocumentForm', 'construct', 'DocumentForm');
         $formProtection->persistSessionToken();
@@ -119,10 +117,8 @@ 
      * @throws Exception if the given string is empty.
      * @throws TypeError if the given string is null
      */
-    public function setCsrfToken(string $csrfToken)
-    {
-        if ($csrfToken === "")
-        {
+    public function setCsrfToken(string $csrfToken) {
+        if ($csrfToken === "") {
             throw new Exception("A forms CSRF token cannot be empty");
         }
         $this->csrfToken = $csrfToken;
@@ -134,8 +130,7 @@ 
      *
      * @return string CSRF token for this form
      */
-    public function getCsrfToken()
-    {
+    public function getCsrfToken() {
         return $this->csrfToken;
     }
 
@@ -145,8 +140,7 @@ 
      *
      * @return bool True, is CSRF token is considered valid. False if the token is invalid or missing.
      */
-    public function hasValidCsrfToken()
-    {
+    public function hasValidCsrfToken() {
         $formProtection = FormProtectionFactory::get();
         return $formProtection->validateToken($this->csrfToken, 'DocumentForm', 'construct', 'DocumentForm');
     }
@@ -155,8 +149,7 @@ 
      *
      * @return integer
      */
-    public function getDocumentUid()
-    {
+    public function getDocumentUid() {
         return $this->documentUid;
     }
 
@@ -164,8 +157,7 @@ 
      *
      * @param integer $documentUid
      */
-    public function setDocumentUid($documentUid)
-    {
+    public function setDocumentUid($documentUid) {
         $this->documentUid = $documentUid;
     }
 
@@ -173,8 +165,7 @@ 
      *
      * @return boolean
      */
-    public function getPrimaryFileMandatory()
-    {
+    public function getPrimaryFileMandatory() {
         return $this->primaryFileMandatory;
     }
 
@@ -182,8 +173,7 @@ 
      *
      * @param boolean $primaryFileMandatory
      */
-    public function setPrimaryFileMandatory($primaryFileMandatory)
-    {
+    public function setPrimaryFileMandatory($primaryFileMandatory) {
         $this->primaryFileMandatory = boolval($primaryFileMandatory);
     }
 
@@ -191,8 +181,7 @@ 
      *
      * @return string
      */
-    public function getFedoraPid()
-    {
+    public function getFedoraPid() {
         return $this->fedoraPid;
     }
 
@@ -200,8 +189,7 @@ 
      *
      * @param string $fedoraPid
      */
-    public function setFedoraPid($fedoraPid)
-    {
+    public function setFedoraPid($fedoraPid) {
         $this->fedoraPid = $fedoraPid;
     }
 
@@ -209,8 +197,7 @@ 
      *
      * @param type \EWW\Dpf\Domain\Model\File $primaryFile
      */
-    public function setPrimaryFile($primaryFile)
-    {
+    public function setPrimaryFile($primaryFile) {
         $this->primaryFile = $primaryFile;
     }
 
@@ -218,59 +205,49 @@ 
      *
      * @return \EWW\Dpf\Domain\Model\File
      */
-    public function getPrimaryFile()
-    {
+    public function getPrimaryFile() {
         return $this->primaryFile;
     }
 
-    public function setSecondaryFiles($secondaryFiles)
-    {
+    public function setSecondaryFiles($secondaryFiles) {
         $this->secondaryFiles = $secondaryFiles;
     }
 
-    public function getSecondaryFiles()
-    {
+    public function getSecondaryFiles() {
         return $this->secondaryFiles;
     }
 
-    public function getDeletedFiles()
-    {
+    public function getDeletedFiles() {
         return $this->deletedFiles;
     }
 
-    public function setDeletedFiles($deletedFiles)
-    {
+    public function setDeletedFiles($deletedFiles) {
         $this->deletedFiles = $deletedFiles;
     }
 
-    public function getNewFiles()
-    {
+    public function getNewFiles() {
         return $this->newFiles;
     }
 
-    public function setNewFiles($newFiles)
-    {
+    public function setNewFiles($newFiles) {
         $this->newFiles = $newFiles;
     }
 
     /**
      * @return bool
      */
-    public function getValid()
-    {
+    public function getValid() {
         return $this->valid;
     }
 
     /**
      * @param bool $valid
      */
-    public function setValid($valid)
-    {
+    public function setValid($valid) {
         $this->valid = boolval($valid);
     }
 
-    public function getNewFileNames()
-    {
+    public function getNewFileNames() {
         $fileNames = array();
         foreach ($this->getNewFiles() as $file) {
             $fileNames[] = $file->getTitle();
@@ -283,8 +260,7 @@ 
      *
      * @return string
      */
-    public function getProcessNumber()
-    {
+    public function getProcessNumber() {
         return $this->processNumber;
     }
 
@@ -293,8 +269,7 @@ 
      *
      * @param string $processNumber
      */
-    public function setProcessNumber($processNumber)
-    {
+    public function setProcessNumber($processNumber) {
         $this->processNumber = $processNumber;
     }
 
@@ -303,8 +278,7 @@ 
      *
      * @return bool
      */
-    public function isTemporary()
-    {
+    public function isTemporary() {
         return $this->temporary;
     }
 
@@ -312,24 +286,21 @@ 
      * Sets if a document is a temporary document or not.
      * @param bool $temporary
      */
-    public function setTemporary($temporary)
-    {
+    public function setTemporary($temporary) {
         $this->temporary = boolval($temporary);
     }
 
     /**
      * @return string
      */
-    public function getComment()
-    {
+    public function getComment() {
         return $this->comment;
     }
 
     /**
      * @param string $comment
      */
-    public function setComment($comment)
-    {
+    public function setComment($comment) {
         $this->comment = $comment;
     }
 }

Classes/Helper/DocumentMapper.php

Braces +5 added lines, -10 removed lines

@@ -20,8 +20,7 @@ 
 use EWW\Dpf\Domain\Workflow\DocumentWorkflow;
 use EWW\Dpf\Services\ProcessNumber\ProcessNumberGenerator;
 
-class DocumentMapper
-{
+class DocumentMapper {
 
     /**
      * objectManager
@@ -91,8 +90,7 @@ 
      *
      * @return mixed
      */
-    public function getSettings()
-    {
+    public function getSettings() {
         $frameworkConfiguration = $this->configurationManager->getConfiguration(
             \TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface::CONFIGURATION_TYPE_FRAMEWORK
         );
@@ -106,8 +104,7 @@ 
      * @param bool $generateEmptyFields
      * @return \EWW\Dpf\Domain\Model\DocumentForm
      */
-    public function getDocumentForm(Document $document, $generateEmptyFields = true)
-    {
+    public function getDocumentForm(Document $document, $generateEmptyFields = true) {
         $documentForm = new \EWW\Dpf\Domain\Model\DocumentForm();
         $documentForm->generateCsrfToken();
         $documentForm->setUid($document->getDocumentType()->getUid());
@@ -344,8 +341,7 @@ 
         return $documentForm;
     }
 
-    public function getDocument($documentForm)
-    {
+    public function getDocument($documentForm) {
         /** @var Document $document */
 
         if ($documentForm->getDocumentUid()) {
@@ -407,8 +403,7 @@ 
         return $document;
     }
 
-    public function getMetadata($documentForm)
-    {
+    public function getMetadata($documentForm) {
 
         foreach ($documentForm->getItems() as $page) {
 

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

@@ -44,7 +44,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\MetadataGroupRepository
      * @TYPO3\CMS\Extbase\Annotation\Inject
      */
-    protected $metadataGroupRepository = null;
+    protected $metadataGroupRepository = NULL;
 
     /**
      * metadataObjectRepository
@@ -52,7 +52,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\MetadataObjectRepository
      * @TYPO3\CMS\Extbase\Annotation\Inject
      */
-    protected $metadataObjectRepository = null;
+    protected $metadataObjectRepository = NULL;
 
     /**
      * documentTypeRepository
@@ -60,7 +60,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DocumentTypeRepository
      * @TYPO3\CMS\Extbase\Annotation\Inject
      */
-    protected $documentTypeRepository = null;
+    protected $documentTypeRepository = NULL;
 
     /**
      * documentRepository
@@ -68,7 +68,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DocumentRepository
      * @TYPO3\CMS\Extbase\Annotation\Inject
      */
-    protected $documentRepository = null;
+    protected $documentRepository = NULL;
 
     /**
      * fileRepository
@@ -76,7 +76,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\FileRepository
      * @TYPO3\CMS\Extbase\Annotation\Inject
      */
-    protected $fileRepository = null;
+    protected $fileRepository = NULL;
 
     /**
      * depositLicenseRepository
@@ -84,7 +84,7 @@ 
      * @var \EWW\Dpf\Domain\Repository\DepositLicenseRepository
      * @TYPO3\CMS\Extbase\Annotation\Inject
      */
-    protected $depositLicenseRepository = null;
+    protected $depositLicenseRepository = NULL;
 
     /**
      * Get typoscript settings
@@ -106,7 +106,7 @@ 
      * @param bool $generateEmptyFields
      * @return \EWW\Dpf\Domain\Model\DocumentForm
      */
-    public function getDocumentForm(Document $document, $generateEmptyFields = true)
+    public function getDocumentForm(Document $document, $generateEmptyFields = TRUE)
     {
         $documentForm = new \EWW\Dpf\Domain\Model\DocumentForm();
         $documentForm->generateCsrfToken();
@@ -120,7 +120,7 @@ 
                 (
                     $this->getSettings()['deactivatePrimaryFileMandatoryCheck'] ||
                     $document->getDocumentType()->getVirtualType()
-                )? false : true
+                )? FALSE : TRUE
             )
         );
 
@@ -183,7 +183,7 @@ 
                 // build mapping path, previous fixed attributes which are differ from
                 // the own fixed attributes are excluded
                 $queryGroupMapping = $metadataGroup->getAbsoluteMapping();
-                if (strpos($queryGroupMapping, "@displayLabel") === false && is_array($excludeGroupAttributes[$groupMappingName])) {
+                if (strpos($queryGroupMapping, "@displayLabel") === FALSE && is_array($excludeGroupAttributes[$groupMappingName])) {
                     foreach ($excludeGroupAttributes[$groupMappingName] as $excludeAttr => $excludeAttrValue) {
                         if (!in_array($excludeAttr, $fixedGroupAttributes)) {
                             $queryGroupMapping .= $excludeAttrValue;
@@ -241,7 +241,7 @@ 
 
                                 // ensure that e.g. <mods:detail> and <mods:detail type="volume">
                                 // are not recognized as the same node
-                                if ((strpos($value, "@") === false) && ($value != '.')) {
+                                if ((strpos($value, "@") === FALSE) && ($value != '.')) {
                                     $objectMappingPath[$key] .= "[not(@*)]";
                                 }
                             }
@@ -298,7 +298,7 @@ 
                     }
                 } else {
 
-                    $documentFormGroup->setEmptyGroup(true);
+                    $documentFormGroup->setEmptyGroup(TRUE);
 
                     foreach ($metadataGroup->getMetadataObject() as $metadataObject) {
                         $documentFormField = new \EWW\Dpf\Domain\Model\DocumentFormField();