Passed
Push — master ( c576cf...1a08ad )
by Gaetano
07:23
created
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.
tests/4ClientTest.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -24,14 +24,16 @@
 block discarded – undo
24 24
         $this->client = new xmlrpc_client('/NOTEXIST.php', $this->args['HTTPSERVER'], 80);
25 25
         $this->client->setDebug($this->args['DEBUG']);
26 26
 
27
-        if ($this->args['DEBUG'] == 1)
28
-            ob_start();
27
+        if ($this->args['DEBUG'] == 1) {
28
+                    ob_start();
29
+        }
29 30
     }
30 31
 
31 32
     protected function tear_down()
32 33
     {
33
-        if ($this->args['DEBUG'] != 1)
34
-            return;
34
+        if ($this->args['DEBUG'] != 1) {
35
+                    return;
36
+        }
35 37
         $out = ob_get_clean();
36 38
         $status = $this->getStatus();
37 39
         if ($status == BaseTestRunner::STATUS_ERROR
Please login to merge, or discard this patch.
tests/3EncoderTest.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,14 +21,16 @@
 block discarded – undo
21 21
     protected function set_up()
22 22
     {
23 23
         $this->args = argParser::getArgs();
24
-        if ($this->args['DEBUG'] == 1)
25
-            ob_start();
24
+        if ($this->args['DEBUG'] == 1) {
25
+                    ob_start();
26
+        }
26 27
     }
27 28
 
28 29
     protected function tear_down()
29 30
     {
30
-        if ($this->args['DEBUG'] != 1)
31
-            return;
31
+        if ($this->args['DEBUG'] != 1) {
32
+                    return;
33
+        }
32 34
         $out = ob_get_clean();
33 35
         $status = $this->getStatus();
34 36
         if ($status == BaseTestRunner::STATUS_ERROR
Please login to merge, or discard this patch.
tests/2MessageTest.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,14 +21,16 @@
 block discarded – undo
21 21
     protected function set_up()
22 22
     {
23 23
         $this->args = argParser::getArgs();
24
-        if ($this->args['DEBUG'] == 1)
25
-            ob_start();
24
+        if ($this->args['DEBUG'] == 1) {
25
+                    ob_start();
26
+        }
26 27
     }
27 28
 
28 29
     protected function tear_down()
29 30
     {
30
-        if ($this->args['DEBUG'] != 1)
31
-            return;
31
+        if ($this->args['DEBUG'] != 1) {
32
+                    return;
33
+        }
32 34
         $out = ob_get_clean();
33 35
         $status = $this->getStatus();
34 36
         if ($status == BaseTestRunner::STATUS_ERROR
Please login to merge, or discard this patch.
tests/6HTTPTest.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.
tests/1ValueTest.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,14 +21,16 @@
 block discarded – undo
21 21
     protected function set_up()
22 22
     {
23 23
         $this->args = argParser::getArgs();
24
-        if ($this->args['DEBUG'] == 1)
25
-            ob_start();
24
+        if ($this->args['DEBUG'] == 1) {
25
+                    ob_start();
26
+        }
26 27
     }
27 28
 
28 29
     protected function tear_down()
29 30
     {
30
-        if ($this->args['DEBUG'] != 1)
31
-            return;
31
+        if ($this->args['DEBUG'] != 1) {
32
+                    return;
33
+        }
32 34
         $out = ob_get_clean();
33 35
         $status = $this->getStatus();
34 36
         if ($status == BaseTestRunner::STATUS_ERROR
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
@@ -12,7 +12,12 @@
 block discarded – undo
12 12
 <html lang="en">
13 13
 <head>
14 14
     <link rel="icon" type="image/vnd.microsoft.icon" href="favicon.ico">
15
-    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) echo 'JSONRPC'; else echo 'XMLRPC'; ?> Debugger</title>
15
+    <title><?php if (defined('DEFAULT_WSTYPE') && DEFAULT_WSTYPE == 1) {
16
+    echo 'JSONRPC';
17
+} else {
18
+    echo 'XMLRPC';
19
+}
20
+?> Debugger</title>
16 21
 </head>
17 22
 <frameset rows="360,*">
18 23
     <frame name="frmcontroller" src="controller.php<?php echo htmlspecialchars($query); ?>" marginwidth="0"
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.