XOOPS / XoopsCore

htdocs/modules/publisher/include/common.php

Spacing +9 added lines, -9 removed lines

@@ -22,18 +22,18 @@
 
 $xoops = \Xoops::getInstance();
 define("PUBLISHER_DIRNAME", basename(dirname(__DIR__)));
-define("PUBLISHER_URL", $xoops->url('modules/' . PUBLISHER_DIRNAME));
-define("PUBLISHER_ADMIN_URL", PUBLISHER_URL . '/admin');
-define("PUBLISHER_UPLOADS_URL", $xoops->url('uploads/' . PUBLISHER_DIRNAME));
-define("PUBLISHER_ROOT_PATH", $xoops->path('modules/' . PUBLISHER_DIRNAME));
-define("PUBLISHER_UPLOADS_PATH", $xoops->path('uploads/' . PUBLISHER_DIRNAME));
+define("PUBLISHER_URL", $xoops->url('modules/'.PUBLISHER_DIRNAME));
+define("PUBLISHER_ADMIN_URL", PUBLISHER_URL.'/admin');
+define("PUBLISHER_UPLOADS_URL", $xoops->url('uploads/'.PUBLISHER_DIRNAME));
+define("PUBLISHER_ROOT_PATH", $xoops->path('modules/'.PUBLISHER_DIRNAME));
+define("PUBLISHER_UPLOADS_PATH", $xoops->path('uploads/'.PUBLISHER_DIRNAME));
 
 $path = dirname(__DIR__);
 XoopsLoad::addMap(array(
-    'publishermetagen'   => $path . '/class/metagen.php',
-    'publisher'          => $path . '/class/helper.php',
-    'publisherutils'     => $path . '/class/utils.php',
-    'publisherblockform' => $path . '/class/blockform.php',
+    'publishermetagen'   => $path.'/class/metagen.php',
+    'publisher'          => $path.'/class/helper.php',
+    'publisherutils'     => $path.'/class/utils.php',
+    'publisherblockform' => $path.'/class/blockform.php',
 ));
 
 $publisher = Publisher::getInstance();

htdocs/modules/publisher/include/search.inc.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
         $hightlight_key = '';
     } else {
         $keywords = implode('+', $queryarray);
-        $hightlight_key = "&keywords=" . $keywords;
+        $hightlight_key = "&keywords=".$keywords;
     }
     $itemsObjs = $publisher->getItemHandler()
             ->getItemsFromSearch($queryarray, $andor, $limit, $offset, $userid, $categories, $sortby, $searchin, $extra);
@@ -36,9 +36,9 @@ 
     foreach ($itemsObjs as $obj) {
         $item['image'] = "images/item_icon.gif";
         $item['link'] = $obj->getItemUrl();
-        $item['link'] .= (!empty($hightlight_key) && (strpos($item['link'], '.php?') === false)) ? "?" . ltrim($hightlight_key, '&') : $hightlight_key;
+        $item['link'] .= (!empty($hightlight_key) && (strpos($item['link'], '.php?') === false)) ? "?".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();
         }
@@ -56,7 +56,7 @@ 
             $start = max(($pos - 100), 0);
             $length = strlen($query) + 200; //xoops_local("strlen", $query) + 200;
             $context = $obj->highlight(XoopsLocale::substr($text, $start, $length, " [...]"), $query);
-            $sanitized_text .= "<p>[...] " . $context . "</p>";
+            $sanitized_text .= "<p>[...] ".$context."</p>";
         }
 
         //End of highlight

htdocs/modules/publisher/backend.php

Indentation +5 added lines, -5 removed lines

@@ -77,11 +77,11 @@
         /* @var $item PublisherItem */
         foreach ($sarray as $item) {
             $tpl->append('items',
-                         array('title' => htmlspecialchars($item->title(), ENT_QUOTES),
-                               'link' => $item->getItemUrl(),
-                               'guid' => $item->getItemUrl(),
-                               'pubdate' => XoopsLocale::formatTimestamp($item->getVar('datesub'), 'rss'),
-                               'description' => htmlspecialchars($item->getBlockSummary(300, true), ENT_QUOTES)));
+                            array('title' => htmlspecialchars($item->title(), ENT_QUOTES),
+                                'link' => $item->getItemUrl(),
+                                'guid' => $item->getItemUrl(),
+                                'pubdate' => XoopsLocale::formatTimestamp($item->getVar('datesub'), 'rss'),
+                                'description' => htmlspecialchars($item->getBlockSummary(300, true), ENT_QUOTES)));
         }
     }
 }

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $xoops->disableErrorReporting();
 
@@ -36,10 +36,10 @@ 
     $categoryObj = $publisher->getCategoryHandler()->get($categoryid);
 }
 
-header('Content-Type:text/xml; charset=' . XoopsLocale::getCharset());
+header('Content-Type:text/xml; charset='.XoopsLocale::getCharset());
 $tpl = new XoopsTpl();
 $tpl->caching = 2;
-$tpl->cache_lifetime= 0;
+$tpl->cache_lifetime = 0;
 
 $myts = \Xoops\Core\Text\Sanitizer::getInstance();
 if (!$tpl->isCached('module:publisher/publisher_rss.tpl')) {
@@ -53,14 +53,14 @@ 
     $tpl->assign('channel_editor', $xoops->getConfig('adminmail'));
 
     if ($categoryid != -1) {
-        $channel_category .= " > " . $categoryObj->getVar('name');
+        $channel_category .= " > ".$categoryObj->getVar('name');
     }
 
     $tpl->assign('channel_category', htmlspecialchars($channel_category));
     $tpl->assign('channel_generator', $publisher->getModule()->getVar('name'));
     $tpl->assign('channel_language', XoopsLocale::getLangCode());
-    $tpl->assign('image_url', \XoopsBaseConfig::get('url') . '/images/logo.gif');
-    $dimention = getimagesize(\XoopsBaseConfig::get('root-path') . '/images/logo.gif');
+    $tpl->assign('image_url', \XoopsBaseConfig::get('url').'/images/logo.gif');
+    $dimention = getimagesize(\XoopsBaseConfig::get('root-path').'/images/logo.gif');
     if (empty($dimention[0])) {
         $width = 140;
         $height = 140;

htdocs/modules/publisher/visit.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
  * @version         $Id$
  */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $publisher = Publisher::getInstance();
@@ -49,10 +49,10 @@ 
 $fileObj->updateCounter();
 
 if (!preg_match("/^ed2k*:\/\//i", $fileObj->getFileUrl())) {
-    header("Location: " . $fileObj->getFileUrl());
+    header("Location: ".$fileObj->getFileUrl());
 }
 
 $myts = \Xoops\Core\Text\Sanitizer::getInstance();
 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL="
-    . $myts->htmlSpecialChars($fileObj->getFileUrl()) . "\"></head><body></body></html>";
+    . $myts->htmlSpecialChars($fileObj->getFileUrl())."\"></head><body></body></html>";
 exit();

htdocs/modules/publisher/makepdf.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
  * @author          Sina Asghari (AKA stranger) <stranger@impresscms.ir>
  */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $xoops->disableErrorReporting();
 
@@ -70,6 +70,6 @@ 
 $xoops->service('htmltopdf')->setKeywords($itemObj->getVar('meta_keywords'));
 $xoops->service('htmltopdf')->setSubject($categoryObj->getVar('name'));
 $xoops->service('htmltopdf')->addHtml($content);
-$name = $itemObj->getVar('short_url') . '.pdf';
+$name = $itemObj->getVar('short_url').'.pdf';
 $xoops->service('htmltopdf')->outputPdfInline($name);
 exit();

htdocs/modules/publisher/blocks/items_columns.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 /**
  * Function To Show Publisher Items From Categories In Their Own Columns
@@ -133,7 +133,7 @@ 
     $block['columns'] = $columns;
     $block['columnwidth'] = (int)(100 / $opt_num_columns);
 
-    $xoTheme->addStylesheet(\XoopsBaseConfig::get('url') . '/modules/' . PUBLISHER_DIRNAME . '/css/publisher.css');
+    $xoTheme->addStylesheet(\XoopsBaseConfig::get('url').'/modules/'.PUBLISHER_DIRNAME.'/css/publisher.css');
 
     return $block;
 }

htdocs/modules/publisher/blocks/date_to_date.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_date_to_date_show($options)
 {
@@ -59,7 +59,7 @@ 
         $block['lang_poster'] = _MB_PUBLISHER_POSTEDBY;
         $block['lang_date'] = _MB_PUBLISHER_DATE;
         $modulename = $myts->displayTarea($publisher->getModule()->getVar('name'));
-        $block['lang_visitItem'] = _MB_PUBLISHER_VISITITEM . " " . $modulename;
+        $block['lang_visitItem'] = _MB_PUBLISHER_VISITITEM." ".$modulename;
         $block['lang_articles_from_to'] = sprintf(_MB_PUBLISHER_ARTICLES_FROM_TO, $options[0], $options[1]);
     }
 

htdocs/modules/publisher/blocks/items_new.php

Spacing +6 added lines, -6 removed lines

@@ -22,7 +22,7 @@ 
 
 defined("XOOPS_ROOT_PATH") or die("XOOPS root path not defined");
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_items_new_show($options)
 {
@@ -48,7 +48,7 @@ 
         $criteria = null;
     } else {
         $criteria = new CriteriaCompo();
-        $criteria->add(new Criteria('categoryid', '(' . $options[0] . ')', 'IN'));
+        $criteria->add(new Criteria('categoryid', '('.$options[0].')', 'IN'));
     }
     $xoops = \Xoops::getInstance();
     $thumbService = $xoops->service('thumbnail');
@@ -64,12 +64,12 @@ 
             $item['poster'] = $itemsObj[$i]->posterName(); // for make poster name linked, use linkedPosterName() instead of posterName()
 
             if ($image === 'article') {
-                $item['image'] = \XoopsBaseConfig::get('url') . '/uploads/blank.gif';
+                $item['image'] = \XoopsBaseConfig::get('url').'/uploads/blank.gif';
                 $item['image_name'] = '';
                 $images = $itemsObj[$i]->getImages();
                 if (is_object($images['main'])) {
                     $item['image'] = $thumbService
-                        ->getImgUrl('uploads/' . $images['main']->getVar('image_name'), 50, 0)
+                        ->getImgUrl('uploads/'.$images['main']->getVar('image_name'), 50, 0)
                         ->getValue();
 
                     $item['image_name'] = $images['main']->getVar('image_nicename');
@@ -80,11 +80,11 @@ 
             } elseif ($image === 'avatar') {
                 $auid = $itemsObj[$i]->getVar('uid');
                 if ($auid == '0') {
-                    $item['image'] = \XoopsBaseConfig::get('url') . '/uploads/blank.gif';
+                    $item['image'] = \XoopsBaseConfig::get('url').'/uploads/blank.gif';
                     $images = $itemsObj[$i]->getImages();
                     if (is_object($images['main'])) {
                         $item['image'] = $thumbService
-                            ->getImgUrl('uploads/' . $images['main']->getVar('image_name'), 50, 0)
+                            ->getImgUrl('uploads/'.$images['main']->getVar('image_name'), 50, 0)
                             ->getValue();
                     }
                 } else {

htdocs/modules/publisher/blocks/search.php

Spacing +17 added lines, -17 removed lines

@@ -20,7 +20,7 @@ 
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_search_show($options)
 {
@@ -52,13 +52,13 @@ 
     $type_select = "<select name=\"andor\">";
     $type_select .= "<option value=\"OR\"";
     if ("OR" === $andor) $type_select .= " selected=\"selected\"";
-    $type_select .= ">" . XoopsLocale::ANY_OR . "</option>";
+    $type_select .= ">".XoopsLocale::ANY_OR."</option>";
     $type_select .= "<option value=\"AND\"";
     if ("AND" === $andor) $type_select .= " selected=\"selected\"";
-    $type_select .= ">" . XoopsLocale::ALL . "</option>";
+    $type_select .= ">".XoopsLocale::ALL."</option>";
     $type_select .= "<option value=\"EXACT\"";
     if ("exact" === $andor) $type_select .= " selected=\"selected\"";
-    $type_select .= ">" . XoopsLocale::EXACT_MATCH . "</option>";
+    $type_select .= ">".XoopsLocale::EXACT_MATCH."</option>";
     $type_select .= "</select>";
 
     /* category */
@@ -66,11 +66,11 @@ 
     $select_category = "<select name=\"category[]\" size=\"5\" multiple=\"multiple\" width=\"150\" style=\"width:150px;\">";
     $select_category .= "<option value=\"all\"";
     if (empty($category) || count($category) == 0) $select_category .= "selected=\"selected\"";
-    $select_category .= ">" . XoopsLocale::ALL . "</option>";
+    $select_category .= ">".XoopsLocale::ALL."</option>";
     foreach ($categories as $id => $cat) {
-        $select_category .= "<option value=\"" . $id . "\"";
+        $select_category .= "<option value=\"".$id."\"";
         if (in_array($id, $category)) $select_category .= "selected=\"selected\"";
-        $select_category .= ">" . $cat . "</option>";
+        $select_category .= ">".$cat."</option>";
     }
     $select_category .= "</select>";
 
@@ -78,37 +78,37 @@ 
     $searchin_select = "";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"title\"";
     if (in_array("title", $searchin)) $searchin_select .= " checked";
-    $searchin_select .= " />" . _CO_PUBLISHER_TITLE . "&nbsp;&nbsp;";
+    $searchin_select .= " />"._CO_PUBLISHER_TITLE."&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"subtitle\"";
     if (in_array("subtitle", $searchin)) $searchin_select .= " checked";
-    $searchin_select .= " />" . _CO_PUBLISHER_SUBTITLE . "&nbsp;&nbsp;";
+    $searchin_select .= " />"._CO_PUBLISHER_SUBTITLE."&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"summary\"";
     if (in_array("summary", $searchin)) $searchin_select .= " checked";
-    $searchin_select .= " />" . _CO_PUBLISHER_SUMMARY . "&nbsp;&nbsp;";
+    $searchin_select .= " />"._CO_PUBLISHER_SUMMARY."&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"text\"";
     if (in_array("body", $searchin)) $searchin_select .= " checked";
-    $searchin_select .= " />" . _CO_PUBLISHER_BODY . "&nbsp;&nbsp;";
+    $searchin_select .= " />"._CO_PUBLISHER_BODY."&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"keywords\"";
     if (in_array("meta_keywords", $searchin)) $searchin_select .= " checked";
-    $searchin_select .= " />" . _CO_PUBLISHER_ITEM_META_KEYWORDS . "&nbsp;&nbsp;";
+    $searchin_select .= " />"._CO_PUBLISHER_ITEM_META_KEYWORDS."&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"all\"";
     if (in_array("all", $searchin) || empty($searchin)) $searchin_select .= " checked";
-    $searchin_select .= " />" . XoopsLocale::ALL . "&nbsp;&nbsp;";
+    $searchin_select .= " />".XoopsLocale::ALL."&nbsp;&nbsp;";
 
     /* sortby */
     $sortby_select = "<select name=\"sortby\">";
     $sortby_select .= "<option value=\"itemid\"";
     if ("itemid" === $sortby || empty($sortby)) $sortby_select .= " selected=\"selected\"";
-    $sortby_select .= ">" . XoopsLocale::NONE . "</option>";
+    $sortby_select .= ">".XoopsLocale::NONE."</option>";
     $sortby_select .= "<option value=\"datesub\"";
     if ("datesub" === $sortby) $sortby_select .= " selected=\"selected\"";
-    $sortby_select .= ">" . _CO_PUBLISHER_DATESUB . "</option>";
+    $sortby_select .= ">"._CO_PUBLISHER_DATESUB."</option>";
     $sortby_select .= "<option value=\"title\"";
     if ("title" === $sortby) $sortby_select .= " selected=\"selected\"";
-    $sortby_select .= ">" . _CO_PUBLISHER_TITLE . "</option>";
+    $sortby_select .= ">"._CO_PUBLISHER_TITLE."</option>";
     $sortby_select .= "<option value=\"categoryid\"";
     if ("categoryid" === $sortby) $sortby_select .= " selected=\"selected\"";
-    $sortby_select .= ">" . _CO_PUBLISHER_CATEGORY . "</option>";
+    $sortby_select .= ">"._CO_PUBLISHER_CATEGORY."</option>";
     $sortby_select .= "</select>";
 
     $block["type_select"] = $type_select;

Braces +48 added lines, -16 removed lines

@@ -28,7 +28,9 @@ 
     $xoops = Xoops::getInstance();
     $publisher = Publisher::getInstance();
     $categories = $publisher->getCategoryHandler()->getCategoriesForSearch();
-    if (count($categories) == 0) return $block;
+    if (count($categories) == 0) {
+        return $block;
+    }
 
     $andor = isset($_POST["andor"]) ? $_POST["andor"] : (isset($_GET["andor"]) ? $_GET["andor"] : "");
 
@@ -51,13 +53,19 @@ 
     /* type */
     $type_select = "<select name=\"andor\">";
     $type_select .= "<option value=\"OR\"";
-    if ("OR" === $andor) $type_select .= " selected=\"selected\"";
+    if ("OR" === $andor) {
+        $type_select .= " selected=\"selected\"";
+    }
     $type_select .= ">" . XoopsLocale::ANY_OR . "</option>";
     $type_select .= "<option value=\"AND\"";
-    if ("AND" === $andor) $type_select .= " selected=\"selected\"";
+    if ("AND" === $andor) {
+        $type_select .= " selected=\"selected\"";
+    }
     $type_select .= ">" . XoopsLocale::ALL . "</option>";
     $type_select .= "<option value=\"EXACT\"";
-    if ("exact" === $andor) $type_select .= " selected=\"selected\"";
+    if ("exact" === $andor) {
+        $type_select .= " selected=\"selected\"";
+    }
     $type_select .= ">" . XoopsLocale::EXACT_MATCH . "</option>";
     $type_select .= "</select>";
 
@@ -65,11 +73,15 @@ 
 
     $select_category = "<select name=\"category[]\" size=\"5\" multiple=\"multiple\" width=\"150\" style=\"width:150px;\">";
     $select_category .= "<option value=\"all\"";
-    if (empty($category) || count($category) == 0) $select_category .= "selected=\"selected\"";
+    if (empty($category) || count($category) == 0) {
+        $select_category .= "selected=\"selected\"";
+    }
     $select_category .= ">" . XoopsLocale::ALL . "</option>";
     foreach ($categories as $id => $cat) {
         $select_category .= "<option value=\"" . $id . "\"";
-        if (in_array($id, $category)) $select_category .= "selected=\"selected\"";
+        if (in_array($id, $category)) {
+            $select_category .= "selected=\"selected\"";
+        }
         $select_category .= ">" . $cat . "</option>";
     }
     $select_category .= "</select>";
@@ -77,37 +89,57 @@ 
     /* scope */
     $searchin_select = "";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"title\"";
-    if (in_array("title", $searchin)) $searchin_select .= " checked";
+    if (in_array("title", $searchin)) {
+        $searchin_select .= " checked";
+    }
     $searchin_select .= " />" . _CO_PUBLISHER_TITLE . "&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"subtitle\"";
-    if (in_array("subtitle", $searchin)) $searchin_select .= " checked";
+    if (in_array("subtitle", $searchin)) {
+        $searchin_select .= " checked";
+    }
     $searchin_select .= " />" . _CO_PUBLISHER_SUBTITLE . "&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"summary\"";
-    if (in_array("summary", $searchin)) $searchin_select .= " checked";
+    if (in_array("summary", $searchin)) {
+        $searchin_select .= " checked";
+    }
     $searchin_select .= " />" . _CO_PUBLISHER_SUMMARY . "&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"text\"";
-    if (in_array("body", $searchin)) $searchin_select .= " checked";
+    if (in_array("body", $searchin)) {
+        $searchin_select .= " checked";
+    }
     $searchin_select .= " />" . _CO_PUBLISHER_BODY . "&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"keywords\"";
-    if (in_array("meta_keywords", $searchin)) $searchin_select .= " checked";
+    if (in_array("meta_keywords", $searchin)) {
+        $searchin_select .= " checked";
+    }
     $searchin_select .= " />" . _CO_PUBLISHER_ITEM_META_KEYWORDS . "&nbsp;&nbsp;";
     $searchin_select .= "<input type=\"checkbox\" name=\"searchin[]\" value=\"all\"";
-    if (in_array("all", $searchin) || empty($searchin)) $searchin_select .= " checked";
+    if (in_array("all", $searchin) || empty($searchin)) {
+        $searchin_select .= " checked";
+    }
     $searchin_select .= " />" . XoopsLocale::ALL . "&nbsp;&nbsp;";
 
     /* sortby */
     $sortby_select = "<select name=\"sortby\">";
     $sortby_select .= "<option value=\"itemid\"";
-    if ("itemid" === $sortby || empty($sortby)) $sortby_select .= " selected=\"selected\"";
+    if ("itemid" === $sortby || empty($sortby)) {
+        $sortby_select .= " selected=\"selected\"";
+    }
     $sortby_select .= ">" . XoopsLocale::NONE . "</option>";
     $sortby_select .= "<option value=\"datesub\"";
-    if ("datesub" === $sortby) $sortby_select .= " selected=\"selected\"";
+    if ("datesub" === $sortby) {
+        $sortby_select .= " selected=\"selected\"";
+    }
     $sortby_select .= ">" . _CO_PUBLISHER_DATESUB . "</option>";
     $sortby_select .= "<option value=\"title\"";
-    if ("title" === $sortby) $sortby_select .= " selected=\"selected\"";
+    if ("title" === $sortby) {
+        $sortby_select .= " selected=\"selected\"";
+    }
     $sortby_select .= ">" . _CO_PUBLISHER_TITLE . "</option>";
     $sortby_select .= "<option value=\"categoryid\"";
-    if ("categoryid" === $sortby) $sortby_select .= " selected=\"selected\"";
+    if ("categoryid" === $sortby) {
+        $sortby_select .= " selected=\"selected\"";
+    }
     $sortby_select .= ">" . _CO_PUBLISHER_CATEGORY . "</option>";
     $sortby_select .= "</select>";