View GitHub Repository phpxmlrpc

Schedule Inspection
Subscribe
Passed
Push — master ( ea57c6...209714 )
by Gaetano
07:44
created
Status
Category
Toggle Patches
src/Helper/Charset.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -176,8 +176,7 @@
 
 block discarded – undo
176 176 
                         } else {
177 177 
                             $escapedData .= $ch;
178 178 
                         }
179 
-                    }
180 
-                    else if ($ii < 128) {
179 
+                    } else if ($ii < 128) {
181 180 
                         /// @todo shall we replace this with a (supposedly) faster str_replace?
182 181 
                         /// @todo to be 'print safe', should we encode as well character 127 (DEL) ?
183 182 
                         switch ($ii) {
Please login to merge, or discard this patch.
debugger/action.php 1 patch
Braces   +9 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -15,7 +15,12 @@  discard block
 
 block discarded – undo
15 15 
 <html lang="en">
16 16 
 <head>
17 17 
     <link rel="icon" type="image/vnd.microsoft.icon" href="favicon.ico">
18 
-    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) echo 'JSON-RPC'; else echo 'XML-RPC'; ?> Debugger</title>
18 
+    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) {
19 
+    echo 'JSON-RPC';
20 
+} else {
21 
+    echo 'XML-RPC';
22 
+}
23 
+?> Debugger</title>
19 24 
     <meta name="robots" content="index,nofollow"/>
20 25 
     <style type="text/css">
21 26 
         <!--
 
@@ -367,7 +372,9 @@  discard block
 
 block discarded – undo
367 372 
                                 echo "<code>OUT:&nbsp;" . htmlspecialchars($ret->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) . "<br />IN: (";
368 373 
                                 if ($x->count() > 1) {
369 374 
                                     foreach($x as $k => $y) {
370 
-                                        if ($k == 0) continue;
375 
+                                        if ($k == 0) {
376 
+                                            continue;
377 
+                                        }
371 378 
                                         echo htmlspecialchars($y->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding);
372 379 
                                         if ($wstype == 1) {
373 380 
                                             switch($y->scalarval()) {
Please login to merge, or discard this patch.
debugger/index.php 1 patch
Braces   +6 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -13,7 +13,12 @@
 
 block discarded – undo
13 13 
 <html lang="en">
14 14 
 <head>
15 15 
     <link rel="icon" type="image/vnd.microsoft.icon" href="favicon.ico">
16 
-    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) echo 'JSON-RPC'; else echo 'XML-RPC'; ?> Debugger</title>
16 
+    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) {
17 
+    echo 'JSON-RPC';
18 
+} else {
19 
+    echo 'XML-RPC';
20 
+}
21 
+?> Debugger</title>
17 22 
 </head>
18 23 
 <frameset rows="360,*">
19 24 
     <frame name="frmcontroller" src="controller.php<?php echo htmlspecialchars($query); ?>" marginwidth="0"
Please login to merge, or discard this patch.
debugger/controller.php 1 patch
Braces   +16 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -66,7 +66,12 @@  discard block
 
 block discarded – undo
66 66 
 <html lang="en">
67 67 
 <head>
68 68 
     <link rel="icon" type="image/vnd.microsoft.icon" href="favicon.ico">
69 
-    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) echo 'JSON-RPC'; else echo 'XML-RPC'; ?> Debugger</title>
69 
+    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) {
70 
+    echo 'JSON-RPC';
71 
+} else {
72 
+    echo 'XML-RPC';
73 
+}
74 
+?> Debugger</title>
70 75 
     <meta name="robots" content="index,nofollow"/>
71 76 
     <script type="text/javascript" language="Javascript">
72 77 
         if (window.name != 'frmcontroller')
 
@@ -266,19 +271,26 @@  discard block
 
 block discarded – undo
266 271 
     </script>
267 272 
 </head>
268 273 
 <body
269 
-    onload="<?php if ($hasjsonrpcclient) echo "switchtransport($wstype); " ?>switchaction(); switchssl(); switchauth(); swicthcainfo();<?php if ($run) {
274 
+    onload="<?php if ($hasjsonrpcclient) {
275 
+    echo "switchtransport($wstype); " ?>switchaction(); switchssl(); switchauth(); swicthcainfo();<?php if ($run) {
270 276 
         echo ' document.frmaction.submit();';
277 
+}
271 278 
     } ?>">
272 279 
 <h1>XML-RPC
273 280 
 <?php if ($hasjsonrpcclient) {
274 281 
     echo '<form name="frmxmlrpc" style="display: inline;" action="."><input name="yes" type="radio" onclick="switchtransport(0);"';
275 282 
     // q: does this if make sense at all?
276 
-    if (!class_exists('\PhpXmlRpc\Client')) echo ' disabled="disabled"';
283 
+    if (!class_exists('\PhpXmlRpc\Client')) {
284 
+        echo ' disabled="disabled"';
285 
+    }
277 286 
     echo ' /></form> / <form name="frmjsonrpc" style="display: inline;" action="."><input name="yes" type="radio" onclick="switchtransport(1);"/></form>
278 287 
     JSON-RPC';
279 288 
 } ?>
280 289 
 Debugger</h1><h3>(based on <a href="https://gggeek.github.io/phpxmlrpc/">PHPXMLRPC</a>, ver. <?php echo htmlspecialchars(\PhpXmlRpc\PhpXmlRpc::$xmlrpcVersion)?>
281 
-<?php if (class_exists('\PhpXmlRpc\JsonRpc\PhpJsonRpc')) echo ' and <a href="https://gggeek.github.io/phpxmlrpc-jsonrpc/">PHPJOSNRPC</a>, ver. ' . htmlspecialchars(\PhpXmlRpc\JsonRpc\PhpJsonRpc::$jsonrpcVersion); ?>)</h3>
290 
+<?php if (class_exists('\PhpXmlRpc\JsonRpc\PhpJsonRpc')) {
291 
+    echo ' and <a href="https://gggeek.github.io/phpxmlrpc-jsonrpc/">PHPJOSNRPC</a>, ver. ' . htmlspecialchars(\PhpXmlRpc\JsonRpc\PhpJsonRpc::$jsonrpcVersion);
292 
+}
293 
+?>)</h3>
282 294 
 <form name="frmaction" method="get" action="action.php" target="frmaction" onSubmit="switchFormMethod();">
283 295
284 296 
     <table id="serverblock">
Please login to merge, or discard this patch.
src/Wrapper.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -605,8 +605,9 @@
 
 block discarded – undo
605 605 
         }
606 606 
         foreach ($parsVariations as $i => $pars) {
607 607 
             $innerCode .= "  if (\$paramCount == " . count($pars) . ") \$retVal = {$catchWarnings}$realFuncName(" . implode(',', $pars) . ");\n";
608 
-            if ($i < (count($parsVariations) - 1))
609 
-                $innerCode .= "  else\n";
608 
+            if ($i < (count($parsVariations) - 1)) {
609 
+                            $innerCode .= "  else\n";
610 
+            }
610 611 
         }
611 612 
         $innerCode .= "  if (is_a(\$retVal, '" . static::$namespace . "Response'))\n    return \$retVal;\n  else\n";
612 613 
         /// q: why not do the same for int, float, bool, string?
Please login to merge, or discard this patch.
demo/server/methodProviders/testsuite.php 1 patch
Braces   +6 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -88,11 +88,12 @@
 
 block discarded – undo
88 88 
     $a = $req->getParam(0);
89 89 
     $b = $req->getParam(1);
90 90
91 
-    if ($a->scalarTyp() == Value::$xmlrpcNull)
92 
-        return new Response(new Value(plain_findstate($b->scalarVal())));
93 
-    else
94 
-        return new Response(new Value(plain_findstate($a->scalarVal())));
95 
-}
91 
+    if ($a->scalarTyp() == Value::$xmlrpcNull) {
92 
+            return new Response(new Value(plain_findstate($b->scalarVal())));
93 
+    } else {
94 
+            return new Response(new Value(plain_findstate($a->scalarVal())));
95 
+    }
96 
+    }
96 97
97 98 
 $sleep_sig = array(array(Value::$xmlrpcInt, Value::$xmlrpcInt));
98 99 
 $sleep_doc = 'Sleeps for the requested number of seconds (between 1 and 60), before sending back the response';
Please login to merge, or discard this patch.
tests/09HTTPTest.php 1 patch
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -304,8 +304,7 @@  discard block
 
 block discarded – undo
304 304 
         if (!function_exists('curl_init'))
305 305 
         {
306 306 
             $this->markTestSkipped('CURL missing: cannot test https functionality');
307 
-        }
308 
-        else if ($this->args['HTTPSSERVER'] == '')
307 
+        } else if ($this->args['HTTPSSERVER'] == '')
309 308 
         {
310 309 
             $this->markTestSkipped('HTTPS SERVER definition missing: cannot test https');
311 310 
         }
 
@@ -480,8 +479,7 @@  discard block
 
 block discarded – undo
480 479 
         if (!function_exists('curl_init'))
481 480 
         {
482 481 
             $this->markTestSkipped('CURL missing: cannot test http 1.1 w. proxy');
483 
-        }
484 
-        else if ($this->args['PROXYSERVER'] == '')
482 
+        } else if ($this->args['PROXYSERVER'] == '')
485 483 
         {
486 484 
             $this->markTestSkipped('PROXYSERVER definition missing: cannot test proxy w. http 1.1');
487 485 
         }
 
@@ -505,12 +503,10 @@  discard block
 
 block discarded – undo
505 503 
         if (!function_exists('curl_init'))
506 504 
         {
507 505 
             $this->markTestSkipped('CURL missing: cannot test https w. proxy');
508 
-        }
509 
-        else if ($this->args['PROXYSERVER'] == '')
506 
+        } else if ($this->args['PROXYSERVER'] == '')
510 507 
         {
511 508 
             $this->markTestSkipped('PROXYSERVER definition missing: cannot test proxy w. https');
512 
-        }
513 
-        else if ($this->args['HTTPSSERVER'] == '')
509 
+        } else if ($this->args['HTTPSSERVER'] == '')
514 510 
         {
515 511 
             $this->markTestSkipped('HTTPS SERVER definition missing: cannot test https w. proxy');
516 512 
         }
Please login to merge, or discard this patch.