geekwright / XoopsCore

htdocs/modules/publisher/admin/clone.php

Spacing +6 added lines, -6 removed lines

@@ -23,7 +23,7 @@ 
 use Xoops\Form\ThemeForm;
 use XoopsModules\Publisher;
 
-require_once __DIR__ . '/admin_header.php';
+require_once __DIR__.'/admin_header.php';
 
 $xoops = Xoops::getInstance();
 Publisher\Utils::cpHeader();
@@ -44,7 +44,7 @@ 
     }
 
     // Check wether the cloned module exists or not
-    if ($clone && is_dir(XoopsBaseConfig::get('root-path') . '/modules/' . $clone)) {
+    if ($clone && is_dir(XoopsBaseConfig::get('root-path').'/modules/'.$clone)) {
         $xoops->redirect('clone.php', 3, sprintf(_AM_PUBLISHER_CLONE_EXISTS, $clone));
     }
 
@@ -60,8 +60,8 @@ 
     $logocreated = publisher_createLogo(mb_strtolower($clone));
 
     $msg = '';
-    if (is_dir(XoopsBaseConfig::get('root-path') . '/modules/' . mb_strtolower($clone))) {
-        $msg .= sprintf(_AM_PUBLISHER_CLONE_CONGRAT, "<a href='" . XoopsBaseConfig::get('url') . "/modules/system/admin.php?fct=modulesadmin'>" . ucfirst(mb_strtolower($clone)) . '</a>') . "<br>\n";
+    if (is_dir(XoopsBaseConfig::get('root-path').'/modules/'.mb_strtolower($clone))) {
+        $msg .= sprintf(_AM_PUBLISHER_CLONE_CONGRAT, "<a href='".XoopsBaseConfig::get('url')."/modules/system/admin.php?fct=modulesadmin'>".ucfirst(mb_strtolower($clone)).'</a>')."<br>\n";
         if (!$logocreated) {
             $msg .= _AM_PUBLISHER_CLONE_IMAGEFAIL;
         }
@@ -155,7 +155,7 @@ 
         }
     }
 
-    if (!XoopsLoad::fileExists($imageBase = XoopsBaseConfig::get('root-path') . '/modules/' . $dirname . '/images/module_logo.png') || !XoopsLoad::fileExists($font = XoopsBaseConfig::get('root-path') . '/modules/' . $dirname . '/images/VeraBd.ttf')) {
+    if (!XoopsLoad::fileExists($imageBase = XoopsBaseConfig::get('root-path').'/modules/'.$dirname.'/images/module_logo.png') || !XoopsLoad::fileExists($font = XoopsBaseConfig::get('root-path').'/modules/'.$dirname.'/images/VeraBd.ttf')) {
         return false;
     }
 
@@ -174,7 +174,7 @@ 
     $white = imagecolorallocatealpha($imageModule, 255, 255, 255, 127);
     imagefill($imageModule, 0, 0, $white);
     imagecolortransparent($imageModule, $white);
-    imagepng($imageModule, XoopsBaseConfig::get('root-path') . '/modules/' . $dirname . '/images/module_logo.png');
+    imagepng($imageModule, XoopsBaseConfig::get('root-path').'/modules/'.$dirname.'/images/module_logo.png');
     imagedestroy($imageModule);
 
     return true;

htdocs/modules/publisher/makepdf.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
  * @author          trabis <lusopoemas@gmail.com>
  * @author          Sina Asghari (AKA stranger) <stranger@impresscms.ir>
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $xoops->disableErrorReporting();
 
@@ -73,6 +73,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/search.php

Spacing +29 added lines, -29 removed lines

@@ -24,7 +24,7 @@ 
  * @author          Taiwen Jiang <phppp@users.sourceforge.net>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 
 $helper = Helper::getInstance();
@@ -34,7 +34,7 @@ 
 //Checking general permissions
 $xoopsConfigSearch = $xoops->getConfigs();
 if (empty($xoopsConfigSearch['enable_search'])) {
-    $xoops->redirect(PUBLISHER_URL . '/index.php', 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
+    $xoops->redirect(PUBLISHER_URL.'/index.php', 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
 }
 
 $groups = $xoops->getUserGroups();
@@ -93,12 +93,12 @@ 
             }
         }
         if (0 == count($queries)) {
-            $xoops->redirect(PUBLISHER_URL . '/search.php', 2, sprintf(XoopsLocale::EF_KEYWORDS_MUST_BE_GREATER_THAN, $xoopsConfigSearch['keyword_min']));
+            $xoops->redirect(PUBLISHER_URL.'/search.php', 2, sprintf(XoopsLocale::EF_KEYWORDS_MUST_BE_GREATER_THAN, $xoopsConfigSearch['keyword_min']));
             exit();
         }
     } else {
         if (mb_strlen($query) < $xoopsConfigSearch['keyword_min']) {
-            $xoops->redirect(PUBLISHER_URL . '/search.php', 2, sprintf(XoopsLocale::EF_KEYWORDS_MUST_BE_GREATER_THAN, $xoopsConfigSearch['keyword_min']));
+            $xoops->redirect(PUBLISHER_URL.'/search.php', 2, sprintf(XoopsLocale::EF_KEYWORDS_MUST_BE_GREATER_THAN, $xoopsConfigSearch['keyword_min']));
             exit();
         }
         $queries = [$query];
@@ -110,8 +110,8 @@ 
     if (!empty($search_username)) {
         $uname_required = true;
         $search_username = $search_username;
-        if (!$result = $xoopsDB->query('SELECT uid FROM ' . $xoopsDB->prefix('users') . ' WHERE uname LIKE ' . $xoopsDB->quoteString("%$search_username%"))) {
-            $xoops->redirect(PUBLISHER_URL . '/search.php', 1, _CO_PUBLISHER_ERROR);
+        if (!$result = $xoopsDB->query('SELECT uid FROM '.$xoopsDB->prefix('users').' WHERE uname LIKE '.$xoopsDB->quoteString("%$search_username%"))) {
+            $xoops->redirect(PUBLISHER_URL.'/search.php', 1, _CO_PUBLISHER_ERROR);
             exit();
         }
         $uid = [];
@@ -156,29 +156,29 @@ 
         unset($next_search);
         unset($items);
     }
-    $search_url = $helper->url('search.php?' . $paras);
+    $search_url = $helper->url('search.php?'.$paras);
 
     if (count($results)) {
         $next = $start + $limit;
         $queries = implode(',', $queries);
-        $search_url_next = $search_url . "&start={$next}";
-        $search_next = '<a href="' . htmlspecialchars($search_url_next) . '">' . XoopsLocale::NEXT . '</a>';
+        $search_url_next = $search_url."&start={$next}";
+        $search_next = '<a href="'.htmlspecialchars($search_url_next).'">'.XoopsLocale::NEXT.'</a>';
         $xoopsTpl->assign('search_next', $search_next);
     }
     if ($start > 0) {
         $prev = $start - $limit;
-        $search_url_prev = $search_url . "&start={$prev}";
-        $search_prev = '<a href="' . htmlspecialchars($search_url_prev) . '">' . XoopsLocale::PREVIOUS . '</a>';
+        $search_url_prev = $search_url."&start={$prev}";
+        $search_prev = '<a href="'.htmlspecialchars($search_url_prev).'">'.XoopsLocale::PREVIOUS.'</a>';
         $xoopsTpl->assign('search_prev', $search_prev);
     }
 
     unset($results);
-    $search_info = XoopsLocale::KEYWORDS . ': ' . $myts->htmlSpecialChars($term);
+    $search_info = XoopsLocale::KEYWORDS.': '.$myts->htmlSpecialChars($term);
     if ($uname_required) {
         if ($search_info) {
             $search_info .= '<br>';
         }
-        $search_info .= _CO_PUBLISHER_UID . ': ' . $myts->htmlSpecialChars($search_username);
+        $search_info .= _CO_PUBLISHER_UID.': '.$myts->htmlSpecialChars($search_username);
     }
     $xoopsTpl->assign('search_info', $search_info);
 }
@@ -189,17 +189,17 @@ 
 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 */
@@ -210,13 +210,13 @@ 
 if (empty($category) || 0 == count($category)) {
     $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>';
 
@@ -226,32 +226,32 @@ 
 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">';
@@ -259,22 +259,22 @@ 
 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>';
 
 $xoopsTpl->assign('type_select', $type_select);

htdocs/modules/publisher/rate.php

Spacing +5 added lines, -5 removed lines

@@ -20,7 +20,7 @@ 
  * @author          trabis <lusopoemas@gmail.com>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 
@@ -36,11 +36,11 @@ 
 //Checking permissions
 if (!$helper->getConfig('perm_rating')
     || !$gpermHandler->checkRight('global', _PUBLISHER_RATE, $groups, $module_id)) {
-    $xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
+    $xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
 }
 
 if ($rating > 5 || $rating < 1) {
-    $xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, _MD_PUBLISHER_VOTE_BAD);
+    $xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, _MD_PUBLISHER_VOTE_BAD);
 }
 
 $criteria = new Criteria('itemid', $itemid);
@@ -61,7 +61,7 @@ 
 }
 
 if ($voted) {
-    $xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, _MD_PUBLISHER_VOTE_ALREADY);
+    $xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, _MD_PUBLISHER_VOTE_ALREADY);
 }
 
 $newRatingObj = $helper->getRatingHandler()->create();
@@ -78,4 +78,4 @@ 
 $helper->getItemHandler()->updateAll('rating', number_format($current_rating / $count, 4), $criteria, true);
 $helper->getItemHandler()->updateAll('votes', $count, $criteria, true);
 
-$xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, _MD_PUBLISHER_VOTE_THANKS);
+$xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, _MD_PUBLISHER_VOTE_THANKS);

htdocs/modules/publisher/footer.php

Spacing +4 added lines, -4 removed lines

@@ -23,17 +23,17 @@
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once __DIR__ . '/include/common.php';
+require_once __DIR__.'/include/common.php';
 
 $helper = Helper::getInstance();
 
 $xoops = Xoops::getInstance();
-$xoops->theme()->addStylesheet(PUBLISHER_URL . '/css/publisher.css');
+$xoops->theme()->addStylesheet(PUBLISHER_URL.'/css/publisher.css');
 
 $xoopsTpl = $xoops->tpl();
-$xoopsTpl->assign('xoops_module_header', '<link rel="alternate" type="application/rss+xml" title="' . $helper->getModule()->getVar('name') . '" href="' . $helper->url('backend.php') . '">' . @$xoopsTpl->getTemplateVars('xoops_module_header'));
+$xoopsTpl->assign('xoops_module_header', '<link rel="alternate" type="application/rss+xml" title="'.$helper->getModule()->getVar('name').'" href="'.$helper->url('backend.php').'">'.@$xoopsTpl->getTemplateVars('xoops_module_header'));
 
-$xoopsTpl->assign('publisher_adminpage', "<a href='" . $helper->url('admin/index.php') . "'>" . _MD_PUBLISHER_ADMIN_PAGE . '</a>');
+$xoopsTpl->assign('publisher_adminpage', "<a href='".$helper->url('admin/index.php')."'>"._MD_PUBLISHER_ADMIN_PAGE.'</a>');
 $xoopsTpl->assign('isAdmin', Publisher\Utils::IsUserAdmin());
 $xoopsTpl->assign('publisher_url', $helper->url());
 $xoopsTpl->assign('publisher_images_url', $helper->url('images'));

htdocs/modules/publisher/print.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
  * @author          trabis <lusopoemas@gmail.com>
  * @author          The SmartFactory <www.smartfactory.ca>
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $xoops->disableErrorReporting();
 
@@ -58,10 +58,10 @@ 
 
 $mainImage = $itemObj->getMainImage();
 if ('' != $mainImage['image_path']) {
-    $item['image'] = '<img src="' . $mainImage['image_path'] . '" alt="' . $myts->undoHtmlSpecialChars($mainImage['image_name']) . '">';
+    $item['image'] = '<img src="'.$mainImage['image_path'].'" alt="'.$myts->undoHtmlSpecialChars($mainImage['image_name']).'">';
 }
 $xoopsTpl->assign('item', $item);
-$xoopsTpl->assign('printtitle', $xoops->getConfig('sitename') . ' - ' . Publisher\Utils::html2text($categoryObj->getCategoryPath()) . ' > ' . $myts->displayTarea($itemObj->title()));
+$xoopsTpl->assign('printtitle', $xoops->getConfig('sitename').' - '.Publisher\Utils::html2text($categoryObj->getCategoryPath()).' > '.$myts->displayTarea($itemObj->title()));
 $xoopsTpl->assign('printlogourl', $helper->getConfig('print_logourl'));
 $xoopsTpl->assign('printheader', $myts->displayTarea($helper->getConfig('print_header'), 1));
 $xoopsTpl->assign('lang_category', _CO_PUBLISHER_CATEGORY);