gggeek / phpxmlrpc

src/Server.php

Doc Comments +2 added lines, -1 removed lines

@@ -348,7 +348,8 @@
     /**
      * Parse http headers received along with xmlrpc request. If needed, inflate request.
      *
-     * @return mixed Response|null on success or an error Response
+     * @param string $data
+     * @return Response|null Response|null on success or an error Response
      */
     protected function parseRequestHeaders(&$data, &$reqEncoding, &$respEncoding, &$respCompression)
     {

src/Wrapper.php

Doc Comments +3 added lines, -3 removed lines

@@ -392,7 +392,7 @@ 
      * @todo validate params? In theory all validation is left to the dispatch map...
      * @todo add support for $catchWarnings
      *
-     * @param $callable
+     * @param callable $callable
      * @param array $extraOptions
      * @param string $plainFuncName
      * @param string $funcDesc
@@ -454,7 +454,7 @@ 
 
     /**
      * Return a name for a new function, based on $callable, insuring its uniqueness
-     * @param mixed $callable a php callable, or the name of an xmlrpc method
+     * @param callable $callable a php callable, or the name of an xmlrpc method
      * @param string $newFuncName when not empty, it is used instead of the calculated version
      * @return string
      */
@@ -492,7 +492,7 @@ 
     }
 
     /**
-     * @param $callable
+     * @param callable $callable
      * @param string $newFuncName
      * @param array $extraOptions
      * @param string $plainFuncName

tests/1ParsingBugsTest.php

Doc Comments +3 added lines

@@ -33,6 +33,9 @@
         }
     }
 
+    /**
+     * @param string $methodName
+     */
     protected function newMsg($methodName, $params = array())
     {
         $msg = new xmlrpcmsg($methodName, $params);

src/Client.php

Doc Comments -1 removed lines

@@ -201,7 +201,6 @@
      * This option can be very useful when debugging servers as it allows you to see exactly what the client sends and
      * the server returns.
      *
-     * @param integer $in values 0, 1 and 2 are supported (2 = echo sent msg too, before received response)
      */
     public function setDebug($level)
     {

lib/xmlrpc_wrappers.inc

Indentation +3 added lines, -3 removed lines

@@ -141,9 +141,9 @@
  * @deprecated
  */
 function build_remote_method_wrapper_code($client, $methodName, $xmlrpcFuncName,
-     $mSig, $mDesc = '', $timeout = 0, $protocol = '', $clientCopyMode = 0, $prefix = 'xmlrpc',
-     $decodePhpObjects = false, $encodePhpObjects = false, $decodeFault = false,
-     $faultResponse = '', $namespace = '\\PhpXmlRpc\\')
+        $mSig, $mDesc = '', $timeout = 0, $protocol = '', $clientCopyMode = 0, $prefix = 'xmlrpc',
+        $decodePhpObjects = false, $encodePhpObjects = false, $decodeFault = false,
+        $faultResponse = '', $namespace = '\\PhpXmlRpc\\')
 {
     $code = "function $xmlrpcFuncName (";
     if ($clientCopyMode < 2) {

Doc Comments +2 added lines, -1 removed lines

@@ -87,7 +87,7 @@ 
  * @see PhpXmlRpc\Wrapper::wrapXmlrpcMethod
  * @param xmlrpc_client $client
  * @param string $methodName
- * @param int|array $extraOptions the usage of an int as signature number is deprecated, use an option in $extraOptions
+ * @param integer $extraOptions the usage of an int as signature number is deprecated, use an option in $extraOptions
  * @param int $timeout            deprecated, use an option in $extraOptions
  * @param string $protocol        deprecated, use an option in $extraOptions
  * @param string $newFuncName     deprecated, use an option in $extraOptions
@@ -217,6 +217,7 @@ 
 
 /**
  * @deprecated
+ * @param integer $verbatim_client_copy
  */
 function build_client_wrapper_code($client, $verbatim_client_copy, $prefix='xmlrpc')
 {

lib/xmlrpc.inc

Doc Comments +6 added lines

@@ -168,6 +168,9 @@ 
     return PhpXmlRpc\Helper\Charset::instance()->encodeEntitites($data, $srcEncoding, $destEncoding);
 }
 
+/**
+ * @param integer $timeT
+ */
 function iso8601_encode($timeT, $utc=0)
 {
     return PhpXmlRpc\Helper\Date::iso8601Encode($timeT, $utc);
@@ -195,6 +198,9 @@ 
     return $encoder->encode($phpVal, $options);
 }
 
+/**
+ * @param string $xmlVal
+ */
 function php_xmlrpc_decode_xml($xmlVal, $options=array())
 {
     $encoder = new PhpXmlRpc\Encoder();

src/Value.php

Doc Comments +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      *
      * When no value or type is passed in, the value is left uninitialized, and the value can be added later.
      *
-     * @param mixed $val if passing in an array, all array elements should be PhpXmlRpc\Value themselves
+     * @param integer $val if passing in an array, all array elements should be PhpXmlRpc\Value themselves
      * @param string $type any valid xmlrpc type name (lowercase): i4, int, boolean, string, double, dateTime.iso8601,
      *                     base64, array, struct, null.
      *                     If null, 'string' is assumed.
@@ -95,7 +95,7 @@ 
      * If the xmlrpc value is empty (uninitialized), this method makes it a scalar value, and sets that value.
      * Fails if the xmlrpc value is not an array and already initialized.
      *
-     * @param mixed $val
+     * @param string $val
      * @param string $type allowed values: i4, i8, int, boolean, string, double, dateTime.iso8601, base64, null.
      *
      * @return int 1 or 0 on failure

tests/3LocalhostTest.php

Doc Comments +5 added lines, -1 removed lines

@@ -114,7 +114,7 @@ 
 
     /**
      * @param PhpXmlRpc\Request|array $msg
-     * @param int|array $errorCode
+     * @param integer $errorCode
      * @param bool $returnResponse
      * @return mixed|\PhpXmlRpc\Response|\PhpXmlRpc\Response[]|\PhpXmlRpc\Value|string|null
      */
@@ -431,6 +431,10 @@ 
         }
     }
 
+    /**
+     * @param string $method
+     * @param PhpXmlRpc\Value[] $params
+     */
     public function _multicall_msg($method, $params)
     {
         $struct['methodName'] = new xmlrpcval($method, 'string');