Test Failed
Push — master ( 398493...d4ef72 )
by Michael
11:04
created
htdocs/include/site-closed.php 2 patches
Indentation   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -47,19 +47,19 @@
 block discarded – undo
47 47
         'type' => 'text/javascript'));
48 48
     $xoopsTpl = $xoTheme->template;
49 49
     $xoopsTpl->assign(array(
50
-                          'xoops_theme'       => $xoopsConfig['theme_set'],
51
-                          'xoops_imageurl'    => XOOPS_THEME_URL . '/' . $xoopsConfig['theme_set'] . '/',
52
-                          'xoops_themecss'    => xoops_getcss($xoopsConfig['theme_set']),
53
-                          'xoops_requesturi'  => htmlspecialchars($_SERVER['REQUEST_URI'], ENT_QUOTES),
54
-                          'xoops_sitename'    => htmlspecialchars($xoopsConfig['sitename'], ENT_QUOTES),
55
-                          'xoops_slogan'      => htmlspecialchars($xoopsConfig['slogan'], ENT_QUOTES),
56
-                          'xoops_dirname'     => @$xoopsModule ? $xoopsModule->getVar('dirname') : 'system',
57
-                          'xoops_banner'      => $xoopsConfig['banners'] ? xoops_getbanner() : ' ',
58
-                          'xoops_pagetitle'   => isset($xoopsModule) && is_object($xoopsModule) ? $xoopsModule->getVar('name') : htmlspecialchars($xoopsConfig['slogan'], ENT_QUOTES),
59
-                          'lang_login'        => _LOGIN,
60
-                          'lang_username'     => _USERNAME,
61
-                          'lang_password'     => _PASSWORD,
62
-                          'lang_siteclosemsg' => $xoopsConfig['closesite_text']));
50
+                            'xoops_theme'       => $xoopsConfig['theme_set'],
51
+                            'xoops_imageurl'    => XOOPS_THEME_URL . '/' . $xoopsConfig['theme_set'] . '/',
52
+                            'xoops_themecss'    => xoops_getcss($xoopsConfig['theme_set']),
53
+                            'xoops_requesturi'  => htmlspecialchars($_SERVER['REQUEST_URI'], ENT_QUOTES),
54
+                            'xoops_sitename'    => htmlspecialchars($xoopsConfig['sitename'], ENT_QUOTES),
55
+                            'xoops_slogan'      => htmlspecialchars($xoopsConfig['slogan'], ENT_QUOTES),
56
+                            'xoops_dirname'     => @$xoopsModule ? $xoopsModule->getVar('dirname') : 'system',
57
+                            'xoops_banner'      => $xoopsConfig['banners'] ? xoops_getbanner() : ' ',
58
+                            'xoops_pagetitle'   => isset($xoopsModule) && is_object($xoopsModule) ? $xoopsModule->getVar('name') : htmlspecialchars($xoopsConfig['slogan'], ENT_QUOTES),
59
+                            'lang_login'        => _LOGIN,
60
+                            'lang_username'     => _USERNAME,
61
+                            'lang_password'     => _PASSWORD,
62
+                            'lang_siteclosemsg' => $xoopsConfig['closesite_text']));
63 63
     /* @var $config_handler XoopsConfigHandler  */
64 64
     $config_handler = xoops_getHandler('config');
65 65
     $criteria       = new CriteriaCompo(new Criteria('conf_modid', 0));
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
     $xoopsTpl = $xoTheme->template;
49 49
     $xoopsTpl->assign(array(
50 50
                           'xoops_theme'       => $xoopsConfig['theme_set'],
51
-                          'xoops_imageurl'    => XOOPS_THEME_URL . '/' . $xoopsConfig['theme_set'] . '/',
51
+                          'xoops_imageurl'    => XOOPS_THEME_URL.'/'.$xoopsConfig['theme_set'].'/',
52 52
                           'xoops_themecss'    => xoops_getcss($xoopsConfig['theme_set']),
53 53
                           'xoops_requesturi'  => htmlspecialchars($_SERVER['REQUEST_URI'], ENT_QUOTES),
54 54
                           'xoops_sitename'    => htmlspecialchars($xoopsConfig['sitename'], ENT_QUOTES),
Please login to merge, or discard this patch.
htdocs/include/comment_new.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 include_once $GLOBALS['xoops']->path('include/comment_constants.php');
22 22
 
23 23
 if (('system' !== $xoopsModule->getVar('dirname') && XOOPS_COMMENT_APPROVENONE == $xoopsModuleConfig['com_rule']) || (!is_object($xoopsUser) && !$xoopsModuleConfig['com_anonpost']) || !is_object($xoopsModule)) {
24
-    redirect_header(XOOPS_URL . '/user.php', 1, _NOPERM);
24
+    redirect_header(XOOPS_URL.'/user.php', 1, _NOPERM);
25 25
 }
26 26
 
27 27
 xoops_loadLanguage('comment');
@@ -32,14 +32,14 @@  discard block
 block discarded – undo
32 32
     if (isset($com_replytitle)) {
33 33
         if (isset($com_replytext)) {
34 34
             echo '<table cellpadding="4" cellspacing="1" width="98%" class="outer">
35
-                  <tr><td class="head">' . $com_replytitle . '</td></tr>
36
-                  <tr><td><br>' . $com_replytext . '<br></td></tr>
35
+                  <tr><td class="head">' . $com_replytitle.'</td></tr>
36
+                  <tr><td><br>' . $com_replytext.'<br></td></tr>
37 37
                   </table>';
38 38
         }
39 39
         $myts      = MyTextSanitizer::getInstance();
40 40
         $com_title = $myts->htmlSpecialChars($com_replytitle);
41
-        if (!preg_match('/^' . _RE . '/i', $com_title)) {
42
-            $com_title = _RE . ' ' . xoops_substr($com_title, 0, 56);
41
+        if (!preg_match('/^'._RE.'/i', $com_title)) {
42
+            $com_title = _RE.' '.xoops_substr($com_title, 0, 56);
43 43
         }
44 44
     } else {
45 45
         $com_title = '';
Please login to merge, or discard this patch.
htdocs/include/searchform.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -25,9 +25,9 @@
 block discarded – undo
25 25
 $search_form->addElement(new XoopsFormText(_SR_KEYWORDS, 'query', 30, 255, htmlspecialchars(stripslashes(implode(' ', $queries)), ENT_QUOTES)), true);
26 26
 $type_select = new XoopsFormSelect(_SR_TYPE, 'andor', $andor);
27 27
 $type_select->addOptionArray(array(
28
-                                 'AND'   => _SR_ALL,
29
-                                 'OR'    => _SR_ANY,
30
-                                 'exact' => _SR_EXACT));
28
+                                    'AND'   => _SR_ALL,
29
+                                    'OR'    => _SR_ANY,
30
+                                    'exact' => _SR_EXACT));
31 31
 $search_form->addElement($type_select);
32 32
 if (!empty($mids)) {
33 33
     $mods_checkbox = new XoopsFormCheckBox(_SR_SEARCHIN, 'mids[]', $mids);
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
     $criteria->add(new Criteria('hassearch', 1));
40 40
     $criteria->add(new Criteria('isactive', 1));
41 41
     if (!empty($available_modules)) {
42
-        $criteria->add(new Criteria('mid', '(' . implode(',', $available_modules) . ')', 'IN'));
42
+        $criteria->add(new Criteria('mid', '('.implode(',', $available_modules).')', 'IN'));
43 43
     }
44 44
     /* @var $module_handler XoopsModuleHandler */
45 45
     $module_handler = xoops_getHandler('module');
Please login to merge, or discard this patch.
htdocs/include/formdhtmltextarea_preview.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
  * @author              Vinod <[email protected]>
17 17
  * @author              Taiwen Jiang <[email protected]>
18 18
  */
19
-include_once dirname(__DIR__) . '/mainfile.php';
19
+include_once dirname(__DIR__).'/mainfile.php';
20 20
 
21 21
 $xoopsLogger->activated = false;
22 22
 $myts                   = MyTextSanitizer::getInstance();
@@ -31,17 +31,17 @@  discard block
 block discarded – undo
31 31
 $content = $myts->displayTarea($content, $html, 1, 1, 1, 1);
32 32
 if (preg_match_all('/%u([[:alnum:]]{4})/', $content, $matches)) {
33 33
     foreach ($matches[1] as $uniord) {
34
-        $utf     = '&#x' . $uniord . ';';
35
-        $content = str_replace('%u' . $uniord, $utf, $content);
34
+        $utf     = '&#x'.$uniord.';';
35
+        $content = str_replace('%u'.$uniord, $utf, $content);
36 36
     }
37 37
     $content = urldecode($content);
38 38
 }
39 39
 
40 40
 if (!headers_sent()) {
41 41
     $charset = (defined('_CHARSET') ? _CHARSET : 'UTF-8');
42
-    header('Content-Type:text/html; charset=' . $charset);
42
+    header('Content-Type:text/html; charset='.$charset);
43 43
     header('Expires: Mon, 26 Jul 1997 05:00:00 GMT');
44 44
     header('Cache-Control: private, no-cache');
45 45
     header('Pragma: no-cache');
46 46
 }
47
-echo '<div>' . $content . '</div>';
47
+echo '<div>'.$content.'</div>';
Please login to merge, or discard this patch.
htdocs/include/functions.encoding.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
     $r   = '';
38 38
     $len = strlen($hex);
39 39
     for ($a = 0; $a < $len; $a += 2) {
40
-        $r .= chr(hexdec($hex{$a} . $hex{$a + 1}));
40
+        $r .= chr(hexdec($hex{$a}.$hex{$a + 1}));
41 41
     }
42 42
 
43 43
     return $r;
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
     $accepted = array('0', '1', '2', '3', '4', '5', '6', '7', '8', '9', 'a', 'b', 'c', 'd', 'e', 'f');
67 67
     $len      = strlen($hex);
68 68
     if ($checklen > $len) {
69
-        $checklen = $len;//And???
69
+        $checklen = $len; //And???
70 70
     }
71 71
     $hex = strtolower($hex);
72 72
     for ($i = 0; $i < $len; ++$i) {
Please login to merge, or discard this patch.
htdocs/include/notification_functions.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     if (!isset($module_id)) {
75 75
         global $xoopsModule;
76 76
         $module_id = !empty($xoopsModule) ? $xoopsModule->getVar('mid') : 0;
77
-        $module    =& $xoopsModule;
77
+        $module    = & $xoopsModule;
78 78
     } else {
79 79
         /* @var $module_handler XoopsModuleHandler */
80 80
         $module_handler = xoops_getHandler('module');
@@ -110,12 +110,12 @@  discard block
 block discarded – undo
110 110
 function &notificationCommentCategoryInfo($module_id = null)
111 111
 {
112 112
     $ret            = false;
113
-    $all_categories =& notificationCategoryInfo('', $module_id);
113
+    $all_categories = & notificationCategoryInfo('', $module_id);
114 114
     if (empty($all_categories)) {
115 115
         return $ret;
116 116
     }
117 117
     foreach ($all_categories as $category) {
118
-        $all_events =& notificationEvents($category['name'], false, $module_id);
118
+        $all_events = & notificationEvents($category['name'], false, $module_id);
119 119
         if (empty($all_events)) {
120 120
             continue;
121 121
         }
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
     if (!isset($module_id)) {
146 146
         global $xoopsModule;
147 147
         $module_id = !empty($xoopsModule) ? $xoopsModule->getVar('mid') : 0;
148
-        $module    =& $xoopsModule;
148
+        $module    = & $xoopsModule;
149 149
     } else {
150 150
         /* @var $module_handler XoopsModuleHandler */
151 151
         $module_handler = xoops_getHandler('module');
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
     $config_handler = xoops_getHandler('config');
157 157
     $mod_config     = $config_handler->getConfigsByCat(0, $module_id);
158 158
 
159
-    $category =& notificationCategoryInfo($category_name, $module_id);
159
+    $category = & notificationCategoryInfo($category_name, $module_id);
160 160
 
161 161
     global $xoopsConfig;
162 162
     $event_array = array();
@@ -167,8 +167,8 @@  discard block
 block discarded – undo
167 167
 
168 168
     foreach ($not_config['event'] as $event) {
169 169
         if ($event['category'] == $category_name) {
170
-            if (!is_dir($dir = XOOPS_ROOT_PATH . '/modules/' . $module->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/mail_template/')) {
171
-                $dir = XOOPS_ROOT_PATH . '/modules/' . $module->getVar('dirname') . '/language/english/mail_template/';
170
+            if (!is_dir($dir = XOOPS_ROOT_PATH.'/modules/'.$module->getVar('dirname').'/language/'.$xoopsConfig['language'].'/mail_template/')) {
171
+                $dir = XOOPS_ROOT_PATH.'/modules/'.$module->getVar('dirname').'/language/english/mail_template/';
172 172
             }
173 173
             $event['mail_template_dir'] = $dir;
174 174
             if (!$enabled_only || notificationEventEnabled($category, $event, $module)) {
@@ -192,8 +192,8 @@  discard block
 block discarded – undo
192 192
     if ($module->getVar('hascomments')) {
193 193
         $com_config = $module->getInfo('comments');
194 194
         if (!empty($category['item_name']) && $category['item_name'] == $com_config['itemName']) {
195
-            if (!is_dir($dir = XOOPS_ROOT_PATH . '/language/' . $xoopsConfig['language'] . '/mail_template/')) {
196
-                $dir = XOOPS_ROOT_PATH . '/language/english/mail_template/';
195
+            if (!is_dir($dir = XOOPS_ROOT_PATH.'/language/'.$xoopsConfig['language'].'/mail_template/')) {
196
+                $dir = XOOPS_ROOT_PATH.'/language/english/mail_template/';
197 197
             }
198 198
             $mail_template_dir = $dir;
199 199
 
@@ -319,7 +319,7 @@  discard block
 block discarded – undo
319 319
  */
320 320
 function &notificationEventInfo($category_name, $event_name, $module_id = null)
321 321
 {
322
-    $all_events =& notificationEvents($category_name, false, $module_id);
322
+    $all_events = & notificationEvents($category_name, false, $module_id);
323 323
     foreach ($all_events as $event) {
324 324
         if ($event['name'] == $event_name) {
325 325
             return $event;
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
 
341 341
 function &notificationSubscribableCategoryInfo($module_id = null)
342 342
 {
343
-    $all_categories =& notificationCategoryInfo('', $module_id);
343
+    $all_categories = & notificationCategoryInfo('', $module_id);
344 344
 
345 345
     // FIXME: better or more standardized way to do this?
346 346
     $script_url  = explode('/', $_SERVER['PHP_SELF']);
@@ -403,10 +403,10 @@  discard block
 block discarded – undo
403 403
     switch ($type) {
404 404
         case 'option_value':
405 405
         case 'name':
406
-            return 'notify:' . $category['name'] . '-' . $event['name'];
406
+            return 'notify:'.$category['name'].'-'.$event['name'];
407 407
             break;
408 408
         case 'option_name':
409
-            return $category['name'] . '-' . $event['name'];
409
+            return $category['name'].'-'.$event['name'];
410 410
             break;
411 411
         default:
412 412
             return false;
Please login to merge, or discard this patch.
htdocs/include/comment_edit.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
 include_once $GLOBALS['xoops']->path('include/comment_constants.php');
24 24
 
25 25
 if (('system' !== $xoopsModule->getVar('dirname') && XOOPS_COMMENT_APPROVENONE == $xoopsModuleConfig['com_rule']) || (!is_object($xoopsUser) && !$xoopsModuleConfig['com_anonpost']) || !is_object($xoopsModule)) {
26
-    redirect_header(XOOPS_URL . '/user.php', 1, _NOPERM);
26
+    redirect_header(XOOPS_URL.'/user.php', 1, _NOPERM);
27 27
 }
28 28
 
29 29
 xoops_loadLanguage('comment');
Please login to merge, or discard this patch.
htdocs/include/license.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
  */
22 22
 defined('XOOPS_ROOT_PATH') || exit('Restricted access');
23 23
 
24
-$licenseFile = XOOPS_VAR_PATH . '/data/license.php';
24
+$licenseFile = XOOPS_VAR_PATH.'/data/license.php';
25 25
 if (file_exists($licenseFile)) {
26 26
     include_once $licenseFile;
27 27
 } else {
Please login to merge, or discard this patch.
htdocs/include/xoops13_header.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -41,10 +41,10 @@
 block discarded – undo
41 41
 }
42 42
 if ($xoopsUser != '') {
43 43
     $xoopsTpl->assign(array(
44
-                          'xoops_isuser'  => true,
45
-                          'xoops_userid'  => $xoopsUser->getVar('uid'),
46
-                          'xoops_uname'   => $xoopsUser->getVar('uname'),
47
-                          'xoops_isadmin' => $xoopsUserIsAdmin));
44
+                            'xoops_isuser'  => true,
45
+                            'xoops_userid'  => $xoopsUser->getVar('uid'),
46
+                            'xoops_uname'   => $xoopsUser->getVar('uname'),
47
+                            'xoops_isadmin' => $xoopsUserIsAdmin));
48 48
 }
49 49
 $xoopsTpl->assign('xoops_requesturi', htmlspecialchars($GLOBALS['xoopsRequestUri'], ENT_QUOTES));
50 50
 include XOOPS_ROOT_PATH . '/include/old_functions.php';
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -19,10 +19,10 @@  discard block
 block discarded – undo
19 19
 
20 20
 $xoopsOption['theme_use_smarty'] = 0;
21 21
 
22
-if (file_exists(XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/lang-' . $xoopsConfig['language'] . '.php')) {
23
-    include XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/lang-' . $xoopsConfig['language'] . '.php';
24
-} elseif (file_exists(XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/lang-english.php')) {
25
-    include XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/lang-english.php';
22
+if (file_exists(XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/lang-'.$xoopsConfig['language'].'.php')) {
23
+    include XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/lang-'.$xoopsConfig['language'].'.php';
24
+} elseif (file_exists(XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/lang-english.php')) {
25
+    include XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/lang-english.php';
26 26
 }
27 27
 
28 28
 /* @var $config_handler XoopsConfigHandler  */
@@ -30,11 +30,11 @@  discard block
 block discarded – undo
30 30
 $xoopsConfigMetaFooter = $config_handler->getConfigsByCat(XOOPS_CONF_METAFOOTER);
31 31
 
32 32
 xoops_header(false);
33
-include XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/theme.php';
33
+include XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/theme.php';
34 34
 $xoopsOption['show_rblock'] = (!empty($xoopsOption['show_rblock'])) ? $xoopsOption['show_rblock'] : 0;
35 35
 
36 36
 // include Smarty template engine and initialize it
37
-require_once XOOPS_ROOT_PATH . '/class/template.php';
37
+require_once XOOPS_ROOT_PATH.'/class/template.php';
38 38
 $xoopsTpl = new XoopsTpl();
39 39
 if ($xoopsConfig['debug_mode'] == 3) {
40 40
     $xoopsTpl->debugging = true;
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
                           'xoops_isadmin' => $xoopsUserIsAdmin));
48 48
 }
49 49
 $xoopsTpl->assign('xoops_requesturi', htmlspecialchars($GLOBALS['xoopsRequestUri'], ENT_QUOTES));
50
-include XOOPS_ROOT_PATH . '/include/old_functions.php';
50
+include XOOPS_ROOT_PATH.'/include/old_functions.php';
51 51
 
52 52
 if ($xoopsOption['show_cblock'] || (!empty($xoopsModule) && preg_match("/index\.php$/i", xoops_getenv('PHP_SELF')) && $xoopsConfig['startpage'] == $xoopsModule->getVar('dirname'))) {
53 53
     $xoopsOption['show_rblock'] = $xoopsOption['show_cblock'] = 1;
Please login to merge, or discard this patch.