Passed
Pull Request — master (#610)
by Richard
14:27
created
htdocs/modules/smilies/class/form/smilies.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -41,13 +41,13 @@
 block discarded – undo
41 41
         $this->addElement(new Xoops\Form\Text(_AM_SMILIES_DESCRIPTION, 'smiley_emotion', 4, 50, $obj->getVar('smiley_emotion')), true);
42 42
 
43 43
         $imgtray_img = new Xoops\Form\ElementTray(_AM_SMILIES_FILE, '<br />');
44
-        $imgpath_img = sprintf(_AM_SMILIES_IMAGE_PATH, $xoops->path('uploads/smilies') . '/');
44
+        $imgpath_img = sprintf(_AM_SMILIES_IMAGE_PATH, $xoops->path('uploads/smilies').'/');
45 45
         $imageselect_img = new Xoops\Form\Select($imgpath_img, 'smiley_url', $blank_img);
46 46
         \Xoops\Core\Lists\ImageFile::setOptionsArray($imageselect_img, $xoops->path('uploads/smilies'));
47 47
 
48
-        $imageselect_img->setExtra('onchange="showImgSelected(\'xo-smilies-img\', \'smiley_url\', \'smilies\', \'\', \'' . \XoopsBaseConfig::get('uploads-url') . '\' )"');
48
+        $imageselect_img->setExtra('onchange="showImgSelected(\'xo-smilies-img\', \'smiley_url\', \'smilies\', \'\', \''.\XoopsBaseConfig::get('uploads-url').'\' )"');
49 49
         $imgtray_img->addElement($imageselect_img, false);
50
-        $imgtray_img->addElement(new Xoops\Form\Label('', "<br /><img src='" . \XoopsBaseConfig::get('uploads-url') . "/smilies/" . $blank_img . "' name='image_img' id='xo-smilies-img' alt=''>"));
50
+        $imgtray_img->addElement(new Xoops\Form\Label('', "<br /><img src='".\XoopsBaseConfig::get('uploads-url')."/smilies/".$blank_img."' name='image_img' id='xo-smilies-img' alt=''>"));
51 51
 
52 52
         $fileseltray_img = new Xoops\Form\ElementTray('<br />', '<br /><br />');
53 53
         $fileseltray_img->addElement(new Xoops\Form\File(_AM_SMILIES_UPLOADS, 'smiley_url'), false);
Please login to merge, or discard this patch.
htdocs/modules/page/viewpage.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -48,11 +48,11 @@  discard block
 block discarded – undo
48 48
 
49 49
 // hits
50 50
 if ($view_content->getVar('content_author') != $uid && $view_content->getVar('content_dohits') != false) {
51
-    if (!isset( $_SESSION['page_hits' . $content_id] ) || isset( $_SESSION['page_hits' . $content_id] ) && ($_SESSION['page_hits' . $content_id]['content_time'] + $interval) <  time()) {
51
+    if (!isset($_SESSION['page_hits'.$content_id]) || isset($_SESSION['page_hits'.$content_id]) && ($_SESSION['page_hits'.$content_id]['content_time'] + $interval) < time()) {
52 52
         $hits = $view_content->getVar('content_hits') + 1;
53 53
         $view_content->setVar('content_hits', $hits);
54 54
         $content_Handler->insert($view_content);
55
-        $_SESSION['page_hits' . $content_id]['content_time'] = time();
55
+        $_SESSION['page_hits'.$content_id]['content_time'] = time();
56 56
     }
57 57
 }
58 58
 
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 // get token for rating
71 71
 $xoops->tpl()->assign('security', $xoops->security()->createToken());
72 72
 // Meta
73
-$xoops->tpl()->assign('xoops_pagetitle', strip_tags($view_content->getVar('content_title')  . ' - ' . $xoops->module->name()));
73
+$xoops->tpl()->assign('xoops_pagetitle', strip_tags($view_content->getVar('content_title').' - '.$xoops->module->name()));
74 74
 $xoops->theme()->addMeta('meta', 'description', strip_tags($view_content->getVar('content_mdescription')));
75 75
 $xoops->theme()->addMeta('meta', 'keywords', strip_tags($view_content->getVar('content_mkeyword')));
76 76
 $xoops->footer();
Please login to merge, or discard this patch.
htdocs/modules/page/admin/content.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
  * @author          Mage Grégory (AKA Mage)
22 22
  * @version         $Id$
23 23
  */
24
-include __DIR__ . '/header.php';
24
+include __DIR__.'/header.php';
25 25
 
26 26
 // Call header
27 27
 $xoops->header('admin:page/page_admin_content.tpl');
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
 
131 131
         if (preg_match('/^\d+$/', Request::getInt('content_weight', 0)) == false) {
132 132
             $error = true;
133
-            $error_message .= PageLocale::E_WEIGHT . '<br />';
133
+            $error_message .= PageLocale::E_WEIGHT.'<br />';
134 134
             $obj->setVar('content_weight', 0);
135 135
         } else {
136 136
             $obj->setVar('content_weight', Request::getInt('content_weight', 0));
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
                     $tags = array();
151 151
                     $tags['MODULE_NAME'] = 'page';
152 152
                     $tags['ITEM_NAME'] = Request::getString('content_title', '');
153
-                    $tags['ITEM_URL'] = \XoopsBaseConfig::get('url') . '/modules/page/viewpage.php?id=' . $newcontent_id;
153
+                    $tags['ITEM_URL'] = \XoopsBaseConfig::get('url').'/modules/page/viewpage.php?id='.$newcontent_id;
154 154
                     $notification_handler->triggerEvent('global', 0, 'newcontent', $tags);
155 155
                     $notification_handler->triggerEvent('item', $newcontent_id, 'newcontent', $tags);
156 156
                 }
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
                 array('ok' => 1, 'content_id' => $content_id, 'op' => 'delete'),
201 201
                 'content.php',
202 202
                 XoopsLocale::Q_ARE_YOU_SURE_YOU_WANT_TO_DELETE_THIS_ITEM
203
-                . '<br /><span class="red">' . $obj->getvar('content_title') . '<span>'
203
+                . '<br /><span class="red">'.$obj->getvar('content_title').'<span>'
204 204
             );
205 205
         }
206 206
         break;
Please login to merge, or discard this patch.
htdocs/modules/page/admin/related.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
  * @author          Mage Grégory (AKA Mage)
22 22
  * @version         $Id$
23 23
  */
24
-include __DIR__ . '/header.php';
24
+include __DIR__.'/header.php';
25 25
 
26 26
 // Call header
27 27
 $xoops->header('admin:page/page_admin_related.tpl');
@@ -106,8 +106,8 @@  discard block
 block discarded – undo
106 106
             if (count($datas_delete) != 0) {
107 107
                 $criteria = $criteria = new CriteriaCompo();
108 108
                 $criteria->add(new Criteria('link_related_id', $related_id));
109
-                $criteria->add(new Criteria('link_content_id', '(' . implode(', ', $datas_delete) . ')', 'IN'));
110
-                $links_ids =  $link_Handler->getIds($criteria);
109
+                $criteria->add(new Criteria('link_content_id', '('.implode(', ', $datas_delete).')', 'IN'));
110
+                $links_ids = $link_Handler->getIds($criteria);
111 111
                 if (!$link_Handler->DeleteByIds($links_ids)) {
112 112
                 }
113 113
             }
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
                 array('ok' => 1, 'related_id' => $related_id, 'op' => 'delete'),
173 173
                 'related.php',
174 174
                 XoopsLocale::Q_ARE_YOU_SURE_YOU_WANT_TO_DELETE_THIS_ITEM
175
-                . '<br /><span class="red">' . $obj->getvar('related_name') . '<span>'
175
+                . '<br /><span class="red">'.$obj->getvar('related_name').'<span>'
176 176
             );
177 177
         }
178 178
         break;
Please login to merge, or discard this patch.
htdocs/modules/page/admin/menu.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@
 block discarded – undo
34 34
 $adminmenu[$i]['link'] = 'admin/related.php';
35 35
 $adminmenu[$i]['icon'] = 'groupmod.png';
36 36
 ++$i;
37
-$adminmenu[$i]['title'] =PageLocale::SYSTEM_PERMISSIONS;
37
+$adminmenu[$i]['title'] = PageLocale::SYSTEM_PERMISSIONS;
38 38
 $adminmenu[$i]['link'] = 'admin/permissions.php';
39 39
 $adminmenu[$i]['icon'] = 'permissions.png';
40 40
 ++$i;
Please login to merge, or discard this patch.
htdocs/modules/page/admin/permissions.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
  * @version         $Id$
23 23
  */
24 24
 
25
-include __DIR__ . '/header.php';
25
+include __DIR__.'/header.php';
26 26
 
27 27
 // Get Action type
28 28
 $op = Request::getString('op', 'global');
@@ -71,13 +71,13 @@  discard block
 block discarded – undo
71 71
                 $groups_ids_view = $gperm_Handler->getGroupIds('page_view_item', $content_id, $module_id);
72 72
                 $groups_ids_view = array_values($groups_ids_view);
73 73
                 foreach (array_keys($group_list) as $j) {
74
-                    $perms .= '<img id="loading_display' . $content_id . '_' . $j .'" src="' . $xoops->url('media/xoops/images/spinner.gif') . '" style="display:none;" alt="' . XoopsLocale::LOADING . '" />';
74
+                    $perms .= '<img id="loading_display'.$content_id.'_'.$j.'" src="'.$xoops->url('media/xoops/images/spinner.gif').'" style="display:none;" alt="'.XoopsLocale::LOADING.'" />';
75 75
                     if (in_array($j, $groups_ids_view)) {
76
-                        $perms .= "<img class=\"cursorpointer\" id=\"display" . $content_id . "_" . $j . "\" onclick=\"Xoops.changeStatus( 'permissions.php', { op: 'update_view', content_id: " . $content_id . ", group: " . $j . ", status: 'no' }, 'display" . $content_id . "_" . $j ."', 'permissions.php' )\" src=\"" . $xoops->url('modules/system/images/icons/default/success.png') . "\" alt=\"" . XoopsLocale::A_DISABLE . "\" title=\"" . XoopsLocale::A_DISABLE . "\" />";
76
+                        $perms .= "<img class=\"cursorpointer\" id=\"display".$content_id."_".$j."\" onclick=\"Xoops.changeStatus( 'permissions.php', { op: 'update_view', content_id: ".$content_id.", group: ".$j.", status: 'no' }, 'display".$content_id."_".$j."', 'permissions.php' )\" src=\"".$xoops->url('modules/system/images/icons/default/success.png')."\" alt=\"".XoopsLocale::A_DISABLE."\" title=\"".XoopsLocale::A_DISABLE."\" />";
77 77
                     } else {
78
-                        $perms .= "<img class=\"cursorpointer\" id=\"display" . $content_id . "_" . $j . "\" onclick=\"Xoops.changeStatus( 'permissions.php', { op: 'update_view', content_id: " . $content_id . ", group: " . $j . ", status: 'yes' }, 'display" . $content_id . "_" . $j ."', 'permissions.php' )\" src=\"" . $xoops->url('modules/system/images/icons/default/cancel.png') . "\" alt=\"" . XoopsLocale::A_ENABLE . "\" title=\"" . XoopsLocale::A_ENABLE . "\" />";
78
+                        $perms .= "<img class=\"cursorpointer\" id=\"display".$content_id."_".$j."\" onclick=\"Xoops.changeStatus( 'permissions.php', { op: 'update_view', content_id: ".$content_id.", group: ".$j.", status: 'yes' }, 'display".$content_id."_".$j."', 'permissions.php' )\" src=\"".$xoops->url('modules/system/images/icons/default/cancel.png')."\" alt=\"".XoopsLocale::A_ENABLE."\" title=\"".XoopsLocale::A_ENABLE."\" />";
79 79
                     }
80
-                    $perms .= $group_list[$j] . '<br />';
80
+                    $perms .= $group_list[$j].'<br />';
81 81
                 }
82 82
                 $content['id'] = $content_id;
83 83
                 $content['title'] = $content_arr[$i]->getVar('content_title');
Please login to merge, or discard this patch.
htdocs/modules/page/admin/index.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  * @author          Mage Grégory (AKA Mage)
20 20
  * @version         $Id$
21 21
  */
22
-include __DIR__ . '/header.php';
22
+include __DIR__.'/header.php';
23 23
 
24 24
 // heaser
25 25
 $xoops->header();
Please login to merge, or discard this patch.
htdocs/modules/page/admin/header.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
  * @author          Mage Grégory (AKA Mage)
22 22
  * @version         $Id$
23 23
  */
24
-require_once dirname(dirname(dirname(__DIR__))) . '/include/cp_header.php';
24
+require_once dirname(dirname(dirname(__DIR__))).'/include/cp_header.php';
25 25
 
26 26
 // Get main instance
27 27
 XoopsLoad::load('system', 'system');
Please login to merge, or discard this patch.
htdocs/modules/page/pdf.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
 
58 58
 $tpl->assign('xoops_sitename', $xoops->getConfig('sitename'));
59 59
 // Meta
60
-$tpl->assign('xoops_pagetitle', strip_tags($view_content->getVar('content_title') . ' - ' . XoopsLocale::A_PRINT . ' - ' . $xoopsModule->name()));
60
+$tpl->assign('xoops_pagetitle', strip_tags($view_content->getVar('content_title').' - '.XoopsLocale::A_PRINT.' - '.$xoopsModule->name()));
61 61
 
62 62
 $content = $tpl->fetch('module:page/page_pdf.tpl');
63 63
 
Please login to merge, or discard this patch.