View GitHub Repository phpxmlrpc

Schedule Inspection
Subscribe
Passed
Push — master ( 9930b7...ad2d4b )
by Gaetano
17:14 queued 07:13
created
Status
Category
Toggle Patches
tests/12ExtraFilesTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@
 
 block discarded – undo
1 1 
 <?php
2 2
3 
-include_once __DIR__ . '/WebTestCase.php';
3 
+include_once __DIR__.'/WebTestCase.php';
4 4
5 5 
 /**
6 6 
  * Tests for php files in the 'extras' directory.
Please login to merge, or discard this patch.
tests/10DemofilesTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@
 
 block discarded – undo
1 1 
 <?php
2 2
3 
-include_once __DIR__ . '/WebTestCase.php';
3 
+include_once __DIR__.'/WebTestCase.php';
4 4
5 5 
 /**
6 6 
  * Tests for php files in the 'demo' directory.
Please login to merge, or discard this patch.
tests/11DebuggerTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@
 
 block discarded – undo
1 1 
 <?php
2 2
3 
-include_once __DIR__ . '/WebTestCase.php';
3 
+include_once __DIR__.'/WebTestCase.php';
4 4
5 5 
 /**
6 6 
  * Tests for the bundled debugger.
Please login to merge, or discard this patch.
tests/01CharsetTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -3,7 +3,7 @@  discard block
 
 block discarded – undo
3 3 
  * @author JoakimLofgren
4 4 
  */
5 5
6 
-include_once __DIR__ . '/PolyfillTestCase.php';
6 
+include_once __DIR__.'/PolyfillTestCase.php';
7 7
8 8 
 use PhpXmlRpc\Helper\Charset;
9 9 
 
@@ -48,10 +48,10 @@  discard block
 
 block discarded – undo
48 48 
     public function testUtf8ToLatin1All()
49 49 
     {
50 50 
         $latinString = "\n\r\t";
51 
-        for($i = 32; $i < 127; $i++) {
51 
+        for ($i = 32; $i<127; $i++) {
52 52 
             $latinString .= chr($i);
53 53 
         }
54 
-        for($i = 160; $i < 256; $i++) {
54 
+        for ($i = 160; $i<256; $i++) {
55 55 
             $latinString .= chr($i);
56 56 
         }
57 57
Please login to merge, or discard this patch.
tests/03MessagesTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@
 
 block discarded – undo
1 1 
 <?php
2 2
3 
-include_once __DIR__ . '/LoggerAwareTestCase.php';
3 
+include_once __DIR__.'/LoggerAwareTestCase.php';
4 4
5 5 
 /**
6 6 
  * Tests involving Requests and Responses, except for the parsing part
Please login to merge, or discard this patch.
tests/06EncoderTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@  discard block
 
 block discarded – undo
1 1 
 <?php
2 2
3 
-include_once __DIR__ . '/LoggerAwareTestCase.php';
3 
+include_once __DIR__.'/LoggerAwareTestCase.php';
4 4
5 5 
 /**
6 6 
  * Tests involving automatic encoding/decoding of php values into xmlrpc values (the Encoder class).
 
@@ -75,7 +75,7 @@  discard block
 
 block discarded – undo
75 75 
         /// @todo it seems that old php versions can not automatically transform latin to utf8 upon xml parsing.
76 76 
         ///       We should fix that, then re-enable this test
77 77 
         if (version_compare(PHP_VERSION, '5.6.0', '>=')) {
78 
-            $i = $e->decodeXml('<?xml version="1.0" encoding="ISO-8859-15" ?><value><string>' . $string . '</string></value>');
78 
+            $i = $e->decodeXml('<?xml version="1.0" encoding="ISO-8859-15" ?><value><string>'.$string.'</string></value>');
79 79 
             $this->assertEquals($string, $i->scalarVal());
80 80 
         }
81 81
Please login to merge, or discard this patch.
demo/client/wrap.php 1 patch
Spacing   +4 added lines, -4 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 - Webservice wrapper demo</title></head>
 
@@ -20,7 +20,7 @@  discard block
 
 block discarded – undo
20 20 
 $client->setOption(\PhpXmlRpc\Client::OPT_RETURN_TYPE, 'phpvals'); // let client give us back php values instead of xmlrpcvals
21 21 
 $resp = $client->send(new PhpXmlRpc\Request('system.listMethods'));
22 22 
 if ($resp->faultCode()) {
23 
-    output("<p>Server methods list could not be retrieved: error {$resp->faultCode()} '" . htmlspecialchars($resp->faultString()) . "'</p>\n");
23 
+    output("<p>Server methods list could not be retrieved: error {$resp->faultCode()} '".htmlspecialchars($resp->faultString())."'</p>\n");
24 24 
 } else {
25 25 
     output("<p>Server methods list retrieved, now wrapping it up...</p>\n<ul>\n");
26 26 
     flush();
 
@@ -32,9 +32,9 @@  discard block
 
 block discarded – undo
32 32 
         if ($methodName == 'examples.getStateName') {
33 33 
             $callable = $wrapper->wrapXmlrpcMethod($client, $methodName);
34 34 
             if ($callable) {
35 
-                output("<li>Remote server method " . htmlspecialchars($methodName) . " wrapped into php function</li>\n");
35 
+                output("<li>Remote server method ".htmlspecialchars($methodName)." wrapped into php function</li>\n");
36 36 
             } else {
37 
-                output("<li>Remote server method " . htmlspecialchars($methodName) . " could not be wrapped!</li>\n");
37 
+                output("<li>Remote server method ".htmlspecialchars($methodName)." could not be wrapped!</li>\n");
38 38 
             }
39 39 
             break;
40 40 
         }
Please login to merge, or discard this patch.
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.