XOOPS / XoopsCore

htdocs/modules/publisher/src/GroupPermHandler.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
  * @author          trabis <lusopoemas@gmail.com>
  * @version         $Id$
  */
-require_once \dirname(__DIR__) . '/include/common.php';
+require_once \dirname(__DIR__).'/include/common.php';
 
 /**
  * Class GroupPermHandler

htdocs/modules/publisher/src/RatingHandler.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
  * @author          trabis <lusopoemas@gmail.com>
  * @version         $Id$
  */
-require_once \dirname(__DIR__) . '/include/common.php';
+require_once \dirname(__DIR__).'/include/common.php';
 
 /**
  * Class RatingHandler

htdocs/modules/publisher/src/rating.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
  * @author          trabis <lusopoemas@gmail.com>
  * @version         $Id$
  */
-require_once \dirname(__DIR__) . '/include/common.php';
+require_once \dirname(__DIR__).'/include/common.php';
 
 /**
  * Class Rating

htdocs/modules/publisher/src/category.php

Spacing +6 added lines, -6 removed lines

@@ -26,7 +26,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once \dirname(__DIR__) . '/include/common.php';
+require_once \dirname(__DIR__).'/include/common.php';
 
 /**
  * Class Category
@@ -133,7 +133,7 @@ 
                 if ($parentObj->notLoaded()) {
                     exit;
                 }
-                $ret = $parentObj->getCategoryPath($withAllLink) . ' > ' . $ret;
+                $ret = $parentObj->getCategoryPath($withAllLink).' > '.$ret;
             }
             $this->_categoryPath = $ret;
         }
@@ -188,10 +188,10 @@ 
     public function getCategoryLink($class = false): ?string
     {
         if ($class) {
-            return "<a class='$class' href='" . $this->getCategoryUrl() . "'>" . $this->getVar('name') . '</a>';
+            return "<a class='$class' href='".$this->getCategoryUrl()."'>".$this->getVar('name').'</a>';
         }
 
-        return "<a href='" . $this->getCategoryUrl() . "'>" . $this->getVar('name') . '</a>';
+        return "<a href='".$this->getCategoryUrl()."'>".$this->getVar('name').'</a>';
     }
 
     /**
@@ -249,7 +249,7 @@ 
             $category['last_title_link'] = $this->getVar('last_title_link', 'n');
         }
         if ('blank.png' !== $this->image()) {
-            $category['image_path'] = Publisher\Utils::getImageDir('category', false) . $this->image();
+            $category['image_path'] = Publisher\Utils::getImageDir('category', false).$this->image();
         } else {
             $category['image_path'] = '';
         }
@@ -272,7 +272,7 @@ 
             $category['last_title_link'] = $this->getVar('last_title_link', 'n');
         }
         if ('blank.png' !== $this->image()) {
-            $category['image_path'] = Publisher\Utils::getImageDir('category', false) . $this->image();
+            $category['image_path'] = Publisher\Utils::getImageDir('category', false).$this->image();
         } else {
             $category['image_path'] = '';
         }

htdocs/modules/publisher/src/ItemHandler.php

Indentation +3 added lines, -3 removed lines

@@ -655,9 +655,9 @@
             'c.parentid',
             'ASC'
         )->addOrderBy(
-                                                                                                                                                                                                                                                                              'i.categoryid',
-                                                                                                                                                                                                                                                                              'ASC'
-                                                                                                                                                                                                                                                                          );
+                                                                                                                                                                                                                                                                                'i.categoryid',
+                                                                                                                                                                                                                                                                                'ASC'
+                                                                                                                                                                                                                                                                            );
         if ((int)$cat_id > 0) {
             $qb->andWhere($qb->expr()->eq('i.categoryid', ':catid'))->setParameter(':catid', $cat_id, PDO::PARAM_INT);
         }

Spacing +13 added lines, -13 removed lines

@@ -32,7 +32,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once \dirname(__DIR__) . '/include/common.php';
+require_once \dirname(__DIR__).'/include/common.php';
 
 /**
  * Items handler class.
@@ -210,7 +210,7 @@ 
             // Categories for which user has access
             $categoriesGranted = $this->helper->getPermissionHandler()->getGrantedItems('category_read');
             if (!empty($categoriesGranted)) {
-                $grantedCategories = new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN');
+                $grantedCategories = new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN');
                 $criteriaPermissions->add($grantedCategories, 'AND');
             } else {
                 return 0;
@@ -357,7 +357,7 @@ 
             // Categories for which user has access
             $categoriesGranted = $this->helper->getPermissionHandler()->getGrantedItems('category_read');
             if (!empty($categoriesGranted)) {
-                $grantedCategories = new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN');
+                $grantedCategories = new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN');
                 $criteriaPermissions->add($grantedCategories, 'AND');
             } else {
                 return [];
@@ -495,7 +495,7 @@ 
         if (\is_array($userid) && \count($userid) > 0) {
             $userid = \array_map('\intval', $userid);
             $criteriaUser = new CriteriaCompo();
-            $criteriaUser->add(new Criteria('uid', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
+            $criteriaUser->add(new Criteria('uid', '('.\implode(',', $userid).')', 'IN'), 'OR');
         } elseif (\is_numeric($userid) && $userid > 0) {
             $criteriaUser = new CriteriaCompo();
             $criteriaUser->add(new Criteria('uid', $userid), 'OR');
@@ -506,19 +506,19 @@ 
             foreach ($queryarray as $iValue) {
                 $criteriaKeyword = new CriteriaCompo();
                 if (\in_array('title', $searchin)) {
-                    $criteriaKeyword->add(new Criteria('title', '%' . $iValue . '%', 'LIKE'), 'OR');
+                    $criteriaKeyword->add(new Criteria('title', '%'.$iValue.'%', 'LIKE'), 'OR');
                 }
                 if (\in_array('subtitle', $searchin)) {
-                    $criteriaKeyword->add(new Criteria('subtitle', '%' . $iValue . '%', 'LIKE'), 'OR');
+                    $criteriaKeyword->add(new Criteria('subtitle', '%'.$iValue.'%', 'LIKE'), 'OR');
                 }
                 if (\in_array('body', $searchin)) {
-                    $criteriaKeyword->add(new Criteria('body', '%' . $iValue . '%', 'LIKE'), 'OR');
+                    $criteriaKeyword->add(new Criteria('body', '%'.$iValue.'%', 'LIKE'), 'OR');
                 }
                 if (\in_array('summary', $searchin)) {
-                    $criteriaKeyword->add(new Criteria('summary', '%' . $iValue . '%', 'LIKE'), 'OR');
+                    $criteriaKeyword->add(new Criteria('summary', '%'.$iValue.'%', 'LIKE'), 'OR');
                 }
                 if (\in_array('meta_keywords', $searchin)) {
-                    $criteriaKeyword->add(new Criteria('meta_keywords', '%' . $iValue . '%', 'LIKE'), 'OR');
+                    $criteriaKeyword->add(new Criteria('meta_keywords', '%'.$iValue.'%', 'LIKE'), 'OR');
                 }
                 $criteriaKeywords->add($criteriaKeyword, $andor);
                 unset($criteriaKeyword);
@@ -534,11 +534,11 @@ 
             if (0 == \count($categoriesGranted)) {
                 return $ret;
             }
-            $grantedCategories = new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN');
+            $grantedCategories = new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN');
             $criteriaPermissions->add($grantedCategories, 'AND');
         } elseif (\count($categories) > 0) {
             $criteriaPermissions = new CriteriaCompo();
-            $grantedCategories = new Criteria('categoryid', '(' . \implode(',', $categories) . ')', 'IN');
+            $grantedCategories = new Criteria('categoryid', '('.\implode(',', $categories).')', 'IN');
             $criteriaPermissions->add($grantedCategories, 'AND');
         }
         $criteriaItemsStatus = new CriteriaCompo();
@@ -603,7 +603,7 @@ 
 
         $subqb = $this->db2->createXoopsQueryBuilder();
         $subqb->select('categoryid', 'MAX(datesub) AS date')->fromPrefix('publisher_items', '')->where($subqb->expr()->in('status', $status))->andWhere($subqb->expr()->in('categoryid', $catIds))->groupBy('categoryid');
-        $subquery = '(' . $subqb->getSQL() . ')';
+        $subquery = '('.$subqb->getSQL().')';
 
         $qb->from($subquery, 'mo')->joinPrefix('mo', 'publisher_items', 'mi', 'mi.datesub = mo.date');
 
@@ -626,7 +626,7 @@ 
      */
     public function countArticlesByCat($parentid, &$catsCount, $spaces = '', $resultCatCounts = []): int
     {
-        $newspaces = $spaces . '--';
+        $newspaces = $spaces.'--';
         $thecount = 0;
         foreach ($catsCount[$parentid] as $subCatId => $count) {
             $thecount = $thecount + $count;

htdocs/modules/publisher/src/item.php

Spacing +24 added lines, -24 removed lines

@@ -37,7 +37,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once \dirname(__DIR__) . '/include/common.php';
+require_once \dirname(__DIR__).'/include/common.php';
 
 /**
  * Class Item
@@ -203,7 +203,7 @@ 
     public function wrappage($file_name): string
     {
         $content = '';
-        $page = Publisher\Utils::getUploadDir(true, 'content') . $file_name;
+        $page = Publisher\Utils::getUploadDir(true, 'content').$file_name;
         if (XoopsLoad::fileExists($page)) {
             // this page uses smarty template
             \ob_start();
@@ -257,7 +257,7 @@ 
         if ($this->helper->getConfig('item_disp_blocks_summary')) {
             $summary = $this->summary($maxLength, $format, $stripTags);
             if ($summary) {
-                $ret = $this->summary() . '<br><br>' . $ret;
+                $ret = $this->summary().'<br><br>'.$ret;
             }
         }
         if (!empty($stripTags)) {
@@ -388,7 +388,7 @@ 
 
     public function getCategoryImagePath(): string
     {
-        return Publisher\Utils::getImageDir('category', false) . $this->category()->image();
+        return Publisher\Utils::getImageDir('category', false).$this->category()->image();
     }
 
     /**
@@ -407,35 +407,35 @@ 
             if (Publisher\Utils::IsUserAdmin() || Publisher\Utils::IsUserAuthor($this) || Publisher\Utils::IsUserModerator($this)) {
                 if ($this->helper->getConfig('perm_edit') || Publisher\Utils::IsUserModerator($this) || Publisher\Utils::IsUserAdmin()) {
                     // Edit button
-                    $adminLinks .= "<a href='" . \PUBLISHER_URL . '/submit.php?itemid=' . $this->getVar('itemid') . "'><img src='" . \PUBLISHER_URL . "/images/links/edit.gif'" . " title='" . _CO_PUBLISHER_EDIT . "' alt='" . _CO_PUBLISHER_EDIT . "'></a>";
+                    $adminLinks .= "<a href='".\PUBLISHER_URL.'/submit.php?itemid='.$this->getVar('itemid')."'><img src='".\PUBLISHER_URL."/images/links/edit.gif'"." title='"._CO_PUBLISHER_EDIT."' alt='"._CO_PUBLISHER_EDIT."'></a>";
                     $adminLinks .= ' ';
                 }
                 if ($this->helper->getConfig('perm_delete') || Publisher\Utils::IsUserModerator($this) || Publisher\Utils::IsUserAdmin()) {
                     // Delete button
-                    $adminLinks .= "<a href='" . \PUBLISHER_URL . '/submit.php?op=del&amp;itemid=' . $this->getVar('itemid') . "'><img src='" . \PUBLISHER_URL . "/images/links/delete.png'" . " title='" . _CO_PUBLISHER_DELETE . "' alt='" . _CO_PUBLISHER_DELETE . "'></a>";
+                    $adminLinks .= "<a href='".\PUBLISHER_URL.'/submit.php?op=del&amp;itemid='.$this->getVar('itemid')."'><img src='".\PUBLISHER_URL."/images/links/delete.png'"." title='"._CO_PUBLISHER_DELETE."' alt='"._CO_PUBLISHER_DELETE."'></a>";
                     $adminLinks .= ' ';
                 }
             }
             if ($this->helper->getConfig('perm_clone') || Publisher\Utils::IsUserModerator($this) || Publisher\Utils::IsUserAdmin()) {
                 // Duplicate button
-                $adminLinks .= "<a href='" . \PUBLISHER_URL . '/submit.php?op=clone&amp;itemid=' . $this->getVar('itemid') . "'><img src='" . \PUBLISHER_URL . "/images/links/clone.gif'" . " title='" . _CO_PUBLISHER_CLONE . "' alt='" . _CO_PUBLISHER_CLONE . "'></a>";
+                $adminLinks .= "<a href='".\PUBLISHER_URL.'/submit.php?op=clone&amp;itemid='.$this->getVar('itemid')."'><img src='".\PUBLISHER_URL."/images/links/clone.gif'"." title='"._CO_PUBLISHER_CLONE."' alt='"._CO_PUBLISHER_CLONE."'></a>";
                 $adminLinks .= ' ';
             }
         }
         // PDF button
         if ($xoops->service('htmltopdf')->isAvailable()) {
-            $adminLinks .= "<a href='" . \PUBLISHER_URL . '/makepdf.php?itemid=' . $this->getVar('itemid') . "' rel='nofollow' target='_blank'><img src='" . \PUBLISHER_URL . "/images/links/pdf.gif' title='" . _CO_PUBLISHER_PDF . "' alt='" . _CO_PUBLISHER_PDF . "'></a>";
+            $adminLinks .= "<a href='".\PUBLISHER_URL.'/makepdf.php?itemid='.$this->getVar('itemid')."' rel='nofollow' target='_blank'><img src='".\PUBLISHER_URL."/images/links/pdf.gif' title='"._CO_PUBLISHER_PDF."' alt='"._CO_PUBLISHER_PDF."'></a>";
             $adminLinks .= ' ';
         }
         // Print button
-        $adminLinks .= "<a href='" . Publisher\Utils::seoGenUrl('print', $this->getVar('itemid'), $this->getVar('short_url')) . "' rel='nofollow' target='_blank'><img src='" . \PUBLISHER_URL . "/images/links/print.gif' title='" . _CO_PUBLISHER_PRINT . "' alt='" . _CO_PUBLISHER_PRINT . "'></a>";
+        $adminLinks .= "<a href='".Publisher\Utils::seoGenUrl('print', $this->getVar('itemid'), $this->getVar('short_url'))."' rel='nofollow' target='_blank'><img src='".\PUBLISHER_URL."/images/links/print.gif' title='"._CO_PUBLISHER_PRINT."' alt='"._CO_PUBLISHER_PRINT."'></a>";
         $adminLinks .= ' ';
         // Email button
         if ($xoops->isActiveModule('tellafriend')) {
             $subject = \sprintf(_CO_PUBLISHER_INTITEMFOUND, $xoops->getConfig('sitename'));
             $subject = $this->_convert_for_japanese($subject);
             $maillink = Publisher\Utils::tellafriend($subject);
-            $adminLinks .= '<a href="' . $maillink . '"><img src="' . \PUBLISHER_URL . '/images/links/friend.gif" title="' . _CO_PUBLISHER_MAIL . '" alt="' . _CO_PUBLISHER_MAIL . '"></a>';
+            $adminLinks .= '<a href="'.$maillink.'"><img src="'.\PUBLISHER_URL.'/images/links/friend.gif" title="'._CO_PUBLISHER_MAIL.'" alt="'._CO_PUBLISHER_MAIL.'"></a>';
             $adminLinks .= ' ';
         }
 
@@ -454,19 +454,19 @@ 
             $tags['MODULE_NAME'] = $this->helper->getModule()->getVar('name');
             $tags['ITEM_NAME'] = $this->title();
             $tags['CATEGORY_NAME'] = $this->getCategoryName();
-            $tags['CATEGORY_URL'] = \PUBLISHER_URL . '/category.php?categoryid=' . $this->getVar('categoryid');
+            $tags['CATEGORY_URL'] = \PUBLISHER_URL.'/category.php?categoryid='.$this->getVar('categoryid');
             $tags['ITEM_BODY'] = $this->body();
             $tags['DATESUB'] = $this->datesub();
             foreach ($notifications as $notification) {
                 switch ($notification) {
                     case \_PUBLISHER_NOT_ITEM_PUBLISHED:
-                        $tags['ITEM_URL'] = \PUBLISHER_URL . '/item.php?itemid=' . $this->getVar('itemid');
+                        $tags['ITEM_URL'] = \PUBLISHER_URL.'/item.php?itemid='.$this->getVar('itemid');
                         $notificationHandler->triggerEvent('global', 0, 'published', $tags, [], $this->helper->getModule()->getVar('mid'));
                         $notificationHandler->triggerEvent('category', $this->getVar('categoryid'), 'published', $tags, [], $this->helper->getModule()->getVar('mid'));
                         $notificationHandler->triggerEvent('item', $this->getVar('itemid'), 'approved', $tags, [], $this->helper->getModule()->getVar('mid'));
                         break;
                     case \_PUBLISHER_NOT_ITEM_SUBMITTED:
-                        $tags['WAITINGFILES_URL'] = \PUBLISHER_URL . '/admin/item.php?itemid=' . $this->getVar('itemid');
+                        $tags['WAITINGFILES_URL'] = \PUBLISHER_URL.'/admin/item.php?itemid='.$this->getVar('itemid');
                         $notificationHandler->triggerEvent('global', 0, 'submitted', $tags, [], $this->helper->getModule()->getVar('mid'));
                         $notificationHandler->triggerEvent('category', $this->getVar('categoryid'), 'submitted', $tags, [], $this->helper->getModule()->getVar('mid'));
                         break;
@@ -500,10 +500,10 @@ 
     public function getItemLink($class = false, $maxsize = 0): ?string
     {
         if ($class) {
-            return '<a class=' . $class . ' href="' . $this->getItemUrl() . '">' . $this->title($maxsize) . '</a>';
+            return '<a class='.$class.' href="'.$this->getItemUrl().'">'.$this->title($maxsize).'</a>';
         }
 
-        return '<a href="' . $this->getItemUrl() . '">' . $this->title($maxsize) . '</a>';
+        return '<a href="'.$this->getItemUrl().'">'.$this->title($maxsize).'</a>';
     }
 
     public function getWhoAndWhen(): string
@@ -583,7 +583,7 @@ 
             $imageObjs = [];
             if (\count($images_ids) > 0) {
                 $imageHandler = Images::getInstance()->getHandlerImages();
-                $criteria = new CriteriaCompo(new Criteria('image_id', '(' . \implode(',', $images_ids) . ')', 'IN'));
+                $criteria = new CriteriaCompo(new Criteria('image_id', '('.\implode(',', $images_ids).')', 'IN'));
                 $imageObjs = $imageHandler->getObjects($criteria, true);
                 unset($criteria);
             }
@@ -698,12 +698,12 @@ 
         if (\is_object($images['main'])) {
             /* @var \ImagesImage $image */
             $image = $images['main'];
-            $dimensions = \getimagesize(XoopsBaseConfig::get('root-path') . '/uploads/' . $image->getVar('image_name'));
+            $dimensions = \getimagesize(XoopsBaseConfig::get('root-path').'/uploads/'.$image->getVar('image_name'));
             $item['image_width'] = $dimensions[0];
             $item['image_height'] = $dimensions[1];
-            $item['image_path'] = XoopsBaseConfig::get('url') . '/uploads/' . $image->getVar('image_name');
+            $item['image_path'] = XoopsBaseConfig::get('url').'/uploads/'.$image->getVar('image_name');
             // pass this on since some consumers build custom thumbnails
-            $item['image_vpath'] = 'uploads/' . $image->getVar('image_name');
+            $item['image_vpath'] = 'uploads/'.$image->getVar('image_name');
             $item['image_thumb'] = Xoops::getInstance()->service('thumbnail')->getImgUrl($item['image_vpath'], 0, 180)->getValue();
             $item['image_name'] = $image->getVar('image_nicename');
         }
@@ -722,11 +722,11 @@ 
         $i = 0;
         /* @var \ImagesImage $image */
         foreach ($images['others'] as $image) {
-            $dimensions = \getimagesize(XoopsBaseConfig::get('root-path') . '/uploads/' . $image->getVar('image_name'));
+            $dimensions = \getimagesize(XoopsBaseConfig::get('root-path').'/uploads/'.$image->getVar('image_name'));
             $item['images'][$i]['width'] = $dimensions[0];
             $item['images'][$i]['height'] = $dimensions[1];
-            $item['images'][$i]['path'] = XoopsBaseConfig::get('url') . '/uploads/' . $image->getVar('image_name');
-            $item['images'][$i]['thumb'] = $thumbService->getImgUrl('uploads/' . $image->getVar('image_name'), 240, 0)->getValue();
+            $item['images'][$i]['path'] = XoopsBaseConfig::get('url').'/uploads/'.$image->getVar('image_name');
+            $item['images'][$i]['thumb'] = $thumbService->getImgUrl('uploads/'.$image->getVar('image_name'), 240, 0)->getValue();
             $item['images'][$i]['name'] = $image->getVar('image_nicename');
             ++$i;
         }
@@ -744,9 +744,9 @@ 
     {
         $color = $this->helper->getConfig('format_highlight_color');
         if (0 !== \mb_strpos($color, '#')) {
-            $color = '#' . $color;
+            $color = '#'.$color;
         }
-        $pre = '<span style="font-weight: bolder; background-color: ' . $color . ';">';
+        $pre = '<span style="font-weight: bolder; background-color: '.$color.';">';
         $post = '</span>';
 
         return Highlighter::apply($keywords, $content, $pre, $post);

htdocs/modules/publisher/src/CategoryHandler.php

Spacing +8 added lines, -8 removed lines

@@ -29,7 +29,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once \dirname(__DIR__) . '/include/common.php';
+require_once \dirname(__DIR__).'/include/common.php';
 
 /**
  * Categories handler class.
@@ -160,7 +160,7 @@ 
         if (!Publisher\Utils::IsUserAdmin()) {
             $categoriesGranted = $this->helper->getPermissionHandler()->getGrantedItems('category_read');
             if (\count($categoriesGranted) > 0) {
-                $criteria->add(new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN'));
+                $criteria->add(new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN'));
             } else {
                 return $ret;
             }
@@ -190,7 +190,7 @@ 
         for ($j = 0; $j < $level; ++$j) {
             $spaces .= '--';
         }
-        $theresult[$category['categoryid']] = $spaces . $category['name'];
+        $theresult[$category['categoryid']] = $spaces.$category['name'];
         if (isset($cat_array[$category['categoryid']])) {
             $level = $level + 1;
             foreach ($cat_array[$category['categoryid']] as $cat) {
@@ -210,7 +210,7 @@ 
         if (!Publisher\Utils::IsUserAdmin()) {
             $categoriesGranted = $this->helper->getPermissionHandler()->getGrantedItems('item_submit');
             if (\count($categoriesGranted) > 0) {
-                $criteria->add(new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN'));
+                $criteria->add(new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN'));
             } else {
                 return $ret;
             }
@@ -250,7 +250,7 @@ 
         if (!Publisher\Utils::IsUserAdmin()) {
             $categoriesGranted = $this->helper->getPermissionHandler()->getGrantedItems('category_read');
             if (\count($categoriesGranted) > 0) {
-                $criteria->add(new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN'));
+                $criteria->add(new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN'));
             } else {
                 return $ret;
             }
@@ -294,7 +294,7 @@ 
             if (!Publisher\Utils::IsUserAdmin()) {
                 $categoriesGranted = $this->helper->getPermissionHandler()->getGrantedItems('category_read');
                 if (\count($categoriesGranted) > 0) {
-                    $criteria->add(new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN'));
+                    $criteria->add(new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN'));
                 } else {
                     return 0;
                 }
@@ -315,12 +315,12 @@ 
     public function &getSubCats($categories): array
     {
         $xoops = Xoops::getInstance();
-        $criteria = new CriteriaCompo(new Criteria('parentid', '(' . \implode(',', \array_keys($categories)) . ')', 'IN'));
+        $criteria = new CriteriaCompo(new Criteria('parentid', '('.\implode(',', \array_keys($categories)).')', 'IN'));
         $ret = [];
         if (!Publisher\Utils::IsUserAdmin()) {
             $categoriesGranted = $this->helper->getPermissionHandler()->getGrantedItems('category_read');
             if (\count($categoriesGranted) > 0) {
-                $criteria->add(new Criteria('categoryid', '(' . \implode(',', $categoriesGranted) . ')', 'IN'));
+                $criteria->add(new Criteria('categoryid', '('.\implode(',', $categoriesGranted).')', 'IN'));
             } else {
                 return $ret;
             }

htdocs/modules/publisher/src/plugin/NotificationsPlugin.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
 use Xoops\Module\Plugin\PluginAbstract;
 use XoopsModules\Publisher;
 
-require_once \dirname(\dirname(__DIR__)) . '/include/common.php';
+require_once \dirname(\dirname(__DIR__)).'/include/common.php';
 
 /**
  * Class NotificationsPlugin
@@ -57,7 +57,7 @@ 
 
         if ('category' === $category) {
             // Assume we have a valid category id
-            $sql = 'SELECT name, short_url FROM ' . $xoopsDB->prefix('publisher_categories') . ' WHERE categoryid  = ' . $item_id;
+            $sql = 'SELECT name, short_url FROM '.$xoopsDB->prefix('publisher_categories').' WHERE categoryid  = '.$item_id;
             $result = $xoopsDB->query($sql); // TODO: error check
             $result_array = $xoopsDB->fetchArray($result);
             $item['name'] = $result_array['name'];
@@ -68,7 +68,7 @@ 
 
         if ('item' === $category) {
             // Assume we have a valid story id
-            $sql = 'SELECT title, short_url FROM ' . $xoopsDB->prefix('publisher_items') . ' WHERE itemid = ' . $item_id;
+            $sql = 'SELECT title, short_url FROM '.$xoopsDB->prefix('publisher_items').' WHERE itemid = '.$item_id;
             $result = $xoopsDB->query($sql); // TODO: error check
             $result_array = $xoopsDB->fetchArray($result);
             $item['name'] = $result_array['title'];

htdocs/modules/publisher/src/plugin/SearchPlugin.php

Spacing +3 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
             $hightlight_key = '';
         } else {
             $keywords = \implode('+', $queryarray);
-            $hightlight_key = '&keywords=' . $keywords;
+            $hightlight_key = '&keywords='.$keywords;
         }
         $itemsObjs = $helper->getItemHandler()->getItemsFromSearch($queryarray, $andor, $limit, $offset, $userid, $categories, $sortby, $searchin, $extra);
         $withCategoryPath = $helper->getConfig('search_cat_path');
@@ -79,9 +79,9 @@ 
         foreach ($itemsObjs as $obj) {
             $item['image'] = 'images/item_icon.gif';
             $item['link'] = $obj->getItemUrl();
-            $item['link'] .= (!empty($hightlight_key) && (false === \mb_strpos($item['link'], '.php?'))) ? '?' . \ltrim($hightlight_key, '&') : $hightlight_key;
+            $item['link'] .= (!empty($hightlight_key) && (false === \mb_strpos($item['link'], '.php?'))) ? '?'.\ltrim($hightlight_key, '&') : $hightlight_key;
             if ($withCategoryPath) {
-                $item['title'] = $obj->getCategoryPath(false) . ' > ' . $obj->title();
+                $item['title'] = $obj->getCategoryPath(false).' > '.$obj->title();
             } else {
                 $item['title'] = $obj->title();
             }