geekwright / XoopsCore

htdocs/modules/publisher/src/helper.php

Spacing +1 added lines, -1 removed lines

@@ -121,7 +121,7 @@
         $ret = false;
         //        /** @var Connection $db */
         $db = XoopsDatabaseFactory::getConnection();
-        $class = '\\XoopsModules\\' . \ucfirst(mb_strtolower(\basename(\dirname(__DIR__)))) . '\\' . $name . 'Handler';
+        $class = '\\XoopsModules\\'.\ucfirst(mb_strtolower(\basename(\dirname(__DIR__)))).'\\'.$name.'Handler';
         $ret = new $class($db);
 
         return $ret;

htdocs/modules/publisher/author_items.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
  * @author          trabis <lusopoemas@gmail.com>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $uid = Request::getInt('uid');
@@ -88,7 +88,7 @@ 
 
 $xoopsTpl->assign('categories', $categories);
 
-$title = _MD_PUBLISHER_ITEMS_SAME_AUTHOR . ' - ' . $author_name;
+$title = _MD_PUBLISHER_ITEMS_SAME_AUTHOR.' - '.$author_name;
 
 /**
  * Generating meta information for this page

htdocs/modules/publisher/category.php

Spacing +7 added lines, -7 removed lines

@@ -24,7 +24,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $helper = Helper::getInstance();
@@ -60,7 +60,7 @@ 
 
 // Added by skalpa: custom template support
 if (!$template = $categoryObj->template()) {
-    $template = 'module:publisher/publisher_display_' . $helper->getConfig('idxcat_items_display_type') . '.tpl';
+    $template = 'module:publisher/publisher_display_'.$helper->getConfig('idxcat_items_display_type').'.tpl';
 }
 
 $xoops->header($template);
@@ -181,7 +181,7 @@ 
     for ($i = 0; $i < $totalItemOnPage; ++$i) {
         $item = $itemsObj[$i]->toArray('default', $helper->getConfig('item_title_size'));
         $item['categoryname'] = $categoryObj->getVar('name');
-        $item['categorylink'] = "<a href='" . Publisher\Utils::seoGenUrl('category', $itemsObj[$i]->getVar('categoryid'), $categoryObj->getVar('short_url')) . "'>" . $categoryObj->getVar('name') . '</a>';
+        $item['categorylink'] = "<a href='".Publisher\Utils::seoGenUrl('category', $itemsObj[$i]->getVar('categoryid'), $categoryObj->getVar('short_url'))."'>".$categoryObj->getVar('name').'</a>';
         $item['who_when'] = $itemsObj[$i]->getWhoAndWhen();
         $xoopsTpl->append('items', $item);
     }
@@ -211,11 +211,11 @@ 
 $xoopsTpl->assign('selected_category', $categoryid);
 
 // The Navigation Bar
-$pagenav = new XoopsPageNav($thiscategory_itemcount, $helper->getConfig('idxcat_index_perpage'), $start, 'start', 'categoryid=' . $categoryObj->getVar('categoryid'));
+$pagenav = new XoopsPageNav($thiscategory_itemcount, $helper->getConfig('idxcat_index_perpage'), $start, 'start', 'categoryid='.$categoryObj->getVar('categoryid'));
 if (1 == $helper->getConfig('format_image_nav')) {
-    $navbar = '<div style="text-align:right;">' . $pagenav->renderImageNav() . '</div>';
+    $navbar = '<div style="text-align:right;">'.$pagenav->renderImageNav().'</div>';
 } else {
-    $navbar = '<div style="text-align:right;">' . $pagenav->renderNav() . '</div>';
+    $navbar = '<div style="text-align:right;">'.$pagenav->renderNav().'</div>';
 }
 $xoopsTpl->assign('navbar', $navbar);
 
@@ -227,7 +227,7 @@ 
 
 // RSS Link
 if (1 == $helper->getConfig('idxcat_show_rss_link')) {
-    $link = sprintf("<a href='%s' title='%s'><img src='%s' border=0 alt='%s'></a>", PUBLISHER_URL . '/backend.php?categoryid=' . $categoryid, _MD_PUBLISHER_RSSFEED, PUBLISHER_URL . '/images/rss.gif', _MD_PUBLISHER_RSSFEED);
+    $link = sprintf("<a href='%s' title='%s'><img src='%s' border=0 alt='%s'></a>", PUBLISHER_URL.'/backend.php?categoryid='.$categoryid, _MD_PUBLISHER_RSSFEED, PUBLISHER_URL.'/images/rss.gif', _MD_PUBLISHER_RSSFEED);
     $xoopsTpl->assign('rssfeed_link', $link);
 }
 

htdocs/modules/publisher/list.tag.php

Spacing +2 added lines, -2 removed lines

@@ -19,5 +19,5 @@
  */
 
 // todo: Check this file
-require __DIR__ . '/header.php';
-include XoopsBaseConfig::get('root-path') . '/modules/tag/list.tag.php';
+require __DIR__.'/header.php';
+include XoopsBaseConfig::get('root-path').'/modules/tag/list.tag.php';

htdocs/modules/publisher/item.php

Spacing +11 added lines, -11 removed lines

@@ -25,7 +25,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $helper = Helper::getInstance();
 $myts = Sanitizer::getInstance();
@@ -49,9 +49,9 @@ 
 $xoops->header('module:publisher/publisher_item.tpl');
 $xoopsTpl = $xoops->tpl();
 $xoTheme = $xoops->theme();
-$xoTheme->addStylesheet(PUBLISHER_URL . '/css/jquery.popeye.style.css');
+$xoTheme->addStylesheet(PUBLISHER_URL.'/css/jquery.popeye.style.css');
 $xoTheme->addBaseScriptAssets('@jquery');
-$xoTheme->addScript(PUBLISHER_URL . '/js/jquery.popeye-2.0.4.js');
+$xoTheme->addScript(PUBLISHER_URL.'/js/jquery.popeye-2.0.4.js');
 $xoTheme->addBaseScriptAssets('modules/publisher/js/publisher.js');
 
 XoopsLoad::loadFile($helper->path('footer.php'));
@@ -137,7 +137,7 @@ 
     if (-1 == $item_page_id) {
         $item_page_id = 0;
     }
-    $pagenav = new XoopsPageNav($itemObj->getVar('pagescount'), 1, $item_page_id, 'page', 'itemid=' . $itemObj->getVar('itemid'));
+    $pagenav = new XoopsPageNav($itemObj->getVar('pagescount'), 1, $item_page_id, 'page', 'itemid='.$itemObj->getVar('itemid'));
     $xoopsTpl->assign('pagenav', $pagenav->renderNav());
 }
 
@@ -163,8 +163,8 @@ 
 
     if ('application/x-shockwave-flash' === $fileObj->getVar('mimetype')) {
         $file['content'] = $fileObj->displayFlash();
-        if (mb_strpos($item['maintext'], '[flash-' . $fileObj->getVar('fileid') . ']')) {
-            $item['maintext'] = str_replace('[flash-' . $fileObj->getVar('fileid') . ']', $file['content'], $item['maintext']);
+        if (mb_strpos($item['maintext'], '[flash-'.$fileObj->getVar('fileid').']')) {
+            $item['maintext'] = str_replace('[flash-'.$fileObj->getVar('fileid').']', $file['content'], $item['maintext']);
         } else {
             $embeded_files[] = $file;
         }
@@ -185,12 +185,12 @@ 
 unset($file, $embeded_files, $filesObj, $fileObj);
 
 // Language constants
-$xoopsTpl->assign('mail_link', 'mailto:?subject=' . sprintf(_CO_PUBLISHER_INTITEM, $xoops->getConfig('sitename')) . '&amp;body=' . sprintf(_CO_PUBLISHER_INTITEMFOUND, $xoops->getConfig('sitename')) . ': ' . $itemObj->getItemUrl());
+$xoopsTpl->assign('mail_link', 'mailto:?subject='.sprintf(_CO_PUBLISHER_INTITEM, $xoops->getConfig('sitename')).'&amp;body='.sprintf(_CO_PUBLISHER_INTITEMFOUND, $xoops->getConfig('sitename')).': '.$itemObj->getItemUrl());
 $xoopsTpl->assign('itemid', $itemObj->getVar('itemid'));
 $xoopsTpl->assign('sectionname', $helper->getModule()->getVar('name'));
 $xoopsTpl->assign('modulename', $helper->getModule()->getVar('dirname'));
 $xoopsTpl->assign('module_home', Publisher\Utils::moduleHome($helper->getConfig('format_linked_path')));
-$xoopsTpl->assign('categoryPath', $item['categoryPath'] . ' > ' . $item['title']);
+$xoopsTpl->assign('categoryPath', $item['categoryPath'].' > '.$item['title']);
 $xoopsTpl->assign('commentatarticlelevel', $helper->getConfig('perm_com_art_level'));
 $xoopsTpl->assign('com_rule', $helper->getConfig('com_rule'));
 $xoopsTpl->assign('other_items', $helper->getConfig('item_other_items_type'));
@@ -199,7 +199,7 @@ 
 
 // tags support
 if ($xoops->isActiveModule('tag')) {
-    require_once XoopsBaseConfig::get('root-path') . '/modules/tag/include/tagbar.php';
+    require_once XoopsBaseConfig::get('root-path').'/modules/tag/include/tagbar.php';
     $xoopsTpl->assign('tagbar', tagBar($itemid, $catid = 0));
 }
 
@@ -227,8 +227,8 @@ 
 if ($helper->getConfig('perm_rating')) {
     $xoopsTpl->assign('rating_enabled', true);
     $item['ratingbar'] = Publisher\Utils::ratingBar($itemid);
-    $xoTheme->addScript(PUBLISHER_URL . '/js/behavior.js');
-    $xoTheme->addScript(PUBLISHER_URL . '/js/rating.js');
+    $xoTheme->addScript(PUBLISHER_URL.'/js/behavior.js');
+    $xoTheme->addScript(PUBLISHER_URL.'/js/rating.js');
 }
 
 $xoopsTpl->assign('item', $item);

htdocs/modules/publisher/view.tag.php

Spacing +2 added lines, -2 removed lines

@@ -19,5 +19,5 @@
  */
 
 //todo: check this file
-require_once __DIR__ . '/header.php';
-require_once XoopsBaseConfig::get('root-path') . '/modules/tag/view.tag.php';
+require_once __DIR__.'/header.php';
+require_once XoopsBaseConfig::get('root-path').'/modules/tag/view.tag.php';

htdocs/modules/publisher/index.php

Spacing +7 added lines, -7 removed lines

@@ -25,7 +25,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $helper = Helper::getInstance();
@@ -45,7 +45,7 @@ 
     $xoops->redirect(XoopsBaseConfig::get('url'), 2, _MD_PUBLISHER_NO_TOP_PERMISSIONS);
 }
 
-$xoops->header('module:publisher/publisher_display' . '_' . $helper->getConfig('idxcat_items_display_type') . '.tpl');
+$xoops->header('module:publisher/publisher_display'.'_'.$helper->getConfig('idxcat_items_display_type').'.tpl');
 $xoopsTpl = $xoops->tpl();
 XoopsLoad::loadFile($helper->path('footer.php'));
 
@@ -192,16 +192,16 @@ 
 // Category Navigation Bar
 $pagenav = new XoopsPageNav($totalCategories, $helper->getConfig('idxcat_cat_perpage'), $catstart, 'catstart', '');
 if (1 == $helper->getConfig('format_image_nav')) {
-    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">' . $pagenav->renderImageNav() . '</div>');
+    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">'.$pagenav->renderImageNav().'</div>');
 } else {
-    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">' . $pagenav->renderNav() . '</div>');
+    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">'.$pagenav->renderNav().'</div>');
 }
 // ITEM Navigation Bar
 $pagenav = new XoopsPageNav($real_total_items, $helper->getConfig('idxcat_index_perpage'), $start, 'start', '');
 if (1 == $helper->getConfig('format_image_nav')) {
-    $xoopsTpl->assign('navbar', '<div style="text-align:right;">' . $pagenav->renderImageNav() . '</div>');
+    $xoopsTpl->assign('navbar', '<div style="text-align:right;">'.$pagenav->renderImageNav().'</div>');
 } else {
-    $xoopsTpl->assign('navbar', '<div style="text-align:right;">' . $pagenav->renderNav() . '</div>');
+    $xoopsTpl->assign('navbar', '<div style="text-align:right;">'.$pagenav->renderNav().'</div>');
 }
 //show subcategories
 $xoopsTpl->assign('show_subcats', $helper->getConfig('idxcat_show_subcats'));
@@ -215,7 +215,7 @@ 
 
 // RSS Link
 if (1 == $helper->getConfig('idxcat_show_rss_link')) {
-    $link = sprintf("<a href='%s' title='%s'><img src='%s' border=0 alt='%s'></a>", PUBLISHER_URL . '/backend.php', _MD_PUBLISHER_RSSFEED, PUBLISHER_URL . '/images/rss.gif', _MD_PUBLISHER_RSSFEED);
+    $link = sprintf("<a href='%s' title='%s'><img src='%s' border=0 alt='%s'></a>", PUBLISHER_URL.'/backend.php', _MD_PUBLISHER_RSSFEED, PUBLISHER_URL.'/images/rss.gif', _MD_PUBLISHER_RSSFEED);
     $xoopsTpl->assign('rssfeed_link', $link);
 }
 

htdocs/modules/publisher/file.php

Indentation +5 added lines, -5 removed lines

@@ -111,11 +111,11 @@
             // no confirm: show deletion condition
             $xoops->header();
             echo $xoops->confirm([
-                                     'op' => 'del',
-                                     'fileid' => $fileObj->getVar('fileid'),
-                                     'confirm' => 1,
-                                     'name' => $fileObj->getVar('name'),
-                                 ], 'file.php', _AM_PUBLISHER_DELETETHISFILE . ' <br>' . $fileObj->getVar('name') . ' <br> <br>', _AM_PUBLISHER_DELETE);
+                                        'op' => 'del',
+                                        'fileid' => $fileObj->getVar('fileid'),
+                                        'confirm' => 1,
+                                        'name' => $fileObj->getVar('name'),
+                                    ], 'file.php', _AM_PUBLISHER_DELETETHISFILE . ' <br>' . $fileObj->getVar('name') . ' <br> <br>', _AM_PUBLISHER_DELETE);
             $xoops->footer();
         }
         break;

Spacing +6 added lines, -6 removed lines

@@ -24,7 +24,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $helper = Helper::getInstance();
 $helper->loadLanguage('admin');
@@ -95,18 +95,18 @@ 
         }
 
         if (!$helper->getFileHandler()->insert($fileObj)) {
-            $xoops->redirect('item.php?itemid=' . $fileObj->getVar('itemid'), 3, _AM_PUBLISHER_FILE_EDITING_ERROR . Publisher\Utils::formatErrors($fileObj->getErrors()));
+            $xoops->redirect('item.php?itemid='.$fileObj->getVar('itemid'), 3, _AM_PUBLISHER_FILE_EDITING_ERROR.Publisher\Utils::formatErrors($fileObj->getErrors()));
         }
-        $xoops->redirect('item.php?itemid=' . $fileObj->getVar('itemid'), 2, _AM_PUBLISHER_FILE_EDITING_SUCCESS);
+        $xoops->redirect('item.php?itemid='.$fileObj->getVar('itemid'), 2, _AM_PUBLISHER_FILE_EDITING_SUCCESS);
         break;
     case 'del':
         $confirm = $_POST['confirm'] ?? 0;
 
         if ($confirm) {
             if (!$helper->getFileHandler()->delete($fileObj)) {
-                $xoops->redirect('item.php?itemid=' . $fileObj->getVar('itemid'), 2, _AM_PUBLISHER_FILE_DELETE_ERROR);
+                $xoops->redirect('item.php?itemid='.$fileObj->getVar('itemid'), 2, _AM_PUBLISHER_FILE_DELETE_ERROR);
             }
-            $xoops->redirect('item.php?itemid=' . $fileObj->getVar('itemid'), 2, sprintf(_AM_PUBLISHER_FILEISDELETED, $fileObj->getVar('name')));
+            $xoops->redirect('item.php?itemid='.$fileObj->getVar('itemid'), 2, sprintf(_AM_PUBLISHER_FILEISDELETED, $fileObj->getVar('name')));
         } else {
             // no confirm: show deletion condition
             $xoops->header();
@@ -115,7 +115,7 @@ 
                                      'fileid' => $fileObj->getVar('fileid'),
                                      'confirm' => 1,
                                      'name' => $fileObj->getVar('name'),
-                                 ], 'file.php', _AM_PUBLISHER_DELETETHISFILE . ' <br>' . $fileObj->getVar('name') . ' <br> <br>', _AM_PUBLISHER_DELETE);
+                                 ], 'file.php', _AM_PUBLISHER_DELETETHISFILE.' <br>'.$fileObj->getVar('name').' <br> <br>', _AM_PUBLISHER_DELETE);
             $xoops->footer();
         }
         break;

htdocs/modules/publisher/submit.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
  * @author          The SmartFactory <www.smartfactory.ca>
  * @version         $Id$
  */
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 $xoops = Xoops::getInstance();
 $helper = Helper::getInstance();
 $helper->loadLanguage('admin');
@@ -117,7 +117,7 @@ 
     'author_alias',
 ];
 foreach ($elements as $element) {
-    if (isset($_REQUEST[$element]) && !in_array(constant('_PUBLISHER_' . mb_strtoupper($element)), $form_view)) {
+    if (isset($_REQUEST[$element]) && !in_array(constant('_PUBLISHER_'.mb_strtoupper($element)), $form_view)) {
         $xoops->redirect('index.php', 1, _MD_PUBLISHER_SUBMIT_ERROR);
     }
 }
@@ -131,12 +131,12 @@ 
 
         if ($confirm) {
             if (!$helper->getItemHandler()->delete($itemObj)) {
-                $xoops->redirect('index.php', 2, _AM_PUBLISHER_ITEM_DELETE_ERROR . Publisher\Utils::formatErrors($itemObj->getErrors()));
+                $xoops->redirect('index.php', 2, _AM_PUBLISHER_ITEM_DELETE_ERROR.Publisher\Utils::formatErrors($itemObj->getErrors()));
             }
             $xoops->redirect('index.php', 2, sprintf(_AM_PUBLISHER_ITEMISDELETED, $itemObj->title()));
         } else {
             $xoops->header();
-            echo $xoops->confirm(['op' => 'del', 'itemid' => $itemObj->getVar('itemid'), 'confirm' => 1, 'name' => $itemObj->title()], 'submit.php', _AM_PUBLISHER_DELETETHISITEM . " <br>'" . $itemObj->title() . "'. <br> <br>", _AM_PUBLISHER_DELETE);
+            echo $xoops->confirm(['op' => 'del', 'itemid' => $itemObj->getVar('itemid'), 'confirm' => 1, 'name' => $itemObj->title()], 'submit.php', _AM_PUBLISHER_DELETETHISITEM." <br>'".$itemObj->title()."'. <br> <br>", _AM_PUBLISHER_DELETE);
             $xoops->footer();
         }
         break;
@@ -229,7 +229,7 @@ 
         $xoops->header('module:publisher/publisher_submit.tpl');
         $xoopsTpl = $xoops->tpl();
         $xoTheme = $xoops->theme();
-        $xoTheme->addScript(PUBLISHER_URL . '/js/publisher.js');
+        $xoTheme->addScript(PUBLISHER_URL.'/js/publisher.js');
         XoopsLoad::loadFile($helper->path('footer.php'));
 
         $itemObj->setVarsFromRequest();