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()

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         }
 
         // Steps for X pages backward / forward. Double page view uses double steps.
-        $basePageSteps = $this->settings['pageStep'] ?: 10;
+        $basePageSteps = $this->settings['pageStep'] ? : 10;
         $pageSteps = $basePageSteps * ($this->requestData['double'] + 1);
 
         $this->view->assign('basePageSteps', $basePageSteps);

Classes/Controller/ToolboxController.php

Indentation +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
      *
      * @return void
      */
-	// TODO(client-side)
+    // TODO(client-side)
     public function annotationtool()
     {
         if (

Classes/Controller/MetadataController.php

Braces +2 added lines, -1 removed lines

@@ -299,7 +299,8 @@
                 // Collect IDs of all logical structures. This is a flattened tree, so the
                 // order also works for rootline configurations.
                 $allIds = [];
-                function getIds($toc, &$output) {
+                function getIds($toc, &$output)
+                {
                     foreach ($toc as $entry) {
                         $output[$entry['id']] = true;
                         if (is_array($entry['children'])) {

Classes/Controller/DocumentController.php

Indentation +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
  */
 class DocumentController extends AbstractController
 {
-   /**
+    /**
      * The main method of the PlugIn
      *
      * @access public

Spacing +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
         //       page: \d+
         //       double: 0|1
 
-        $make = function ($page, $double, $pagegrid) {
+        $make = function($page, $double, $pagegrid) {
             $result = $this->uriBuilder->reset()
                 ->setTargetPageUid($GLOBALS['TSFE']->id)
                 ->setCreateAbsoluteUri(!empty($this->settings['forceAbsoluteUrl']) ? true : false)

Braces +2 added lines, -1 removed lines

@@ -135,7 +135,8 @@
         //       page: \d+
         //       double: 0|1
 
-        $make = function ($page, $double, $pagegrid) {
+        $make = function ($page, $double, $pagegrid)
+        {
             $result = $this->uriBuilder->reset()
                 ->setTargetPageUid($GLOBALS['TSFE']->id)
                 ->setCreateAbsoluteUri(!empty($this->settings['forceAbsoluteUrl']) ? true : false)

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) {