Completed
Pull Request — master (#592)
by
unknown
14:06
created
xoops_lib/Xmf/Module/Helper/Session.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
      */
40 40
     public function init()
41 41
     {
42
-        $this->prefix = $this->module->getVar('dirname') . '_';
42
+        $this->prefix = $this->module->getVar('dirname').'_';
43 43
     }
44 44
 
45 45
     /**
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
      */
52 52
     protected function prefix($name)
53 53
     {
54
-        $prefixedName = $this->prefix . $name;
54
+        $prefixedName = $this->prefix.$name;
55 55
 
56 56
         return $prefixedName;
57 57
     }
Please login to merge, or discard this patch.
xoops_lib/Xmf/Module/Helper/AbstractHelper.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
      */
91 91
     public function setDebug($bool = true)
92 92
     {
93
-        $this->debug = (bool) $bool;
93
+        $this->debug = (bool)$bool;
94 94
     }
95 95
 
96 96
     /**
@@ -127,6 +127,6 @@  discard block
 block discarded – undo
127 127
         if (!is_string($value)) {
128 128
             $value = json_encode($value);
129 129
         }
130
-        return (string) $value;
130
+        return (string)$value;
131 131
     }
132 132
 }
Please login to merge, or discard this patch.
xoops_lib/Xmf/Module/Helper/Permission.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
      **/
67 67
     public function checkPermission($gperm_name, $gperm_itemid, $trueifadmin = true)
68 68
     {
69
-        $gperm_itemid = (int) $gperm_itemid;
69
+        $gperm_itemid = (int)$gperm_itemid;
70 70
         $gperm_groupid = \Xoops::getInstance()->getUserGroups();
71 71
 
72 72
         return $this->permissionHandler->checkRight(
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
             $gperm_itemid,
75 75
             $gperm_groupid,
76 76
             $this->mid,
77
-            (bool) $trueifadmin
77
+            (bool)$trueifadmin
78 78
         );
79 79
     }
80 80
 
@@ -98,14 +98,14 @@  discard block
 block discarded – undo
98 98
         $message = '',
99 99
         $trueifadmin = true
100 100
     ) {
101
-        $gperm_itemid = (int) $gperm_itemid;
101
+        $gperm_itemid = (int)$gperm_itemid;
102 102
         $gperm_groupid = \Xoops::getInstance()->getUserGroups();
103 103
         $permission = $this->permissionHandler->checkRight(
104 104
             $gperm_name,
105 105
             $gperm_itemid,
106 106
             $gperm_groupid,
107 107
             $this->mid,
108
-            (bool) $trueifadmin
108
+            (bool)$trueifadmin
109 109
         );
110 110
         if (!$permission) {
111 111
             $helper = Helper::getHelper($this->dirname);
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
      **/
124 124
     public function getGroupsForItem($gperm_name, $gperm_itemid)
125 125
     {
126
-        $gperm_itemid = (int) $gperm_itemid;
126
+        $gperm_itemid = (int)$gperm_itemid;
127 127
         return $this->permissionHandler->getGroupIds($gperm_name, $gperm_itemid, $this->mid);
128 128
     }
129 129
 
@@ -138,9 +138,9 @@  discard block
 block discarded – undo
138 138
      **/
139 139
     public function savePermissionForItem($gperm_name, $gperm_itemid, $groups)
140 140
     {
141
-        $gperm_itemid = (int) $gperm_itemid;
141
+        $gperm_itemid = (int)$gperm_itemid;
142 142
         foreach ($groups as $index => $group) {
143
-            $groups[$index] = (int) $group;
143
+            $groups[$index] = (int)$group;
144 144
         }
145 145
 
146 146
         $result = true;
@@ -173,9 +173,9 @@  discard block
 block discarded – undo
173 173
      */
174 174
     public function deletePermissionForItem($gperm_name, $gperm_itemid)
175 175
     {
176
-        $gperm_itemid = (int) $gperm_itemid;
176
+        $gperm_itemid = (int)$gperm_itemid;
177 177
         if (!is_array($gperm_name)) {
178
-            $gperm_name = (array) $gperm_name;
178
+            $gperm_name = (array)$gperm_name;
179 179
         }
180 180
         $return = true;
181 181
         foreach ($gperm_name as $pname) {
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
         if (empty($name)) {
212 212
             $name = $this->defaultFieldName($gperm_name, $gperm_itemid);
213 213
         }
214
-        $gperm_itemid = (int) $gperm_itemid;
214
+        $gperm_itemid = (int)$gperm_itemid;
215 215
         $value = $this->getGroupsForItem($gperm_name, $gperm_itemid);
216 216
         $element = new SelectGroup(
217 217
             $caption,
@@ -236,9 +236,9 @@  discard block
 block discarded – undo
236 236
      */
237 237
     public function defaultFieldName($gperm_name, $gperm_itemid)
238 238
     {
239
-        $gperm_itemid = (int) $gperm_itemid;
240
-        $name = $this->module->getVar('dirname') . '_' .
241
-            $gperm_name . '_' . $gperm_itemid;
239
+        $gperm_itemid = (int)$gperm_itemid;
240
+        $name = $this->module->getVar('dirname').'_'.
241
+            $gperm_name.'_'.$gperm_itemid;
242 242
 
243 243
         return $name;
244 244
     }
Please login to merge, or discard this patch.
xoops_lib/Xmf/Module/Helper/GenericHelper.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
             return $default;
104 104
         }
105 105
 
106
-        $this->addLog("Getting config '{$name}' : " . $this->serializeForHelperLog($this->configs[$name]));
106
+        $this->addLog("Getting config '{$name}' : ".$this->serializeForHelperLog($this->configs[$name]));
107 107
 
108 108
         return $this->configs[$name];
109 109
     }
@@ -183,15 +183,15 @@  discard block
 block discarded – undo
183 183
      */
184 184
     protected function initHandler($name)
185 185
     {
186
-        $this->addLog('INIT ' . $name . ' HANDLER');
186
+        $this->addLog('INIT '.$name.' HANDLER');
187 187
 
188 188
         if (!isset($this->handlers[$name])) {
189
-            $hnd_file = XOOPS_ROOT_PATH . "/modules/{$this->dirname}/class/{$name}.php";
189
+            $hnd_file = XOOPS_ROOT_PATH."/modules/{$this->dirname}/class/{$name}.php";
190 190
             if (file_exists($hnd_file)) {
191 191
                 include_once $hnd_file;
192 192
             }
193 193
             $class = ucfirst(strtolower($this->dirname))
194
-                . ucfirst(strtolower($name)) . 'Handler';
194
+                . ucfirst(strtolower($name)).'Handler';
195 195
             if (class_exists($class)) {
196 196
                 $db = \XoopsDatabaseFactory::getDatabaseConnection();
197 197
                 $this->handlers[$name] = new $class($db);
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
      */
255 255
     public function url($url = '')
256 256
     {
257
-        return XOOPS_URL . '/modules/' . $this->dirname . '/' . $url;
257
+        return XOOPS_URL.'/modules/'.$this->dirname.'/'.$url;
258 258
     }
259 259
 
260 260
     /**
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
      */
267 267
     public function path($path = '')
268 268
     {
269
-        return XOOPS_ROOT_PATH . '/modules/' . $this->dirname . '/' . $path;
269
+        return XOOPS_ROOT_PATH.'/modules/'.$this->dirname.'/'.$path;
270 270
     }
271 271
 
272 272
     /**
Please login to merge, or discard this patch.
xoops_lib/Xmf/Module/Helper/Cache.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
      */
45 45
     public function init()
46 46
     {
47
-        $this->prefix = 'module/' . $this->module->getVar('dirname');
47
+        $this->prefix = 'module/'.$this->module->getVar('dirname');
48 48
         $this->cache = \Xoops::getInstance()->cache();
49 49
     }
50 50
 
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
      */
58 58
     protected function prefix($name)
59 59
     {
60
-        return $this->prefix . '/'. $name;
60
+        return $this->prefix.'/'.$name;
61 61
     }
62 62
 
63 63
     /**
Please login to merge, or discard this patch.
xoops_lib/Xmf/Module/Admin.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -276,10 +276,10 @@  discard block
 block discarded – undo
276 276
      */
277 277
     public function addConfigError($value = '')
278 278
     {
279
-        $path = XOOPS_URL . '/Frameworks/moduleclasses/icons/16/';
279
+        $path = XOOPS_URL.'/Frameworks/moduleclasses/icons/16/';
280 280
         $line = "";
281 281
         $line .= "<span style='color : red; font-weight : bold;'>";
282
-        $line .= "<img src='" . $path . "0.png' >";
282
+        $line .= "<img src='".$path."0.png' >";
283 283
         $line .= $value;
284 284
         $line .= "</span>";
285 285
         $value = $line;
@@ -297,10 +297,10 @@  discard block
 block discarded – undo
297 297
      */
298 298
     public function addConfigAccept($value = '')
299 299
     {
300
-        $path = XOOPS_URL . '/Frameworks/moduleclasses/icons/16/';
300
+        $path = XOOPS_URL.'/Frameworks/moduleclasses/icons/16/';
301 301
         $line = "";
302 302
         $line .= "<span style='color : green;'>";
303
-        $line .= "<img src='" . $path . "1.png' >";
303
+        $line .= "<img src='".$path."1.png' >";
304 304
         $line .= $value;
305 305
         $line .= "</span>";
306 306
         $value = $line;
@@ -318,10 +318,10 @@  discard block
 block discarded – undo
318 318
      */
319 319
     public function addConfigWarning($value = '')
320 320
     {
321
-        $path = XOOPS_URL . '/Frameworks/moduleclasses/icons/16/';
321
+        $path = XOOPS_URL.'/Frameworks/moduleclasses/icons/16/';
322 322
         $line = "";
323 323
         $line .= "<span style='color : orange; font-weight : bold;'>";
324
-        $line .= "<img src='" . $path . "warning.png' >";
324
+        $line .= "<img src='".$path."warning.png' >";
325 325
         $line .= $value;
326 326
         $line .= "</span>";
327 327
         $value = $line;
@@ -403,7 +403,7 @@  discard block
 block discarded – undo
403 403
         } else {
404 404
             $path = '../../Frameworks/moduleclasses/icons/32/';
405 405
 
406
-            return($path . $image);
406
+            return($path.$image);
407 407
         }
408 408
     }
409 409
 
@@ -439,12 +439,12 @@  discard block
 block discarded – undo
439 439
         }
440 440
 
441 441
         if (static::isXng()) {
442
-            $path = '/media/xoops/images/icons/' . $path;
442
+            $path = '/media/xoops/images/icons/'.$path;
443 443
         } else {
444
-            $path = '/Frameworks/moduleclasses/icons/' . $path;
444
+            $path = '/Frameworks/moduleclasses/icons/'.$path;
445 445
         }
446 446
 
447
-        return(XOOPS_URL . $path . $name);
447
+        return(XOOPS_URL.$path.$name);
448 448
     }
449 449
 
450 450
     /**
Please login to merge, or discard this patch.
xoops_lib/Xmf/Request.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
     public static function getString($name, $default = '', $hash = 'default', $mask = 0)
235 235
     {
236 236
         // Cast to string, in case static::MASK_ALLOW_RAW was specified for mask
237
-        return (string) static::getVar($name, $default, $hash, 'string', $mask);
237
+        return (string)static::getVar($name, $default, $hash, 'string', $mask);
238 238
     }
239 239
 
240 240
     /**
@@ -262,7 +262,7 @@  discard block
 block discarded – undo
262 262
      */
263 263
     public static function getText($name, $default = '', $hash = 'default')
264 264
     {
265
-        return (string) static::getVar($name, $default, $hash, 'string', static::MASK_ALLOW_RAW);
265
+        return (string)static::getVar($name, $default, $hash, 'string', static::MASK_ALLOW_RAW);
266 266
     }
267 267
 
268 268
     /**
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
276 276
      */
277 277
     public static function getUrl($name, $default = '', $hash = 'default')
278 278
     {
279
-        return (string) static::getVar($name, $default, $hash, 'weburl');
279
+        return (string)static::getVar($name, $default, $hash, 'weburl');
280 280
     }
281 281
 
282 282
     /**
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
      */
291 291
     public static function getPath($name, $default = '', $hash = 'default')
292 292
     {
293
-        return (string) static::getVar($name, $default, $hash, 'path');
293
+        return (string)static::getVar($name, $default, $hash, 'path');
294 294
     }
295 295
 
296 296
     /**
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
      */
305 305
     public static function getEmail($name, $default = '', $hash = 'default')
306 306
     {
307
-        $ret = (string) static::getVar($name, $default, $hash, 'email');
307
+        $ret = (string)static::getVar($name, $default, $hash, 'email');
308 308
         return empty($ret) ? $default : $ret;
309 309
     }
310 310
 
@@ -319,7 +319,7 @@  discard block
 block discarded – undo
319 319
      */
320 320
     public static function getIP($name, $default = '', $hash = 'default')
321 321
     {
322
-        $ret = (string) static::getVar($name, $default, $hash, 'ip');
322
+        $ret = (string)static::getVar($name, $default, $hash, 'ip');
323 323
         return empty($ret) ? $default : $ret;
324 324
     }
325 325
 
Please login to merge, or discard this patch.
xoops_lib/Xmf/Debug.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
      */
127 127
     public static function useEventDumper($value = true)
128 128
     {
129
-        self::$eventDumper = (bool) $value;
129
+        self::$eventDumper = (bool)$value;
130 130
     }
131 131
 
132 132
     /**
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
         if ($events->hasListeners($eventName)) {
175 175
             $events->triggerEvent($eventName, $name);
176 176
         } else {
177
-            echo $name . ' - ' . (int)(microtime(true) - self::$times[$name]) . " \n";
177
+            echo $name.' - '.(int)(microtime(true) - self::$times[$name])." \n";
178 178
         }
179 179
     }
180 180
 
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
             ini_set('xdebug.collect_params', $collect_params);
256 256
             ini_set('xdebug.collect_return', $collect_return);
257 257
             if ($tracefile == '') {
258
-                $tracefile = \XoopsBaseConfig::get('var-path') . '/logs/php_trace';
258
+                $tracefile = \XoopsBaseConfig::get('var-path').'/logs/php_trace';
259 259
             }
260 260
             xdebug_start_trace($tracefile);
261 261
         }
Please login to merge, or discard this patch.
xoops_lib/Xmf/Yaml.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -134,7 +134,7 @@
 block discarded – undo
134 134
     {
135 135
         try {
136 136
             $yamlString = VendorYaml::dump($var, $inline, $indent);
137
-            $ret = empty($yamlString) ? false : "<?php\n/*\n---\n" . $yamlString . "\n...\n*/\n";
137
+            $ret = empty($yamlString) ? false : "<?php\n/*\n---\n".$yamlString."\n...\n*/\n";
138 138
         } catch (\Exception $e) {
139 139
             static::logError($e);
140 140
             $ret = false;
Please login to merge, or discard this patch.