Test Failed
Pull Request — master (#592)
by Lucio
10:17
created
htdocs/class/xml/rpc/movabletypeapi.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
  */
19 19
 
20 20
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
21
-require_once XOOPS_ROOT_PATH . '/class/xml/rpc/xmlrpcapi.php';
21
+require_once XOOPS_ROOT_PATH.'/class/xml/rpc/xmlrpcapi.php';
22 22
 
23 23
 /**
24 24
  * Class MovableTypeApi
@@ -40,9 +40,9 @@  discard block
 block discarded – undo
40 40
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
41 41
             $this->response->add(new XoopsXmlRpcFault(104));
42 42
         } else {
43
-            $xoopsapi =& $this->_getXoopsApi($this->params);
43
+            $xoopsapi = & $this->_getXoopsApi($this->params);
44 44
             $xoopsapi->_setUser($this->user, $this->isadmin);
45
-            $ret =& $xoopsapi->getCategories(false);
45
+            $ret = & $xoopsapi->getCategories(false);
46 46
             if (is_array($ret)) {
47 47
                 $arr = new XoopsXmlRpcArray();
48 48
                 foreach ($ret as $id => $name) {
Please login to merge, or discard this patch.
htdocs/class/xml/rpc/xoopsapi.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
  */
19 19
 
20 20
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
21
-require_once XOOPS_ROOT_PATH . '/class/xml/rpc/xmlrpcapi.php';
21
+require_once XOOPS_ROOT_PATH.'/class/xml/rpc/xmlrpcapi.php';
22 22
 
23 23
 /**
24 24
  * Class XoopsApi
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
41 41
             $this->response->add(new XoopsXmlRpcFault(104));
42 42
         } else {
43
-            if (!$fields =& $this->_getPostFields(null, $this->params[0])) {
43
+            if (!$fields = & $this->_getPostFields(null, $this->params[0])) {
44 44
                 $this->response->add(new XoopsXmlRpcFault(106));
45 45
             } else {
46 46
                 $missing = array();
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
                                 $missing[] = $tag;
53 53
                             }
54 54
                         } else {
55
-                            $post[$tag] =& $data;
55
+                            $post[$tag] = & $data;
56 56
                         }
57 57
                     } else {
58 58
                         $post[$tag] = $this->params[3][$tag];
@@ -61,12 +61,12 @@  discard block
 block discarded – undo
61 61
                 if (count($missing) > 0) {
62 62
                     $msg = '';
63 63
                     foreach ($missing as $m) {
64
-                        $msg .= '<' . $m . '> ';
64
+                        $msg .= '<'.$m.'> ';
65 65
                     }
66 66
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
67 67
                 } else {
68 68
                     // will be removed... don't worry if this looks bad
69
-                    include_once XOOPS_ROOT_PATH . '/modules/news/class/class.newsstory.php';
69
+                    include_once XOOPS_ROOT_PATH.'/modules/news/class/class.newsstory.php';
70 70
                     $story = new NewsStory();
71 71
                     $error = false;
72 72
                     if ((int)$this->params[4] > 0) {
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
129 129
             $this->response->add(new XoopsXmlRpcFault(104));
130 130
         } else {
131
-            if (!$fields =& $this->_getPostFields($this->params[0])) {
131
+            if (!$fields = & $this->_getPostFields($this->params[0])) {
132 132
             } else {
133 133
                 $missing = array();
134 134
                 foreach ($fields as $tag => $detail) {
@@ -148,12 +148,12 @@  discard block
 block discarded – undo
148 148
                 if (count($missing) > 0) {
149 149
                     $msg = '';
150 150
                     foreach ($missing as $m) {
151
-                        $msg .= '<' . $m . '> ';
151
+                        $msg .= '<'.$m.'> ';
152 152
                     }
153 153
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
154 154
                 } else {
155 155
                     // will be removed... don't worry if this looks bad
156
-                    include_once XOOPS_ROOT_PATH . '/modules/news/class/class.newsstory.php';
156
+                    include_once XOOPS_ROOT_PATH.'/modules/news/class/class.newsstory.php';
157 157
                     $story   = new NewsStory($this->params[0]);
158 158
                     $storyid = $story->storyid();
159 159
                     if (empty($storyid)) {
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
                 $this->response->add(new XoopsXmlRpcFault(111));
196 196
             } else {
197 197
                 // will be removed... don't worry if this looks bad
198
-                include_once XOOPS_ROOT_PATH . '/modules/news/class/class.newsstory.php';
198
+                include_once XOOPS_ROOT_PATH.'/modules/news/class/class.newsstory.php';
199 199
                 $story = new NewsStory($this->params[0]);
200 200
                 if (!$story->delete()) {
201 201
                     $this->response->add(new XoopsXmlRpcFault(106));
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
             $this->response->add(new XoopsXmlRpcFault(104));
219 219
         } else {
220 220
             // will be removed... don't worry if this looks bad
221
-            include_once XOOPS_ROOT_PATH . '/modules/news/class/class.newsstory.php';
221
+            include_once XOOPS_ROOT_PATH.'/modules/news/class/class.newsstory.php';
222 222
             $story = new NewsStory($this->params[0]);
223 223
             $ret   = array(
224 224
                 'uid'       => $story->uid(),
@@ -246,14 +246,14 @@  discard block
 block discarded – undo
246 246
                                 break;
247 247
                             case 'storyid':
248 248
                                 $struct->add('postid', new XoopsXmlRpcString($value));
249
-                                $struct->add('link', new XoopsXmlRpcString(XOOPS_URL . '/modules/news/article.php?item_id=' . $value));
250
-                                $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL . '/modules/news/article.php?item_id=' . $value));
249
+                                $struct->add('link', new XoopsXmlRpcString(XOOPS_URL.'/modules/news/article.php?item_id='.$value));
250
+                                $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL.'/modules/news/article.php?item_id='.$value));
251 251
                                 break;
252 252
                             case 'title':
253 253
                                 $struct->add('title', new XoopsXmlRpcString($value));
254 254
                                 break;
255 255
                             default :
256
-                                $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
256
+                                $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
257 257
                                 break;
258 258
                         }
259 259
                     }
@@ -276,11 +276,11 @@  discard block
 block discarded – undo
276 276
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
277 277
             $this->response->add(new XoopsXmlRpcFault(104));
278 278
         } else {
279
-            include_once XOOPS_ROOT_PATH . '/modules/news/class/class.newsstory.php';
279
+            include_once XOOPS_ROOT_PATH.'/modules/news/class/class.newsstory.php';
280 280
             if (isset($this->params[4]) && (int)$this->params[4] > 0) {
281
-                $stories =& NewsStory::getAllPublished((int)$this->params[3], 0, $this->params[4]);
281
+                $stories = & NewsStory::getAllPublished((int)$this->params[3], 0, $this->params[4]);
282 282
             } else {
283
-                $stories =& NewsStory::getAllPublished((int)$this->params[3]);
283
+                $stories = & NewsStory::getAllPublished((int)$this->params[3]);
284 284
             }
285 285
             $scount = count($stories);
286 286
             $ret    = array();
@@ -315,14 +315,14 @@  discard block
 block discarded – undo
315 315
                                     break;
316 316
                                 case 'storyid':
317 317
                                     $struct->add('postid', new XoopsXmlRpcString($value));
318
-                                    $struct->add('link', new XoopsXmlRpcString(XOOPS_URL . '/modules/news/article.php?item_id=' . $value));
319
-                                    $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL . '/modules/news/article.php?item_id=' . $value));
318
+                                    $struct->add('link', new XoopsXmlRpcString(XOOPS_URL.'/modules/news/article.php?item_id='.$value));
319
+                                    $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL.'/modules/news/article.php?item_id='.$value));
320 320
                                     break;
321 321
                                 case 'title':
322 322
                                     $struct->add('title', new XoopsXmlRpcString($value));
323 323
                                     break;
324 324
                                 default :
325
-                                    $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
325
+                                    $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
326 326
                                     break;
327 327
                             }
328 328
                         }
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
349 349
             $this->response->add(new XoopsXmlRpcFault(104));
350 350
         } else {
351
-            include_once XOOPS_ROOT_PATH . '/class/xoopstopic.php';
351
+            include_once XOOPS_ROOT_PATH.'/class/xoopstopic.php';
352 352
             $this->db = XoopsDatabaseFactory::getDatabaseConnection();
353 353
             $xt       = new XoopsTopic($db->prefix('topics'));
354 354
             $ret      = $xt->getTopicsList();
Please login to merge, or discard this patch.
htdocs/class/xml/rpc/xmlrpctag.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -123,10 +123,10 @@
 block discarded – undo
123 123
             "#||\\1||#",
124 124
             '&amp;',
125 125
             "&\\1;"), str_replace(array(
126
-                                      '<',
127
-                                      '>'), array(
128
-                                      '&lt;',
129
-                                      '&gt;'), $text));
126
+                                        '<',
127
+                                        '>'), array(
128
+                                        '&lt;',
129
+                                        '&gt;'), $text));
130 130
 
131 131
         return $text;
132 132
     }
Please login to merge, or discard this patch.
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
      */
37 37
     public function add(&$tagobj)
38 38
     {
39
-        $this->_tags[] =& $tagobj;
39
+        $this->_tags[] = & $tagobj;
40 40
     }
41 41
 
42 42
     public function render()
@@ -60,11 +60,11 @@  discard block
 block discarded – undo
60 60
             if (!$this->_tags[$i]->isFault()) {
61 61
                 $payload .= $this->_tags[$i]->render();
62 62
             } else {
63
-                return '<?xml version="1.0"?><methodResponse>' . $this->_tags[$i]->render() . '</methodResponse>';
63
+                return '<?xml version="1.0"?><methodResponse>'.$this->_tags[$i]->render().'</methodResponse>';
64 64
             }
65 65
         }
66 66
 
67
-        return '<?xml version="1.0"?><methodResponse><params><param>' . $payload . '</param></params></methodResponse>';
67
+        return '<?xml version="1.0"?><methodResponse><params><param>'.$payload.'</param></params></methodResponse>';
68 68
     }
69 69
 }
70 70
 
@@ -91,10 +91,10 @@  discard block
 block discarded – undo
91 91
         $count   = count($this->_tags);
92 92
         $payload = '';
93 93
         for ($i = 0; $i < $count; ++$i) {
94
-            $payload .= '<param>' . $this->_tags[$i]->render() . '</param>';
94
+            $payload .= '<param>'.$this->_tags[$i]->render().'</param>';
95 95
         }
96 96
 
97
-        return '<?xml version="1.0"?><methodCall><methodName>' . $this->methodName . '</methodName><params>' . $payload . '</params></methodCall>';
97
+        return '<?xml version="1.0"?><methodCall><methodName>'.$this->methodName.'</methodName><params>'.$payload.'</params></methodCall>';
98 98
     }
99 99
 }
100 100
 
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
      */
137 137
     public function setFault($fault = true)
138 138
     {
139
-        $this->_fault = ((int)$fault > 0);// ? true : false;
139
+        $this->_fault = ((int)$fault > 0); // ? true : false;
140 140
     }
141 141
 
142 142
     /**
@@ -214,9 +214,9 @@  discard block
 block discarded – undo
214 214
                 $string = 'Method response error';
215 215
                 break;
216 216
         }
217
-        $string .= "\n" . $this->_extra;
217
+        $string .= "\n".$this->_extra;
218 218
 
219
-        return '<fault><value><struct><member><name>faultCode</name><value>' . $this->_code . '</value></member><member><name>faultString</name><value>' . $this->encode($string) . '</value></member></struct></value></fault>';
219
+        return '<fault><value><struct><member><name>faultCode</name><value>'.$this->_code.'</value></member><member><name>faultString</name><value>'.$this->encode($string).'</value></member></struct></value></fault>';
220 220
     }
221 221
 }
222 222
 
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
      */
241 241
     public function render()
242 242
     {
243
-        return '<value><int>' . $this->_value . '</int></value>';
243
+        return '<value><int>'.$this->_value.'</int></value>';
244 244
     }
245 245
 }
246 246
 
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
      */
265 265
     public function render()
266 266
     {
267
-        return '<value><double>' . $this->_value . '</double></value>';
267
+        return '<value><double>'.$this->_value.'</double></value>';
268 268
     }
269 269
 }
270 270
 
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
      */
289 289
     public function render()
290 290
     {
291
-        return '<value><boolean>' . $this->_value . '</boolean></value>';
291
+        return '<value><boolean>'.$this->_value.'</boolean></value>';
292 292
     }
293 293
 }
294 294
 
@@ -312,7 +312,7 @@  discard block
 block discarded – undo
312 312
      */
313 313
     public function render()
314 314
     {
315
-        return '<value><string>' . $this->encode($this->_value) . '</string></value>';
315
+        return '<value><string>'.$this->encode($this->_value).'</string></value>';
316 316
     }
317 317
 }
318 318
 
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
      */
341 341
     public function render()
342 342
     {
343
-        return '<value><dateTime.iso8601>' . gmstrftime('%Y%m%dT%H:%M:%S', $this->_value) . '</dateTime.iso8601></value>';
343
+        return '<value><dateTime.iso8601>'.gmstrftime('%Y%m%dT%H:%M:%S', $this->_value).'</dateTime.iso8601></value>';
344 344
     }
345 345
 }
346 346
 
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
      */
365 365
     public function render()
366 366
     {
367
-        return '<value><base64>' . $this->_value . '</base64></value>';
367
+        return '<value><base64>'.$this->_value.'</base64></value>';
368 368
     }
369 369
 }
370 370
 
@@ -387,7 +387,7 @@  discard block
 block discarded – undo
387 387
      */
388 388
     public function add(&$tagobj)
389 389
     {
390
-        $this->_tags[] =& $tagobj;
390
+        $this->_tags[] = & $tagobj;
391 391
     }
392 392
 
393 393
     /**
@@ -437,7 +437,7 @@  discard block
 block discarded – undo
437 437
         $count = count($this->_tags);
438 438
         $ret   = '<value><struct>';
439 439
         for ($i = 0; $i < $count; ++$i) {
440
-            $ret .= '<member><name>' . $this->encode($this->_tags[$i]['name']) . '</name>' . $this->_tags[$i]['value']->render() . '</member>';
440
+            $ret .= '<member><name>'.$this->encode($this->_tags[$i]['name']).'</name>'.$this->_tags[$i]['value']->render().'</member>';
441 441
         }
442 442
         $ret .= '</struct></value>';
443 443
 
Please login to merge, or discard this patch.
htdocs/class/xml/rpc/xmlrpcparser.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,8 +18,8 @@  discard block
 block discarded – undo
18 18
  */
19 19
 
20 20
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
21
-require_once XOOPS_ROOT_PATH . '/class/xml/saxparser.php';
22
-require_once XOOPS_ROOT_PATH . '/class/xml/xmltaghandler.php';
21
+require_once XOOPS_ROOT_PATH.'/class/xml/saxparser.php';
22
+require_once XOOPS_ROOT_PATH.'/class/xml/xmltaghandler.php';
23 23
 
24 24
 /**
25 25
  * Class RSS Parser
@@ -565,7 +565,7 @@  discard block
 block discarded – undo
565 565
      */
566 566
     public function handleEndElement($parser)
567 567
     {
568
-        $member =& $parser->getTempMember();
568
+        $member = & $parser->getTempMember();
569 569
         $parser->releaseWorkingLevel();
570 570
         $parser->setTempStruct($member);
571 571
     }
Please login to merge, or discard this patch.
htdocs/class/xml/rpc/metaweblogapi.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
  */
19 19
 
20 20
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
21
-require_once XOOPS_ROOT_PATH . '/class/xml/rpc/xmlrpcapi.php';
21
+require_once XOOPS_ROOT_PATH.'/class/xml/rpc/xmlrpcapi.php';
22 22
 
23 23
 /**
24 24
  * Class MetaWeblogApi
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
45 45
             $this->response->add(new XoopsXmlRpcFault(104));
46 46
         } else {
47
-            if (!$fields =& $this->_getPostFields(null, $this->params[0])) {
47
+            if (!$fields = & $this->_getPostFields(null, $this->params[0])) {
48 48
                 $this->response->add(new XoopsXmlRpcFault(106));
49 49
             } else {
50 50
                 $missing = array();
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
                 if (count($missing) > 0) {
68 68
                     $msg = '';
69 69
                     foreach ($missing as $m) {
70
-                        $msg .= '<' . $m . '> ';
70
+                        $msg .= '<'.$m.'> ';
71 71
                         echo $m;
72 72
                     }
73 73
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
                     $newparams[1] = $this->params[1];
78 78
                     $newparams[2] = $this->params[2];
79 79
                     foreach ($post as $key => $value) {
80
-                        $newparams[3][$key] =& $value;
80
+                        $newparams[3][$key] = & $value;
81 81
                         unset($value);
82 82
                     }
83 83
                     $newparams[3]['xoops_text'] = $this->params[3]['description'];
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
                         }
88 88
                     }
89 89
                     $newparams[4] = $this->params[4];
90
-                    $xoopsapi     =& $this->_getXoopsApi($newparams);
90
+                    $xoopsapi     = & $this->_getXoopsApi($newparams);
91 91
                     $xoopsapi->_setUser($this->user, $this->isadmin);
92 92
                     $xoopsapi->newPost();
93 93
                 }
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
101 101
             $this->response->add(new XoopsXmlRpcFault(104));
102 102
         } else {
103
-            if (!$fields =& $this->_getPostFields($this->params[0])) {
103
+            if (!$fields = & $this->_getPostFields($this->params[0])) {
104 104
             } else {
105 105
                 $missing = array();
106 106
                 $post    = array();
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
                 if (count($missing) > 0) {
123 123
                     $msg = '';
124 124
                     foreach ($missing as $m) {
125
-                        $msg .= '<' . $m . '> ';
125
+                        $msg .= '<'.$m.'> ';
126 126
                     }
127 127
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
128 128
                 } else {
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
                     $newparams[1] = $this->params[1];
132 132
                     $newparams[2] = $this->params[2];
133 133
                     foreach ($post as $key => $value) {
134
-                        $newparams[3][$key] =& $value;
134
+                        $newparams[3][$key] = & $value;
135 135
                         unset($value);
136 136
                     }
137 137
                     if (isset($this->params[3]['categories']) && is_array($this->params[3]['categories'])) {
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
                     }
142 142
                     $newparams[3]['xoops_text'] = $this->params[3]['description'];
143 143
                     $newparams[4]               = $this->params[4];
144
-                    $xoopsapi                   =& $this->_getXoopsApi($newparams);
144
+                    $xoopsapi                   = & $this->_getXoopsApi($newparams);
145 145
                     $xoopsapi->_setUser($this->user, $this->isadmin);
146 146
                     $xoopsapi->editPost();
147 147
                 }
@@ -154,9 +154,9 @@  discard block
 block discarded – undo
154 154
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
155 155
             $this->response->add(new XoopsXmlRpcFault(104));
156 156
         } else {
157
-            $xoopsapi =& $this->_getXoopsApi($this->params);
157
+            $xoopsapi = & $this->_getXoopsApi($this->params);
158 158
             $xoopsapi->_setUser($this->user, $this->isadmin);
159
-            $ret =& $xoopsapi->getPost(false);
159
+            $ret = & $xoopsapi->getPost(false);
160 160
             if (is_array($ret)) {
161 161
                 $struct  = new XoopsXmlRpcStruct();
162 162
                 $content = '';
@@ -171,14 +171,14 @@  discard block
 block discarded – undo
171 171
                             break;
172 172
                         case 'postid':
173 173
                             $struct->add('postid', new XoopsXmlRpcString($value));
174
-                            $struct->add('link', new XoopsXmlRpcString(XOOPS_URL . '/modules/xoopssections/item.php?item=' . $value));
175
-                            $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL . '/modules/xoopssections/item.php?item=' . $value));
174
+                            $struct->add('link', new XoopsXmlRpcString(XOOPS_URL.'/modules/xoopssections/item.php?item='.$value));
175
+                            $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL.'/modules/xoopssections/item.php?item='.$value));
176 176
                             break;
177 177
                         case 'title':
178 178
                             $struct->add('title', new XoopsXmlRpcString($value));
179 179
                             break;
180 180
                         default :
181
-                            $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
181
+                            $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
182 182
                             break;
183 183
                     }
184 184
                 }
@@ -195,9 +195,9 @@  discard block
 block discarded – undo
195 195
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
196 196
             $this->response->add(new XoopsXmlRpcFault(104));
197 197
         } else {
198
-            $xoopsapi =& $this->_getXoopsApi($this->params);
198
+            $xoopsapi = & $this->_getXoopsApi($this->params);
199 199
             $xoopsapi->_setUser($this->user, $this->isadmin);
200
-            $ret =& $xoopsapi->getRecentPosts(false);
200
+            $ret = & $xoopsapi->getRecentPosts(false);
201 201
             if (is_array($ret)) {
202 202
                 $arr   = new XoopsXmlRpcArray();
203 203
                 $count = count($ret);
@@ -218,14 +218,14 @@  discard block
 block discarded – undo
218 218
                                     break;
219 219
                                 case 'postid':
220 220
                                     $struct->add('postid', new XoopsXmlRpcString($value));
221
-                                    $struct->add('link', new XoopsXmlRpcString(XOOPS_URL . '/modules/news/article.php?item_id=' . $value));
222
-                                    $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL . '/modules/news/article.php?item_id=' . $value));
221
+                                    $struct->add('link', new XoopsXmlRpcString(XOOPS_URL.'/modules/news/article.php?item_id='.$value));
222
+                                    $struct->add('permaLink', new XoopsXmlRpcString(XOOPS_URL.'/modules/news/article.php?item_id='.$value));
223 223
                                     break;
224 224
                                 case 'title':
225 225
                                     $struct->add('title', new XoopsXmlRpcString($value));
226 226
                                     break;
227 227
                                 default :
228
-                                    $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
228
+                                    $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
229 229
                                     break;
230 230
                             }
231 231
                         }
@@ -246,15 +246,15 @@  discard block
 block discarded – undo
246 246
         if (!$this->_checkUser($this->params[1], $this->params[2])) {
247 247
             $this->response->add(new XoopsXmlRpcFault(104));
248 248
         } else {
249
-            $xoopsapi =& $this->_getXoopsApi($this->params);
249
+            $xoopsapi = & $this->_getXoopsApi($this->params);
250 250
             $xoopsapi->_setUser($this->user, $this->isadmin);
251
-            $ret =& $xoopsapi->getCategories(false);
251
+            $ret = & $xoopsapi->getCategories(false);
252 252
             if (is_array($ret)) {
253 253
                 $arr = new XoopsXmlRpcArray();
254 254
                 foreach ($ret as $id => $detail) {
255 255
                     $struct = new XoopsXmlRpcStruct();
256 256
                     $struct->add('description', new XoopsXmlRpcString($detail));
257
-                    $struct->add('htmlUrl', new XoopsXmlRpcString(XOOPS_URL . '/modules/news/index.php?storytopic=' . $id));
257
+                    $struct->add('htmlUrl', new XoopsXmlRpcString(XOOPS_URL.'/modules/news/index.php?storytopic='.$id));
258 258
                     $struct->add('rssUrl', new XoopsXmlRpcString(''));
259 259
                     $catstruct = new XoopsXmlRpcStruct();
260 260
                     $catstruct->add($detail['title'], $struct);
Please login to merge, or discard this patch.
htdocs/class/xml/rpc/bloggerapi.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
  */
19 19
 
20 20
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
21
-require_once XOOPS_ROOT_PATH . '/class/xml/rpc/xmlrpcapi.php';
21
+require_once XOOPS_ROOT_PATH.'/class/xml/rpc/xmlrpcapi.php';
22 22
 
23 23
 /**
24 24
  * Class BloggerApi
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
         if (!$this->_checkUser($this->params[2], $this->params[3])) {
44 44
             $this->response->add(new XoopsXmlRpcFault(104));
45 45
         } else {
46
-            if (!$fields =& $this->_getPostFields(null, $this->params[1])) {
46
+            if (!$fields = & $this->_getPostFields(null, $this->params[1])) {
47 47
                 $this->response->add(new XoopsXmlRpcFault(106));
48 48
             } else {
49 49
                 $missing = array();
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
                 if (count($missing) > 0) {
63 63
                     $msg = '';
64 64
                     foreach ($missing as $m) {
65
-                        $msg .= '<' . $m . '> ';
65
+                        $msg .= '<'.$m.'> ';
66 66
                     }
67 67
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
68 68
                 } else {
@@ -72,12 +72,12 @@  discard block
 block discarded – undo
72 72
                     $newparams[1] = $this->params[2];
73 73
                     $newparams[2] = $this->params[3];
74 74
                     foreach ($post as $key => $value) {
75
-                        $newparams[3][$key] =& $value;
75
+                        $newparams[3][$key] = & $value;
76 76
                         unset($value);
77 77
                     }
78 78
                     $newparams[3]['xoops_text'] = $this->params[4];
79 79
                     $newparams[4]               = $this->params[5];
80
-                    $xoopsapi                   =& $this->_getXoopsApi($newparams);
80
+                    $xoopsapi                   = & $this->_getXoopsApi($newparams);
81 81
                     $xoopsapi->_setUser($this->user, $this->isadmin);
82 82
                     $xoopsapi->newPost();
83 83
                 }
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
         if (!$this->_checkUser($this->params[2], $this->params[3])) {
91 91
             $this->response->add(new XoopsXmlRpcFault(104));
92 92
         } else {
93
-            if (!$fields =& $this->_getPostFields($this->params[1])) {
93
+            if (!$fields = & $this->_getPostFields($this->params[1])) {
94 94
             } else {
95 95
                 $missing = array();
96 96
                 $post    = array();
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
                 if (count($missing) > 0) {
108 108
                     $msg = '';
109 109
                     foreach ($missing as $m) {
110
-                        $msg .= '<' . $m . '> ';
110
+                        $msg .= '<'.$m.'> ';
111 111
                     }
112 112
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
113 113
                 } else {
@@ -117,12 +117,12 @@  discard block
 block discarded – undo
117 117
                     $newparams[1] = $this->params[2];
118 118
                     $newparams[2] = $this->params[3];
119 119
                     foreach ($post as $key => $value) {
120
-                        $newparams[3][$key] =& $value;
120
+                        $newparams[3][$key] = & $value;
121 121
                         unset($value);
122 122
                     }
123 123
                     $newparams[3]['xoops_text'] = $this->params[4];
124 124
                     $newparams[4]               = $this->params[5];
125
-                    $xoopsapi                   =& $this->_getXoopsApi($newparams);
125
+                    $xoopsapi                   = & $this->_getXoopsApi($newparams);
126 126
                     $xoopsapi->_setUser($this->user, $this->isadmin);
127 127
                     $xoopsapi->editPost();
128 128
                 }
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
         } else {
138 138
             // XOOPS API ignores App key (index 0 of params)
139 139
             array_shift($this->params);
140
-            $xoopsapi =& $this->_getXoopsApi($this->params);
140
+            $xoopsapi = & $this->_getXoopsApi($this->params);
141 141
             $xoopsapi->_setUser($this->user, $this->isadmin);
142 142
             $xoopsapi->deletePost();
143 143
         }
@@ -150,9 +150,9 @@  discard block
 block discarded – undo
150 150
         } else {
151 151
             // XOOPS API ignores App key (index 0 of params)
152 152
             array_shift($this->params);
153
-            $xoopsapi =& $this->_getXoopsApi($this->params);
153
+            $xoopsapi = & $this->_getXoopsApi($this->params);
154 154
             $xoopsapi->_setUser($this->user, $this->isadmin);
155
-            $ret =& $xoopsapi->getPost(false);
155
+            $ret = & $xoopsapi->getPost(false);
156 156
             if (is_array($ret)) {
157 157
                 $struct  = new XoopsXmlRpcStruct();
158 158
                 $content = '';
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
                             $struct->add('postid', new XoopsXmlRpcString($value));
170 170
                             break;
171 171
                         default :
172
-                            $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
172
+                            $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
173 173
                             break;
174 174
                     }
175 175
                 }
@@ -188,9 +188,9 @@  discard block
 block discarded – undo
188 188
         } else {
189 189
             // XOOPS API ignores App key (index 0 of params)
190 190
             array_shift($this->params);
191
-            $xoopsapi =& $this->_getXoopsApi($this->params);
191
+            $xoopsapi = & $this->_getXoopsApi($this->params);
192 192
             $xoopsapi->_setUser($this->user, $this->isadmin);
193
-            $ret =& $xoopsapi->getRecentPosts(false);
193
+            $ret = & $xoopsapi->getRecentPosts(false);
194 194
             if (is_array($ret)) {
195 195
                 $arr   = new XoopsXmlRpcArray();
196 196
                 $count = count($ret);
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
                                     $struct->add('postid', new XoopsXmlRpcString($value));
214 214
                                     break;
215 215
                                 default :
216
-                                    $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
216
+                                    $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
217 217
                                     break;
218 218
                             }
219 219
                         }
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
         } else {
237 237
             $arr    = new XoopsXmlRpcArray();
238 238
             $struct = new XoopsXmlRpcStruct();
239
-            $struct->add('url', new XoopsXmlRpcString(XOOPS_URL . '/modules/' . $this->module->getVar('dirname') . '/'));
239
+            $struct->add('url', new XoopsXmlRpcString(XOOPS_URL.'/modules/'.$this->module->getVar('dirname').'/'));
240 240
             $struct->add('blogid', new XoopsXmlRpcString($this->module->getVar('mid')));
241 241
             $struct->add('blogName', new XoopsXmlRpcString('XOOPS Blog'));
242 242
             $arr->add($struct);
Please login to merge, or discard this patch.
htdocs/class/xml/rss/xmlrss2parser.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -18,8 +18,8 @@  discard block
 block discarded – undo
18 18
  */
19 19
 
20 20
 defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');
21
-require_once(XOOPS_ROOT_PATH . '/class/xml/saxparser.php');
22
-require_once(XOOPS_ROOT_PATH . '/class/xml/xmltaghandler.php');
21
+require_once(XOOPS_ROOT_PATH.'/class/xml/saxparser.php');
22
+require_once(XOOPS_ROOT_PATH.'/class/xml/xmltaghandler.php');
23 23
 
24 24
 /**
25 25
  * Class XoopsXmlRss2Parser
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
     public function setChannelData($name, &$value)
71 71
     {
72 72
         if (!isset($this->_channelData[$name])) {
73
-            $this->_channelData[$name] =& $value;
73
+            $this->_channelData[$name] = & $value;
74 74
         } else {
75 75
             $this->_channelData[$name] .= $value;
76 76
         }
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
      */
101 101
     public function setImageData($name, &$value)
102 102
     {
103
-        $this->_imageData[$name] =& $value;
103
+        $this->_imageData[$name] = & $value;
104 104
     }
105 105
 
106 106
     /**
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
      */
128 128
     public function setItems(&$itemarr)
129 129
     {
130
-        $this->_items[] =& $itemarr;
130
+        $this->_items[] = & $itemarr;
131 131
     }
132 132
 
133 133
     /**
@@ -146,9 +146,9 @@  discard block
 block discarded – undo
146 146
     public function setTempArr($name, &$value, $delim = '')
147 147
     {
148 148
         if (!isset($this->_tempArr[$name])) {
149
-            $this->_tempArr[$name] =& $value;
149
+            $this->_tempArr[$name] = & $value;
150 150
         } else {
151
-            $this->_tempArr[$name] .= $delim . $value;
151
+            $this->_tempArr[$name] .= $delim.$value;
152 152
         }
153 153
     }
154 154
 
Please login to merge, or discard this patch.
htdocs/class/xml/saxparser.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -260,7 +260,7 @@  discard block
 block discarded – undo
260 260
      */
261 261
     public function handleCharacterData($parser, $data)
262 262
     {
263
-        $tagHandler =& $this->tagHandlers[$this->getCurrentTag()];
263
+        $tagHandler = & $this->tagHandlers[$this->getCurrentTag()];
264 264
         if (isset($tagHandler) && is_subclass_of($tagHandler, 'XmlTagHandler')) {
265 265
             $tagHandler->handleCharacterData($this, $data);
266 266
         } else {
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
             $ret = '';
387 387
             if (count($this->errors) > 0) {
388 388
                 foreach ($this->errors as $error) {
389
-                    $ret .= $error . '<br>';
389
+                    $ret .= $error.'<br>';
390 390
                 }
391 391
             }
392 392
 
Please login to merge, or discard this patch.
htdocs/class/tardownloader.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -20,11 +20,11 @@  discard block
 block discarded – undo
20 20
 /**
21 21
  * base class
22 22
  */
23
-include_once XOOPS_ROOT_PATH . '/class/downloader.php';
23
+include_once XOOPS_ROOT_PATH.'/class/downloader.php';
24 24
 /**
25 25
  * Class to handle tar files
26 26
  */
27
-include_once XOOPS_ROOT_PATH . '/class/class.tar.php';
27
+include_once XOOPS_ROOT_PATH.'/class/class.tar.php';
28 28
 
29 29
 /**
30 30
  * Class XoopsTarDownloader
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
      */
94 94
     public function addFileData(&$data, $filename, $time = 0)
95 95
     {
96
-        $dummyfile = XOOPS_CACHE_PATH . '/dummy_' . time() . '.tpl';
96
+        $dummyfile = XOOPS_CACHE_PATH.'/dummy_'.time().'.tpl';
97 97
         $fp        = fopen($dummyfile, 'w');
98 98
         fwrite($fp, $data);
99 99
         fclose($fp);
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
      */
121 121
     public function addBinaryFileData(&$data, $filename, $time = 0)
122 122
     {
123
-        $dummyfile = XOOPS_CACHE_PATH . '/dummy_' . time() . '.tpl';
123
+        $dummyfile = XOOPS_CACHE_PATH.'/dummy_'.time().'.tpl';
124 124
         $fp        = fopen($dummyfile, 'wb');
125 125
         fwrite($fp, $data);
126 126
         fclose($fp);
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
      */
147 147
     public function download($name, $gzip = true)
148 148
     {
149
-        $this->_header($name . $this->ext);
150
-        echo $this->archiver->toTarOutput($name . $this->ext, $gzip);
149
+        $this->_header($name.$this->ext);
150
+        echo $this->archiver->toTarOutput($name.$this->ext, $gzip);
151 151
     }
152 152
 }
Please login to merge, or discard this patch.