Passed
Pull Request — master (#585)
by
unknown
17:41
created
htdocs/modules/debugbar/class/debugbarlogger.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -177,12 +177,12 @@  discard block
 block discarded – undo
177 177
                 $excludes = array(
178 178
                     //'/vendor/font-awesome/', // font-awesome needs special process
179 179
                     //'/vendor/highlightjs/',  // highlightjs has some negative side effects
180
-                    '/vendor/jquery/',       // jquery is already available
180
+                    '/vendor/jquery/', // jquery is already available
181 181
                 );
182 182
 
183 183
                 $cssAssets = array_filter(
184 184
                     $cssAssets,
185
-                    function ($filename) use ($excludes) {
185
+                    function($filename) use ($excludes) {
186 186
                         foreach ($excludes as $exclude) {
187 187
                             if (false !== strpos($filename, $exclude)) {
188 188
                                 return false;
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
 
195 195
                 $jsAssets = array_filter(
196 196
                     $jsAssets,
197
-                    function ($filename) use ($excludes) {
197
+                    function($filename) use ($excludes) {
198 198
                         foreach ($excludes as $exclude) {
199 199
                             if (false !== strpos($filename, $exclude)) {
200 200
                                 return false;
@@ -390,7 +390,7 @@  discard block
 block discarded – undo
390 390
     {
391 391
         if ($this->activated) {
392 392
             $this->debugbar->stackData();
393
-            $this->activated=false;
393
+            $this->activated = false;
394 394
             $this->renderingEnabled = false;
395 395
         }
396 396
     }
@@ -431,9 +431,9 @@  discard block
 block discarded – undo
431 431
         $pattern = '<!--<xo-logger-output>-->';
432 432
         $pos = strpos($output, $pattern);
433 433
         if ($pos !== false) {
434
-            return substr($output, 0, $pos) . $log . substr($output, $pos + strlen($pattern));
434
+            return substr($output, 0, $pos).$log.substr($output, $pos + strlen($pattern));
435 435
         } else {
436
-            return $output . $log;
436
+            return $output.$log;
437 437
         }
438 438
     }
439 439
 
@@ -452,15 +452,15 @@  discard block
 block discarded – undo
452 452
             }
453 453
             $this->addToTheme();
454 454
             $this->addExtra(_MD_DEBUGBAR_PHP_VERSION, PHP_VERSION);
455
-            $this->addExtra(_MD_DEBUGBAR_INCLUDED_FILES, (string) count(get_included_files()));
455
+            $this->addExtra(_MD_DEBUGBAR_INCLUDED_FILES, (string)count(get_included_files()));
456 456
             $conn = \Xoops::getInstance()->db()->getWrappedConnection();
457 457
             if ($conn instanceof \PDO) {
458 458
                 $this->addExtra(
459
-                    $conn->getAttribute(\PDO::ATTR_DRIVER_NAME) . ' version',
459
+                    $conn->getAttribute(\PDO::ATTR_DRIVER_NAME).' version',
460 460
                     $conn->getAttribute(\PDO::ATTR_SERVER_VERSION)
461 461
                 );
462 462
                 $this->addExtra(
463
-                    $conn->getAttribute(\PDO::ATTR_DRIVER_NAME) . ' stats',
463
+                    $conn->getAttribute(\PDO::ATTR_DRIVER_NAME).' stats',
464 464
                     $conn->getAttribute(\PDO::ATTR_SERVER_INFO)
465 465
                 );
466 466
             }
@@ -638,7 +638,7 @@  discard block
 block discarded – undo
638 638
             switch ($chan) {
639 639
                 case 'blocks':
640 640
                     $channel = 'Blocks';
641
-                    $msg = $message . ': ';
641
+                    $msg = $message.': ';
642 642
                     if ($context['cached']) {
643 643
                         $msg .= sprintf(_MD_DEBUGBAR_CACHED, (int)($context['cachetime']));
644 644
                     } else {
@@ -651,7 +651,7 @@  discard block
 block discarded – undo
651 651
                     break;
652 652
                 case 'extra':
653 653
                     $channel = 'Extra';
654
-                    $msg = $context['name'] . ': ' . $message;
654
+                    $msg = $context['name'].': '.$message;
655 655
                     break;
656 656
                 case 'queries':
657 657
                     $channel = 'Queries';
@@ -663,11 +663,11 @@  discard block
 block discarded – undo
663 663
                         //    \Xmf\Debug::dump($context);
664 664
                         //}
665 665
                         $msg .= ' -- Error number: '
666
-                            . (is_scalar($context['errno']) ?  $context['errno'] : '?')
666
+                            . (is_scalar($context['errno']) ? $context['errno'] : '?')
667 667
                             . ' Error message: '
668
-                            . (is_scalar($context['error']) ?  $context['error'] : '?');
668
+                            . (is_scalar($context['error']) ? $context['error'] : '?');
669 669
                     }
670
-                    $msg = $qt . $msg;
670
+                    $msg = $qt.$msg;
671 671
                     break;
672 672
             }
673 673
         }
Please login to merge, or discard this patch.
htdocs/modules/debugbar/class/plugin/userconfigs.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
      */
59 59
     public function configs()
60 60
     {
61
-        $config[]=array(
61
+        $config[] = array(
62 62
             'name' => 'debugbar_enable',
63 63
             'title' => '_MI_DEBUGBAR_UCONF_ENABLE_BAR',
64 64
             'description' => '',
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
             'category' => 'cat_options'
70 70
         );
71 71
 
72
-        $config[]=array(
72
+        $config[] = array(
73 73
             'name' => 'debug_smarty_enable',
74 74
             'title' => '_MI_DEBUGBAR_UCONF_ENABLE_SMARTY',
75 75
             'description' => '',
Please login to merge, or discard this patch.
htdocs/modules/publisher/pop.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,4 +23,4 @@
 block discarded – undo
23 23
 $doNotStartPrint = true;
24 24
 $noTitle = true;
25 25
 $smartPopup = true;
26
-include_once __DIR__ . '/print.php';
26
+include_once __DIR__.'/print.php';
Please login to merge, or discard this patch.
htdocs/modules/publisher/rate.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
  * @version         $Id$
21 21
  */
22 22
 
23
-include_once __DIR__ . '/header.php';
23
+include_once __DIR__.'/header.php';
24 24
 
25 25
 $xoops = Xoops::getInstance();
26 26
 
@@ -37,11 +37,11 @@  discard block
 block discarded – undo
37 37
 if (!$publisher->getConfig('perm_rating')
38 38
     || !$gperm_handler->checkRight('global', _PUBLISHER_RATE, $groups, $module_id)
39 39
 ) {
40
-    $xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
40
+    $xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, XoopsLocale::E_NO_ACCESS_PERMISSION);
41 41
 }
42 42
 
43 43
 if ($rating > 5 || $rating < 1) {
44
-    $xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, _MD_PUBLISHER_VOTE_BAD);
44
+    $xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, _MD_PUBLISHER_VOTE_BAD);
45 45
 }
46 46
 
47 47
 $criteria = new Criteria('itemid', $itemid);
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 }
63 63
 
64 64
 if ($voted) {
65
-    $xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, _MD_PUBLISHER_VOTE_ALREADY);
65
+    $xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, _MD_PUBLISHER_VOTE_ALREADY);
66 66
 }
67 67
 
68 68
 $newRatingObj = $publisher->getRatingHandler()->create();
@@ -79,4 +79,4 @@  discard block
 block discarded – undo
79 79
 $publisher->getItemHandler()->updateAll('rating', number_format($current_rating / $count, 4), $criteria, true);
80 80
 $publisher->getItemHandler()->updateAll('votes', $count, $criteria, true);
81 81
 
82
-$xoops->redirect(PUBLISHER_URL . '/item.php?itemid=' . $itemid, 2, _MD_PUBLISHER_VOTE_THANKS);
82
+$xoops->redirect(PUBLISHER_URL.'/item.php?itemid='.$itemid, 2, _MD_PUBLISHER_VOTE_THANKS);
Please login to merge, or discard this patch.
htdocs/modules/publisher/admin/about.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
  * @version      $Id$
21 21
  */
22 22
 
23
-include __DIR__ . '/admin_header.php';
23
+include __DIR__.'/admin_header.php';
24 24
 $xoops = Xoops::getInstance();
25 25
 $xoops->header();
26 26
 
Please login to merge, or discard this patch.
htdocs/modules/publisher/admin/import/smartsection.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -20,10 +20,10 @@  discard block
 block discarded – undo
20 20
  * @version         $Id$
21 21
  */
22 22
 
23
-include_once dirname(__DIR__) . '/admin_header.php';
23
+include_once dirname(__DIR__).'/admin_header.php';
24 24
 $myts = \Xoops\Core\Text\Sanitizer::getInstance();
25 25
 
26
-$importFromModuleName = "Smartsection " . @$_POST['smartsection_version'];
26
+$importFromModuleName = "Smartsection ".@$_POST['smartsection_version'];
27 27
 
28 28
 $scriptname = "smartsection.php";
29 29
 
@@ -39,26 +39,26 @@  discard block
 block discarded – undo
39 39
     //publisher_adminMenu(-1, _AM_PUBLISHER_IMPORT);
40 40
     PublisherUtils::openCollapsableBar('newsimport', 'newsimporticon', sprintf(_AM_PUBLISHER_IMPORT_FROM, $importFromModuleName), _AM_PUBLISHER_IMPORT_INFO);
41 41
 
42
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix("smartsection_categories"));
42
+    $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix("smartsection_categories"));
43 43
     list ($totalCat) = $xoopsDB->fetchRow($result);
44 44
 
45 45
     if ($totalCat == 0) {
46
-        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . _AM_PUBLISHER_IMPORT_NO_CATEGORY . "</span>";
46
+        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">"._AM_PUBLISHER_IMPORT_NO_CATEGORY."</span>";
47 47
     } else {
48
-        include_once \XoopsBaseConfig::get('root-path') . '/class/xoopstree.php';
48
+        include_once \XoopsBaseConfig::get('root-path').'/class/xoopstree.php';
49 49
 
50
-        $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix('smartsection_items'));
50
+        $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix('smartsection_items'));
51 51
         list ($totalArticles) = $xoopsDB->fetchRow($result);
52 52
 
53 53
         if ($totalArticles == 0) {
54
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles) . "</span>";
54
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles)."</span>";
55 55
         } else {
56
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat) . "</span>";
56
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat)."</span>";
57 57
 
58
-            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL . "/import/{$scriptname}");
58
+            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL."/import/{$scriptname}");
59 59
 
60 60
             // Categories to be imported
61
-            $sql = "SELECT cat.categoryid, cat.parentid, cat.name, COUNT(art.itemid) FROM " . $xoopsDB->prefix('smartsection_categories') . " AS cat INNER JOIN " . $xoopsDB->prefix('smartsection_items') . " AS art ON cat.categoryid=art.categoryid GROUP BY art.categoryid";
61
+            $sql = "SELECT cat.categoryid, cat.parentid, cat.name, COUNT(art.itemid) FROM ".$xoopsDB->prefix('smartsection_categories')." AS cat INNER JOIN ".$xoopsDB->prefix('smartsection_items')." AS art ON cat.categoryid=art.categoryid GROUP BY art.categoryid";
62 62
 
63 63
             $result = $xoopsDB->query($sql);
64 64
             $cat_cbox_options = array();
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 
112 112
     $parentId = $_POST['parent_category'];
113 113
 
114
-    $sql = "SELECT * FROM " . $xoopsDB->prefix('smartsection_categories');
114
+    $sql = "SELECT * FROM ".$xoopsDB->prefix('smartsection_categories');
115 115
 
116 116
     $resultCat = $xoopsDB->query($sql);
117 117
 
@@ -131,20 +131,20 @@  discard block
 block discarded – undo
131 131
 
132 132
         // Copy category image
133 133
         if (($arrCat['image'] !== 'blank.gif') && ($arrCat['image'] != '')) {
134
-            copy(\XoopsBaseConfig::get('root-path') . "/uploads/smartsection/images/category/" . $arrCat['image'], \XoopsBaseConfig::get('root-path') . "/uploads/publisher/images/category/" . $arrCat['image']);
134
+            copy(\XoopsBaseConfig::get('root-path')."/uploads/smartsection/images/category/".$arrCat['image'], \XoopsBaseConfig::get('root-path')."/uploads/publisher/images/category/".$arrCat['image']);
135 135
         }
136 136
 
137 137
         if (!$publisher->getCategoryHandler()->insert($categoryObj)) {
138
-            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['name']) . "<br/>";
138
+            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['name'])."<br/>";
139 139
             continue;
140 140
         }
141 141
 
142 142
         $newCat['newid'] = $categoryObj->getVar('categoryid');
143 143
         ++$cnt_imported_cat;
144 144
 
145
-        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name')) . "<br\>";
145
+        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name'))."<br\>";
146 146
 
147
-        $sql = "SELECT * FROM " . $xoopsDB->prefix('smartsection_items') . " WHERE categoryid=" . $arrCat['categoryid'];
147
+        $sql = "SELECT * FROM ".$xoopsDB->prefix('smartsection_items')." WHERE categoryid=".$arrCat['categoryid'];
148 148
         $resultArticles = $xoopsDB->query($sql);
149 149
 
150 150
         while (false !== ($arrArticle = $xoopsDB->fetchArray($resultArticles))) {
@@ -172,31 +172,31 @@  discard block
 block discarded – undo
172 172
             */
173 173
 
174 174
             if (!$itemObj->store()) {
175
-                echo sprintf("  " . _AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title']) . "<br/>";
175
+                echo sprintf("  "._AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title'])."<br/>";
176 176
                 continue;
177 177
             } else {
178 178
 
179 179
                 // Linkes files
180
-                $sql = "SELECT * FROM " . $xoopsDB->prefix("smartsection_files") . " WHERE itemid=" . $arrArticle['itemid'];
180
+                $sql = "SELECT * FROM ".$xoopsDB->prefix("smartsection_files")." WHERE itemid=".$arrArticle['itemid'];
181 181
                 $resultFiles = $xoopsDB->query($sql);
182 182
                 $allowed_mimetypes = null;
183 183
                 while (false !== ($arrFile = $xoopsDB->fetchArray($resultFiles))) {
184
-                    $filename = \XoopsBaseConfig::get('root-path') . "/uploads/smartsection/" . $arrFile['filename'];
184
+                    $filename = \XoopsBaseConfig::get('root-path')."/uploads/smartsection/".$arrFile['filename'];
185 185
                     if (XoopsLoad::fileExists($filename)) {
186
-                        if (copy($filename, \XoopsBaseConfig::get('root-path') . "/uploads/publisher/" . $arrFile['filename'])) {
186
+                        if (copy($filename, \XoopsBaseConfig::get('root-path')."/uploads/publisher/".$arrFile['filename'])) {
187 187
                             $fileObj = $publisher->getFileHandler()->create();
188 188
                             $fileObj->setVars($arrFile);
189 189
                             $fileObj->setVar('fileid', 0);
190 190
 
191 191
                             if ($fileObj->store($allowed_mimetypes, true, false)) {
192
-                                echo "&nbsp;&nbsp;&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE_FILE, $arrFile['filename']) . "<br />";
192
+                                echo "&nbsp;&nbsp;&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE_FILE, $arrFile['filename'])."<br />";
193 193
                             }
194 194
                         }
195 195
                     }
196 196
                 }
197 197
 
198 198
                 $newArticleArray[$arrArticle['itemid']] = $itemObj->getVar('itemid');
199
-                echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title()) . "<br />";
199
+                echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title())."<br />";
200 200
                 ++$cnt_imported_articles;
201 201
             }
202 202
         }
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
     }
229 229
 
230 230
     // Looping through the comments to link them to the new articles and module
231
-    echo _AM_PUBLISHER_IMPORT_COMMENTS . "<br />";
231
+    echo _AM_PUBLISHER_IMPORT_COMMENTS."<br />";
232 232
 
233 233
     $publisher_module_id = $publisher->getModule()->mid();
234 234
 
@@ -241,17 +241,17 @@  discard block
 block discarded – undo
241 241
         $comment->setVar('com_modid', $publisher_module_id);
242 242
         $comment->setNew();
243 243
         if (!$comment_handler->insert($comment)) {
244
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title')) . "<br />";
244
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title'))."<br />";
245 245
         } else {
246
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title')) . "<br />";
246
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title'))."<br />";
247 247
         }
248 248
 
249 249
     }
250 250
 
251 251
     echo "<br/><br/>Done.<br/>";
252
-    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat) . "<br/>";
253
-    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles) . "<br/>";
254
-    echo "<br/><a href='" . PUBLISHER_URL . "/'>" . _AM_PUBLISHER_IMPORT_GOTOMODULE . "</a><br/>";
252
+    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat)."<br/>";
253
+    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles)."<br/>";
254
+    echo "<br/><a href='".PUBLISHER_URL."/'>"._AM_PUBLISHER_IMPORT_GOTOMODULE."</a><br/>";
255 255
 
256 256
     PublisherUtils::closeCollapsableBar('newsimportgo', 'newsimportgoicon');
257 257
     $xoops->footer();
Please login to merge, or discard this patch.
htdocs/modules/publisher/admin/import/wfsection.php 1 patch
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -20,10 +20,10 @@  discard block
 block discarded – undo
20 20
  * @version         $Id$
21 21
  */
22 22
 
23
-include_once dirname(__DIR__) . '/admin_header.php';
23
+include_once dirname(__DIR__).'/admin_header.php';
24 24
 $myts = \Xoops\Core\Text\Sanitizer::getInstance();
25 25
 
26
-$importFromModuleName = "WF-Section " . @$_POST['wfs_version'];
26
+$importFromModuleName = "WF-Section ".@$_POST['wfs_version'];
27 27
 
28 28
 $scriptname = "wfsection.php";
29 29
 
@@ -39,26 +39,26 @@  discard block
 block discarded – undo
39 39
     //publisher_adminMenu(-1, _AM_PUBLISHER_IMPORT);
40 40
     PublisherUtils::openCollapsableBar('wfsectionimport', 'wfsectionimporticon', sprintf(_AM_PUBLISHER_IMPORT_FROM, $importFromModuleName), _AM_PUBLISHER_IMPORT_INFO);
41 41
 
42
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix("wfs_category"));
42
+    $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix("wfs_category"));
43 43
     list ($totalCat) = $xoopsDB->fetchRow($result);
44 44
 
45 45
     if ($totalCat == 0) {
46
-        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . _AM_PUBLISHER_IMPORT_NOCATSELECTED . "</span>";
46
+        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">"._AM_PUBLISHER_IMPORT_NOCATSELECTED."</span>";
47 47
     } else {
48
-        include_once \XoopsBaseConfig::get('root-path') . "/class/xoopstree.php";
48
+        include_once \XoopsBaseConfig::get('root-path')."/class/xoopstree.php";
49 49
 
50
-        $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix("wfs_article"));
50
+        $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix("wfs_article"));
51 51
         list ($totalArticles) = $xoopsDB->fetchRow($result);
52 52
 
53 53
         if ($totalArticles == 0) {
54
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles) . "</span>";
54
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles)."</span>";
55 55
         } else {
56
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat) . "</span>";
56
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat)."</span>";
57 57
 
58
-            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL . "/import/$scriptname");
58
+            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL."/import/$scriptname");
59 59
 
60 60
             // Categories to be imported
61
-            $sql = "SELECT cat.id, cat.pid, cat.title, COUNT(art.articleid) FROM " . $xoopsDB->prefix("wfs_category") . " AS cat INNER JOIN " . $xoopsDB->prefix("wfs_article") . " AS art ON cat.id=art.categoryid GROUP BY art.categoryid";
61
+            $sql = "SELECT cat.id, cat.pid, cat.title, COUNT(art.articleid) FROM ".$xoopsDB->prefix("wfs_category")." AS cat INNER JOIN ".$xoopsDB->prefix("wfs_article")." AS art ON cat.id=art.categoryid GROUP BY art.categoryid";
62 62
             $result = $xoopsDB->query($sql);
63 63
             $cat_cbox_values = array();
64 64
             $cat_cbox_options = array();
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
         $orders = 'orders';
110 110
     }
111 111
     //$sql = "SELECT * FROM ".$xoopsDB->prefix("wfs_category")." ORDER by orders";
112
-    $sql = "SELECT * FROM " . $xoopsDB->prefix("wfs_category") . " ORDER by $orders";
112
+    $sql = "SELECT * FROM ".$xoopsDB->prefix("wfs_category")." ORDER by $orders";
113 113
     //end added to support 2.0.7
114 114
     $resultCat = $xoopsDB->query($sql);
115 115
 
@@ -134,13 +134,13 @@  discard block
 block discarded – undo
134 134
 
135 135
         // Category image
136 136
         if (($arrCat['imgurl'] !== 'blank.gif') && ($arrCat['imgurl'])) {
137
-            if (copy(XOOPS_ROOT_PATH . "/modules/wfsection/images/category/" . $arrCat['imgurl'], PUBLISHER_UPLOADS_PATH . "/images/category/" . $arrCat['imgurl'])) {
137
+            if (copy(XOOPS_ROOT_PATH."/modules/wfsection/images/category/".$arrCat['imgurl'], PUBLISHER_UPLOADS_PATH."/images/category/".$arrCat['imgurl'])) {
138 138
                 $categoryObj->setVar('image', $arrCat['imgurl']);
139 139
             }
140 140
         }
141 141
 
142 142
         if (!$categoryObj->store(false)) {
143
-            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['title']) . "<br/>";
143
+            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['title'])."<br/>";
144 144
             continue;
145 145
         }
146 146
 
@@ -151,9 +151,9 @@  discard block
 block discarded – undo
151 151
 
152 152
         ++$cnt_imported_cat;
153 153
 
154
-        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name')) . "<br\>";
154
+        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name'))."<br\>";
155 155
 
156
-        $sql = "SELECT * FROM " . $xoopsDB->prefix("wfs_article") . " WHERE categoryid=" . $arrCat['id'] . " ORDER BY weight";
156
+        $sql = "SELECT * FROM ".$xoopsDB->prefix("wfs_article")." WHERE categoryid=".$arrCat['id']." ORDER BY weight";
157 157
         $resultArticles = $xoopsDB->query($sql);
158 158
         while (false !== ($arrArticle = $xoopsDB->fetchArray($resultArticles))) {
159 159
             // insert article
@@ -175,28 +175,28 @@  discard block
 block discarded – undo
175 175
 
176 176
             // HTML Wrap
177 177
             if ($arrArticle['htmlpage']) {
178
-                $pagewrap_filename = \XoopsBaseConfig::get('root-path') . "/modules/wfsection/html/" . $arrArticle['htmlpage'];
178
+                $pagewrap_filename = \XoopsBaseConfig::get('root-path')."/modules/wfsection/html/".$arrArticle['htmlpage'];
179 179
                 if (XoopsLoad::fileExists($pagewrap_filename)) {
180
-                    if (copy($pagewrap_filename, PUBLISHER_UPLOADS_PATH . "/content/" . $arrArticle['htmlpage'])) {
181
-                        $itemObj->setVar('body', "[pagewrap=" . $arrArticle['htmlpage'] . "]");
182
-                        echo sprintf("&nbsp;&nbsp;&nbsp;&nbsp;" . _AM_PUBLISHER_IMPORT_ARTICLE_WRAP, $arrArticle['htmlpage']) . "<br/>";
180
+                    if (copy($pagewrap_filename, PUBLISHER_UPLOADS_PATH."/content/".$arrArticle['htmlpage'])) {
181
+                        $itemObj->setVar('body', "[pagewrap=".$arrArticle['htmlpage']."]");
182
+                        echo sprintf("&nbsp;&nbsp;&nbsp;&nbsp;"._AM_PUBLISHER_IMPORT_ARTICLE_WRAP, $arrArticle['htmlpage'])."<br/>";
183 183
                     }
184 184
                 }
185 185
             }
186 186
 
187 187
             if (!$itemObj->store()) {
188
-                echo sprintf("  " . _AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title']) . "<br/>";
188
+                echo sprintf("  "._AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title'])."<br/>";
189 189
                 continue;
190 190
             } else {
191 191
                 // Linkes files
192 192
 
193
-                $sql = "SELECT * FROM " . $xoopsDB->prefix("wfs_files") . " WHERE articleid=" . $arrArticle['articleid'];
193
+                $sql = "SELECT * FROM ".$xoopsDB->prefix("wfs_files")." WHERE articleid=".$arrArticle['articleid'];
194 194
                 $resultFiles = $xoopsDB->query($sql);
195 195
                 $allowed_mimetypes = '';
196 196
                 while (false !== ($arrFile = $xoopsDB->fetchArray($resultFiles))) {
197
-                    $filename = \XoopsBaseConfig::get('root-path') . "/modules/wfsection/cache/uploaded/" . $arrFile['filerealname'];
197
+                    $filename = \XoopsBaseConfig::get('root-path')."/modules/wfsection/cache/uploaded/".$arrFile['filerealname'];
198 198
                     if (XoopsLoad::fileExists($filename)) {
199
-                        if (copy($filename, PUBLISHER_UPLOADS_PATH . "/" . $arrFile['filerealname'])) {
199
+                        if (copy($filename, PUBLISHER_UPLOADS_PATH."/".$arrFile['filerealname'])) {
200 200
                             $fileObj = $publisher->getFileHandler()->create();
201 201
                             $fileObj->setVar('name', $arrFile['fileshowname']);
202 202
                             $fileObj->setVar('description', $arrFile['filedescript']);
@@ -209,14 +209,14 @@  discard block
 block discarded – undo
209 209
                             $fileObj->setVar('filename', $arrFile['filerealname']);
210 210
 
211 211
                             if ($fileObj->store($allowed_mimetypes, true, false)) {
212
-                                echo "&nbsp;&nbsp;&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE_FILE, $arrFile['filerealname']) . "<br />";
212
+                                echo "&nbsp;&nbsp;&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE_FILE, $arrFile['filerealname'])."<br />";
213 213
                             }
214 214
                         }
215 215
                     }
216 216
                 }
217 217
 
218 218
                 $newArticleArray[$arrArticle['articleid']] = $itemObj->getVar('itemid');
219
-                echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title()) . "<br />";
219
+                echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title())."<br />";
220 220
                 ++$cnt_imported_articles;
221 221
             }
222 222
         }
@@ -239,7 +239,7 @@  discard block
 block discarded – undo
239 239
     }
240 240
 
241 241
     // Looping through the comments to link them to the new articles and module
242
-    echo _AM_PUBLISHER_IMPORT_COMMENTS . "<br />";
242
+    echo _AM_PUBLISHER_IMPORT_COMMENTS."<br />";
243 243
 
244 244
     $module_handler = xoops_getHandler('module');
245 245
     $moduleObj = $module_handler->getByDirname('wfsection');
@@ -256,17 +256,17 @@  discard block
 block discarded – undo
256 256
         $comment->setVar('com_modid', $publisher_module_id);
257 257
         $comment->setNew();
258 258
         if (!$comment_handler->insert($comment)) {
259
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title')) . "<br />";
259
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title'))."<br />";
260 260
         } else {
261
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title')) . "<br />";
261
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title'))."<br />";
262 262
         }
263 263
 
264 264
     }
265 265
 
266 266
     echo "<br/><br/>Done.<br/>";
267
-    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat) . "<br/>";
268
-    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles) . "<br/>";
269
-    echo "<br/><a href='" . PUBLISHER_URL . "/'>" . _AM_PUBLISHER_IMPORT_GOTOMODULE . "</a><br/>";
267
+    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat)."<br/>";
268
+    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles)."<br/>";
269
+    echo "<br/><a href='".PUBLISHER_URL."/'>"._AM_PUBLISHER_IMPORT_GOTOMODULE."</a><br/>";
270 270
 
271 271
     PublisherUtils::closeCollapsableBar('wfsectionimportgo', 'wfsectionimportgoicon');
272 272
     $xoops->footer();
Please login to merge, or discard this patch.
htdocs/modules/publisher/admin/import/news.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -20,10 +20,10 @@  discard block
 block discarded – undo
20 20
  * @version         $Id$
21 21
  */
22 22
 
23
-include_once dirname(__DIR__) . '/admin_header.php';
23
+include_once dirname(__DIR__).'/admin_header.php';
24 24
 $myts = \Xoops\Core\Text\Sanitizer::getInstance();
25 25
 
26
-$importFromModuleName = "News " . @$_POST['news_version'];
26
+$importFromModuleName = "News ".@$_POST['news_version'];
27 27
 
28 28
 $scriptname = "news.php";
29 29
 
@@ -39,26 +39,26 @@  discard block
 block discarded – undo
39 39
     //publisher_adminMenu(-1, _AM_PUBLISHER_IMPORT);
40 40
     PublisherUtils::openCollapsableBar('newsimport', 'newsimporticon', sprintf(_AM_PUBLISHER_IMPORT_FROM, $importFromModuleName), _AM_PUBLISHER_IMPORT_INFO);
41 41
 
42
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix("topics"));
42
+    $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix("topics"));
43 43
     list ($totalCat) = $xoopsDB->fetchRow($result);
44 44
 
45 45
     if ($totalCat == 0) {
46
-        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . _AM_PUBLISHER_IMPORT_NO_CATEGORY . "</span>";
46
+        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">"._AM_PUBLISHER_IMPORT_NO_CATEGORY."</span>";
47 47
     } else {
48
-        include_once \XoopsBaseConfig::get('root-path') . '/class/xoopstree.php';
48
+        include_once \XoopsBaseConfig::get('root-path').'/class/xoopstree.php';
49 49
 
50
-        $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix("stories"));
50
+        $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix("stories"));
51 51
         list ($totalArticles) = $xoopsDB->fetchRow($result);
52 52
 
53 53
         if ($totalArticles == 0) {
54
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles) . "</span>";
54
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles)."</span>";
55 55
         } else {
56
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat) . "</span>";
56
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat)."</span>";
57 57
 
58
-            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL . "/import/$scriptname");
58
+            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL."/import/$scriptname");
59 59
 
60 60
             // Categories to be imported
61
-            $sql = "SELECT cat.topic_id, cat.topic_pid, cat.topic_title, COUNT(art.storyid) FROM " . $xoopsDB->prefix("topics") . " AS cat INNER JOIN " . $xoopsDB->prefix("stories") . " AS art ON cat.topic_id=art.topicid GROUP BY art.topicid";
61
+            $sql = "SELECT cat.topic_id, cat.topic_pid, cat.topic_title, COUNT(art.storyid) FROM ".$xoopsDB->prefix("topics")." AS cat INNER JOIN ".$xoopsDB->prefix("stories")." AS art ON cat.topic_id=art.topicid GROUP BY art.topicid";
62 62
 
63 63
             $result = $xoopsDB->query($sql);
64 64
             $cat_cbox_options = array();
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 
112 112
     $parentId = $_POST['parent_category'];
113 113
 
114
-    $sql = "SELECT * FROM " . $xoopsDB->prefix('topics');
114
+    $sql = "SELECT * FROM ".$xoopsDB->prefix('topics');
115 115
 
116 116
     $resultCat = $xoopsDB->query($sql);
117 117
 
@@ -133,22 +133,22 @@  discard block
 block discarded – undo
133 133
 
134 134
         // Category image
135 135
         if (($arrCat['topic_imgurl'] !== 'blank.gif') && ($arrCat['topic_imgurl'] != '')) {
136
-            if (copy(\XoopsBaseConfig::get('root-path') . "/modules/news/images/topics/" . $arrCat['topic_imgurl'], \XoopsBaseConfig::get('root-path') . "/uploads/publisher/images/category/" . $arrCat['topic_imgurl'])) {
136
+            if (copy(\XoopsBaseConfig::get('root-path')."/modules/news/images/topics/".$arrCat['topic_imgurl'], \XoopsBaseConfig::get('root-path')."/uploads/publisher/images/category/".$arrCat['topic_imgurl'])) {
137 137
                 $categoryObj->setVar('image', $arrCat['topic_imgurl']);
138 138
             }
139 139
         }
140 140
 
141 141
         if (!$publisher->getCategoryHandler()->insert($categoryObj)) {
142
-            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['topic_title']) . "<br/>";
142
+            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['topic_title'])."<br/>";
143 143
             continue;
144 144
         }
145 145
 
146 146
         $newCat['newid'] = $categoryObj->getVar('categoryid');
147 147
         ++$cnt_imported_cat;
148 148
 
149
-        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name')) . "<br\>";
149
+        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name'))."<br\>";
150 150
 
151
-        $sql = "SELECT * FROM " . $xoopsDB->prefix('stories') . " WHERE topicid=" . $arrCat['topic_id'];
151
+        $sql = "SELECT * FROM ".$xoopsDB->prefix('stories')." WHERE topicid=".$arrCat['topic_id'];
152 152
         $resultArticles = $xoopsDB->query($sql);
153 153
         while (false !== ($arrArticle = $xoopsDB->fetchArray($resultArticles))) {
154 154
             // insert article
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
              */
187 187
 
188 188
             if (!$itemObj->store()) {
189
-                echo sprintf("  " . _AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title']) . "<br/>";
189
+                echo sprintf("  "._AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title'])."<br/>";
190 190
                 continue;
191 191
             } else {
192 192
                 /*
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
                  }
219 219
                  */
220 220
                 $newArticleArray[$arrArticle['storyid']] = $itemObj->getVar('itemid');
221
-                echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title()) . "<br />";
221
+                echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title())."<br />";
222 222
                 ++$cnt_imported_articles;
223 223
             }
224 224
         }
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
     }
251 251
 
252 252
     // Looping through the comments to link them to the new articles and module
253
-    echo _AM_PUBLISHER_IMPORT_COMMENTS . "<br />";
253
+    echo _AM_PUBLISHER_IMPORT_COMMENTS."<br />";
254 254
 
255 255
     $publisher_module_id = $publisher->getModule()->mid();
256 256
 
@@ -263,17 +263,17 @@  discard block
 block discarded – undo
263 263
         $comment->setVar('com_modid', $publisher_module_id);
264 264
         $comment->setNew();
265 265
         if (!$comment_handler->insert($comment)) {
266
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title')) . "<br />";
266
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title'))."<br />";
267 267
         } else {
268
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title')) . "<br />";
268
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title'))."<br />";
269 269
         }
270 270
 
271 271
     }
272 272
 
273 273
     echo "<br/><br/>Done.<br/>";
274
-    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat) . "<br/>";
275
-    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles) . "<br/>";
276
-    echo "<br/><a href='" . PUBLISHER_URL . "/'>" . _AM_PUBLISHER_IMPORT_GOTOMODULE . "</a><br/>";
274
+    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat)."<br/>";
275
+    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles)."<br/>";
276
+    echo "<br/><a href='".PUBLISHER_URL."/'>"._AM_PUBLISHER_IMPORT_GOTOMODULE."</a><br/>";
277 277
 
278 278
     PublisherUtils::closeCollapsableBar('newsimportgo', 'newsimportgoicon');
279 279
     $xoops->footer();
Please login to merge, or discard this patch.
htdocs/modules/publisher/admin/import/xfsection.php 1 patch
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -20,10 +20,10 @@  discard block
 block discarded – undo
20 20
  * @version         $Id$
21 21
  */
22 22
 
23
-include_once dirname(__DIR__) . '/admin_header.php';
23
+include_once dirname(__DIR__).'/admin_header.php';
24 24
 $myts = \Xoops\Core\Text\Sanitizer::getInstance();
25 25
 
26
-$importFromModuleName = "XF-Section " . @$_POST['xfs_version'];
26
+$importFromModuleName = "XF-Section ".@$_POST['xfs_version'];
27 27
 
28 28
 $scriptname = "xfsection.php";
29 29
 
@@ -39,26 +39,26 @@  discard block
 block discarded – undo
39 39
     //publisher_adminMenu(-1, _AM_PUBLISHER_IMPORT);
40 40
     PublisherUtils::openCollapsableBar('xfsectionimport', 'xfsectionimporticon', sprintf(_AM_PUBLISHER_IMPORT_FROM, $importFromModuleName), _AM_PUBLISHER_IMPORT_INFO);
41 41
 
42
-    $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix("xfs_category"));
42
+    $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix("xfs_category"));
43 43
     list ($totalCat) = $xoopsDB->fetchRow($result);
44 44
 
45 45
     if ($totalCat == 0) {
46
-        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . _AM_PUBLISHER_IMPORT_NOCATSELECTED . "</span>";
46
+        echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">"._AM_PUBLISHER_IMPORT_NOCATSELECTED."</span>";
47 47
     } else {
48
-        include_once \XoopsBaseConfig::get('root-path') . '/class/xoopstree.php';
48
+        include_once \XoopsBaseConfig::get('root-path').'/class/xoopstree.php';
49 49
 
50
-        $result = $xoopsDB->query("SELECT COUNT(*) FROM " . $xoopsDB->prefix("xfs_article"));
50
+        $result = $xoopsDB->query("SELECT COUNT(*) FROM ".$xoopsDB->prefix("xfs_article"));
51 51
         list ($totalArticles) = $xoopsDB->fetchRow($result);
52 52
 
53 53
         if ($totalArticles == 0) {
54
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles) . "</span>";
54
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND_NO_ITEMS, $importFromModuleName, $totalArticles)."</span>";
55 55
         } else {
56
-            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">" . sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat) . "</span>";
56
+            echo "<span style=\"color: #567; margin: 3px 0 12px 0; font-size: small; display: block; \">".sprintf(_AM_PUBLISHER_IMPORT_MODULE_FOUND, $importFromModuleName, $totalArticles, $totalCat)."</span>";
57 57
 
58
-            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL . "/import/$scriptname");
58
+            $form = new Xoops\Form\ThemeForm(_AM_PUBLISHER_IMPORT_SETTINGS, 'import_form', PUBLISHER_ADMIN_URL."/import/$scriptname");
59 59
 
60 60
             // Categories to be imported
61
-            $sql = "SELECT cat.id, cat.pid, cat.title, COUNT(art.articleid) FROM " . $xoopsDB->prefix("xfs_category") . " AS cat INNER JOIN " . $xoopsDB->prefix("xfs_article") . " AS art ON cat.id=art.categoryid GROUP BY art.categoryid";
61
+            $sql = "SELECT cat.id, cat.pid, cat.title, COUNT(art.articleid) FROM ".$xoopsDB->prefix("xfs_category")." AS cat INNER JOIN ".$xoopsDB->prefix("xfs_article")." AS art ON cat.id=art.categoryid GROUP BY art.categoryid";
62 62
             $result = $xoopsDB->query($sql);
63 63
             $cat_cbox_values = array();
64 64
             $cat_cbox_options = array();
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
     $parentId = $_POST['parent_category'];
105 105
 
106 106
 
107
-    $sql = "SELECT * FROM " . $xoopsDB->prefix("xfs_category") . " ORDER by orders";
107
+    $sql = "SELECT * FROM ".$xoopsDB->prefix("xfs_category")." ORDER by orders";
108 108
 
109 109
     $resultCat = $xoopsDB->query($sql);
110 110
 
@@ -127,13 +127,13 @@  discard block
 block discarded – undo
127 127
 
128 128
         // Category image
129 129
         if (($arrCat['imgurl'] !== 'blank.gif') && ($arrCat['imgurl'])) {
130
-            if (copy(\XoopsBaseConfig::get('root-path') . "/modules/xfsection/images/category/" . $arrCat['imgurl'], PUBLISHER_UPLOADS_PATH . "/images/category/" . $arrCat['imgurl'])) {
130
+            if (copy(\XoopsBaseConfig::get('root-path')."/modules/xfsection/images/category/".$arrCat['imgurl'], PUBLISHER_UPLOADS_PATH."/images/category/".$arrCat['imgurl'])) {
131 131
                 $categoryObj->setVar('image', $arrCat['imgurl']);
132 132
             }
133 133
         }
134 134
 
135 135
         if (!$categoryObj->store(false)) {
136
-            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['title']) . "<br/>";
136
+            echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_ERROR, $arrCat['title'])."<br/>";
137 137
             continue;
138 138
         }
139 139
 
@@ -144,9 +144,9 @@  discard block
 block discarded – undo
144 144
 
145 145
         ++$cnt_imported_cat;
146 146
 
147
-        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name')) . "<br\>";
147
+        echo sprintf(_AM_PUBLISHER_IMPORT_CATEGORY_SUCCESS, $categoryObj->getVar('name'))."<br\>";
148 148
 
149
-        $sql = "SELECT * FROM " . $xoopsDB->prefix("xfs_article") . " WHERE categoryid=" . $arrCat['id'] . " ORDER BY weight";
149
+        $sql = "SELECT * FROM ".$xoopsDB->prefix("xfs_article")." WHERE categoryid=".$arrCat['id']." ORDER BY weight";
150 150
         $resultArticles = $xoopsDB->query($sql);
151 151
         while (false !== ($arrArticle = $xoopsDB->fetchArray($resultArticles))) {
152 152
             // insert article
@@ -174,28 +174,28 @@  discard block
 block discarded – undo
174 174
 
175 175
             // HTML Wrap
176 176
             if ($arrArticle['htmlpage']) {
177
-                $pagewrap_filename = \XoopsBaseConfig::get('root-path') . "/modules/xfsection/html/" . $arrArticle['htmlpage'];
177
+                $pagewrap_filename = \XoopsBaseConfig::get('root-path')."/modules/xfsection/html/".$arrArticle['htmlpage'];
178 178
                 if (XoopsLoad::fileExists($pagewrap_filename)) {
179
-                    if (copy($pagewrap_filename, PUBLISHER_UPLOADS_PATH . "/content/" . $arrArticle['htmlpage'])) {
180
-                        $itemObj->setVar('body', "[pagewrap=" . $arrArticle['htmlpage'] . "]");
181
-                        echo sprintf("&nbsp;&nbsp;&nbsp;&nbsp;" . _AM_PUBLISHER_IMPORT_ARTICLE_WRAP, $arrArticle['htmlpage']) . "<br/>";
179
+                    if (copy($pagewrap_filename, PUBLISHER_UPLOADS_PATH."/content/".$arrArticle['htmlpage'])) {
180
+                        $itemObj->setVar('body', "[pagewrap=".$arrArticle['htmlpage']."]");
181
+                        echo sprintf("&nbsp;&nbsp;&nbsp;&nbsp;"._AM_PUBLISHER_IMPORT_ARTICLE_WRAP, $arrArticle['htmlpage'])."<br/>";
182 182
                     }
183 183
                 }
184 184
             }
185 185
 
186 186
             if (!$itemObj->store()) {
187
-                echo sprintf("  " . _AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title']) . "<br/>";
187
+                echo sprintf("  "._AM_PUBLISHER_IMPORT_ARTICLE_ERROR, $arrArticle['title'])."<br/>";
188 188
                 continue;
189 189
             } else {
190 190
                 // Linkes files
191 191
 
192
-                $sql = "SELECT * FROM " . $xoopsDB->prefix("xfs_files") . " WHERE articleid=" . $arrArticle['articleid'];
192
+                $sql = "SELECT * FROM ".$xoopsDB->prefix("xfs_files")." WHERE articleid=".$arrArticle['articleid'];
193 193
                 $resultFiles = $xoopsDB->query($sql);
194 194
                 $allowed_mimetypes = '';
195 195
                 while (false !== ($arrFile = $xoopsDB->fetchArray($resultFiles))) {
196
-                    $filename = \XoopsBaseConfig::get('root-path') . "/modules/xfsection/cache/uploaded/" . $arrFile['filerealname'];
196
+                    $filename = \XoopsBaseConfig::get('root-path')."/modules/xfsection/cache/uploaded/".$arrFile['filerealname'];
197 197
                     if (XoopsLoad::fileExists($filename)) {
198
-                        if (copy($filename, PUBLISHER_UPLOADS_PATH . "/" . $arrFile['filerealname'])) {
198
+                        if (copy($filename, PUBLISHER_UPLOADS_PATH."/".$arrFile['filerealname'])) {
199 199
                             $fileObj = $publisher->getFileHandler()->create();
200 200
                             $fileObj->setVar('name', $arrFile['fileshowname']);
201 201
                             $fileObj->setVar('description', $arrFile['filedescript']);
@@ -208,14 +208,14 @@  discard block
 block discarded – undo
208 208
                             $fileObj->setVar('filename', $arrFile['filerealname']);
209 209
 
210 210
                             if ($fileObj->store($allowed_mimetypes, true, false)) {
211
-                                echo "&nbsp;&nbsp;&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE_FILE, $arrFile['filerealname']) . "<br />";
211
+                                echo "&nbsp;&nbsp;&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE_FILE, $arrFile['filerealname'])."<br />";
212 212
                             }
213 213
                         }
214 214
                     }
215 215
                 }
216 216
 
217 217
                 $newArticleArray[$arrArticle['articleid']] = $itemObj->getVar('itemid');
218
-                echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title()) . "<br />";
218
+                echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_ARTICLE, $itemObj->title())."<br />";
219 219
                 ++$cnt_imported_articles;
220 220
             }
221 221
         }
@@ -238,7 +238,7 @@  discard block
 block discarded – undo
238 238
     }
239 239
 
240 240
     // Looping through the comments to link them to the new articles and module
241
-    echo _AM_PUBLISHER_IMPORT_COMMENTS . "<br />";
241
+    echo _AM_PUBLISHER_IMPORT_COMMENTS."<br />";
242 242
 
243 243
     $module_handler = xoops_getHandler('module');
244 244
     $moduleObj = $module_handler->getByDirname('xfsection');
@@ -255,17 +255,17 @@  discard block
 block discarded – undo
255 255
         $comment->setVar('com_modid', $publisher_module_id);
256 256
         $comment->setNew();
257 257
         if (!$comment_handler->insert($comment)) {
258
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title')) . "<br />";
258
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT_ERROR, $comment->getVar('com_title'))."<br />";
259 259
         } else {
260
-            echo "&nbsp;&nbsp;" . sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title')) . "<br />";
260
+            echo "&nbsp;&nbsp;".sprintf(_AM_PUBLISHER_IMPORTED_COMMENT, $comment->getVar('com_title'))."<br />";
261 261
         }
262 262
 
263 263
     }
264 264
 
265 265
     echo "<br/><br/>Done.<br/>";
266
-    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat) . "<br/>";
267
-    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles) . "<br/>";
268
-    echo "<br/><a href='" . publisher_URL . "'>" . _AM_PUBLISHER_IMPORT_GOTOMODULE . "</a><br/>";
266
+    echo sprintf(_AM_PUBLISHER_IMPORTED_CATEGORIES, $cnt_imported_cat)."<br/>";
267
+    echo sprintf(_AM_PUBLISHER_IMPORTED_ARTICLES, $cnt_imported_articles)."<br/>";
268
+    echo "<br/><a href='".publisher_URL."'>"._AM_PUBLISHER_IMPORT_GOTOMODULE."</a><br/>";
269 269
 
270 270
     PublisherUtils::closeCollapsableBar('xfsectionimportgo', 'xfsectionimportgoicon');
271 271
     $xoops->footer();
Please login to merge, or discard this patch.