XOOPS / XoopsCore

htdocs/modules/publisher/blocks/items_random_item.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_items_random_item_show($options)
 {

Braces +3 added lines, -1 removed lines

@@ -29,7 +29,9 @@
     // creating the ITEM object
     $itemsObj = $publisher->getItemHandler()->getRandomItem('', array(_PUBLISHER_STATUS_PUBLISHED));
 
-    if (!is_object($itemsObj)) return $block;
+    if (!is_object($itemsObj)) {
+        return $block;
+    }
 
     $block['content'] = $itemsObj->getBlockSummary(300, true); //show complete summary  but truncate to 300 if only body available
     $block['id'] = $itemsObj->getVar('itemid');

htdocs/modules/publisher/blocks/items_recent.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_items_recent_show($options)
 {
@@ -47,7 +47,7 @@ 
         $criteria = null;
     } else {
         $criteria = new CriteriaCompo();
-        $criteria->add(new Criteria('categoryid', '(' . $options[0] . ')', 'IN'));
+        $criteria->add(new Criteria('categoryid', '('.$options[0].')', 'IN'));
     }
     $itemsObj = $publisher->getItemHandler()->getItems($limit, $start, array(_PUBLISHER_STATUS_PUBLISHED), -1, $sort, $order, '', true, $criteria, true);
 
@@ -73,7 +73,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;
     }
 
     return $block;

htdocs/modules/publisher/blocks/category_items_sel.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_category_items_sel_show($options)
 {

htdocs/modules/publisher/blocks/latest_files.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_latest_files_show($options)
 {

htdocs/modules/publisher/blocks/items_spot.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_items_spot_show($options)
 {
@@ -48,7 +48,7 @@ 
                 $category['name'] = $cat->getVar('name');
                 $category['categoryurl'] = $cat->getCategoryUrl();
                 if ($cat->image() !== 'blank.png') {
-                    $category['image_path'] = PublisherUtils::getImageDir('category', false) . $cat->image();
+                    $category['image_path'] = PublisherUtils::getImageDir('category', false).$cat->image();
                 } else {
                     $category['image_path'] = '';
                 }

htdocs/modules/publisher/blocks/latest_news.php

Spacing +81 added lines, -81 removed lines

@@ -21,7 +21,7 @@ 
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_latest_news_show($options)
 {
@@ -57,14 +57,14 @@ 
         $criteria = null;
     } else {
         $criteria = new CriteriaCompo();
-        $criteria->add(new Criteria('categoryid', '(' . $options[29] . ')', 'IN'));
+        $criteria->add(new Criteria('categoryid', '('.$options[29].')', 'IN'));
     }
 
     // Use specific ITEMS
     if ($selected_stories != 0) {
         unset($criteria); //removes category option
         $criteria = new CriteriaCompo();
-        $criteria->add(new Criteria('itemid', '(' . $selected_stories . ')', 'IN'));
+        $criteria->add(new Criteria('itemid', '('.$selected_stories.')', 'IN'));
     }
 
     $itemsObj = $publisher->getItemHandler()->getItems($limit, $start, array(_PUBLISHER_STATUS_PUBLISHED), -1, $sort, $order, '', true, $criteria, 'itemid');
@@ -96,7 +96,7 @@ 
 
         $ls_height = '';
         if ($options[12] != 0) {
-            $ls_height = 'height="' . $imgheight . '" ';
+            $ls_height = 'height="'.$imgheight.'" ';
         }
 
         if ($options[15] === 'LEFT') {
@@ -113,17 +113,17 @@ 
 
         //Image
         if ($options[10] == 1 && $item['image_path'] != '') {
-            $startdiv = '<div style="' . $imgposition . '"><a href="' . $item['itemurl'] . '">';
-            $style = 'style="margin' . $ls_margin . ': 10px; padding: 2px; border: ' . $border . 'px solid #' . $bordercolor . '"';
-            $enddiv = 'width="' . $imgwidth . '" ' . $ls_height . '/></a></div>';
-            $image = $startdiv . '<img ' . $style . ' src="' . $item['item_image'] . '" alt="' . $item['image_name'] . '" ' . $enddiv;
+            $startdiv = '<div style="'.$imgposition.'"><a href="'.$item['itemurl'].'">';
+            $style = 'style="margin'.$ls_margin.': 10px; padding: 2px; border: '.$border.'px solid #'.$bordercolor.'"';
+            $enddiv = 'width="'.$imgwidth.'" '.$ls_height.'/></a></div>';
+            $image = $startdiv.'<img '.$style.' src="'.$item['item_image'].'" alt="'.$item['image_name'].'" '.$enddiv;
 
             $item['image'] = $image;
         }
 
         if ($publisher->isUserAdmin()) {
-            $item['admin'] = "<a href='" . PUBLISHER_URL . "/submit.php?itemid=" . $itemObj->getVar('itemid'). "'><img src='" . PUBLISHER_URL . "/images/links/edit.gif'" . " title='" . _CO_PUBLISHER_EDIT . "' alt='" . _CO_PUBLISHER_EDIT . "' /></a>&nbsp;";
-            $item['admin'] .= "<a href='" . PUBLISHER_URL . "/admin/item.php?op=del&amp;itemid=" . $itemObj->getVar('itemid'). "'><img src='" . PUBLISHER_URL . "/images/links/delete.png'" . " title='" . _CO_PUBLISHER_DELETE . "' alt='" . _CO_PUBLISHER_DELETE . "' /></a>";
+            $item['admin'] = "<a href='".PUBLISHER_URL."/submit.php?itemid=".$itemObj->getVar('itemid')."'><img src='".PUBLISHER_URL."/images/links/edit.gif'"." title='"._CO_PUBLISHER_EDIT."' alt='"._CO_PUBLISHER_EDIT."' /></a>&nbsp;";
+            $item['admin'] .= "<a href='".PUBLISHER_URL."/admin/item.php?op=del&amp;itemid=".$itemObj->getVar('itemid')."'><img src='".PUBLISHER_URL."/images/links/delete.png'"." title='"._CO_PUBLISHER_DELETE."' alt='"._CO_PUBLISHER_DELETE."' /></a>";
         } else {
             $item['admin'] = '';
         }
@@ -136,40 +136,40 @@ 
          */
         $block['archivelink'] = '';
         if ($options[17] == 1) {
-            $block['archivelink'] = '| <a href="' . PUBLISHER_URL . '/archive.php">' . _MB_PUBLISHER_ARCHIVE . '</a> ';
+            $block['archivelink'] = '| <a href="'.PUBLISHER_URL.'/archive.php">'._MB_PUBLISHER_ARCHIVE.'</a> ';
         }
 
         //TODO: Should we not show link to Anonymous?
         $block['submitlink'] = '';
         if ($options[18] == 1 && $xoops->isUser()) {
-            $block['submitlink'] = '| <a href="' . PUBLISHER_URL . '/submit.php">' . _MB_PUBLISHER_SUBMITNEWS . '</a> ';
+            $block['submitlink'] = '| <a href="'.PUBLISHER_URL.'/submit.php">'._MB_PUBLISHER_SUBMITNEWS.'</a> ';
         }
 
         $item['poster'] = '';
         if ($options[19] == 1) {
-            $item['poster'] = _MB_PUBLISHER_POSTER . ' ' . $itemObj->posterName();
+            $item['poster'] = _MB_PUBLISHER_POSTER.' '.$itemObj->posterName();
         }
 
         $item['posttime'] = '';
         if ($options[20] == 1) {
             //todo, check this concatenation
-            $item['posttime'] = strtolower(XoopsLocale::ON) . ' ' . $itemObj->datesub();
+            $item['posttime'] = strtolower(XoopsLocale::ON).' '.$itemObj->datesub();
         }
 
         $item['topic_title'] = '';
         if ($options[21] == 1) {
-            $item['topic_title'] = $itemObj->getCategoryLink() . _MB_PUBLISHER_SP;
+            $item['topic_title'] = $itemObj->getCategoryLink()._MB_PUBLISHER_SP;
         }
 
         $item['read'] = '';
         if ($options[22] == 1) {
             //todo, check this concatenation
-            $item['read'] = '&nbsp;(' . $itemObj->getVar('counter') . ' ' . strtolower(XoopsLocale::READS) . ')';
+            $item['read'] = '&nbsp;('.$itemObj->getVar('counter').' '.strtolower(XoopsLocale::READS).')';
         }
 
         $item['more'] = '';
         if ($itemObj->body() != '' || $itemObj->getVar('comments') > 0) {
-            $item['more'] = '<a class="publisher_spotlight_readmore" href="' . $itemObj->getItemUrl() . '">' . _MB_PUBLISHER_READMORE . '</a>';
+            $item['more'] = '<a class="publisher_spotlight_readmore" href="'.$itemObj->getItemUrl().'">'._MB_PUBLISHER_READMORE.'</a>';
         }
 
         $comments = $itemObj->getVar('comments');
@@ -178,23 +178,23 @@ 
                 //shows 1 comment instead of 1 comm. if comments ==1
                 //langugage file modified accordingly
                 if ($comments == 1) {
-                    $item['comment'] = '&nbsp;' . _MB_PUBLISHER_ONECOMMENT . '&nbsp;';
+                    $item['comment'] = '&nbsp;'._MB_PUBLISHER_ONECOMMENT.'&nbsp;';
                 } else {
-                    $item['comment'] = '&nbsp;' . $comments . '&nbsp;' . _MB_PUBLISHER_COMMENTS . '&nbsp;';
+                    $item['comment'] = '&nbsp;'.$comments.'&nbsp;'._MB_PUBLISHER_COMMENTS.'&nbsp;';
                 }
             } else {
-                $item['comment'] = '&nbsp;' . _MB_PUBLISHER_NO_COMMENTS . '&nbsp;';
+                $item['comment'] = '&nbsp;'._MB_PUBLISHER_NO_COMMENTS.'&nbsp;';
             }
         }
 
         $item['print'] = '';
         if ($options[24] == 1) {
-            $item['print'] = '<a href="' . PublisherUtils::seoGenUrl("print", $itemObj->getVar('itemid'), $itemObj->getVar('short_url')) . '" rel="nofollow"><img src="' . PUBLISHER_URL . '/images/links/print.gif" title="' . _CO_PUBLISHER_PRINT . '" alt="' . _CO_PUBLISHER_PRINT . '" /></a>&nbsp;';
+            $item['print'] = '<a href="'.PublisherUtils::seoGenUrl("print", $itemObj->getVar('itemid'), $itemObj->getVar('short_url')).'" rel="nofollow"><img src="'.PUBLISHER_URL.'/images/links/print.gif" title="'._CO_PUBLISHER_PRINT.'" alt="'._CO_PUBLISHER_PRINT.'" /></a>&nbsp;';
         }
 
         $item['pdf'] = '';
         if ($options[25] == 1) {
-            $item['pdf'] = "<a href='" . PUBLISHER_URL . "/makepdf.php?itemid=" . $itemObj->getVar('itemid'). "' rel='nofollow'><img src='" . PUBLISHER_URL . "/images/links/pdf.gif' title='" . _CO_PUBLISHER_PDF . "' alt='" . _CO_PUBLISHER_PDF . "' /></a>&nbsp;";
+            $item['pdf'] = "<a href='".PUBLISHER_URL."/makepdf.php?itemid=".$itemObj->getVar('itemid')."' rel='nofollow'><img src='".PUBLISHER_URL."/images/links/pdf.gif' title='"._CO_PUBLISHER_PDF."' alt='"._CO_PUBLISHER_PDF."' /></a>&nbsp;";
         }
 
         $item['email'] = '';
@@ -203,12 +203,12 @@ 
             $subject = $itemObj->_convert_for_japanese($subject);
             $maillink = PublisherUtils::tellafriend($subject);
 
-            $item['email'] = '<a href="' . $maillink . '"><img src="' . PUBLISHER_URL . '/images/links/friend.gif" title="' . _CO_PUBLISHER_MAIL . '" alt="' . _CO_PUBLISHER_MAIL . '" /></a>&nbsp;';
+            $item['email'] = '<a href="'.$maillink.'"><img src="'.PUBLISHER_URL.'/images/links/friend.gif" title="'._CO_PUBLISHER_MAIL.'" alt="'._CO_PUBLISHER_MAIL.'" /></a>&nbsp;';
         }
 
         $block['morelink'] = '';
         if ($options[27] == 1) {
-            $block['morelink'] = '<a href="' . PUBLISHER_URL . '/index.php">' . _MB_PUBLISHER_MORE_ITEMS . '</a> ';
+            $block['morelink'] = '<a href="'.PUBLISHER_URL.'/index.php">'._MB_PUBLISHER_MORE_ITEMS.'</a> ';
         }
 
         $block['latestnews_scroll'] = false;
@@ -249,24 +249,24 @@ 
     $tabletag4 = '</td></tr>';
 
     $form = "<table border='0' cellpadding='0' cellspacing='0'>";
-    $form .= $tabletag3 . _MB_PUBLISHER_GENERALCONFIG . $tabletag4; // General Options
-    $form .= $tabletag1 . _MB_PUBLISHER_FIRST . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[0] . "' size='4'>&nbsp;" . _MB_PUBLISHER_ITEMS . "</td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_DISP . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[1] . "' size='4'>&nbsp;" . _MB_PUBLISHER_ITEMS . "</td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_COLUMNS . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[2] . "' size='4'>&nbsp;" . _MB_PUBLISHER_COLUMN . "</td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_TEXTLENGTH . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[3] . "' size='4'>&nbsp;" . _MB_PUBLISHER_LETTER . "</td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_SELECTEDSTORIES . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[4] . "' size='16'></td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_SCROLL . $tabletag2;
+    $form .= $tabletag3._MB_PUBLISHER_GENERALCONFIG.$tabletag4; // General Options
+    $form .= $tabletag1._MB_PUBLISHER_FIRST.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[0]."' size='4'>&nbsp;"._MB_PUBLISHER_ITEMS."</td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_DISP.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[1]."' size='4'>&nbsp;"._MB_PUBLISHER_ITEMS."</td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_COLUMNS.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[2]."' size='4'>&nbsp;"._MB_PUBLISHER_COLUMN."</td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_TEXTLENGTH.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[3]."' size='4'>&nbsp;"._MB_PUBLISHER_LETTER."</td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_SELECTEDSTORIES.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[4]."' size='16'></td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_SCROLL.$tabletag2;
     $form .= publisher_mk_chkbox($options, 5);
-    $form .= $tabletag1 . _MB_PUBLISHER_SCROLLHEIGHT . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[6] . "' size='4'></td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_SCROLLSPEED . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[7] . "' size='4'></td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_SCROLLDIR . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_SCROLLHEIGHT.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[6]."' size='4'></td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_SCROLLSPEED.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[7]."' size='4'></td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_SCROLLDIR.$tabletag2;
 
     $form .= "<select size='1' name='options[8]'>";
 
@@ -280,90 +280,90 @@ 
     }
     $form .= "</select></td></tr>";
 
-    $form .= $tabletag1 . _MB_PUBLISHER_ORDER . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_ORDER.$tabletag2;
 
     $form .= "<select name='options[9]'>";
     $form .= "<option value='datesub'";
     if ($options[9] === "datesub") {
         $form .= " selected='selected'";
     }
-    $form .= ">" . _MB_PUBLISHER_DATE . "</option>";
+    $form .= ">"._MB_PUBLISHER_DATE."</option>";
 
     $form .= "<option value='counter'";
     if ($options[9] === "counter") {
         $form .= " selected='selected'";
     }
-    $form .= ">" . _MB_PUBLISHER_HITS . "</option>";
+    $form .= ">"._MB_PUBLISHER_HITS."</option>";
 
     $form .= "<option value='weight'";
     if ($options[9] === "weight") {
         $form .= " selected='selected'";
     }
-    $form .= ">" . _MB_PUBLISHER_WEIGHT . "</option>";
+    $form .= ">"._MB_PUBLISHER_WEIGHT."</option>";
 
     $form .= "</select></td></tr>";
 
-    $form .= $tabletag3 . _MB_PUBLISHER_PHOTOSCONFIG . $tabletag4; // Photos Options
-    $form .= $tabletag1 . _MB_PUBLISHER_IMGDISPLAY . $tabletag2;
+    $form .= $tabletag3._MB_PUBLISHER_PHOTOSCONFIG.$tabletag4; // Photos Options
+    $form .= $tabletag1._MB_PUBLISHER_IMGDISPLAY.$tabletag2;
     $form .= publisher_mk_chkbox($options, 10);
-    $form .= $tabletag1 . _MB_PUBLISHER_IMGWIDTH . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[11] . "' size='4'>&nbsp;" . _MB_PUBLISHER_PIXEL . "</td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_IMGHEIGHT . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[12] . "' size='4'>&nbsp;" . _MB_PUBLISHER_PIXEL . "</td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_BORDER . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[13] . "' size='4'>&nbsp;" . _MB_PUBLISHER_PIXEL . "</td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_BORDERCOLOR . $tabletag2;
-    $form .= "<input type='text' name='options[]' value='" . $options[14] . "' size='8'></td></tr>";
-    $form .= $tabletag1 . _MB_PUBLISHER_IMGPOSITION . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_IMGWIDTH.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[11]."' size='4'>&nbsp;"._MB_PUBLISHER_PIXEL."</td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_IMGHEIGHT.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[12]."' size='4'>&nbsp;"._MB_PUBLISHER_PIXEL."</td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_BORDER.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[13]."' size='4'>&nbsp;"._MB_PUBLISHER_PIXEL."</td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_BORDERCOLOR.$tabletag2;
+    $form .= "<input type='text' name='options[]' value='".$options[14]."' size='8'></td></tr>";
+    $form .= $tabletag1._MB_PUBLISHER_IMGPOSITION.$tabletag2;
     $form .= "<select name='options[]'>";
     $form .= "<option value='LEFT'";
     if ($options[15] === 'LEFT') {
         $form .= " selected='selected'";
     }
-    $form .= '>' . XoopsLocale::LEFT . "</option>\n";
+    $form .= '>'.XoopsLocale::LEFT."</option>\n";
 
     $form .= "<option value='CENTER'";
     if ($options[15] === 'CENTER') {
         $form .= " selected='selected'";
     }
-    $form .= '>' . XoopsLocale::CENTER . "</option>\n";
+    $form .= '>'.XoopsLocale::CENTER."</option>\n";
 
     $form .= "<option value='RIGHT'";
     if ($options[15] === 'RIGHT') {
         $form .= " selected='selected'";
     }
-    $form .= '>' . XoopsLocale::RIGHT . '</option>';
+    $form .= '>'.XoopsLocale::RIGHT.'</option>';
     $form .= "</select></td></tr>";
 
-    $form .= $tabletag3 . _MB_PUBLISHER_LINKSCONFIG . $tabletag4; // Links Options
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_TOPICLINK . $tabletag2;
+    $form .= $tabletag3._MB_PUBLISHER_LINKSCONFIG.$tabletag4; // Links Options
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_TOPICLINK.$tabletag2;
     $form .= publisher_mk_chkbox($options, 16);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_ARCHIVELINK . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_ARCHIVELINK.$tabletag2;
     $form .= publisher_mk_chkbox($options, 17);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_SUBMITLINK . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_SUBMITLINK.$tabletag2;
     $form .= publisher_mk_chkbox($options, 18);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_POSTEDBY . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_POSTEDBY.$tabletag2;
     $form .= publisher_mk_chkbox($options, 19);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_POSTTIME . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_POSTTIME.$tabletag2;
     $form .= publisher_mk_chkbox($options, 20);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_TOPICTITLE . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_TOPICTITLE.$tabletag2;
     $form .= publisher_mk_chkbox($options, 21);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_READ . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_READ.$tabletag2;
     $form .= publisher_mk_chkbox($options, 22);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_COMMENT . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_COMMENT.$tabletag2;
     $form .= publisher_mk_chkbox($options, 23);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_PRINT . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_PRINT.$tabletag2;
     $form .= publisher_mk_chkbox($options, 24);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_PDF . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_PDF.$tabletag2;
     $form .= publisher_mk_chkbox($options, 25);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_EMAIL . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_EMAIL.$tabletag2;
     $form .= publisher_mk_chkbox($options, 26);
-    $form .= $tabletag1 . _MB_PUBLISHER_DISPLAY_MORELINK . $tabletag2;
+    $form .= $tabletag1._MB_PUBLISHER_DISPLAY_MORELINK.$tabletag2;
     $form .= publisher_mk_chkbox($options, 27);
 
 
-    $form .= $tabletag3 . _MB_PUBLISHER_TEMPLATESCONFIG . $tabletag4; // Templates Options
-    $form .= $tabletag1 . _MB_PUBLISHER_TEMPLATE . $tabletag2;
+    $form .= $tabletag3._MB_PUBLISHER_TEMPLATESCONFIG.$tabletag4; // Templates Options
+    $form .= $tabletag1._MB_PUBLISHER_TEMPLATE.$tabletag2;
     $form .= "<select size='1' name='options[28]'>";
 
     $templates = array('normal' => _MB_PUBLISHER_TEMPLATE_NORMAL, 'extended' => _MB_PUBLISHER_TEMPLATE_EXTENDED, 'ticker' => _MB_PUBLISHER_TEMPLATE_TICKER, 'slider1' => _MB_PUBLISHER_TEMPLATE_SLIDER1, 'slider2' => _MB_PUBLISHER_TEMPLATE_SLIDER2);
@@ -377,8 +377,8 @@ 
     $form .= "</select></td></tr>";
 
     //Select Which Categories To Show
-    $form .= $tabletag3 . _MB_PUBLISHER_TOPICSCONFIG . $tabletag4; // Topics Options
-    $form .= $tabletag1 . _MB_PUBLISHER_TOPICSDISPLAY . $tabletag2;
+    $form .= $tabletag3._MB_PUBLISHER_TOPICSCONFIG.$tabletag4; // Topics Options
+    $form .= $tabletag1._MB_PUBLISHER_TOPICSDISPLAY.$tabletag2;
     $form .= PublisherUtils::createCategorySelect($options[29], 0, true, 'options[29]');
     $form .= '</td></tr>';
 
@@ -392,12 +392,12 @@ 
     if ($options[$number] == 1) {
         $chk = " checked='checked'";
     }
-    $chkbox = "<input type='radio' name='options[{$number}]' value='1'" . $chk . " />&nbsp;" . XoopsLocale::YES . "&nbsp;&nbsp;";
+    $chkbox = "<input type='radio' name='options[{$number}]' value='1'".$chk." />&nbsp;".XoopsLocale::YES."&nbsp;&nbsp;";
     $chk = "";
     if ($options[$number] == 0) {
         $chk = " checked='checked'";
     }
-    $chkbox .= "<input type='radio' name='options[{$number}]' value='0'" . $chk . " />&nbsp;" . XoopsLocale::NO . "</td></tr>";
+    $chkbox .= "<input type='radio' name='options[{$number}]' value='0'".$chk." />&nbsp;".XoopsLocale::NO."</td></tr>";
     return $chkbox;
 }
 
@@ -407,16 +407,16 @@ 
     if ($options[$number] == 2) {
         $slc = " checked='checked'";
     }
-    $select = "<input type='radio' name='options[{$number}]' value='2'" . $slc . " />&nbsp;" . XoopsLocale::LEFT . "&nbsp;&nbsp;";
+    $select = "<input type='radio' name='options[{$number}]' value='2'".$slc." />&nbsp;".XoopsLocale::LEFT."&nbsp;&nbsp;";
     $slc = "";
     if ($options[$number] == 1) {
         $slc = " checked='checked'";
     }
-    $select .= "<input type='radio' name='options[{$number}]' value='1'" . $slc . " />&nbsp;" . XoopsLocale::CENTER . "&nbsp;&nbsp;";
+    $select .= "<input type='radio' name='options[{$number}]' value='1'".$slc." />&nbsp;".XoopsLocale::CENTER."&nbsp;&nbsp;";
     $slc = "";
     if ($options[$number] == 0) {
         $slc = " checked='checked'";
     }
-    $select .= "<input type='radio' name='options[{$number}]' value='0'" . $slc . " />&nbsp;" . XoopsLocale::RIGHT . "</td></tr>";
+    $select .= "<input type='radio' name='options[{$number}]' value='0'".$slc." />&nbsp;".XoopsLocale::RIGHT."</td></tr>";
     return $select;
 }

htdocs/modules/publisher/blocks/items_menu.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 function publisher_items_menu_show($options)
 {

Braces +3 added lines, -1 removed lines

@@ -31,7 +31,9 @@
     // Getting all top cats
     $block_categoriesObj = $publisher->getCategoryHandler()->getCategories(0, 0, 0);
 
-    if (count($block_categoriesObj) == 0) return $block;
+    if (count($block_categoriesObj) == 0) {
+        return $block;
+    }
 
     // Are we in Publisher ?
     $block['inModule'] = $publisher->isCurrentModule();

htdocs/modules/publisher/class/file.php

Spacing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
  * @version         $Id$
  */
 
-include_once dirname(__DIR__) . '/include/common.php';
+include_once dirname(__DIR__).'/include/common.php';
 
 // File status
 define("_PUBLISHER_STATUS_FILE_NOTSET", -1);
@@ -114,9 +114,9 @@ 
         if (!is_dir(PublisherUtils::getUploadDir())) {
             mkdir(PublisherUtils::getUploadDir(), 0757);
         }
-        $uploader = new XoopsMediaUploader(PublisherUtils::getUploadDir() . '/', $allowed_mimetypes, $maxfilesize, $maxfilewidth, $maxfileheight);
+        $uploader = new XoopsMediaUploader(PublisherUtils::getUploadDir().'/', $allowed_mimetypes, $maxfilesize, $maxfilewidth, $maxfileheight);
         if ($uploader->fetchMedia($post_field)) {
-            $uploader->setTargetFileName($itemid . "_" . $uploader->getMediaName());
+            $uploader->setTargetFileName($itemid."_".$uploader->getMediaName());
             if ($uploader->upload()) {
                 $this->setVar('filename', $uploader->getSavedFileName());
                 if ($this->getVar('name') == '') {
@@ -184,7 +184,7 @@ 
      */
     public function getFileUrl()
     {
-        return PublisherUtils::getUploadDir(false) . $this->getVar('filename');
+        return PublisherUtils::getUploadDir(false).$this->getVar('filename');
     }
 
     /**
@@ -192,7 +192,7 @@ 
      */
     public function getFilePath()
     {
-        return PublisherUtils::getUploadDir() . $this->getVar('filename');
+        return PublisherUtils::getUploadDir().$this->getVar('filename');
     }
 
     /**
@@ -200,7 +200,7 @@ 
      */
     public function getFileLink()
     {
-        return "<a href='" . PUBLISHER_URL . "/visit.php?fileid=" . $this->getVar('fileid') . "'>" . $this->getVar('name') . "</a>";
+        return "<a href='".PUBLISHER_URL."/visit.php?fileid=".$this->getVar('fileid')."'>".$this->getVar('name')."</a>";
     }
 
     /**
@@ -208,7 +208,7 @@ 
      */
     public function getItemLink()
     {
-        return "<a href='" . PUBLISHER_URL . "/item.php?itemid=" . $this->getVar('itemid') . "'>" . $this->getVar('name') . "</a>";
+        return "<a href='".PUBLISHER_URL."/item.php?itemid=".$this->getVar('itemid')."'>".$this->getVar('name')."</a>";
     }
 
     /**

htdocs/modules/publisher/class/utils.php

Spacing +48 added lines, -48 removed lines

@@ -151,7 +151,7 @@ 
         if (!$withLink) {
             return $publisher->getModule()->getVar('name');
         } else {
-            return '<a href="' . $xoops->url(PUBLISHER_URL) . '/">' . $publisher->getModule()->getVar('name') . '</a>';
+            return '<a href="'.$xoops->url(PUBLISHER_URL).'/">'.$publisher->getModule()->getVar('name').'</a>';
         }
     }
 
@@ -227,10 +227,10 @@ 
             $path_status = _AM_PUBLISHER_AVAILABLE;
         } elseif (!@is_dir($thePath)) {
             $pathCheckResult = -1;
-            $path_status = _AM_PUBLISHER_NOTAVAILABLE . " <a href='" . $publisher->url("admin/index.php?op=createdir&amp;path={$item}") . "'>" . _AM_PUBLISHER_CREATETHEDIR . "</a>";
+            $path_status = _AM_PUBLISHER_NOTAVAILABLE." <a href='".$publisher->url("admin/index.php?op=createdir&amp;path={$item}")."'>"._AM_PUBLISHER_CREATETHEDIR."</a>";
         } else {
             $pathCheckResult = -2;
-            $path_status = _AM_PUBLISHER_NOTWRITABLE . " <a href='" . $publisher->url("admin/index.php?op=setperm&amp;path={$item}") . "'>" . _AM_SCS_SETMPERM . "</a>";
+            $path_status = _AM_PUBLISHER_NOTWRITABLE." <a href='".$publisher->url("admin/index.php?op=setperm&amp;path={$item}")."'>"._AM_SCS_SETMPERM."</a>";
         }
         if (!$getStatus) {
             return $path_status;
@@ -298,16 +298,16 @@ 
             if ($item === 'root') {
                 $item = '';
             } else {
-                $item = $item . '/';
+                $item = $item.'/';
             }
         } else {
             $item = '';
         }
 
         if ($hasPath) {
-            return $xoops->path(PUBLISHER_UPLOADS_PATH . '/' . $item);
+            return $xoops->path(PUBLISHER_UPLOADS_PATH.'/'.$item);
         } else {
-            return $xoops->url(PUBLISHER_UPLOADS_URL . '/' . $item);
+            return $xoops->url(PUBLISHER_UPLOADS_URL.'/'.$item);
         }
     }
 
@@ -337,7 +337,7 @@ 
     {
         $ret = '';
         foreach ($errors as $value) {
-            $ret .= '<br /> - ' . $value;
+            $ret .= '<br /> - '.$value;
         }
 
         return $ret;
@@ -428,11 +428,11 @@ 
             $display = 'block';
         }
 
-        echo "<h3 style=\"color: #2F5376; font-weight: bold; font-size: 14px; margin: 6px 0 0 0; \"><a href='javascript:;' onclick=\"toggle('" . $tablename . "'); toggleIcon('" . $iconname . "')\";>";
-        echo "<img id='" . $iconname . "' src='" . $publisher->url("images/links/" . $image) . "' alt='' /></a>&nbsp;" . $tabletitle . "</h3>";
-        echo "<div id='" . $tablename . "' style='display: " . $display . ";'>";
+        echo "<h3 style=\"color: #2F5376; font-weight: bold; font-size: 14px; margin: 6px 0 0 0; \"><a href='javascript:;' onclick=\"toggle('".$tablename."'); toggleIcon('".$iconname."')\";>";
+        echo "<img id='".$iconname."' src='".$publisher->url("images/links/".$image)."' alt='' /></a>&nbsp;".$tabletitle."</h3>";
+        echo "<div id='".$tablename."' style='display: ".$display.";'>";
         if ($tabledsc != '') {
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . $tabledsc . "</span>";
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".$tabledsc."</span>";
         }
     }
 
@@ -449,14 +449,14 @@ 
         $urls = self::getCurrentUrls();
         $path = $urls['phpself'];
 
-        $cookie_name = $path . '_publisher_collaps_' . $name;
+        $cookie_name = $path.'_publisher_collaps_'.$name;
         $cookie_name = str_replace('.', '_', $cookie_name);
         $cookie = self::getCookieVar($cookie_name, '');
 
         if ($cookie === 'none') {
             echo '
         <script type="text/javascript"><!--
-        toggle("' . $name . '"); toggleIcon("' . $icon . '");
+        toggle("' . $name.'"); toggleIcon("'.$icon.'");
         //-->
         </script>
         ';
@@ -504,10 +504,10 @@ 
         $querystring = isset($_SERVER['QUERY_STRING']) ? $_SERVER['QUERY_STRING'] : '';
 
         if ($querystring != '') {
-            $querystring = '?' . $querystring;
+            $querystring = '?'.$querystring;
         }
 
-        $currenturl = $http . $httphost . $phpself . $querystring;
+        $currenturl = $http.$httphost.$phpself.$querystring;
 
         $urls = array();
         $urls['http'] = $http;
@@ -547,13 +547,13 @@ 
             $spaces .= '--';
         }
 
-        $ret .= "<option value='" . $categoryObj->getVar('categoryid') . "'";
+        $ret .= "<option value='".$categoryObj->getVar('categoryid')."'";
         if (is_array($selectedid) && in_array($categoryObj->getVar('categoryid'), $selectedid)) {
             $ret .= " selected='selected'";
         } elseif ($categoryObj->getVar('categoryid') == $selectedid) {
             $ret .= " selected='selected'";
         }
-        $ret .= ">" . $spaces . $categoryObj->getVar('name') . "</option>\n";
+        $ret .= ">".$spaces.$categoryObj->getVar('name')."</option>\n";
 
         $subCategoriesObj = $publisher->getCategoryHandler()->getCategories(0, 0, $categoryObj->getVar('categoryid'));
         if (count($subCategoriesObj) > 0) {
@@ -581,13 +581,13 @@ 
 
         $selectedid = explode(',', $selectedid);
 
-        $ret = "<select name='" . $selectname . "[]' multiple='multiple' size='10'>";
+        $ret = "<select name='".$selectname."[]' multiple='multiple' size='10'>";
         if ($allCatOption) {
             $ret .= "<option value='0'";
             if (in_array(0, $selectedid)) {
                 $ret .= " selected='selected'";
             }
-            $ret .= ">" . _MB_PUBLISHER_ALLCAT . "</option>";
+            $ret .= ">"._MB_PUBLISHER_ALLCAT."</option>";
         }
 
         // Creating category objects
@@ -618,7 +618,7 @@ 
         $ret = "";
         if ($allCatOption) {
             $ret .= "<option value='0'";
-            $ret .= ">" . _MB_PUBLISHER_ALLCAT . "</option>\n";
+            $ret .= ">"._MB_PUBLISHER_ALLCAT."</option>\n";
         }
 
         // Creating category objects
@@ -642,12 +642,12 @@ 
     public static function renderErrors(&$err_arr, $reseturl = '')
     {
         if (is_array($err_arr) && count($err_arr) > 0) {
-            echo '<div id="readOnly" class="errorMsg" style="border:1px solid #D24D00; background:#FEFECC url(' . PUBLISHER_URL . '/images/important-32.png) no-repeat 7px 50%;color:#333;padding-left:45px;">';
+            echo '<div id="readOnly" class="errorMsg" style="border:1px solid #D24D00; background:#FEFECC url('.PUBLISHER_URL.'/images/important-32.png) no-repeat 7px 50%;color:#333;padding-left:45px;">';
 
-            echo '<h4 style="text-align:left;margin:0; padding-top:0">' . _AM_PUBLISHER_MSG_SUBMISSION_ERR;
+            echo '<h4 style="text-align:left;margin:0; padding-top:0">'._AM_PUBLISHER_MSG_SUBMISSION_ERR;
 
             if ($reseturl) {
-                echo ' <a href="' . $reseturl . '">[' . _AM_PUBLISHER_TEXT_SESSION_RESET . ']</a>';
+                echo ' <a href="'.$reseturl.'">['._AM_PUBLISHER_TEXT_SESSION_RESET.']</a>';
             }
 
             echo '</h4><ul>';
@@ -655,10 +655,10 @@ 
             foreach ($err_arr as $key => $error) {
                 if (is_array($error)) {
                     foreach ($error as $err) {
-                        echo '<li><a href="#' . $key . '" onclick="var e = xoopsGetElementById(\'' . $key . '\'); e.focus();">' . htmlspecialchars($err) . '</a></li>';
+                        echo '<li><a href="#'.$key.'" onclick="var e = xoopsGetElementById(\''.$key.'\'); e.focus();">'.htmlspecialchars($err).'</a></li>';
                     }
                 } else {
-                    echo '<li><a href="#' . $key . '" onclick="var e = xoopsGetElementById(\'' . $key . '\'); e.focus();">' . htmlspecialchars($error) . '</a></li>';
+                    echo '<li><a href="#'.$key.'" onclick="var e = xoopsGetElementById(\''.$key.'\'); e.focus();">'.htmlspecialchars($error).'</a></li>';
                 }
             }
             echo "</ul></div><br />";
@@ -687,11 +687,11 @@ 
 
         $qs = '';
         foreach ($vars as $key => $value) {
-            $qs .= $joinStr . $key . '=' . $value;
+            $qs .= $joinStr.$key.'='.$value;
             $joinStr = $amp;
         }
 
-        return $page . '?' . $qs;
+        return $page.'?'.$qs;
     }
 
     /**
@@ -707,7 +707,7 @@ 
         }
         $target_uri = $xoops->url($_SERVER['REQUEST_URI']);
 
-        return $xoops->url('modules/tellafriend/index.php?target_uri=' . rawurlencode($target_uri) . '&amp;subject=' . rawurlencode($subject));
+        return $xoops->url('modules/tellafriend/index.php?target_uri='.rawurlencode($target_uri).'&amp;subject='.rawurlencode($subject));
     }
 
     /**
@@ -755,7 +755,7 @@ 
 
                 $message = sprintf(_CO_PUBLISHER_MESSAGE_FILE_ERROR, $errorstxt);
                 if ($withRedirect) {
-                    $xoops->redirect("file.php?op=mod&itemid=" . $itemid, 5, $message);
+                    $xoops->redirect("file.php?op=mod&itemid=".$itemid, 5, $message);
                 } else {
                     return $message;
                 }
@@ -765,15 +765,15 @@ 
         // Storing the file
         if (!$fileObj->store($allowed_mimetypes)) {
             if ($withRedirect) {
-                $xoops->redirect("file.php?op=mod&itemid=" . $fileObj->getVar('itemid'), 3, _CO_PUBLISHER_FILEUPLOAD_ERROR . self::formatErrors($fileObj->getErrors()));
+                $xoops->redirect("file.php?op=mod&itemid=".$fileObj->getVar('itemid'), 3, _CO_PUBLISHER_FILEUPLOAD_ERROR.self::formatErrors($fileObj->getErrors()));
             } else {
-                return _CO_PUBLISHER_FILEUPLOAD_ERROR . self::formatErrors($fileObj->getErrors());
+                return _CO_PUBLISHER_FILEUPLOAD_ERROR.self::formatErrors($fileObj->getErrors());
             }
         }
 
         if ($withRedirect) {
             $redirect_page = $another ? 'file.php' : 'item.php';
-            $xoops->redirect($redirect_page . "?op=mod&itemid=" . $fileObj->getVar('itemid'), 2, _CO_PUBLISHER_FILEUPLOAD_SUCCESS);
+            $xoops->redirect($redirect_page."?op=mod&itemid=".$fileObj->getVar('itemid'), 2, _CO_PUBLISHER_FILEUPLOAD_SUCCESS);
         }
 
         return true;
@@ -784,7 +784,7 @@ 
      */
     public static function newFeatureTag()
     {
-        $ret = '<span style="padding-right: 4px; font-weight: bold; color: red;">' . _CO_PUBLISHER_NEW_FEATURE . '</span>';
+        $ret = '<span style="padding-right: 4px; font-weight: bold; color: red;">'._CO_PUBLISHER_NEW_FEATURE.'</span>';
 
         return $ret;
     }
@@ -824,7 +824,7 @@ 
                 $string = self::closeTags($string);
             }
 
-            return $string . $etc;
+            return $string.$etc;
         } else {
             return $string;
         }
@@ -862,7 +862,7 @@ 
 
             $complete_tags = array_reverse($complete_tags);
             for ($i = 0; $i < count($complete_tags); ++$i) {
-                $string .= '</' . $complete_tags[$i] . '>';
+                $string .= '</'.$complete_tags[$i].'>';
             }
         }
 
@@ -911,26 +911,26 @@ 
 
         if (!$gperm_handler->checkRight('global', _PUBLISHER_RATE, $groups, $publisher->getModule()->getVar('mid'))) {
             $static_rater = array();
-            $static_rater[] .= "\n" . '<div class="publisher_ratingblock">';
-            $static_rater[] .= '<div id="unit_long' . $itemid . '">';
-            $static_rater[] .= '<div id="unit_ul' . $itemid . '" class="publisher_unit-rating" style="width:' . $rating_unitwidth * $units . 'px;">';
-            $static_rater[] .= '<div class="publisher_current-rating" style="width:' . $rating_width . 'px;">' . _MD_PUBLISHER_VOTE_RATING . ' ' . $rating2 . '/' . $units . '</div>';
+            $static_rater[] .= "\n".'<div class="publisher_ratingblock">';
+            $static_rater[] .= '<div id="unit_long'.$itemid.'">';
+            $static_rater[] .= '<div id="unit_ul'.$itemid.'" class="publisher_unit-rating" style="width:'.$rating_unitwidth * $units.'px;">';
+            $static_rater[] .= '<div class="publisher_current-rating" style="width:'.$rating_width.'px;">'._MD_PUBLISHER_VOTE_RATING.' '.$rating2.'/'.$units.'</div>';
             $static_rater[] .= '</div>';
-            $static_rater[] .= '<div class="publisher_static">' . _MD_PUBLISHER_VOTE_RATING . ': <strong> ' . $rating1 . '</strong>/' . $units . ' (' . $count . ' ' . $tense . ') <br /><em>' . _MD_PUBLISHER_VOTE_DISABLE . '</em></div>';
+            $static_rater[] .= '<div class="publisher_static">'._MD_PUBLISHER_VOTE_RATING.': <strong> '.$rating1.'</strong>/'.$units.' ('.$count.' '.$tense.') <br /><em>'._MD_PUBLISHER_VOTE_DISABLE.'</em></div>';
             $static_rater[] .= '</div>';
-            $static_rater[] .= '</div>' . "\n\n";
+            $static_rater[] .= '</div>'."\n\n";
 
             return join("\n", $static_rater);
         } else {
             $rater = '';
             $rater .= '<div class="publisher_ratingblock">';
-            $rater .= '<div id="unit_long' . $itemid . '">';
-            $rater .= '<div id="unit_ul' . $itemid . '" class="publisher_unit-rating" style="width:' . $rating_unitwidth * $units . 'px;">';
-            $rater .= '<div class="publisher_current-rating" style="width:' . $rating_width . 'px;">' . _MD_PUBLISHER_VOTE_RATING . ' ' . $rating2 . '/' . $units . '</div>';
+            $rater .= '<div id="unit_long'.$itemid.'">';
+            $rater .= '<div id="unit_ul'.$itemid.'" class="publisher_unit-rating" style="width:'.$rating_unitwidth * $units.'px;">';
+            $rater .= '<div class="publisher_current-rating" style="width:'.$rating_width.'px;">'._MD_PUBLISHER_VOTE_RATING.' '.$rating2.'/'.$units.'</div>';
 
             for ($ncount = 1; $ncount <= $units; ++$ncount) { // loop from 1 to the number of units
                 if (!$voted) { // if the user hasn't yet voted, draw the voting stars
-                    $rater .= '<div><a href="' . PUBLISHER_URL . '/rate.php?itemid=' . $itemid . '&amp;rating=' . $ncount . '" title="' . $ncount . ' ' . _MD_PUBLISHER_VOTE_OUTOF . ' ' . $units . '" class="publisher_r' . $ncount . '-unit rater" rel="nofollow">' . $ncount . '</a></div>';
+                    $rater .= '<div><a href="'.PUBLISHER_URL.'/rate.php?itemid='.$itemid.'&amp;rating='.$ncount.'" title="'.$ncount.' '._MD_PUBLISHER_VOTE_OUTOF.' '.$units.'" class="publisher_r'.$ncount.'-unit rater" rel="nofollow">'.$ncount.'</a></div>';
                 }
             }
 
@@ -941,7 +941,7 @@ 
                 $rater .= ' class="publisher_voted"';
             }
 
-            $rater .= '>' . _MD_PUBLISHER_VOTE_RATING . ': <strong> ' . $rating1 . '</strong>/' . $units . ' (' . $count . ' ' . $tense . ')';
+            $rater .= '>'._MD_PUBLISHER_VOTE_RATING.': <strong> '.$rating1.'</strong>/'.$units.' ('.$count.' '.$tense.')';
             $rater .= '  </div>';
             $rater .= '</div>';
             $rater .= '</div>';
@@ -1081,12 +1081,12 @@ 
         $publisher = Publisher::getInstance();
         if ($publisher->getConfig('seo_url_rewrite') !== 'none') {
             if (!empty($short_url)) {
-                $short_url = $short_url . '.html';
+                $short_url = $short_url.'.html';
             }
 
             if ($publisher->getConfig('seo_url_rewrite') === 'htaccess') {
                 // generate SEO url using htaccess
-                return \XoopsBaseConfig::get('url') . '/' . $publisher->getConfig('seo_module_name') . ".${op}.${id}/${short_url}";
+                return \XoopsBaseConfig::get('url').'/'.$publisher->getConfig('seo_module_name').".${op}.${id}/${short_url}";
             } else {
                 if ($publisher->getConfig('seo_url_rewrite') === 'path-info') {
                     // generate SEO url using path-info