Passed
Push — master ( ea57c6...209714 )
by Gaetano
07:44
created
tests/LoggerAwareTestCase.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -1,11 +1,11 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-include_once __DIR__ . '/../lib/xmlrpc.inc';
4
-include_once __DIR__ . '/../lib/xmlrpcs.inc';
3
+include_once __DIR__.'/../lib/xmlrpc.inc';
4
+include_once __DIR__.'/../lib/xmlrpcs.inc';
5 5
 
6
-include_once __DIR__ . '/parse_args.php';
6
+include_once __DIR__.'/parse_args.php';
7 7
 
8
-include_once __DIR__ . '/PolyfillTestCase.php';
8
+include_once __DIR__.'/PolyfillTestCase.php';
9 9
 
10 10
 use PHPUnit\Runner\BaseTestRunner;
11 11
 
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 
33 33
     protected function tear_down()
34 34
     {
35
-        if ($this->args['DEBUG'] > 0) {
35
+        if ($this->args['DEBUG']>0) {
36 36
             return;
37 37
         }
38 38
 
@@ -50,16 +50,16 @@  discard block
 block discarded – undo
50 50
 
51 51
     public function debug($message, $context = array())
52 52
     {
53
-        $this->buffer .= $message . "\n";
53
+        $this->buffer .= $message."\n";
54 54
     }
55 55
 
56 56
     public function error($message, $context = array())
57 57
     {
58
-        $this->buffer .= $message . "\n";
58
+        $this->buffer .= $message."\n";
59 59
     }
60 60
 
61 61
     public function warning($message, $context = array())
62 62
     {
63
-        $this->buffer .= $message . "\n";
63
+        $this->buffer .= $message."\n";
64 64
     }
65 65
 }
Please login to merge, or discard this patch.
demo/client/loggerinjection.php 1 patch
Spacing   +9 added lines, -9 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
 /**
5 5
  * Demoing how to inject a custom logger for use by the library
@@ -21,17 +21,17 @@  discard block
 block discarded – undo
21 21
     // logger API
22 22
     public function debug($message, $context = array())
23 23
     {
24
-        $this->debugBuffer .= $message . "\n";
24
+        $this->debugBuffer .= $message."\n";
25 25
     }
26 26
 
27 27
     public function error($message, $context = array())
28 28
     {
29
-        $this->errorBuffer .= $message . "\n";
29
+        $this->errorBuffer .= $message."\n";
30 30
     }
31 31
 
32 32
     public function warning($message, $context = array())
33 33
     {
34
-        $this->warningBuffer .= $message . "\n";
34
+        $this->warningBuffer .= $message."\n";
35 35
     }
36 36
 
37 37
     public function getDebug()
@@ -62,8 +62,8 @@  discard block
 block discarded – undo
62 62
 
63 63
 $input = array(
64 64
     array('name' => 'Dave', 'age' => 24),
65
-    array('name' => 'Edd',  'age' => 45),
66
-    array('name' => 'Joe',  'age' => 37),
65
+    array('name' => 'Edd', 'age' => 45),
66
+    array('name' => 'Joe', 'age' => 37),
67 67
     array('name' => 'Fred', 'age' => 27),
68 68
 );
69 69
 
@@ -85,6 +85,6 @@  discard block
 block discarded – undo
85 85
 $response = $client->send($request);
86 86
 output("Response received.<br>");
87 87
 
88
-output("The client error info is:<pre>\n" . $logger->getError() . "\n</pre>");
89
-output("The client warning info is:<pre>\n" . $logger->getWarning() . "\n</pre>");
90
-output("The client debug info is:<pre>\n" . $logger->getDebug() . "\n</pre>");
88
+output("The client error info is:<pre>\n".$logger->getError()."\n</pre>");
89
+output("The client warning info is:<pre>\n".$logger->getWarning()."\n</pre>");
90
+output("The client debug info is:<pre>\n".$logger->getDebug()."\n</pre>");
Please login to merge, or discard this patch.
demo/client/symfony/ClientController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
         if ($response->faultCode()) {
29 29
             throw new HttpException(502, $response->faultString());
30 30
         } else {
31
-            return new Response("<html><body>State number $stateNo is: " . $response->value()->scalarVal() . '</body></html>');
31
+            return new Response("<html><body>State number $stateNo is: ".$response->value()->scalarVal().'</body></html>');
32 32
         }
33 33
     }
34 34
 }
Please login to merge, or discard this patch.
demo/server/codegen.php 1 patch
Spacing   +17 added lines, -17 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
 /**
5 5
  * Demoing the code-generation capabilities of the library: create all that is required to expose as xml-rpc methods
@@ -38,33 +38,33 @@  discard block
 block discarded – undo
38 38
 // generate a file with a class definition
39 39
 
40 40
 // the generated code does not have an autoloader included - we need to add in one
41
-$autoloader = __DIR__ . "/_prepend.php";
41
+$autoloader = __DIR__."/_prepend.php";
42 42
 
43 43
 file_put_contents($targetClassFile,
44
-    "<?php\n\n" .
45
-    "require_once '$autoloader';\n\n" .
44
+    "<?php\n\n".
45
+    "require_once '$autoloader';\n\n".
46 46
     "class MyServerClass\n{\n\n"
47 47
 ) || die('uh oh');
48 48
 
49 49
 // we mangle a bit the code we get from wrapPhpClass to turn it into a php class definition instead of a bunch of functions
50 50
 
51
-foreach($code as $methodName => $methodDef) {
52
-    file_put_contents($targetClassFile, '  ' . str_replace(array('function ', "\n"), array('public static function ', "\n  "), $methodDef['source']) . "\n\n", FILE_APPEND) || die('uh oh');
53
-    $code[$methodName]['function'] = 'MyServerClass::' . $methodDef['function'];
51
+foreach ($code as $methodName => $methodDef) {
52
+    file_put_contents($targetClassFile, '  '.str_replace(array('function ', "\n"), array('public static function ', "\n  "), $methodDef['source'])."\n\n", FILE_APPEND) || die('uh oh');
53
+    $code[$methodName]['function'] = 'MyServerClass::'.$methodDef['function'];
54 54
     unset($code[$methodName]['source']);
55 55
 }
56 56
 file_put_contents($targetClassFile, "}\n", FILE_APPEND) || die('uh oh');
57 57
 
58 58
 // generate separate files with the xml-rpc server instantiation and its dispatch map
59 59
 
60
-file_put_contents($targetDispatchMapFile, "<?php\n\nreturn " . var_export($code, true) . ";\n");
60
+file_put_contents($targetDispatchMapFile, "<?php\n\nreturn ".var_export($code, true).";\n");
61 61
 
62 62
 file_put_contents($targetControllerFile,
63
-    "<?php\n\n" .
63
+    "<?php\n\n".
64 64
 
65
-    "require_once '$autoloader';\n\n" .
65
+    "require_once '$autoloader';\n\n".
66 66
 
67
-    "require_once '$targetClassFile';\n\n" .
67
+    "require_once '$targetClassFile';\n\n".
68 68
 
69 69
     // NB: since we are running the generated code within the same script, the existing CommentManager instance will be
70 70
     // available for usage by the methods of MyServerClass, as we keep a reference to them within the variable Wrapper::$objHolder
@@ -74,12 +74,12 @@  discard block
 block discarded – undo
74 74
     //     Wrapper::holdObject('xmlrpc_CommentManager_addComment', $cm);
75 75
     //     Wrapper::holdObject('xmlrpc_CommentManager_getComments', $cm);
76 76
 
77
-    "\$dm = require_once '$targetDispatchMapFile';\n" .
78
-    '$s = new \PhpXmlRpc\Server($dm, false);' . "\n\n" .
79
-    '// NB: do not leave these 2 debug lines enabled on publicly accessible servers!' . "\n" .
80
-    '$s->setOption(\PhpXmlRpc\Server::OPT_DEBUG, 2);' . "\n" .
81
-    '$s->setOption(\PhpXmlRpc\Server::OPT_EXCEPTION_HANDLING, 1);' . "\n\n" .
82
-    '$s->service();' . "\n"
77
+    "\$dm = require_once '$targetDispatchMapFile';\n".
78
+    '$s = new \PhpXmlRpc\Server($dm, false);'."\n\n".
79
+    '// NB: do not leave these 2 debug lines enabled on publicly accessible servers!'."\n".
80
+    '$s->setOption(\PhpXmlRpc\Server::OPT_DEBUG, 2);'."\n".
81
+    '$s->setOption(\PhpXmlRpc\Server::OPT_EXCEPTION_HANDLING, 1);'."\n\n".
82
+    '$s->service();'."\n"
83 83
 ) || die('uh oh');
84 84
 
85 85
 // test that everything worked by running it in realtime (note that this script will return an xml-rpc error message if
Please login to merge, or discard this patch.
lib/xmlrpc.inc 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
         $val = reset($ar);
130 130
         $typ = key($ar);
131 131
 
132
-        return '<value>' . $this->serializeData($typ, $val) . "</value>\n";
132
+        return '<value>'.$this->serializeData($typ, $val)."</value>\n";
133 133
         //}
134 134
     }
135 135
 
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
         // preserve back compatibility
149 149
 
150 150
         if (is_array($b)) {
151
-            foreach($b as $id => $cont) {
151
+            foreach ($b as $id => $cont) {
152 152
                 $b[$id] = $cont->scalarVal();
153 153
             }
154 154
         }
@@ -156,10 +156,10 @@  discard block
 block discarded – undo
156 156
         // add support for structures directly encoding php objects
157 157
         if (is_object($b)) {
158 158
             $t = get_object_vars($b);
159
-            foreach($t as $id => $cont) {
159
+            foreach ($t as $id => $cont) {
160 160
                 $t[$id] = $cont->scalarVal();
161 161
             }
162
-            foreach($t as $id => $cont) {
162
+            foreach ($t as $id => $cont) {
163 163
                 @$b->$id = $cont;
164 164
             }
165 165
         }
@@ -201,17 +201,17 @@  discard block
 block discarded – undo
201 201
 /* Expose as global functions the ones which are now class methods */
202 202
 
203 203
 /// Wrong speling, but we are adamant on backwards compatibility!
204
-function xmlrpc_encode_entitites($data, $srcEncoding='', $destEncoding='')
204
+function xmlrpc_encode_entitites($data, $srcEncoding = '', $destEncoding = '')
205 205
 {
206 206
     return Charset::instance()->encodeEntities($data, $srcEncoding, $destEncoding);
207 207
 }
208 208
 
209
-function iso8601_encode($timeT, $utc=0)
209
+function iso8601_encode($timeT, $utc = 0)
210 210
 {
211 211
     return Date::iso8601Encode($timeT, $utc);
212 212
 }
213 213
 
214
-function iso8601_decode($iDate, $utc=0)
214
+function iso8601_decode($iDate, $utc = 0)
215 215
 {
216 216
     return Date::iso8601Decode($iDate, $utc);
217 217
 }
@@ -221,25 +221,25 @@  discard block
 block discarded – undo
221 221
     return Http::decodeChunked($buffer);
222 222
 }
223 223
 
224
-function php_xmlrpc_decode($xmlrpcVal, $options=array())
224
+function php_xmlrpc_decode($xmlrpcVal, $options = array())
225 225
 {
226 226
     $encoder = new Encoder();
227 227
     return $encoder->decode($xmlrpcVal, $options);
228 228
 }
229 229
 
230
-function php_xmlrpc_encode($phpVal, $options=array())
230
+function php_xmlrpc_encode($phpVal, $options = array())
231 231
 {
232 232
     $encoder = new Encoder();
233 233
     return $encoder->encode($phpVal, $options);
234 234
 }
235 235
 
236
-function php_xmlrpc_decode_xml($xmlVal, $options=array())
236
+function php_xmlrpc_decode_xml($xmlVal, $options = array())
237 237
 {
238 238
     $encoder = new Encoder();
239 239
     return $encoder->decodeXml($xmlVal, $options);
240 240
 }
241 241
 
242
-function guess_encoding($httpHeader='', $xmlChunk='', $encodingPrefs=null)
242
+function guess_encoding($httpHeader = '', $xmlChunk = '', $encodingPrefs = null)
243 243
 {
244 244
     return XMLParser::guessEncoding($httpHeader, $xmlChunk, $encodingPrefs);
245 245
 }
Please login to merge, or discard this patch.
demo/client/parallel.php 1 patch
Spacing   +25 added lines, -25 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
 use PhpXmlRpc\Encoder;
5 5
 use PhpXmlRpc\Client;
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
         $handles = array();
34 34
         $curl = curl_multi_init();
35 35
 
36
-        foreach($requests as $k => $req) {
36
+        foreach ($requests as $k => $req) {
37 37
             $req->setDebug($this->debug);
38 38
             $handle = $this->createCurlHandle($req, $method, $this->server, $this->port, $this->path, $opts);
39 39
             curl_multi_add_handle($curl, $handle);
@@ -43,20 +43,20 @@  discard block
 block discarded – undo
43 43
         $running = 0;
44 44
         do {
45 45
             curl_multi_exec($curl, $running);
46
-        } while($running > 0);
46
+        } while ($running>0);
47 47
 
48 48
         $responses = array();
49 49
         $errors = array();
50
-        foreach($handles as $k => $h) {
50
+        foreach ($handles as $k => $h) {
51 51
             $responses[$k] = curl_multi_getcontent($handles[$k]);
52 52
 
53
-            if ($this->debug > 1) {
53
+            if ($this->debug>1) {
54 54
                 $message = "---CURL INFO---\n";
55 55
                 foreach (curl_getinfo($h) as $name => $val) {
56 56
                     if (is_array($val)) {
57 57
                         $val = implode("\n", $val);
58 58
                     }
59
-                    $message .= $name . ': ' . $val . "\n";
59
+                    $message .= $name.': '.$val."\n";
60 60
                 }
61 61
                 $message .= '---END---';
62 62
                 $this->getLogger()->debugMessage($message);
@@ -71,9 +71,9 @@  discard block
 block discarded – undo
71 71
         }
72 72
         curl_multi_close($curl);
73 73
 
74
-        foreach($responses as $k => $resp) {
74
+        foreach ($responses as $k => $resp) {
75 75
             if (!$resp) {
76
-                $responses[$k] = new Response(0, PhpXmlRpc::$xmlrpcerr['curl_fail'], PhpXmlRpc::$xmlrpcstr['curl_fail'] . ': ' . $errors[$k]);
76
+                $responses[$k] = new Response(0, PhpXmlRpc::$xmlrpcerr['curl_fail'], PhpXmlRpc::$xmlrpcstr['curl_fail'].': '.$errors[$k]);
77 77
             } else {
78 78
                 $responses[$k] = $requests[$k]->parseResponse($resp, true, $this->return_type);
79 79
             }
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 $value = $encoder->encode($data, array('auto_dates'));
94 94
 $req = new Request('interopEchoTests.echoValue', array($value));
95 95
 $reqs = array();
96
-for ($i = 0; $i < $num_tests; $i++) {
96
+for ($i = 0; $i<$num_tests; $i++) {
97 97
     $reqs[] = $req;
98 98
 }
99 99
 
@@ -102,40 +102,40 @@  discard block
 block discarded – undo
102 102
 // avoid storing http info in the responses, to make the checksums comparable
103 103
 $client->setDebug(-1);
104 104
 
105
-echo "Making $num_tests calls to method interopEchoTests.echoValue on server " . XMLRPCSERVER . " ...\n";
105
+echo "Making $num_tests calls to method interopEchoTests.echoValue on server ".XMLRPCSERVER." ...\n";
106 106
 flush();
107 107
 
108
-$client->setOption(Client::OPT_NO_MULTICALL,  true);
108
+$client->setOption(Client::OPT_NO_MULTICALL, true);
109 109
 $t = microtime(true);
110 110
 $resp = $client->send($reqs);
111
-$t = microtime(true) - $t;
112
-echo "Sequential send: " . sprintf('%.3f', $t) . " secs.\n";
113
-echo "Response checksum: " . md5(var_export($resp, true)) . "\n";
111
+$t = microtime(true)-$t;
112
+echo "Sequential send: ".sprintf('%.3f', $t)." secs.\n";
113
+echo "Response checksum: ".md5(var_export($resp, true))."\n";
114 114
 flush();
115 115
 
116 116
 if (strpos(XMLRPCSERVER, 'http://') === 0) {
117
-    $client->setOption(Client::OPT_USE_CURL,  Client::USE_CURL_ALWAYS);
117
+    $client->setOption(Client::OPT_USE_CURL, Client::USE_CURL_ALWAYS);
118 118
     $t = microtime(true);
119 119
     $resp = $client->send($reqs);
120
-    $t = microtime(true) - $t;
121
-    echo "Sequential send, curl (w. keepalive): " . sprintf('%.3f', $t) . " secs.\n";
122
-    echo "Response checksum: " . md5(var_export($resp, true)) . "\n";
120
+    $t = microtime(true)-$t;
121
+    echo "Sequential send, curl (w. keepalive): ".sprintf('%.3f', $t)." secs.\n";
122
+    echo "Response checksum: ".md5(var_export($resp, true))."\n";
123 123
     flush();
124 124
 }
125 125
 
126 126
 $t = microtime(true);
127 127
 $resp = $client->sendParallel($reqs);
128
-$t = microtime(true) - $t;
129
-echo "Parallel send: " . sprintf('%.3f', $t) . " secs.\n";
130
-echo "Response checksum: " . md5(var_export($resp, true)) . "\n";
128
+$t = microtime(true)-$t;
129
+echo "Parallel send: ".sprintf('%.3f', $t)." secs.\n";
130
+echo "Response checksum: ".md5(var_export($resp, true))."\n";
131 131
 flush();
132 132
 
133 133
 $client->setOption(Client::OPT_NO_MULTICALL, false);
134 134
 // make sure we don't reuse the keepalive handle
135
-$client->setOption(Client::OPT_USE_CURL,  Client::USE_CURL_NEVER);
135
+$client->setOption(Client::OPT_USE_CURL, Client::USE_CURL_NEVER);
136 136
 $t = microtime(true);
137 137
 $resp = $client->send($reqs);
138
-$t = microtime(true) - $t;
139
-echo "Multicall send: " . sprintf('%.3f', $t) . " secs.\n";
140
-echo "Response checksum: " . md5(var_export($resp, true)) . "\n";
138
+$t = microtime(true)-$t;
139
+echo "Multicall send: ".sprintf('%.3f', $t)." secs.\n";
140
+echo "Response checksum: ".md5(var_export($resp, true))."\n";
141 141
 flush();
Please login to merge, or discard this patch.
demo/client/introspect.php 1 patch
Spacing   +9 added lines, -9 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
 output('<html lang="en">
5 5
 <head><title>phpxmlrpc - Introspect demo</title></head>
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 function display_error($r)
18 18
 {
19 19
     output("An error occurred: ");
20
-    output("Code: " . $r->faultCode() . " Reason: '" . $r->faultString() . "'<br/>");
20
+    output("Code: ".$r->faultCode()." Reason: '".$r->faultString()."'<br/>");
21 21
 }
22 22
 
23 23
 $client = new Client(XMLRPCSERVER);
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 $client->setOption(Client::OPT_RETURN_TYPE, XMLRPCParser::RETURN_PHP);
26 26
 
27 27
 // First off, let's retrieve the list of methods available on the remote server
28
-output("<h3>methods available at http://" . $client->getUrl(PHP_URL_HOST) . $client->getUrl(PHP_URL_PATH) . "</h3>\n");
28
+output("<h3>methods available at http://".$client->getUrl(PHP_URL_HOST).$client->getUrl(PHP_URL_PATH)."</h3>\n");
29 29
 $req = new Request('system.listMethods');
30 30
 $resp = $client->send($req);
31 31
 
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 
48 48
     // Then, retrieve the signature and help text of each available method
49 49
     foreach ($v as $methodName) {
50
-        output("<h4>" . htmlspecialchars($methodName) . "</h4>\n");
50
+        output("<h4>".htmlspecialchars($methodName)."</h4>\n");
51 51
         // build requests first, add params later
52 52
         $r1 = new PhpXmlRpc\Request('system.methodHelp');
53 53
         $r2 = new PhpXmlRpc\Request('system.methodSignature');
@@ -83,12 +83,12 @@  discard block
 block discarded – undo
83 83
                         continue;
84 84
                     }
85 85
                     $ret = $sig[0];
86
-                    output("<code>" . htmlspecialchars($ret) . " "
87
-                        . htmlspecialchars($methodName) . "(");
88
-                    if (count($sig) > 1) {
89
-                        for ($k = 1; $k < count($sig); $k++) {
86
+                    output("<code>".htmlspecialchars($ret)." "
87
+                        . htmlspecialchars($methodName)."(");
88
+                    if (count($sig)>1) {
89
+                        for ($k = 1; $k<count($sig); $k++) {
90 90
                             output(htmlspecialchars($sig[$k]));
91
-                            if ($k < count($sig) - 1) {
91
+                            if ($k<count($sig)-1) {
92 92
                                 output(", ");
93 93
                             }
94 94
                         }
Please login to merge, or discard this patch.
src/Client.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -2011,9 +2011,9 @@
 block discarded – undo
2011 2011
      * @return false|\CurlHandle|resource
2012 2012
      */
2013 2013
     protected function prepareCurlHandle($req, $server, $port, $timeout = 0, $username = '', $password = '',
2014
-         $authType = 1, $cert = '', $certPass = '', $caCert = '', $caCertDir = '', $proxyHost = '', $proxyPort = 0,
2015
-         $proxyUsername = '', $proxyPassword = '', $proxyAuthType = 1, $method = 'https', $keepAlive = false, $key = '',
2016
-         $keyPass = '', $sslVersion = 0)
2014
+            $authType = 1, $cert = '', $certPass = '', $caCert = '', $caCertDir = '', $proxyHost = '', $proxyPort = 0,
2015
+            $proxyUsername = '', $proxyPassword = '', $proxyAuthType = 1, $method = 'https', $keepAlive = false, $key = '',
2016
+            $keyPass = '', $sslVersion = 0)
2017 2017
     {
2018 2018
         $this->logDeprecationUnlessCalledBy('sendViaCURL');
2019 2019
 
Please login to merge, or discard this patch.
Spacing   +108 added lines, -108 removed lines patch added patch discarded remove patch
@@ -338,10 +338,10 @@  discard block
 block discarded – undo
338 338
             $server = $parts['host'];
339 339
             $path = isset($parts['path']) ? $parts['path'] : '';
340 340
             if (isset($parts['query'])) {
341
-                $path .= '?' . $parts['query'];
341
+                $path .= '?'.$parts['query'];
342 342
             }
343 343
             if (isset($parts['fragment'])) {
344
-                $path .= '#' . $parts['fragment'];
344
+                $path .= '#'.$parts['fragment'];
345 345
             }
346 346
             if (isset($parts['port'])) {
347 347
                 $port = $parts['port'];
@@ -357,7 +357,7 @@  discard block
 block discarded – undo
357 357
             }
358 358
         }
359 359
         if ($path == '' || $path[0] != '/') {
360
-            $this->path = '/' . $path;
360
+            $this->path = '/'.$path;
361 361
         } else {
362 362
             $this->path = $path;
363 363
         }
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
         //$this->accepted_charset_encodings = $ch->knownCharsets();
390 390
 
391 391
         // initialize user_agent string
392
-        $this->user_agent = PhpXmlRpc::$xmlrpcName . ' ' . PhpXmlRpc::$xmlrpcVersion;
392
+        $this->user_agent = PhpXmlRpc::$xmlrpcName.' '.PhpXmlRpc::$xmlrpcVersion;
393 393
     }
394 394
 
395 395
     /**
@@ -561,7 +561,7 @@  discard block
 block discarded – undo
561 561
      */
562 562
     public function setSSLVerifyPeer($i)
563 563
     {
564
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
564
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
565 565
 
566 566
         $this->verifypeer = $i;
567 567
         return $this;
@@ -578,7 +578,7 @@  discard block
 block discarded – undo
578 578
      */
579 579
     public function setSSLVerifyHost($i)
580 580
     {
581
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
581
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
582 582
 
583 583
         $this->verifyhost = $i;
584 584
         return $this;
@@ -596,7 +596,7 @@  discard block
 block discarded – undo
596 596
      */
597 597
     public function setSSLVersion($i)
598 598
     {
599
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
599
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
600 600
 
601 601
         $this->sslversion = $i;
602 602
         return $this;
@@ -661,7 +661,7 @@  discard block
 block discarded – undo
661 661
      */
662 662
     public function setRequestCompression($compMethod)
663 663
     {
664
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
664
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
665 665
 
666 666
         $this->request_compression = $compMethod;
667 667
         return $this;
@@ -711,7 +711,7 @@  discard block
 block discarded – undo
711 711
      */
712 712
     public function setCurlOptions($options)
713 713
     {
714
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
714
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
715 715
 
716 716
         $this->extracurlopts = $options;
717 717
         return $this;
@@ -725,7 +725,7 @@  discard block
 block discarded – undo
725 725
      */
726 726
     public function setUseCurl($useCurlMode)
727 727
     {
728
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
728
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
729 729
 
730 730
         $this->use_curl = $useCurlMode;
731 731
         return $this;
@@ -742,7 +742,7 @@  discard block
 block discarded – undo
742 742
      */
743 743
     public function setUserAgent($agentString)
744 744
     {
745
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
745
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
746 746
 
747 747
         $this->user_agent = $agentString;
748 748
         return $this;
@@ -774,12 +774,12 @@  discard block
 block discarded – undo
774 774
             }
775 775
         }
776 776
 
777
-        $url = $this->method . '://' . $this->server;
777
+        $url = $this->method.'://'.$this->server;
778 778
         if ($this->port == 0 || ($this->port == 80 && in_array($this->method, array('http', 'http10', 'http11', 'h2c'))) ||
779 779
             ($this->port == 443 && in_array($this->method, array('https', 'h2')))) {
780
-            return $url . $this->path;
780
+            return $url.$this->path;
781 781
         } else {
782
-            return $url . ':' . $this->port . $this->path;
782
+            return $url.':'.$this->port.$this->path;
783 783
         }
784 784
     }
785 785
 
@@ -822,7 +822,7 @@  discard block
 block discarded – undo
822 822
     public function send($req, $timeout = 0, $method = '')
823 823
     {
824 824
         if ($method !== '' || $timeout !== 0) {
825
-            $this->logDeprecation("Using non-default values for arguments 'method' and 'timeout' when calling method " . __METHOD__ . ' is deprecated');
825
+            $this->logDeprecation("Using non-default values for arguments 'method' and 'timeout' when calling method ".__METHOD__.' is deprecated');
826 826
         }
827 827
 
828 828
         // if user does not specify http protocol, use native method of this client
@@ -946,7 +946,7 @@  discard block
 block discarded – undo
946 946
                     $payload = $a;
947 947
                     $encodingHdr = "Content-Encoding: gzip\r\n";
948 948
                 } else {
949
-                    $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': gzencode failure in compressing request');
949
+                    $this->getLogger()->warning('XML-RPC: '.__METHOD__.': gzencode failure in compressing request');
950 950
                 }
951 951
             } else if (function_exists('gzcompress')) {
952 952
                 $a = @gzcompress($payload);
@@ -954,14 +954,14 @@  discard block
 block discarded – undo
954 954
                     $payload = $a;
955 955
                     $encodingHdr = "Content-Encoding: deflate\r\n";
956 956
                 } else {
957
-                    $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': gzcompress failure in compressing request');
957
+                    $this->getLogger()->warning('XML-RPC: '.__METHOD__.': gzcompress failure in compressing request');
958 958
                 }
959 959
             } else {
960
-                $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': desired request compression method is unsupported by this PHP install');
960
+                $this->getLogger()->warning('XML-RPC: '.__METHOD__.': desired request compression method is unsupported by this PHP install');
961 961
             }
962 962
         } else {
963 963
             if ($opts['request_compression'] != '') {
964
-                $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': desired request compression method is unsupported');
964
+                $this->getLogger()->warning('XML-RPC: '.__METHOD__.': desired request compression method is unsupported');
965 965
             }
966 966
         }
967 967
 
@@ -969,16 +969,16 @@  discard block
 block discarded – undo
969 969
         $credentials = '';
970 970
         if ($opts['username'] != '') {
971 971
             if ($opts['authtype'] != 1) {
972
-                $this->getLogger()->error('XML-RPC: ' . __METHOD__ . ': warning. Only Basic auth is supported with HTTP 1.0');
972
+                $this->getLogger()->error('XML-RPC: '.__METHOD__.': warning. Only Basic auth is supported with HTTP 1.0');
973 973
                 return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['unsupported_option'],
974
-                    PhpXmlRpc::$xmlrpcerr['unsupported_option'] . ': only Basic auth is supported with HTTP 1.0');
974
+                    PhpXmlRpc::$xmlrpcerr['unsupported_option'].': only Basic auth is supported with HTTP 1.0');
975 975
             }
976
-            $credentials = 'Authorization: Basic ' . base64_encode($opts['username'] . ':' . $opts['password']) . "\r\n";
976
+            $credentials = 'Authorization: Basic '.base64_encode($opts['username'].':'.$opts['password'])."\r\n";
977 977
         }
978 978
 
979 979
         $acceptedEncoding = '';
980 980
         if (is_array($opts['accepted_compression']) && count($opts['accepted_compression'])) {
981
-            $acceptedEncoding = 'Accept-Encoding: ' . implode(', ', $opts['accepted_compression']) . "\r\n";
981
+            $acceptedEncoding = 'Accept-Encoding: '.implode(', ', $opts['accepted_compression'])."\r\n";
982 982
         }
983 983
 
984 984
         if ($port == 0) {
@@ -999,21 +999,21 @@  discard block
 block discarded – undo
999 999
             } elseif ($method === 'h2') {
1000 1000
                 $protocol = 'https';
1001 1001
             } else if (strpos($protocol, ':') !== false) {
1002
-                $this->getLogger()->error('XML-RPC: ' . __METHOD__ . ": warning - attempted hacking attempt?. The protocol requested for the call is: '$protocol'");
1003
-                return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['unsupported_option'], PhpXmlRpc::$xmlrpcerr['unsupported_option'] .
1002
+                $this->getLogger()->error('XML-RPC: '.__METHOD__.": warning - attempted hacking attempt?. The protocol requested for the call is: '$protocol'");
1003
+                return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['unsupported_option'], PhpXmlRpc::$xmlrpcerr['unsupported_option'].
1004 1004
                     " attempted hacking attempt?. The protocol requested for the call is: '$protocol'");
1005 1005
             }
1006 1006
             /// @todo this does not work atm (tested at least with an http proxy forwarding to an https server) - we
1007 1007
             ///       should implement the CONNECT protocol
1008
-            $uri = $protocol . '://' . $server . ':' . $port . $path;
1008
+            $uri = $protocol.'://'.$server.':'.$port.$path;
1009 1009
             if ($opts['proxy_user'] != '') {
1010 1010
                 if ($opts['proxy_authtype'] != 1) {
1011
-                    $this->getLogger()->error('XML-RPC: ' . __METHOD__ . ': warning. Only Basic auth to proxy is supported with HTTP 1.0');
1011
+                    $this->getLogger()->error('XML-RPC: '.__METHOD__.': warning. Only Basic auth to proxy is supported with HTTP 1.0');
1012 1012
                     return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['unsupported_option'],
1013
-                        PhpXmlRpc::$xmlrpcerr['unsupported_option'] . ': only Basic auth to proxy is supported with socket transport');
1013
+                        PhpXmlRpc::$xmlrpcerr['unsupported_option'].': only Basic auth to proxy is supported with socket transport');
1014 1014
                 }
1015
-                $proxyCredentials = 'Proxy-Authorization: Basic ' . base64_encode($opts['proxy_user'] . ':' .
1016
-                    $opts['proxy_pass']) . "\r\n";
1015
+                $proxyCredentials = 'Proxy-Authorization: Basic '.base64_encode($opts['proxy_user'].':'.
1016
+                    $opts['proxy_pass'])."\r\n";
1017 1017
             }
1018 1018
         } else {
1019 1019
             $connectServer = $server;
@@ -1030,14 +1030,14 @@  discard block
 block discarded – undo
1030 1030
             $version = '';
1031 1031
             foreach ($opts['cookies'] as $name => $cookie) {
1032 1032
                 /// @todo should we sanitize the cookie value on behalf of the user? See setCookie comments
1033
-                $cookieHeader .= ' ' . $name . '=' . $cookie['value'] . ";";
1033
+                $cookieHeader .= ' '.$name.'='.$cookie['value'].";";
1034 1034
             }
1035
-            $cookieHeader = 'Cookie:' . $version . substr($cookieHeader, 0, -1) . "\r\n";
1035
+            $cookieHeader = 'Cookie:'.$version.substr($cookieHeader, 0, -1)."\r\n";
1036 1036
         }
1037 1037
 
1038 1038
         $extraHeaders = '';
1039 1039
         if (is_array($this->extra_headers) && $this->extra_headers) {
1040
-            $extraHeaders = implode("\r\n", $this->extra_headers) . "\r\n";
1040
+            $extraHeaders = implode("\r\n", $this->extra_headers)."\r\n";
1041 1041
         }
1042 1042
 
1043 1043
         // omit port if default
@@ -1045,24 +1045,24 @@  discard block
 block discarded – undo
1045 1045
         if (($port == 80 && in_array($method, array('http', 'http10', 'http11'))) || ($port == 443 && $method == 'https')) {
1046 1046
             $port = '';
1047 1047
         } else {
1048
-            $port = ':' . $port;
1049
-        }
1050
-
1051
-        $op = 'POST ' . $uri . " HTTP/1.0\r\n" .
1052
-            'User-Agent: ' . $opts['user_agent'] . "\r\n" .
1053
-            'Host: ' . $server . $port . "\r\n" .
1054
-            $credentials .
1055
-            $proxyCredentials .
1056
-            $acceptedEncoding .
1057
-            $encodingHdr .
1058
-            'Accept-Charset: ' . implode(',', $opts['accepted_charset_encodings']) . "\r\n" .
1059
-            $cookieHeader .
1060
-            'Content-Type: ' . $req->getContentType() . "\r\n" .
1061
-            $extraHeaders .
1062
-            'Content-Length: ' . strlen($payload) . "\r\n\r\n" .
1048
+            $port = ':'.$port;
1049
+        }
1050
+
1051
+        $op = 'POST '.$uri." HTTP/1.0\r\n".
1052
+            'User-Agent: '.$opts['user_agent']."\r\n".
1053
+            'Host: '.$server.$port."\r\n".
1054
+            $credentials.
1055
+            $proxyCredentials.
1056
+            $acceptedEncoding.
1057
+            $encodingHdr.
1058
+            'Accept-Charset: '.implode(',', $opts['accepted_charset_encodings'])."\r\n".
1059
+            $cookieHeader.
1060
+            'Content-Type: '.$req->getContentType()."\r\n".
1061
+            $extraHeaders.
1062
+            'Content-Length: '.strlen($payload)."\r\n\r\n".
1063 1063
             $payload;
1064 1064
 
1065
-        if ($opts['debug'] > 1) {
1065
+        if ($opts['debug']>1) {
1066 1066
             $this->getLogger()->debug("---SENDING---\n$op\n---END---");
1067 1067
         }
1068 1068
 
@@ -1091,13 +1091,13 @@  discard block
 block discarded – undo
1091 1091
                 ///      https://www.php.net/manual/en/function.stream-socket-enable-crypto.php
1092 1092
                 ///      https://www.php.net/manual/en/migration56.openssl.php,
1093 1093
                 ///      https://wiki.php.net/rfc/improved-tls-constants
1094
-                switch($opts['sslversion']) {
1094
+                switch ($opts['sslversion']) {
1095 1095
                     case 1: // TLSv1x
1096 1096
                         if (version_compare(PHP_VERSION, '7.2.0', '>=')) {
1097 1097
                             $contextOptions['ssl']['crypto_method'] = STREAM_CRYPTO_METHOD_TLS_CLIENT;
1098 1098
                         } else {
1099 1099
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['unsupported_option'],
1100
-                                PhpXmlRpc::$xmlrpcerr['unsupported_option'] . ': TLS-any only is supported with PHP 7.2 or later');
1100
+                                PhpXmlRpc::$xmlrpcerr['unsupported_option'].': TLS-any only is supported with PHP 7.2 or later');
1101 1101
                         }
1102 1102
                         break;
1103 1103
                     case 2: // SSLv2
@@ -1120,12 +1120,12 @@  discard block
 block discarded – undo
1120 1120
                             $contextOptions['ssl']['crypto_method'] = STREAM_CRYPTO_METHOD_TLSv1_3_CLIENT;
1121 1121
                         } else {
1122 1122
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['unsupported_option'],
1123
-                                PhpXmlRpc::$xmlrpcerr['unsupported_option'] . ': TLS-1.3 only is supported with PHP 7.4 or later');
1123
+                                PhpXmlRpc::$xmlrpcerr['unsupported_option'].': TLS-1.3 only is supported with PHP 7.4 or later');
1124 1124
                         }
1125 1125
                         break;
1126 1126
                     default:
1127 1127
                         return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['unsupported_option'],
1128
-                            PhpXmlRpc::$xmlrpcerr['unsupported_option'] . ': Unsupported required TLS version');
1128
+                            PhpXmlRpc::$xmlrpcerr['unsupported_option'].': Unsupported required TLS version');
1129 1129
                 }
1130 1130
             }
1131 1131
         }
@@ -1138,7 +1138,7 @@  discard block
 block discarded – undo
1138 1138
 
1139 1139
         $context = stream_context_create($contextOptions);
1140 1140
 
1141
-        if ($opts['timeout'] <= 0) {
1141
+        if ($opts['timeout']<=0) {
1142 1142
             $connectTimeout = ini_get('default_socket_timeout');
1143 1143
         } else {
1144 1144
             $connectTimeout = $opts['timeout'];
@@ -1157,7 +1157,7 @@  discard block
 block discarded – undo
1157 1157
         $fp = @stream_socket_client("$transport://$connectServer:$connectPort", $this->errno, $this->errstr, $connectTimeout,
1158 1158
             STREAM_CLIENT_CONNECT, $context);
1159 1159
         if ($fp) {
1160
-            if ($opts['timeout'] > 0) {
1160
+            if ($opts['timeout']>0) {
1161 1161
                 stream_set_timeout($fp, $opts['timeout'], 0);
1162 1162
             }
1163 1163
         } else {
@@ -1166,8 +1166,8 @@  discard block
 block discarded – undo
1166 1166
                 $this->errstr = $err['message'];
1167 1167
             }
1168 1168
 
1169
-            $this->errstr = 'Connect error: ' . $this->errstr;
1170
-            return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['http_error'], $this->errstr . ' (' . $this->errno . ')');
1169
+            $this->errstr = 'Connect error: '.$this->errstr;
1170
+            return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['http_error'], $this->errstr.' ('.$this->errno.')');
1171 1171
         }
1172 1172
 
1173 1173
         /// @todo from here onwards, we can inject the results of stream_get_meta_data in the response. We could
@@ -1250,19 +1250,19 @@  discard block
 block discarded – undo
1250 1250
             $opts['keepalive'], $opts['key'], $opts['keypass'], $opts['sslversion']);
1251 1251
 
1252 1252
         if (!$curl) {
1253
-            return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['curl_fail'], PhpXmlRpc::$xmlrpcstr['curl_fail'] .
1253
+            return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['curl_fail'], PhpXmlRpc::$xmlrpcstr['curl_fail'].
1254 1254
                 ': error during curl initialization. Check php error log for details');
1255 1255
         }
1256 1256
 
1257 1257
         $result = curl_exec($curl);
1258 1258
 
1259
-        if ($opts['debug'] > 1) {
1259
+        if ($opts['debug']>1) {
1260 1260
             $message = "---CURL INFO---\n";
1261 1261
             foreach (curl_getinfo($curl) as $name => $val) {
1262 1262
                 if (is_array($val)) {
1263 1263
                     $val = implode("\n", $val);
1264 1264
                 }
1265
-                $message .= $name . ': ' . $val . "\n";
1265
+                $message .= $name.': '.$val."\n";
1266 1266
             }
1267 1267
             $message .= '---END---';
1268 1268
             $this->getLogger()->debug($message);
@@ -1272,8 +1272,8 @@  discard block
 block discarded – undo
1272 1272
             /// @todo we should use a better check here - what if we get back '' or '0'?
1273 1273
 
1274 1274
             $this->errstr = 'no response';
1275
-            $resp = new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['curl_fail'], PhpXmlRpc::$xmlrpcstr['curl_fail'] .
1276
-                ': ' . curl_error($curl));
1275
+            $resp = new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['curl_fail'], PhpXmlRpc::$xmlrpcstr['curl_fail'].
1276
+                ': '.curl_error($curl));
1277 1277
             curl_close($curl);
1278 1278
             if ($opts['keepalive']) {
1279 1279
                 $this->xmlrpc_curl_handle = null;
@@ -1331,7 +1331,7 @@  discard block
 block discarded – undo
1331 1331
                     $payload = $a;
1332 1332
                     $encodingHdr = 'Content-Encoding: gzip';
1333 1333
                 } else {
1334
-                    $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': gzencode failure in compressing request');
1334
+                    $this->getLogger()->warning('XML-RPC: '.__METHOD__.': gzencode failure in compressing request');
1335 1335
                 }
1336 1336
             } else if (function_exists('gzcompress')) {
1337 1337
                 $a = @gzcompress($payload);
@@ -1339,14 +1339,14 @@  discard block
 block discarded – undo
1339 1339
                     $payload = $a;
1340 1340
                     $encodingHdr = 'Content-Encoding: deflate';
1341 1341
                 } else {
1342
-                    $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': gzcompress failure in compressing request');
1342
+                    $this->getLogger()->warning('XML-RPC: '.__METHOD__.': gzcompress failure in compressing request');
1343 1343
                 }
1344 1344
             } else {
1345
-                $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': desired request compression method is unsupported by this PHP install');
1345
+                $this->getLogger()->warning('XML-RPC: '.__METHOD__.': desired request compression method is unsupported by this PHP install');
1346 1346
             }
1347 1347
         } else {
1348 1348
             if ($opts['request_compression'] != '') {
1349
-                $this->getLogger()->warning('XML-RPC: ' . __METHOD__ . ': desired request compression method is unsupported');
1349
+                $this->getLogger()->warning('XML-RPC: '.__METHOD__.': desired request compression method is unsupported');
1350 1350
             }
1351 1351
         }
1352 1352
 
@@ -1360,12 +1360,12 @@  discard block
 block discarded – undo
1360 1360
                     // http, https
1361 1361
                     $protocol = $method;
1362 1362
                     if (strpos($protocol, ':') !== false) {
1363
-                        $this->getLogger()->error('XML-RPC: ' . __METHOD__ . ": warning - attempted hacking attempt?. The curl protocol requested for the call is: '$protocol'");
1363
+                        $this->getLogger()->error('XML-RPC: '.__METHOD__.": warning - attempted hacking attempt?. The curl protocol requested for the call is: '$protocol'");
1364 1364
                         return false;
1365 1365
                     }
1366 1366
                 }
1367 1367
             }
1368
-            $curl = curl_init($protocol . '://' . $server . ':' . $port . $path);
1368
+            $curl = curl_init($protocol.'://'.$server.':'.$port.$path);
1369 1369
             if (!$curl) {
1370 1370
                 return false;
1371 1371
             }
@@ -1379,7 +1379,7 @@  discard block
 block discarded – undo
1379 1379
         // results into variable
1380 1380
         curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
1381 1381
 
1382
-        if ($opts['debug'] > 1) {
1382
+        if ($opts['debug']>1) {
1383 1383
             curl_setopt($curl, CURLOPT_VERBOSE, true);
1384 1384
             /// @todo redirect curlopt_stderr to some stream which can be piped to the logger
1385 1385
         }
@@ -1404,7 +1404,7 @@  discard block
 block discarded – undo
1404 1404
             }
1405 1405
         }
1406 1406
         // extra headers
1407
-        $headers = array('Content-Type: ' . $req->getContentType(), 'Accept-Charset: ' . implode(',', $opts['accepted_charset_encodings']));
1407
+        $headers = array('Content-Type: '.$req->getContentType(), 'Accept-Charset: '.implode(',', $opts['accepted_charset_encodings']));
1408 1408
         // if no keepalive is wanted, let the server know it in advance
1409 1409
         if (!$opts['keepalive']) {
1410 1410
             $headers[] = 'Connection: close';
@@ -1441,7 +1441,7 @@  discard block
 block discarded – undo
1441 1441
                 if (defined('CURL_HTTP_VERSION_2_PRIOR_KNOWLEDGE')) {
1442 1442
                     curl_setopt($curl, CURLOPT_HTTP_VERSION, CURL_HTTP_VERSION_2_PRIOR_KNOWLEDGE);
1443 1443
                 } else {
1444
-                    $this->getLogger()->error('XML-RPC: ' . __METHOD__ . ': warning. HTTP2 is not supported by the current PHP/curl install');
1444
+                    $this->getLogger()->error('XML-RPC: '.__METHOD__.': warning. HTTP2 is not supported by the current PHP/curl install');
1445 1445
                     curl_close($curl);
1446 1446
                     return false;
1447 1447
                 }
@@ -1452,11 +1452,11 @@  discard block
 block discarded – undo
1452 1452
         }
1453 1453
 
1454 1454
         if ($opts['username'] && $opts['password']) {
1455
-            curl_setopt($curl, CURLOPT_USERPWD, $opts['username'] . ':' . $opts['password']);
1455
+            curl_setopt($curl, CURLOPT_USERPWD, $opts['username'].':'.$opts['password']);
1456 1456
             if (defined('CURLOPT_HTTPAUTH')) {
1457 1457
                 curl_setopt($curl, CURLOPT_HTTPAUTH, $opts['authtype']);
1458 1458
             } elseif ($opts['authtype'] != 1) {
1459
-                $this->getLogger()->error('XML-RPC: ' . __METHOD__ . ': warning. Only Basic auth is supported by the current PHP/curl install');
1459
+                $this->getLogger()->error('XML-RPC: '.__METHOD__.': warning. Only Basic auth is supported by the current PHP/curl install');
1460 1460
                 curl_close($curl);
1461 1461
                 return false;
1462 1462
             }
@@ -1501,13 +1501,13 @@  discard block
 block discarded – undo
1501 1501
             if ($opts['proxyport'] == 0) {
1502 1502
                 $opts['proxyport'] = 8080; // NB: even for HTTPS, local connection is on port 8080
1503 1503
             }
1504
-            curl_setopt($curl, CURLOPT_PROXY, $opts['proxy'] . ':' . $opts['proxyport']);
1504
+            curl_setopt($curl, CURLOPT_PROXY, $opts['proxy'].':'.$opts['proxyport']);
1505 1505
             if ($opts['proxy_user']) {
1506
-                curl_setopt($curl, CURLOPT_PROXYUSERPWD, $opts['proxy_user'] . ':' . $opts['proxy_pass']);
1506
+                curl_setopt($curl, CURLOPT_PROXYUSERPWD, $opts['proxy_user'].':'.$opts['proxy_pass']);
1507 1507
                 if (defined('CURLOPT_PROXYAUTH')) {
1508 1508
                     curl_setopt($curl, CURLOPT_PROXYAUTH, $opts['proxy_authtype']);
1509 1509
                 } elseif ($opts['proxy_authtype'] != 1) {
1510
-                    $this->getLogger()->error('XML-RPC: ' . __METHOD__ . ': warning. Only Basic auth to proxy is supported by the current PHP/curl install');
1510
+                    $this->getLogger()->error('XML-RPC: '.__METHOD__.': warning. Only Basic auth to proxy is supported by the current PHP/curl install');
1511 1511
                     curl_close($curl);
1512 1512
                     return false;
1513 1513
                 }
@@ -1519,7 +1519,7 @@  discard block
 block discarded – undo
1519 1519
         if (count($opts['cookies'])) {
1520 1520
             $cookieHeader = '';
1521 1521
             foreach ($opts['cookies'] as $name => $cookie) {
1522
-                $cookieHeader .= $name . '=' . $cookie['value'] . '; ';
1522
+                $cookieHeader .= $name.'='.$cookie['value'].'; ';
1523 1523
             }
1524 1524
             curl_setopt($curl, CURLOPT_COOKIE, substr($cookieHeader, 0, -2));
1525 1525
         }
@@ -1528,7 +1528,7 @@  discard block
 block discarded – undo
1528 1528
             curl_setopt($curl, $opt, $val);
1529 1529
         }
1530 1530
 
1531
-        if ($opts['debug'] > 1) {
1531
+        if ($opts['debug']>1) {
1532 1532
             $this->getLogger()->debug("---SENDING---\n$payload\n---END---");
1533 1533
         }
1534 1534
 
@@ -1629,7 +1629,7 @@  discard block
 block discarded – undo
1629 1629
             $call['methodName'] = new Value($req->method(), 'string');
1630 1630
             $numParams = $req->getNumParams();
1631 1631
             $params = array();
1632
-            for ($i = 0; $i < $numParams; $i++) {
1632
+            for ($i = 0; $i<$numParams; $i++) {
1633 1633
                 $params[$i] = $req->getParam($i);
1634 1634
             }
1635 1635
             $call['params'] = new Value($params, 'array');
@@ -1651,7 +1651,7 @@  discard block
 block discarded – undo
1651 1651
         $response = array();
1652 1652
 
1653 1653
         if ($this->return_type == 'xml') {
1654
-            for ($i = 0; $i < count($reqs); $i++) {
1654
+            for ($i = 0; $i<count($reqs); $i++) {
1655 1655
                 $response[] = new static::$responseClass($rets, 0, '', 'xml', $result->httpResponse());
1656 1656
             }
1657 1657
 
@@ -1659,21 +1659,21 @@  discard block
 block discarded – undo
1659 1659
             if (!is_array($rets)) {
1660 1660
                 // bad return type from system.multicall
1661 1661
                 return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1662
-                    PhpXmlRpc::$xmlrpcstr['multicall_error'] . ': not an array', 'phpvals', $result->httpResponse());
1662
+                    PhpXmlRpc::$xmlrpcstr['multicall_error'].': not an array', 'phpvals', $result->httpResponse());
1663 1663
             }
1664 1664
             $numRets = count($rets);
1665 1665
             if ($numRets != count($reqs)) {
1666 1666
                 // wrong number of return values.
1667 1667
                 return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1668
-                    PhpXmlRpc::$xmlrpcstr['multicall_error'] . ': incorrect number of responses', 'phpvals',
1668
+                    PhpXmlRpc::$xmlrpcstr['multicall_error'].': incorrect number of responses', 'phpvals',
1669 1669
                     $result->httpResponse());
1670 1670
             }
1671 1671
 
1672
-            for ($i = 0; $i < $numRets; $i++) {
1672
+            for ($i = 0; $i<$numRets; $i++) {
1673 1673
                 $val = $rets[$i];
1674 1674
                 if (!is_array($val)) {
1675 1675
                     return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1676
-                        PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i is not an array or struct",
1676
+                        PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i is not an array or struct",
1677 1677
                         'phpvals', $result->httpResponse());
1678 1678
                 }
1679 1679
                 switch (count($val)) {
@@ -1681,7 +1681,7 @@  discard block
 block discarded – undo
1681 1681
                         if (!isset($val[0])) {
1682 1682
                             // Bad value
1683 1683
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1684
-                                PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has no value",
1684
+                                PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has no value",
1685 1685
                                 'phpvals', $result->httpResponse());
1686 1686
                         }
1687 1687
                         // Normal return value
@@ -1693,20 +1693,20 @@  discard block
 block discarded – undo
1693 1693
                         if (!is_int($code)) {
1694 1694
                             /// @todo should we check that it is != 0?
1695 1695
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1696
-                                PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has invalid or no faultCode",
1696
+                                PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has invalid or no faultCode",
1697 1697
                                 'phpvals', $result->httpResponse());
1698 1698
                         }
1699 1699
                         $str = @$val['faultString'];
1700 1700
                         if (!is_string($str)) {
1701 1701
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1702
-                                PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has invalid or no FaultString",
1702
+                                PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has invalid or no FaultString",
1703 1703
                                 'phpvals', $result->httpResponse());
1704 1704
                         }
1705 1705
                         $response[$i] = new static::$responseClass(0, $code, $str, 'phpvals', $result->httpResponse());
1706 1706
                         break;
1707 1707
                     default:
1708 1708
                         return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1709
-                            PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has too many items",
1709
+                            PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has too many items",
1710 1710
                             'phpvals', $result->httpResponse());
1711 1711
                 }
1712 1712
             }
@@ -1715,14 +1715,14 @@  discard block
 block discarded – undo
1715 1715
             // return type == 'xmlrpcvals'
1716 1716
             if ($rets->kindOf() != 'array') {
1717 1717
                 return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1718
-                    PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i is not an array", 'xmlrpcvals',
1718
+                    PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i is not an array", 'xmlrpcvals',
1719 1719
                     $result->httpResponse());
1720 1720
             }
1721 1721
             $numRets = $rets->count();
1722 1722
             if ($numRets != count($reqs)) {
1723 1723
                 // wrong number of return values.
1724 1724
                 return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1725
-                    PhpXmlRpc::$xmlrpcstr['multicall_error'] . ': incorrect number of responses', 'xmlrpcvals',
1725
+                    PhpXmlRpc::$xmlrpcstr['multicall_error'].': incorrect number of responses', 'xmlrpcvals',
1726 1726
                     $result->httpResponse());
1727 1727
             }
1728 1728
 
@@ -1731,7 +1731,7 @@  discard block
 block discarded – undo
1731 1731
                     case 'array':
1732 1732
                         if ($val->count() != 1) {
1733 1733
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1734
-                                PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has too many items",
1734
+                                PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has too many items",
1735 1735
                                 'phpvals', $result->httpResponse());
1736 1736
                         }
1737 1737
                         // Normal return value
@@ -1740,28 +1740,28 @@  discard block
 block discarded – undo
1740 1740
                     case 'struct':
1741 1741
                         if ($val->count() != 2) {
1742 1742
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1743
-                                PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has too many items",
1743
+                                PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has too many items",
1744 1744
                                 'phpvals', $result->httpResponse());
1745 1745
                         }
1746 1746
                         /** @var Value $code */
1747 1747
                         $code = $val['faultCode'];
1748 1748
                         if ($code->kindOf() != 'scalar' || $code->scalarTyp() != 'int') {
1749 1749
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1750
-                                PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has invalid or no faultCode",
1750
+                                PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has invalid or no faultCode",
1751 1751
                                 'xmlrpcvals', $result->httpResponse());
1752 1752
                         }
1753 1753
                         /** @var Value $str */
1754 1754
                         $str = $val['faultString'];
1755 1755
                         if ($str->kindOf() != 'scalar' || $str->scalarTyp() != 'string') {
1756 1756
                             return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1757
-                                PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i has invalid or no faultCode",
1757
+                                PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i has invalid or no faultCode",
1758 1758
                                 'xmlrpcvals', $result->httpResponse());
1759 1759
                         }
1760 1760
                         $response[] = new static::$responseClass(0, $code->scalarVal(), $str->scalarVal(), 'xmlrpcvals', $result->httpResponse());
1761 1761
                         break;
1762 1762
                     default:
1763 1763
                         return new static::$responseClass(0, PhpXmlRpc::$xmlrpcerr['multicall_error'],
1764
-                            PhpXmlRpc::$xmlrpcstr['multicall_error'] . ": response element $i is not an array or struct",
1764
+                            PhpXmlRpc::$xmlrpcstr['multicall_error'].": response element $i is not an array or struct",
1765 1765
                             'xmlrpcvals', $result->httpResponse());
1766 1766
                 }
1767 1767
             }
@@ -1796,7 +1796,7 @@  discard block
 block discarded – undo
1796 1796
         $authType = 1, $proxyHost = '', $proxyPort = 0, $proxyUsername = '', $proxyPassword = '', $proxyAuthType = 1,
1797 1797
         $method = 'http')
1798 1798
     {
1799
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
1799
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
1800 1800
 
1801 1801
         return $this->sendPayloadSocket($req, $server, $port, $timeout, $username, $password, $authType, null, null,
1802 1802
             null, null, $proxyHost, $proxyPort, $proxyUsername, $proxyPassword, $proxyAuthType, $method);
@@ -1834,7 +1834,7 @@  discard block
 block discarded – undo
1834 1834
         $proxyUsername = '', $proxyPassword = '', $proxyAuthType = 1, $keepAlive = false, $key = '', $keyPass = '',
1835 1835
         $sslVersion = 0)
1836 1836
     {
1837
-        $this->logDeprecation('Method ' . __METHOD__ . ' is deprecated');
1837
+        $this->logDeprecation('Method '.__METHOD__.' is deprecated');
1838 1838
 
1839 1839
         return $this->sendPayloadCURL($req, $server, $port, $timeout, $username,
1840 1840
             $password, $authType, $cert, $certPass, $caCert, $caCertDir, $proxyHost, $proxyPort,
@@ -2047,7 +2047,7 @@  discard block
 block discarded – undo
2047 2047
     public function &__get($name)
2048 2048
     {
2049 2049
         if (in_array($name, static::$options)) {
2050
-            $this->logDeprecation('Getting property Client::' . $name . ' is deprecated');
2050
+            $this->logDeprecation('Getting property Client::'.$name.' is deprecated');
2051 2051
             return $this->$name;
2052 2052
         }
2053 2053
 
@@ -2058,12 +2058,12 @@  discard block
 block discarded – undo
2058 2058
             case 'server':
2059 2059
             case 'port':
2060 2060
             case 'path':
2061
-                $this->logDeprecation('Getting property Client::' . $name . ' is deprecated');
2061
+                $this->logDeprecation('Getting property Client::'.$name.' is deprecated');
2062 2062
                 return $this->$name;
2063 2063
             default:
2064 2064
                 /// @todo throw instead? There are very few other places where the lib trigger errors which can potentially reach stdout...
2065 2065
                 $trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);
2066
-                trigger_error('Undefined property via __get(): ' . $name . ' in ' . $trace[0]['file'] . ' on line ' . $trace[0]['line'], E_USER_WARNING);
2066
+                trigger_error('Undefined property via __get(): '.$name.' in '.$trace[0]['file'].' on line '.$trace[0]['line'], E_USER_WARNING);
2067 2067
                 $result = null;
2068 2068
                 return $result;
2069 2069
         }
@@ -2072,7 +2072,7 @@  discard block
 block discarded – undo
2072 2072
     public function __set($name, $value)
2073 2073
     {
2074 2074
         if (in_array($name, static::$options)) {
2075
-            $this->logDeprecation('Setting property Client::' . $name . ' is deprecated');
2075
+            $this->logDeprecation('Setting property Client::'.$name.' is deprecated');
2076 2076
             $this->$name = $value;
2077 2077
             return;
2078 2078
         }
@@ -2084,20 +2084,20 @@  discard block
 block discarded – undo
2084 2084
             case 'server':
2085 2085
             case 'port':
2086 2086
             case 'path':
2087
-                $this->logDeprecation('Setting property Client::' . $name . ' is deprecated');
2087
+                $this->logDeprecation('Setting property Client::'.$name.' is deprecated');
2088 2088
                 $this->$name = $value;
2089 2089
                 return;
2090 2090
             default:
2091 2091
                 /// @todo throw instead? There are very few other places where the lib trigger errors which can potentially reach stdout...
2092 2092
                 $trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);
2093
-                trigger_error('Undefined property via __set(): ' . $name . ' in ' . $trace[0]['file'] . ' on line ' . $trace[0]['line'], E_USER_WARNING);
2093
+                trigger_error('Undefined property via __set(): '.$name.' in '.$trace[0]['file'].' on line '.$trace[0]['line'], E_USER_WARNING);
2094 2094
         }
2095 2095
     }
2096 2096
 
2097 2097
     public function __isset($name)
2098 2098
     {
2099 2099
         if (in_array($name, static::$options)) {
2100
-            $this->logDeprecation('Checking property Client::' . $name . ' is deprecated');
2100
+            $this->logDeprecation('Checking property Client::'.$name.' is deprecated');
2101 2101
             return isset($this->$name);
2102 2102
         }
2103 2103
 
@@ -2108,7 +2108,7 @@  discard block
 block discarded – undo
2108 2108
             case 'server':
2109 2109
             case 'port':
2110 2110
             case 'path':
2111
-                $this->logDeprecation('Checking property Client::' . $name . ' is deprecated');
2111
+                $this->logDeprecation('Checking property Client::'.$name.' is deprecated');
2112 2112
                 return isset($this->$name);
2113 2113
             default:
2114 2114
                 return false;
@@ -2118,7 +2118,7 @@  discard block
 block discarded – undo
2118 2118
     public function __unset($name)
2119 2119
     {
2120 2120
         if (in_array($name, static::$options)) {
2121
-            $this->logDeprecation('Unsetting property Client::' . $name . ' is deprecated');
2121
+            $this->logDeprecation('Unsetting property Client::'.$name.' is deprecated');
2122 2122
             unset($this->$name);
2123 2123
             return;
2124 2124
         }
@@ -2130,13 +2130,13 @@  discard block
 block discarded – undo
2130 2130
             case 'server':
2131 2131
             case 'port':
2132 2132
             case 'path':
2133
-                $this->logDeprecation('Unsetting property Client::' . $name . ' is deprecated');
2133
+                $this->logDeprecation('Unsetting property Client::'.$name.' is deprecated');
2134 2134
                 unset($this->$name);
2135 2135
                 return;
2136 2136
             default:
2137 2137
                 /// @todo throw instead? There are very few other places where the lib trigger errors which can potentially reach stdout...
2138 2138
                 $trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);
2139
-                trigger_error('Undefined property via __unset(): ' . $name . ' in ' . $trace[0]['file'] . ' on line ' . $trace[0]['line'], E_USER_WARNING);
2139
+                trigger_error('Undefined property via __unset(): '.$name.' in '.$trace[0]['file'].' on line '.$trace[0]['line'], E_USER_WARNING);
2140 2140
         }
2141 2141
     }
2142 2142
 }
Please login to merge, or discard this patch.
src/Traits/DeprecationLogger.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
             return;
15 15
         }
16 16
 
17
-        $this->getLogger()->warning('XML-RPC Deprecated: ' . $message);
17
+        $this->getLogger()->warning('XML-RPC Deprecated: '.$message);
18 18
     }
19 19
 
20 20
     /**
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
             return;
35 35
         }
36 36
 
37
-        $this->getLogger()->warning('XML-RPC Deprecated: ' . $trace[1]['class'] . '::' . $trace[1]['function'] .
38
-            ' is only supposed to be called by ' . $expectedCaller);
37
+        $this->getLogger()->warning('XML-RPC Deprecated: '.$trace[1]['class'].'::'.$trace[1]['function'].
38
+            ' is only supposed to be called by '.$expectedCaller);
39 39
     }
40 40
 }
Please login to merge, or discard this patch.