Passed
Push — master ( f31a58...bb780a )
by Gaetano
09:56
created
debugger/action.php 1 patch
Spacing   +115 added lines, -115 removed lines patch added patch discarded remove patch
@@ -78,13 +78,13 @@  discard block
 block discarded – undo
78 78
 <body>
79 79
 <?php
80 80
 
81
-include __DIR__ . '/common.php';
81
+include __DIR__.'/common.php';
82 82
 
83 83
 if ($action) {
84 84
 
85 85
     // avoid php hanging when using the builtin webserver and sending requests to itself
86 86
     $skip = false;
87
-    if (php_sapi_name() === 'cli-server' && ((int)getenv('PHP_CLI_SERVER_WORKERS') < 2)) {
87
+    if (php_sapi_name() === 'cli-server' && ((int) getenv('PHP_CLI_SERVER_WORKERS')<2)) {
88 88
         $localHost = explode(':', $_SERVER['HTTP_HOST']);
89 89
         /// @todo support also case where port is null (on either side), and when there is a Proxy in the parameters,
90 90
         ///       and that proxy is us
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     if (!$skip) {
98 98
         // make sure the script waits long enough for the call to complete...
99 99
         if ($timeout) {
100
-            set_time_limit($timeout + 10);
100
+            set_time_limit($timeout+10);
101 101
         }
102 102
 
103 103
         if ($wstype == 1) {
@@ -118,13 +118,13 @@  discard block
 block discarded – undo
118 118
             $server = "$host$path";
119 119
         }
120 120
         if ($protocol == 2 || $protocol == 3) {
121
-            $server = 'https://' . $server;
121
+            $server = 'https://'.$server;
122 122
         } else {
123
-            $server = 'http://' . $server;
123
+            $server = 'http://'.$server;
124 124
         }
125 125
         if ($proxy != '') {
126 126
             $pproxy = explode(':', $proxy);
127
-            if (count($pproxy) > 1) {
127
+            if (count($pproxy)>1) {
128 128
                 $pport = $pproxy[1];
129 129
             } else {
130 130
                 $pport = 8080;
@@ -199,9 +199,9 @@  discard block
 block discarded – undo
199 199
             case 'wrap':
200 200
                 $msg[0] = new $requestClass('system.methodHelp', array(), $id);
201 201
                 $msg[0]->addparam(new PhpXmlRpc\Value($method));
202
-                $msg[1] = new $requestClass('system.methodSignature', array(), (int)$id + 1);
202
+                $msg[1] = new $requestClass('system.methodSignature', array(), (int) $id+1);
203 203
                 $msg[1]->addparam(new PhpXmlRpc\Value($method));
204
-                $actionname = 'Description of method "' . $method . '"';
204
+                $actionname = 'Description of method "'.$method.'"';
205 205
                 break;
206 206
             case 'list':
207 207
                 $msg[0] = new $requestClass('system.listMethods', array(), $id);
@@ -214,9 +214,9 @@  discard block
 block discarded – undo
214 214
                 $msg[0] = new $requestClass($method, array(), $id);
215 215
                 // hack! build payload by hand
216 216
                 if ($wstype == 1) {
217
-                    $msg[0]->payload = "{\n" .
218
-                        '"method": "' . $method . "\",\n\"params\": [" .
219
-                        $payload .
217
+                    $msg[0]->payload = "{\n".
218
+                        '"method": "'.$method."\",\n\"params\": [".
219
+                        $payload.
220 220
                         "\n],\n\"id\": ";
221 221
                     // fix: if user gave an empty string, use NULL, or we'll break json syntax
222 222
                     if ($id == "") {
@@ -229,21 +229,21 @@  discard block
 block discarded – undo
229 229
                         }
230 230
                     }
231 231
                 } else {
232
-                    $msg[0]->payload = $msg[0]->xml_header($inputcharset) .
233
-                        '<methodName>' . $method . "</methodName>\n<params>" .
234
-                        $payload .
235
-                        "</params>\n" . $msg[0]->xml_footer();
232
+                    $msg[0]->payload = $msg[0]->xml_header($inputcharset).
233
+                        '<methodName>'.$method."</methodName>\n<params>".
234
+                        $payload.
235
+                        "</params>\n".$msg[0]->xml_footer();
236 236
                 }
237
-                $actionname = 'Execution of method ' . $method;
237
+                $actionname = 'Execution of method '.$method;
238 238
                 break;
239 239
             default: // give a warning
240
-                $actionname = '[ERROR: unknown action] "' . $action . '"';
240
+                $actionname = '[ERROR: unknown action] "'.$action.'"';
241 241
         }
242 242
     }
243 243
 
244 244
     // Before calling execute, echo out brief description of action taken + date and time ???
245 245
     // this gives good user feedback for long-running methods...
246
-    echo '<h2>' . htmlspecialchars($actionname, ENT_COMPAT, $inputcharset) . ' on server ' . htmlspecialchars($server, ENT_COMPAT, $inputcharset) . " ...</h2>\n";
246
+    echo '<h2>'.htmlspecialchars($actionname, ENT_COMPAT, $inputcharset).' on server '.htmlspecialchars($server, ENT_COMPAT, $inputcharset)." ...</h2>\n";
247 247
     flush();
248 248
 
249 249
     $response = null;
@@ -260,7 +260,7 @@  discard block
 block discarded – undo
260 260
             break;
261 261
         }
262 262
     }
263
-    $time = microtime(true) - $time;
263
+    $time = microtime(true)-$time;
264 264
     if ($debug) {
265 265
         echo "</div>\n";
266 266
     }
@@ -270,8 +270,8 @@  discard block
 block discarded – undo
270 270
             // call failed! echo out error msg!
271 271
             //echo '<h2>'.htmlspecialchars($actionname, ENT_COMPAT, $inputcharset).' on server '.htmlspecialchars($server, ENT_COMPAT, $inputcharset).'</h2>';
272 272
             echo "<h3>$protoName call FAILED!</h3>\n";
273
-            echo "<p>Fault code: [" . htmlspecialchars($response->faultCode(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) .
274
-                "] Reason: '" . htmlspecialchars($response->faultString(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) . "'</p>\n";
273
+            echo "<p>Fault code: [".htmlspecialchars($response->faultCode(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding).
274
+                "] Reason: '".htmlspecialchars($response->faultString(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding)."'</p>\n";
275 275
             echo(date("d/M/Y:H:i:s\n"));
276 276
         } else {
277 277
             // call succeeded: parse results
@@ -287,36 +287,36 @@  discard block
 block discarded – undo
287 287
                         $max = $v->count();
288 288
                         echo "<table border=\"0\" cellspacing=\"0\" cellpadding=\"0\">\n";
289 289
                         echo "<thead>\n<tr><th>Method ($max)</th><th>Description</th></tr>\n</thead>\n<tbody>\n";
290
-                        foreach($v as $i => $rec) {
290
+                        foreach ($v as $i => $rec) {
291 291
                             if ($i % 2) {
292 292
                                 $class = ' class="oddrow"';
293 293
                             } else {
294 294
                                 $class = ' class="evenrow"';
295 295
                             }
296
-                            echo("<tr><td$class>" . htmlspecialchars($rec->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) . "</td><td$class><form action=\"controller.php\" method=\"get\" target=\"frmcontroller\">" .
297
-                                "<input type=\"hidden\" name=\"host\" value=\"" . htmlspecialchars($host, ENT_COMPAT, $inputcharset) . "\" />" .
298
-                                "<input type=\"hidden\" name=\"port\" value=\"" . htmlspecialchars($port, ENT_COMPAT, $inputcharset) . "\" />" .
299
-                                "<input type=\"hidden\" name=\"path\" value=\"" . htmlspecialchars($path, ENT_COMPAT, $inputcharset) . "\" />" .
300
-                                "<input type=\"hidden\" name=\"id\" value=\"" . htmlspecialchars($id, ENT_COMPAT, $inputcharset) . "\" />" .
301
-                                "<input type=\"hidden\" name=\"debug\" value=\"$debug\" />" .
302
-                                "<input type=\"hidden\" name=\"username\" value=\"" . htmlspecialchars($username, ENT_COMPAT, $inputcharset) . "\" />" .
303
-                                "<input type=\"hidden\" name=\"password\" value=\"" . htmlspecialchars($password, ENT_COMPAT, $inputcharset) . "\" />" .
304
-                                "<input type=\"hidden\" name=\"authtype\" value=\"$authtype\" />" .
305
-                                "<input type=\"hidden\" name=\"verifyhost\" value=\"$verifyhost\" />" .
306
-                                "<input type=\"hidden\" name=\"verifypeer\" value=\"$verifypeer\" />" .
307
-                                "<input type=\"hidden\" name=\"cainfo\" value=\"" . htmlspecialchars($cainfo, ENT_COMPAT, $inputcharset) . "\" />" .
308
-                                "<input type=\"hidden\" name=\"proxy\" value=\"" . htmlspecialchars($proxy, ENT_COMPAT, $inputcharset) . "\" />" .
309
-                                "<input type=\"hidden\" name=\"proxyuser\" value=\"" . htmlspecialchars($proxyuser, ENT_COMPAT, $inputcharset) . "\" />" .
310
-                                "<input type=\"hidden\" name=\"proxypwd\" value=\"" . htmlspecialchars($proxypwd, ENT_COMPAT, $inputcharset) . "\" />" .
311
-                                "<input type=\"hidden\" name=\"responsecompression\" value=\"$responsecompression\" />" .
312
-                                "<input type=\"hidden\" name=\"requestcompression\" value=\"$requestcompression\" />" .
313
-                                "<input type=\"hidden\" name=\"clientcookies\" value=\"" . htmlspecialchars($clientcookies, ENT_COMPAT, $inputcharset) . "\" />" .
314
-                                "<input type=\"hidden\" name=\"protocol\" value=\"$protocol\" />" .
315
-                                "<input type=\"hidden\" name=\"timeout\" value=\"" . htmlspecialchars($timeout, ENT_COMPAT, $inputcharset) . "\" />" .
316
-                                "<input type=\"hidden\" name=\"method\" value=\"" . htmlspecialchars($rec->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) . "\" />" .
317
-                                "<input type=\"hidden\" name=\"wstype\" value=\"$wstype\" />" .
318
-                                "<input type=\"hidden\" name=\"action\" value=\"describe\" />" .
319
-                                "<input type=\"hidden\" name=\"run\" value=\"now\" />" .
296
+                            echo("<tr><td$class>".htmlspecialchars($rec->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding)."</td><td$class><form action=\"controller.php\" method=\"get\" target=\"frmcontroller\">".
297
+                                "<input type=\"hidden\" name=\"host\" value=\"".htmlspecialchars($host, ENT_COMPAT, $inputcharset)."\" />".
298
+                                "<input type=\"hidden\" name=\"port\" value=\"".htmlspecialchars($port, ENT_COMPAT, $inputcharset)."\" />".
299
+                                "<input type=\"hidden\" name=\"path\" value=\"".htmlspecialchars($path, ENT_COMPAT, $inputcharset)."\" />".
300
+                                "<input type=\"hidden\" name=\"id\" value=\"".htmlspecialchars($id, ENT_COMPAT, $inputcharset)."\" />".
301
+                                "<input type=\"hidden\" name=\"debug\" value=\"$debug\" />".
302
+                                "<input type=\"hidden\" name=\"username\" value=\"".htmlspecialchars($username, ENT_COMPAT, $inputcharset)."\" />".
303
+                                "<input type=\"hidden\" name=\"password\" value=\"".htmlspecialchars($password, ENT_COMPAT, $inputcharset)."\" />".
304
+                                "<input type=\"hidden\" name=\"authtype\" value=\"$authtype\" />".
305
+                                "<input type=\"hidden\" name=\"verifyhost\" value=\"$verifyhost\" />".
306
+                                "<input type=\"hidden\" name=\"verifypeer\" value=\"$verifypeer\" />".
307
+                                "<input type=\"hidden\" name=\"cainfo\" value=\"".htmlspecialchars($cainfo, ENT_COMPAT, $inputcharset)."\" />".
308
+                                "<input type=\"hidden\" name=\"proxy\" value=\"".htmlspecialchars($proxy, ENT_COMPAT, $inputcharset)."\" />".
309
+                                "<input type=\"hidden\" name=\"proxyuser\" value=\"".htmlspecialchars($proxyuser, ENT_COMPAT, $inputcharset)."\" />".
310
+                                "<input type=\"hidden\" name=\"proxypwd\" value=\"".htmlspecialchars($proxypwd, ENT_COMPAT, $inputcharset)."\" />".
311
+                                "<input type=\"hidden\" name=\"responsecompression\" value=\"$responsecompression\" />".
312
+                                "<input type=\"hidden\" name=\"requestcompression\" value=\"$requestcompression\" />".
313
+                                "<input type=\"hidden\" name=\"clientcookies\" value=\"".htmlspecialchars($clientcookies, ENT_COMPAT, $inputcharset)."\" />".
314
+                                "<input type=\"hidden\" name=\"protocol\" value=\"$protocol\" />".
315
+                                "<input type=\"hidden\" name=\"timeout\" value=\"".htmlspecialchars($timeout, ENT_COMPAT, $inputcharset)."\" />".
316
+                                "<input type=\"hidden\" name=\"method\" value=\"".htmlspecialchars($rec->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding)."\" />".
317
+                                "<input type=\"hidden\" name=\"wstype\" value=\"$wstype\" />".
318
+                                "<input type=\"hidden\" name=\"action\" value=\"describe\" />".
319
+                                "<input type=\"hidden\" name=\"run\" value=\"now\" />".
320 320
                                 "<input type=\"submit\" value=\"Describe\" /></form></td>");
321 321
                             //echo("</tr>\n");
322 322
 
@@ -343,7 +343,7 @@  discard block
 block discarded – undo
343 343
                     $r2 = $resp[1]->value();
344 344
 
345 345
                     echo "<table border=\"0\" cellspacing=\"0\" cellpadding=\"0\">\n";
346
-                    echo "<thead>\n<tr><th>Method</th><th>" . htmlspecialchars($method, ENT_COMPAT, $inputcharset) . "</th><th>&nbsp;</th><th>&nbsp;</th></tr>\n</thead>\n<tbody>\n";
346
+                    echo "<thead>\n<tr><th>Method</th><th>".htmlspecialchars($method, ENT_COMPAT, $inputcharset)."</th><th>&nbsp;</th><th>&nbsp;</th></tr>\n</thead>\n<tbody>\n";
347 347
                     $desc = htmlspecialchars($r1->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding);
348 348
                     if ($desc == "") {
349 349
                         $desc = "-";
@@ -353,24 +353,24 @@  discard block
 block discarded – undo
353 353
                     if ($r2->kindOf() != "array") {
354 354
                         echo "<tr><td class=\"oddrow\">Signature</td><td class=\"oddrow\">Unknown</td><td class=\"oddrow\">&nbsp;</td></tr>\n";
355 355
                     } else {
356
-                        foreach($r2 as $i => $x) {
356
+                        foreach ($r2 as $i => $x) {
357 357
                             $payload = "";
358 358
                             $alt_payload = "";
359
-                            if ($i + 1 % 2) {
359
+                            if ($i+1 % 2) {
360 360
                                 $class = ' class="oddrow"';
361 361
                             } else {
362 362
                                 $class = ' class="evenrow"';
363 363
                             }
364
-                            echo "<tr><td$class>Signature&nbsp;" . ($i + 1) . "</td><td$class>";
364
+                            echo "<tr><td$class>Signature&nbsp;".($i+1)."</td><td$class>";
365 365
                             if ($x->kindOf() == "array") {
366 366
                                 $ret = $x[0];
367
-                                echo "<code>OUT:&nbsp;" . htmlspecialchars($ret->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) . "<br />IN: (";
368
-                                if ($x->count() > 1) {
369
-                                    foreach($x as $k => $y) {
367
+                                echo "<code>OUT:&nbsp;".htmlspecialchars($ret->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding)."<br />IN: (";
368
+                                if ($x->count()>1) {
369
+                                    foreach ($x as $k => $y) {
370 370
                                         if ($k == 0) continue;
371 371
                                         echo htmlspecialchars($y->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding);
372 372
                                         if ($wstype == 1) {
373
-                                            switch($y->scalarval()) {
373
+                                            switch ($y->scalarval()) {
374 374
                                                 case 'string':
375 375
                                                 case 'dateTime.iso8601':
376 376
                                                 case 'base64':
@@ -403,22 +403,22 @@  discard block
 block discarded – undo
403 403
                                         } else {
404 404
                                             $type = $y->scalarval();
405 405
                                             $payload .= '<param><value>';
406
-                                            switch($type) {
406
+                                            switch ($type) {
407 407
                                                 case 'undefined':
408 408
                                                     break;
409 409
                                                 case 'null';
410 410
                                                     $type = 'nil';
411 411
                                                     // fall thru intentionally
412 412
                                                 default:
413
-                                                    $payload .= '<' .
414
-                                                        htmlspecialchars($type, ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) .
415
-                                                        '></' . htmlspecialchars($type, ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) .
413
+                                                    $payload .= '<'.
414
+                                                        htmlspecialchars($type, ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding).
415
+                                                        '></'.htmlspecialchars($type, ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding).
416 416
                                                         '>';
417 417
                                             }
418 418
                                             $payload .= "</value></param>\n";
419 419
                                         }
420 420
                                         $alt_payload .= $y->scalarval();
421
-                                        if ($k < $x->count() - 1) {
421
+                                        if ($k<$x->count()-1) {
422 422
                                             $alt_payload .= ';';
423 423
                                             if ($wstype == 1) {
424 424
                                                 $payload .= ', ';
@@ -434,63 +434,63 @@  discard block
 block discarded – undo
434 434
                             echo '</td>';
435 435
                             // button to test this method
436 436
                             //$payload="<methodCall>\n<methodName>$method</methodName>\n<params>\n$payload</params>\n</methodCall>";
437
-                            echo "<td$class><form action=\"controller.php\" target=\"frmcontroller\" method=\"get\">" .
438
-                                "<input type=\"hidden\" name=\"host\" value=\"" . htmlspecialchars($host, ENT_COMPAT, $inputcharset) . "\" />" .
439
-                                "<input type=\"hidden\" name=\"port\" value=\"" . htmlspecialchars($port, ENT_COMPAT, $inputcharset) . "\" />" .
440
-                                "<input type=\"hidden\" name=\"path\" value=\"" . htmlspecialchars($path, ENT_COMPAT, $inputcharset) . "\" />" .
441
-                                "<input type=\"hidden\" name=\"id\" value=\"" . htmlspecialchars($id, ENT_COMPAT, $inputcharset) . "\" />" .
442
-                                "<input type=\"hidden\" name=\"debug\" value=\"$debug\" />" .
443
-                                "<input type=\"hidden\" name=\"username\" value=\"" . htmlspecialchars($username, ENT_COMPAT, $inputcharset) . "\" />" .
444
-                                "<input type=\"hidden\" name=\"password\" value=\"" . htmlspecialchars($password, ENT_COMPAT, $inputcharset) . "\" />" .
445
-                                "<input type=\"hidden\" name=\"authtype\" value=\"$authtype\" />" .
446
-                                "<input type=\"hidden\" name=\"verifyhost\" value=\"$verifyhost\" />" .
447
-                                "<input type=\"hidden\" name=\"verifypeer\" value=\"$verifypeer\" />" .
448
-                                "<input type=\"hidden\" name=\"cainfo\" value=\"" . htmlspecialchars($cainfo, ENT_COMPAT, $inputcharset) . "\" />" .
449
-                                "<input type=\"hidden\" name=\"proxy\" value=\"" . htmlspecialchars($proxy, ENT_COMPAT, $inputcharset) . "\" />" .
450
-                                "<input type=\"hidden\" name=\"proxyuser\" value=\"" . htmlspecialchars($proxyuser, ENT_COMPAT, $inputcharset) . "\" />" .
451
-                                "<input type=\"hidden\" name=\"proxypwd\" value=\"" . htmlspecialchars($proxypwd, ENT_COMPAT, $inputcharset) . "\" />" .
452
-                                "<input type=\"hidden\" name=\"responsecompression\" value=\"$responsecompression\" />" .
453
-                                "<input type=\"hidden\" name=\"requestcompression\" value=\"$requestcompression\" />" .
454
-                                "<input type=\"hidden\" name=\"clientcookies\" value=\"" . htmlspecialchars($clientcookies, ENT_COMPAT, $inputcharset) . "\" />" .
455
-                                "<input type=\"hidden\" name=\"protocol\" value=\"$protocol\" />" .
456
-                                "<input type=\"hidden\" name=\"timeout\" value=\"" . htmlspecialchars($timeout, ENT_COMPAT, $inputcharset) . "\" />" .
457
-                                "<input type=\"hidden\" name=\"method\" value=\"" . htmlspecialchars($method, ENT_COMPAT, $inputcharset) . "\" />" .
458
-                                "<input type=\"hidden\" name=\"methodpayload\" value=\"" . htmlspecialchars($payload, ENT_COMPAT, $inputcharset) . "\" />" .
459
-                                "<input type=\"hidden\" name=\"altmethodpayload\" value=\"" . htmlspecialchars($alt_payload, ENT_COMPAT, $inputcharset) . "\" />" .
460
-                                "<input type=\"hidden\" name=\"wstype\" value=\"$wstype\" />" .
437
+                            echo "<td$class><form action=\"controller.php\" target=\"frmcontroller\" method=\"get\">".
438
+                                "<input type=\"hidden\" name=\"host\" value=\"".htmlspecialchars($host, ENT_COMPAT, $inputcharset)."\" />".
439
+                                "<input type=\"hidden\" name=\"port\" value=\"".htmlspecialchars($port, ENT_COMPAT, $inputcharset)."\" />".
440
+                                "<input type=\"hidden\" name=\"path\" value=\"".htmlspecialchars($path, ENT_COMPAT, $inputcharset)."\" />".
441
+                                "<input type=\"hidden\" name=\"id\" value=\"".htmlspecialchars($id, ENT_COMPAT, $inputcharset)."\" />".
442
+                                "<input type=\"hidden\" name=\"debug\" value=\"$debug\" />".
443
+                                "<input type=\"hidden\" name=\"username\" value=\"".htmlspecialchars($username, ENT_COMPAT, $inputcharset)."\" />".
444
+                                "<input type=\"hidden\" name=\"password\" value=\"".htmlspecialchars($password, ENT_COMPAT, $inputcharset)."\" />".
445
+                                "<input type=\"hidden\" name=\"authtype\" value=\"$authtype\" />".
446
+                                "<input type=\"hidden\" name=\"verifyhost\" value=\"$verifyhost\" />".
447
+                                "<input type=\"hidden\" name=\"verifypeer\" value=\"$verifypeer\" />".
448
+                                "<input type=\"hidden\" name=\"cainfo\" value=\"".htmlspecialchars($cainfo, ENT_COMPAT, $inputcharset)."\" />".
449
+                                "<input type=\"hidden\" name=\"proxy\" value=\"".htmlspecialchars($proxy, ENT_COMPAT, $inputcharset)."\" />".
450
+                                "<input type=\"hidden\" name=\"proxyuser\" value=\"".htmlspecialchars($proxyuser, ENT_COMPAT, $inputcharset)."\" />".
451
+                                "<input type=\"hidden\" name=\"proxypwd\" value=\"".htmlspecialchars($proxypwd, ENT_COMPAT, $inputcharset)."\" />".
452
+                                "<input type=\"hidden\" name=\"responsecompression\" value=\"$responsecompression\" />".
453
+                                "<input type=\"hidden\" name=\"requestcompression\" value=\"$requestcompression\" />".
454
+                                "<input type=\"hidden\" name=\"clientcookies\" value=\"".htmlspecialchars($clientcookies, ENT_COMPAT, $inputcharset)."\" />".
455
+                                "<input type=\"hidden\" name=\"protocol\" value=\"$protocol\" />".
456
+                                "<input type=\"hidden\" name=\"timeout\" value=\"".htmlspecialchars($timeout, ENT_COMPAT, $inputcharset)."\" />".
457
+                                "<input type=\"hidden\" name=\"method\" value=\"".htmlspecialchars($method, ENT_COMPAT, $inputcharset)."\" />".
458
+                                "<input type=\"hidden\" name=\"methodpayload\" value=\"".htmlspecialchars($payload, ENT_COMPAT, $inputcharset)."\" />".
459
+                                "<input type=\"hidden\" name=\"altmethodpayload\" value=\"".htmlspecialchars($alt_payload, ENT_COMPAT, $inputcharset)."\" />".
460
+                                "<input type=\"hidden\" name=\"wstype\" value=\"$wstype\" />".
461 461
                                 "<input type=\"hidden\" name=\"action\" value=\"execute\" />";
462 462
                             //if ($wstype != 1) {
463 463
                                 echo "<input type=\"submit\" value=\"Load method synopsis\" />";
464 464
                             //}
465 465
                             echo "</form></td>\n";
466 466
 
467
-                            echo "<td$class><form action=\"controller.php\" target=\"frmcontroller\" method=\"get\">" .
468
-                                "<input type=\"hidden\" name=\"host\" value=\"" . htmlspecialchars($host, ENT_COMPAT, $inputcharset) . "\" />" .
469
-                                "<input type=\"hidden\" name=\"port\" value=\"" . htmlspecialchars($port, ENT_COMPAT, $inputcharset) . "\" />" .
470
-                                "<input type=\"hidden\" name=\"path\" value=\"" . htmlspecialchars($path, ENT_COMPAT, $inputcharset) . "\" />" .
471
-                                "<input type=\"hidden\" name=\"id\" value=\"" . htmlspecialchars($id, ENT_COMPAT, $inputcharset) . "\" />" .
472
-                                "<input type=\"hidden\" name=\"debug\" value=\"$debug\" />" .
473
-                                "<input type=\"hidden\" name=\"username\" value=\"" . htmlspecialchars($username, ENT_COMPAT, $inputcharset) . "\" />" .
474
-                                "<input type=\"hidden\" name=\"password\" value=\"" . htmlspecialchars($password, ENT_COMPAT, $inputcharset) . "\" />" .
475
-                                "<input type=\"hidden\" name=\"authtype\" value=\"$authtype\" />" .
476
-                                "<input type=\"hidden\" name=\"verifyhost\" value=\"$verifyhost\" />" .
477
-                                "<input type=\"hidden\" name=\"verifypeer\" value=\"$verifypeer\" />" .
478
-                                "<input type=\"hidden\" name=\"cainfo\" value=\"" . htmlspecialchars($cainfo, ENT_COMPAT, $inputcharset) . "\" />" .
479
-                                "<input type=\"hidden\" name=\"proxy\" value=\"" . htmlspecialchars($proxy, ENT_COMPAT, $inputcharset) . "\" />" .
480
-                                "<input type=\"hidden\" name=\"proxyuser\" value=\"" . htmlspecialchars($proxyuser, ENT_COMPAT, $inputcharset) . "\" />" .
481
-                                "<input type=\"hidden\" name=\"proxypwd\" value=\"" . htmlspecialchars($proxypwd, ENT_COMPAT, $inputcharset) . "\" />" .
482
-                                "<input type=\"hidden\" name=\"responsecompression\" value=\"$responsecompression\" />" .
483
-                                "<input type=\"hidden\" name=\"requestcompression\" value=\"$requestcompression\" />" .
484
-                                "<input type=\"hidden\" name=\"clientcookies\" value=\"" . htmlspecialchars($clientcookies, ENT_COMPAT, $inputcharset) . "\" />" .
485
-                                "<input type=\"hidden\" name=\"protocol\" value=\"$protocol\" />" .
486
-                                "<input type=\"hidden\" name=\"timeout\" value=\"" . htmlspecialchars($timeout, ENT_COMPAT, $inputcharset) . "\" />" .
487
-                                "<input type=\"hidden\" name=\"method\" value=\"" . htmlspecialchars($method, ENT_COMPAT, $inputcharset) . "\" />" .
488
-                                "<input type=\"hidden\" name=\"methodsig\" value=\"" . $i . "\" />" .
489
-                                "<input type=\"hidden\" name=\"methodpayload\" value=\"" . htmlspecialchars($payload, ENT_COMPAT, $inputcharset) . "\" />" .
490
-                                "<input type=\"hidden\" name=\"altmethodpayload\" value=\"" . htmlspecialchars($alt_payload, ENT_COMPAT, $inputcharset) . "\" />" .
491
-                                "<input type=\"hidden\" name=\"wstype\" value=\"$wstype\" />" .
492
-                                "<input type=\"hidden\" name=\"action\" value=\"wrap\" />" .
493
-                                "<input type=\"hidden\" name=\"run\" value=\"now\" />" .
467
+                            echo "<td$class><form action=\"controller.php\" target=\"frmcontroller\" method=\"get\">".
468
+                                "<input type=\"hidden\" name=\"host\" value=\"".htmlspecialchars($host, ENT_COMPAT, $inputcharset)."\" />".
469
+                                "<input type=\"hidden\" name=\"port\" value=\"".htmlspecialchars($port, ENT_COMPAT, $inputcharset)."\" />".
470
+                                "<input type=\"hidden\" name=\"path\" value=\"".htmlspecialchars($path, ENT_COMPAT, $inputcharset)."\" />".
471
+                                "<input type=\"hidden\" name=\"id\" value=\"".htmlspecialchars($id, ENT_COMPAT, $inputcharset)."\" />".
472
+                                "<input type=\"hidden\" name=\"debug\" value=\"$debug\" />".
473
+                                "<input type=\"hidden\" name=\"username\" value=\"".htmlspecialchars($username, ENT_COMPAT, $inputcharset)."\" />".
474
+                                "<input type=\"hidden\" name=\"password\" value=\"".htmlspecialchars($password, ENT_COMPAT, $inputcharset)."\" />".
475
+                                "<input type=\"hidden\" name=\"authtype\" value=\"$authtype\" />".
476
+                                "<input type=\"hidden\" name=\"verifyhost\" value=\"$verifyhost\" />".
477
+                                "<input type=\"hidden\" name=\"verifypeer\" value=\"$verifypeer\" />".
478
+                                "<input type=\"hidden\" name=\"cainfo\" value=\"".htmlspecialchars($cainfo, ENT_COMPAT, $inputcharset)."\" />".
479
+                                "<input type=\"hidden\" name=\"proxy\" value=\"".htmlspecialchars($proxy, ENT_COMPAT, $inputcharset)."\" />".
480
+                                "<input type=\"hidden\" name=\"proxyuser\" value=\"".htmlspecialchars($proxyuser, ENT_COMPAT, $inputcharset)."\" />".
481
+                                "<input type=\"hidden\" name=\"proxypwd\" value=\"".htmlspecialchars($proxypwd, ENT_COMPAT, $inputcharset)."\" />".
482
+                                "<input type=\"hidden\" name=\"responsecompression\" value=\"$responsecompression\" />".
483
+                                "<input type=\"hidden\" name=\"requestcompression\" value=\"$requestcompression\" />".
484
+                                "<input type=\"hidden\" name=\"clientcookies\" value=\"".htmlspecialchars($clientcookies, ENT_COMPAT, $inputcharset)."\" />".
485
+                                "<input type=\"hidden\" name=\"protocol\" value=\"$protocol\" />".
486
+                                "<input type=\"hidden\" name=\"timeout\" value=\"".htmlspecialchars($timeout, ENT_COMPAT, $inputcharset)."\" />".
487
+                                "<input type=\"hidden\" name=\"method\" value=\"".htmlspecialchars($method, ENT_COMPAT, $inputcharset)."\" />".
488
+                                "<input type=\"hidden\" name=\"methodsig\" value=\"".$i."\" />".
489
+                                "<input type=\"hidden\" name=\"methodpayload\" value=\"".htmlspecialchars($payload, ENT_COMPAT, $inputcharset)."\" />".
490
+                                "<input type=\"hidden\" name=\"altmethodpayload\" value=\"".htmlspecialchars($alt_payload, ENT_COMPAT, $inputcharset)."\" />".
491
+                                "<input type=\"hidden\" name=\"wstype\" value=\"$wstype\" />".
492
+                                "<input type=\"hidden\" name=\"action\" value=\"wrap\" />".
493
+                                "<input type=\"hidden\" name=\"run\" value=\"now\" />".
494 494
                                 "<input type=\"submit\" value=\"Generate method call stub code\" />";
495 495
                             echo "</form></td></tr>\n";
496 496
                         }
@@ -502,14 +502,14 @@  discard block
 block discarded – undo
502 502
                 case 'wrap':
503 503
                     $r1 = $resp[0]->value();
504 504
                     $r2 = $resp[1]->value();
505
-                    if ($r2->kindOf() != "array" || $r2->count() <= $methodsig) {
505
+                    if ($r2->kindOf() != "array" || $r2->count()<=$methodsig) {
506 506
                         echo "Error: signature unknown\n";
507 507
                     } else {
508 508
                         $mdesc = $r1->scalarval();
509 509
                         $encoder = new PhpXmlRpc\Encoder();
510 510
                         $msig = $encoder->decode($r2);
511 511
                         $msig = $msig[$methodsig];
512
-                        $proto = ($protocol == 1) ? 'http11' : ( $protocol == 2 ? 'https' : ( $protocol == 3 ? 'h2' : ( $protocol == 4 ? 'h2c' : '' ) ) );
512
+                        $proto = ($protocol == 1) ? 'http11' : ($protocol == 2 ? 'https' : ($protocol == 3 ? 'h2' : ($protocol == 4 ? 'h2c' : '')));
513 513
                         if ($proxy == '' && $username == '' && !$requestcompression && !$responsecompression &&
514 514
                             $clientcookies == '') {
515 515
                             $opts = 1; // simple client copy in stub code
@@ -530,12 +530,12 @@  discard block
 block discarded – undo
530 530
                             $client,
531 531
                             $method,
532 532
                             array('timeout' => $timeout, 'protocol' => $proto, 'simple_client_copy' => $opts, 'prefix' => $prefix, 'throw_on_fault' => true),
533
-                            str_replace('.', '_', $prefix . '_' . $method), $msig, $mdesc
533
+                            str_replace('.', '_', $prefix.'_'.$method), $msig, $mdesc
534 534
                         );
535 535
                         //if ($code)
536 536
                         //{
537 537
                         echo "<div id=\"phpcode\">\n";
538
-                        highlight_string("<?php\n" . $code['docstring'] . $code['source']);
538
+                        highlight_string("<?php\n".$code['docstring'].$code['source']);
539 539
                         echo "\n</div>";
540 540
                         //}
541 541
                         //else
@@ -546,7 +546,7 @@  discard block
 block discarded – undo
546 546
                     break;
547 547
 
548 548
                 case 'execute':
549
-                    echo '<div id="response"><h2>Response:</h2>' . htmlspecialchars($response->serialize()) . '</div>';
549
+                    echo '<div id="response"><h2>Response:</h2>'.htmlspecialchars($response->serialize()).'</div>';
550 550
                     break;
551 551
 
552 552
                 default: // give a warning
Please login to merge, or discard this patch.
src/Helper/Charset.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -91,14 +91,14 @@  discard block
 block discarded – undo
91 91
                 if (count($this->xml_iso88591_Entities['in'])) {
92 92
                     return;
93 93
                 }
94
-                for ($i = 0; $i < 32; $i++) {
94
+                for ($i = 0; $i<32; $i++) {
95 95
                     $this->xml_iso88591_Entities["in"][] = chr($i);
96 96
                     $this->xml_iso88591_Entities["out"][] = "&#{$i};";
97 97
                 }
98 98
 
99 99
                 /// @todo to be 'print safe', should we encode as well character 127 (DEL) ?
100 100
 
101
-                for ($i = 160; $i < 256; $i++) {
101
+                for ($i = 160; $i<256; $i++) {
102 102
                     $this->xml_iso88591_Entities["in"][] = chr($i);
103 103
                     $this->xml_iso88591_Entities["out"][] = "&#{$i};";
104 104
                 }
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
                 break;*/
127 127
 
128 128
             default:
129
-                throw new \Exception('Unsupported table: ' . $tableName);
129
+                throw new \Exception('Unsupported table: '.$tableName);
130 130
         }
131 131
     }
132 132
 
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
             $srcEncoding = 'UTF-8';
179 179
         }
180 180
 
181
-        $conversion = strtoupper($srcEncoding . '_' . $destEncoding);
181
+        $conversion = strtoupper($srcEncoding.'_'.$destEncoding);
182 182
 
183 183
         // list ordered with (expected) most common scenarios first
184 184
         switch ($conversion) {
@@ -196,20 +196,20 @@  discard block
 block discarded – undo
196 196
                 // NB: this will choke on invalid UTF-8, going most likely beyond EOF
197 197
                 $escapedData = '';
198 198
                 // be kind to users creating string xml-rpc values out of different php types
199
-                $data = (string)$data;
199
+                $data = (string) $data;
200 200
                 $ns = strlen($data);
201
-                for ($nn = 0; $nn < $ns; $nn++) {
201
+                for ($nn = 0; $nn<$ns; $nn++) {
202 202
                     $ch = $data[$nn];
203 203
                     $ii = ord($ch);
204 204
                     // 7 bits in 1 byte: 0bbbbbbb (127)
205
-                    if ($ii < 32) {
205
+                    if ($ii<32) {
206 206
                         if ($conversion == 'UTF-8_US-ASCII') {
207 207
                             $escapedData .= sprintf('&#%d;', $ii);
208 208
                         } else {
209 209
                             $escapedData .= $ch;
210 210
                         }
211 211
                     }
212
-                    else if ($ii < 128) {
212
+                    else if ($ii<128) {
213 213
                         /// @todo shall we replace this with a (supposedly) faster str_replace?
214 214
                         /// @todo to be 'print safe', should we encode as well character 127 (DEL) ?
215 215
                         switch ($ii) {
@@ -234,25 +234,25 @@  discard block
 block discarded – undo
234 234
                     } // 11 bits in 2 bytes: 110bbbbb 10bbbbbb (2047)
235 235
                     elseif ($ii >> 5 == 6) {
236 236
                         $b1 = ($ii & 31);
237
-                        $b2 = (ord($data[$nn + 1]) & 63);
238
-                        $ii = ($b1 * 64) + $b2;
237
+                        $b2 = (ord($data[$nn+1]) & 63);
238
+                        $ii = ($b1 * 64)+$b2;
239 239
                         $escapedData .= sprintf('&#%d;', $ii);
240 240
                         $nn += 1;
241 241
                     } // 16 bits in 3 bytes: 1110bbbb 10bbbbbb 10bbbbbb
242 242
                     elseif ($ii >> 4 == 14) {
243 243
                         $b1 = ($ii & 15);
244
-                        $b2 = (ord($data[$nn + 1]) & 63);
245
-                        $b3 = (ord($data[$nn + 2]) & 63);
246
-                        $ii = ((($b1 * 64) + $b2) * 64) + $b3;
244
+                        $b2 = (ord($data[$nn+1]) & 63);
245
+                        $b3 = (ord($data[$nn+2]) & 63);
246
+                        $ii = ((($b1 * 64)+$b2) * 64)+$b3;
247 247
                         $escapedData .= sprintf('&#%d;', $ii);
248 248
                         $nn += 2;
249 249
                     } // 21 bits in 4 bytes: 11110bbb 10bbbbbb 10bbbbbb 10bbbbbb
250 250
                     elseif ($ii >> 3 == 30) {
251 251
                         $b1 = ($ii & 7);
252
-                        $b2 = (ord($data[$nn + 1]) & 63);
253
-                        $b3 = (ord($data[$nn + 2]) & 63);
254
-                        $b4 = (ord($data[$nn + 3]) & 63);
255
-                        $ii = ((((($b1 * 64) + $b2) * 64) + $b3) * 64) + $b4;
252
+                        $b2 = (ord($data[$nn+1]) & 63);
253
+                        $b3 = (ord($data[$nn+2]) & 63);
254
+                        $b4 = (ord($data[$nn+3]) & 63);
255
+                        $ii = ((((($b1 * 64)+$b2) * 64)+$b3) * 64)+$b4;
256 256
                         $escapedData .= sprintf('&#%d;', $ii);
257 257
                         $nn += 3;
258 258
                     }
@@ -305,14 +305,14 @@  discard block
 block discarded – undo
305 305
                     // If src is UTF8, we run htmlspecialchars before converting to the target charset, as
306 306
                     // htmlspecialchars has limited charset support, but it groks utf8
307 307
                     if ($srcEncoding === 'UTF-8') {
308
-                        $data = htmlspecialchars($data,  defined('ENT_XML1') ? ENT_XML1 | ENT_QUOTES : ENT_QUOTES, 'UTF-8');
308
+                        $data = htmlspecialchars($data, defined('ENT_XML1') ? ENT_XML1 | ENT_QUOTES : ENT_QUOTES, 'UTF-8');
309 309
                     }
310 310
                     if ($srcEncoding !== $destEncoding) {
311 311
                         $data = mb_convert_encoding($data, str_replace('US-ASCII', 'ASCII', $destEncoding), str_replace('US-ASCII', 'ASCII', $srcEncoding));
312 312
                     }
313 313
                     if ($data === false) {
314 314
                         $escapedData = '';
315
-                        $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ": Converting from $srcEncoding to $destEncoding via mbstring: failed...");
315
+                        $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.": Converting from $srcEncoding to $destEncoding via mbstring: failed...");
316 316
                     } else {
317 317
                         if ($srcEncoding === 'UTF-8') {
318 318
                             $escapedData = $data;
@@ -322,7 +322,7 @@  discard block
 block discarded – undo
322 322
                     }
323 323
                 } else {
324 324
                     $escapedData = '';
325
-                    $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ": Converting from $srcEncoding to $destEncoding: not supported...");
325
+                    $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.": Converting from $srcEncoding to $destEncoding: not supported...");
326 326
                 }
327 327
         }
328 328
 
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
         if (function_exists('mb_list_encodings')) {
341 341
             $knownCharsets = array_unique(array_merge($knownCharsets, array_diff(mb_list_encodings(), array(
342 342
                 'pass', 'auto', 'wchar', 'BASE64', 'UUENCODE', 'ASCII', 'HTML-ENTITIES', 'Quoted-Printable',
343
-                '7bit','8bit', 'byte2be', 'byte2le', 'byte4be', 'byte4le'
343
+                '7bit', '8bit', 'byte2be', 'byte2le', 'byte4be', 'byte4le'
344 344
             ))));
345 345
         }
346 346
         return $knownCharsets;
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
             case 'iso88591':
395 395
                 return $this->xml_iso88591_Entities;
396 396
             default:
397
-                throw new \Exception('Unsupported charset: ' . $charset);
397
+                throw new \Exception('Unsupported charset: '.$charset);
398 398
         }
399 399
     }
400 400
 }
Please login to merge, or discard this patch.
src/Server.php 1 patch
Spacing   +50 added lines, -50 removed lines patch added patch discarded remove patch
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
      */
242 242
     public static function xmlrpc_debugmsg($msg)
243 243
     {
244
-        static::$_xmlrpc_debuginfo .= $msg . "\n";
244
+        static::$_xmlrpc_debuginfo .= $msg."\n";
245 245
     }
246 246
 
247 247
     /**
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
      */
254 254
     public static function error_occurred($msg)
255 255
     {
256
-        static::$_xmlrpcs_occurred_errors .= $msg . "\n";
256
+        static::$_xmlrpcs_occurred_errors .= $msg."\n";
257 257
     }
258 258
 
259 259
     /**
@@ -274,10 +274,10 @@  discard block
 block discarded – undo
274 274
         // user debug info should be encoded by the end user using the INTERNAL_ENCODING
275 275
         $out = '';
276 276
         if ($this->debug_info != '') {
277
-            $out .= "<!-- SERVER DEBUG INFO (BASE64 ENCODED):\n" . base64_encode($this->debug_info) . "\n-->\n";
277
+            $out .= "<!-- SERVER DEBUG INFO (BASE64 ENCODED):\n".base64_encode($this->debug_info)."\n-->\n";
278 278
         }
279 279
         if (static::$_xmlrpc_debuginfo != '') {
280
-            $out .= "<!-- DEBUG INFO:\n" . $this->getCharsetEncoder()->encodeEntities(str_replace('--', '_-', static::$_xmlrpc_debuginfo), PhpXmlRpc::$xmlrpc_internalencoding, $charsetEncoding) . "\n-->\n";
280
+            $out .= "<!-- DEBUG INFO:\n".$this->getCharsetEncoder()->encodeEntities(str_replace('--', '_-', static::$_xmlrpc_debuginfo), PhpXmlRpc::$xmlrpc_internalencoding, $charsetEncoding)."\n-->\n";
281 281
             // NB: a better solution MIGHT be to use CDATA, but we need to insert it
282 282
             // into return payload AFTER the beginning tag
283 283
             //$out .= "<![CDATA[ DEBUG INFO:\n\n" . str_replace(']]>', ']_]_>', static::$_xmlrpc_debuginfo) . "\n]]>\n";
@@ -306,8 +306,8 @@  discard block
 block discarded – undo
306 306
         $this->debug_info = '';
307 307
 
308 308
         // Save what we received, before parsing it
309
-        if ($this->debug > 1) {
310
-            $this->debugmsg("+++GOT+++\n" . $data . "\n+++END+++");
309
+        if ($this->debug>1) {
310
+            $this->debugmsg("+++GOT+++\n".$data."\n+++END+++");
311 311
         }
312 312
 
313 313
         $resp = $this->parseRequestHeaders($data, $reqCharset, $respCharset, $respEncoding);
@@ -324,14 +324,14 @@  discard block
 block discarded – undo
324 324
             $resp->raw_data = $rawData;
325 325
         }
326 326
 
327
-        if ($this->debug > 2 && static::$_xmlrpcs_occurred_errors != '') {
328
-            $this->debugmsg("+++PROCESSING ERRORS AND WARNINGS+++\n" .
329
-                static::$_xmlrpcs_occurred_errors . "+++END+++");
327
+        if ($this->debug>2 && static::$_xmlrpcs_occurred_errors != '') {
328
+            $this->debugmsg("+++PROCESSING ERRORS AND WARNINGS+++\n".
329
+                static::$_xmlrpcs_occurred_errors."+++END+++");
330 330
         }
331 331
 
332 332
         $payload = $this->xml_header($respCharset);
333
-        if ($this->debug > 0) {
334
-            $payload = $payload . $this->serializeDebug($respCharset);
333
+        if ($this->debug>0) {
334
+            $payload = $payload.$this->serializeDebug($respCharset);
335 335
         }
336 336
 
337 337
         // Do not create response serialization if it has already happened. Helps to build json magic
@@ -339,7 +339,7 @@  discard block
 block discarded – undo
339 339
         if (empty($resp->payload)) {
340 340
             $resp->serialize($respCharset);
341 341
         }
342
-        $payload = $payload . $resp->payload;
342
+        $payload = $payload.$resp->payload;
343 343
 
344 344
         if ($returnPayload) {
345 345
             return $payload;
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
         // if we get a warning/error that has output some text before here, then we cannot
349 349
         // add a new header. We cannot say we are sending xml, either...
350 350
         if (!headers_sent()) {
351
-            header('Content-Type: ' . $resp->content_type);
351
+            header('Content-Type: '.$resp->content_type);
352 352
             // we do not know if client actually told us an accepted charset, but if it did we have to tell it what we did
353 353
             header("Vary: Accept-Charset");
354 354
 
@@ -371,10 +371,10 @@  discard block
 block discarded – undo
371 371
             // Note that Apache/mod_php will add (and even alter!) the Content-Length header on its own, but only for
372 372
             // responses up to 8000 bytes
373 373
             if ($phpNoSelfCompress) {
374
-                header('Content-Length: ' . (int)strlen($payload));
374
+                header('Content-Length: '.(int) strlen($payload));
375 375
             }
376 376
         } else {
377
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': http headers already sent before response is fully generated. Check for php warning or error messages');
377
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': http headers already sent before response is fully generated. Check for php warning or error messages');
378 378
         }
379 379
 
380 380
         print $payload;
@@ -432,9 +432,9 @@  discard block
 block discarded – undo
432 432
             $numParams = count($in);
433 433
         }
434 434
         foreach ($sigs as $curSig) {
435
-            if (count($curSig) == $numParams + 1) {
435
+            if (count($curSig) == $numParams+1) {
436 436
                 $itsOK = 1;
437
-                for ($n = 0; $n < $numParams; $n++) {
437
+                for ($n = 0; $n<$numParams; $n++) {
438 438
                     if (is_object($in)) {
439 439
                         $p = $in->getParam($n);
440 440
                         if ($p->kindOf() == 'scalar') {
@@ -447,10 +447,10 @@  discard block
 block discarded – undo
447 447
                     }
448 448
 
449 449
                     // param index is $n+1, as first member of sig is return type
450
-                    if ($pt != $curSig[$n + 1] && $curSig[$n + 1] != Value::$xmlrpcValue) {
450
+                    if ($pt != $curSig[$n+1] && $curSig[$n+1] != Value::$xmlrpcValue) {
451 451
                         $itsOK = 0;
452
-                        $pno = $n + 1;
453
-                        $wanted = $curSig[$n + 1];
452
+                        $pno = $n+1;
453
+                        $wanted = $curSig[$n+1];
454 454
                         $got = $pt;
455 455
                         break;
456 456
                     }
@@ -477,10 +477,10 @@  discard block
 block discarded – undo
477 477
         // check if $_SERVER is populated: it might have been disabled via ini file
478 478
         // (this is true even when in CLI mode)
479 479
         if (count($_SERVER) == 0) {
480
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': cannot parse request headers as $_SERVER is not populated');
480
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': cannot parse request headers as $_SERVER is not populated');
481 481
         }
482 482
 
483
-        if ($this->debug > 1) {
483
+        if ($this->debug>1) {
484 484
             if (function_exists('getallheaders')) {
485 485
                 $this->debugmsg(''); // empty line
486 486
                 foreach (getallheaders() as $name => $val) {
@@ -505,13 +505,13 @@  discard block
 block discarded – undo
505 505
                 if (function_exists('gzinflate') && in_array($contentEncoding, $this->accepted_compression)) {
506 506
                     if ($contentEncoding == 'deflate' && $degzdata = @gzuncompress($data)) {
507 507
                         $data = $degzdata;
508
-                        if ($this->debug > 1) {
509
-                            $this->debugmsg("\n+++INFLATED REQUEST+++[" . strlen($data) . " chars]+++\n" . $data . "\n+++END+++");
508
+                        if ($this->debug>1) {
509
+                            $this->debugmsg("\n+++INFLATED REQUEST+++[".strlen($data)." chars]+++\n".$data."\n+++END+++");
510 510
                         }
511 511
                     } elseif ($contentEncoding == 'gzip' && $degzdata = @gzinflate(substr($data, 10))) {
512 512
                         $data = $degzdata;
513
-                        if ($this->debug > 1) {
514
-                            $this->debugmsg("+++INFLATED REQUEST+++[" . strlen($data) . " chars]+++\n" . $data . "\n+++END+++");
513
+                        if ($this->debug>1) {
514
+                            $this->debugmsg("+++INFLATED REQUEST+++[".strlen($data)." chars]+++\n".$data."\n+++END+++");
515 515
                         }
516 516
                     } else {
517 517
                         $r = new Response(0, PhpXmlRpc::$xmlrpcerr['server_decompress_fail'],
@@ -598,7 +598,7 @@  discard block
 block discarded – undo
598 598
                     if ($reqEncoding == 'ISO-8859-1') {
599 599
                         $data = utf8_encode($data);
600 600
                     } else {
601
-                        $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': unsupported charset encoding of received request: ' . $reqEncoding);
601
+                        $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': unsupported charset encoding of received request: '.$reqEncoding);
602 602
                     }
603 603
                 }
604 604
             }
@@ -625,7 +625,7 @@  discard block
 block discarded – undo
625 625
             preg_match('/^XML error ([0-9]+)/', $xmlRpcParser->_xh['isf_reason'], $matches);
626 626
             return new Response(
627 627
                 0,
628
-                PhpXmlRpc::$xmlrpcerrxml + (int)$matches[1],
628
+                PhpXmlRpc::$xmlrpcerrxml+(int) $matches[1],
629 629
                 $xmlRpcParser->_xh['isf_reason']);
630 630
         } elseif ($xmlRpcParser->_xh['isf']) {
631 631
             /// @todo separate better the various cases, as we have done in Request::parseResponse: invalid xml-rpc,
@@ -633,7 +633,7 @@  discard block
 block discarded – undo
633 633
             return new Response(
634 634
                 0,
635 635
                 PhpXmlRpc::$xmlrpcerr['invalid_request'],
636
-                PhpXmlRpc::$xmlrpcstr['invalid_request'] . ' ' . $xmlRpcParser->_xh['isf_reason']);
636
+                PhpXmlRpc::$xmlrpcstr['invalid_request'].' '.$xmlRpcParser->_xh['isf_reason']);
637 637
         } else {
638 638
             // small layering violation in favor of speed and memory usage: we should allow the 'execute' method handle
639 639
             // this, but in the most common scenario (xml-rpc values type server with some methods registered as phpvals)
@@ -643,20 +643,20 @@  discard block
 block discarded – undo
643 643
                     ($this->dmap[$xmlRpcParser->_xh['method']]['parameters_type'] != 'xmlrpcvals')
644 644
                 )
645 645
             ) {
646
-                if ($this->debug > 1) {
647
-                    $this->debugmsg("\n+++PARSED+++\n" . var_export($xmlRpcParser->_xh['params'], true) . "\n+++END+++");
646
+                if ($this->debug>1) {
647
+                    $this->debugmsg("\n+++PARSED+++\n".var_export($xmlRpcParser->_xh['params'], true)."\n+++END+++");
648 648
                 }
649 649
 
650 650
                 return $this->execute($xmlRpcParser->_xh['method'], $xmlRpcParser->_xh['params'], $xmlRpcParser->_xh['pt']);
651 651
             } else {
652 652
                 // build a Request object with data parsed from xml and add parameters in
653 653
                 $req = new Request($xmlRpcParser->_xh['method']);
654
-                for ($i = 0; $i < count($xmlRpcParser->_xh['params']); $i++) {
654
+                for ($i = 0; $i<count($xmlRpcParser->_xh['params']); $i++) {
655 655
                     $req->addParam($xmlRpcParser->_xh['params'][$i]);
656 656
                 }
657 657
 
658
-                if ($this->debug > 1) {
659
-                    $this->debugmsg("\n+++PARSED+++\n" . var_export($req, true) . "\n+++END+++");
658
+                if ($this->debug>1) {
659
+                    $this->debugmsg("\n+++PARSED+++\n".var_export($req, true)."\n+++END+++");
660 660
                 }
661 661
 
662 662
                 return $this->execute($req);
@@ -708,7 +708,7 @@  discard block
 block discarded – undo
708 708
                 return new Response(
709 709
                     0,
710 710
                     PhpXmlRpc::$xmlrpcerr['incorrect_params'],
711
-                    PhpXmlRpc::$xmlrpcstr['incorrect_params'] . ": {$errStr}"
711
+                    PhpXmlRpc::$xmlrpcstr['incorrect_params'].": {$errStr}"
712 712
                 );
713 713
             }
714 714
         }
@@ -723,7 +723,7 @@  discard block
 block discarded – undo
723 723
         // build string representation of function 'name'
724 724
         if (is_array($func)) {
725 725
             if (is_object($func[0])) {
726
-                $funcName = get_class($func[0]) . '->' . $func[1];
726
+                $funcName = get_class($func[0]).'->'.$func[1];
727 727
             } else {
728 728
                 $funcName = implode('::', $func);
729 729
             }
@@ -735,17 +735,17 @@  discard block
 block discarded – undo
735 735
 
736 736
         // verify that function to be invoked is in fact callable
737 737
         if (!is_callable($func)) {
738
-            $this->getLogger()->errorLog("XML-RPC: " . __METHOD__ . ": function '$funcName' registered as method handler is not callable");
738
+            $this->getLogger()->errorLog("XML-RPC: ".__METHOD__.": function '$funcName' registered as method handler is not callable");
739 739
             return new Response(
740 740
                 0,
741 741
                 PhpXmlRpc::$xmlrpcerr['server_error'],
742
-                PhpXmlRpc::$xmlrpcstr['server_error'] . ": no function matches method"
742
+                PhpXmlRpc::$xmlrpcstr['server_error'].": no function matches method"
743 743
             );
744 744
         }
745 745
 
746 746
         // If debug level is 3, we should catch all errors generated during processing of user function, and log them
747 747
         // as part of response
748
-        if ($this->debug > 2) {
748
+        if ($this->debug>2) {
749 749
             self::$_xmlrpcs_prev_ehandler = set_error_handler(array('\PhpXmlRpc\Server', '_xmlrpcs_errorHandler'));
750 750
         }
751 751
 
@@ -759,14 +759,14 @@  discard block
 block discarded – undo
759 759
                     $r = call_user_func($func, $req);
760 760
                 }
761 761
                 if (!is_a($r, 'PhpXmlRpc\Response')) {
762
-                    $this->getLogger()->errorLog("XML-RPC: " . __METHOD__ . ": function '$funcName' registered as method handler does not return an xmlrpc response object but a " . gettype($r));
762
+                    $this->getLogger()->errorLog("XML-RPC: ".__METHOD__.": function '$funcName' registered as method handler does not return an xmlrpc response object but a ".gettype($r));
763 763
                     if (is_a($r, 'PhpXmlRpc\Value')) {
764 764
                         $r = new Response($r);
765 765
                     } else {
766 766
                         $r = new Response(
767 767
                             0,
768 768
                             PhpXmlRpc::$xmlrpcerr['server_error'],
769
-                            PhpXmlRpc::$xmlrpcstr['server_error'] . ": function does not return xmlrpc response object"
769
+                            PhpXmlRpc::$xmlrpcstr['server_error'].": function does not return xmlrpc response object"
770 770
                         );
771 771
                     }
772 772
                 }
@@ -781,7 +781,7 @@  discard block
 block discarded – undo
781 781
                         $r = call_user_func_array($func, array($methodName, $params, $this->user_data));
782 782
                         // mimic EPI behaviour: if we get an array that looks like an error, make it an error response
783 783
                         if (is_array($r) && array_key_exists('faultCode', $r) && array_key_exists('faultString', $r)) {
784
-                            $r = new Response(0, (integer)$r['faultCode'], (string)$r['faultString']);
784
+                            $r = new Response(0, (integer) $r['faultCode'], (string) $r['faultString']);
785 785
                         } else {
786 786
                             // functions using EPI api should NOT return resp objects, so make sure we encode the
787 787
                             // return type correctly
@@ -805,7 +805,7 @@  discard block
 block discarded – undo
805 805
             // proper error-response
806 806
             switch ($this->exception_handling) {
807 807
                 case 2:
808
-                    if ($this->debug > 2) {
808
+                    if ($this->debug>2) {
809 809
                         if (self::$_xmlrpcs_prev_ehandler) {
810 810
                             set_error_handler(self::$_xmlrpcs_prev_ehandler);
811 811
                         } else {
@@ -828,7 +828,7 @@  discard block
 block discarded – undo
828 828
             // proper error-response
829 829
             switch ($this->exception_handling) {
830 830
                 case 2:
831
-                    if ($this->debug > 2) {
831
+                    if ($this->debug>2) {
832 832
                         if (self::$_xmlrpcs_prev_ehandler) {
833 833
                             set_error_handler(self::$_xmlrpcs_prev_ehandler);
834 834
                         } else {
@@ -848,7 +848,7 @@  discard block
 block discarded – undo
848 848
             }
849 849
         }
850 850
 
851
-        if ($this->debug > 2) {
851
+        if ($this->debug>2) {
852 852
             // note: restore the error handler we found before calling the user func, even if it has been changed
853 853
             // inside the func itself
854 854
             if (self::$_xmlrpcs_prev_ehandler) {
@@ -914,7 +914,7 @@  discard block
 block discarded – undo
914 914
      */
915 915
     protected function debugmsg($string)
916 916
     {
917
-        $this->debug_info .= $string . "\n";
917
+        $this->debug_info .= $string."\n";
918 918
     }
919 919
 
920 920
     /**
@@ -924,9 +924,9 @@  discard block
 block discarded – undo
924 924
     protected function xml_header($charsetEncoding = '')
925 925
     {
926 926
         if ($charsetEncoding != '') {
927
-            return "<?xml version=\"1.0\" encoding=\"$charsetEncoding\"?" . ">\n";
927
+            return "<?xml version=\"1.0\" encoding=\"$charsetEncoding\"?".">\n";
928 928
         } else {
929
-            return "<?xml version=\"1.0\"?" . ">\n";
929
+            return "<?xml version=\"1.0\"?".">\n";
930 930
         }
931 931
     }
932 932
 
@@ -1219,7 +1219,7 @@  discard block
 block discarded – undo
1219 1219
                 $i++; // for error message, we count params from 1
1220 1220
                 return static::_xmlrpcs_multicall_error(new Response(0,
1221 1221
                     PhpXmlRpc::$xmlrpcerr['incorrect_params'],
1222
-                    PhpXmlRpc::$xmlrpcstr['incorrect_params'] . ": probable xml error in param " . $i));
1222
+                    PhpXmlRpc::$xmlrpcstr['incorrect_params'].": probable xml error in param ".$i));
1223 1223
             }
1224 1224
         }
1225 1225
 
@@ -1300,7 +1300,7 @@  discard block
 block discarded – undo
1300 1300
             }
1301 1301
         } else {
1302 1302
             $numCalls = count($req);
1303
-            for ($i = 0; $i < $numCalls; $i++) {
1303
+            for ($i = 0; $i<$numCalls; $i++) {
1304 1304
                 $result[$i] = static::_xmlrpcs_multicall_do_call_phpvals($server, $req[$i]);
1305 1305
             }
1306 1306
         }
Please login to merge, or discard this patch.
src/Helper/XMLParser.php 1 patch
Spacing   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -175,12 +175,12 @@  discard block
 block discarded – undo
175 175
         foreach ($mergedOptions as $key => $val) {
176 176
             // q: can php be built without ctype? should we use a regexp?
177 177
             if (is_string($key) && !ctype_digit($key)) {
178
-                switch($key) {
178
+                switch ($key) {
179 179
                     case 'target_charset':
180 180
                         if (function_exists('mb_convert_encoding')) {
181 181
                             $this->current_parsing_options['target_charset'] = $val;
182 182
                         } else {
183
-                            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ": 'target_charset' option is unsupported without mbstring");
183
+                            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.": 'target_charset' option is unsupported without mbstring");
184 184
                         }
185 185
                         break;
186 186
 
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
                             //$this->_xh['isf'] = 4;
192 192
                             //$this->_xh['isf_reason'] = "Callback passed as 'methodname_callback' is not callable";
193 193
                             //return;
194
-                            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ": Callback passed as 'methodname_callback' is not callable");
194
+                            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.": Callback passed as 'methodname_callback' is not callable");
195 195
                         }
196 196
                         break;
197 197
 
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
                         break;
202 202
 
203 203
                     default:
204
-                        $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ": unsupported option: $key");
204
+                        $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.": unsupported option: $key");
205 205
                 }
206 206
                 unset($mergedOptions[$key]);
207 207
             }
@@ -244,10 +244,10 @@  discard block
 block discarded – undo
244 244
 
245 245
         try {
246 246
             // @see ticket #70 - we have to parse big xml docs in chunks to avoid errors
247
-            for ($offset = 0; $offset < $len; $offset += $this->maxChunkLength) {
247
+            for ($offset = 0; $offset<$len; $offset += $this->maxChunkLength) {
248 248
                 $chunk = substr($data, $offset, $this->maxChunkLength);
249 249
                 // error handling: xml not well formed
250
-                if (!xml_parse($parser, $chunk, $offset + $this->maxChunkLength >= $len)) {
250
+                if (!xml_parse($parser, $chunk, $offset+$this->maxChunkLength>=$len)) {
251 251
                     $errCode = xml_get_error_code($parser);
252 252
                     $errStr = sprintf('XML error %s: %s at line %d, column %d', $errCode, xml_error_string($errCode),
253 253
                         xml_get_current_line_number($parser), xml_get_current_column_number($parser));
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
                     break;
258 258
                 }
259 259
                 // no need to parse further if we already have a fatal error
260
-                if ($this->_xh['isf'] >= 2) {
260
+                if ($this->_xh['isf']>=2) {
261 261
                     break;
262 262
                 }
263 263
             }
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
     public function xmlrpc_se($parser, $name, $attrs, $acceptSingleVals = false)
290 290
     {
291 291
         // if invalid xml-rpc already detected, skip all processing
292
-        if ($this->_xh['isf'] >= 2) {
292
+        if ($this->_xh['isf']>=2) {
293 293
             return;
294 294
         }
295 295
 
@@ -312,7 +312,7 @@  discard block
 block discarded – undo
312 312
                 $this->_xh['rt'] = strtolower($name);
313 313
             } else {
314 314
                 $this->_xh['isf'] = 2;
315
-                $this->_xh['isf_reason'] = 'missing top level xmlrpc element. Found: ' . $name;
315
+                $this->_xh['isf_reason'] = 'missing top level xmlrpc element. Found: '.$name;
316 316
 
317 317
                 return;
318 318
             }
@@ -414,7 +414,7 @@  discard block
 block discarded – undo
414 414
 
415 415
             case 'MEMBER':
416 416
                 // set member name to null, in case we do not find in the xml later on
417
-                $this->_xh['valuestack'][count($this->_xh['valuestack']) - 1]['name'] = '';
417
+                $this->_xh['valuestack'][count($this->_xh['valuestack'])-1]['name'] = '';
418 418
                 //$this->_xh['ac']='';
419 419
                 // Drop trough intentionally
420 420
 
@@ -486,7 +486,7 @@  discard block
 block discarded – undo
486 486
      */
487 487
     public function xmlrpc_ee($parser, $name, $rebuildXmlrpcvals = 1)
488 488
     {
489
-        if ($this->_xh['isf'] >= 2) {
489
+        if ($this->_xh['isf']>=2) {
490 490
             return;
491 491
 
492 492
         }
@@ -508,7 +508,7 @@  discard block
 block discarded – undo
508 508
                     $this->_xh['value'] = mb_convert_encoding($this->_xh['value'], $this->current_parsing_options['target_charset'], 'UTF-8');
509 509
                 }
510 510
 
511
-                if ($rebuildXmlrpcvals > 0) {
511
+                if ($rebuildXmlrpcvals>0) {
512 512
                     // build the xml-rpc val out of the data received, and substitute it
513 513
                     $temp = new Value($this->_xh['value'], $this->_xh['vt']);
514 514
                     // in case we got info about underlying php class, save it in the object we're rebuilding
@@ -516,15 +516,15 @@  discard block
 block discarded – undo
516 516
                         $temp->_php_class = $this->_xh['php_class'];
517 517
                     }
518 518
                     $this->_xh['value'] = $temp;
519
-                } elseif ($rebuildXmlrpcvals < 0) {
519
+                } elseif ($rebuildXmlrpcvals<0) {
520 520
                     if ($this->_xh['vt'] == Value::$xmlrpcDateTime) {
521
-                        $this->_xh['value'] = (object)array(
521
+                        $this->_xh['value'] = (object) array(
522 522
                             'xmlrpc_type' => 'datetime',
523 523
                             'scalar' => $this->_xh['value'],
524 524
                             'timestamp' => \PhpXmlRpc\Helper\Date::iso8601Decode($this->_xh['value'])
525 525
                         );
526 526
                     } elseif ($this->_xh['vt'] == Value::$xmlrpcBase64) {
527
-                        $this->_xh['value'] = (object)array(
527
+                        $this->_xh['value'] = (object) array(
528 528
                             'xmlrpc_type' => 'base64',
529 529
                             'scalar' => $this->_xh['value']
530 530
                         );
@@ -539,8 +539,8 @@  discard block
 block discarded – undo
539 539
                 // check if we are inside an array or struct:
540 540
                 // if value just built is inside an array, let's move it into array on the stack
541 541
                 $vscount = count($this->_xh['valuestack']);
542
-                if ($vscount && $this->_xh['valuestack'][$vscount - 1]['type'] == 'ARRAY') {
543
-                    $this->_xh['valuestack'][$vscount - 1]['values'][] = $this->_xh['value'];
542
+                if ($vscount && $this->_xh['valuestack'][$vscount-1]['type'] == 'ARRAY') {
543
+                    $this->_xh['valuestack'][$vscount-1]['values'][] = $this->_xh['value'];
544 544
                 }
545 545
                 break;
546 546
 
@@ -560,11 +560,11 @@  discard block
 block discarded – undo
560 560
                     $this->_xh['value'] = $this->_xh['ac'];
561 561
                 } elseif ($name == 'DATETIME.ISO8601') {
562 562
                     if (!preg_match(PhpXmlRpc::$xmlrpc_datetime_format, $this->_xh['ac'])) {
563
-                        $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': invalid value received in DATETIME: ' . $this->_xh['ac']);
563
+                        $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': invalid value received in DATETIME: '.$this->_xh['ac']);
564 564
                     }
565 565
                     $this->_xh['vt'] = Value::$xmlrpcDateTime;
566 566
                     if ($this->current_parsing_options['xmlrpc_return_datetimes']) {
567
-                        $this->_xh['value'] =  new \DateTime($this->_xh['ac']);
567
+                        $this->_xh['value'] = new \DateTime($this->_xh['ac']);
568 568
                     } else {
569 569
                         $this->_xh['value'] = $this->_xh['ac'];
570 570
                     }
@@ -581,7 +581,7 @@  discard block
 block discarded – undo
581 581
                     } else {
582 582
                         // log if receiving something strange, even though we set the value to false anyway
583 583
                         if ($this->_xh['ac'] != '0' && strcasecmp($this->_xh['ac'], 'false') != 0) {
584
-                            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': invalid value received in BOOLEAN: ' . $this->_xh['ac']);
584
+                            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': invalid value received in BOOLEAN: '.$this->_xh['ac']);
585 585
                         }
586 586
                         $this->_xh['value'] = false;
587 587
                     }
@@ -591,38 +591,38 @@  discard block
 block discarded – undo
591 591
                     // NOTE: regexp could be much stricter than this...
592 592
                     if (!preg_match('/^[+-eE0123456789 \t.]+$/', $this->_xh['ac'])) {
593 593
                         /// @todo: find a better way of throwing an error than this!
594
-                        $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': non numeric value received in DOUBLE: ' . $this->_xh['ac']);
594
+                        $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': non numeric value received in DOUBLE: '.$this->_xh['ac']);
595 595
                         $this->_xh['value'] = 'ERROR_NON_NUMERIC_FOUND';
596 596
                     } else {
597 597
                         // it's ok, add it on
598
-                        $this->_xh['value'] = (double)$this->_xh['ac'];
598
+                        $this->_xh['value'] = (double) $this->_xh['ac'];
599 599
                     }
600 600
                 } else {
601 601
                     // we have an I4/I8/INT
602 602
                     // we must check that only 0123456789-<space> are characters here
603 603
                     if (!preg_match('/^[+-]?[0123456789 \t]+$/', $this->_xh['ac'])) {
604 604
                         /// @todo find a better way of throwing an error than this!
605
-                        $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': non numeric value received in INT: ' . $this->_xh['ac']);
605
+                        $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': non numeric value received in INT: '.$this->_xh['ac']);
606 606
                         $this->_xh['value'] = 'ERROR_NON_NUMERIC_FOUND';
607 607
                     } else {
608 608
                         // it's ok, add it on
609
-                        $this->_xh['value'] = (int)$this->_xh['ac'];
609
+                        $this->_xh['value'] = (int) $this->_xh['ac'];
610 610
                     }
611 611
                 }
612 612
                 $this->_xh['lv'] = 3; // indicate we've found a value
613 613
                 break;
614 614
 
615 615
             case 'NAME':
616
-                $this->_xh['valuestack'][count($this->_xh['valuestack']) - 1]['name'] = $this->_xh['ac'];
616
+                $this->_xh['valuestack'][count($this->_xh['valuestack'])-1]['name'] = $this->_xh['ac'];
617 617
                 break;
618 618
 
619 619
             case 'MEMBER':
620 620
                 // add to array in the stack the last element built, unless no VALUE was found
621 621
                 if ($this->_xh['vt']) {
622 622
                     $vscount = count($this->_xh['valuestack']);
623
-                    $this->_xh['valuestack'][$vscount - 1]['values'][$this->_xh['valuestack'][$vscount - 1]['name']] = $this->_xh['value'];
623
+                    $this->_xh['valuestack'][$vscount-1]['values'][$this->_xh['valuestack'][$vscount-1]['name']] = $this->_xh['value'];
624 624
                 } else {
625
-                    $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': missing VALUE inside STRUCT in received xml');
625
+                    $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': missing VALUE inside STRUCT in received xml');
626 626
                 }
627 627
                 break;
628 628
 
@@ -647,7 +647,7 @@  discard block
 block discarded – undo
647 647
                     $this->_xh['params'][] = $this->_xh['value'];
648 648
                     $this->_xh['pt'][] = $this->_xh['vt'];
649 649
                 } else {
650
-                    $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': missing VALUE inside PARAM in received xml');
650
+                    $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': missing VALUE inside PARAM in received xml');
651 651
                 }
652 652
                 break;
653 653
 
@@ -722,7 +722,7 @@  discard block
 block discarded – undo
722 722
     public function xmlrpc_cd($parser, $data)
723 723
     {
724 724
         // skip processing if xml fault already detected
725
-        if ($this->_xh['isf'] >= 2) {
725
+        if ($this->_xh['isf']>=2) {
726 726
             return;
727 727
         }
728 728
 
@@ -744,7 +744,7 @@  discard block
 block discarded – undo
744 744
     public function xmlrpc_dh($parser, $data)
745 745
     {
746 746
         // skip processing if xml fault already detected
747
-        if ($this->_xh['isf'] >= 2) {
747
+        if ($this->_xh['isf']>=2) {
748 748
             return;
749 749
         }
750 750
 
@@ -818,8 +818,8 @@  discard block
 block discarded – undo
818 818
         // Details:
819 819
         // SPACE:         (#x20 | #x9 | #xD | #xA)+ === [ \x9\xD\xA]+
820 820
         // EQ:            SPACE?=SPACE? === [ \x9\xD\xA]*=[ \x9\xD\xA]*
821
-        if (preg_match('/^<\?xml\s+version\s*=\s*' . "((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))" .
822
-            '\s+encoding\s*=\s*' . "((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
821
+        if (preg_match('/^<\?xml\s+version\s*=\s*'."((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))".
822
+            '\s+encoding\s*=\s*'."((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
823 823
             $xmlChunk, $matches)) {
824 824
             return strtoupper(substr($matches[2], 1, -1));
825 825
         }
@@ -837,7 +837,7 @@  discard block
 block discarded – undo
837 837
             // NB: mb_detect likes to call it ascii, xml parser likes to call it US_ASCII...
838 838
             // IANA also likes better US-ASCII, so go with it
839 839
             if ($enc == 'ASCII') {
840
-                $enc = 'US-' . $enc;
840
+                $enc = 'US-'.$enc;
841 841
             }
842 842
 
843 843
             return $enc;
@@ -874,8 +874,8 @@  discard block
 block discarded – undo
874 874
         // Details:
875 875
         // SPACE:         (#x20 | #x9 | #xD | #xA)+ === [ \x9\xD\xA]+
876 876
         // EQ:            SPACE?=SPACE? === [ \x9\xD\xA]*=[ \x9\xD\xA]*
877
-        if (preg_match('/^<\?xml\s+version\s*=\s*' . "((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))" .
878
-            '\s+encoding\s*=\s*' . "((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
877
+        if (preg_match('/^<\?xml\s+version\s*=\s*'."((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))".
878
+            '\s+encoding\s*=\s*'."((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
879 879
             $xmlChunk)) {
880 880
             return true;
881 881
         }
@@ -895,7 +895,7 @@  discard block
 block discarded – undo
895 895
                 break;
896 896
             default:
897 897
                 $trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
898
-                trigger_error('Undefined property via __set(): ' . $name . ' in ' . $trace[0]['file'] . ' on line ' . $trace[0]['line'], E_USER_WARNING);
898
+                trigger_error('Undefined property via __set(): '.$name.' in '.$trace[0]['file'].' on line '.$trace[0]['line'], E_USER_WARNING);
899 899
         }
900 900
     }
901 901
 
@@ -919,7 +919,7 @@  discard block
 block discarded – undo
919 919
                 break;
920 920
             default:
921 921
                 $trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
922
-                trigger_error('Undefined property via __unset(): ' . $name . ' in ' . $trace[0]['file'] . ' on line ' . $trace[0]['line'], E_USER_WARNING);
922
+                trigger_error('Undefined property via __unset(): '.$name.' in '.$trace[0]['file'].' on line '.$trace[0]['line'], E_USER_WARNING);
923 923
         }
924 924
     }
925 925
 }
Please login to merge, or discard this patch.
demo/server/codegen.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-require_once __DIR__ . "/_prepend.php";
2
+require_once __DIR__."/_prepend.php";
3 3
 
4 4
 require_once __DIR__.'/methodProviders/CommentManager.php';
5 5
 
@@ -23,19 +23,19 @@  discard block
 block discarded – undo
23 23
 // generate a file with a class definition
24 24
 
25 25
 // the generated code does not have an autoloader included - we need to add in one
26
-$autoloader = __DIR__ . "/_prepend.php";
26
+$autoloader = __DIR__."/_prepend.php";
27 27
 
28 28
 file_put_contents($targetClassFile,
29
-    "<?php\n\n" .
30
-    "require_once '$autoloader';\n\n" .
29
+    "<?php\n\n".
30
+    "require_once '$autoloader';\n\n".
31 31
     "class MyServerClass\n{\n\n"
32 32
 ) || die('uh oh');
33 33
 
34 34
 // we mangle a bit the code we get from wrapPhpClass to generate a php class instead of a bunch of functions
35 35
 
36
-foreach($code as $methodName => $methodDef) {
37
-    file_put_contents($targetClassFile, '  ' . str_replace(array('function ', "\n"), array('public static function ', "\n  "), $methodDef['source']) . "\n\n", FILE_APPEND) || die('uh oh');
38
-    $code[$methodName]['function'] = 'MyServerClass::' . $methodDef['function'];
36
+foreach ($code as $methodName => $methodDef) {
37
+    file_put_contents($targetClassFile, '  '.str_replace(array('function ', "\n"), array('public static function ', "\n  "), $methodDef['source'])."\n\n", FILE_APPEND) || die('uh oh');
38
+    $code[$methodName]['function'] = 'MyServerClass::'.$methodDef['function'];
39 39
     unset($code[$methodName]['source']);
40 40
 }
41 41
 file_put_contents($targetClassFile, "}\n", FILE_APPEND) || die('uh oh');
@@ -43,11 +43,11 @@  discard block
 block discarded – undo
43 43
 // generate a separate file with the xml-rpc server instantiation and its dispatch map
44 44
 
45 45
 file_put_contents($targetServerFile,
46
-    "<?php\n\n" .
46
+    "<?php\n\n".
47 47
 
48
-    "require_once '$autoloader';\n\n" .
48
+    "require_once '$autoloader';\n\n".
49 49
 
50
-    "require_once '$targetClassFile';\n\n" .
50
+    "require_once '$targetClassFile';\n\n".
51 51
 
52 52
     // NB: since we are running the generated code within the same script, the existing CommentManager instance will be
53 53
     // available for usage by the methods of MyServerClass, as we keep a reference to them within the variable Wrapper::$objHolder
@@ -57,12 +57,12 @@  discard block
 block discarded – undo
57 57
     //     Wrapper::holdObject('xmlrpc_CommentManager_addComment', $cm);
58 58
     //     Wrapper::holdObject('xmlrpc_CommentManager_getComments', $cm);
59 59
 
60
-    '$dm = ' . var_export($code, true) . ";\n" .
61
-    '$s = new \PhpXmlRpc\Server($dm, false);' . "\n" .
62
-    '// NB: do not leave these 2 debug lines enabled on publicly accessible servers!' . "\n" .
63
-    '$s->setDebug(2);' . "\n" .
64
-    '$s->exception_handling = 1;' . "\n" .
65
-    '$s->service();' . "\n"
60
+    '$dm = '.var_export($code, true).";\n".
61
+    '$s = new \PhpXmlRpc\Server($dm, false);'."\n".
62
+    '// NB: do not leave these 2 debug lines enabled on publicly accessible servers!'."\n".
63
+    '$s->setDebug(2);'."\n".
64
+    '$s->exception_handling = 1;'."\n".
65
+    '$s->service();'."\n"
66 66
 ) || die('uh oh');
67 67
 
68 68
 // test that everything worked by running it in realtime (note that this will return an xml-rpc error message if run
Please login to merge, or discard this patch.
src/Wrapper.php 1 patch
Spacing   +96 added lines, -96 removed lines patch added patch discarded remove patch
@@ -190,20 +190,20 @@  discard block
 block discarded – undo
190 190
             $callable = explode('::', $callable);
191 191
         }
192 192
         if (is_array($callable)) {
193
-            if (count($callable) < 2 || (!is_string($callable[0]) && !is_object($callable[0]))) {
194
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': syntax for function to be wrapped is wrong');
193
+            if (count($callable)<2 || (!is_string($callable[0]) && !is_object($callable[0]))) {
194
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': syntax for function to be wrapped is wrong');
195 195
                 return false;
196 196
             }
197 197
             if (is_string($callable[0])) {
198 198
                 $plainFuncName = implode('::', $callable);
199 199
             } elseif (is_object($callable[0])) {
200
-                $plainFuncName = get_class($callable[0]) . '->' . $callable[1];
200
+                $plainFuncName = get_class($callable[0]).'->'.$callable[1];
201 201
             }
202 202
             $exists = method_exists($callable[0], $callable[1]);
203 203
         } else if ($callable instanceof \Closure) {
204 204
             // we do not support creating code which wraps closures, as php does not allow to serialize them
205 205
             if (!$buildIt) {
206
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': a closure can not be wrapped in generated source code');
206
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': a closure can not be wrapped in generated source code');
207 207
                 return false;
208 208
             }
209 209
 
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
         }
216 216
 
217 217
         if (!$exists) {
218
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': function to be wrapped is not defined: ' . $plainFuncName);
218
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': function to be wrapped is not defined: '.$plainFuncName);
219 219
             return false;
220 220
         }
221 221
 
@@ -259,23 +259,23 @@  discard block
 block discarded – undo
259 259
         if (is_array($callable)) {
260 260
             $func = new \ReflectionMethod($callable[0], $callable[1]);
261 261
             if ($func->isPrivate()) {
262
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': method to be wrapped is private: ' . $plainFuncName);
262
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': method to be wrapped is private: '.$plainFuncName);
263 263
                 return false;
264 264
             }
265 265
             if ($func->isProtected()) {
266
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': method to be wrapped is protected: ' . $plainFuncName);
266
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': method to be wrapped is protected: '.$plainFuncName);
267 267
                 return false;
268 268
             }
269 269
             if ($func->isConstructor()) {
270
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': method to be wrapped is the constructor: ' . $plainFuncName);
270
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': method to be wrapped is the constructor: '.$plainFuncName);
271 271
                 return false;
272 272
             }
273 273
             if ($func->isDestructor()) {
274
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': method to be wrapped is the destructor: ' . $plainFuncName);
274
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': method to be wrapped is the destructor: '.$plainFuncName);
275 275
                 return false;
276 276
             }
277 277
             if ($func->isAbstract()) {
278
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': method to be wrapped is abstract: ' . $plainFuncName);
278
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': method to be wrapped is abstract: '.$plainFuncName);
279 279
                 return false;
280 280
             }
281 281
             /// @todo add more checks for static vs. nonstatic?
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
         if ($func->isInternal()) {
286 286
             /// @todo from PHP 5.1.0 onward, we should be able to use invokeargs instead of getparameters to fully
287 287
             ///       reflect internal php functions
288
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': function to be wrapped is internal: ' . $plainFuncName);
288
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': function to be wrapped is internal: '.$plainFuncName);
289 289
             return false;
290 290
         }
291 291
 
@@ -337,7 +337,7 @@  discard block
 block discarded – undo
337 337
         $i = 0;
338 338
         foreach ($func->getParameters() as $paramObj) {
339 339
             $params[$i] = array();
340
-            $params[$i]['name'] = '$' . $paramObj->getName();
340
+            $params[$i]['name'] = '$'.$paramObj->getName();
341 341
             $params[$i]['isoptional'] = $paramObj->isOptional();
342 342
             $i++;
343 343
         }
@@ -401,7 +401,7 @@  discard block
 block discarded – undo
401 401
             // build a signature
402 402
             $sig = array($this->php2XmlrpcType($funcDesc['returns']));
403 403
             $pSig = array($funcDesc['returnsDocs']);
404
-            for ($i = 0; $i < count($pars); $i++) {
404
+            for ($i = 0; $i<count($pars); $i++) {
405 405
                 $name = strtolower($funcDesc['params'][$i]['name']);
406 406
                 if (isset($funcDesc['paramDocs'][$name]['type'])) {
407 407
                     $sig[] = $this->php2XmlrpcType($funcDesc['paramDocs'][$name]['type']);
@@ -457,7 +457,7 @@  discard block
 block discarded – undo
457 457
                 }
458 458
             }
459 459
             $numPars = $req->getNumParams();
460
-            if ($numPars < $minPars || $numPars > $maxPars) {
460
+            if ($numPars<$minPars || $numPars>$maxPars) {
461 461
                 return new $responseClass(0, 3, 'Incorrect parameters passed to method');
462 462
             }
463 463
 
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
 
471 471
             $result = call_user_func_array($callable, $params);
472 472
 
473
-            if (! is_a($result, $responseClass)) {
473
+            if (!is_a($result, $responseClass)) {
474 474
                 // q: why not do the same for int, float, bool, string?
475 475
                 if ($funcDesc['returns'] == Value::$xmlrpcDateTime || $funcDesc['returns'] == Value::$xmlrpcBase64) {
476 476
                     $result = new $valueClass($result, $funcDesc['returns']);
@@ -509,9 +509,9 @@  discard block
 block discarded – undo
509 509
         if ($newFuncName == '') {
510 510
             if (is_array($callable)) {
511 511
                 if (is_string($callable[0])) {
512
-                    $xmlrpcFuncName = "{$prefix}_" . implode('_', $callable);
512
+                    $xmlrpcFuncName = "{$prefix}_".implode('_', $callable);
513 513
                 } else {
514
-                    $xmlrpcFuncName = "{$prefix}_" . get_class($callable[0]) . '_' . $callable[1];
514
+                    $xmlrpcFuncName = "{$prefix}_".get_class($callable[0]).'_'.$callable[1];
515 515
                 }
516 516
             } else {
517 517
                 if ($callable instanceof \Closure) {
@@ -543,9 +543,9 @@  discard block
 block discarded – undo
543 543
      */
544 544
     protected function buildWrapFunctionSource($callable, $newFuncName, $extraOptions, $plainFuncName, $funcDesc)
545 545
     {
546
-        $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool)$extraOptions['encode_nulls'] : false;
547
-        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
548
-        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
546
+        $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool) $extraOptions['encode_nulls'] : false;
547
+        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool) $extraOptions['encode_php_objs'] : false;
548
+        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool) $extraOptions['decode_php_objs'] : false;
549 549
         $catchWarnings = isset($extraOptions['suppress_warnings']) && $extraOptions['suppress_warnings'] ? '@' : '';
550 550
 
551 551
         $i = 0;
@@ -580,9 +580,9 @@  discard block
 block discarded – undo
580 580
         // build body of new function
581 581
 
582 582
         $innerCode = "  \$paramCount = \$req->getNumParams();\n";
583
-        $innerCode .= "  if (\$paramCount < $minPars || \$paramCount > $maxPars) return new " . static::$namespace . "Response(0, " . PhpXmlRpc::$xmlrpcerr['incorrect_params'] . ", '" . PhpXmlRpc::$xmlrpcstr['incorrect_params'] . "');\n";
583
+        $innerCode .= "  if (\$paramCount < $minPars || \$paramCount > $maxPars) return new ".static::$namespace."Response(0, ".PhpXmlRpc::$xmlrpcerr['incorrect_params'].", '".PhpXmlRpc::$xmlrpcstr['incorrect_params']."');\n";
584 584
 
585
-        $innerCode .= "  \$encoder = new " . static::$namespace . "Encoder();\n";
585
+        $innerCode .= "  \$encoder = new ".static::$namespace."Encoder();\n";
586 586
         if ($decodePhpObjects) {
587 587
             $innerCode .= "  \$params = \$encoder->decode(\$req, array('decode_php_objs'));\n";
588 588
         } else {
@@ -595,23 +595,23 @@  discard block
 block discarded – undo
595 595
             static::holdObject($newFuncName, $callable[0]);
596 596
             $class = get_class($callable[0]);
597 597
             if ($class[0] !== '\\') {
598
-                $class = '\\' . $class;
598
+                $class = '\\'.$class;
599 599
             }
600 600
             $innerCode .= "  /// @var $class \$obj\n";
601 601
             $innerCode .= "  \$obj = PhpXmlRpc\\Wrapper::getHeldObject('$newFuncName');\n";
602
-            $realFuncName = '$obj->' . $callable[1];
602
+            $realFuncName = '$obj->'.$callable[1];
603 603
         } else {
604 604
             $realFuncName = $plainFuncName;
605 605
         }
606 606
         foreach ($parsVariations as $i => $pars) {
607
-            $innerCode .= "  if (\$paramCount == " . count($pars) . ") \$retVal = {$catchWarnings}$realFuncName(" . implode(',', $pars) . ");\n";
608
-            if ($i < (count($parsVariations) - 1))
607
+            $innerCode .= "  if (\$paramCount == ".count($pars).") \$retVal = {$catchWarnings}$realFuncName(".implode(',', $pars).");\n";
608
+            if ($i<(count($parsVariations)-1))
609 609
                 $innerCode .= "  else\n";
610 610
         }
611
-        $innerCode .= "  if (is_a(\$retVal, '" . static::$namespace . "Response'))\n    return \$retVal;\n  else\n";
611
+        $innerCode .= "  if (is_a(\$retVal, '".static::$namespace."Response'))\n    return \$retVal;\n  else\n";
612 612
         /// q: why not do the same for int, float, bool, string?
613 613
         if ($funcDesc['returns'] == Value::$xmlrpcDateTime || $funcDesc['returns'] == Value::$xmlrpcBase64) {
614
-            $innerCode .= "    return new " . static::$namespace . "Response(new " . static::$namespace . "Value(\$retVal, '{$funcDesc['returns']}'));";
614
+            $innerCode .= "    return new ".static::$namespace."Response(new ".static::$namespace."Value(\$retVal, '{$funcDesc['returns']}'));";
615 615
         } else {
616 616
             $encodeOptions = array();
617 617
             if ($encodeNulls) {
@@ -622,18 +622,18 @@  discard block
 block discarded – undo
622 622
             }
623 623
 
624 624
             if ($encodeOptions) {
625
-                $innerCode .= "    return new " . static::$namespace . "Response(\$encoder->encode(\$retVal, array('" .
626
-                    implode("', '", $encodeOptions) . "')));";
625
+                $innerCode .= "    return new ".static::$namespace."Response(\$encoder->encode(\$retVal, array('".
626
+                    implode("', '", $encodeOptions)."')));";
627 627
             } else {
628
-                $innerCode .= "    return new " . static::$namespace . "Response(\$encoder->encode(\$retVal));";
628
+                $innerCode .= "    return new ".static::$namespace."Response(\$encoder->encode(\$retVal));";
629 629
             }
630 630
         }
631 631
         // shall we exclude functions returning by ref?
632 632
         // if ($func->returnsReference())
633 633
         //     return false;
634 634
 
635
-        $code = "/**\n * @param \PhpXmlRpc\Request \$req\n * @return \PhpXmlRpc\Response\n * @throws \\Exception\n */\n" .
636
-            "function $newFuncName(\$req)\n{\n" . $innerCode . "\n}";
635
+        $code = "/**\n * @param \PhpXmlRpc\Request \$req\n * @return \PhpXmlRpc\Response\n * @throws \\Exception\n */\n".
636
+            "function $newFuncName(\$req)\n{\n".$innerCode."\n}";
637 637
 
638 638
         return $code;
639 639
     }
@@ -687,7 +687,7 @@  discard block
 block discarded – undo
687 687
     protected function generateMethodNameForClassMethod($className, $classMethod, $extraOptions = array())
688 688
     {
689 689
         if (isset($extraOptions['replace_class_name']) && $extraOptions['replace_class_name']) {
690
-            return (isset($extraOptions['prefix']) ?  $extraOptions['prefix'] : '') . $classMethod;
690
+            return (isset($extraOptions['prefix']) ? $extraOptions['prefix'] : '').$classMethod;
691 691
         }
692 692
 
693 693
         if (is_object($className)) {
@@ -695,7 +695,7 @@  discard block
 block discarded – undo
695 695
         } else {
696 696
             $realClassName = $className;
697 697
         }
698
-        return (isset($extraOptions['prefix']) ?  $extraOptions['prefix'] : '') . "$realClassName.$classMethod";
698
+        return (isset($extraOptions['prefix']) ? $extraOptions['prefix'] : '')."$realClassName.$classMethod";
699 699
     }
700 700
 
701 701
     /**
@@ -778,21 +778,21 @@  discard block
 block discarded – undo
778 778
      */
779 779
     protected function retrieveMethodSignature($client, $methodName, array $extraOptions = array())
780 780
     {
781
-        $reqClass = static::$namespace . 'Request';
782
-        $valClass = static::$namespace . 'Value';
783
-        $decoderClass = static::$namespace . 'Encoder';
781
+        $reqClass = static::$namespace.'Request';
782
+        $valClass = static::$namespace.'Value';
783
+        $decoderClass = static::$namespace.'Encoder';
784 784
 
785 785
         $debug = isset($extraOptions['debug']) ? ($extraOptions['debug']) : 0;
786
-        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
786
+        $timeout = isset($extraOptions['timeout']) ? (int) $extraOptions['timeout'] : 0;
787 787
         $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
788
-        $sigNum = isset($extraOptions['signum']) ? (int)$extraOptions['signum'] : 0;
788
+        $sigNum = isset($extraOptions['signum']) ? (int) $extraOptions['signum'] : 0;
789 789
 
790 790
         $req = new $reqClass('system.methodSignature');
791 791
         $req->addparam(new $valClass($methodName));
792 792
         $client->setDebug($debug);
793 793
         $response = $client->send($req, $timeout, $protocol);
794 794
         if ($response->faultCode()) {
795
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': could not retrieve method signature from remote server for method ' . $methodName);
795
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': could not retrieve method signature from remote server for method '.$methodName);
796 796
             return false;
797 797
         }
798 798
 
@@ -803,8 +803,8 @@  discard block
 block discarded – undo
803 803
             $mSig = $decoder->decode($mSig);
804 804
         }
805 805
 
806
-        if (!is_array($mSig) || count($mSig) <= $sigNum) {
807
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': could not retrieve method signature nr.' . $sigNum . ' from remote server for method ' . $methodName);
806
+        if (!is_array($mSig) || count($mSig)<=$sigNum) {
807
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': could not retrieve method signature nr.'.$sigNum.' from remote server for method '.$methodName);
808 808
             return false;
809 809
         }
810 810
 
@@ -819,11 +819,11 @@  discard block
 block discarded – undo
819 819
      */
820 820
     protected function retrieveMethodHelp($client, $methodName, array $extraOptions = array())
821 821
     {
822
-        $reqClass = static::$namespace . 'Request';
823
-        $valClass = static::$namespace . 'Value';
822
+        $reqClass = static::$namespace.'Request';
823
+        $valClass = static::$namespace.'Value';
824 824
 
825 825
         $debug = isset($extraOptions['debug']) ? ($extraOptions['debug']) : 0;
826
-        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
826
+        $timeout = isset($extraOptions['timeout']) ? (int) $extraOptions['timeout'] : 0;
827 827
         $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
828 828
 
829 829
         $mDesc = '';
@@ -857,11 +857,11 @@  discard block
 block discarded – undo
857 857
         $clientClone = clone $client;
858 858
         $function = function() use($clientClone, $methodName, $extraOptions, $mSig)
859 859
         {
860
-            $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
860
+            $timeout = isset($extraOptions['timeout']) ? (int) $extraOptions['timeout'] : 0;
861 861
             $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
862
-            $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
863
-            $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
864
-            $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool)$extraOptions['encode_nulls'] : false;
862
+            $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool) $extraOptions['encode_php_objs'] : false;
863
+            $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool) $extraOptions['decode_php_objs'] : false;
864
+            $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool) $extraOptions['encode_nulls'] : false;
865 865
             $throwFault = false;
866 866
             $decodeFault = false;
867 867
             $faultResponse = null;
@@ -872,9 +872,9 @@  discard block
 block discarded – undo
872 872
                 $faultResponse = $extraOptions['return_on_fault'];
873 873
             }
874 874
 
875
-            $reqClass = static::$namespace . 'Request';
876
-            $encoderClass = static::$namespace . 'Encoder';
877
-            $valueClass = static::$namespace . 'Value';
875
+            $reqClass = static::$namespace.'Request';
876
+            $encoderClass = static::$namespace.'Encoder';
877
+            $valueClass = static::$namespace.'Value';
878 878
 
879 879
             $encoder = new $encoderClass();
880 880
             $encodeOptions = array();
@@ -956,14 +956,14 @@  discard block
 block discarded – undo
956 956
      * @param string $mDesc
957 957
      * @return string[] keys: source, docstring
958 958
      */
959
-    public function buildWrapMethodSource($client, $methodName, array $extraOptions, $newFuncName, $mSig, $mDesc='')
959
+    public function buildWrapMethodSource($client, $methodName, array $extraOptions, $newFuncName, $mSig, $mDesc = '')
960 960
     {
961
-        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
961
+        $timeout = isset($extraOptions['timeout']) ? (int) $extraOptions['timeout'] : 0;
962 962
         $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
963
-        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
964
-        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
965
-        $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool)$extraOptions['encode_nulls'] : false;
966
-        $clientCopyMode = isset($extraOptions['simple_client_copy']) ? (int)($extraOptions['simple_client_copy']) : 0;
963
+        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool) $extraOptions['encode_php_objs'] : false;
964
+        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool) $extraOptions['decode_php_objs'] : false;
965
+        $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool) $extraOptions['encode_nulls'] : false;
966
+        $clientCopyMode = isset($extraOptions['simple_client_copy']) ? (int) ($extraOptions['simple_client_copy']) : 0;
967 967
         $prefix = isset($extraOptions['prefix']) ? $extraOptions['prefix'] : 'xmlrpc';
968 968
         $throwFault = false;
969 969
         $decodeFault = false;
@@ -976,10 +976,10 @@  discard block
 block discarded – undo
976 976
         }
977 977
 
978 978
         $code = "function $newFuncName(";
979
-        if ($clientCopyMode < 2) {
979
+        if ($clientCopyMode<2) {
980 980
             // client copy mode 0 or 1 == full / partial client copy in emitted code
981 981
             $verbatimClientCopy = !$clientCopyMode;
982
-            $innerCode = '  ' . str_replace("\n", "\n  ", $this->buildClientWrapperCode($client, $verbatimClientCopy, $prefix, static::$namespace));
982
+            $innerCode = '  '.str_replace("\n", "\n  ", $this->buildClientWrapperCode($client, $verbatimClientCopy, $prefix, static::$namespace));
983 983
             $innerCode .= "\$client->setDebug(\$debug);\n";
984 984
             $this_ = '';
985 985
         } else {
@@ -987,28 +987,28 @@  discard block
 block discarded – undo
987 987
             $innerCode = '';
988 988
             $this_ = 'this->';
989 989
         }
990
-        $innerCode .= "  \$req = new " . static::$namespace . "Request('$methodName');\n";
990
+        $innerCode .= "  \$req = new ".static::$namespace."Request('$methodName');\n";
991 991
 
992 992
         if ($mDesc != '') {
993 993
             // take care that PHP comment is not terminated unwillingly by method description
994 994
             /// @todo according to the spec, method desc can have html in it. We should run it through strip_tags...
995
-            $mDesc = "/**\n * " . str_replace(array("\n", '*/'), array("\n * ", '* /'), $mDesc) . "\n";
995
+            $mDesc = "/**\n * ".str_replace(array("\n", '*/'), array("\n * ", '* /'), $mDesc)."\n";
996 996
         } else {
997 997
             $mDesc = "/**\n * Function $newFuncName.\n";
998 998
         }
999 999
 
1000 1000
         // param parsing
1001
-        $innerCode .= "  \$encoder = new " . static::$namespace . "Encoder();\n";
1001
+        $innerCode .= "  \$encoder = new ".static::$namespace."Encoder();\n";
1002 1002
         $plist = array();
1003 1003
         $pCount = count($mSig);
1004
-        for ($i = 1; $i < $pCount; $i++) {
1004
+        for ($i = 1; $i<$pCount; $i++) {
1005 1005
             $plist[] = "\$p$i";
1006 1006
             $pType = $mSig[$i];
1007 1007
             if ($pType == 'i4' || $pType == 'i8' || $pType == 'int' || $pType == 'boolean' || $pType == 'double' ||
1008 1008
                 $pType == 'string' || $pType == 'dateTime.iso8601' || $pType == 'base64' || $pType == 'null'
1009 1009
             ) {
1010 1010
                 // only build directly xml-rpc values when type is known and scalar
1011
-                $innerCode .= "  \$p$i = new " . static::$namespace . "Value(\$p$i, '$pType');\n";
1011
+                $innerCode .= "  \$p$i = new ".static::$namespace."Value(\$p$i, '$pType');\n";
1012 1012
             } else {
1013 1013
                 if ($encodePhpObjects || $encodeNulls) {
1014 1014
                     $encOpts = array();
@@ -1019,26 +1019,26 @@  discard block
 block discarded – undo
1019 1019
                         $encOpts[] = 'null_extension';
1020 1020
                     }
1021 1021
 
1022
-                    $innerCode .= "  \$p$i = \$encoder->encode(\$p$i, array( '" . implode("', '", $encOpts) . "'));\n";
1022
+                    $innerCode .= "  \$p$i = \$encoder->encode(\$p$i, array( '".implode("', '", $encOpts)."'));\n";
1023 1023
                 } else {
1024 1024
                     $innerCode .= "  \$p$i = \$encoder->encode(\$p$i);\n";
1025 1025
                 }
1026 1026
             }
1027 1027
             $innerCode .= "  \$req->addparam(\$p$i);\n";
1028
-            $mDesc .= " * @param " . $this->xmlrpc2PhpType($pType) . " \$p$i\n";
1028
+            $mDesc .= " * @param ".$this->xmlrpc2PhpType($pType)." \$p$i\n";
1029 1029
         }
1030
-        if ($clientCopyMode < 2) {
1030
+        if ($clientCopyMode<2) {
1031 1031
             $plist[] = '$debug = 0';
1032 1032
             $mDesc .= " * @param int \$debug when 1 (or 2) will enable debugging of the underlying {$prefix} call (defaults to 0)\n";
1033 1033
         }
1034 1034
         $plist = implode(', ', $plist);
1035
-        $mDesc .= ' * @return ' . $this->xmlrpc2PhpType($mSig[0]);
1035
+        $mDesc .= ' * @return '.$this->xmlrpc2PhpType($mSig[0]);
1036 1036
         if ($throwFault) {
1037
-            $mDesc .= "\n * @throws " . (is_string($throwFault) ? $throwFault : '\\Exception');
1037
+            $mDesc .= "\n * @throws ".(is_string($throwFault) ? $throwFault : '\\Exception');
1038 1038
         } else if ($decodeFault) {
1039
-            $mDesc .= '|' . gettype($faultResponse) . " (a " . gettype($faultResponse) . " if call fails)";
1039
+            $mDesc .= '|'.gettype($faultResponse)." (a ".gettype($faultResponse)." if call fails)";
1040 1040
         } else {
1041
-            $mDesc .= '|' . static::$namespace . "Response (a " . static::$namespace . "Response obj instance if call fails)";
1041
+            $mDesc .= '|'.static::$namespace."Response (a ".static::$namespace."Response obj instance if call fails)";
1042 1042
         }
1043 1043
         $mDesc .= "\n */\n";
1044 1044
 
@@ -1050,9 +1050,9 @@  discard block
 block discarded – undo
1050 1050
             $respCode = "throw new $throwFault(\$res->faultString(), \$res->faultCode())";
1051 1051
         } else if ($decodeFault) {
1052 1052
             if (is_string($faultResponse) && ((strpos($faultResponse, '%faultCode%') !== false) || (strpos($faultResponse, '%faultString%') !== false))) {
1053
-                $respCode = "return str_replace(array('%faultCode%', '%faultString%'), array(\$res->faultCode(), \$res->faultString()), '" . str_replace("'", "''", $faultResponse) . "')";
1053
+                $respCode = "return str_replace(array('%faultCode%', '%faultString%'), array(\$res->faultCode(), \$res->faultString()), '".str_replace("'", "''", $faultResponse)."')";
1054 1054
             } else {
1055
-                $respCode = 'return ' . var_export($faultResponse, true);
1055
+                $respCode = 'return '.var_export($faultResponse, true);
1056 1056
             }
1057 1057
         } else {
1058 1058
             $respCode = 'return $res';
@@ -1063,7 +1063,7 @@  discard block
 block discarded – undo
1063 1063
             $innerCode .= "  if (\$res->faultCode()) $respCode; else return \$encoder->decode(\$res->value());";
1064 1064
         }
1065 1065
 
1066
-        $code = $code . $plist . ")\n{\n" . $innerCode . "\n}\n";
1066
+        $code = $code.$plist.")\n{\n".$innerCode."\n}\n";
1067 1067
 
1068 1068
         return array('source' => $code, 'docstring' => $mDesc);
1069 1069
     }
@@ -1090,25 +1090,25 @@  discard block
 block discarded – undo
1090 1090
     public function wrapXmlrpcServer($client, $extraOptions = array())
1091 1091
     {
1092 1092
         $methodFilter = isset($extraOptions['method_filter']) ? $extraOptions['method_filter'] : '';
1093
-        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
1093
+        $timeout = isset($extraOptions['timeout']) ? (int) $extraOptions['timeout'] : 0;
1094 1094
         $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
1095 1095
         $newClassName = isset($extraOptions['new_class_name']) ? $extraOptions['new_class_name'] : '';
1096
-        $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool)$extraOptions['encode_nulls'] : false;
1097
-        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
1098
-        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
1096
+        $encodeNulls = isset($extraOptions['encode_nulls']) ? (bool) $extraOptions['encode_nulls'] : false;
1097
+        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool) $extraOptions['encode_php_objs'] : false;
1098
+        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool) $extraOptions['decode_php_objs'] : false;
1099 1099
         $verbatimClientCopy = isset($extraOptions['simple_client_copy']) ? !($extraOptions['simple_client_copy']) : true;
1100
-        $throwOnFault = isset($extraOptions['throw_on_fault']) ? (bool)$extraOptions['throw_on_fault'] : false;
1100
+        $throwOnFault = isset($extraOptions['throw_on_fault']) ? (bool) $extraOptions['throw_on_fault'] : false;
1101 1101
         $buildIt = isset($extraOptions['return_source']) ? !($extraOptions['return_source']) : true;
1102 1102
         $prefix = isset($extraOptions['prefix']) ? $extraOptions['prefix'] : 'xmlrpc';
1103 1103
 
1104
-        $reqClass = static::$namespace . 'Request';
1105
-        $decoderClass = static::$namespace . 'Encoder';
1104
+        $reqClass = static::$namespace.'Request';
1105
+        $decoderClass = static::$namespace.'Encoder';
1106 1106
 
1107 1107
         // retrieve the list of methods
1108 1108
         $req = new $reqClass('system.listMethods');
1109 1109
         $response = $client->send($req, $timeout, $protocol);
1110 1110
         if ($response->faultCode()) {
1111
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': could not retrieve method list from remote server');
1111
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': could not retrieve method list from remote server');
1112 1112
 
1113 1113
             return false;
1114 1114
         }
@@ -1119,7 +1119,7 @@  discard block
 block discarded – undo
1119 1119
             $mList = $decoder->decode($mList);
1120 1120
         }
1121 1121
         if (!is_array($mList) || !count($mList)) {
1122
-            $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': could not retrieve meaningful method list from remote server');
1122
+            $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': could not retrieve meaningful method list from remote server');
1123 1123
 
1124 1124
             return false;
1125 1125
         }
@@ -1128,8 +1128,8 @@  discard block
 block discarded – undo
1128 1128
         if ($newClassName != '') {
1129 1129
             $xmlrpcClassName = $newClassName;
1130 1130
         } else {
1131
-            $xmlrpcClassName = $prefix . '_' . preg_replace(array('/\./', '/[^a-zA-Z0-9_\x7f-\xff]/'),
1132
-                    array('_', ''), $client->server) . '_client';
1131
+            $xmlrpcClassName = $prefix.'_'.preg_replace(array('/\./', '/[^a-zA-Z0-9_\x7f-\xff]/'),
1132
+                    array('_', ''), $client->server).'_client';
1133 1133
         }
1134 1134
         while ($buildIt && class_exists($xmlrpcClassName)) {
1135 1135
             $xmlrpcClassName .= 'x';
@@ -1138,7 +1138,7 @@  discard block
 block discarded – undo
1138 1138
         /// @todo add method setDebug() to new class, to enable/disable debugging
1139 1139
         $source = "class $xmlrpcClassName\n{\n  public \$client;\n\n";
1140 1140
         $source .= "  function __construct()\n  {\n";
1141
-        $source .= '    ' . str_replace("\n", "\n    ", $this->buildClientWrapperCode($client, $verbatimClientCopy, $prefix, static::$namespace));
1141
+        $source .= '    '.str_replace("\n", "\n    ", $this->buildClientWrapperCode($client, $verbatimClientCopy, $prefix, static::$namespace));
1142 1142
         $source .= "\$this->client = \$client;\n  }\n\n";
1143 1143
         $opts = array(
1144 1144
             'return_source' => true,
@@ -1161,26 +1161,26 @@  discard block
 block discarded – undo
1161 1161
                 $methodWrap = $this->wrapXmlrpcMethod($client, $mName, $opts);
1162 1162
                 if ($methodWrap) {
1163 1163
                     if ($buildIt) {
1164
-                        $source .= $methodWrap['source'] . "\n";
1164
+                        $source .= $methodWrap['source']."\n";
1165 1165
 
1166 1166
                     } else {
1167
-                        $source .= '  ' . str_replace("\n", "\n  ", $methodWrap['docstring']);
1168
-                        $source .= str_replace("\n", "\n  ", $methodWrap['source']). "\n";
1167
+                        $source .= '  '.str_replace("\n", "\n  ", $methodWrap['docstring']);
1168
+                        $source .= str_replace("\n", "\n  ", $methodWrap['source'])."\n";
1169 1169
                     }
1170 1170
 
1171 1171
                 } else {
1172
-                    $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': will not create class method to wrap remote method ' . $mName);
1172
+                    $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': will not create class method to wrap remote method '.$mName);
1173 1173
                 }
1174 1174
             }
1175 1175
         }
1176 1176
         $source .= "}\n";
1177 1177
         if ($buildIt) {
1178 1178
             $allOK = 0;
1179
-            eval($source . '$allOK=1;');
1179
+            eval($source.'$allOK=1;');
1180 1180
             if ($allOK) {
1181 1181
                 return $xmlrpcClassName;
1182 1182
             } else {
1183
-                $this->getLogger()->errorLog('XML-RPC: ' . __METHOD__ . ': could not create class ' . $xmlrpcClassName . ' to wrap remote server ' . $client->server);
1183
+                $this->getLogger()->errorLog('XML-RPC: '.__METHOD__.': could not create class '.$xmlrpcClassName.' to wrap remote server '.$client->server);
1184 1184
                 return false;
1185 1185
             }
1186 1186
         } else {
@@ -1199,8 +1199,8 @@  discard block
 block discarded – undo
1199 1199
      */
1200 1200
     protected function buildClientWrapperCode($client, $verbatimClientCopy, $prefix = 'xmlrpc', $namespace = '\\PhpXmlRpc\\')
1201 1201
     {
1202
-        $code = "\$client = new {$namespace}Client('" . str_replace(array("\\", "'"), array("\\\\", "\'"), $client->path) .
1203
-            "', '" . str_replace(array("\\", "'"), array("\\\\", "\'"), $client->server) . "', $client->port);\n";
1202
+        $code = "\$client = new {$namespace}Client('".str_replace(array("\\", "'"), array("\\\\", "\'"), $client->path).
1203
+            "', '".str_replace(array("\\", "'"), array("\\\\", "\'"), $client->server)."', $client->port);\n";
1204 1204
 
1205 1205
         // copy all client fields to the client that will be generated runtime
1206 1206
         // (this provides for future expansion or subclassing of client obj)
Please login to merge, or discard this patch.