kitodo / kitodo-publication

Classes/Services/Email/Notifier.php

Braces +6 added lines, -12 removed lines

@@ -18,8 +18,7 @@ 
 use \TYPO3\CMS\Core\Log\LogManager;
 use \TYPO3\CMS\Core\Utility\GeneralUtility;
 
-class Notifier
-{
+class Notifier {
 
     /**
      * clientRepository
@@ -39,8 +38,7 @@ 
     protected $documentTypeRepository = null;
 
 
-    public function sendNewDocumentNotification(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function sendNewDocumentNotification(\EWW\Dpf\Domain\Model\Document $document) {
 
         try {
             $client = $this->clientRepository->findAll()->current();
@@ -117,8 +115,7 @@ 
 
     }
 
-    public function sendIngestNotification(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function sendIngestNotification(\EWW\Dpf\Domain\Model\Document $document) {
 
         try {
             $client = $this->clientRepository->findAll()->current();
@@ -230,8 +227,7 @@ 
         }
     }
 
-    public function sendRegisterNotification(\EWW\Dpf\Domain\Model\Document $document)
-    {
+    public function sendRegisterNotification(\EWW\Dpf\Domain\Model\Document $document) {
 
         try {
             $client = $this->clientRepository->findAll()->current();
@@ -290,8 +286,7 @@ 
 
     }
 
-    protected function replaceMarkers($message, $args)
-    {
+    protected function replaceMarkers($message, $args) {
         if (is_array($args)) {
             foreach ($args as $key => $value) {
                 $message = str_replace($key, $value, $message);
@@ -301,8 +296,7 @@ 
     }
 
 
-    protected function sendMail($reveiver, $subject, $body, $args, $mailType)
-    {
+    protected function sendMail($reveiver, $subject, $body, $args, $mailType) {
         $emailReceiver = array();
         $emailReceiver[$reveiver] = $reveiver;
         $message = (new \TYPO3\CMS\Core\Mail\MailMessage())

Classes/Services/Transfer/FedoraRepository.php

Braces +10 added lines, -20 removed lines

@@ -22,8 +22,7 @@ 
 use \EWW\Dpf\Services\Logger\TransferLogger;
 use \Httpful\Request;
 
-class FedoraRepository implements Repository
-{
+class FedoraRepository implements Repository {
 
     /**
      * documentTransferLogRepository
@@ -64,8 +63,7 @@ 
      * @return string
      * @throws \Exception
      */
-    public function ingest($document, $metsXml)
-    {
+    public function ingest($document, $metsXml) {
         try {
 
             $response = Request::post($this->clientConfigurationManager->getSwordHost() . "/sword/" . $this->getSWORDCollection())
@@ -84,8 +82,7 @@ 
                 TransferLogger::Log('INGEST', $document->getUid(), null, $response);
                 throw new \EWW\Dpf\Exceptions\IngestDocumentErrorException("Fedora error while ingest document.");
             }
-        }
-        catch (\Exception $exception) {
+        } catch (\Exception $exception) {
             TransferLogger::Log('INGEST', $document->getUid(), null, $exception->getMessage());
 
             if ($exception instanceof \Httpful\Exception\ConnectionErrorException) {
@@ -107,8 +104,7 @@ 
      * @return string
      * @throws \Exception
      */
-    public function update($document, $metsXml)
-    {
+    public function update($document, $metsXml) {
 
         $remoteId = $document->getObjectIdentifier();
 
@@ -149,8 +145,7 @@ 
      * @return string
      * @throws \Exception
      */
-    public function retrieve($remoteId)
-    {
+    public function retrieve($remoteId) {
 
         try {
             $response = Request::get($this->clientConfigurationManager->getFedoraHost() . "/fedora/objects/" . $remoteId . "/methods/qucosa:SDef/getMETSDissemination")
@@ -186,8 +181,7 @@ 
      * @return string
      * @throws \Exception
      */
-    public function getNextDocumentId()
-    {
+    public function getNextDocumentId() {
         try {
             $response = Request::get($this->clientConfigurationManager->getFedoraHost() . "/fedora/management/getNextPID?numPIDs=1&namespace=qucosa&xml=true")
                 ->authenticateWith($this->clientConfigurationManager->getFedoraUser(), $this->clientConfigurationManager->getFedoraPassword())
@@ -224,8 +218,7 @@ 
      * @return boolean
      * @throws \Exception
      */
-    public function delete($document, $state)
-    {
+    public function delete($document, $state) {
 
         $remoteId = $document->getObjectIdentifier();
 
@@ -274,8 +267,7 @@ 
      * @param  \Httpful\Response $response
      * @return string
      */
-    protected function getRemoteDocumentId($response)
-    {
+    protected function getRemoteDocumentId($response) {
 
         // Get repository ID and write into document
         $responseDom = new \DOMDocument();
@@ -292,8 +284,7 @@ 
         return null;
     }
 
-    protected function getOwnerId($pid = NULL)
-    {
+    protected function getOwnerId($pid = NULL) {
         if ($pid) {
             $this->clientConfigurationManager->setConfigurationPid($pid);
         }
@@ -306,8 +297,7 @@ 
         return $ownerId;
     }
 
-    protected function getSWORDCollection()
-    {
+    protected function getSWORDCollection() {
         return $this->clientConfigurationManager->getSwordCollectionNamespace();
     }
 }

Classes/Tasks/EmbargoTask.php

Braces +1 added lines, -2 removed lines

@@ -9,8 +9,7 @@
 use EWW\Dpf\Domain\Repository\DocumentRepository;
 
 
-class EmbargoTask extends \TYPO3\CMS\Scheduler\Task\AbstractTask
-{
+class EmbargoTask extends \TYPO3\CMS\Scheduler\Task\AbstractTask {
 
     public function execute() {
 

Classes/Domain/Model/Document.php

Braces +62 added lines, -124 removed lines

@@ -20,8 +20,7 @@ 
 /**
  * Document
  */
-class Document extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity
-{
+class Document extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
     /**
      * title
      *
@@ -183,8 +182,7 @@ 
     /**
      * __construct
      */
-    public function __construct()
-    {
+    public function __construct() {
         //Do not remove the next line: It would break the functionality
         $this->initStorageObjects();
     }
@@ -194,8 +192,7 @@ 
      *
      * @return void
      */
-    protected function initStorageObjects()
-    {
+    protected function initStorageObjects() {
         $this->file = new \TYPO3\CMS\Extbase\Persistence\ObjectStorage();
     }
 
@@ -204,8 +201,7 @@ 
      *
      * @return string $title
      */
-    public function getTitle()
-    {
+    public function getTitle() {
         return $this->title;
     }
 
@@ -215,8 +211,7 @@ 
      * @param string $title
      * @return void
      */
-    public function setTitle($title)
-    {
+    public function setTitle($title) {
         $this->title = $title;
         //htmlspecialchars_decode($title,ENT_QUOTES);
     }
@@ -226,8 +221,7 @@ 
      *
      * @return array $authors
      */
-    public function getAuthors()
-    {
+    public function getAuthors() {
         return array_map('trim', explode(";", $this->authors));
     }
 
@@ -237,8 +231,7 @@ 
      * @param array $authors
      * @return void
      */
-    public function setAuthors($authors)
-    {
+    public function setAuthors($authors) {
         $authors = implode("; ", $authors);
         $this->authors = $authors;
     }
@@ -248,8 +241,7 @@ 
      *
      * @return string $xmlData
      */
-    public function getXmlData()
-    {
+    public function getXmlData() {
         return $this->xmlData;
     }
 
@@ -259,8 +251,7 @@ 
      * @param string $xmlData
      * @return void
      */
-    public function setXmlData($xmlData)
-    {
+    public function setXmlData($xmlData) {
         $this->xmlData = $xmlData;
     }
 
@@ -269,8 +260,7 @@ 
      *
      * @return string $slubInfoData
      */
-    public function getSlubInfoData()
-    {
+    public function getSlubInfoData() {
         return $this->slubInfoData;
     }
 
@@ -279,8 +269,7 @@ 
      *
      * @return string $slubInfoData
      */
-    public function setSlubInfoData($slubInfoData)
-    {
+    public function setSlubInfoData($slubInfoData) {
         $this->slubInfoData = $slubInfoData;
     }
 
@@ -289,8 +278,7 @@ 
      *
      * @return \EWW\Dpf\Domain\Model\DocumentType $documentType
      */
-    public function getDocumentType()
-    {
+    public function getDocumentType() {
         return $this->documentType;
     }
 
@@ -300,8 +288,7 @@ 
      * @param \EWW\Dpf\Domain\Model\DocumentType $documentType
      * @return void
      */
-    public function setDocumentType(\EWW\Dpf\Domain\Model\DocumentType $documentType)
-    {
+    public function setDocumentType(\EWW\Dpf\Domain\Model\DocumentType $documentType) {
         $this->documentType = $documentType;
     }
 
@@ -310,8 +297,7 @@ 
      *
      * @return string
      */
-    public function getObjectIdentifier()
-    {
+    public function getObjectIdentifier() {
         return $this->objectIdentifier;
     }
 
@@ -321,8 +307,7 @@ 
      * @param string $objectIdentifier
      * @return void
      */
-    public function setObjectIdentifier($objectIdentifier)
-    {
+    public function setObjectIdentifier($objectIdentifier) {
         // Due to uniqe key uc_object_identifier, which should ignore empty object identifiers.
         $this->objectIdentifier = empty($objectIdentifier)? null : $objectIdentifier;
     }
@@ -332,8 +317,7 @@ 
      *
      * @return string
      */
-    public function getReservedObjectIdentifier()
-    {
+    public function getReservedObjectIdentifier() {
         return $this->reservedObjectIdentifier;
     }
 
@@ -343,8 +327,7 @@ 
      * @param string $reservedObjectIdentifier
      * @return void
      */
-    public function setReservedObjectIdentifier($reservedObjectIdentifier)
-    {
+    public function setReservedObjectIdentifier($reservedObjectIdentifier) {
         $this->reservedObjectIdentifier = $reservedObjectIdentifier;
     }
 
@@ -353,8 +336,7 @@ 
      *
      * @return string
      */
-    public function getTransferStatus()
-    {
+    public function getTransferStatus() {
         return $this->transferStatus;
     }
 
@@ -364,8 +346,7 @@ 
      * @param string
      * @return void
      */
-    public function setTransferStatus($transferStatus)
-    {
+    public function setTransferStatus($transferStatus) {
         $this->transferStatus = $transferStatus;
     }
 
@@ -374,8 +355,7 @@ 
      *
      * @return integer
      */
-    public function getTransferDate()
-    {
+    public function getTransferDate() {
         return $this->transferDate;
     }
 
@@ -385,8 +365,7 @@ 
      * @param integer $transferDate
      * @return void
      */
-    public function setTransferDate($transferDate)
-    {
+    public function setTransferDate($transferDate) {
         $this->transferDate = $transferDate;
     }
 
@@ -395,8 +374,7 @@ 
      *
      * @var integer
      */
-    public function getTransferErrorCode()
-    {
+    public function getTransferErrorCode() {
         return $this->transferErrorCode;
     }
 
@@ -406,8 +384,7 @@ 
      * @param integer $transferErrorCode
      * @return void
      */
-    public function setTransferErrorCode($transferErrorCode)
-    {
+    public function setTransferErrorCode($transferErrorCode) {
         $this->transferErrorCode = $transferErrorCode;
     }
 
@@ -416,8 +393,7 @@ 
      *
      * @var string
      */
-    public function getTransferResponse()
-    {
+    public function getTransferResponse() {
         return $this->transferResponse;
     }
 
@@ -427,8 +403,7 @@ 
      * @param string $transferResponse
      * @return void
      */
-    public function setTransferResponse($transferResponse)
-    {
+    public function setTransferResponse($transferResponse) {
         $this->transferResponse = $transferResponse;
     }
 
@@ -437,8 +412,7 @@ 
      *
      * @var integer
      */
-    public function getTransferHttpStatus()
-    {
+    public function getTransferHttpStatus() {
         return $this->transferHttpStatus;
     }
 
@@ -448,8 +422,7 @@ 
      * @param integer $transferHttpStatus
      * @return void
      */
-    public function setTransferHttpStatus($transferHttpStatus)
-    {
+    public function setTransferHttpStatus($transferHttpStatus) {
         $this->transferHttpStatus = $transferHttpStatus;
     }
 
@@ -459,8 +432,7 @@ 
      * @param \EWW\Dpf\Domain\Model\File $file
      * @return void
      */
-    public function addFile(\EWW\Dpf\Domain\Model\File $file)
-    {
+    public function addFile(\EWW\Dpf\Domain\Model\File $file) {
         $this->file->attach($file);
     }
 
@@ -470,8 +442,7 @@ 
      * @param \EWW\Dpf\Domain\Model\File $fileToRemove The File to be removed
      * @return void
      */
-    public function removeFile(\EWW\Dpf\Domain\Model\File $fileToRemove)
-    {
+    public function removeFile(\EWW\Dpf\Domain\Model\File $fileToRemove) {
         $this->file->detach($fileToRemove);
     }
 
@@ -480,8 +451,7 @@ 
      *
      * @return \TYPO3\CMS\Extbase\Persistence\ObjectStorage<\EWW\Dpf\Domain\Model\File> $file
      */
-    public function getFile()
-    {
+    public function getFile() {
         return $this->file;
     }
 
@@ -491,8 +461,7 @@ 
      * @param \TYPO3\CMS\Extbase\Persistence\ObjectStorage<\EWW\Dpf\Domain\Model\File> $file
      * @return void
      */
-    public function setFile(\TYPO3\CMS\Extbase\Persistence\ObjectStorage $file)
-    {
+    public function setFile(\TYPO3\CMS\Extbase\Persistence\ObjectStorage $file) {
         $this->file = $file;
     }
 
@@ -501,8 +470,7 @@ 
      *
      * @return array
      */
-    public function getFileData()
-    {
+    public function getFileData() {
 
         $fileId = new \EWW\Dpf\Services\Transfer\FileId($this);
 
@@ -552,8 +520,7 @@ 
      *
      * @return array
      */
-    public function getCurrentFileData()
-    {
+    public function getCurrentFileData() {
 
         $fileId = new \EWW\Dpf\Services\Transfer\FileId($this);
 
@@ -601,8 +568,7 @@ 
      *
      * @return boolean $changed
      */
-    public function getChanged()
-    {
+    public function getChanged() {
         return $this->changed;
     }
 
@@ -612,8 +578,7 @@ 
      * @param boolean $changed
      * @return void
      */
-    public function setChanged($changed)
-    {
+    public function setChanged($changed) {
         $this->changed = $changed;
     }
 
@@ -622,8 +587,7 @@ 
      *
      * @return boolean $valid
      */
-    public function getValid()
-    {
+    public function getValid() {
         return $this->valid;
     }
 
@@ -633,8 +597,7 @@ 
      * @param boolean $valid
      * @return void
      */
-    public function setValid($valid)
-    {
+    public function setValid($valid) {
         $this->valid = $valid;
     }
 
@@ -643,8 +606,7 @@ 
      *
      * @return string
      */
-    public function getDateIssued()
-    {
+    public function getDateIssued() {
         return empty($this->dateIssued) ? '' : $this->dateIssued;
     }
 
@@ -654,8 +616,7 @@ 
      * @param string $dateIssued
      * @return void
      */
-    public function setDateIssued($dateIssued)
-    {
+    public function setDateIssued($dateIssued) {
         $this->dateIssued = empty($dateIssued) ? '' : $dateIssued;
     }
 
@@ -665,8 +626,7 @@ 
      *
      * @return string
      */
-    public function getProcessNumber()
-    {
+    public function getProcessNumber() {
         return $this->processNumber;
     }
 
@@ -676,8 +636,7 @@ 
      * @param string $processNumber
      * @return void
      */
-    public function setProcessNumber($processNumber)
-    {
+    public function setProcessNumber($processNumber) {
         $this->processNumber = trim($processNumber);
     }
 
@@ -687,8 +646,7 @@ 
      *
      * @return string
      */
-    public function getSubmitterName()
-    {
+    public function getSubmitterName() {
         try {
             $slub = new \EWW\Dpf\Helper\Slub($this->getSlubInfoData());
             return $slub->getSubmitterName();
@@ -702,8 +660,7 @@ 
      *
      * @return string
      */
-    public function getQucosaUrn()
-    {
+    public function getQucosaUrn() {
         $mods = new \EWW\Dpf\Helper\Mods($this->getXmlData());
         return $mods->getQucosaUrn();
     }
@@ -713,8 +670,7 @@ 
      *
      * @return int
      */
-    public function getCreator()
-    {
+    public function getCreator() {
         return $this->creator? $this->creator : 0;
     }
 
@@ -724,23 +680,19 @@ 
      * @param int $creator
      * @return void
      */
-    public function setCreator($creator)
-    {
+    public function setCreator($creator) {
         $this->creator = $creator;
     }
 
-    public function getState()
-    {
+    public function getState() {
         return $this->state;
     }
 
-    public function setState($state)
-    {
+    public function setState($state) {
         $this->state = $state;
     }
 
-    public function getRemoteState()
-    {
+    public function getRemoteState() {
         $state = explode(':', $this->state);
         if (is_array($state) && array_key_exists(1, $state)) {
             return $state[1];
@@ -778,8 +730,7 @@ 
     /**
      * @return string
      */
-    public function getRemoteLastModDate()
-    {
+    public function getRemoteLastModDate() {
         return $this->remoteLastModDate;
     }
 
@@ -787,16 +738,14 @@ 
      * @param string $remoteLastModDate
      * @return void
      */
-    public function setRemoteLastModDate($remoteLastModDate)
-    {
+    public function setRemoteLastModDate($remoteLastModDate) {
         $this->remoteLastModDate = $remoteLastModDate;
     }
 
     /**
      * @return integer
      */
-    public function getTstamp()
-    {
+    public function getTstamp() {
         return $this->tstamp;
     }
 
@@ -811,8 +760,7 @@ 
     /**
      * @param bool $suggestion
      */
-    public function setSuggestion(bool $suggestion)
-    {
+    public function setSuggestion(bool $suggestion) {
         $this->suggestion = $suggestion;
     }
 
@@ -827,8 +775,7 @@ 
     /**
      * @param string $linkedUid
      */
-    public function setLinkedUid(string $linkedUid)
-    {
+    public function setLinkedUid(string $linkedUid) {
         $this->linkedUid = $linkedUid;
     }
 
@@ -843,8 +790,7 @@ 
     /**
      * @param string $comment
      */
-    public function setComment(string $comment)
-    {
+    public function setComment(string $comment) {
         $this->comment = $comment;
     }
 
@@ -881,8 +827,7 @@ 
      *
      * @return string|int
      */
-    public function getDocumentIdentifier()
-    {
+    public function getDocumentIdentifier() {
         return $this->getObjectIdentifier()? $this->getObjectIdentifier() : $this->getUid();
     }
 
@@ -891,8 +836,7 @@ 
      *
      * @return bool
      */
-    public function isWorkingCopy()
-    {
+    public function isWorkingCopy() {
         return $this->getObjectIdentifier() && !$this->isTemporary() && !$this->isSuggestion();
     }
 
@@ -902,8 +846,7 @@ 
      *
      * @return bool
      */
-    public function isTemporaryCopy()
-    {
+    public function isTemporaryCopy() {
         return $this->getObjectIdentifier() && $this->isTemporary() && !$this->isSuggestion();
     }
 
@@ -913,8 +856,7 @@ 
      *
      * @return string|null
      */
-    public function getPublicationYear()
-    {
+    public function getPublicationYear() {
         $mods = new Mods($this->getXmlData());
         $year =  $mods->getPublishingYear();
         return $year? $year : "";
@@ -925,8 +867,7 @@ 
      *
      * @return string|null
      */
-    public function getMainTitle()
-    {
+    public function getMainTitle() {
         $mods = new Mods($this->getXmlData());
         $title = $mods->getTitle();
         return $title? $title : "";
@@ -938,8 +879,7 @@ 
      *
      * @return string|null
      */
-    public function getSourceDetails()
-    {
+    public function getSourceDetails() {
         $mods = new Mods($this->getXmlData());
         $data = $mods->getSourceDetails();
         return $data;
@@ -956,8 +896,7 @@ 
     /**
      * @param \DateTime|null $embargoDate
      */
-    public function setEmbargoDate(?\DateTime $embargoDate)
-    {
+    public function setEmbargoDate(?\DateTime $embargoDate) {
         $this->embargoDate = $embargoDate;
     }
 
@@ -972,8 +911,7 @@ 
     /**
      * @param \DateTime|null $embargoDate
      */
-    public function setEmbargoDate(?\DateTime $embargoDate)
-    {
+    public function setEmbargoDate(?\DateTime $embargoDate) {
         $this->embargoDate = $embargoDate;
     }
 

Classes/Domain/Model/MetadataObject.php

Braces +37 added lines, -74 removed lines

@@ -17,8 +17,7 @@ 
 /**
  * MetadataObject
  */
-class MetadataObject extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity implements MetadataMandatoryInterface
-{
+class MetadataObject extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity implements MetadataMandatoryInterface {
 
     /**
      * name
@@ -158,8 +157,7 @@ 
      *
      * @return string $name
      */
-    public function getName()
-    {
+    public function getName() {
         return $this->name;
     }
 
@@ -169,8 +167,7 @@ 
      * @param string $name
      * @return void
      */
-    public function setName($name)
-    {
+    public function setName($name) {
         $this->name = $name;
     }
 
@@ -179,8 +176,7 @@ 
      *
      * @return string $displayName
      */
-    public function getDisplayName()
-    {
+    public function getDisplayName() {
         return $this->displayName;
     }
 
@@ -190,8 +186,7 @@ 
      * @param string $displayName
      * @return void
      */
-    public function setDisplayName($displayName)
-    {
+    public function setDisplayName($displayName) {
         $this->displayName = $displayName;
     }
 
@@ -200,8 +195,7 @@ 
      *
      * @return integer $maxIteration
      */
-    public function getMaxIteration()
-    {
+    public function getMaxIteration() {
         return $this->maxIteration;
     }
 
@@ -211,8 +205,7 @@ 
      * @param integer $maxIteration
      * @return void
      */
-    public function setMaxIteration($maxIteration)
-    {
+    public function setMaxIteration($maxIteration) {
         $this->maxIteration = $maxIteration;
     }
 
@@ -221,8 +214,7 @@ 
      *
      * @return string $mandatory
      */
-    public function getMandatory()
-    {
+    public function getMandatory() {
         return $this->mandatory;
     }
 
@@ -232,8 +224,7 @@ 
      * @param string $mandatory
      * @return void
      */
-    public function setMandatory($mandatory)
-    {
+    public function setMandatory($mandatory) {
         $this->mandatory = $mandatory;
     }
 
@@ -242,8 +233,7 @@ 
      *
      * @return boolean $modsExtension
      */
-    public function getModsExtension()
-    {
+    public function getModsExtension() {
         return $this->modsExtension;
     }
 
@@ -253,8 +243,7 @@ 
      * @param boolean $modsExtension
      * @return void
      */
-    public function setModsExtension($modsExtension)
-    {
+    public function setModsExtension($modsExtension) {
         $this->modsExtension = $modsExtension;
     }
 
@@ -263,8 +252,7 @@ 
      *
      * @return boolean
      */
-    public function isModsExtension()
-    {
+    public function isModsExtension() {
         return $this->modsExtension;
     }
 
@@ -273,8 +261,7 @@ 
      *
      * @return string $mapping
      */
-    public function getMapping()
-    {
+    public function getMapping() {
         return $this->mapping;
     }
 
@@ -284,8 +271,7 @@ 
      * @param string $mapping
      * @return void
      */
-    public function setMapping($mapping)
-    {
+    public function setMapping($mapping) {
         $this->mapping = $mapping;
     }
 
@@ -294,8 +280,7 @@ 
      *
      * @return string $relativeMapping
      */
-    public function getRelativeMapping()
-    {
+    public function getRelativeMapping() {
         $modsRegExp = "/^.*?mods:mods/i";
         $mapping    = preg_replace($modsRegExp, "", $this->mapping);
         return trim($mapping, " /");
@@ -306,8 +291,7 @@ 
      *
      * @return integer $inputField
      */
-    public function getInputField()
-    {
+    public function getInputField() {
         return $this->inputField;
     }
 
@@ -317,8 +301,7 @@ 
      * @param integer $inputField
      * @return void
      */
-    public function setInputField($inputField)
-    {
+    public function setInputField($inputField) {
         $this->inputField = $inputField;
     }
 
@@ -327,8 +310,7 @@ 
      *
      * @return \TYPO3\CMS\Extbase\Persistence\ObjectStorage<\EWW\Dpf\Domain\Model\MetadataObject> $metadataObject
      */
-    public function getChildren()
-    {
+    public function getChildren() {
         return null;
     }
 
@@ -337,8 +319,7 @@ 
      *
      * @return \EWW\Dpf\Domain\Model\InputOptionList $inputOptionList
      */
-    public function getInputOptionList()
-    {
+    public function getInputOptionList() {
         return $this->inputOptionList;
     }
 
@@ -348,8 +329,7 @@ 
      * @param \EWW\Dpf\Domain\Model\InputOptionList $inputOptionList
      * @return void
      */
-    public function setInputOptionList(\EWW\Dpf\Domain\Model\InputOptionList $inputOptionList)
-    {
+    public function setInputOptionList(\EWW\Dpf\Domain\Model\InputOptionList $inputOptionList) {
         $this->inputOptionList = $inputOptionList;
     }
 
@@ -358,8 +338,7 @@ 
      *
      * @return string $fillOutService
      */
-    public function getFillOutService()
-    {
+    public function getFillOutService() {
         return $this->fillOutService;
     }
 
@@ -369,8 +348,7 @@ 
      * @param string $fillOutService
      * @return void
      */
-    public function setFillOutService($fillOutService)
-    {
+    public function setFillOutService($fillOutService) {
         $this->fillOutService = $fillOutService;
     }
 
@@ -379,8 +357,7 @@ 
      *
      * @return array $accessRestrictionRoles
      */
-    public function getAccessRestrictionRoles()
-    {
+    public function getAccessRestrictionRoles() {
         if ($this->accessRestrictionRoles) {
             return array_map('trim', explode(',', $this->accessRestrictionRoles));
         } else {
@@ -394,8 +371,7 @@ 
      * @param array $accessRestrictionRoles
      * @return void
      */
-    public function setAccessRestrictionRoles($accessRestrictionRoles)
-    {
+    public function setAccessRestrictionRoles($accessRestrictionRoles) {
         $this->accessRestrictionRoles = implode(',', $accessRestrictionRoles);
     }
 
@@ -404,8 +380,7 @@ 
      *
      * @return boolean $consent
      */
-    public function getConsent()
-    {
+    public function getConsent() {
         return $this->consent;
     }
 
@@ -415,8 +390,7 @@ 
      * @param boolean $consent
      * @return void
      */
-    public function setConsent($consent)
-    {
+    public function setConsent($consent) {
         $this->consent = $consent;
     }
 
@@ -425,8 +399,7 @@ 
      *
      * @return string $defaultValue
      */
-    public function getDefaultValue()
-    {
+    public function getDefaultValue() {
         return $this->defaultValue;
     }
 
@@ -436,8 +409,7 @@ 
      * @param string $defaultValue
      * @return void
      */
-    public function setDefaultValue($defaultValue)
-    {
+    public function setDefaultValue($defaultValue) {
         $this->defaultValue = $defaultValue;
     }
 
@@ -446,8 +418,7 @@ 
      *
      * @return string $validation
      */
-    public function getValidation()
-    {
+    public function getValidation() {
         return $this->validation;
     }
 
@@ -457,8 +428,7 @@ 
      * @param string $validation
      * @return void
      */
-    public function setValidation($validation)
-    {
+    public function setValidation($validation) {
         $this->validation = $validation;
     }
 
@@ -467,8 +437,7 @@ 
      *
      * @return string $dataType
      */
-    public function getDataType()
-    {
+    public function getDataType() {
         return $this->dataType;
     }
 
@@ -478,32 +447,28 @@ 
      * @param string $dataType
      * @return void
      */
-    public function setDataType($dataType)
-    {
+    public function setDataType($dataType) {
         $this->dataType = $dataType;
     }
 
     /**
      * @return string
      */
-    public function getGndFieldUid()
-    {
+    public function getGndFieldUid() {
         return $this->gndFieldUid;
     }
 
     /**
      * @param string $gndFieldUid
      */
-    public function setGndFieldUid($gndFieldUid)
-    {
+    public function setGndFieldUid($gndFieldUid) {
         $this->gndFieldUid = $gndFieldUid;
     }
 
     /**
      * @return integer
      */
-    public function getMaxInputLength()
-    {
+    public function getMaxInputLength() {
         if ($this->maxInputLength == 0) {
             if ($this->inputField == self::input) {
                 return 255;
@@ -518,8 +483,7 @@ 
     /**
      * @return integer
      */
-    public function setMaxInputLength($maxInputLength)
-    {
+    public function setMaxInputLength($maxInputLength) {
         $this->maxInputLength = $maxInputLength;
     }
 
@@ -534,8 +498,7 @@ 
     /**
      * @param bool $embargo
      */
-    public function setEmbargo(bool $embargo)
-    {
+    public function setEmbargo(bool $embargo) {
         $this->embargo = $embargo;
     }
 

Classes/Domain/Model/DocumentFormField.php

Braces +16 added lines, -32 removed lines

@@ -14,8 +14,7 @@ 
  * The TYPO3 project - inspiring people to share!
  */
 
-class DocumentFormField extends AbstractFormElement
-{
+class DocumentFormField extends AbstractFormElement {
 
     protected $value;
 
@@ -55,13 +54,11 @@ 
      */
     protected $maxInputLength;
 
-    public function getValue()
-    {
+    public function getValue() {
         return $this->value;
     }
 
-    public function setValue($value, $defaultValue = '')
-    {
+    public function setValue($value, $defaultValue = '') {
 
         $this->hasDefaultValue = !empty($defaultValue);
 
@@ -92,13 +89,11 @@ 
         }
     }
 
-    public function getInputField()
-    {
+    public function getInputField() {
         return $this->inputField;
     }
 
-    public function setInputField($inputField)
-    {
+    public function setInputField($inputField) {
         $this->inputField = $inputField;
     }
 
@@ -106,8 +101,7 @@ 
      *
      * @return array
      */
-    public function getInputOptions()
-    {
+    public function getInputOptions() {
         return $this->inputOptions;
     }
 
@@ -115,8 +109,7 @@ 
      *
      * @param \EWW\Dpf\Domain\Model\InputOptionList $inputOptionList
      */
-    public function setInputOptions(\EWW\Dpf\Domain\Model\InputOptionList $inputOptionList = null)
-    {
+    public function setInputOptions(\EWW\Dpf\Domain\Model\InputOptionList $inputOptionList = null) {
 
         $this->inputOptions = array();
 
@@ -136,8 +129,7 @@ 
      *
      * @return string $fillOutService
      */
-    public function getFillOutService()
-    {
+    public function getFillOutService() {
         return $this->fillOutService;
     }
 
@@ -147,8 +139,7 @@ 
      * @param string $fillOutService
      * @return void
      */
-    public function setFillOutService($fillOutService)
-    {
+    public function setFillOutService($fillOutService) {
         $this->fillOutService = $fillOutService;
     }
 
@@ -157,8 +148,7 @@ 
      *
      * @return boolean $consent
      */
-    public function getConsent()
-    {
+    public function getConsent() {
         return $this->consent;
     }
 
@@ -168,23 +158,19 @@ 
      * @param boolean $consent
      * @return void
      */
-    public function setConsent($consent)
-    {
+    public function setConsent($consent) {
         $this->consent = $consent;
     }
 
-    public function getHasDefaultValue()
-    {
+    public function getHasDefaultValue() {
         return $this->hasDefaultValue;
     }
 
-    public function getValidation()
-    {
+    public function getValidation() {
         return $this->validation;
     }
 
-    public function setValidation($validation)
-    {
+    public function setValidation($validation) {
         $this->validation = $validation;
     }
 
@@ -193,8 +179,7 @@ 
      *
      * @return string
      */
-    public function getDataType()
-    {
+    public function getDataType() {
         return $this->dataType;
     }
 
@@ -204,8 +189,7 @@ 
      * @param string $dataType
      * @return void
      */
-    public function setDataType($dataType)
-    {
+    public function setDataType($dataType) {
         $this->dataType = $dataType;
     }
 

Classes/Services/Document/DocumentManager.php

Braces +11 added lines, -19 removed lines

@@ -9,8 +9,7 @@ 
 use EWW\Dpf\Domain\Workflow\DocumentWorkflow;
 use EWW\Dpf\Controller\AbstractController;
 
-class DocumentManager
-{
+class DocumentManager {
     /**
      * objectManager
      *
@@ -75,8 +74,7 @@ 
      * @return Document|null
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\IllegalObjectTypeException
      */
-    public function read($identifier)
-    {
+    public function read($identifier) {
         if (!$identifier) {
             return null;
         }
@@ -121,8 +119,7 @@ 
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\IllegalObjectTypeException
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\UnknownObjectException
      */
-    public function update(Document $document, $workflowTransition = null, $deletedFiles = [], $newFiles = [])
-    {
+    public function update(Document $document, $workflowTransition = null, $deletedFiles = [], $newFiles = []) {
         // xml data fields are limited to 64 KB
         if (strlen($document->getXmlData()) >= 64 * 1024 || strlen($document->getSlubInfoData() >= 64 * 1024)) {
             throw new \EWW\Dpf\Exceptions\DocumentMaxSizeErrorException("Maximum document size exceeded.");
@@ -183,7 +180,7 @@ 
                 }
 
                 // check embargo
-                if(!$this->hasActiveEmbargo($document)){
+                if(!$this->hasActiveEmbargo($document)) {
                     $this->removeDocument($document);
                 } else {
                     $document->setState(DocumentWorkflow::LOCAL_STATE_IN_PROGRESS . ':' . $document->getRemoteState());
@@ -221,8 +218,7 @@ 
     /**
      * @return DocumentTransferManager
      */
-    protected function getDocumentTransferManager()
-    {
+    protected function getDocumentTransferManager() {
         /** @var DocumentTransferManager $documentTransferManager */
         $documentTransferManager = $this->objectManager->get(DocumentTransferManager::class);
 
@@ -244,8 +240,7 @@ 
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\IllegalObjectTypeException
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\UnknownObjectException
      */
-    protected function updateFiles(Document $document, $deletedFiles, $newFiles)
-    {
+    protected function updateFiles(Document $document, $deletedFiles, $newFiles) {
         // Delete files
         /** @var File $deleteFile */
         foreach ($deletedFiles as $deleteFile) {
@@ -272,8 +267,7 @@ 
      * @param $document
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\IllegalObjectTypeException
      */
-    protected function removeDocument($document)
-    {
+    protected function removeDocument($document) {
         $files = $this->fileRepository->findByDocument($document->getUid());
         foreach ($files as $file) {
             $this->fileRepository->remove($file);
@@ -292,8 +286,7 @@ 
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\IllegalObjectTypeException
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\UnknownObjectException
      */
-    protected function updateRemotely($document, $workflowTransition = null, $deletedFiles = [], $newFiles = [])
-    {
+    protected function updateRemotely($document, $workflowTransition = null, $deletedFiles = [], $newFiles = []) {
         $lastModDate = $this->getDocumentTransferManager()->getLastModDate($document->getObjectIdentifier());
         $docLastModDate = $document->getRemoteLastModDate();
         if ($lastModDate !== $docLastModDate && !empty($docLastModDate)) {
@@ -330,7 +323,7 @@ 
 
         if ($this->getDocumentTransferManager()->update($document)) {
 
-            if(!$this->hasActiveEmbargo($document)){
+            if(!$this->hasActiveEmbargo($document)) {
                 $this->removeDocument($document);
             } else {
                 $document->setState(DocumentWorkflow::LOCAL_STATE_IN_PROGRESS . ':' . $document->getRemoteState());
@@ -346,10 +339,9 @@ 
      * @return bool (true: if no embargo is set or embargo is expired, false: embargo is active)
      * @throws \Exception
      */
-    protected function hasActiveEmbargo($document)
-    {
+    protected function hasActiveEmbargo($document) {
         $currentDate = new \DateTime('now');
-        if($currentDate > $document->getEmbargoDate()){
+        if($currentDate > $document->getEmbargoDate()) {
             // embargo is expired
             return false;
         } else {