kitodo / kitodo-presentation

Classes/Controller/CalendarController.php

Spacing +4 added lines, -4 removed lines

@@ -129,7 +129,7 @@ 
                 foreach ($year['children'] as $month) {
                     foreach ($month['children'] as $day) {
                         foreach ($day['children'] as $issue) {
-                            $title = $issue['label'] ?: $issue['orderlabel'];
+                            $title = $issue['label'] ? : $issue['orderlabel'];
                             if (strtotime($title) !== false) {
                                 $title = strftime('%x', strtotime($title));
                             }
@@ -220,8 +220,8 @@ 
         $this->view->assign('calendarData', $calendarData);
         $this->view->assign('documentId', $this->document->getUid());
         $this->view->assign('yearLinkTitle', $yearLinkTitle);
-        $this->view->assign('parentDocumentId', $this->document->getPartof() ?: $this->document->getDoc()->tableOfContents[0]['points']);
-        $this->view->assign('allYearDocTitle', $this->document->getDoc()->getTitle($this->document->getPartof()) ?: $this->document->getDoc()->tableOfContents[0]['label']);
+        $this->view->assign('parentDocumentId', $this->document->getPartof() ? : $this->document->getDoc()->tableOfContents[0]['points']);
+        $this->view->assign('allYearDocTitle', $this->document->getDoc()->getTitle($this->document->getPartof()) ? : $this->document->getDoc()->tableOfContents[0]['label']);
     }
 
     /**
@@ -257,7 +257,7 @@ 
 
                 if (empty($yearLabel)) {
                     // if neither order nor orderlabel is set, use the id...
-                    $yearLabel = (string)$id;
+                    $yearLabel = (string) $id;
                 }
 
                 $years[] = [

Classes/Hooks/ItemsProcFunc.php

Braces +2 added lines, -1 removed lines

@@ -58,7 +58,8 @@
      *
      * @return void
      */
-    public function getTyposcriptConfigFromPluginSiteRoot($params) {
+    public function getTyposcriptConfigFromPluginSiteRoot($params)
+    {
         $objectManager = GeneralUtility::makeInstance(ObjectManager::class);
         $pid = $params['flexParentDatabaseRow']['pid'];
         $rootline = \TYPO3\CMS\Backend\Utility\BackendUtility::BEgetRootLine($pid);

Classes/Controller/View3DController.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
             $settingsParts = explode("/", $model);
             $fileName = end($settingsParts);
-            $path = substr($model, 0,  strrpos($model, $fileName));
+            $path = substr($model, 0, strrpos($model, $fileName));
             $modelSettings = $path . "metadata/" . $fileName . "_viewer";
 
             if (!empty($modelConverted)) {

Classes/Controller/NavigationController.php

Braces +2 added lines, -1 removed lines

@@ -30,7 +30,8 @@
      * @param \Kitodo\Dlf\Domain\Model\PageSelectForm|NULL $pageSelectForm
      * @return void
      */
-    public function pageSelectAction(\Kitodo\Dlf\Domain\Model\PageSelectForm $pageSelectForm = NULL) {
+    public function pageSelectAction(\Kitodo\Dlf\Domain\Model\PageSelectForm $pageSelectForm = NULL)
+    {
         if ($pageSelectForm) {
             $uriBuilder = $this->getControllerContext()->getUriBuilder();
             $uri = $uriBuilder->reset()

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

@@ -30,7 +30,7 @@
      * @param \Kitodo\Dlf\Domain\Model\PageSelectForm|NULL $pageSelectForm
      * @return void
      */
-    public function pageSelectAction(\Kitodo\Dlf\Domain\Model\PageSelectForm $pageSelectForm = NULL) {
+    public function pageSelectAction(\Kitodo\Dlf\Domain\Model\PageSelectForm $pageSelectForm = null) {
         if ($pageSelectForm) {
             $uriBuilder = $this->getControllerContext()->getUriBuilder();
             $uri = $uriBuilder->reset()

Classes/Controller/SearchController.php

Spacing +3 added lines, -3 removed lines

@@ -107,13 +107,13 @@
         }
 
         // sanitize date search input
-        if(empty($this->searchParams['dateFrom']) && !empty($this->searchParams['dateTo'])) {
+        if (empty($this->searchParams['dateFrom']) && !empty($this->searchParams['dateTo'])) {
             $this->searchParams['dateFrom'] = '*';
         }
-        if(empty($this->searchParams['dateTo']) && !empty($this->searchParams['dateFrom'])) {
+        if (empty($this->searchParams['dateTo']) && !empty($this->searchParams['dateFrom'])) {
             $this->searchParams['dateTo'] = 'NOW';
         }
-        if($this->searchParams['dateFrom'] > $this->searchParams['dateTo']) {
+        if ($this->searchParams['dateFrom'] > $this->searchParams['dateTo']) {
             $tmpDate = $this->searchParams['dateFrom'];
             $this->searchParams['dateFrom'] = $this->searchParams['dateTo'];
             $this->searchParams['dateTo'] = $tmpDate;

Classes/Format/Mods.php

Braces +20 added lines, -10 removed lines

@@ -71,7 +71,8 @@ 
      *
      * @return void
      */
-    private function getAuthors() {
+    private function getAuthors()
+    {
         $authors = $this->xml->xpath('./mods:name[./mods:role/mods:roleTerm[@type="code" and @authority="marcrelator"]="aut"]');
 
         // Get "author" and "author_sorting" again if that was too sophisticated.
@@ -93,7 +94,8 @@ 
         }
     }
 
-    private function getAuthorFromOrcidApi($orcidId, $authors, $i) {
+    private function getAuthorFromOrcidApi($orcidId, $authors, $i)
+    {
         $profile = new OrcidProfile($orcidId);
         $name = $profile->getFullName();
         if (!empty($name)) {
@@ -107,7 +109,8 @@ 
         }
     }
 
-    private function getAuthorFromXml($authors, $i) {
+    private function getAuthorFromXml($authors, $i)
+    {
         $this->getAuthorFromXmlDisplayForm($authors, $i);
 
         $nameParts = $authors[$i]->xpath('./mods:namePart');
@@ -150,7 +153,8 @@ 
         }
     }
 
-    private function getAuthorFromXmlDisplayForm($authors, $i) {
+    private function getAuthorFromXmlDisplayForm($authors, $i)
+    {
         $displayForm = $authors[$i]->xpath('./mods:displayForm');
         if ($displayForm) {
             $this->metadata['author'][$i] = (string) $displayForm[0];
@@ -164,7 +168,8 @@ 
      *
      * @return void
      */
-    private function getHolders() {
+    private function getHolders()
+    {
         $holders = $this->xml->xpath('./mods:name[./mods:role/mods:roleTerm[@type="code" and @authority="marcrelator"]="prv"]');
 
         if (!empty($holders)) {
@@ -181,7 +186,8 @@ 
         }
     }
 
-    private function getHolderFromViafApi($viafId, $holders, $i) {
+    private function getHolderFromViafApi($viafId, $holders, $i)
+    {
         $profile = new ViafProfile($viafId);
         $name = $profile->getFullName();
         if (!empty($name)) {
@@ -195,7 +201,8 @@ 
         }
     }
 
-    private function getHolderFromXml($holders, $i) {
+    private function getHolderFromXml($holders, $i)
+    {
         $this->getHolderFromXmlDisplayForm($holders, $i);
         // Append "valueURI" to name using Unicode unit separator.
         if (isset($holders[$i]['valueURI'])) {
@@ -203,7 +210,8 @@ 
         }
     }
 
-    private function getHolderFromXmlDisplayForm($holders, $i) {
+    private function getHolderFromXmlDisplayForm($holders, $i)
+    {
         // Check if there is a display form.
         $displayForm = $holders[$i]->xpath('./mods:displayForm');
         if ($displayForm) {
@@ -218,7 +226,8 @@ 
      *
      * @return void
      */
-    private function getPlaces() {
+    private function getPlaces()
+    {
         $places = $this->xml->xpath('./mods:originInfo[not(./mods:edition="[Electronic ed.]")]/mods:place/mods:placeTerm');
         // Get "place" and "place_sorting" again if that was to sophisticated.
         if (empty($places)) {
@@ -242,7 +251,8 @@ 
      *
      * @return void
      */
-    private function getYears() {
+    private function getYears()
+    {
         // Get "year_sorting".
         if (($years_sorting = $this->xml->xpath('./mods:originInfo[not(./mods:edition="[Electronic ed.]")]/mods:dateOther[@type="order" and @encoding="w3cdtf"]'))) {
             foreach ($years_sorting as $year_sorting) {

Classes/Common/Indexer.php

Spacing +2 added lines, -2 removed lines

@@ -337,7 +337,7 @@ 
             $solrDoc->setField('title', $metadata['title'][0], self::$fields['fieldboost']['title']);
             $solrDoc->setField('volume', $metadata['volume'][0], self::$fields['fieldboost']['volume']);
             // verify date formatting
-            if(strtotime($metadata['date'][0])) {
+            if (strtotime($metadata['date'][0])) {
                 // do not alter dates YYYY or YYYY-MM or YYYY-MM-DD
                 if (
                     preg_match("/^[\d]{4}$/", $metadata['date'][0])
@@ -346,7 +346,7 @@ 
                 ) {
                     $solrDoc->setField('date', $metadata['date'][0]);
                 // change date YYYYMMDD to YYYY-MM-DD
-                } elseif (preg_match("/^[\d]{8}$/", $metadata['date'][0])){
+                } elseif (preg_match("/^[\d]{8}$/", $metadata['date'][0])) {
                     $solrDoc->setField('date', date("Y-m-d", strtotime($metadata['date'][0])));
                 // convert any datetime to proper ISO extended datetime format and timezone for SOLR
                 } else {

Braces +5 added lines, -3 removed lines

@@ -346,7 +346,7 @@ 
                 ) {
                     $solrDoc->setField('date', $metadata['date'][0]);
                 // change date YYYYMMDD to YYYY-MM-DD
-                } elseif (preg_match("/^[\d]{8}$/", $metadata['date'][0])){
+                } elseif (preg_match("/^[\d]{8}$/", $metadata['date'][0])) {
                     $solrDoc->setField('date', date("Y-m-d", strtotime($metadata['date'][0])));
                 // convert any datetime to proper ISO extended datetime format and timezone for SOLR
                 } else {
@@ -549,7 +549,8 @@ 
      *
      * @return \Solarium\Core\Query\DocumentInterface
      */
-    private static function getSolrDocument($updateQuery, $document, $unit, $fullText = '') {
+    private static function getSolrDocument($updateQuery, $document, $unit, $fullText = '')
+    {
         $solrDoc = $updateQuery->createDocument();
         // Create unique identifier from document's UID and unit's XML ID.
         $solrDoc->setField('id', $document->getUid() . $unit['id']);
@@ -573,7 +574,8 @@ 
      *
      * @return array|string
      */
-    private static function removeAppendsFromAuthor($authors) {
+    private static function removeAppendsFromAuthor($authors)
+    {
         if (is_array($authors)) {
             foreach ($authors as $i => $author) {
                 $splitName = explode(chr(31), $author);

Classes/Common/SolrSearch.php

Spacing +5 added lines, -5 removed lines

@@ -212,7 +212,7 @@ 
         // Add filter query for date search
         if (!empty($this->searchParams['dateFrom']) && !empty($this->searchParams['dateTo'])) {
             // combine dateFrom and dateTo into range search
-            $params['filterquery'][]['query'] = '{!join from=' . $fields['uid'] . ' to=' . $fields['uid'] . '}'. $fields['date'] . ':[' . $this->searchParams['dateFrom'] . ' TO ' . $this->searchParams['dateTo'] . ']';
+            $params['filterquery'][]['query'] = '{!join from=' . $fields['uid'] . ' to=' . $fields['uid'] . '}' . $fields['date'] . ':[' . $this->searchParams['dateFrom'] . ' TO ' . $this->searchParams['dateTo'] . ']';
         }
 
         // Add filter query for faceting.
@@ -324,8 +324,8 @@ 
                 }
                 if ($documents[$doc['uid']]) {
                     // translate language code if applicable
-                    if($doc['metadata']['language']) {
-                        foreach($doc['metadata']['language'] as $indexName => $language) {
+                    if ($doc['metadata']['language']) {
+                        foreach ($doc['metadata']['language'] as $indexName => $language) {
                             $doc['metadata']['language'][$indexName] = Helper::getLanguageName($doc['metadata']['language'][$indexName]);
                         }
                     }
@@ -425,8 +425,8 @@ 
 
         foreach ($result['documents'] as $doc) {
             // translate language code if applicable
-            if($doc['metadata']['language']) {
-                foreach($doc['metadata']['language'] as $indexName => $language) {
+            if ($doc['metadata']['language']) {
+                foreach ($doc['metadata']['language'] as $indexName => $language) {
                     $doc['metadata']['language'][$indexName] = Helper::getLanguageName($doc['metadata']['language'][$indexName]);
                 }
             }

Classes/Api/Viaf/Client.php

Braces +2 added lines, -1 removed lines

@@ -77,7 +77,8 @@
      *
      * @return void
      */
-    public function setEndpoint($endpoint) {
+    public function setEndpoint($endpoint)
+    {
         $this->endpoint = $endpoint;
     }