Inspection of "add a test for the new demo" - gggeek/phpxmlrpc - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b81870...2e14d2 )

by Gaetano

created 2023-01-17 12:11 UTC

Status

Spacing +24 added lines, -24 removed lines patch added patch discarded remove patch

@@ -1,5 +1,5 @@  discard block
 block discarded – undo
 <?php
-require_once __DIR__ . "/client/_prepend.php";
+require_once __DIR__."/client/_prepend.php";
 
 output('<html lang="en">
 <head><title>phpxmlrpc</title></head>
@@ -10,23 +10,23 @@  discard block
 block discarded – undo
 output("<p>Please note that in most cases you are better off using `new PhpXmlRpc\Encoder()->encode()` to create nested Value objects</p>\n");
 
 $v = new PhpXmlRpc\Value(1234, 'int');
-output("Int: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Int: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 $v = new PhpXmlRpc\Value('Are the following characters escaped? < & >');
-output("String <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("String <PRE>".htmlentities($v->serialize())."</PRE>");
 
 $v = new PhpXmlRpc\Value(true, 'boolean');
-output("Boolean: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Boolean: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 $v = new PhpXmlRpc\Value(1234.5678, 'double');
-output("Double: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Double: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 $v = new PhpXmlRpc\Value(time(), 'dateTime.iso8601');
-output("Datetime: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Datetime: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 $v = new PhpXmlRpc\Value('hello world', 'base64');
-output("Base64: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
-output("(value of base64 string is: '" . $v->scalarval() . "')<BR><BR>");
+output("Base64: <PRE>".htmlentities($v->serialize())."</PRE>");
+output("(value of base64 string is: '".$v->scalarval()."')<BR><BR>");
 
 $v = new PhpXmlRpc\Value(
     array(
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
     ),
     "array"
 );
-output("Array: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Array: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 $v = new PhpXmlRpc\Value(
     array(
@@ -59,10 +59,10 @@  discard block
 block discarded – undo
     ),
     "struct"
 );
-output("Struct: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Struct: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 $w = new PhpXmlRpc\Value(array($v), 'array');
-output("Array containing a struct: <PRE>" . htmlentities($w->serialize()) . "</PRE>");
+output("Array containing a struct: <PRE>".htmlentities($w->serialize())."</PRE>");
 
 class MyClass
 {
@@ -74,49 +74,49 @@  discard block
 block discarded – undo
 // the public property is the only one which will be serialized. As such, it has to be of type Value
 $myObject->public = new \PhpXmlRpc\Value('a public property, wrapped');
 $w = new PhpXmlRpc\Value($myObject, 'struct');
-output("Struct encoding a php object: <PRE>" . htmlentities($w->serialize()) . "</PRE>");
+output("Struct encoding a php object: <PRE>".htmlentities($w->serialize())."</PRE>");
 
 output("<h3>Testing value serialization - xml-rpc extensions</h3>\n");
 $v = new PhpXmlRpc\Value(1234, 'i8');
-output("I8: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("I8: <PRE>".htmlentities($v->serialize())."</PRE>");
 $v = new PhpXmlRpc\Value(null, 'null');
-output("Null: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Null: <PRE>".htmlentities($v->serialize())."</PRE>");
 \PhpXmlRpc\PhpXmlRpc::$xmlrpc_null_apache_encoding = true;
-output("Null, alternative: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Null, alternative: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 output("<h3>Testing value serialization - character encoding</h3>\n");
 // The greek word 'kosme'
 $v = new PhpXmlRpc\Value('κόσμε');
-output("Greek (default encoding): <PRE>" . htmlentities($v->serialize()) . "</PRE>");
-output("Greek (utf8 encoding): <PRE>" . htmlentities($v->serialize('UTF-8')) . "</PRE>");
+output("Greek (default encoding): <PRE>".htmlentities($v->serialize())."</PRE>");
+output("Greek (utf8 encoding): <PRE>".htmlentities($v->serialize('UTF-8'))."</PRE>");
 if (function_exists('mb_convert_encoding')) {
-    output("Greek (ISO-8859-7 encoding): <PRE>" . htmlentities($v->serialize('ISO-8859-7')) . "</PRE>");
+    output("Greek (ISO-8859-7 encoding): <PRE>".htmlentities($v->serialize('ISO-8859-7'))."</PRE>");
 }
 
 output("<h3>Testing request serialization</h3>\n");
 $req = new PhpXmlRpc\Request('examples.getStateName');
 $req->method('examples.getStateName');
 $req->addParam(new PhpXmlRpc\Value(42, 'int'));
-output("<PRE>" . htmlentities($req->serialize()) . "</PRE>");
+output("<PRE>".htmlentities($req->serialize())."</PRE>");
 
 output("<h3>Testing response serialization</h3>\n");
 $resp = new PhpXmlRpc\Response(new PhpXmlRpc\Value('The meaning of life'));
-output("<PRE>" . htmlentities($resp->serialize()) . "</PRE>");
+output("<PRE>".htmlentities($resp->serialize())."</PRE>");
 
 output("<h3>Testing ISO date formatting</h3><pre>\n");
 $t = time();
 $date = PhpXmlRpc\Helper\Date::iso8601Encode($t);
 output("Now is $t --> $date\n");
-output("Or in UTC, that is " . PhpXmlRpc\Helper\Date::iso8601Encode($t, 1) . "\n");
+output("Or in UTC, that is ".PhpXmlRpc\Helper\Date::iso8601Encode($t, 1)."\n");
 $tb = PhpXmlRpc\Helper\Date::iso8601Decode($date);
 output("That is to say $date --> $tb\n");
-output("Which comes out at " . PhpXmlRpc\Helper\Date::iso8601Encode($tb) . "\n");
-output("Which was the time in UTC at " . PhpXmlRpc\Helper\Date::iso8601Encode($tb, 1) . "\n");
+output("Which comes out at ".PhpXmlRpc\Helper\Date::iso8601Encode($tb)."\n");
+output("Which was the time in UTC at ".PhpXmlRpc\Helper\Date::iso8601Encode($tb, 1)."\n");
 output("</pre>\n");
 
 output("<h3>Testing reduced-precision formatting for doubles</h3><pre>\n");
 $v = new PhpXmlRpc\Value(1234.56789, 'double');
 \PhpXmlRpc\PhpXmlRpc::$xmlpc_double_precision = 2;
-output("Double, limited precision: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
+output("Double, limited precision: <PRE>".htmlentities($v->serialize())."</PRE>");
 
 output('</body></html>');

Please login to merge, or discard this patch.

demo/server/methodProviders/functions.php 1 patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -54,11 +54,11 @@  discard block
 block discarded – undo
         $snv = $sno->scalarval();
 
         // look it up in our array (zero-based)
-        if (isset(self::$stateNames[$snv - 1])) {
-            $stateName = self::$stateNames[$snv - 1];
+        if (isset(self::$stateNames[$snv-1])) {
+            $stateName = self::$stateNames[$snv-1];
         } else {
             // not there, so complain
-            $err = "I don't have a state for the index '" . $snv . "'";
+            $err = "I don't have a state for the index '".$snv."'";
         }
 
         if ($err != '') {
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
         $s = $req->getParam(0);
         $t = $req->getParam(1);
 
-        return new Response(new Value($s->scalarval() + $t->scalarval(), Value::$xmlrpcInt));
+        return new Response(new Value($s->scalarval()+$t->scalarval(), Value::$xmlrpcInt));
     }
 
     public static $addtwodouble_sig = array(array('double', 'double', 'double'));
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
         $s = $req->getParam(0);
         $t = $req->getParam(1);
 
-        return new Response(new Value($s->scalarval() + $t->scalarval(), Value::$xmlrpcDouble));
+        return new Response(new Value($s->scalarval()+$t->scalarval(), Value::$xmlrpcDouble));
     }
 
     public static $stringecho_sig = array(array('string', 'string'));
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
         ///       In fact, we miss some API (or extra data) in the Request...
         //$payload = file_get_contents('php://input');
         $payload = $req->serialize(PhpXmlRpc::$xmlrpc_internalencoding);
-        $s = "I got the following message:\n" . $payload;
+        $s = "I got the following message:\n".$payload;
 
         return new Response(new Value($s));
     }
@@ -238,17 +238,17 @@  discard block
 block discarded – undo
 
         /// @todo in real life, we should check for presence of return characters to avoid header injection!
 
-        $msgHdr = "From: " . $mFrom->scalarval() . "\n";
-        $msgHdr .= "To: " . $mTo->scalarval() . "\n";
+        $msgHdr = "From: ".$mFrom->scalarval()."\n";
+        $msgHdr .= "To: ".$mTo->scalarval()."\n";
 
         if ($mCc->scalarval() != "") {
-            $msgHdr .= "Cc: " . $mCc->scalarval() . "\n";
+            $msgHdr .= "Cc: ".$mCc->scalarval()."\n";
         }
         if ($mBcc->scalarval() != "") {
-            $msgHdr .= "Bcc: " . $mBcc->scalarval() . "\n";
+            $msgHdr .= "Bcc: ".$mBcc->scalarval()."\n";
         }
         if ($mMime->scalarval() != "") {
-            $msgHdr .= "Content-type: " . $mMime->scalarval() . "\n";
+            $msgHdr .= "Content-type: ".$mMime->scalarval()."\n";
         }
         $msgHdr .= "X-Mailer: XML-RPC for PHP mailer 1.0";
 

Please login to merge, or discard this patch.

demo/server/proxy.php 1 patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -15,7 +15,7 @@  discard block
 block discarded – undo
  * @license code licensed under the BSD License: see file license.txt
  */
 
-require_once __DIR__ . "/_prepend.php";
+require_once __DIR__."/_prepend.php";
 
 // *** NB: WE BLOCK THIS FROM RUNNING BY DEFAULT IN CASE ACCESS IS GRANTED TO IT IN PRODUCTION BY MISTAKE ***
 // Comment out the following safeguard if you want to use it as is, but remember: this is an open relay !!!
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
     //            fe. any url using the 'file://' protocol might be considered a hacking attempt
     $client = new Client($url);
 
-    if ($req->getNumParams() > 3) {
+    if ($req->getNumParams()>3) {
         // We have to set some options onto the client.
         // Note that if we do not untaint the received values, warnings might be generated...
         $options = $encoder->decode($req->getParam(3));
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
                     $client->setSSLVerifyPeer($val);
                     break;
                 case 'Timeout':
-                    $timeout = (integer)$val;
+                    $timeout = (integer) $val;
                     break;
             } // switch
         }
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
     ///         X-forwarded-for anyway, unless they consider this server as trusted...)
     $reqMethod = $req->getParam(1)->scalarval();
     $req = new Request($reqMethod);
-    if ($req->getNumParams() > 1) {
+    if ($req->getNumParams()>1) {
         $pars = $req->getParam(2);
         foreach ($pars as $par) {
             $req->addParam($par);
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
     }
 
     // add debug info into response we give back to caller
-    Server::xmlrpc_debugmsg("Sending to server $url the payload: " . $req->serialize());
+    Server::xmlrpc_debugmsg("Sending to server $url the payload: ".$req->serialize());
 
     return $client->send($req, $timeout);
 }

Please login to merge, or discard this patch.

demo/server/discuss.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -8,7 +8,7 @@  discard block
 block discarded – undo
  *   `$functions_parameters_type` and `$exception_handling`
  */
 
-require_once __DIR__ . "/_prepend.php";
+require_once __DIR__."/_prepend.php";
 
 require_once __DIR__.'/methodProviders/CommentManager.php';
 
@@ -19,12 +19,12 @@  discard block
 block discarded – undo
 
 $addComment_sig = array(array(Value::$xmlrpcInt, Value::$xmlrpcString, Value::$xmlrpcString, Value::$xmlrpcString));
 
-$addComment_doc = 'Adds a comment to an item. The first parameter is the item ID, the second the name of the commenter, ' .
+$addComment_doc = 'Adds a comment to an item. The first parameter is the item ID, the second the name of the commenter, '.
     'and the third is the comment itself. Returns the number of comments against that ID.';
 
 $getComments_sig = array(array(Value::$xmlrpcArray, Value::$xmlrpcString));
 
-$getComments_doc = 'Returns an array of comments for a given ID, which is the sole argument. Each array item is a struct ' .
+$getComments_doc = 'Returns an array of comments for a given ID, which is the sole argument. Each array item is a struct '.
     'containing name and comment text.';
 
 $srv = new Server();

Please login to merge, or discard this patch.

demo/client/windowscharset.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -1,5 +1,5 @@
 block discarded – undo
 <?php
-require_once __DIR__ . "/_prepend.php";
+require_once __DIR__."/_prepend.php";
 
 /**
  * Demoing the charset conversion of the library: create a client class which uses data in the CP-1252 character set,

Please login to merge, or discard this patch.

		@@ -54,11 +54,11 @@ discard block
		block discarded – undo
54	54	$snv = $sno->scalarval();
55	55
56	56	// look it up in our array (zero-based)
57		- if (isset(self::$stateNames[$snv - 1])) {
58		- $stateName = self::$stateNames[$snv - 1];
	57	+ if (isset(self::$stateNames[$snv-1])) {
	58	+ $stateName = self::$stateNames[$snv-1];
59	59	} else {
60	60	// not there, so complain
61		- $err = "I don't have a state for the index '" . $snv . "'";
	61	+ $err = "I don't have a state for the index '".$snv."'";
62	62	}
63	63
64	64	if ($err != '') {
		@@ -132,7 +132,7 @@ discard block
		block discarded – undo
132	132	$s = $req->getParam(0);
133	133	$t = $req->getParam(1);
134	134
135		- return new Response(new Value($s->scalarval() + $t->scalarval(), Value::$xmlrpcInt));
	135	+ return new Response(new Value($s->scalarval()+$t->scalarval(), Value::$xmlrpcInt));
136	136	}
137	137
138	138	public static $addtwodouble_sig = array(array('double', 'double', 'double'));
		@@ -142,7 +142,7 @@ discard block
		block discarded – undo
142	142	$s = $req->getParam(0);
143	143	$t = $req->getParam(1);
144	144
145		- return new Response(new Value($s->scalarval() + $t->scalarval(), Value::$xmlrpcDouble));
	145	+ return new Response(new Value($s->scalarval()+$t->scalarval(), Value::$xmlrpcDouble));
146	146	}
147	147
148	148	public static $stringecho_sig = array(array('string', 'string'));
		@@ -165,7 +165,7 @@ discard block
		block discarded – undo
165	165	/// In fact, we miss some API (or extra data) in the Request...
166	166	//$payload = file_get_contents('php://input');
167	167	$payload = $req->serialize(PhpXmlRpc::$xmlrpc_internalencoding);
168		- $s = "I got the following message:\n" . $payload;
	168	+ $s = "I got the following message:\n".$payload;
169	169
170	170	return new Response(new Value($s));
171	171	}
		@@ -238,17 +238,17 @@ discard block
		block discarded – undo
238	238
239	239	/// @todo in real life, we should check for presence of return characters to avoid header injection!
240	240
241		- $msgHdr = "From: " . $mFrom->scalarval() . "\n";
242		- $msgHdr .= "To: " . $mTo->scalarval() . "\n";
	241	+ $msgHdr = "From: ".$mFrom->scalarval()."\n";
	242	+ $msgHdr .= "To: ".$mTo->scalarval()."\n";
243	243
244	244	if ($mCc->scalarval() != "") {
245		- $msgHdr .= "Cc: " . $mCc->scalarval() . "\n";
	245	+ $msgHdr .= "Cc: ".$mCc->scalarval()."\n";
246	246	}
247	247	if ($mBcc->scalarval() != "") {
248		- $msgHdr .= "Bcc: " . $mBcc->scalarval() . "\n";
	248	+ $msgHdr .= "Bcc: ".$mBcc->scalarval()."\n";
249	249	}
250	250	if ($mMime->scalarval() != "") {
251		- $msgHdr .= "Content-type: " . $mMime->scalarval() . "\n";
	251	+ $msgHdr .= "Content-type: ".$mMime->scalarval()."\n";
252	252	}
253	253	$msgHdr .= "X-Mailer: XML-RPC for PHP mailer 1.0";
254	254

		@@ -1,5 +1,5 @@ discard block
		block discarded – undo
1	1	<?php
2		-require_once __DIR__ . "/client/_prepend.php";
	2	+require_once __DIR__."/client/_prepend.php";
3	3
4	4	output('<html lang="en">
5	5	<head><title>phpxmlrpc</title></head>
		@@ -10,23 +10,23 @@ discard block
		block discarded – undo
10	10	output("<p>Please note that in most cases you are better off using `new PhpXmlRpc\Encoder()->encode()` to create nested Value objects</p>\n");
11	11
12	12	$v = new PhpXmlRpc\Value(1234, 'int');
13		-output("Int: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	13	+output("Int: <PRE>".htmlentities($v->serialize())."</PRE>");
14	14
15	15	$v = new PhpXmlRpc\Value('Are the following characters escaped? < & >');
16		-output("String <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	16	+output("String <PRE>".htmlentities($v->serialize())."</PRE>");
17	17
18	18	$v = new PhpXmlRpc\Value(true, 'boolean');
19		-output("Boolean: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	19	+output("Boolean: <PRE>".htmlentities($v->serialize())."</PRE>");
20	20
21	21	$v = new PhpXmlRpc\Value(1234.5678, 'double');
22		-output("Double: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	22	+output("Double: <PRE>".htmlentities($v->serialize())."</PRE>");
23	23
24	24	$v = new PhpXmlRpc\Value(time(), 'dateTime.iso8601');
25		-output("Datetime: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	25	+output("Datetime: <PRE>".htmlentities($v->serialize())."</PRE>");
26	26
27	27	$v = new PhpXmlRpc\Value('hello world', 'base64');
28		-output("Base64: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
29		-output("(value of base64 string is: '" . $v->scalarval() . "')<BR><BR>");
	28	+output("Base64: <PRE>".htmlentities($v->serialize())."</PRE>");
	29	+output("(value of base64 string is: '".$v->scalarval()."')<BR><BR>");
30	30
31	31	$v = new PhpXmlRpc\Value(
32	32	array(
		@@ -39,7 +39,7 @@ discard block
		block discarded – undo
39	39	),
40	40	"array"
41	41	);
42		-output("Array: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	42	+output("Array: <PRE>".htmlentities($v->serialize())."</PRE>");
43	43
44	44	$v = new PhpXmlRpc\Value(
45	45	array(
		@@ -59,10 +59,10 @@ discard block
		block discarded – undo
59	59	),
60	60	"struct"
61	61	);
62		-output("Struct: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	62	+output("Struct: <PRE>".htmlentities($v->serialize())."</PRE>");
63	63
64	64	$w = new PhpXmlRpc\Value(array($v), 'array');
65		-output("Array containing a struct: <PRE>" . htmlentities($w->serialize()) . "</PRE>");
	65	+output("Array containing a struct: <PRE>".htmlentities($w->serialize())."</PRE>");
66	66
67	67	class MyClass
68	68	{
		@@ -74,49 +74,49 @@ discard block
		block discarded – undo
74	74	// the public property is the only one which will be serialized. As such, it has to be of type Value
75	75	$myObject->public = new \PhpXmlRpc\Value('a public property, wrapped');
76	76	$w = new PhpXmlRpc\Value($myObject, 'struct');
77		-output("Struct encoding a php object: <PRE>" . htmlentities($w->serialize()) . "</PRE>");
	77	+output("Struct encoding a php object: <PRE>".htmlentities($w->serialize())."</PRE>");
78	78
79	79	output("<h3>Testing value serialization - xml-rpc extensions</h3>\n");
80	80	$v = new PhpXmlRpc\Value(1234, 'i8');
81		-output("I8: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	81	+output("I8: <PRE>".htmlentities($v->serialize())."</PRE>");
82	82	$v = new PhpXmlRpc\Value(null, 'null');
83		-output("Null: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	83	+output("Null: <PRE>".htmlentities($v->serialize())."</PRE>");
84	84	\PhpXmlRpc\PhpXmlRpc::$xmlrpc_null_apache_encoding = true;
85		-output("Null, alternative: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	85	+output("Null, alternative: <PRE>".htmlentities($v->serialize())."</PRE>");
86	86
87	87	output("<h3>Testing value serialization - character encoding</h3>\n");
88	88	// The greek word 'kosme'
89	89	$v = new PhpXmlRpc\Value('κόσμε');
90		-output("Greek (default encoding): <PRE>" . htmlentities($v->serialize()) . "</PRE>");
91		-output("Greek (utf8 encoding): <PRE>" . htmlentities($v->serialize('UTF-8')) . "</PRE>");
	90	+output("Greek (default encoding): <PRE>".htmlentities($v->serialize())."</PRE>");
	91	+output("Greek (utf8 encoding): <PRE>".htmlentities($v->serialize('UTF-8'))."</PRE>");
92	92	if (function_exists('mb_convert_encoding')) {
93		- output("Greek (ISO-8859-7 encoding): <PRE>" . htmlentities($v->serialize('ISO-8859-7')) . "</PRE>");
	93	+ output("Greek (ISO-8859-7 encoding): <PRE>".htmlentities($v->serialize('ISO-8859-7'))."</PRE>");
94	94	}
95	95
96	96	output("<h3>Testing request serialization</h3>\n");
97	97	$req = new PhpXmlRpc\Request('examples.getStateName');
98	98	$req->method('examples.getStateName');
99	99	$req->addParam(new PhpXmlRpc\Value(42, 'int'));
100		-output("<PRE>" . htmlentities($req->serialize()) . "</PRE>");
	100	+output("<PRE>".htmlentities($req->serialize())."</PRE>");
101	101
102	102	output("<h3>Testing response serialization</h3>\n");
103	103	$resp = new PhpXmlRpc\Response(new PhpXmlRpc\Value('The meaning of life'));
104		-output("<PRE>" . htmlentities($resp->serialize()) . "</PRE>");
	104	+output("<PRE>".htmlentities($resp->serialize())."</PRE>");
105	105
106	106	output("<h3>Testing ISO date formatting</h3><pre>\n");
107	107	$t = time();
108	108	$date = PhpXmlRpc\Helper\Date::iso8601Encode($t);
109	109	output("Now is $t --> $date\n");
110		-output("Or in UTC, that is " . PhpXmlRpc\Helper\Date::iso8601Encode($t, 1) . "\n");
	110	+output("Or in UTC, that is ".PhpXmlRpc\Helper\Date::iso8601Encode($t, 1)."\n");
111	111	$tb = PhpXmlRpc\Helper\Date::iso8601Decode($date);
112	112	output("That is to say $date --> $tb\n");
113		-output("Which comes out at " . PhpXmlRpc\Helper\Date::iso8601Encode($tb) . "\n");
114		-output("Which was the time in UTC at " . PhpXmlRpc\Helper\Date::iso8601Encode($tb, 1) . "\n");
	113	+output("Which comes out at ".PhpXmlRpc\Helper\Date::iso8601Encode($tb)."\n");
	114	+output("Which was the time in UTC at ".PhpXmlRpc\Helper\Date::iso8601Encode($tb, 1)."\n");
115	115	output("</pre>\n");
116	116
117	117	output("<h3>Testing reduced-precision formatting for doubles</h3><pre>\n");
118	118	$v = new PhpXmlRpc\Value(1234.56789, 'double');
119	119	\PhpXmlRpc\PhpXmlRpc::$xmlpc_double_precision = 2;
120		-output("Double, limited precision: <PRE>" . htmlentities($v->serialize()) . "</PRE>");
	120	+output("Double, limited precision: <PRE>".htmlentities($v->serialize())."</PRE>");
121	121
122	122	output('</body></html>');

		@@ -15,7 +15,7 @@ discard block
		block discarded – undo
15	15	* @license code licensed under the BSD License: see file license.txt
16	16	*/
17	17
18		-require_once __DIR__ . "/_prepend.php";
	18	+require_once __DIR__."/_prepend.php";
19	19
20	20	// * NB: WE BLOCK THIS FROM RUNNING BY DEFAULT IN CASE ACCESS IS GRANTED TO IT IN PRODUCTION BY MISTAKE *
21	21	// Comment out the following safeguard if you want to use it as is, but remember: this is an open relay !!!
		@@ -46,7 +46,7 @@ discard block
		block discarded – undo
46	46	// fe. any url using the 'file://' protocol might be considered a hacking attempt
47	47	$client = new Client($url);
48	48
49		- if ($req->getNumParams() > 3) {
	49	+ if ($req->getNumParams()>3) {
50	50	// We have to set some options onto the client.
51	51	// Note that if we do not untaint the received values, warnings might be generated...
52	52	$options = $encoder->decode($req->getParam(3));
		@@ -86,7 +86,7 @@ discard block
		block discarded – undo
86	86	$client->setSSLVerifyPeer($val);
87	87	break;
88	88	case 'Timeout':
89		- $timeout = (integer)$val;
	89	+ $timeout = (integer) $val;
90	90	break;
91	91	} // switch
92	92	}
		@@ -99,7 +99,7 @@ discard block
		block discarded – undo
99	99	/// X-forwarded-for anyway, unless they consider this server as trusted...)
100	100	$reqMethod = $req->getParam(1)->scalarval();
101	101	$req = new Request($reqMethod);
102		- if ($req->getNumParams() > 1) {
	102	+ if ($req->getNumParams()>1) {
103	103	$pars = $req->getParam(2);
104	104	foreach ($pars as $par) {
105	105	$req->addParam($par);
		@@ -107,7 +107,7 @@ discard block
		block discarded – undo
107	107	}
108	108
109	109	// add debug info into response we give back to caller
110		- Server::xmlrpc_debugmsg("Sending to server $url the payload: " . $req->serialize());
	110	+ Server::xmlrpc_debugmsg("Sending to server $url the payload: ".$req->serialize());
111	111
112	112	return $client->send($req, $timeout);
113	113	}

		@@ -8,7 +8,7 @@ discard block
		block discarded – undo
8	8	* `$functions_parameters_type` and `$exception_handling`
9	9	*/
10	10
11		-require_once __DIR__ . "/_prepend.php";
	11	+require_once __DIR__."/_prepend.php";
12	12
13	13	require_once __DIR__.'/methodProviders/CommentManager.php';
14	14
		@@ -19,12 +19,12 @@ discard block
		block discarded – undo
19	19
20	20	$addComment_sig = array(array(Value::$xmlrpcInt, Value::$xmlrpcString, Value::$xmlrpcString, Value::$xmlrpcString));
21	21
22		-$addComment_doc = 'Adds a comment to an item. The first parameter is the item ID, the second the name of the commenter, ' .
	22	+$addComment_doc = 'Adds a comment to an item. The first parameter is the item ID, the second the name of the commenter, '.
23	23	'and the third is the comment itself. Returns the number of comments against that ID.';
24	24
25	25	$getComments_sig = array(array(Value::$xmlrpcArray, Value::$xmlrpcString));
26	26
27		-$getComments_doc = 'Returns an array of comments for a given ID, which is the sole argument. Each array item is a struct ' .
	27	+$getComments_doc = 'Returns an array of comments for a given ID, which is the sole argument. Each array item is a struct '.
28	28	'containing name and comment text.';
29	29
30	30	$srv = new Server();

		@@ -1,5 +1,5 @@
		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 charset conversion of the library: create a client class which uses data in the CP-1252 character set,

gggeek / phpxmlrpc

Push — master ( b81870...2e14d2 )

Status

Category

Spacing +24 added lines, -24 removed lines patch added patch discarded remove patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch