View GitHub Repository phpxmlrpc

Schedule Inspection
Subscribe
Passed
Push — master ( 645c9e...18eded )
by Gaetano
10:21 queued 35s
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.
tests/09HTTPTest.php 1 patch
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -201,8 +201,7 @@  discard block
 
 block discarded – undo
201 201 
         {
202 202 
             $this->markTestSkipped('CURL missing: cannot test http 1.1 w. proxy');
203 203 
             return;
204 
-        }
205 
-        else if ($this->args['PROXYSERVER'] == '')
204 
+        } else if ($this->args['PROXYSERVER'] == '')
206 205 
         {
207 206 
             $this->markTestSkipped('PROXYSERVER definition missing: cannot test proxy w. http 1.1');
208 207 
             return;
 
@@ -226,8 +225,7 @@  discard block
 
 block discarded – undo
226 225 
         {
227 226 
             $this->markTestSkipped('CURL missing: cannot test https functionality');
228 227 
             return;
229 
-        }
230 
-        else if ($this->args['HTTPSSERVER'] == '')
228 
+        } else if ($this->args['HTTPSSERVER'] == '')
231 229 
         {
232 230 
             $this->markTestSkipped('HTTPS SERVER definition missing: cannot test https');
233 231 
             return;
 
@@ -278,13 +276,11 @@  discard block
 
 block discarded – undo
278 276 
         {
279 277 
             $this->markTestSkipped('CURL missing: cannot test https w. proxy');
280 278 
             return;
281 
-        }
282 
-        else if ($this->args['PROXYSERVER'] == '')
279 
+        } else if ($this->args['PROXYSERVER'] == '')
283 280 
         {
284 281 
             $this->markTestSkipped('PROXYSERVER definition missing: cannot test proxy w. https');
285 282 
             return;
286 
-        }
287 
-        else if ($this->args['HTTPSSERVER'] == '')
283 
+        } else if ($this->args['HTTPSSERVER'] == '')
288 284 
         {
289 285 
             $this->markTestSkipped('HTTPS SERVER definition missing: cannot test https w. proxy');
290 286 
             return;
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 
@@ -69,11 +69,12 @@
 
 block discarded – undo
69 69 
     $a = $req->getParam(0);
70 70 
     $b = $req->getParam(1);
71 71
72 
-    if ($a->scalarTyp() == Value::$xmlrpcNull)
73 
-        return new Response(new Value(plain_findstate($b->scalarVal())));
74 
-    else
75 
-        return new Response(new Value(plain_findstate($a->scalarVal())));
76 
-}
72 
+    if ($a->scalarTyp() == Value::$xmlrpcNull) {
73 
+            return new Response(new Value(plain_findstate($b->scalarVal())));
74 
+    } else {
75 
+            return new Response(new Value(plain_findstate($a->scalarVal())));
76 
+    }
77 
+    }
77 78
78 79 
 return array(
79 80 
     "tests.getallheaders" => array(
Please login to merge, or discard this patch.