kitodo / kitodo-publication

Classes/Domain/Model/TransformationFile.php

Braces +4 added lines, -8 removed lines

@@ -17,8 +17,7 @@ 
 /**
  * File
  */
-class TransformationFile extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity
-{
+class TransformationFile extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
 
     /**
      * title
@@ -52,8 +51,7 @@ 
     /**
      * @param string $title
      */
-    public function setTitle(string $title)
-    {
+    public function setTitle(string $title) {
         $this->title = $title;
     }
 
@@ -68,8 +66,7 @@ 
     /**
      * @param string $label
      */
-    public function setLabel(string $label)
-    {
+    public function setLabel(string $label) {
         $this->label = $label;
     }
 
@@ -84,8 +81,7 @@ 
     /**
      * @param \TYPO3\CMS\Extbase\Domain\Model\FileReference $file
      */
-    public function setFile(\TYPO3\CMS\Extbase\Domain\Model\FileReference $file)
-    {
+    public function setFile(\TYPO3\CMS\Extbase\Domain\Model\FileReference $file) {
         $this->file = $file;
     }
 

Classes/Domain/Model/DepositLicenseLog.php

Braces +1 added lines, -2 removed lines

@@ -17,8 +17,7 @@
 /**
  * Document
  */
-class DepositLicenseLog extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity
-{
+class DepositLicenseLog extends \TYPO3\CMS\Extbase\DomainObject\AbstractEntity {
     /**
      * @var string
      */

Classes/Domain/Repository/DepositLicenseLogRepository.php

Braces +1 added lines, -2 removed lines

@@ -17,7 +17,6 @@
 /**
  * The repository for DepositLicenseLog
  */
-class DepositLicenseLogRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository
-{
+class DepositLicenseLogRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository {
 
 }

Classes/Domain/Repository/DepositLicenseRepository.php

Braces +1 added lines, -2 removed lines

@@ -17,7 +17,6 @@
 /**
  * The repository for DepositLicense
  */
-class DepositLicenseRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository
-{
+class DepositLicenseRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository {
 
 }

Classes/Domain/Repository/ExternalMetadataRepository.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
     {
         $query = $this->createQuery();
 
-        $constraints = array();
+        $constraints = array ();
         $constraints[] = $query->equals('publication_identifier', $identifier);
         $constraints[] = $query->equals('fe_user', $feUser);
 

Braces +3 added lines, -6 removed lines

@@ -17,8 +17,7 @@ 
 /**
  * The repository for to be imported external metadata
  */
-class ExternalMetadataRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository
-{
+class ExternalMetadataRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository {
     /**
      * Find external metadata by a frontend user uid and a publication identifier.
      *
@@ -26,8 +25,7 @@ 
      * @param string $identifier
      * @return array|\TYPO3\CMS\Extbase\Persistence\QueryResultInterface
      */
-    public function findOneByUserAndPublicationIdentifier($feUser, $identifier)
-    {
+    public function findOneByUserAndPublicationIdentifier($feUser, $identifier) {
         $query = $this->createQuery();
 
         $constraints = array();
@@ -44,8 +42,7 @@ 
      * @param $feUserUid
      * @throws \TYPO3\CMS\Extbase\Persistence\Exception\IllegalObjectTypeException
      */
-    public function clearExternalMetadataByFeUserUid($feUserUid)
-    {
+    public function clearExternalMetadataByFeUserUid($feUserUid) {
         $metadata = $this->findByFeUser($feUserUid);
 
         foreach ($metadata as $metadataItem) {

Classes/Domain/Repository/MetadataGroupRepository.php

Spacing +4 added lines, -4 removed lines

@@ -19,13 +19,13 @@
  */
 class MetadataGroupRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository
 {
-    public function findPersonGroup(){
+    public function findPersonGroup() {
 
         $query = $this->createQuery();
 
-        $constraints = array();
-        $constraints[] =  $query->like('group_type', '%Person%');
-        $constraints[] =  $query->like('group_type', '%Fis%');
+        $constraints = array ();
+        $constraints[] = $query->like('group_type', '%Person%');
+        $constraints[] = $query->like('group_type', '%Fis%');
 
         if (count($constraints)) {
             $query->matching(

Braces +2 added lines, -3 removed lines

@@ -17,9 +17,8 @@
 /**
  * The repository for MetadataGroups
  */
-class MetadataGroupRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository
-{
-    public function findPersonGroup(){
+class MetadataGroupRepository extends \EWW\Dpf\Domain\Repository\AbstractRepository {
+    public function findPersonGroup() {
 
         $query = $this->createQuery();
 

Classes/Domain/Repository/DocumentTypeRepository.php

Spacing +6 added lines, -6 removed lines

@@ -23,13 +23,13 @@ 
     {
         $documentTypes = $this->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,
             );
@@ -58,7 +58,7 @@ 
     {
         $query = $this->createQuery();
         $query->matching(
-            $query->like($externalTypesDbColumn, '%"'.$type.'"%')
+            $query->like($externalTypesDbColumn, '%"' . $type . '"%')
         );
 
         return $query->execute()->getFirst();

Braces +3 added lines, -6 removed lines

@@ -17,10 +17,8 @@ 
 /**
  * The repository for DocumentTypes
  */
-class DocumentTypeRepository extends AbstractRepository
-{
-    public function getDocumentTypesAlphabetically()
-    {
+class DocumentTypeRepository extends AbstractRepository {
+    public function getDocumentTypesAlphabetically() {
         $documentTypes = $this->findAll();
 
         $data = array();
@@ -54,8 +52,7 @@ 
      * @param string $externalTypesDbColumn
      * @return object
      */
-    public function findOneByExternalType($type, $externalTypesDbColumn)
-    {
+    public function findOneByExternalType($type, $externalTypesDbColumn) {
         $query = $this->createQuery();
         $query->matching(
             $query->like($externalTypesDbColumn, '%"'.$type.'"%')

Classes/Configuration/ClientConfigurationManager.php

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

@@ -51,7 +51,7 @@ 
      *
      * @var \EWW\Dpf\Domain\Model\Client
      */
-    protected $client = null;
+    protected $client = NULL;
 
     /**
      * extensionConfiguration
@@ -71,7 +71,7 @@ 
                 $this->client = $clientRepository->findAll()->current();
 
                 $configurationManager = $objectManager->get(BackendConfigurationManager::class);
-                $settings = $configurationManager->getConfiguration(null, null);
+                $settings = $configurationManager->getConfiguration(NULL, NULL);
                 $this->settings = $settings; //['settings'];
             }
 
@@ -102,9 +102,9 @@ 
      *
      * @var array
      */
-    public function getSetting($settingName, $extConfig = null)
+    public function getSetting($settingName, $extConfig = NULL)
     {
-        $setting = null;
+        $setting = NULL;
         if ($this->client) {
             $setting = trim($this->client->{"get" . ucfirst($settingName)}());
         }

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
      *
      * @var array
      */
-    protected $settings = array();
+    protected $settings = array ();
 
 
     /**
@@ -44,7 +44,7 @@ 
      *
      * @var array
      */
-    protected $extensionConfiguration = array();
+    protected $extensionConfiguration = array ();
 
     public function __construct()
     {
@@ -52,7 +52,7 @@ 
         $clientRepository = $objectManager->get(ClientRepository::class);
 
         if (TYPO3_MODE === 'BE') {
-            $selectedPageId = (int)GeneralUtility::_GP('id');
+            $selectedPageId = (int) GeneralUtility::_GP('id');
             if ($selectedPageId) {
                 $this->client = $clientRepository->findAll()->current();
 

Braces +58 added lines, -116 removed lines

@@ -22,8 +22,7 @@ 
 use EWW\Dpf\Domain\Repository\ClientRepository;
 use TYPO3\CMS\Core\Configuration\ExtensionConfiguration;
 
-class ClientConfigurationManager
-{
+class ClientConfigurationManager {
     /**
      * settings
      *
@@ -46,8 +45,7 @@ 
      */
     protected $extensionConfiguration = array();
 
-    public function __construct()
-    {
+    public function __construct() {
         $objectManager = GeneralUtility::makeInstance(ObjectManager::class);
         $clientRepository = $objectManager->get(ClientRepository::class);
 
@@ -77,8 +75,7 @@ 
 
     }
 
-    public function setConfigurationPid($pid)
-    {
+    public function setConfigurationPid($pid) {
         $objectManager = GeneralUtility::makeInstance(ObjectManager::class);
         $clientRepository = $objectManager->get(ClientRepository::class);
 
@@ -88,8 +85,7 @@ 
     /**
      * @return int|null
      */
-    public function getClientPid()
-    {
+    public function getClientPid() {
         return $this->client->getPid();
     }
 
@@ -98,8 +94,7 @@ 
      *
      * @var array
      */
-    public function getSetting($settingName, $extConfig = null)
-    {
+    public function getSetting($settingName, $extConfig = null) {
         $setting = null;
         if ($this->client) {
             $setting = trim($this->client->{"get" . ucfirst($settingName)}());
@@ -114,270 +109,217 @@ 
     }
 
 
-    public function getOwnerId()
-    {
+    public function getOwnerId() {
         return $this->getSetting("ownerId");
     }
 
-    public function getSwordHost()
-    {
+    public function getSwordHost() {
         return $this->getSetting("swordHost", "swordHost");
     }
 
-    public function getSwordUser()
-    {
+    public function getSwordUser() {
         return $this->getSetting("swordUser", "swordUser");
     }
 
-    public function getSwordPassword()
-    {
+    public function getSwordPassword() {
         return $this->getSetting("swordPassword", "swordPassword");
     }
 
-    public function getSwordCollectionNamespace()
-    {
+    public function getSwordCollectionNamespace() {
         return $this->getSetting("swordCollectionNamespace", "swordCollectionNamespace");
     }
 
-    public function getFedoraHost()
-    {
+    public function getFedoraHost() {
         return $this->getSetting("fedoraHost", "fedoraHost");
     }
 
-    public function getFedoraUser()
-    {
+    public function getFedoraUser() {
         return $this->getSetting("fedoraUser", "fedoraUser");
     }
 
-    public function getFedoraPassword()
-    {
+    public function getFedoraPassword() {
         return $this->getSetting("fedoraPassword", "fedoraPassword");
     }
 
-    public function getElasticSearchHost()
-    {
+    public function getElasticSearchHost() {
         return $this->getSetting("elasticSearchHost", "elasticSearchHost");
     }
 
-    public function getElasticSearchPort()
-    {
+    public function getElasticSearchPort() {
         return $this->getSetting("elasticSearchPort", "elasticSearchPort");
     }
 
-    public function getElasticSearchIndexName()
-    {
+    public function getElasticSearchIndexName() {
         return $this->getSetting("elasticSearchIndexName", "elasticSearchIndexName");
     }
 
-    public function getUploadDirectory()
-    {
+    public function getUploadDirectory() {
         return $this->getSetting("uploadDirectory", "uploadDirectory");
     }
 
-    public function getUploadDomain()
-    {
+    public function getUploadDomain() {
         return $this->getSetting("uploadDomain", "uploadDomain");
     }
 
-    public function getSuggestionFlashMessage()
-    {
+    public function getSuggestionFlashMessage() {
         return $this->getSetting("suggestionFlashmessage", "suggestionFlashmessage");
     }
 
-    public function getFileXpath()
-    {
+    public function getFileXpath() {
         return $this->getSetting("fileXpath", "fileXpath");
     }
 
-    public function getStateXpath()
-    {
+    public function getStateXpath() {
         return $this->getSetting("stateXpath", "stateXpath");
     }
 
-    public function getTypeXpath()
-    {
+    public function getTypeXpath() {
         return $this->getSetting("typeXpath", "typeXpath");
     }
 
-    public function getTypeXpathInput()
-    {
+    public function getTypeXpathInput() {
         return $this->getSetting("typeXpathInput", "typeXpathInput");
     }
 
-    public function getUrnXpath()
-    {
+    public function getUrnXpath() {
         return $this->getSetting("urnXpath", "urnXpath");
     }
 
-    public function getPrimaryUrnXpath()
-    {
+    public function getPrimaryUrnXpath() {
         return $this->getSetting("primaryUrnXpath", "primaryUrnXpath");
     }
 
-    public function getDateXpath()
-    {
+    public function getDateXpath() {
         return $this->getSetting("dateXpath", "dateXpath");
     }
 
-    public function getPublishingYearXpath()
-    {
+    public function getPublishingYearXpath() {
         return $this->getSetting("publishingYearXpath", "publishingYearXpath");
     }
 
-    public function getNamespaces()
-    {
+    public function getNamespaces() {
         return $this->getSetting("namespaces", "namespaces");
     }
 
-    public function getTitleXpath()
-    {
+    public function getTitleXpath() {
         return $this->getSetting("titleXpath", "titleXpath");
     }
 
-    public function getOriginalSourceTitleXpath()
-    {
+    public function getOriginalSourceTitleXpath() {
         return $this->getSetting("originalSourceTitleXpath", "originalSourceTitleXpath");
     }
 
-    public function getProcessNumberXpath()
-    {
+    public function getProcessNumberXpath() {
         return $this->getSetting("processnumberXpath", "processnumberXpath");
     }
 
-    public function getSubmitterNameXpath()
-    {
+    public function getSubmitterNameXpath() {
         return $this->getSetting("submitterNameXpath", "submitterNameXpath");
     }
 
-    public function getSubmitterEmailXpath()
-    {
+    public function getSubmitterEmailXpath() {
         return $this->getSetting("submitterEmailXpath", "submitterEmailXpath");
     }
 
-    public function getSubmitterNoticeXpath()
-    {
+    public function getSubmitterNoticeXpath() {
         return $this->getSetting("submitterNoticeXpath", "submitterNoticeXpath");
     }
 
-    public function getCreatorXpath()
-    {
+    public function getCreatorXpath() {
         return $this->getSetting("creatorXpath", "creatorXpath");
     }
 
-    public function getCreationDateXpath()
-    {
+    public function getCreationDateXpath() {
         return $this->getSetting("creationDateXpath", "creationDateXpath");
     }
 
-    public function getRepositoryCreationDateXpath()
-    {
+    public function getRepositoryCreationDateXpath() {
         return $this->getSetting("repositoryCreationDateXpath", "repositoryCreationDateXpath");
     }
 
-    public function getRepositoryLastModDateXpath()
-    {
+    public function getRepositoryLastModDateXpath() {
         return $this->getSetting("repositoryLastModDateXpath", "repositoryLastModDateXpath");
     }
 
-    public function getDepositLicenseXpath()
-    {
+    public function getDepositLicenseXpath() {
         return $this->getSetting("depositLicenseXpath", "depositLicenseXpath");
     }
 
-    public function getAllNotesXpath()
-    {
+    public function getAllNotesXpath() {
         return $this->getSetting("allNotesXpath", "allNotesXpath");
     }
 
-    public function getPrivateNotesXpath()
-    {
+    public function getPrivateNotesXpath() {
         return $this->getSetting("privateNotesXpath", "privateNotesXpath");
     }
 
-    public function getInputTransformation()
-    {
+    public function getInputTransformation() {
         return $this->client->getInputTransformation()->current();
     }
 
-    public function getOutputTransformation()
-    {
+    public function getOutputTransformation() {
         return $this->client->getOutputTransformation()->current();
     }
 
-    public function getPersonXpath()
-    {
+    public function getPersonXpath() {
         return $this->getSetting("personXpath", "personXpath");
     }
 
-    public function getPersonFamilyXpath()
-    {
+    public function getPersonFamilyXpath() {
         return $this->getSetting("personFamilyXpath", "personFamilyXpath");
     }
 
-    public function getPersonGivenXpath()
-    {
+    public function getPersonGivenXpath() {
         return $this->getSetting("personGivenXpath", "personGivenXpath");
     }
 
-    public function getPersonRoleXpath()
-    {
+    public function getPersonRoleXpath() {
         return $this->getSetting("personRoleXpath", "personRoleXpath");
     }
 
-    public function getPersonFisIdentifierXpath()
-    {
+    public function getPersonFisIdentifierXpath() {
         return $this->getSetting("personFisIdentifierXpath", "personFisIdentifierXpath");
     }
 
-    public function getPersonAffiliationXpath()
-    {
+    public function getPersonAffiliationXpath() {
         return $this->getSetting("personAffiliationXpath", "personAffiliationXpath");
     }
 
-    public function getPersonAffiliationIdentifierXpath()
-    {
+    public function getPersonAffiliationIdentifierXpath() {
         return $this->getSetting("personAffiliationIdentifierXpath", "personAffiliationIdentifierXpath");
     }
 
-    public function getPersonAuthorRole()
-    {
+    public function getPersonAuthorRole() {
         return $this->getSetting("personAuthorRole", "personAuthorRole");
     }
 
-    public function getPersonPublisherRole()
-    {
+    public function getPersonPublisherRole() {
         return $this->getSetting("personPublisherRole", "personPublisherRole");
     }
 
-    public function getValidationXpath()
-    {
+    public function getValidationXpath() {
         return $this->getSetting("validationXpath", "validationXpath");
     }
 
-    public function fisIdXpath()
-    {
+    public function fisIdXpath() {
         return $this->getSetting("fisIdXpath", "fisIdXpath");
     }
 
-    public function getSourceDetailsXpaths()
-    {
+    public function getSourceDetailsXpaths() {
         return $this->getSetting("sourceDetailsXpaths", "sourceDetailsXpaths");
     }
 
-    public function getFedoraNamespace()
-    {
+    public function getFedoraNamespace() {
         $settings = $this->getTypoScriptSettings();
         return $settings['fedoraNamespace'];
     }
 
-    public function getUniversityCollection()
-    {
+    public function getUniversityCollection() {
         $settings = $this->getTypoScriptSettings();
         return $settings['universityCollection'];
     }
 
-    public function getTypoScriptSettings()
-    {
+    public function getTypoScriptSettings() {
         $objectManager = GeneralUtility::makeInstance(ObjectManager::class);
         $configurationManager = $objectManager->get(ConfigurationManager::class);
         $settings = $configurationManager->getConfiguration(

Classes/Helper/XSLTransformator.php

Braces +2 added lines, -4 removed lines

@@ -23,8 +23,7 @@ 
 use \TYPO3\CMS\Core\Log\LogLevel;
 use \TYPO3\CMS\Core\Log\LogManager;
 
-class XSLTransformator
-{
+class XSLTransformator {
 
     /**
      * clientConfigurationManager
@@ -97,8 +96,7 @@ 
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return string The transformed xml
      */
-    public function getTransformedOutputXML($document, $xmlData = false)
-    {
+    public function getTransformedOutputXML($document, $xmlData = false) {
         $objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(ObjectManager::class);
         $this->clientConfigurationManager = $objectManager->get(ClientConfigurationManager::class);
 

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

@@ -80,7 +80,7 @@
      * @param \EWW\Dpf\Domain\Model\Document $document
      * @return string The transformed xml
      */
-    public function getTransformedOutputXML($document, $xmlData = false)
+    public function getTransformedOutputXML($document, $xmlData = FALSE)
     {
         $objectManager = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(ObjectManager::class);
         $this->clientConfigurationManager = $objectManager->get(ClientConfigurationManager::class);

Spacing +4 added lines, -4 removed lines

@@ -87,7 +87,7 @@ 
             $logger->log(
                 LogLevel::WARNING,
                 "Input XML: No transformation file is present. The given xml data was taken over as it is",
-                array(
+                array (
                     'documentTypeName' => $documentTypeName
                 )
             );
@@ -117,7 +117,7 @@ 
             $filePath = $transformationFile->getFile()->getOriginalResource()->getIdentifier();
             $documentTransformer = new DocumentTransformer();
 
-            if ( !$document->getRemoteState() || $document->getRemoteState() == 'NONE' ) {
+            if (!$document->getRemoteState() || $document->getRemoteState() == 'NONE') {
                 $remoteState = 'ACTIVE';
             } else {
                 $remoteState = $document->getRemoteState();
@@ -136,7 +136,7 @@ 
                 );
             } else {
                 $transformedXml = $documentTransformer->transform(
-                    Environment::getPublicPath(). '/fileadmin' . $filePath, $document->getXmlData(), $transformParams
+                    Environment::getPublicPath() . '/fileadmin' . $filePath, $document->getXmlData(), $transformParams
                 );
             }
 
@@ -152,7 +152,7 @@ 
             $logger->log(
                 LogLevel::WARNING,
                 "Output XML: No transformation file is present. The generated xml data was taken over as it is",
-                array(
+                array (
                     'documentTypeName' => $documentTypeName
                 )
             );