Completed
Push — master ( 46dc99...a217e2 )
by Michael
50:47 queued 35:27
created
htdocs/modules/system/admin/services/main.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -41,9 +41,9 @@  discard block
 block discarded – undo
41 41
     // we need to decide how to handle that. Right now, some js will redirect us from
42 42
     // provider sort to service select just before the token should expire.
43 43
     if (isset($_POST['token']) && $security->validateToken($_POST['token'], false)) {
44
-        if (isset($_POST['op']) && $_POST['op']==='order') {
44
+        if (isset($_POST['op']) && $_POST['op'] === 'order') {
45 45
             if (isset($_POST['service'])) {
46
-                $service=$_POST['service'];
46
+                $service = $_POST['service'];
47 47
                 if (isset($_POST[$service]) && is_array($_POST[$service])) {
48 48
                     $service_order = array_flip($_POST[$service]);
49 49
                     $sm = Manager::getInstance();
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 $xoops->header('admin:system/system_services.tpl');
67 67
 
68 68
 $admin_page = new \Xoops\Module\Admin();
69
-$admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
69
+$admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
70 70
 $admin_page->addBreadcrumbLink(
71 71
     SystemLocale::SERVICES_MANAGER,
72 72
     $system->adminVersion('services', 'adminpath')
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 $admin_page->renderBreadcrumb();
77 77
 $admin_page->renderTips();
78 78
 
79
-$selected_service='';
79
+$selected_service = '';
80 80
 if (isset($_GET['service'])) {
81 81
     $selected_service = strtolower(XoopsFilterInput::clean($_GET['service'], 'WORD'));
82 82
 }
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
     $service_list[] = array(
101 101
         'name' => $v,
102 102
         'display' => ucfirst($v),
103
-        'active' => ($v==$selected_service),
103
+        'active' => ($v == $selected_service),
104 104
         );
105 105
 }
106 106
 $xoops->tpl()->assign('service_list', $service_list);
Please login to merge, or discard this patch.
htdocs/modules/system/admin/modulesadmin/main.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
         $xoops->theme()->addScript('modules/system/js/module.js');
59 59
         // Define Breadcrumb and tips
60 60
         $admin_page = new \Xoops\Module\Admin();
61
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
61
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
62 62
         $admin_page->addBreadcrumbLink(
63 63
             SystemLocale::MODULES_ADMINISTRATION,
64 64
             $system->adminVersion('modulesadmin', 'adminpath')
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
         $xoops->theme()->addStylesheet('modules/system/css/admin.css');
183 183
         // Define Breadcrumb and tips
184 184
         $admin_page = new \Xoops\Module\Admin();
185
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
185
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
186 186
         $admin_page->addBreadcrumbLink(
187 187
             SystemLocale::MODULES_ADMINISTRATION,
188 188
             $system->adminVersion('modulesadmin', 'adminpath')
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
         $xoops->theme()->addStylesheet('modules/system/css/admin.css');
223 223
         // Define Breadcrumb and tips
224 224
         $admin_page = new \Xoops\Module\Admin();
225
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
225
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
226 226
         $admin_page->addBreadcrumbLink(
227 227
             SystemLocale::MODULES_ADMINISTRATION,
228 228
             $system->adminVersion('modulesadmin', 'adminpath')
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
         $xoops->theme()->addStylesheet('modules/system/css/admin.css');
260 260
         // Define Breadcrumb and tips
261 261
         $admin_page = new \Xoops\Module\Admin();
262
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
262
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
263 263
         $admin_page->addBreadcrumbLink(
264 264
             SystemLocale::MODULES_ADMINISTRATION,
265 265
             $system->adminVersion('modulesadmin', 'adminpath')
Please login to merge, or discard this patch.
htdocs/modules/system/admin/extensions/main.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -146,7 +146,7 @@
 block discarded – undo
146 146
         }
147 147
         $folder = array(1, 2, 3);
148 148
         $system->cleanCache($folder);
149
-         //Set active modules in cache folder
149
+            //Set active modules in cache folder
150 150
         $xoops->setActiveModules();
151 151
         // Call Footer
152 152
         $xoops->footer();
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
         $xoops->theme()->addScript('modules/system/js/module.js');
59 59
         // Define Breadcrumb and tips
60 60
         $admin_page = new \Xoops\Module\Admin();
61
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
61
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
62 62
         $admin_page->addBreadcrumbLink(
63 63
             SystemLocale::EXTENSIONS_ADMINISTRATION,
64 64
             $system->adminVersion('extensions', 'adminpath')
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
         $xoops->theme()->addStylesheet('modules/system/css/admin.css');
87 87
         // Define Breadcrumb and tips
88 88
         $admin_page = new \Xoops\Module\Admin();
89
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
89
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
90 90
         $admin_page->addBreadcrumbLink(
91 91
             SystemLocale::EXTENSIONS_ADMINISTRATION,
92 92
             $system->adminVersion('extensions', 'adminpath')
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
         $xoops->theme()->addStylesheet('modules/system/css/admin.css');
127 127
         // Define Breadcrumb and tips
128 128
         $admin_page = new \Xoops\Module\Admin();
129
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
129
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
130 130
         $admin_page->addBreadcrumbLink(
131 131
             SystemLocale::EXTENSIONS_ADMINISTRATION,
132 132
             $system->adminVersion('extensions', 'adminpath')
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
         $xoops->theme()->addStylesheet('modules/system/css/admin.css');
164 164
         // Define Breadcrumb and tips
165 165
         $admin_page = new \Xoops\Module\Admin();
166
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
166
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
167 167
         $admin_page->addBreadcrumbLink(
168 168
             SystemLocale::EXTENSIONS_ADMINISTRATION,
169 169
             $system->adminVersion('extensions', 'adminpath')
Please login to merge, or discard this patch.
htdocs/modules/system/admin/users/main.php 1 patch
Spacing   +81 added lines, -81 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
     case 'users_edit':
65 65
         // Assign Breadcrumb menu
66 66
         $admin_page = new \Xoops\Module\Admin();
67
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
67
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
68 68
         $admin_page->addBreadcrumbLink(SystemLocale::USERS_MANAGEMENT, $system->adminVersion('users', 'adminpath'));
69 69
         $admin_page->addBreadcrumbLink(SystemLocale::EDIT_USER);
70 70
         $admin_page->renderBreadcrumb();
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
     case 'users_add':
80 80
         // Assign Breadcrumb menu
81 81
         $admin_page = new \Xoops\Module\Admin();
82
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
82
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
83 83
         $admin_page->addBreadcrumbLink(SystemLocale::USERS_MANAGEMENT, $system->adminVersion('users', 'adminpath'));
84 84
         $admin_page->addBreadcrumbLink(SystemLocale::ADD_USER);
85 85
         $admin_page->renderBreadcrumb();
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
     case 'users_delete':
94 94
         // Assign Breadcrumb menu
95 95
         $admin_page = new \Xoops\Module\Admin();
96
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
96
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
97 97
         $admin_page->addBreadcrumbLink(SystemLocale::USERS_MANAGEMENT, $system->adminVersion('users', 'adminpath'));
98 98
         $admin_page->addBreadcrumbLink(SystemLocale::DELETE_USER);
99 99
         $admin_page->renderBreadcrumb();
@@ -118,12 +118,12 @@  discard block
 block discarded – undo
118 118
             }
119 119
         } else {
120 120
             //Assign Breadcrumb menu
121
-            $system_breadcrumb->addHelp(system_adminVersion('users', 'help') . '#delete');
121
+            $system_breadcrumb->addHelp(system_adminVersion('users', 'help').'#delete');
122 122
             $system_breadcrumb->addLink(SystemLocale::DELETE_USER);
123 123
             $system_breadcrumb->render();
124 124
             echo $xoops->confirm(array(
125 125
                 'ok' => 1, 'uid' => $uid, 'op' => 'users_delete'
126
-            ), "admin.php?fct=users", sprintf(SystemLocale::F_DELETE_USER, $user->getVar('uname')) . '<br />');
126
+            ), "admin.php?fct=users", sprintf(SystemLocale::F_DELETE_USER, $user->getVar('uname')).'<br />');
127 127
         }
128 128
         break;
129 129
 
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
                     if ($_REQUEST['password'] != $_REQUEST['pass2']) {
215 215
                         $xoops->header();
216 216
                         echo "
217
-                        <strong>" . SystemLocale::E_NEW_PASSWORDS_NOT_MATCH_TRY_AGAIN . "</strong>";
217
+                        <strong>" . SystemLocale::E_NEW_PASSWORDS_NOT_MATCH_TRY_AGAIN."</strong>";
218 218
                         $xoops->footer();
219 219
                         exit();
220 220
                     }
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
                 $member_handler = $xoops->getHandlerMember();
256 256
                 // make sure the username doesnt exist yet
257 257
                 if ($member_handler->getUserCount(new Criteria('uname', $_REQUEST['username'])) > 0) {
258
-                    $adduser_errormsg = 'User name ' . $myts->htmlSpecialChars($_REQUEST['username']) . ' already exists';
258
+                    $adduser_errormsg = 'User name '.$myts->htmlSpecialChars($_REQUEST['username']).' already exists';
259 259
                 } else {
260 260
                     $newuser = $member_handler->createUser();
261 261
                     if (isset($user_viewemail)) {
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
279 279
                     if ($_REQUEST['pass2'] != "") {
280 280
                         if ($_REQUEST['password'] != $_REQUEST['pass2']) {
281 281
                             $xoops->header();
282
-                            echo "<strong>" . SystemLocale::E_NEW_PASSWORDS_NOT_MATCH_TRY_AGAIN . "</strong>";
282
+                            echo "<strong>".SystemLocale::E_NEW_PASSWORDS_NOT_MATCH_TRY_AGAIN."</strong>";
283 283
                             $xoops->footer();
284 284
                             exit();
285 285
                         }
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
                             }
309 309
                         }
310 310
                         if (!empty($groups_failed)) {
311
-                            $group_names = $member_handler->getGroupList(new Criteria('groupid', "(" . implode(", ", $groups_failed) . ")", 'IN'));
311
+                            $group_names = $member_handler->getGroupList(new Criteria('groupid', "(".implode(", ", $groups_failed).")", 'IN'));
312 312
                             $adduser_errormsg = sprintf(SystemLocale::EF_COULD_NOT_ADD_USER_TO_GROUPS, implode(", ", $group_names));
313 313
                         } else {
314 314
                             XoopsUserUtility::sendWelcome($newuser);
@@ -356,7 +356,7 @@  discard block
 block discarded – undo
356 356
         if (isset($_REQUEST['complet_search'])) {
357 357
             // Assign Breadcrumb menu
358 358
             $admin_page = new \Xoops\Module\Admin();
359
-            $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
359
+            $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
360 360
             $admin_page->addBreadcrumbLink(SystemLocale::USERS_MANAGEMENT, $system->adminVersion('users', 'adminpath'));
361 361
             $admin_page->addBreadcrumbLink(XoopsLocale::ADVANCED_SEARCH);
362 362
             $admin_page->renderBreadcrumb();
@@ -370,7 +370,7 @@  discard block
 block discarded – undo
370 370
             /* @var $group XoopsGroup */
371 371
             foreach ($group_arr as $group) {
372 372
                 if ($group->getVar("groupid") != FixedGroups::ANONYMOUS) {
373
-                    $group_select->addOption("" . $group->getVar("groupid") . "", "" . $group->getVar("name") . "");
373
+                    $group_select->addOption("".$group->getVar("groupid")."", "".$group->getVar("name")."");
374 374
                 }
375 375
             }
376 376
             unset($group);
@@ -477,7 +477,7 @@  discard block
 block discarded – undo
477 477
             //Display data
478 478
             // Assign Breadcrumb menu
479 479
             $admin_page = new \Xoops\Module\Admin();
480
-            $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
480
+            $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
481 481
             $admin_page->addBreadcrumbLink(SystemLocale::USERS_MANAGEMENT, $system->adminVersion('users', 'adminpath'));
482 482
             $admin_page->addBreadcrumbLink(XoopsLocale::LIST_);
483 483
             $admin_page->renderBreadcrumb();
@@ -487,7 +487,7 @@  discard block
 block discarded – undo
487 487
             $admin_page->addItemButton(SystemLocale::ADD_USER, 'admin.php?fct=users&amp;op=users_add', 'add');
488 488
             $admin_page->renderButton();
489 489
 
490
-            $requete_search = '<br /><br /><strong>' . SystemLocale::C_SEE_SEARCH_REQUEST . '</strong><br /><br />';
490
+            $requete_search = '<br /><br /><strong>'.SystemLocale::C_SEE_SEARCH_REQUEST.'</strong><br /><br />';
491 491
             $requete_pagenav = '';
492 492
 
493 493
             $criteria = new CriteriaCompo();
@@ -496,84 +496,84 @@  discard block
 block discarded – undo
496 496
             if (!empty($value)) {
497 497
                 $match = Request::getInt('user_uname_match', XOOPS_MATCH_START);
498 498
                 addCriteria($criteria, 'uname', $value, $match);
499
-                $requete_pagenav .= '&amp;user_uname=' . $myts->htmlSpecialChars($value) . '&amp;user_uname_match=' . $match;
500
-                $requete_search .= 'uname : ' . $value . ' and user_uname_match=' . $match . '<br />';
499
+                $requete_pagenav .= '&amp;user_uname='.$myts->htmlSpecialChars($value).'&amp;user_uname_match='.$match;
500
+                $requete_search .= 'uname : '.$value.' and user_uname_match='.$match.'<br />';
501 501
             }
502 502
 
503 503
             $value = Request::getString('user_name', '');
504 504
             if (!empty($value)) {
505 505
                 $match = Request::getInt('user_name_match', XOOPS_MATCH_START);
506 506
                 addCriteria($criteria, 'name', $value, $match);
507
-                $requete_pagenav .= '&amp;user_name=' . $myts->htmlSpecialChars($value) . '&amp;user_name_match=' . $match;
508
-                $requete_search .= 'name : ' . $value . ' and user_name_match=' . $match . '<br />';
507
+                $requete_pagenav .= '&amp;user_name='.$myts->htmlSpecialChars($value).'&amp;user_name_match='.$match;
508
+                $requete_search .= 'name : '.$value.' and user_name_match='.$match.'<br />';
509 509
             }
510 510
 
511 511
             $value = Request::getString('user_email', '');
512 512
             if (!empty($value)) {
513 513
                 $match = Request::getInt('user_email_match', XOOPS_MATCH_START);
514 514
                 addCriteria($criteria, 'email', $value, $match);
515
-                $requete_pagenav .= '&amp;user_email=' . $myts->htmlSpecialChars($value) . '&amp;user_email_match=' . $match;
516
-                $requete_search .= 'email : ' . $value . ' and user_email_match=' . $match . '<br />';
515
+                $requete_pagenav .= '&amp;user_email='.$myts->htmlSpecialChars($value).'&amp;user_email_match='.$match;
516
+                $requete_search .= 'email : '.$value.' and user_email_match='.$match.'<br />';
517 517
             }
518 518
 
519 519
             $value = Request::getString('user_url', '');
520 520
             if (!empty($value)) {
521 521
                 //$url = $xoops->formatURL(trim($_REQUEST['user_url']));
522
-                $criteria->add(new Criteria('url', '%' . $value . '%', 'LIKE'));
523
-                $requete_search .= 'url : ' . $value . '<br />';
522
+                $criteria->add(new Criteria('url', '%'.$value.'%', 'LIKE'));
523
+                $requete_search .= 'url : '.$value.'<br />';
524 524
             }
525 525
 
526 526
             $value = Request::getInt('user_icq', 0);
527 527
             if (!empty($value)) {
528 528
                 $match = Request::getInt('user_icq_match', XOOPS_MATCH_START);
529
-                addCriteria($criteria, 'user_icq', (string) $value, $match);
530
-                $requete_pagenav .= '&amp;user_icq=' . $value . '&amp;user_icq_match=' . $match;
531
-                $requete_search .= 'icq : ' . $value . ' and user_icq_match=' . $match . '<br />';
529
+                addCriteria($criteria, 'user_icq', (string)$value, $match);
530
+                $requete_pagenav .= '&amp;user_icq='.$value.'&amp;user_icq_match='.$match;
531
+                $requete_search .= 'icq : '.$value.' and user_icq_match='.$match.'<br />';
532 532
             }
533 533
 
534 534
             $value = Request::getString('user_aim', '');
535 535
             if (!empty($value)) {
536 536
                 $match = Request::getInt('user_aim_match', XOOPS_MATCH_START);
537 537
                 addCriteria($criteria, 'user_aim', $value, $match);
538
-                $requete_pagenav .= '&amp;user_aim=' . $myts->htmlSpecialChars($value) . '&amp;user_aim_match=' . $match;
539
-                $requete_search .= 'aim : ' . $value . ' and user_aim_match=' . $match . '<br />';
538
+                $requete_pagenav .= '&amp;user_aim='.$myts->htmlSpecialChars($value).'&amp;user_aim_match='.$match;
539
+                $requete_search .= 'aim : '.$value.' and user_aim_match='.$match.'<br />';
540 540
             }
541 541
 
542 542
             $value = Request::getString('user_yim', '');
543 543
             if (!empty($value)) {
544 544
                 $match = Request::getInt('user_yim_match', XOOPS_MATCH_START);
545 545
                 addCriteria($criteria, 'user_yim', $value, $match);
546
-                $requete_pagenav .= '&amp;user_yim=' . $myts->htmlSpecialChars($value) . '&amp;user_yim_match=' . $match;
547
-                $requete_search .= 'yim : ' . $value . ' and user_yim_match=' . $match . '<br />';
546
+                $requete_pagenav .= '&amp;user_yim='.$myts->htmlSpecialChars($value).'&amp;user_yim_match='.$match;
547
+                $requete_search .= 'yim : '.$value.' and user_yim_match='.$match.'<br />';
548 548
             }
549 549
 
550 550
             $value = Request::getString('user_msnm', '');
551 551
             if (!empty($value)) {
552 552
                 $match = Request::getInt('user_msnm_match', XOOPS_MATCH_START);
553 553
                 addCriteria($criteria, 'user_msnm', $value, $match);
554
-                $requete_pagenav .= '&amp;user_msnm=' . $myts->htmlSpecialChars($value) . '&amp;user_msnm_match=' . $match;
555
-                $requete_search .= 'msnm : ' . $value . ' and user_msnm_match=' . $match . '<br />';
554
+                $requete_pagenav .= '&amp;user_msnm='.$myts->htmlSpecialChars($value).'&amp;user_msnm_match='.$match;
555
+                $requete_search .= 'msnm : '.$value.' and user_msnm_match='.$match.'<br />';
556 556
             }
557 557
 
558 558
             $value = Request::getString('user_from', '');
559 559
             if (!empty($value)) {
560
-                $criteria->add(new Criteria('user_from', '%' . $value . '%', 'LIKE'));
561
-                $requete_pagenav .= '&amp;user_from=' . $myts->htmlSpecialChars($value);
562
-                $requete_search .= 'from : ' . $value . '<br />';
560
+                $criteria->add(new Criteria('user_from', '%'.$value.'%', 'LIKE'));
561
+                $requete_pagenav .= '&amp;user_from='.$myts->htmlSpecialChars($value);
562
+                $requete_search .= 'from : '.$value.'<br />';
563 563
             }
564 564
 
565 565
             $value = Request::getString('user_intrest', '');
566 566
             if (!empty($value)) {
567
-                $criteria->add(new Criteria('user_intrest', '%' . $value . '%', 'LIKE'));
568
-                $requete_pagenav .= '&amp;user_intrest=' . $myts->htmlSpecialChars($value);
569
-                $requete_search .= 'interet : ' . $value . '<br />';
567
+                $criteria->add(new Criteria('user_intrest', '%'.$value.'%', 'LIKE'));
568
+                $requete_pagenav .= '&amp;user_intrest='.$myts->htmlSpecialChars($value);
569
+                $requete_search .= 'interet : '.$value.'<br />';
570 570
             }
571 571
 
572 572
             $value = Request::getString('user_occ', '');
573 573
             if (!empty($value)) {
574
-                $criteria->add(new Criteria('user_occ', '%' . $value . '%', 'LIKE'));
575
-                $requete_pagenav .= '&amp;user_occ=' . $myts->htmlSpecialChars($value);
576
-                $requete_search .= 'location : ' . $value . '<br />';
574
+                $criteria->add(new Criteria('user_occ', '%'.$value.'%', 'LIKE'));
575
+                $requete_pagenav .= '&amp;user_occ='.$myts->htmlSpecialChars($value);
576
+                $requete_search .= 'location : '.$value.'<br />';
577 577
             }
578 578
 
579 579
             $value = Request::getInt('user_lastlog_more', 0);
@@ -581,8 +581,8 @@  discard block
 block discarded – undo
581 581
                 $time = time() - (60 * 60 * 24 * $value);
582 582
                 if ($time > 0) {
583 583
                     $criteria->add(new Criteria('last_login', $time, '<'));
584
-                    $requete_pagenav .= '&amp;user_lastlog_more=' . $value;
585
-                    $requete_search .= 'derniere connexion apres : ' . $value . '<br />';
584
+                    $requete_pagenav .= '&amp;user_lastlog_more='.$value;
585
+                    $requete_search .= 'derniere connexion apres : '.$value.'<br />';
586 586
                 }
587 587
             }
588 588
 
@@ -591,8 +591,8 @@  discard block
 block discarded – undo
591 591
                 $time = time() - (60 * 60 * 24 * $value);
592 592
                 if ($time > 0) {
593 593
                     $criteria->add(new Criteria('last_login', $time, '>'));
594
-                    $requete_pagenav .= '&amp;user_lastlog_less=' . $value;
595
-                    $requete_search .= 'derniere connexion avant : ' . $value . '<br />';
594
+                    $requete_pagenav .= '&amp;user_lastlog_less='.$value;
595
+                    $requete_search .= 'derniere connexion avant : '.$value.'<br />';
596 596
                 }
597 597
             }
598 598
 
@@ -601,8 +601,8 @@  discard block
 block discarded – undo
601 601
                 $time = time() - (60 * 60 * 24 * $value);
602 602
                 if ($time > 0) {
603 603
                     $criteria->add(new Criteria('user_regdate', $time, '<'));
604
-                    $requete_pagenav .= '&amp;user_reg_more=' . $value;
605
-                    $requete_search .= 'enregistre apres : ' . $value . '<br />';
604
+                    $requete_pagenav .= '&amp;user_reg_more='.$value;
605
+                    $requete_search .= 'enregistre apres : '.$value.'<br />';
606 606
                 }
607 607
             }
608 608
 
@@ -611,31 +611,31 @@  discard block
 block discarded – undo
611 611
                 $time = time() - (60 * 60 * 24 * $value);
612 612
                 if ($time > 0) {
613 613
                     $criteria->add(new Criteria('user_regdate', $time, '>'));
614
-                    $requete_pagenav .= '&amp;user_reg_less=' . $value;
615
-                    $requete_search .= 'enregistre avant : ' . $value . '<br />';
614
+                    $requete_pagenav .= '&amp;user_reg_less='.$value;
615
+                    $requete_search .= 'enregistre avant : '.$value.'<br />';
616 616
                 }
617 617
             }
618 618
 
619 619
             $value = Request::getInt('user_posts_more', 0);
620 620
             if (!empty($value)) {
621 621
                 $criteria->add(new Criteria('posts', $value, '>'));
622
-                $requete_pagenav .= '&amp;user_posts_more=' . $value;
623
-                $requete_search .= 'posts plus de : ' . $value . '<br />';
622
+                $requete_pagenav .= '&amp;user_posts_more='.$value;
623
+                $requete_search .= 'posts plus de : '.$value.'<br />';
624 624
             }
625 625
 
626 626
             $value = Request::getInt('user_posts_less', 0);
627 627
             if (!empty($value)) {
628 628
                 $criteria->add(new Criteria('posts', $value, '<'));
629
-                $requete_pagenav .= '&amp;user_posts_less=' . $value;
630
-                $requete_search .= 'post moins de : ' . $value . '<br />';
629
+                $requete_pagenav .= '&amp;user_posts_less='.$value;
630
+                $requete_search .= 'post moins de : '.$value.'<br />';
631 631
             }
632 632
 
633 633
             $value = Request::getWord('user_mailok', '');
634 634
             if (!empty($value) && ($value !== 'both')) {
635 635
                 $ok = ($value === 'mailok') ? 1 : 0;
636 636
                 $criteria->add(new Criteria('user_mailok', $ok));
637
-                $requete_pagenav .= '&amp;user_mailok=' . $value;
638
-                $requete_search .= 'accept email : ' . $value . '<br />';
637
+                $requete_pagenav .= '&amp;user_mailok='.$value;
638
+                $requete_search .= 'accept email : '.$value.'<br />';
639 639
             }
640 640
 
641 641
             $user_type = Request::getWord('user_type', '');
@@ -645,31 +645,31 @@  discard block
 block discarded – undo
645 645
                 } elseif ($user_type === "actv") {
646 646
                     $criteria->add(new Criteria('level', 0, '>'));
647 647
                 }
648
-                $requete_search .= 'actif ou inactif : ' . $user_type . '<br />';
649
-                $requete_pagenav .= '&amp;user_type=' . $user_type;
648
+                $requete_search .= 'actif ou inactif : '.$user_type.'<br />';
649
+                $requete_pagenav .= '&amp;user_type='.$user_type;
650 650
             }
651 651
 
652 652
             //$groups = empty($_REQUEST['selgroups']) ? array() : array_map("intval", $_REQUEST['selgroups']);
653 653
             $validsort = array("uname", "email", "last_login", "user_regdate", "posts");
654 654
             $sort = Request::getWord('user_sort', 'user_regdate');
655 655
             $sort = (!in_array($sort, $validsort)) ? "user_regdate" : $sort;
656
-            $requete_pagenav .= '&amp;user_sort=' . $sort;
657
-            $requete_search .= 'order by : ' . $sort . '<br />';
656
+            $requete_pagenav .= '&amp;user_sort='.$sort;
657
+            $requete_search .= 'order by : '.$sort.'<br />';
658 658
             $criteria->setSort($sort);
659 659
 
660 660
             $order = Request::getWord('user_order', 'DESC');
661
-            $requete_pagenav .= '&amp;user_order=' . $order;
662
-            $requete_search .= 'tris : ' . $order . '<br />';
661
+            $requete_pagenav .= '&amp;user_order='.$order;
662
+            $requete_search .= 'tris : '.$order.'<br />';
663 663
             $criteria->setOrder($order);
664 664
 
665 665
             $user_limit = $xoops->getModuleConfig('users_pager', 'system');
666 666
             if (isset($_REQUEST['user_limit'])) {
667 667
                 $user_limit = $_REQUEST['user_limit'];
668
-                $requete_pagenav .= '&amp;user_limit=' . $myts->htmlSpecialChars($_REQUEST['user_limit']);
669
-                $requete_search .= 'limit : ' . $user_limit . '<br />';
668
+                $requete_pagenav .= '&amp;user_limit='.$myts->htmlSpecialChars($_REQUEST['user_limit']);
669
+                $requete_search .= 'limit : '.$user_limit.'<br />';
670 670
             } else {
671
-                $requete_pagenav .= '&amp;user_limit=' . $xoops->getModuleConfig('users_pager', 'system');
672
-                $requete_search .= 'limit : ' . $user_limit . '<br />';
671
+                $requete_pagenav .= '&amp;user_limit='.$xoops->getModuleConfig('users_pager', 'system');
672
+                $requete_search .= 'limit : '.$user_limit.'<br />';
673 673
             }
674 674
 
675 675
             $start = (!empty($_REQUEST['start'])) ? (int)($_REQUEST['start']) : 0;
@@ -684,7 +684,7 @@  discard block
 block discarded – undo
684 684
                 } else {
685 685
                     $groups = array();
686 686
                 }
687
-                $requete_pagenav .= '&amp;selgroups=' . $myts->htmlSpecialChars($_REQUEST['selgroups']);
687
+                $requete_pagenav .= '&amp;selgroups='.$myts->htmlSpecialChars($_REQUEST['selgroups']);
688 688
             } else {
689 689
                 $groups = array();
690 690
             }
@@ -693,7 +693,7 @@  discard block
 block discarded – undo
693 693
             $users_count = $member_handler->getUserCountByGroupLink($groups, $criteria);
694 694
             $users_arr = array();
695 695
             if ($start < $users_count) {
696
-                echo sprintf(XoopsLocale::F_USERS_FOUND, $users_count) . "<br />";
696
+                echo sprintf(XoopsLocale::F_USERS_FOUND, $users_count)."<br />";
697 697
                 $criteria->setSort($sort);
698 698
                 $criteria->setOrder($order);
699 699
                 $criteria->setLimit($user_limit);
@@ -716,32 +716,32 @@  discard block
 block discarded – undo
716 716
             $user_uname = (!isset($_REQUEST['user_uname'])) ? '' : $_REQUEST['user_uname'];
717 717
             //Form tris
718 718
             $form = '<form action="admin.php?fct=users" method="post">
719
-                    ' . SystemLocale::C_SEARCH_USER . '<input type="text" name="user_uname" value="' . $myts->htmlSpecialChars($user_uname) . '" size="15">
719
+                    ' . SystemLocale::C_SEARCH_USER.'<input type="text" name="user_uname" value="'.$myts->htmlSpecialChars($user_uname).'" size="15">
720 720
                     <select name="selgroups">
721
-                        <option value="" selected="selected">' . XoopsLocale::ALL_GROUPS . '</option>';
721
+                        <option value="" selected="selected">' . XoopsLocale::ALL_GROUPS.'</option>';
722 722
             $group_handler = $xoops->getHandlerGroup();
723 723
             $group_arr = $group_handler->getObjects();
724 724
             /* @var $group XoopsGroup */
725 725
             foreach ($group_arr as $group) {
726 726
                 if ($group->getVar("groupid") != FixedGroups::ANONYMOUS) {
727
-                    $form .= '<option value="' . $group->getVar("groupid") . '"  ' . ($selgroups == $group->getVar("groupid") ? ' selected="selected"' : '') . '>' . $group->getVar("name") . '</option>';
727
+                    $form .= '<option value="'.$group->getVar("groupid").'"  '.($selgroups == $group->getVar("groupid") ? ' selected="selected"' : '').'>'.$group->getVar("name").'</option>';
728 728
                 }
729 729
             }
730 730
             unset($group);
731 731
             $form .= '</select>&nbsp;
732 732
                 <select name="user_type">
733
-                    <option value="" ' . ($user_type == '' ? ' selected="selected"' : '') . '>' . XoopsLocale::ALL_USERS . '</option>
734
-                    <option value="actv" ' . ($user_type === 'actv' ? ' selected="selected"' : '') . '>' . SystemLocale::ONLY_ACTIVE_USERS . '</option>
735
-                    <option value="inactv" ' . ($user_type === 'inactv' ? ' selected="selected"' : '') . '>' . SystemLocale::ONLY_INACTIVE_USERS . '</option>
733
+                    <option value="" ' . ($user_type == '' ? ' selected="selected"' : '').'>'.XoopsLocale::ALL_USERS.'</option>
734
+                    <option value="actv" ' . ($user_type === 'actv' ? ' selected="selected"' : '').'>'.SystemLocale::ONLY_ACTIVE_USERS.'</option>
735
+                    <option value="inactv" ' . ($user_type === 'inactv' ? ' selected="selected"' : '').'>'.SystemLocale::ONLY_INACTIVE_USERS.'</option>
736 736
                 </select>&nbsp;
737 737
                 <select name="user_limit">
738
-                    <option value="20" ' . ($user_limit == 20 ? ' selected="selected"' : '') . '>20</option>
739
-                    <option value="50" ' . ($user_limit == 50 ? ' selected="selected"' : '') . '>50</option>
740
-                    <option value="100" ' . ($user_limit == 100 ? ' selected="selected"' : '') . '>100</option>
738
+                    <option value="20" ' . ($user_limit == 20 ? ' selected="selected"' : '').'>20</option>
739
+                    <option value="50" ' . ($user_limit == 50 ? ' selected="selected"' : '').'>50</option>
740
+                    <option value="100" ' . ($user_limit == 100 ? ' selected="selected"' : '').'>100</option>
741 741
                 </select>&nbsp;
742 742
                 <input type="hidden" name="user_uname_match" value="XOOPS_MATCH_START" />
743
-                <input class="btn" type="submit" value="' . XoopsLocale::A_SEARCH . '" name="speed_search">&nbsp;
744
-                <input class="btn success" type="submit" value="' . XoopsLocale::ADVANCED_SEARCH . '" name="complet_search"></form>
743
+                <input class="btn" type="submit" value="' . XoopsLocale::A_SEARCH.'" name="speed_search">&nbsp;
744
+                <input class="btn success" type="submit" value="' . XoopsLocale::ADVANCED_SEARCH.'" name="complet_search"></form>
745 745
                 ';
746 746
 
747 747
             //select groupe
@@ -752,7 +752,7 @@  discard block
 block discarded – undo
752 752
             /* @var $group XoopsGroup */
753 753
             foreach ($group_arr as $group) {
754 754
                 if ($group->getVar("groupid") != FixedGroups::ANONYMOUS) {
755
-                    $form_select_groups .= '<option value="' . $group->getVar("groupid") . '"  ' . ($selgroups == $group->getVar("groupid") ? ' selected="selected"' : '') . '>' . $group->getVar("name") . '</option>';
755
+                    $form_select_groups .= '<option value="'.$group->getVar("groupid").'"  '.($selgroups == $group->getVar("groupid") ? ' selected="selected"' : '').'>'.$group->getVar("name").'</option>';
756 756
                 }
757 757
             }
758 758
             unset($group);
@@ -807,7 +807,7 @@  discard block
 block discarded – undo
807 807
             }
808 808
 
809 809
             if ($users_count > $user_limit) {
810
-                $nav = new XoopsPageNav($users_count, $user_limit, $start, 'start', 'fct=users&amp;op=default' . $requete_pagenav);
810
+                $nav = new XoopsPageNav($users_count, $user_limit, $start, 'start', 'fct=users&amp;op=default'.$requete_pagenav);
811 811
                 $xoops->tpl()->assign('nav', $nav->renderNav());
812 812
             }
813 813
         }
@@ -832,17 +832,17 @@  discard block
 block discarded – undo
832 832
     switch ($match) {
833 833
         default:
834 834
         case XOOPS_MATCH_START:
835
-            $value = $value . '%';
835
+            $value = $value.'%';
836 836
             break;
837 837
         case XOOPS_MATCH_END:
838
-            $value = '%' . $value;
838
+            $value = '%'.$value;
839 839
             break;
840 840
         case XOOPS_MATCH_EQUAL:
841 841
             //$value = $value;
842 842
             $relation = '=';
843 843
             break;
844 844
         case XOOPS_MATCH_CONTAIN:
845
-            $value = '%' . $value . '%';
845
+            $value = '%'.$value.'%';
846 846
             break;
847 847
     }
848 848
     $criteria->add(new Criteria($column, $value, $relation));
Please login to merge, or discard this patch.
htdocs/modules/system/admin/users/jquery.php 2 patches
Switch Indentation   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -40,28 +40,28 @@
 block discarded – undo
40 40
 switch ($op) {
41 41
 
42 42
     // Display post
43
-    case 'display_post':
44
-        include_once $xoops->path('modules/system/include/functions.php');
43
+        case 'display_post':
44
+            include_once $xoops->path('modules/system/include/functions.php');
45 45
 
46
-        $uid = $system->cleanVars($_REQUEST, 'uid', 'int');
47
-        $total_posts = 0;
46
+            $uid = $system->cleanVars($_REQUEST, 'uid', 'int');
47
+            $total_posts = 0;
48 48
 
49
-        /* @var $plugin SystemPluginInterface */
50
-        $plugins = \Xoops\Module\Plugin::getPlugins();
51
-        foreach ($plugins as $plugin) {
52
-            if ($res = $plugin->userPosts($uid)) {
53
-                $total_posts += $res;
49
+            /* @var $plugin SystemPluginInterface */
50
+            $plugins = \Xoops\Module\Plugin::getPlugins();
51
+            foreach ($plugins as $plugin) {
52
+                if ($res = $plugin->userPosts($uid)) {
53
+                    $total_posts += $res;
54
+                }
54 55
             }
55
-        }
56 56
 
57
-        $qb = $xoops->db()->createXoopsQueryBuilder();
58
-        $eb = $qb->expr();
59
-        $sql = $qb->updatePrefix('system_user')
60
-            ->set('posts', ':posts')
61
-            ->where('uid = :uid')
62
-            ->setParameter(':posts', $total_posts)
63
-            ->setParameter(':uid', $uid);
64
-        $row_count = $sql->execute();
65
-        echo $row_count;
66
-        break;
57
+            $qb = $xoops->db()->createXoopsQueryBuilder();
58
+            $eb = $qb->expr();
59
+            $sql = $qb->updatePrefix('system_user')
60
+                ->set('posts', ':posts')
61
+                ->where('uid = :uid')
62
+                ->setParameter(':posts', $total_posts)
63
+                ->setParameter(':uid', $uid);
64
+            $row_count = $sql->execute();
65
+            echo $row_count;
66
+            break;
67 67
 }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  * @version     $Id$
20 20
  */
21 21
 
22
-include dirname(dirname(__DIR__)) . '/header.php';
22
+include dirname(dirname(__DIR__)).'/header.php';
23 23
 
24 24
 $xoops = Xoops::getInstance();
25 25
 $system = System::getInstance();
Please login to merge, or discard this patch.
htdocs/modules/system/admin/preferences/main.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
 
80 80
         // Define Breadcrumb and tips
81 81
         $admin_page = new \Xoops\Module\Admin();
82
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
82
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
83 83
         $admin_page->addBreadcrumbLink(XoopsLocale::PREFERENCES, $system->adminVersion('extensions', 'adminpath'));
84 84
         $admin_page->addBreadcrumbLink($module->getVar('name'));
85 85
         $admin_page->renderBreadcrumb();
Please login to merge, or discard this patch.
htdocs/modules/system/admin/tplsets/main.php 1 patch
Spacing   +46 added lines, -46 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
     default:
56 56
         // Define Breadcrumb and tips
57 57
         $admin_page = new \Xoops\Module\Admin();
58
-        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
58
+        $admin_page->addBreadcrumbLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
59 59
         $admin_page->addBreadcrumbLink(SystemLocale::TEMPLATES_MANAGER, $system->adminVersion('tplsets', 'adminpath'));
60 60
         $admin_page->renderBreadcrumb();
61 61
         $admin_page->addTips(SystemLocale::TEMPLATES_TIPS);
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
             $xoops->redirect('admin.php?fct=tplsets', 3, implode('<br />', $xoops->security()->getErrors()));
105 105
         }
106 106
         // Assign Breadcrumb menu
107
-        $system_breadcrumb->addHelp(system_adminVersion('tplsets', 'help') . '#override');
107
+        $system_breadcrumb->addHelp(system_adminVersion('tplsets', 'help').'#override');
108 108
         $system_breadcrumb->addLink(SystemLocale::FILES_GENERATED);
109 109
         $system_breadcrumb->render();
110 110
 
@@ -113,8 +113,8 @@  discard block
 block discarded – undo
113 113
             if (isset($_REQUEST['select_theme']) && isset($_REQUEST['force_generated'])) {
114 114
                 //on verifie si le dossier module existe
115 115
                 $template_overload = \XoopsBaseConfig::get('themes-path')
116
-                    . '/' . $_REQUEST['select_theme'] . '/modules';
117
-                $indexFile = \XoopsBaseConfig::get('root-path') . "/modules/system/include/index.html";
116
+                    . '/'.$_REQUEST['select_theme'].'/modules';
117
+                $indexFile = \XoopsBaseConfig::get('root-path')."/modules/system/include/index.html";
118 118
                 $verif_write = false;
119 119
                 $text = '';
120 120
 
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
                         mkdir($template_overload, 0777);
126 126
                     }
127 127
                     chmod($template_overload, 0777);
128
-                    copy($indexFile, $template_overload . "/index.html");
128
+                    copy($indexFile, $template_overload."/index.html");
129 129
                 }
130 130
 
131 131
                 $tplset = $system->cleanVars($POST, 'tplset', 'default', 'string');
@@ -153,42 +153,42 @@  discard block
 block discarded – undo
153 153
                                 $module = $xoops->getModuleByDirname($moddir);
154 154
                                 if (is_object($module)) {
155 155
                                     // create module folder
156
-                                    if (!is_dir($template_overload . '/' . $module->getVar('dirname'))) {
157
-                                        mkdir($template_overload . '/' . $module->getVar('dirname'), 0777);
158
-                                        chmod($template_overload . '/' . $module->getVar('dirname'), 0777);
159
-                                        copy($indexFile, $template_overload . '/' . $module->getVar('dirname') . '/index.html');
156
+                                    if (!is_dir($template_overload.'/'.$module->getVar('dirname'))) {
157
+                                        mkdir($template_overload.'/'.$module->getVar('dirname'), 0777);
158
+                                        chmod($template_overload.'/'.$module->getVar('dirname'), 0777);
159
+                                        copy($indexFile, $template_overload.'/'.$module->getVar('dirname').'/index.html');
160 160
                                     }
161 161
 
162 162
                                     // create block folder
163
-                                    if (!is_dir($template_overload . '/' . $module->getVar('dirname') . '/blocks')) {
164
-                                        if (!is_dir($template_overload . '/' . $module->getVar('dirname') . '/blocks')) {
165
-                                            mkdir($template_overload . '/' . $module->getVar('dirname') . '/blocks', 0777);
163
+                                    if (!is_dir($template_overload.'/'.$module->getVar('dirname').'/blocks')) {
164
+                                        if (!is_dir($template_overload.'/'.$module->getVar('dirname').'/blocks')) {
165
+                                            mkdir($template_overload.'/'.$module->getVar('dirname').'/blocks', 0777);
166 166
                                         }
167
-                                        chmod($template_overload . '/' . $module->getVar('dirname') . '/blocks', 0777);
168
-                                        copy($indexFile, $template_overload . '/' . $module->getVar('dirname') . '/blocks' . '/index.html');
167
+                                        chmod($template_overload.'/'.$module->getVar('dirname').'/blocks', 0777);
168
+                                        copy($indexFile, $template_overload.'/'.$module->getVar('dirname').'/blocks'.'/index.html');
169 169
                                     }
170 170
 
171 171
                                     $class = "odd";
172
-                                    $text .= '<table cellspacing="1" class="outer"><tr><th colspan="3" align="center">' . XoopsLocale::C_MODULES . ucfirst($module->getVar('dirname')) . '</th></tr><tr><th align="center">' . XoopsLocale::TYPES . '</th><th  align="center">' . XoopsLocale::FILES . '</th><th>' . XoopsLocale::STATUS . '</th></tr>';
172
+                                    $text .= '<table cellspacing="1" class="outer"><tr><th colspan="3" align="center">'.XoopsLocale::C_MODULES.ucfirst($module->getVar('dirname')).'</th></tr><tr><th align="center">'.XoopsLocale::TYPES.'</th><th  align="center">'.XoopsLocale::FILES.'</th><th>'.XoopsLocale::STATUS.'</th></tr>';
173 173
 
174 174
                                     // create template
175 175
                                     $templates = $tpltpl_handler->find($tplsetname, 'module', null, $moddir);
176 176
                                     for ($j = 0; $j < count($templates); ++$j) {
177 177
                                         $filename = $templates[$j]->getVar('tpl_file');
178 178
                                         if ($tplsetname == $tplset) {
179
-                                            $physical_file = \XoopsBaseConfig::get('themes-path') . '/' . $_REQUEST['select_theme'] . '/modules/' . $moddir . '/' . $filename;
179
+                                            $physical_file = \XoopsBaseConfig::get('themes-path').'/'.$_REQUEST['select_theme'].'/modules/'.$moddir.'/'.$filename;
180 180
 
181 181
                                             $tplfile = $tpltpl_handler->get($templates[$j]->getVar('tpl_id'), true);
182 182
 
183 183
                                             if (is_object($tplfile)) {
184 184
                                                 if (!XoopsLoad::fileExists($physical_file) || $_REQUEST['force_generated'] == 1) {
185 185
                                                     $open = fopen($physical_file, 'w+');
186
-                                                    if (fwrite($open, "" . html_entity_decode($tplfile->getVar('tpl_source', 'E'), ENT_QUOTES))) {
187
-                                                        $text .= '<tr class="' . $class . '"><td align="center">' . XoopsLocale::TEMPLATES . '</td><td>' . $physical_file . '</td><td align="center">';
186
+                                                    if (fwrite($open, "".html_entity_decode($tplfile->getVar('tpl_source', 'E'), ENT_QUOTES))) {
187
+                                                        $text .= '<tr class="'.$class.'"><td align="center">'.XoopsLocale::TEMPLATES.'</td><td>'.$physical_file.'</td><td align="center">';
188 188
                                                         if (XoopsLoad::fileExists($physical_file)) {
189
-                                                            $text .= '<img width="16" src="' . system_AdminIcons('success.png') . '" /></td></tr>';
189
+                                                            $text .= '<img width="16" src="'.system_AdminIcons('success.png').'" /></td></tr>';
190 190
                                                         } else {
191
-                                                            $text .= '<img width="16" src="' . system_AdminIcons('cancel.png') . '" /></td></tr>';
191
+                                                            $text .= '<img width="16" src="'.system_AdminIcons('cancel.png').'" /></td></tr>';
192 192
                                                         }
193 193
                                                         $verif_write = true;
194 194
                                                     }
@@ -204,18 +204,18 @@  discard block
 block discarded – undo
204 204
                                     for ($k = 0; $k < count($btemplates); ++$k) {
205 205
                                         $filename = $btemplates[$k]->getVar('tpl_file');
206 206
                                         if ($tplsetname == $tplset) {
207
-                                            $physical_file = \XoopsBaseConfig::get('themes-path') . '/' . $_REQUEST['select_theme'] . '/modules/' . $moddir . '/blocks/' . $filename;
207
+                                            $physical_file = \XoopsBaseConfig::get('themes-path').'/'.$_REQUEST['select_theme'].'/modules/'.$moddir.'/blocks/'.$filename;
208 208
                                             $btplfile = $tpltpl_handler->get($btemplates[$k]->getVar('tpl_id'), true);
209 209
 
210 210
                                             if (is_object($btplfile)) {
211 211
                                                 if (!XoopsLoad::fileExists($physical_file) || $_REQUEST['force_generated'] == 1) {
212 212
                                                     $open = fopen($physical_file, 'w+');
213
-                                                    if (fwrite($open, "" . utf8_encode(html_entity_decode($btplfile->getVar('tpl_source', 'E'))) . "")) {
214
-                                                        $text .= '<tr class="' . $class . '"><td align="center">' . XoopsLocale::BLOCKS . '</td><td>' . $physical_file . '</td><td align="center">';
213
+                                                    if (fwrite($open, "".utf8_encode(html_entity_decode($btplfile->getVar('tpl_source', 'E')))."")) {
214
+                                                        $text .= '<tr class="'.$class.'"><td align="center">'.XoopsLocale::BLOCKS.'</td><td>'.$physical_file.'</td><td align="center">';
215 215
                                                         if (XoopsLoad::fileExists($physical_file)) {
216
-                                                            $text .= '<img width="16" src="' . system_AdminIcons('success.png') . '" /></td></tr>';
216
+                                                            $text .= '<img width="16" src="'.system_AdminIcons('success.png').'" /></td></tr>';
217 217
                                                         } else {
218
-                                                            $text .= '<img width="16" src="' . system_AdminIcons('cancel.png') . '" /></td></tr>';
218
+                                                            $text .= '<img width="16" src="'.system_AdminIcons('cancel.png').'" /></td></tr>';
219 219
                                                         }
220 220
                                                         $verif_write = true;
221 221
                                                     }
@@ -241,23 +241,23 @@  discard block
 block discarded – undo
241 241
                             $module = $xoops->getModuleByDirname($moddir);
242 242
                             if (is_object($module)) {
243 243
                                 // create module folder
244
-                                if (!is_dir($template_overload . '/' . $module->getVar('dirname'))) {
245
-                                    mkdir($template_overload . '/' . $module->getVar('dirname'), 0777);
246
-                                    chmod($template_overload . '/' . $module->getVar('dirname'), 0777);
247
-                                    copy($indexFile, $template_overload . '/' . $module->getVar('dirname') . '/index.html');
244
+                                if (!is_dir($template_overload.'/'.$module->getVar('dirname'))) {
245
+                                    mkdir($template_overload.'/'.$module->getVar('dirname'), 0777);
246
+                                    chmod($template_overload.'/'.$module->getVar('dirname'), 0777);
247
+                                    copy($indexFile, $template_overload.'/'.$module->getVar('dirname').'/index.html');
248 248
                                 }
249 249
 
250 250
                                 // create block folder
251
-                                if (!is_dir($template_overload . '/' . $module->getVar('dirname') . '/blocks')) {
252
-                                    if (!is_dir($template_overload . '/' . $module->getVar('dirname') . '/blocks')) {
253
-                                        mkdir($template_overload . '/' . $module->getVar('dirname') . '/blocks', 0777);
251
+                                if (!is_dir($template_overload.'/'.$module->getVar('dirname').'/blocks')) {
252
+                                    if (!is_dir($template_overload.'/'.$module->getVar('dirname').'/blocks')) {
253
+                                        mkdir($template_overload.'/'.$module->getVar('dirname').'/blocks', 0777);
254 254
                                     }
255
-                                    chmod($template_overload . '/' . $module->getVar('dirname') . '/blocks', 0777);
256
-                                    copy($indexFile, $template_overload . '/' . $module->getVar('dirname') . '/blocks' . '/index.html');
255
+                                    chmod($template_overload.'/'.$module->getVar('dirname').'/blocks', 0777);
256
+                                    copy($indexFile, $template_overload.'/'.$module->getVar('dirname').'/blocks'.'/index.html');
257 257
                                 }
258 258
 
259 259
                                 $class = "odd";
260
-                                $text .= '<table cellspacing="1" class="outer"><tr><th colspan="3" align="center">' . XoopsLocale::C_MODULES . ucfirst($module->getVar('dirname')) . '</th></tr><tr><th align="center">' . XoopsLocale::TYPES . '</th><th  align="center">' . XoopsLocale::FILES . '</th><th>' . XoopsLocale::STATUS . '</th></tr>';
260
+                                $text .= '<table cellspacing="1" class="outer"><tr><th colspan="3" align="center">'.XoopsLocale::C_MODULES.ucfirst($module->getVar('dirname')).'</th></tr><tr><th align="center">'.XoopsLocale::TYPES.'</th><th  align="center">'.XoopsLocale::FILES.'</th><th>'.XoopsLocale::STATUS.'</th></tr>';
261 261
                                 $select_templates_modules = $_REQUEST['select_templates_modules'];
262 262
                                 for ($l = 0; $l < count($_REQUEST['select_templates_modules']); ++$l) {
263 263
                                     // create template
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
                                     for ($j = 0; $j < count($templates); ++$j) {
266 266
                                         $filename = $templates[$j]->getVar('tpl_file');
267 267
                                         if ($tplsetname == $tplset) {
268
-                                            $physical_file = \XoopsBaseConfig::get('themes-path') . '/' . $_REQUEST['select_theme'] . '/modules/' . $moddir . '/' . $filename;
268
+                                            $physical_file = \XoopsBaseConfig::get('themes-path').'/'.$_REQUEST['select_theme'].'/modules/'.$moddir.'/'.$filename;
269 269
 
270 270
                                             $tplfile = $tpltpl_handler->get($templates[$j]->getVar('tpl_id'), true);
271 271
 
@@ -274,11 +274,11 @@  discard block
 block discarded – undo
274 274
                                                     if ($select_templates_modules[$l] == $filename) {
275 275
                                                         $open = fopen($physical_file, 'w+');
276 276
                                                         if (fwrite($open, html_entity_decode($tplfile->getVar('tpl_source', 'E'), ENT_QUOTES))) {
277
-                                                            $text .= '<tr class="' . $class . '"><td align="center">' . XoopsLocale::TEMPLATES . '</td><td>' . $physical_file . '</td><td align="center">';
277
+                                                            $text .= '<tr class="'.$class.'"><td align="center">'.XoopsLocale::TEMPLATES.'</td><td>'.$physical_file.'</td><td align="center">';
278 278
                                                             if (XoopsLoad::fileExists($physical_file)) {
279
-                                                                $text .= '<img width="16" src="' . system_AdminIcons('success.png') . '" /></td></tr>';
279
+                                                                $text .= '<img width="16" src="'.system_AdminIcons('success.png').'" /></td></tr>';
280 280
                                                             } else {
281
-                                                                $text .= '<img width="16" src="' . system_AdminIcons('cancel.png') . '" /></td></tr>';
281
+                                                                $text .= '<img width="16" src="'.system_AdminIcons('cancel.png').'" /></td></tr>';
282 282
                                                             }
283 283
                                                             $verif_write = true;
284 284
                                                         }
@@ -295,19 +295,19 @@  discard block
 block discarded – undo
295 295
                                     for ($k = 0; $k < count($btemplates); ++$k) {
296 296
                                         $filename = $btemplates[$k]->getVar('tpl_file');
297 297
                                         if ($tplsetname == $tplset) {
298
-                                            $physical_file = \XoopsBaseConfig::get('themes-path') . '/' . $_REQUEST['select_theme'] . '/modules/' . $moddir . '/blocks/' . $filename;
298
+                                            $physical_file = \XoopsBaseConfig::get('themes-path').'/'.$_REQUEST['select_theme'].'/modules/'.$moddir.'/blocks/'.$filename;
299 299
                                             $btplfile = $tpltpl_handler->get($btemplates[$k]->getVar('tpl_id'), true);
300 300
 
301 301
                                             if (is_object($btplfile)) {
302 302
                                                 if (!XoopsLoad::fileExists($physical_file) || $_REQUEST['force_generated'] == 1) {
303 303
                                                     if ($select_templates_modules[$l] == $filename) {
304 304
                                                         $open = fopen($physical_file, 'w+');
305
-                                                        if (fwrite($open, utf8_encode(html_entity_decode($btplfile->getVar('tpl_source', 'E'))) . "")) {
306
-                                                            $text .= '<tr class="' . $class . '"><td align="center">' . XoopsLocale::BLOCKS . '</td><td>' . $physical_file . '</td><td align="center">';
305
+                                                        if (fwrite($open, utf8_encode(html_entity_decode($btplfile->getVar('tpl_source', 'E')))."")) {
306
+                                                            $text .= '<tr class="'.$class.'"><td align="center">'.XoopsLocale::BLOCKS.'</td><td>'.$physical_file.'</td><td align="center">';
307 307
                                                             if (XoopsLoad::fileExists($physical_file)) {
308
-                                                                $text .= '<img width="16" src="' . system_AdminIcons('success.png') . '" /></td></tr>';
308
+                                                                $text .= '<img width="16" src="'.system_AdminIcons('success.png').'" /></td></tr>';
309 309
                                                             } else {
310
-                                                                $text .= '<img width="16" src="' . system_AdminIcons('cancel.png') . '" /></td></tr>';
310
+                                                                $text .= '<img width="16" src="'.system_AdminIcons('cancel.png').'" /></td></tr>';
311 311
                                                             }
312 312
                                                             $verif_write = true;
313 313
                                                         }
@@ -365,14 +365,14 @@  discard block
 block discarded – undo
365 365
         }
366 366
         $clean_path_file = Request::getString('path_file', '');
367 367
         if (!empty($clean_path_file)) {
368
-            $path_file = realpath(\XoopsBaseConfig::get('themes-path') . '/' . trim($clean_path_file));
368
+            $path_file = realpath(\XoopsBaseConfig::get('themes-path').'/'.trim($clean_path_file));
369 369
             $path_file = str_replace('\\', '/', $path_file);
370 370
             $pathInfo = pathinfo($path_file);
371 371
             if (!in_array($pathInfo['extension'], array('css', 'html', 'tpl'))) {
372 372
                 $xoops->redirect("admin.php?fct=tplsets", 2, XoopsLocale::E_NOT_DONE);
373 373
             }
374 374
             // copy file
375
-            $copy_file = $path_file . '.back';
375
+            $copy_file = $path_file.'.back';
376 376
             copy($path_file, $copy_file);
377 377
             // Save modif
378 378
             if (isset($_REQUEST['templates'])) {
Please login to merge, or discard this patch.
htdocs/modules/system/admin/tplsets/jquery.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
  * @package     system
21 21
  */
22 22
 
23
-include dirname(dirname(__DIR__)) . '/header.php';
23
+include dirname(dirname(__DIR__)).'/header.php';
24 24
 
25 25
 $xoops = Xoops::getInstance();
26 26
 $xoops->logger()->quiet();
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
     exit(XoopsLocale::E_NO_ACCESS_PERMISSION);
30 30
 }
31 31
 
32
-if (isset($_REQUEST['dir']) && strpos($_REQUEST['dir'],'..') !== false || isset($_REQUEST['path_file']) && strpos($_REQUEST['path_file'],'..') !== false) {
32
+if (isset($_REQUEST['dir']) && strpos($_REQUEST['dir'], '..') !== false || isset($_REQUEST['path_file']) && strpos($_REQUEST['path_file'], '..') !== false) {
33 33
     exit(XoopsLocale::E_NO_ACCESS_PERMISSION);
34 34
 }
35 35
 
@@ -42,33 +42,33 @@  discard block
 block discarded – undo
42 42
     case "tpls_display_folder":
43 43
         $_REQUEST['dir'] = urldecode($_REQUEST['dir']);
44 44
         $root = \XoopsBaseConfig::get('themes-path');
45
-        if (XoopsLoad::fileExists($root . $_REQUEST['dir'])) {
46
-            $files = scandir($root . $_REQUEST['dir']);
45
+        if (XoopsLoad::fileExists($root.$_REQUEST['dir'])) {
46
+            $files = scandir($root.$_REQUEST['dir']);
47 47
             natcasesort($files);
48 48
             if (count($files) > 2) { /* The 2 accounts for . and .. */
49 49
                 echo "<ul class=\"jqueryFileTree\" style=\"display: none;\">";
50 50
                 // All dirs
51 51
                 foreach ($files as $file) {
52 52
 
53
-                    if (XoopsLoad::fileExists($root . $_REQUEST['dir'] . $file) && $file !== '.' && $file !== '..' && is_dir($root . $_REQUEST['dir'] . $file)) {
53
+                    if (XoopsLoad::fileExists($root.$_REQUEST['dir'].$file) && $file !== '.' && $file !== '..' && is_dir($root.$_REQUEST['dir'].$file)) {
54 54
                         //retirer .svn
55 55
                         $file_no_valid = array('.svn', 'icons', 'img', 'images', 'language', 'locale');
56 56
 
57 57
                         if (!in_array($file, $file_no_valid)) {
58
-                            echo "<li class=\"directory collapsed\"><a href=\"#\" rel=\"" . htmlentities($_REQUEST['dir'] . $file) . "/\">" . htmlentities($file) . "</a></li>";
58
+                            echo "<li class=\"directory collapsed\"><a href=\"#\" rel=\"".htmlentities($_REQUEST['dir'].$file)."/\">".htmlentities($file)."</a></li>";
59 59
                         }
60 60
                     }
61 61
                 }
62 62
                 // All files
63 63
                 foreach ($files as $file) {
64
-                    if (XoopsLoad::fileExists($root . $_REQUEST['dir'] . $file) && $file !== '.' && $file !== '..' && !is_dir($root . $_REQUEST['dir'] . $file) && $file !== 'index.html') {
64
+                    if (XoopsLoad::fileExists($root.$_REQUEST['dir'].$file) && $file !== '.' && $file !== '..' && !is_dir($root.$_REQUEST['dir'].$file) && $file !== 'index.html') {
65 65
                         $ext = preg_replace('/^.*\./', '', $file);
66 66
 
67 67
                         $extensions = array('.tpl', '.html', '.htm', '.css');
68 68
                         $extension_verif = strrchr($file, '.');
69 69
 
70 70
                         if (in_array($extension_verif, $extensions)) {
71
-                            echo "<li class=\"file ext_$ext\"><a href=\"#\" onclick=\"tpls_edit_file('" . htmlentities($_REQUEST['dir'] . $file) . "', '" . htmlentities($_REQUEST['dir']) . "', '" . htmlentities($file) . "', '" . $ext . "');\" rel=\"tpls_edit_file('" . htmlentities($_REQUEST['dir'] . $file) . "', '" . htmlentities($_REQUEST['dir']) . "', '" . htmlentities($file) . "', '" . $ext . "');\">" . htmlentities($file) . "</a></li>";
71
+                            echo "<li class=\"file ext_$ext\"><a href=\"#\" onclick=\"tpls_edit_file('".htmlentities($_REQUEST['dir'].$file)."', '".htmlentities($_REQUEST['dir'])."', '".htmlentities($file)."', '".$ext."');\" rel=\"tpls_edit_file('".htmlentities($_REQUEST['dir'].$file)."', '".htmlentities($_REQUEST['dir'])."', '".htmlentities($file)."', '".$ext."');\">".htmlentities($file)."</a></li>";
72 72
                         } else {
73 73
                             //echo "<li class=\"file ext_$ext\">" . htmlentities($file) . "</li>";
74 74
                         }
@@ -83,12 +83,12 @@  discard block
 block discarded – undo
83 83
         $clean_file = Request::getString('file', '');
84 84
         $clean_path_file = Request::getString('path_file', '');
85 85
         $path_file = realpath(XOOPS_ROOT_PATH.'/themes'.trim($clean_path_file));
86
-        $path_file = str_replace('\\','/',$path_file);
86
+        $path_file = str_replace('\\', '/', $path_file);
87 87
 
88 88
         //Button restore
89
-        if (XoopsLoad::fileExists($path_file . '.back')) {
90
-            $restore = '<button class="ui-corner-all tooltip" type="button" onclick="tpls_restore(\'' . $path_file . '\')" value="' . XoopsLocale::A_RESTORE . '" title="' . XoopsLocale::A_RESTORE . '">
91
-                            <img src="' . system_AdminIcons('revert.png') . '" alt="' . XoopsLocale::A_RESTORE . '" />
89
+        if (XoopsLoad::fileExists($path_file.'.back')) {
90
+            $restore = '<button class="ui-corner-all tooltip" type="button" onclick="tpls_restore(\''.$path_file.'\')" value="'.XoopsLocale::A_RESTORE.'" title="'.XoopsLocale::A_RESTORE.'">
91
+                            <img src="' . system_AdminIcons('revert.png').'" alt="'.XoopsLocale::A_RESTORE.'" />
92 92
                         </button>';
93 93
         } else {
94 94
             $restore = '';
@@ -107,12 +107,12 @@  discard block
 block discarded – undo
107 107
                       <td>
108 108
                           <div class="xo-btn-actions">
109 109
                               <div class="xo-buttons">
110
-                                  <button class="ui-corner-all tooltip" type="submit" value="' . XoopsLocale::A_SAVE . '" title="' . XoopsLocale::A_SAVE . '">
111
-                                      <img src="' . system_AdminIcons('save.png') . '" alt="' . XoopsLocale::A_SAVE . '" />
110
+                                  <button class="ui-corner-all tooltip" type="submit" value="' . XoopsLocale::A_SAVE.'" title="'.XoopsLocale::A_SAVE.'">
111
+                                      <img src="' . system_AdminIcons('save.png').'" alt="'.XoopsLocale::A_SAVE.'" />
112 112
                                   </button>
113
-                                  ' . $restore . '
114
-                                  <button class="ui-corner-all tooltip" type="button" onclick="$(\'#display_contenu\').hide();$(\'#display_form\').fadeIn(\'fast\');" title="' . XoopsLocale::A_CANCEL . '">
115
-                                      <img src="' . system_AdminIcons('cancel.png') . '" alt="' . XoopsLocale::A_CANCEL . '" />
113
+                                  ' . $restore.'
114
+                                  <button class="ui-corner-all tooltip" type="button" onclick="$(\'#display_contenu\').hide();$(\'#display_form\').fadeIn(\'fast\');" title="' . XoopsLocale::A_CANCEL.'">
115
+                                      <img src="' . system_AdminIcons('cancel.png').'" alt="'.XoopsLocale::A_CANCEL.'" />
116 116
                                   </button>
117 117
                                   <div class="clear"></div>
118 118
                              </div>
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
                       </td>
121 121
                   </tr>
122 122
                   <tr>
123
-                      <td><textarea id="code_mirror" name="templates" rows=24 cols=110>' . $content . '</textarea></td>
123
+                      <td><textarea id="code_mirror" name="templates" rows=24 cols=110>' . $content.'</textarea></td>
124 124
                   </tr>
125 125
               </table>';
126 126
         $xoopsToken = new \Xoops\Form\Token();
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
         $extensions = array('.tpl', '.html', '.htm', '.css');
134 134
 
135 135
         //check if the file is inside themes directory
136
-        $valid_dir = stristr(realpath($_REQUEST['path_file']), realpath(\XoopsBaseConfig::get('root-path') . '/themes'));
136
+        $valid_dir = stristr(realpath($_REQUEST['path_file']), realpath(\XoopsBaseConfig::get('root-path').'/themes'));
137 137
 
138
-        $old_file = $_REQUEST['path_file'] . '.back';
138
+        $old_file = $_REQUEST['path_file'].'.back';
139 139
         $new_file = $_REQUEST['path_file'];
140 140
 
141 141
         $extension_verif = strrchr($new_file, '.');
Please login to merge, or discard this patch.
htdocs/modules/system/admin/groupperm.php 2 patches
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
  */
20 20
 
21 21
 // Include XOOPS control panel header
22
-include_once dirname(dirname(dirname(__DIR__))) . '/include/cp_header.php';
22
+include_once dirname(dirname(dirname(__DIR__))).'/include/cp_header.php';
23 23
 
24 24
 $xoops = Xoops::getInstance();
25 25
 $xoops->loadLocale('system');
@@ -46,11 +46,11 @@  discard block
 block discarded – undo
46 46
             continue;
47 47
         }
48 48
         if (false == $gperm_handler->deleteByModule($modid, $perm_name)) {
49
-            $msg[] = sprintf(SystemLocale::EF_COULD_NOT_RESET_GROUP_PERMISSION_FOR_MODULE, $module->getVar('name') . '(' . $perm_name . ')');
49
+            $msg[] = sprintf(SystemLocale::EF_COULD_NOT_RESET_GROUP_PERMISSION_FOR_MODULE, $module->getVar('name').'('.$perm_name.')');
50 50
         }
51
-        if (!array_key_exists('groups', $perm_data)){
51
+        if (!array_key_exists('groups', $perm_data)) {
52 52
             $msg[] = sprintf(SystemLocale::SF_ADDED_PERMISSION_FOR_GROUP, $module->getVar('name'), $perm_name, ' /');
53
-        }else{
53
+        } else {
54 54
             foreach ($perm_data['groups'] as $group_id => $item_ids) {
55 55
                 foreach ($item_ids as $item_id => $selected) {
56 56
                     if ($selected == 1) {
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
                             foreach ($parent_ids as $pid) {
61 61
                                 if ($pid != 0 && !in_array($pid, array_keys($item_ids))) {
62 62
                                     // one of the parent items were not selected, so skip this item
63
-                                    $msg[] = sprintf(SystemLocale::EF_COULD_NOT_ADD_PERMISSION_FOR_GROUP, '<strong>' . $perm_name . '</strong>', '<strong>' . $perm_data['itemname'][$item_id] . '</strong>', '<strong>' . $group_list[$group_id] . '</strong>') . ' (' . XoopsLocale::E_ALL_PARENT_ITEMS_MUST_BE_SELECTED . ')';
63
+                                    $msg[] = sprintf(SystemLocale::EF_COULD_NOT_ADD_PERMISSION_FOR_GROUP, '<strong>'.$perm_name.'</strong>', '<strong>'.$perm_data['itemname'][$item_id].'</strong>', '<strong>'.$group_list[$group_id].'</strong>').' ('.XoopsLocale::E_ALL_PARENT_ITEMS_MUST_BE_SELECTED.')';
64 64
                                     continue 2;
65 65
                                 }
66 66
                             }
@@ -71,9 +71,9 @@  discard block
 block discarded – undo
71 71
                         $gperm->setVar('gperm_modid', $modid);
72 72
                         $gperm->setVar('gperm_itemid', $item_id);
73 73
                         if (!$gperm_handler->insert($gperm)) {
74
-                            $msg[] = sprintf(SystemLocale::EF_COULD_NOT_ADD_PERMISSION_FOR_GROUP, '<strong>' . $perm_name . '</strong>', '<strong>' . $perm_data['itemname'][$item_id] . '</strong>', '<strong>' . $group_list[$group_id] . '</strong>');
74
+                            $msg[] = sprintf(SystemLocale::EF_COULD_NOT_ADD_PERMISSION_FOR_GROUP, '<strong>'.$perm_name.'</strong>', '<strong>'.$perm_data['itemname'][$item_id].'</strong>', '<strong>'.$group_list[$group_id].'</strong>');
75 75
                         } else {
76
-                            $msg[] = sprintf(SystemLocale::SF_ADDED_PERMISSION_FOR_GROUP, '<strong>' . $perm_name . '</strong>', '<strong>' . $perm_data['itemname'][$item_id] . '</strong>', '<strong>' . $group_list[$group_id] . '</strong>');
76
+                            $msg[] = sprintf(SystemLocale::SF_ADDED_PERMISSION_FOR_GROUP, '<strong>'.$perm_name.'</strong>', '<strong>'.$perm_data['itemname'][$item_id].'</strong>', '<strong>'.$group_list[$group_id].'</strong>');
77 77
                         }
78 78
                         unset($gperm);
79 79
                     }
@@ -86,9 +86,9 @@  discard block
 block discarded – undo
86 86
 if ($module->getVar('hasadmin')) {
87 87
     $adminindex = isset($_POST['redirect_url']) ? $_POST['redirect_url'] : $module->getInfo('adminindex');
88 88
     if ($adminindex) {
89
-        $backlink = $xoops->url('modules/' . $module->getVar('dirname') . '/' . $adminindex);
89
+        $backlink = $xoops->url('modules/'.$module->getVar('dirname').'/'.$adminindex);
90 90
     }
91 91
 }
92
-$backlink = ($backlink) ? $backlink : \XoopsBaseConfig::get('url') . '/admin.php';
92
+$backlink = ($backlink) ? $backlink : \XoopsBaseConfig::get('url').'/admin.php';
93 93
 
94 94
 $xoops->redirect($backlink, 2, implode("<br />", $msg));
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
         }
51 51
         if (!array_key_exists('groups', $perm_data)){
52 52
             $msg[] = sprintf(SystemLocale::SF_ADDED_PERMISSION_FOR_GROUP, $module->getVar('name'), $perm_name, ' /');
53
-        }else{
53
+        } else{
54 54
             foreach ($perm_data['groups'] as $group_id => $item_ids) {
55 55
                 foreach ($item_ids as $item_id => $selected) {
56 56
                     if ($selected == 1) {
Please login to merge, or discard this patch.