geekwright / XoopsCore

htdocs/modules/publisher/include/ajax_rating.php

Spacing +8 added lines, -8 removed lines

@@ -34,7 +34,7 @@ 
 use XoopsModules\Publisher;
 use XoopsModules\Publisher\Helper;
 
-require_once dirname(__DIR__) . '/header.php';
+require_once dirname(__DIR__).'/header.php';
 
 $xoops = Xoops::getInstance();
 $xoops->disableErrorReporting();
@@ -56,7 +56,7 @@ 
 
 //Checking permissions
 if (!$helper->getConfig('perm_rating') || !$gpermHandler->checkRight('global', _PUBLISHER_RATE, $groups, $module_id)) {
-    $output = "unit_long$itemid|" . XoopsLocale::E_NO_ACCESS_PERMISSION . "\n";
+    $output = "unit_long$itemid|".XoopsLocale::E_NO_ACCESS_PERMISSION."\n";
     echo $output;
     exit();
 }
@@ -65,7 +65,7 @@ 
 $units = 5;
 
 if ($rating > 5 || $rating < 1) {
-    $output = "unit_long$itemid|" . _MD_PUBLISHER_VOTE_BAD . "\n";
+    $output = "unit_long$itemid|"._MD_PUBLISHER_VOTE_BAD."\n";
     echo $output;
     exit();
 }
@@ -87,7 +87,7 @@ 
 }
 
 if ($voted) {
-    $output = "unit_long$itemid|" . _MD_PUBLISHER_VOTE_ALREADY . "\n";
+    $output = "unit_long$itemid|"._MD_PUBLISHER_VOTE_ALREADY."\n";
     echo $output;
     exit();
 }
@@ -111,8 +111,8 @@ 
 // $new_back is what gets 'drawn' on your page after a successful 'AJAX/Javascript' vote
 $new_back = [];
 
-$new_back[] .= '<div class="publisher_unit-rating" style="width:' . $units * $rating_unitwidth . 'px;">';
-$new_back[] .= '<div class="publisher_current-rating" style="width:' . @number_format($current_rating / $count, 2) * $rating_unitwidth . 'px;">' . _MD_PUBLISHER_VOTE_RATING . '</div>';
+$new_back[] .= '<div class="publisher_unit-rating" style="width:'.$units * $rating_unitwidth.'px;">';
+$new_back[] .= '<div class="publisher_current-rating" style="width:'.@number_format($current_rating / $count, 2) * $rating_unitwidth.'px;">'._MD_PUBLISHER_VOTE_RATING.'</div>';
 $new_back[] .= '<div class="publisher_r1-unit">1</div>';
 $new_back[] .= '<div class="publisher_r2-unit">2</div>';
 $new_back[] .= '<div class="publisher_r3-unit">3</div>';
@@ -124,8 +124,8 @@ 
 $new_back[] .= '<div class="publisher_r9-unit">9</div>';
 $new_back[] .= '<div class="publisher_r10-unit">10</div>';
 $new_back[] .= '</div>';
-$new_back[] .= '<div class="publisher_voted">' . _MD_PUBLISHER_VOTE_RATING . ' <strong>' . @number_format($current_rating / $count, 2) . '</strong>/' . $units . ' (' . $count . ' ' . $tense . ')</div>';
-$new_back[] .= '<div class="publisher_thanks">' . _MD_PUBLISHER_VOTE_THANKS . '</div>';
+$new_back[] .= '<div class="publisher_voted">'._MD_PUBLISHER_VOTE_RATING.' <strong>'.@number_format($current_rating / $count, 2).'</strong>/'.$units.' ('.$count.' '.$tense.')</div>';
+$new_back[] .= '<div class="publisher_thanks">'._MD_PUBLISHER_VOTE_THANKS.'</div>';
 
 $allnewback = implode("\n", $new_back);
 

htdocs/modules/publisher/include/ajax_upload.php

Spacing +4 added lines, -4 removed lines

@@ -29,8 +29,8 @@ 
 use XoopsModules\Publisher;
 use XoopsModules\Publisher\Helper;
 
-include dirname(dirname(dirname(__DIR__))) . '/mainfile.php';
-require_once __DIR__ . '/common.php';
+include dirname(dirname(dirname(__DIR__))).'/mainfile.php';
+require_once __DIR__.'/common.php';
 
 $xoops = Xoops::getInstance();
 $xoops->disableErrorReporting();
@@ -74,7 +74,7 @@ 
 
 $image = null;
 if (false === $error) {
-    $uploader = new XoopsMediaUploader(XoopsBaseConfig::get('uploads-path') . '/images', [
+    $uploader = new XoopsMediaUploader(XoopsBaseConfig::get('uploads-path').'/images', [
         'image/gif',
         'image/jpeg',
         'image/pjpeg',
@@ -107,7 +107,7 @@ 
             }
         }
     } else {
-        $error = sprintf(_CO_PUBLISHER_FAILSAVEIMG, $filename) . '<br>' . implode('<br>', $uploader->getErrors(false));
+        $error = sprintf(_CO_PUBLISHER_FAILSAVEIMG, $filename).'<br>'.implode('<br>', $uploader->getErrors(false));
     }
 }
 

htdocs/modules/publisher/include/common.php

Spacing +5 added lines, -5 removed lines

@@ -24,11 +24,11 @@
 
 $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(

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

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
  */
 use XoopsModules\Publisher\Helper;
 
-require_once dirname(__DIR__) . '/include/common.php';
+require_once dirname(__DIR__).'/include/common.php';
 
 $helper = Helper::getInstance();
 
@@ -53,21 +53,21 @@ 
     // per their requirements.
 
     $url_arr = explode('/modules/', $_SERVER['PHP_SELF']);
-    $newUrl = $url_arr[0] . '/modules/' . PUBLISHER_DIRNAME . '/' . $seoMap[$seoOp];
+    $newUrl = $url_arr[0].'/modules/'.PUBLISHER_DIRNAME.'/'.$seoMap[$seoOp];
 
     $_ENV['PHP_SELF'] = $newUrl;
     $_SERVER['SCRIPT_NAME'] = $newUrl;
     $_SERVER['PHP_SELF'] = $newUrl;
     switch ($seoOp) {
         case 'category':
-            $_SERVER['REQUEST_URI'] = $newUrl . '?categoryid=' . $seoArg;
+            $_SERVER['REQUEST_URI'] = $newUrl.'?categoryid='.$seoArg;
             $_GET['categoryid'] = $seoArg;
             $_REQUEST['categoryid'] = $seoArg;
             break;
         case 'item':
         case 'print':
         default:
-            $_SERVER['REQUEST_URI'] = $newUrl . '?itemid=' . $seoArg;
+            $_SERVER['REQUEST_URI'] = $newUrl.'?itemid='.$seoArg;
             $_GET['itemid'] = $seoArg;
             $_REQUEST['itemid'] = $seoArg;
     }

htdocs/modules/publisher/include/plugin.tag.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         }
     }
     $itemHandler = $helper->getItemHandler();
-    $criteria = new Criteria('itemid', '(' . implode(', ', $items_id) . ')', 'IN');
+    $criteria = new Criteria('itemid', '('.implode(', ', $items_id).')', 'IN');
     $items_obj = $itemHandler->getItemObjects($criteria, 'itemid');
 
     /* @var Publisher\Item $item_obj */

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

Spacing +4 added lines, -4 removed lines

@@ -40,7 +40,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');
@@ -50,9 +50,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();
         }
@@ -70,7 +70,7 @@ 
             $start = max($pos - 100, 0);
             $length = mb_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

Spacing +5 added lines, -5 removed lines

@@ -22,7 +22,7 @@ 
 use Xoops\Core\XoopsTpl;
 use XoopsModules\Publisher;
 
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $xoops->disableErrorReporting();
 
@@ -37,7 +37,7 @@ 
     $categoryObj = $helper->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;
@@ -54,14 +54,14 @@ 
     $tpl->assign('channel_editor', $xoops->getConfig('adminmail'));
 
     if (-1 != $categoryid) {
-        $channel_category .= ' > ' . $categoryObj->getVar('name');
+        $channel_category .= ' > '.$categoryObj->getVar('name');
     }
 
     $tpl->assign('channel_category', htmlspecialchars($channel_category));
     $tpl->assign('channel_generator', $helper->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/pop.php

Spacing +1 added lines, -1 removed lines

@@ -23,4 +23,4 @@
 $doNotStartPrint = true;
 $noTitle         = true;
 $smartPopup      = true;
-require_once __DIR__ . '/print.php';
+require_once __DIR__.'/print.php';

htdocs/modules/publisher/archive.php

Spacing +7 added lines, -7 removed lines

@@ -33,7 +33,7 @@ 
 // # [11-may-2001] Kenneth Lee - http://www.nexgear.com/
 // ######################################################################
 
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $helper = Helper::getInstance();
@@ -61,7 +61,7 @@ 
 }
 
 $myts = Sanitizer::getInstance();
-$xoopsTpl->assign('xoops_pagetitle', $myts->htmlSpecialChars(_MD_PUBLISHER_ARCHIVES) . $pgtitle . ' - ' . $myts->htmlSpecialChars($xoopsModule->getVar('name')));
+$xoopsTpl->assign('xoops_pagetitle', $myts->htmlSpecialChars(_MD_PUBLISHER_ARCHIVES).$pgtitle.' - '.$myts->htmlSpecialChars($xoopsModule->getVar('name')));
 
 $useroffset = '';
 if ($xoops->isUser()) {
@@ -146,7 +146,7 @@ 
     $itemhandler->field_object = 'categoryid';
     // Categories for which user has access
     $categoriesGranted = $helper->getPermissionHandler()->getGrantedItems('category_read');
-    $grantedCategories = new Criteria('l.categoryid', '(' . implode(',', $categoriesGranted) . ')', 'IN');
+    $grantedCategories = new Criteria('l.categoryid', '('.implode(',', $categoriesGranted).')', 'IN');
     $criteria = new CriteriaCompo();
     $criteria->add($grantedCategories, 'AND');
     $criteria->add(new Criteria('o.status', 2), 'AND');
@@ -165,18 +165,18 @@ 
         foreach ($storyarray as $item) {
             $story = [];
             $htmltitle = '';
-            $story['title'] = "<a href='" . XoopsBaseConfig::get('url') . '/modules/publisher/category.php?categoryid=' . $item->getVar('categoryid') . "'>" . $item->getCategoryName() . "</a>: <a href='" . $item->getItemUrl() . "'" . $htmltitle . '>' . $item->title() . '</a>';
+            $story['title'] = "<a href='".XoopsBaseConfig::get('url').'/modules/publisher/category.php?categoryid='.$item->getVar('categoryid')."'>".$item->getCategoryName()."</a>: <a href='".$item->getItemUrl()."'".$htmltitle.'>'.$item->title().'</a>';
             $story['counter'] = $item->getVar('counter');
             $story['date'] = $item->datesub();
-            $story['print_link'] = XoopsBaseConfig::get('url') . '/modules/publisher/print.php?itemid=' . $item->getVar('itemid');
-            $story['mail_link'] = 'mailto:?subject=' . sprintf(_CO_PUBLISHER_INTITEM, $xoops->getConfig('sitename')) . '&amp;body=' . sprintf(_CO_PUBLISHER_INTITEMFOUND, $xoops->getConfig('sitename')) . ':  ' . $item->getItemUrl();
+            $story['print_link'] = XoopsBaseConfig::get('url').'/modules/publisher/print.php?itemid='.$item->getVar('itemid');
+            $story['mail_link'] = 'mailto:?subject='.sprintf(_CO_PUBLISHER_INTITEM, $xoops->getConfig('sitename')).'&amp;body='.sprintf(_CO_PUBLISHER_INTITEMFOUND, $xoops->getConfig('sitename')).':  '.$item->getItemUrl();
 
             $xoopsTpl->append('stories', $story);
         }
     }
     $xoopsTpl->assign('lang_printer', _MD_PUBLISHER_PRINTERFRIENDLY);
     $xoopsTpl->assign('lang_sendstory', _MD_PUBLISHER_SENDSTORY);
-    $xoopsTpl->assign('lang_storytotal', _MD_PUBLISHER_TOTAL_ITEMS . ' ' . $count);
+    $xoopsTpl->assign('lang_storytotal', _MD_PUBLISHER_TOTAL_ITEMS.' '.$count);
 } else {
     $xoopsTpl->assign('show_articles', false);
 }