Erikmitk / kitodo-presentation

Classes/Middleware/DOMDocumentValidation.php

Braces +1 added lines, -2 removed lines

@@ -33,8 +33,7 @@
  * @subpackage dlf
  * @access public
  */
-class DOMDocumentValidation implements MiddlewareInterface
-{
+class DOMDocumentValidation implements MiddlewareInterface {
     use LoggerAwareTrait;
 
     /**

Tests/Unit/Validation/SaxonXslToSvrlValidatorTest.php

Braces +1 added lines, -2 removed lines

@@ -28,8 +28,7 @@
  *
  * @access public
  */
-class SaxonXslToSvrlValidatorTest extends UnitTestCase
-{
+class SaxonXslToSvrlValidatorTest extends UnitTestCase {
     const SVRL = <<<SVRL
         <svrl:schematron-output
             xmlns:svrl="http://purl.oclc.org/dsdl/schematron">

Tests/Unit/Validation/DOMDocumentValidationStackTest.php

Braces +1 added lines, -2 removed lines

@@ -29,8 +29,7 @@
  *
  * @access public
  */
-class DOMDocumentValidationStackTest extends UnitTestCase
-{
+class DOMDocumentValidationStackTest extends UnitTestCase {
     public function setUp(): void
     {
         parent::setUp();

Tests/Unit/Validation/XmlSchemesValidatorTest.php

Braces +1 added lines, -2 removed lines

@@ -26,8 +26,7 @@
  *
  * @access public
  */
-class XmlSchemesValidatorTest extends UnitTestCase
-{
+class XmlSchemesValidatorTest extends UnitTestCase {
     const METS = <<<METS
         <mets:mets
             xmlns:mets="http://www.loc.gov/METS/"

Classes/Validation/SaxonXslToSvrlValidator.php

Braces +2 added lines, -4 removed lines

@@ -30,14 +30,12 @@
  *
  * @access public
  */
-class SaxonXslToSvrlValidator extends AbstractDlfValidator implements LoggerAwareInterface
-{
+class SaxonXslToSvrlValidator extends AbstractDlfValidator implements LoggerAwareInterface {
     private string $jar;
 
     private string $xsl;
 
-    public function __construct(array $configuration)
-    {
+    public function __construct(array $configuration) {
         parent::__construct(DOMDocument::class);
         $this->jar = GeneralUtility::getFileAbsFileName($configuration["jar"] ?? '');
         $this->xsl = GeneralUtility::getFileAbsFileName($configuration["xsl"] ?? '');

Classes/Validation/XmlSchemesValidator.php

Braces +2 added lines, -4 removed lines

@@ -24,14 +24,12 @@
  *
  * @access public
  */
-class XmlSchemesValidator extends AbstractDlfValidator
-{
+class XmlSchemesValidator extends AbstractDlfValidator {
     use LibXmlTrait;
 
     private array $schemes;
 
-    public function __construct(array $configuration)
-    {
+    public function __construct(array $configuration) {
         parent::__construct(DOMDocument::class);
         $this->schemes = $configuration;
     }

Classes/Validation/AbstractDlfValidator.php

Braces +2 added lines, -4 removed lines

@@ -29,8 +29,7 @@ 
  *
  * @access public
  */
-abstract class AbstractDlfValidator extends AbstractValidator
-{
+abstract class AbstractDlfValidator extends AbstractValidator {
     use LoggerAwareTrait;
 
     protected string $valueClassName;
@@ -38,8 +37,7 @@ 
     /**
      * @param $valueClassName string The class name of the value
      */
-    public function __construct(string $valueClassName)
-    {
+    public function __construct(string $valueClassName) {
         $this->logger = GeneralUtility::makeInstance(LogManager::class)->getLogger(static::class);
         $this->valueClassName = $valueClassName;
     }

Classes/Controller/AbstractController.php

Braces +8 added lines, -10 removed lines

@@ -39,8 +39,7 @@ 
  *
  * @abstract
  */
-abstract class AbstractController extends ActionController implements LoggerAwareInterface
-{
+abstract class AbstractController extends ActionController implements LoggerAwareInterface {
     use LoggerAwareTrait;
 
     /**
@@ -248,8 +247,7 @@ 
      *
      * @return null|string|array
      */
-    protected function getParametersSafely(string $parameterName)
-    {
+    protected function getParametersSafely(string $parameterName) {
         if ($this->request->hasArgument($parameterName)) {
             return $this->request->getArgument($parameterName);
         }
@@ -444,7 +442,8 @@ 
         $lastStartRecordNumberGrid = 0; // due to validity outside the loop
         foreach (range($firstPage, $lastPage) as $i) {
             // detect which pagination is active: ListView or GridView
-            if (get_class($pagination) == 'TYPO3\CMS\Core\Pagination\SimplePagination') {  // ListView
+            if (get_class($pagination) == 'TYPO3\CMS\Core\Pagination\SimplePagination') {
+// ListView
                 $lastStartRecordNumberGrid = $i; // save last $startRecordNumber for LastPage button
 
                 $pages[$i] = [
@@ -457,7 +456,8 @@ 
                 if (in_array($i, $aRange)) {
                     array_push($pagesSect, ['label' => $i, 'startRecordNumber' => $i]);
                 };
-            } else { // GridView
+            } else {
+// GridView
                 // to calculate the values for generation the links for the pagination pages
                 /** @var \Kitodo\Dlf\Pagination\PageGridPaginator $paginator */
                 $itemsPerPage = $paginator->getPublicItemsPerPage();
@@ -537,8 +537,7 @@ 
      *
      * @return AbstractDocument
      */
-    private function getDocumentByUid(int $documentId)
-    {
+    private function getDocumentByUid(int $documentId) {
         // TODO: implement multiView as it is in getDocumentByUrl
         $doc = null;
         $this->document = $this->documentRepository->findOneByIdAndSettings($documentId);
@@ -563,8 +562,7 @@ 
      *
      * @return AbstractDocument
      */
-    protected function getDocumentByUrl(string $documentId)
-    {
+    protected function getDocumentByUrl(string $documentId) {
         $doc = AbstractDocument::getInstance($documentId, $this->settings, true);
 
         if (isset($this->settings['multiViewType']) && $doc->tableOfContents[0]['type'] === $this->settings['multiViewType']) {

Classes/Controller/OaiPmhController.php

Braces +19 added lines, -38 removed lines

@@ -27,8 +27,7 @@ 
  *
  * @access public
  */
-class OaiPmhController extends AbstractController
-{
+class OaiPmhController extends AbstractController {
     /**
      * @access protected
      * @var TokenRepository
@@ -42,8 +41,7 @@ 
      *
      * @return void
      */
-    public function injectTokenRepository(TokenRepository $tokenRepository)
-    {
+    public function injectTokenRepository(TokenRepository $tokenRepository) {
         $this->tokenRepository = $tokenRepository;
     }
 
@@ -60,8 +58,7 @@ 
      *
      * @return void
      */
-    public function injectCollectionRepository(CollectionRepository $collectionRepository)
-    {
+    public function injectCollectionRepository(CollectionRepository $collectionRepository) {
         $this->collectionRepository = $collectionRepository;
     }
 
@@ -78,8 +75,7 @@ 
      *
      * @return void
      */
-    public function injectLibraryRepository(LibraryRepository $libraryRepository)
-    {
+    public function injectLibraryRepository(LibraryRepository $libraryRepository) {
         $this->libraryRepository = $libraryRepository;
     }
 
@@ -90,8 +86,7 @@ 
      *
      * @return void
      */
-    public function initializeAction()
-    {
+    public function initializeAction() {
         $this->request = $this->request->withFormat("xml");
     }
 
@@ -136,8 +131,7 @@ 
      *
      * @return void
      */
-    protected function deleteExpiredTokens()
-    {
+    protected function deleteExpiredTokens() {
         // Delete expired resumption tokens.
         $this->tokenRepository->deleteExpiredTokens($this->settings['expired']);
     }
@@ -149,8 +143,7 @@ 
      *
      * @return void
      */
-    protected function getUrlParams()
-    {
+    protected function getUrlParams() {
         $allowedParams = [
             'verb',
             'identifier',
@@ -181,8 +174,7 @@ 
      *
      * @return array The mapped metadata array
      */
-    private function getDublinCoreData(array $record)
-    {
+    private function getDublinCoreData(array $record) {
         $metadata = [];
 
         $metadata[] = ['dc:identifier' => $record['record_id']];
@@ -224,8 +216,7 @@ 
      *
      * @return void
      */
-    private function addDublinCoreData(&$metadata, $key, $value)
-    {
+    private function addDublinCoreData(&$metadata, $key, $value) {
         if (!empty($value)) {
             $metadata[] = [$key => $value];
         }
@@ -241,8 +232,7 @@ 
      *
      * @return string The fetched METS XML
      */
-    protected function getMetsData(array $record)
-    {
+    protected function getMetsData(array $record) {
         $mets = null;
         // Load METS file.
         $xml = new \DOMDocument();
@@ -335,8 +325,7 @@ 
      *
      * @return void
      */
-    protected function verbGetRecord()
-    {
+    protected function verbGetRecord() {
         if (count($this->parameters) !== 3 || empty($this->parameters['metadataPrefix']) || empty($this->parameters['identifier'])) {
             $this->error = 'badArgument';
             return;
@@ -388,8 +377,7 @@ 
      *
      * @return void
      */
-    protected function verbIdentify()
-    {
+    protected function verbIdentify() {
         $library = $this->libraryRepository->findByUid($this->settings['library']);
 
         $oaiIdentifyInfo = [];
@@ -438,8 +426,7 @@ 
      *
      * @return void
      */
-    protected function verbListIdentifiers()
-    {
+    protected function verbListIdentifiers() {
         // If we have a resumption token we can continue our work
         if (!empty($this->parameters['resumptionToken'])) {
             // "resumptionToken" is an exclusive argument.
@@ -493,8 +480,7 @@ 
      *
      * @return void
      */
-    protected function verbListMetadataFormats()
-    {
+    protected function verbListMetadataFormats() {
         $resArray = [];
         // check for the optional "identifier" parameter
         if (isset($this->parameters['identifier'])) {
@@ -526,8 +512,7 @@ 
      *
      * @return void
      */
-    protected function verbListRecords()
-    {
+    protected function verbListRecords() {
         // Check for invalid arguments.
         if (!empty($this->parameters['resumptionToken'])) {
             // "resumptionToken" is an exclusive argument.
@@ -582,8 +567,7 @@ 
      *
      * @return void
      */
-    protected function verbListSets()
-    {
+    protected function verbListSets() {
         // It is required to set a oai_name inside the collection record to be shown in oai-pmh plugin.
         $this->settings['hideEmptyOaiNames'] = true;
 
@@ -729,8 +713,7 @@ 
      *
      * @return array|false
      */
-    private function getDate(string $dateType)
-    {
+    private function getDate(string $dateType) {
         return strptime($this->parameters[$dateType], '%Y-%m-%dT%H:%M:%SZ') ?: strptime($this->parameters[$dateType], '%Y-%m-%d');
     }
 
@@ -785,8 +768,7 @@ 
      *
      * @return array of enriched records
      */
-    protected function generateOutputForDocumentList(array $documentListSet)
-    {
+    protected function generateOutputForDocumentList(array $documentListSet) {
         // check whether any result elements are available
         if (empty($documentListSet) || empty($documentListSet['elements'])) {
             $this->error = 'noRecordsMatch';
@@ -841,8 +823,7 @@ 
      *
      * @return void
      */
-    protected function generateResumptionTokenForDocumentListSet(array $documentListSet, int $numShownDocuments)
-    {
+    protected function generateResumptionTokenForDocumentListSet(array $documentListSet, int $numShownDocuments) {
         // The cursor specifies how many elements have already been returned in previous requests
         // See http://www.openarchives.org/OAI/openarchivesprotocol.html#FlowControl
         $currentCursor = $documentListSet['metadata']['cursor'];