View GitHub Repository phpxmlrpc

Schedule Inspection
Subscribe
Passed
Push — master ( c1cbf5...9efc10 )
by Gaetano
07:04
created
Status
Category
Toggle Patches
lib/xmlrpc_wrappers.inc 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -141,9 +141,9 @@
 
 block discarded – undo
141 141 
  * @deprecated
142 142 
  */
143 143 
 function build_remote_method_wrapper_code($client, $methodName, $xmlrpcFuncName,
144 
-     $mSig, $mDesc = '', $timeout = 0, $protocol = '', $clientCopyMode = 0, $prefix = 'xmlrpc',
145 
-     $decodePhpObjects = false, $encodePhpObjects = false, $decodeFault = false,
146 
-     $faultResponse = '', $namespace = '\\PhpXmlRpc\\')
144 
+        $mSig, $mDesc = '', $timeout = 0, $protocol = '', $clientCopyMode = 0, $prefix = 'xmlrpc',
145 
+        $decodePhpObjects = false, $encodePhpObjects = false, $decodeFault = false,
146 
+        $faultResponse = '', $namespace = '\\PhpXmlRpc\\')
147 147 
 {
148 148 
     $code = "function $xmlrpcFuncName (";
149 149 
     if ($clientCopyMode < 2) {
Please login to merge, or discard this patch.
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.
src/Client.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -951,9 +951,9 @@
 
 block discarded – undo
951 951 
     }
952 952
953 953 
     protected function prepareCurlHandle($req, $server, $port, $timeout = 0, $username = '', $password = '',
954 
-         $authType = 1, $cert = '', $certPass = '', $caCert = '', $caCertDir = '', $proxyHost = '', $proxyPort = 0,
955 
-         $proxyUsername = '', $proxyPassword = '', $proxyAuthType = 1, $method = 'https', $keepAlive = false, $key = '',
956 
-         $keyPass = '', $sslVersion = 0)
954 
+            $authType = 1, $cert = '', $certPass = '', $caCert = '', $caCertDir = '', $proxyHost = '', $proxyPort = 0,
955 
+            $proxyUsername = '', $proxyPassword = '', $proxyAuthType = 1, $method = 'https', $keepAlive = false, $key = '',
956 
+            $keyPass = '', $sslVersion = 0)
957 957 
     {
958 958 
         if ($port == 0) {
959 959 
             if (in_array($method, array('http', 'http10', 'http11', 'h2c'))) {
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 
@@ -66,11 +66,12 @@
 
 block discarded – undo
66 66 
     $a = $req->getParam(0);
67 67 
     $b = $req->getParam(1);
68 68
69 
-    if ($a->scalartyp() == Value::$xmlrpcNull)
70 
-        return new Response(new Value(plain_findstate($b->scalarval())));
71 
-    else
72 
-        return new Response(new Value(plain_findstate($a->scalarval())));
73 
-}
69 
+    if ($a->scalartyp() == Value::$xmlrpcNull) {
70 
+            return new Response(new Value(plain_findstate($b->scalarval())));
71 
+    } else {
72 
+            return new Response(new Value(plain_findstate($a->scalarval())));
73 
+    }
74 
+    }
74 75
75 76 
 return array(
76 77 
     "tests.getallheaders" => array(
Please login to merge, or discard this patch.
demo/server/methodProviders/wrapper.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -26,7 +26,7 @@
 
 block discarded – undo
26 26 
  */
27 27 
 function plain_findstate($stateNo)
28 28 
 {
29 
-     if (isset(exampleMethods::$stateNames[$stateNo - 1])) {
29 
+        if (isset(exampleMethods::$stateNames[$stateNo - 1])) {
30 30 
         return exampleMethods::$stateNames[$stateNo - 1];
31 31 
     } else {
32 32 
         // not, there so complain
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.