Completed
Push — php51 ( 617621...a4e868 )
by Gaetano
16:05 queued 06:04
created
lib/xmlrpcs.inc 1 patch
Spacing   +180 added lines, -180 removed lines patch added patch discarded remove patch
@@ -57,10 +57,10 @@  discard block
 block discarded – undo
57 57
 	);
58 58
 
59 59
 	/* Functions that implement system.XXX methods of xmlrpc servers */
60
-	$_xmlrpcs_getCapabilities_sig=array(array($GLOBALS['xmlrpcStruct']));
61
-	$_xmlrpcs_getCapabilities_doc='This method lists all the capabilites that the XML-RPC server has: the (more or less standard) extensions to the xmlrpc spec that it adheres to';
62
-	$_xmlrpcs_getCapabilities_sdoc=array(array('list of capabilities, described as structs with a version number and url for the spec'));
63
-	function _xmlrpcs_getCapabilities($server, $m=null)
60
+	$_xmlrpcs_getCapabilities_sig = array(array($GLOBALS['xmlrpcStruct']));
61
+	$_xmlrpcs_getCapabilities_doc = 'This method lists all the capabilites that the XML-RPC server has: the (more or less standard) extensions to the xmlrpc spec that it adheres to';
62
+	$_xmlrpcs_getCapabilities_sdoc = array(array('list of capabilities, described as structs with a version number and url for the spec'));
63
+	function _xmlrpcs_getCapabilities($server, $m = null)
64 64
 	{
65 65
 		$outAr = $GLOBALS['xmlrpcs_capabilities'];
66 66
 		// NIL extension
@@ -75,117 +75,117 @@  discard block
 block discarded – undo
75 75
 
76 76
 	// listMethods: signature was either a string, or nothing.
77 77
 	// The useless string variant has been removed
78
-	$_xmlrpcs_listMethods_sig=array(array($GLOBALS['xmlrpcArray']));
79
-	$_xmlrpcs_listMethods_doc='This method lists all the methods that the XML-RPC server knows how to dispatch';
80
-	$_xmlrpcs_listMethods_sdoc=array(array('list of method names'));
81
-	function _xmlrpcs_listMethods($server, $m=null) // if called in plain php values mode, second param is missing
78
+	$_xmlrpcs_listMethods_sig = array(array($GLOBALS['xmlrpcArray']));
79
+	$_xmlrpcs_listMethods_doc = 'This method lists all the methods that the XML-RPC server knows how to dispatch';
80
+	$_xmlrpcs_listMethods_sdoc = array(array('list of method names'));
81
+	function _xmlrpcs_listMethods($server, $m = null) // if called in plain php values mode, second param is missing
82 82
 	{
83 83
 
84
-		$outAr=array();
85
-		foreach($server->dmap as $key => $val)
84
+		$outAr = array();
85
+		foreach ($server->dmap as $key => $val)
86 86
 		{
87
-			$outAr[]=new xmlrpcval($key, 'string');
87
+			$outAr[] = new xmlrpcval($key, 'string');
88 88
 		}
89
-		if($server->allow_system_funcs)
89
+		if ($server->allow_system_funcs)
90 90
 		{
91
-			foreach($GLOBALS['_xmlrpcs_dmap'] as $key => $val)
91
+			foreach ($GLOBALS['_xmlrpcs_dmap'] as $key => $val)
92 92
 			{
93
-				$outAr[]=new xmlrpcval($key, 'string');
93
+				$outAr[] = new xmlrpcval($key, 'string');
94 94
 			}
95 95
 		}
96 96
 		return new xmlrpcresp(new xmlrpcval($outAr, 'array'));
97 97
 	}
98 98
 
99
-	$_xmlrpcs_methodSignature_sig=array(array($GLOBALS['xmlrpcArray'], $GLOBALS['xmlrpcString']));
100
-	$_xmlrpcs_methodSignature_doc='Returns an array of known signatures (an array of arrays) for the method name passed. If no signatures are known, returns a none-array (test for type != array to detect missing signature)';
101
-	$_xmlrpcs_methodSignature_sdoc=array(array('list of known signatures, each sig being an array of xmlrpc type names', 'name of method to be described'));
99
+	$_xmlrpcs_methodSignature_sig = array(array($GLOBALS['xmlrpcArray'], $GLOBALS['xmlrpcString']));
100
+	$_xmlrpcs_methodSignature_doc = 'Returns an array of known signatures (an array of arrays) for the method name passed. If no signatures are known, returns a none-array (test for type != array to detect missing signature)';
101
+	$_xmlrpcs_methodSignature_sdoc = array(array('list of known signatures, each sig being an array of xmlrpc type names', 'name of method to be described'));
102 102
 	function _xmlrpcs_methodSignature($server, $m)
103 103
 	{
104 104
 		// let accept as parameter both an xmlrpcval or string
105 105
 		if (is_object($m))
106 106
 		{
107
-			$methName=$m->getParam(0);
108
-			$methName=$methName->scalarval();
107
+			$methName = $m->getParam(0);
108
+			$methName = $methName->scalarval();
109 109
 		}
110 110
 		else
111 111
 		{
112
-			$methName=$m;
112
+			$methName = $m;
113 113
 		}
114
-		if(strpos($methName, "system.") === 0)
114
+		if (strpos($methName, "system.") === 0)
115 115
 		{
116
-			$dmap=$GLOBALS['_xmlrpcs_dmap']; $sysCall=1;
116
+			$dmap = $GLOBALS['_xmlrpcs_dmap']; $sysCall = 1;
117 117
 		}
118 118
 		else
119 119
 		{
120
-			$dmap=$server->dmap; $sysCall=0;
120
+			$dmap = $server->dmap; $sysCall = 0;
121 121
 		}
122
-		if(isset($dmap[$methName]))
122
+		if (isset($dmap[$methName]))
123 123
 		{
124
-			if(isset($dmap[$methName]['signature']))
124
+			if (isset($dmap[$methName]['signature']))
125 125
 			{
126
-				$sigs=array();
127
-				foreach($dmap[$methName]['signature'] as $inSig)
126
+				$sigs = array();
127
+				foreach ($dmap[$methName]['signature'] as $inSig)
128 128
 				{
129
-					$cursig=array();
130
-					foreach($inSig as $sig)
129
+					$cursig = array();
130
+					foreach ($inSig as $sig)
131 131
 					{
132
-						$cursig[]=new xmlrpcval($sig, 'string');
132
+						$cursig[] = new xmlrpcval($sig, 'string');
133 133
 					}
134
-					$sigs[]=new xmlrpcval($cursig, 'array');
134
+					$sigs[] = new xmlrpcval($cursig, 'array');
135 135
 				}
136
-				$r=new xmlrpcresp(new xmlrpcval($sigs, 'array'));
136
+				$r = new xmlrpcresp(new xmlrpcval($sigs, 'array'));
137 137
 			}
138 138
 			else
139 139
 			{
140 140
 				// NB: according to the official docs, we should be returning a
141 141
 				// "none-array" here, which means not-an-array
142
-				$r=new xmlrpcresp(new xmlrpcval('undef', 'string'));
142
+				$r = new xmlrpcresp(new xmlrpcval('undef', 'string'));
143 143
 			}
144 144
 		}
145 145
 		else
146 146
 		{
147
-			$r=new xmlrpcresp(0,$GLOBALS['xmlrpcerr']['introspect_unknown'], $GLOBALS['xmlrpcstr']['introspect_unknown']);
147
+			$r = new xmlrpcresp(0, $GLOBALS['xmlrpcerr']['introspect_unknown'], $GLOBALS['xmlrpcstr']['introspect_unknown']);
148 148
 		}
149 149
 		return $r;
150 150
 	}
151 151
 
152
-	$_xmlrpcs_methodHelp_sig=array(array($GLOBALS['xmlrpcString'], $GLOBALS['xmlrpcString']));
153
-	$_xmlrpcs_methodHelp_doc='Returns help text if defined for the method passed, otherwise returns an empty string';
154
-	$_xmlrpcs_methodHelp_sdoc=array(array('method description', 'name of the method to be described'));
152
+	$_xmlrpcs_methodHelp_sig = array(array($GLOBALS['xmlrpcString'], $GLOBALS['xmlrpcString']));
153
+	$_xmlrpcs_methodHelp_doc = 'Returns help text if defined for the method passed, otherwise returns an empty string';
154
+	$_xmlrpcs_methodHelp_sdoc = array(array('method description', 'name of the method to be described'));
155 155
 	function _xmlrpcs_methodHelp($server, $m)
156 156
 	{
157 157
 		// let accept as parameter both an xmlrpcval or string
158 158
 		if (is_object($m))
159 159
 		{
160
-			$methName=$m->getParam(0);
161
-			$methName=$methName->scalarval();
160
+			$methName = $m->getParam(0);
161
+			$methName = $methName->scalarval();
162 162
 		}
163 163
 		else
164 164
 		{
165
-			$methName=$m;
165
+			$methName = $m;
166 166
 		}
167
-		if(strpos($methName, "system.") === 0)
167
+		if (strpos($methName, "system.") === 0)
168 168
 		{
169
-			$dmap=$GLOBALS['_xmlrpcs_dmap']; $sysCall=1;
169
+			$dmap = $GLOBALS['_xmlrpcs_dmap']; $sysCall = 1;
170 170
 		}
171 171
 		else
172 172
 		{
173
-			$dmap=$server->dmap; $sysCall=0;
173
+			$dmap = $server->dmap; $sysCall = 0;
174 174
 		}
175
-		if(isset($dmap[$methName]))
175
+		if (isset($dmap[$methName]))
176 176
 		{
177
-			if(isset($dmap[$methName]['docstring']))
177
+			if (isset($dmap[$methName]['docstring']))
178 178
 			{
179
-				$r=new xmlrpcresp(new xmlrpcval($dmap[$methName]['docstring']), 'string');
179
+				$r = new xmlrpcresp(new xmlrpcval($dmap[$methName]['docstring']), 'string');
180 180
 			}
181 181
 			else
182 182
 			{
183
-				$r=new xmlrpcresp(new xmlrpcval('', 'string'));
183
+				$r = new xmlrpcresp(new xmlrpcval('', 'string'));
184 184
 			}
185 185
 		}
186 186
 		else
187 187
 		{
188
-			$r=new xmlrpcresp(0, $GLOBALS['xmlrpcerr']['introspect_unknown'], $GLOBALS['xmlrpcstr']['introspect_unknown']);
188
+			$r = new xmlrpcresp(0, $GLOBALS['xmlrpcerr']['introspect_unknown'], $GLOBALS['xmlrpcstr']['introspect_unknown']);
189 189
 		}
190 190
 		return $r;
191 191
 	}
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
 	$_xmlrpcs_multicall_sdoc = array(array('list of response structs, where each struct has the usual members', 'list of calls, with each call being represented as a struct, with members "methodname" and "params"'));
196 196
 	function _xmlrpcs_multicall_error($err)
197 197
 	{
198
-		if(is_string($err))
198
+		if (is_string($err))
199 199
 		{
200 200
 			$str = $GLOBALS['xmlrpcstr']["multicall_${err}"];
201 201
 			$code = $GLOBALS['xmlrpcerr']["multicall_${err}"];
@@ -213,52 +213,52 @@  discard block
 block discarded – undo
213 213
 
214 214
 	function _xmlrpcs_multicall_do_call($server, $call)
215 215
 	{
216
-		if($call->kindOf() != 'struct')
216
+		if ($call->kindOf() != 'struct')
217 217
 		{
218 218
 			return _xmlrpcs_multicall_error('notstruct');
219 219
 		}
220 220
 		$methName = @$call->structmem('methodName');
221
-		if(!$methName)
221
+		if (!$methName)
222 222
 		{
223 223
 			return _xmlrpcs_multicall_error('nomethod');
224 224
 		}
225
-		if($methName->kindOf() != 'scalar' || $methName->scalartyp() != 'string')
225
+		if ($methName->kindOf() != 'scalar' || $methName->scalartyp() != 'string')
226 226
 		{
227 227
 			return _xmlrpcs_multicall_error('notstring');
228 228
 		}
229
-		if($methName->scalarval() == 'system.multicall')
229
+		if ($methName->scalarval() == 'system.multicall')
230 230
 		{
231 231
 			return _xmlrpcs_multicall_error('recursion');
232 232
 		}
233 233
 
234 234
 		$params = @$call->structmem('params');
235
-		if(!$params)
235
+		if (!$params)
236 236
 		{
237 237
 			return _xmlrpcs_multicall_error('noparams');
238 238
 		}
239
-		if($params->kindOf() != 'array')
239
+		if ($params->kindOf() != 'array')
240 240
 		{
241 241
 			return _xmlrpcs_multicall_error('notarray');
242 242
 		}
243 243
 		$numParams = $params->arraysize();
244 244
 
245 245
 		$msg = new xmlrpcmsg($methName->scalarval());
246
-		for($i = 0; $i < $numParams; $i++)
246
+		for ($i = 0; $i<$numParams; $i++)
247 247
 		{
248
-			if(!$msg->addParam($params->arraymem($i)))
248
+			if (!$msg->addParam($params->arraymem($i)))
249 249
 			{
250 250
 				$i++;
251 251
 				return _xmlrpcs_multicall_error(new xmlrpcresp(0,
252 252
 					$GLOBALS['xmlrpcerr']['incorrect_params'],
253
-					$GLOBALS['xmlrpcstr']['incorrect_params'] . ": probable xml error in param " . $i));
253
+					$GLOBALS['xmlrpcstr']['incorrect_params'].": probable xml error in param ".$i));
254 254
 			}
255 255
 		}
256 256
 
257 257
 		$result = $server->execute($msg);
258 258
 
259
-		if($result->faultCode() != 0)
259
+		if ($result->faultCode() != 0)
260 260
 		{
261
-			return _xmlrpcs_multicall_error($result);		// Method returned fault.
261
+			return _xmlrpcs_multicall_error($result); // Method returned fault.
262 262
 		}
263 263
 
264 264
 		return new xmlrpcval(array($result->value()), 'array');
@@ -266,11 +266,11 @@  discard block
 block discarded – undo
266 266
 
267 267
 	function _xmlrpcs_multicall_do_call_phpvals($server, $call)
268 268
 	{
269
-		if(!is_array($call))
269
+		if (!is_array($call))
270 270
 		{
271 271
 			return _xmlrpcs_multicall_error('notstruct');
272 272
 		}
273
-		if(!array_key_exists('methodName', $call))
273
+		if (!array_key_exists('methodName', $call))
274 274
 		{
275 275
 			return _xmlrpcs_multicall_error('nomethod');
276 276
 		}
@@ -278,15 +278,15 @@  discard block
 block discarded – undo
278 278
 		{
279 279
 			return _xmlrpcs_multicall_error('notstring');
280 280
 		}
281
-		if($call['methodName'] == 'system.multicall')
281
+		if ($call['methodName'] == 'system.multicall')
282 282
 		{
283 283
 			return _xmlrpcs_multicall_error('recursion');
284 284
 		}
285
-		if(!array_key_exists('params', $call))
285
+		if (!array_key_exists('params', $call))
286 286
 		{
287 287
 			return _xmlrpcs_multicall_error('noparams');
288 288
 		}
289
-		if(!is_array($call['params']))
289
+		if (!is_array($call['params']))
290 290
 		{
291 291
 			return _xmlrpcs_multicall_error('notarray');
292 292
 		}
@@ -295,14 +295,14 @@  discard block
 block discarded – undo
295 295
 		// base64 or datetime values, but they will be listed as strings here...
296 296
 		$numParams = count($call['params']);
297 297
 		$pt = array();
298
-		foreach($call['params'] as $val)
298
+		foreach ($call['params'] as $val)
299 299
 			$pt[] = php_2_xmlrpc_type(gettype($val));
300 300
 
301 301
 		$result = $server->execute($call['methodName'], $call['params'], $pt);
302 302
 
303
-		if($result->faultCode() != 0)
303
+		if ($result->faultCode() != 0)
304 304
 		{
305
-			return _xmlrpcs_multicall_error($result);		// Method returned fault.
305
+			return _xmlrpcs_multicall_error($result); // Method returned fault.
306 306
 		}
307 307
 
308 308
 		return new xmlrpcval(array($result->value()), 'array');
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
316 316
 		{
317 317
 			$calls = $m->getParam(0);
318 318
 			$numCalls = $calls->arraysize();
319
-			for($i = 0; $i < $numCalls; $i++)
319
+			for ($i = 0; $i<$numCalls; $i++)
320 320
 			{
321 321
 				$call = $calls->arraymem($i);
322 322
 				$result[$i] = _xmlrpcs_multicall_do_call($server, $call);
@@ -324,8 +324,8 @@  discard block
 block discarded – undo
324 324
 		}
325 325
 		else
326 326
 		{
327
-			$numCalls=count($m);
328
-			for($i = 0; $i < $numCalls; $i++)
327
+			$numCalls = count($m);
328
+			for ($i = 0; $i<$numCalls; $i++)
329 329
 			{
330 330
 				$result[$i] = _xmlrpcs_multicall_do_call_phpvals($server, $m[$i]);
331 331
 			}
@@ -334,7 +334,7 @@  discard block
 block discarded – undo
334 334
 		return new xmlrpcresp(new xmlrpcval($result, 'array'));
335 335
 	}
336 336
 
337
-	$GLOBALS['_xmlrpcs_dmap']=array(
337
+	$GLOBALS['_xmlrpcs_dmap'] = array(
338 338
 		'system.listMethods' => array(
339 339
 			'function' => '_xmlrpcs_listMethods',
340 340
 			'signature' => $_xmlrpcs_listMethods_sig,
@@ -374,24 +374,24 @@  discard block
 block discarded – undo
374 374
 	* NB: in fact a user defined error handler can only handle WARNING, NOTICE and USER_* errors.
375 375
 	*
376 376
 	*/
377
-	function _xmlrpcs_errorHandler($errcode, $errstring, $filename=null, $lineno=null, $context=null)
377
+	function _xmlrpcs_errorHandler($errcode, $errstring, $filename = null, $lineno = null, $context = null)
378 378
 	{
379 379
 		// obey the @ protocol
380 380
 		if (error_reporting() == 0)
381 381
 			return;
382 382
 
383 383
 		//if($errcode != E_NOTICE && $errcode != E_WARNING && $errcode != E_USER_NOTICE && $errcode != E_USER_WARNING)
384
-		if($errcode != E_STRICT)
384
+		if ($errcode != E_STRICT)
385 385
 		{
386
-			$GLOBALS['_xmlrpcs_occurred_errors'] = $GLOBALS['_xmlrpcs_occurred_errors'] . $errstring . "\n";
386
+			$GLOBALS['_xmlrpcs_occurred_errors'] = $GLOBALS['_xmlrpcs_occurred_errors'].$errstring."\n";
387 387
 		}
388 388
 		// Try to avoid as much as possible disruption to the previous error handling
389 389
 		// mechanism in place
390
-		if($GLOBALS['_xmlrpcs_prev_ehandler'] == '')
390
+		if ($GLOBALS['_xmlrpcs_prev_ehandler'] == '')
391 391
 		{
392 392
 			// The previous error handler was the default: all we should do is log error
393 393
 			// to the default error log (if level high enough)
394
-			if(ini_get('log_errors') && (intval(ini_get('error_reporting')) & $errcode))
394
+			if (ini_get('log_errors') && (intval(ini_get('error_reporting')) & $errcode))
395 395
 			{
396 396
 				error_log($errstring);
397 397
 			}
@@ -399,10 +399,10 @@  discard block
 block discarded – undo
399 399
 		else
400 400
 		{
401 401
 			// Pass control on to previous error handler, trying to avoid loops...
402
-			if($GLOBALS['_xmlrpcs_prev_ehandler'] != '_xmlrpcs_errorHandler')
402
+			if ($GLOBALS['_xmlrpcs_prev_ehandler'] != '_xmlrpcs_errorHandler')
403 403
 			{
404 404
 				// NB: this code will NOT work on php < 4.0.2: only 2 params were used for error handlers
405
-				if(is_array($GLOBALS['_xmlrpcs_prev_ehandler']))
405
+				if (is_array($GLOBALS['_xmlrpcs_prev_ehandler']))
406 406
 				{
407 407
 					// the following works both with static class methods and plain object methods as error handler
408 408
 					call_user_func_array($GLOBALS['_xmlrpcs_prev_ehandler'], array($errcode, $errstring, $filename, $lineno, $context));
@@ -415,7 +415,7 @@  discard block
 block discarded – undo
415 415
 		}
416 416
 	}
417 417
 
418
-	$GLOBALS['_xmlrpc_debuginfo']='';
418
+	$GLOBALS['_xmlrpc_debuginfo'] = '';
419 419
 
420 420
 	/**
421 421
 	* Add a string to the debug info that can be later seralized by the server
@@ -427,7 +427,7 @@  discard block
 block discarded – undo
427 427
 	*/
428 428
 	function xmlrpc_debugmsg($m)
429 429
 	{
430
-		$GLOBALS['_xmlrpc_debuginfo'] .= $m . "\n";
430
+		$GLOBALS['_xmlrpc_debuginfo'] .= $m."\n";
431 431
 	}
432 432
 
433 433
 	class xmlrpc_server
@@ -436,20 +436,20 @@  discard block
 block discarded – undo
436 436
 		* Array defining php functions exposed as xmlrpc methods by this server
437 437
 		* @access private
438 438
 		*/
439
-		var $dmap=array();
439
+		var $dmap = array();
440 440
 		/**
441 441
 		* Defines how functions in dmap will be invoked: either using an xmlrpc msg object
442 442
 		* or plain php values.
443 443
 		* valid strings are 'xmlrpcvals', 'phpvals' or 'epivals'
444 444
 		*/
445
-		var $functions_parameters_type='xmlrpcvals';
445
+		var $functions_parameters_type = 'xmlrpcvals';
446 446
 		/**
447 447
 		* Option used for fine-tuning the encoding the php values returned from
448 448
 		* functions registered in the dispatch map when the functions_parameters_types
449 449
 		* member is set to 'phpvals'
450 450
 		* @see php_xmlrpc_encode for a list of values
451 451
 		*/
452
-		var $phpvals_encoding_options = array( 'auto_dates' );
452
+		var $phpvals_encoding_options = array('auto_dates');
453 453
 		/// controls whether the server is going to echo debugging messages back to the client as comments in response body. valid values: 0,1,2,3
454 454
 		var $debug = 1;
455 455
 		/**
@@ -497,11 +497,11 @@  discard block
 block discarded – undo
497 497
 		* @param array $dispmap the dispatch map with definition of exposed services
498 498
 		* @param boolean $servicenow set to false to prevent the server from running upon construction
499 499
 		*/
500
-		function __construct($dispMap=null, $serviceNow=true)
500
+		function __construct($dispMap = null, $serviceNow = true)
501 501
 		{
502 502
 			// if ZLIB is enabled, let the server by default accept compressed requests,
503 503
 			// and compress responses sent to clients that support them
504
-			if(function_exists('gzinflate'))
504
+			if (function_exists('gzinflate'))
505 505
 			{
506 506
 				$this->accepted_compression = array('gzip', 'deflate');
507 507
 				$this->compress_response = true;
@@ -519,10 +519,10 @@  discard block
 block discarded – undo
519 519
 			 * instead, you can use the class add_to_map() function
520 520
 			 * to add functions manually (borrowed from SOAPX4)
521 521
 			 */
522
-			if($dispMap)
522
+			if ($dispMap)
523 523
 			{
524 524
 				$this->dmap = $dispMap;
525
-				if($serviceNow)
525
+				if ($serviceNow)
526 526
 				{
527 527
 					$this->service();
528 528
 				}
@@ -532,7 +532,7 @@  discard block
 block discarded – undo
532 532
 		/**
533 533
 		* @deprecated
534 534
 		*/
535
-		function xmlrpc_client($dispMap=null, $serviceNow=true)
535
+		function xmlrpc_client($dispMap = null, $serviceNow = true)
536 536
 		{
537 537
 			self::__construct($dispMap, $serviceNow);
538 538
 		}
@@ -553,7 +553,7 @@  discard block
 block discarded – undo
553 553
 		*/
554 554
 		function setDebug($in)
555 555
 		{
556
-			$this->debug=$in;
556
+			$this->debug = $in;
557 557
 		}
558 558
 
559 559
 		/**
@@ -561,7 +561,7 @@  discard block
 block discarded – undo
561 561
 		* @param string $charset_encoding the target charset encoding for the serialization
562 562
 		* @return string an XML comment (or two)
563 563
 		*/
564
-		function serializeDebug($charset_encoding='')
564
+		function serializeDebug($charset_encoding = '')
565 565
 		{
566 566
 			// Tough encoding problem: which internal charset should we assume for debug info?
567 567
 			// It might contain a copy of raw data received from client, ie with unknown encoding,
@@ -573,10 +573,10 @@  discard block
 block discarded – undo
573 573
 			{
574 574
 				$out .= "<!-- SERVER DEBUG INFO (BASE64 ENCODED):\n".base64_encode($this->debug_info)."\n-->\n";
575 575
 			}
576
-			if($GLOBALS['_xmlrpc_debuginfo']!='')
576
+			if ($GLOBALS['_xmlrpc_debuginfo'] != '')
577 577
 			{
578 578
 
579
-				$out .= "<!-- DEBUG INFO:\n" . xmlrpc_encode_entitites(str_replace('--', '_-', $GLOBALS['_xmlrpc_debuginfo']), $GLOBALS['xmlrpc_internalencoding'], $charset_encoding) . "\n-->\n";
579
+				$out .= "<!-- DEBUG INFO:\n".xmlrpc_encode_entitites(str_replace('--', '_-', $GLOBALS['_xmlrpc_debuginfo']), $GLOBALS['xmlrpc_internalencoding'], $charset_encoding)."\n-->\n";
580 580
 				// NB: a better solution MIGHT be to use CDATA, but we need to insert it
581 581
 				// into return payload AFTER the beginning tag
582 582
 				//$out .= "<![CDATA[ DEBUG INFO:\n\n" . str_replace(']]>', ']_]_>', $GLOBALS['_xmlrpc_debuginfo']) . "\n]]>\n";
@@ -590,7 +590,7 @@  discard block
 block discarded – undo
590 590
 		* @return xmlrpcresp the response object (usually not used by caller...)
591 591
 		* @access public
592 592
 		*/
593
-		function service($data=null, $return_payload=false)
593
+		function service($data = null, $return_payload = false)
594 594
 		{
595 595
 			if ($data === null)
596 596
 			{
@@ -603,30 +603,30 @@  discard block
 block discarded – undo
603 603
 			$this->debug_info = '';
604 604
 
605 605
 			// Echo back what we received, before parsing it
606
-			if($this->debug > 1)
606
+			if ($this->debug>1)
607 607
 			{
608
-				$this->debugmsg("+++GOT+++\n" . $data . "\n+++END+++");
608
+				$this->debugmsg("+++GOT+++\n".$data."\n+++END+++");
609 609
 			}
610 610
 
611 611
 			$r = $this->parseRequestHeaders($data, $req_charset, $resp_charset, $resp_encoding);
612 612
 			if (!$r)
613 613
 			{
614
-				$r=$this->parseRequest($data, $req_charset);
614
+				$r = $this->parseRequest($data, $req_charset);
615 615
 			}
616 616
 
617 617
 			// save full body of request into response, for more debugging usages
618 618
 			$r->raw_data = $raw_data;
619 619
 
620
-			if($this->debug > 2 && $GLOBALS['_xmlrpcs_occurred_errors'])
620
+			if ($this->debug>2 && $GLOBALS['_xmlrpcs_occurred_errors'])
621 621
 			{
622
-				$this->debugmsg("+++PROCESSING ERRORS AND WARNINGS+++\n" .
623
-					$GLOBALS['_xmlrpcs_occurred_errors'] . "+++END+++");
622
+				$this->debugmsg("+++PROCESSING ERRORS AND WARNINGS+++\n".
623
+					$GLOBALS['_xmlrpcs_occurred_errors']."+++END+++");
624 624
 			}
625 625
 
626
-			$payload=$this->xml_header($resp_charset);
627
-			if($this->debug > 0)
626
+			$payload = $this->xml_header($resp_charset);
627
+			if ($this->debug>0)
628 628
 			{
629
-				$payload = $payload . $this->serializeDebug($resp_charset);
629
+				$payload = $payload.$this->serializeDebug($resp_charset);
630 630
 			}
631 631
 
632 632
 			// G. Giunta 2006-01-27: do not create response serialization if it has
@@ -635,7 +635,7 @@  discard block
 block discarded – undo
635 635
 			{
636 636
 				$r->serialize($resp_charset);
637 637
 			}
638
-			$payload = $payload . $r->payload;
638
+			$payload = $payload.$r->payload;
639 639
 
640 640
 			if ($return_payload)
641 641
 			{
@@ -644,7 +644,7 @@  discard block
 block discarded – undo
644 644
 
645 645
 			// if we get a warning/error that has output some text before here, then we cannot
646 646
 			// add a new header. We cannot say we are sending xml, either...
647
-			if(!headers_sent())
647
+			if (!headers_sent())
648 648
 			{
649 649
 				header('Content-Type: '.$r->content_type);
650 650
 				// we do not know if client actually told us an accepted charset, but if he did
@@ -655,10 +655,10 @@  discard block
 block discarded – undo
655 655
 				// if we can do it, and we want to do it, and client asked us to,
656 656
 				// and php ini settings do not force it already
657 657
 				$php_no_self_compress = !ini_get('zlib.output_compression') && (ini_get('output_handler') != 'ob_gzhandler');
658
-				if($this->compress_response && function_exists('gzencode') && $resp_encoding != ''
658
+				if ($this->compress_response && function_exists('gzencode') && $resp_encoding != ''
659 659
 					&& $php_no_self_compress)
660 660
 				{
661
-					if(strpos($resp_encoding, 'gzip') !== false)
661
+					if (strpos($resp_encoding, 'gzip') !== false)
662 662
 					{
663 663
 						$payload = gzencode($payload);
664 664
 						header("Content-Encoding: gzip");
@@ -674,9 +674,9 @@  discard block
 block discarded – undo
674 674
 
675 675
 				// do not ouput content-length header if php is compressing output for us:
676 676
 				// it will mess up measurements
677
-				if($php_no_self_compress)
677
+				if ($php_no_self_compress)
678 678
 				{
679
-					header('Content-Length: ' . (int)strlen($payload));
679
+					header('Content-Length: '.(int) strlen($payload));
680 680
 				}
681 681
 			}
682 682
 			else
@@ -699,7 +699,7 @@  discard block
 block discarded – undo
699 699
 		* @param array $sigdoc the array of valid method signatures docs (one string per param, one for return type)
700 700
 		* @access public
701 701
 		*/
702
-		function add_to_map($methodname,$function,$sig=null,$doc=false,$sigdoc=false)
702
+		function add_to_map($methodname, $function, $sig = null, $doc = false, $sigdoc = false)
703 703
 		{
704 704
 			$this->dmap[$methodname] = array(
705 705
 				'function'	=> $function,
@@ -733,47 +733,47 @@  discard block
 block discarded – undo
733 733
 			{
734 734
 				$numParams = count($in);
735 735
 			}
736
-			foreach($sig as $cursig)
736
+			foreach ($sig as $cursig)
737 737
 			{
738
-				if(count($cursig)==$numParams+1)
738
+				if (count($cursig) == $numParams+1)
739 739
 				{
740
-					$itsOK=1;
741
-					for($n=0; $n<$numParams; $n++)
740
+					$itsOK = 1;
741
+					for ($n = 0; $n<$numParams; $n++)
742 742
 					{
743 743
 						if (is_object($in))
744 744
 						{
745
-							$p=$in->getParam($n);
746
-							if($p->kindOf() == 'scalar')
745
+							$p = $in->getParam($n);
746
+							if ($p->kindOf() == 'scalar')
747 747
 							{
748
-								$pt=$p->scalartyp();
748
+								$pt = $p->scalartyp();
749 749
 							}
750 750
 							else
751 751
 							{
752
-								$pt=$p->kindOf();
752
+								$pt = $p->kindOf();
753 753
 							}
754 754
 						}
755 755
 						else
756 756
 						{
757
-							$pt= $in[$n] == 'i4' ? 'int' : strtolower($in[$n]); // dispatch maps never use i4...
757
+							$pt = $in[$n] == 'i4' ? 'int' : strtolower($in[$n]); // dispatch maps never use i4...
758 758
 						}
759 759
 
760 760
 						// param index is $n+1, as first member of sig is return type
761
-						if($pt != $cursig[$n+1] && $cursig[$n+1] != $GLOBALS['xmlrpcValue'])
761
+						if ($pt != $cursig[$n+1] && $cursig[$n+1] != $GLOBALS['xmlrpcValue'])
762 762
 						{
763
-							$itsOK=0;
764
-							$pno=$n+1;
765
-							$wanted=$cursig[$n+1];
766
-							$got=$pt;
763
+							$itsOK = 0;
764
+							$pno = $n+1;
765
+							$wanted = $cursig[$n+1];
766
+							$got = $pt;
767 767
 							break;
768 768
 						}
769 769
 					}
770
-					if($itsOK)
770
+					if ($itsOK)
771 771
 					{
772
-						return array(1,'');
772
+						return array(1, '');
773 773
 					}
774 774
 				}
775 775
 			}
776
-			if(isset($wanted))
776
+			if (isset($wanted))
777 777
 			{
778 778
 				return array(0, "Wanted ${wanted}, got ${got} at param ${pno}");
779 779
 			}
@@ -797,12 +797,12 @@  discard block
 block discarded – undo
797 797
 				error_log('XML-RPC: '.__METHOD__.': cannot parse request headers as $_SERVER is not populated');
798 798
 			}
799 799
 
800
-			if($this->debug > 1)
800
+			if ($this->debug>1)
801 801
 			{
802
-				if(function_exists('getallheaders'))
802
+				if (function_exists('getallheaders'))
803 803
 				{
804 804
 					$this->debugmsg(''); // empty line
805
-					foreach(getallheaders() as $name => $val)
805
+					foreach (getallheaders() as $name => $val)
806 806
 					{
807 807
 						$this->debugmsg("HEADER: $name: $val");
808 808
 					}
@@ -810,7 +810,7 @@  discard block
 block discarded – undo
810 810
 
811 811
 			}
812 812
 
813
-			if(isset($_SERVER['HTTP_CONTENT_ENCODING']))
813
+			if (isset($_SERVER['HTTP_CONTENT_ENCODING']))
814 814
 			{
815 815
 				$content_encoding = str_replace('x-', '', $_SERVER['HTTP_CONTENT_ENCODING']);
816 816
 			}
@@ -820,26 +820,26 @@  discard block
 block discarded – undo
820 820
 			}
821 821
 
822 822
 			// check if request body has been compressed and decompress it
823
-			if($content_encoding != '' && strlen($data))
823
+			if ($content_encoding != '' && strlen($data))
824 824
 			{
825
-				if($content_encoding == 'deflate' || $content_encoding == 'gzip')
825
+				if ($content_encoding == 'deflate' || $content_encoding == 'gzip')
826 826
 				{
827 827
 					// if decoding works, use it. else assume data wasn't gzencoded
828
-					if(function_exists('gzinflate') && in_array($content_encoding, $this->accepted_compression))
828
+					if (function_exists('gzinflate') && in_array($content_encoding, $this->accepted_compression))
829 829
 					{
830
-						if($content_encoding == 'deflate' && $degzdata = @gzuncompress($data))
830
+						if ($content_encoding == 'deflate' && $degzdata = @gzuncompress($data))
831 831
 						{
832 832
 							$data = $degzdata;
833
-							if($this->debug > 1)
833
+							if ($this->debug>1)
834 834
 							{
835
-								$this->debugmsg("\n+++INFLATED REQUEST+++[".strlen($data)." chars]+++\n" . $data . "\n+++END+++");
835
+								$this->debugmsg("\n+++INFLATED REQUEST+++[".strlen($data)." chars]+++\n".$data."\n+++END+++");
836 836
 							}
837 837
 						}
838
-						elseif($content_encoding == 'gzip' && $degzdata = @gzinflate(substr($data, 10)))
838
+						elseif ($content_encoding == 'gzip' && $degzdata = @gzinflate(substr($data, 10)))
839 839
 						{
840 840
 							$data = $degzdata;
841
-							if($this->debug > 1)
842
-								$this->debugmsg("+++INFLATED REQUEST+++[".strlen($data)." chars]+++\n" . $data . "\n+++END+++");
841
+							if ($this->debug>1)
842
+								$this->debugmsg("+++INFLATED REQUEST+++[".strlen($data)." chars]+++\n".$data."\n+++END+++");
843 843
 						}
844 844
 						else
845 845
 						{
@@ -912,7 +912,7 @@  discard block
 block discarded – undo
912 912
 		* @return xmlrpcresp
913 913
 		* @access private
914 914
 		*/
915
-		function parseRequest($data, $req_encoding='')
915
+		function parseRequest($data, $req_encoding = '')
916 916
 		{
917 917
 			// 2005/05/07 commented and moved into caller function code
918 918
 			//if($data=='')
@@ -924,16 +924,16 @@  discard block
 block discarded – undo
924 924
 			// so we do not try to convert them into xml character entities
925 925
 			//$data = xmlrpc_html_entity_xlate($data);
926 926
 
927
-			$GLOBALS['_xh']=array();
928
-			$GLOBALS['_xh']['ac']='';
929
-			$GLOBALS['_xh']['stack']=array();
927
+			$GLOBALS['_xh'] = array();
928
+			$GLOBALS['_xh']['ac'] = '';
929
+			$GLOBALS['_xh']['stack'] = array();
930 930
 			$GLOBALS['_xh']['valuestack'] = array();
931
-			$GLOBALS['_xh']['params']=array();
932
-			$GLOBALS['_xh']['pt']=array();
933
-			$GLOBALS['_xh']['isf']=0;
934
-			$GLOBALS['_xh']['isf_reason']='';
935
-			$GLOBALS['_xh']['method']=false; // so we can check later if we got a methodname or not
936
-			$GLOBALS['_xh']['rt']='';
931
+			$GLOBALS['_xh']['params'] = array();
932
+			$GLOBALS['_xh']['pt'] = array();
933
+			$GLOBALS['_xh']['isf'] = 0;
934
+			$GLOBALS['_xh']['isf_reason'] = '';
935
+			$GLOBALS['_xh']['method'] = false; // so we can check later if we got a methodname or not
936
+			$GLOBALS['_xh']['rt'] = '';
937 937
 
938 938
 			// decompose incoming XML into request structure
939 939
 
@@ -951,7 +951,7 @@  discard block
 block discarded – undo
951 951
 						if (extension_loaded('mbstring')) {
952 952
 							$data = mb_convert_encoding($data, 'UTF-8', $req_encoding);
953 953
 						} else {
954
-							error_log('XML-RPC: ' . __METHOD__ . ': invalid charset encoding of received request: ' . $req_encoding);
954
+							error_log('XML-RPC: '.__METHOD__.': invalid charset encoding of received request: '.$req_encoding);
955 955
 						}
956 956
 					}
957 957
 				}
@@ -980,10 +980,10 @@  discard block
 block discarded – undo
980 980
 				xml_set_element_handler($parser, 'xmlrpc_se', 'xmlrpc_ee');
981 981
 			xml_set_character_data_handler($parser, 'xmlrpc_cd');
982 982
 			xml_set_default_handler($parser, 'xmlrpc_dh');
983
-			if(!xml_parse($parser, $data, 1))
983
+			if (!xml_parse($parser, $data, 1))
984 984
 			{
985 985
 				// return XML error as a faultCode
986
-				$r=new xmlrpcresp(0,
986
+				$r = new xmlrpcresp(0,
987 987
 				$GLOBALS['xmlrpcerrxml']+xml_get_error_code($parser),
988 988
 				sprintf('XML error: %s at line %d, column %d',
989 989
 					xml_error_string(xml_get_error_code($parser)),
@@ -993,9 +993,9 @@  discard block
 block discarded – undo
993 993
 			elseif ($GLOBALS['_xh']['isf'])
994 994
 			{
995 995
 				xml_parser_free($parser);
996
-				$r=new xmlrpcresp(0,
996
+				$r = new xmlrpcresp(0,
997 997
 					$GLOBALS['xmlrpcerr']['invalid_request'],
998
-					$GLOBALS['xmlrpcstr']['invalid_request'] . ' ' . $GLOBALS['_xh']['isf_reason']);
998
+					$GLOBALS['xmlrpcstr']['invalid_request'].' '.$GLOBALS['_xh']['isf_reason']);
999 999
 			}
1000 1000
 			else
1001 1001
 			{
@@ -1006,7 +1006,7 @@  discard block
 block discarded – undo
1006 1006
 				// registered as phpvals) that would mean a useless encode+decode pass
1007 1007
 				if ($this->functions_parameters_type != 'xmlrpcvals' || (isset($this->dmap[$GLOBALS['_xh']['method']]['parameters_type']) && ($this->dmap[$GLOBALS['_xh']['method']]['parameters_type'] == 'phpvals')))
1008 1008
 				{
1009
-					if($this->debug > 1)
1009
+					if ($this->debug>1)
1010 1010
 					{
1011 1011
 						$this->debugmsg("\n+++PARSED+++\n".var_export($GLOBALS['_xh']['params'], true)."\n+++END+++");
1012 1012
 					}
@@ -1015,14 +1015,14 @@  discard block
 block discarded – undo
1015 1015
 				else
1016 1016
 				{
1017 1017
 					// build an xmlrpcmsg object with data parsed from xml
1018
-					$m=new xmlrpcmsg($GLOBALS['_xh']['method']);
1018
+					$m = new xmlrpcmsg($GLOBALS['_xh']['method']);
1019 1019
 					// now add parameters in
1020
-					for($i=0; $i<count($GLOBALS['_xh']['params']); $i++)
1020
+					for ($i = 0; $i<count($GLOBALS['_xh']['params']); $i++)
1021 1021
 					{
1022 1022
 						$m->addParam($GLOBALS['_xh']['params'][$i]);
1023 1023
 					}
1024 1024
 
1025
-					if($this->debug > 1)
1025
+					if ($this->debug>1)
1026 1026
 					{
1027 1027
 						$this->debugmsg("\n+++PARSED+++\n".var_export($m, true)."\n+++END+++");
1028 1028
 					}
@@ -1040,7 +1040,7 @@  discard block
 block discarded – undo
1040 1040
 		* @return xmlrpcresp
1041 1041
 		* @access private
1042 1042
 		*/
1043
-		function execute($m, $params=null, $paramtypes=null)
1043
+		function execute($m, $params = null, $paramtypes = null)
1044 1044
 		{
1045 1045
 			if (is_object($m))
1046 1046
 			{
@@ -1053,7 +1053,7 @@  discard block
 block discarded – undo
1053 1053
 			$sysCall = $this->allow_system_funcs && (strpos($methName, "system.") === 0);
1054 1054
 			$dmap = $sysCall ? $GLOBALS['_xmlrpcs_dmap'] : $this->dmap;
1055 1055
 
1056
-			if(!isset($dmap[$methName]['function']))
1056
+			if (!isset($dmap[$methName]['function']))
1057 1057
 			{
1058 1058
 				// No such method
1059 1059
 				return new xmlrpcresp(0,
@@ -1062,7 +1062,7 @@  discard block
 block discarded – undo
1062 1062
 			}
1063 1063
 
1064 1064
 			// Check signature
1065
-			if(isset($dmap[$methName]['signature']))
1065
+			if (isset($dmap[$methName]['signature']))
1066 1066
 			{
1067 1067
 				$sig = $dmap[$methName]['signature'];
1068 1068
 				if (is_object($m))
@@ -1073,37 +1073,37 @@  discard block
 block discarded – undo
1073 1073
 				{
1074 1074
 					list($ok, $errstr) = $this->verifySignature($paramtypes, $sig);
1075 1075
 				}
1076
-				if(!$ok)
1076
+				if (!$ok)
1077 1077
 				{
1078 1078
 					// Didn't match.
1079 1079
 					return new xmlrpcresp(
1080 1080
 						0,
1081 1081
 						$GLOBALS['xmlrpcerr']['incorrect_params'],
1082
-						$GLOBALS['xmlrpcstr']['incorrect_params'] . ": ${errstr}"
1082
+						$GLOBALS['xmlrpcstr']['incorrect_params'].": ${errstr}"
1083 1083
 					);
1084 1084
 				}
1085 1085
 			}
1086 1086
 
1087 1087
 			$func = $dmap[$methName]['function'];
1088 1088
 			// let the 'class::function' syntax be accepted in dispatch maps
1089
-			if(is_string($func) && strpos($func, '::'))
1089
+			if (is_string($func) && strpos($func, '::'))
1090 1090
 			{
1091 1091
 				$func = explode('::', $func);
1092 1092
 			}
1093 1093
 			// verify that function to be invoked is in fact callable
1094
-			if(!is_callable($func))
1094
+			if (!is_callable($func))
1095 1095
 			{
1096 1096
 				error_log("XML-RPC: ".__METHOD__.": function $func registered as method handler is not callable");
1097 1097
 				return new xmlrpcresp(
1098 1098
 					0,
1099 1099
 					$GLOBALS['xmlrpcerr']['server_error'],
1100
-					$GLOBALS['xmlrpcstr']['server_error'] . ": no function matches method"
1100
+					$GLOBALS['xmlrpcstr']['server_error'].": no function matches method"
1101 1101
 				);
1102 1102
 			}
1103 1103
 
1104 1104
 			// If debug level is 3, we should catch all errors generated during
1105 1105
 			// processing of user function, and log them as part of response
1106
-			if($this->debug > 2)
1106
+			if ($this->debug>2)
1107 1107
 			{
1108 1108
 				$GLOBALS['_xmlrpcs_prev_ehandler'] = set_error_handler('_xmlrpcs_errorHandler');
1109 1109
 			}
@@ -1112,7 +1112,7 @@  discard block
 block discarded – undo
1112 1112
 				// Allow mixed-convention servers
1113 1113
 				if (is_object($m))
1114 1114
 				{
1115
-					if($sysCall)
1115
+					if ($sysCall)
1116 1116
 					{
1117 1117
 						$r = call_user_func($func, $this, $m);
1118 1118
 					}
@@ -1132,7 +1132,7 @@  discard block
 block discarded – undo
1132 1132
 							$r = new xmlrpcresp(
1133 1133
 								0,
1134 1134
 								$GLOBALS['xmlrpcerr']['server_error'],
1135
-								$GLOBALS['xmlrpcstr']['server_error'] . ": function does not return xmlrpcresp object"
1135
+								$GLOBALS['xmlrpcstr']['server_error'].": function does not return xmlrpcresp object"
1136 1136
 							);
1137 1137
 						}
1138 1138
 					}
@@ -1140,7 +1140,7 @@  discard block
 block discarded – undo
1140 1140
 				else
1141 1141
 				{
1142 1142
 					// call a 'plain php' function
1143
-					if($sysCall)
1143
+					if ($sysCall)
1144 1144
 					{
1145 1145
 						array_unshift($params, $this);
1146 1146
 						$r = call_user_func_array($func, $params);
@@ -1155,7 +1155,7 @@  discard block
 block discarded – undo
1155 1155
 							// an eror response
1156 1156
 							if (is_array($r) && array_key_exists('faultCode', $r) && array_key_exists('faultString', $r))
1157 1157
 							{
1158
-								$r = new xmlrpcresp(0, (integer)$r['faultCode'], (string)$r['faultString']);
1158
+								$r = new xmlrpcresp(0, (integer) $r['faultCode'], (string) $r['faultString']);
1159 1159
 							}
1160 1160
 							else
1161 1161
 							{
@@ -1178,11 +1178,11 @@  discard block
 block discarded – undo
1178 1178
 					}
1179 1179
 				}
1180 1180
 			}
1181
-			catch(Exception $e)
1181
+			catch (Exception $e)
1182 1182
 			{
1183 1183
 				// (barring errors in the lib) an uncatched exception happened
1184 1184
 				// in the called function, we wrap it in a proper error-response
1185
-				switch($this->exception_handling)
1185
+				switch ($this->exception_handling)
1186 1186
 				{
1187 1187
 					case 2:
1188 1188
 						throw $e;
@@ -1194,11 +1194,11 @@  discard block
 block discarded – undo
1194 1194
 						$r = new xmlrpcresp(0, $GLOBALS['xmlrpcerr']['server_error'], $GLOBALS['xmlrpcstr']['server_error']);
1195 1195
 				}
1196 1196
 			}
1197
-			if($this->debug > 2)
1197
+			if ($this->debug>2)
1198 1198
 			{
1199 1199
 				// note: restore the error handler we found before calling the
1200 1200
 				// user func, even if it has been changed inside the func itself
1201
-				if($GLOBALS['_xmlrpcs_prev_ehandler'])
1201
+				if ($GLOBALS['_xmlrpcs_prev_ehandler'])
1202 1202
 				{
1203 1203
 					set_error_handler($GLOBALS['_xmlrpcs_prev_ehandler']);
1204 1204
 				}
@@ -1223,15 +1223,15 @@  discard block
 block discarded – undo
1223 1223
 		/**
1224 1224
 		* @access private
1225 1225
 		*/
1226
-		function xml_header($charset_encoding='')
1226
+		function xml_header($charset_encoding = '')
1227 1227
 		{
1228 1228
 			if ($charset_encoding != '')
1229 1229
 			{
1230
-				return "<?xml version=\"1.0\" encoding=\"$charset_encoding\"?" . ">\n";
1230
+				return "<?xml version=\"1.0\" encoding=\"$charset_encoding\"?".">\n";
1231 1231
 			}
1232 1232
 			else
1233 1233
 			{
1234
-				return "<?xml version=\"1.0\"?" . ">\n";
1234
+				return "<?xml version=\"1.0\"?".">\n";
1235 1235
 			}
1236 1236
 		}
1237 1237
 
@@ -1241,7 +1241,7 @@  discard block
 block discarded – undo
1241 1241
 		*/
1242 1242
 		function echoInput()
1243 1243
 		{
1244
-			$r=new xmlrpcresp(new xmlrpcval( "'Aha said I: '" . $GLOBALS['HTTP_RAW_POST_DATA'], 'string'));
1244
+			$r = new xmlrpcresp(new xmlrpcval("'Aha said I: '".$GLOBALS['HTTP_RAW_POST_DATA'], 'string'));
1245 1245
 			print $r->serialize();
1246 1246
 		}
1247 1247
 	}
Please login to merge, or discard this patch.