Inspection of "Remove usages of 'each', as it is deprecated in ph..." - gggeek/phpxmlrpc - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 509dee...ac735c )

by Gaetano

created 2017-09-03 19:11 UTC

Status

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

@@ -3,16 +3,16 @@  discard block
 block discarded – undo
 <body>
 <?php
 
-include_once __DIR__ . "/../vendor/autoload.php";
+include_once __DIR__."/../vendor/autoload.php";
 
 $req = new PhpXmlRpc\Request('examples.getStateName');
 
 print "<h3>Testing value serialization</h3>\n";
 
 $v = new PhpXmlRpc\Value(23, "int");
-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
+print "<PRE>".htmlentities($v->serialize())."</PRE>";
 $v = new PhpXmlRpc\Value("What are you saying? >> << &&");
-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
+print "<PRE>".htmlentities($v->serialize())."</PRE>";
 
 $v = new PhpXmlRpc\Value(
     array(
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
     "array"
 );
 
-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
+print "<PRE>".htmlentities($v->serialize())."</PRE>";
 
 $v = new PhpXmlRpc\Value(
     array(
@@ -51,11 +51,11 @@  discard block
 block discarded – undo
     "struct"
 );
 
-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
+print "<PRE>".htmlentities($v->serialize())."</PRE>";
 
 $w = new PhpXmlRpc\Value(array($v, new PhpXmlRpc\Value("That was the struct!")), "array");
 
-print "<PRE>" . htmlentities($w->serialize()) . "</PRE>";
+print "<PRE>".htmlentities($w->serialize())."</PRE>";
 
 $w = new PhpXmlRpc\Value("Mary had a little lamb,
 Whose fleece was white as snow,
@@ -67,26 +67,26 @@  discard block
 block discarded – undo
 Ten thousand volts went down its back
 And turned it into nylon", "base64"
 );
-print "<PRE>" . htmlentities($w->serialize()) . "</PRE>";
-print "<PRE>Value of base64 string is: '" . $w->scalarval() . "'</PRE>";
+print "<PRE>".htmlentities($w->serialize())."</PRE>";
+print "<PRE>Value of base64 string is: '".$w->scalarval()."'</PRE>";
 
 $req->method('');
 $req->addParam(new PhpXmlRpc\Value("41", "int"));
 
 print "<h3>Testing request serialization</h3>\n";
 $op = $req->serialize();
-print "<PRE>" . htmlentities($op) . "</PRE>";
+print "<PRE>".htmlentities($op)."</PRE>";
 
 print "<h3>Testing ISO date format</h3><pre>\n";
 
 $t = time();
 $date = PhpXmlRpc\Helper\Date::iso8601Encode($t);
 print "Now is $t --> $date\n";
-print "Or in UTC, that is " . PhpXmlRpc\Helper\Date::iso8601Encode($t, 1) . "\n";
+print "Or in UTC, that is ".PhpXmlRpc\Helper\Date::iso8601Encode($t, 1)."\n";
 $tb = PhpXmlRpc\Helper\Date::iso8601Decode($date);
 print "That is to say $date --> $tb\n";
-print "Which comes out at " . PhpXmlRpc\Helper\Date::iso8601Encode($tb) . "\n";
-print "Which was the time in UTC at " . PhpXmlRpc\Helper\Date::iso8601Encode($date, 1) . "\n";
+print "Which comes out at ".PhpXmlRpc\Helper\Date::iso8601Encode($tb)."\n";
+print "Which was the time in UTC at ".PhpXmlRpc\Helper\Date::iso8601Encode($date, 1)."\n";
 
 print "</pre>\n";
 

Please login to merge, or discard this patch.

src/Helper/XMLParser.php 1 patch

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

@@ -66,7 +66,7 @@  discard block
 block discarded – undo
     public function xmlrpc_se($parser, $name, $attrs, $acceptSingleVals = false)
     {
         // if invalid xmlrpc already detected, skip all processing
-        if ($this->_xh['isf'] < 2) {
+        if ($this->_xh['isf']<2) {
             // check for correct element nesting
             // top level element can only be of 2 types
             /// @todo optimization creep: save this check into a bool variable, instead of using count() every time:
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
                     $this->_xh['isf'] = 1;
                     break;
                 case 'MEMBER':
-                    $this->_xh['valuestack'][count($this->_xh['valuestack']) - 1]['name'] = ''; // set member name to null, in case we do not find in the xml later on
+                    $this->_xh['valuestack'][count($this->_xh['valuestack'])-1]['name'] = ''; // set member name to null, in case we do not find in the xml later on
                     //$this->_xh['ac']='';
                 // Drop trough intentionally
                 case 'PARAM':
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
      */
     public function xmlrpc_ee($parser, $name, $rebuildXmlrpcvals = true)
     {
-        if ($this->_xh['isf'] < 2) {
+        if ($this->_xh['isf']<2) {
             // push this element name from stack
             // NB: if XML validates, correct opening/closing is guaranteed and
             // we do not have to check for $name == $currElem.
@@ -249,8 +249,8 @@  discard block
 block discarded – undo
                         // check if we are inside an array or struct:
                         // if value just built is inside an array, let's move it into array on the stack
                         $vscount = count($this->_xh['valuestack']);
-                        if ($vscount && $this->_xh['valuestack'][$vscount - 1]['type'] == 'ARRAY') {
-                            $this->_xh['valuestack'][$vscount - 1]['values'][] = $temp;
+                        if ($vscount && $this->_xh['valuestack'][$vscount-1]['type'] == 'ARRAY') {
+                            $this->_xh['valuestack'][$vscount-1]['values'][] = $temp;
                         } else {
                             $this->_xh['value'] = $temp;
                         }
@@ -264,8 +264,8 @@  discard block
 block discarded – undo
                         // check if we are inside an array or struct:
                         // if value just built is inside an array, let's move it into array on the stack
                         $vscount = count($this->_xh['valuestack']);
-                        if ($vscount && $this->_xh['valuestack'][$vscount - 1]['type'] == 'ARRAY') {
-                            $this->_xh['valuestack'][$vscount - 1]['values'][] = $this->_xh['value'];
+                        if ($vscount && $this->_xh['valuestack'][$vscount-1]['type'] == 'ARRAY') {
+                            $this->_xh['valuestack'][$vscount-1]['values'][] = $this->_xh['value'];
                         }
                     }
                     break;
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
                         $this->_xh['value'] = $this->_xh['ac'];
                     } elseif ($name == 'DATETIME.ISO8601') {
                         if (!preg_match('/^[0-9]{8}T[0-9]{2}:[0-9]{2}:[0-9]{2}$/', $this->_xh['ac'])) {
-                            error_log('XML-RPC: ' . __METHOD__ . ': invalid value received in DATETIME: ' . $this->_xh['ac']);
+                            error_log('XML-RPC: '.__METHOD__.': invalid value received in DATETIME: '.$this->_xh['ac']);
                         }
                         $this->_xh['vt'] = Value::$xmlrpcDateTime;
                         $this->_xh['value'] = $this->_xh['ac'];
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
                         } else {
                             // log if receiving something strange, even though we set the value to false anyway
                             if ($this->_xh['ac'] != '0' && strcasecmp($this->_xh['ac'], 'false') != 0) {
-                                error_log('XML-RPC: ' . __METHOD__ . ': invalid value received in BOOLEAN: ' . $this->_xh['ac']);
+                                error_log('XML-RPC: '.__METHOD__.': invalid value received in BOOLEAN: '.$this->_xh['ac']);
                             }
                             $this->_xh['value'] = false;
                         }
@@ -314,37 +314,37 @@  discard block
 block discarded – undo
                         // NOTE: regexp could be much stricter than this...
                         if (!preg_match('/^[+-eE0123456789 \t.]+$/', $this->_xh['ac'])) {
                             /// @todo: find a better way of throwing an error than this!
-                            error_log('XML-RPC: ' . __METHOD__ . ': non numeric value received in DOUBLE: ' . $this->_xh['ac']);
+                            error_log('XML-RPC: '.__METHOD__.': non numeric value received in DOUBLE: '.$this->_xh['ac']);
                             $this->_xh['value'] = 'ERROR_NON_NUMERIC_FOUND';
                         } else {
                             // it's ok, add it on
-                            $this->_xh['value'] = (double)$this->_xh['ac'];
+                            $this->_xh['value'] = (double) $this->_xh['ac'];
                         }
                     } else {
                         // we have an I4/I8/INT
                         // we must check that only 0123456789-<space> are characters here
                         if (!preg_match('/^[+-]?[0123456789 \t]+$/', $this->_xh['ac'])) {
                             /// @todo find a better way of throwing an error than this!
-                            error_log('XML-RPC: ' . __METHOD__ . ': non numeric value received in INT: ' . $this->_xh['ac']);
+                            error_log('XML-RPC: '.__METHOD__.': non numeric value received in INT: '.$this->_xh['ac']);
                             $this->_xh['value'] = 'ERROR_NON_NUMERIC_FOUND';
                         } else {
                             // it's ok, add it on
-                            $this->_xh['value'] = (int)$this->_xh['ac'];
+                            $this->_xh['value'] = (int) $this->_xh['ac'];
                         }
                     }
                     $this->_xh['lv'] = 3; // indicate we've found a value
                     break;
                 case 'NAME':
-                    $this->_xh['valuestack'][count($this->_xh['valuestack']) - 1]['name'] = $this->_xh['ac'];
+                    $this->_xh['valuestack'][count($this->_xh['valuestack'])-1]['name'] = $this->_xh['ac'];
                     break;
                 case 'MEMBER':
                     // add to array in the stack the last element built,
                     // unless no VALUE was found
                     if ($this->_xh['vt']) {
                         $vscount = count($this->_xh['valuestack']);
-                        $this->_xh['valuestack'][$vscount - 1]['values'][$this->_xh['valuestack'][$vscount - 1]['name']] = $this->_xh['value'];
+                        $this->_xh['valuestack'][$vscount-1]['values'][$this->_xh['valuestack'][$vscount-1]['name']] = $this->_xh['value'];
                     } else {
-                        error_log('XML-RPC: ' . __METHOD__ . ': missing VALUE inside STRUCT in received xml');
+                        error_log('XML-RPC: '.__METHOD__.': missing VALUE inside STRUCT in received xml');
                     }
                     break;
                 case 'DATA':
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
                         $this->_xh['params'][] = $this->_xh['value'];
                         $this->_xh['pt'][] = $this->_xh['vt'];
                     } else {
-                        error_log('XML-RPC: ' . __METHOD__ . ': missing VALUE inside PARAM in received xml');
+                        error_log('XML-RPC: '.__METHOD__.': missing VALUE inside PARAM in received xml');
                     }
                     break;
                 case 'METHODNAME':
@@ -409,7 +409,7 @@  discard block
 block discarded – undo
     public function xmlrpc_cd($parser, $data)
     {
         // skip processing if xml fault already detected
-        if ($this->_xh['isf'] < 2) {
+        if ($this->_xh['isf']<2) {
             // "lookforvalue==3" means that we've found an entire value
             // and should discard any further character data
             if ($this->_xh['lv'] != 3) {
@@ -425,7 +425,7 @@  discard block
 block discarded – undo
     public function xmlrpc_dh($parser, $data)
     {
         // skip processing if xml fault already detected
-        if ($this->_xh['isf'] < 2) {
+        if ($this->_xh['isf']<2) {
             if (substr($data, 0, 1) == '&' && substr($data, -1, 1) == ';') {
                 $this->_xh['ac'] .= $data;
             }
@@ -496,8 +496,8 @@  discard block
 block discarded – undo
         // Details:
         // SPACE:         (#x20 | #x9 | #xD | #xA)+ === [ \x9\xD\xA]+
         // EQ:            SPACE?=SPACE? === [ \x9\xD\xA]*=[ \x9\xD\xA]*
-        if (preg_match('/^<\?xml\s+version\s*=\s*' . "((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))" .
-            '\s+encoding\s*=\s*' . "((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
+        if (preg_match('/^<\?xml\s+version\s*=\s*'."((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))".
+            '\s+encoding\s*=\s*'."((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
             $xmlChunk, $matches)) {
             return strtoupper(substr($matches[2], 1, -1));
         }
@@ -515,7 +515,7 @@  discard block
 block discarded – undo
             // NB: mb_detect likes to call it ascii, xml parser likes to call it US_ASCII...
             // IANA also likes better US-ASCII, so go with it
             if ($enc == 'ASCII') {
-                $enc = 'US-' . $enc;
+                $enc = 'US-'.$enc;
             }
 
             return $enc;
@@ -550,8 +550,8 @@  discard block
 block discarded – undo
         // Details:
         // SPACE:         (#x20 | #x9 | #xD | #xA)+ === [ \x9\xD\xA]+
         // EQ:            SPACE?=SPACE? === [ \x9\xD\xA]*=[ \x9\xD\xA]*
-        if (preg_match('/^<\?xml\s+version\s*=\s*' . "((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))" .
-            '\s+encoding\s*=\s*' . "((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
+        if (preg_match('/^<\?xml\s+version\s*=\s*'."((?:\"[a-zA-Z0-9_.:-]+\")|(?:'[a-zA-Z0-9_.:-]+'))".
+            '\s+encoding\s*=\s*'."((?:\"[A-Za-z][A-Za-z0-9._-]*\")|(?:'[A-Za-z][A-Za-z0-9._-]*'))/",
             $xmlChunk, $matches)) {
             return true;
         }

Please login to merge, or discard this patch.

src/Helper/Http.php 1 patch

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

@@ -22,12 +22,12 @@  discard block
 block discarded – undo
 
         // read chunk-size, chunk-extension (if any) and crlf
         // get the position of the linebreak
-        $chunkEnd = strpos($buffer, "\r\n") + 2;
+        $chunkEnd = strpos($buffer, "\r\n")+2;
         $temp = substr($buffer, 0, $chunkEnd);
         $chunkSize = hexdec(trim($temp));
         $chunkStart = $chunkEnd;
-        while ($chunkSize > 0) {
-            $chunkEnd = strpos($buffer, "\r\n", $chunkStart + $chunkSize);
+        while ($chunkSize>0) {
+            $chunkEnd = strpos($buffer, "\r\n", $chunkStart+$chunkSize);
 
             // just in case we got a broken connection
             if ($chunkEnd == false) {
@@ -39,19 +39,19 @@  discard block
 block discarded – undo
             }
 
             // read chunk-data and crlf
-            $chunk = substr($buffer, $chunkStart, $chunkEnd - $chunkStart);
+            $chunk = substr($buffer, $chunkStart, $chunkEnd-$chunkStart);
             // append chunk-data to entity-body
             $new .= $chunk;
             // length := length + chunk-size
             $length += strlen($chunk);
             // read chunk-size and crlf
-            $chunkStart = $chunkEnd + 2;
+            $chunkStart = $chunkEnd+2;
 
-            $chunkEnd = strpos($buffer, "\r\n", $chunkStart) + 2;
+            $chunkEnd = strpos($buffer, "\r\n", $chunkStart)+2;
             if ($chunkEnd == false) {
                 break; //just in case we got a broken connection
             }
-            $temp = substr($buffer, $chunkStart, $chunkEnd - $chunkStart);
+            $temp = substr($buffer, $chunkStart, $chunkEnd-$chunkStart);
             $chunkSize = hexdec(trim($temp));
             $chunkStart = $chunkEnd;
         }
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
      * @return array with keys 'headers' and 'cookies'
      * @throws \Exception
      */
-    public function parseResponseHeaders(&$data, $headersProcessed = false, $debug=0)
+    public function parseResponseHeaders(&$data, $headersProcessed = false, $debug = 0)
     {
         $httpResponse = array('raw_data' => $data, 'headers'=> array(), 'cookies' => array());
 
@@ -78,11 +78,11 @@  discard block
 block discarded – undo
             // (even though it is not valid http)
             $pos = strpos($data, "\r\n\r\n");
             if ($pos || is_int($pos)) {
-                $bd = $pos + 4;
+                $bd = $pos+4;
             } else {
                 $pos = strpos($data, "\n\n");
                 if ($pos || is_int($pos)) {
-                    $bd = $pos + 2;
+                    $bd = $pos+2;
                 } else {
                     // No separation between response headers and body: fault?
                     $bd = 0;
@@ -93,8 +93,8 @@  discard block
 block discarded – undo
                 // maybe we could take them into account, too?
                 $data = substr($data, $bd);
             } else {
-                error_log('XML-RPC: ' . __METHOD__ . ': HTTPS via proxy error, tunnel connection possibly failed');
-                throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'] . ' (HTTPS via proxy error, tunnel connection possibly failed)', PhpXmlRpc::$xmlrpcerr['http_error']);
+                error_log('XML-RPC: '.__METHOD__.': HTTPS via proxy error, tunnel connection possibly failed');
+                throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'].' (HTTPS via proxy error, tunnel connection possibly failed)', PhpXmlRpc::$xmlrpcerr['http_error']);
             }
         }
 
@@ -111,20 +111,20 @@  discard block
 block discarded – undo
             $data = substr($data, $pos);
         }
         if (!preg_match('/^HTTP\/[0-9.]+ 200 /', $data)) {
-            $errstr = substr($data, 0, strpos($data, "\n") - 1);
-            error_log('XML-RPC: ' . __METHOD__ . ': HTTP error, got response: ' . $errstr);
-            throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'] . ' (' . $errstr . ')', PhpXmlRpc::$xmlrpcerr['http_error']);
+            $errstr = substr($data, 0, strpos($data, "\n")-1);
+            error_log('XML-RPC: '.__METHOD__.': HTTP error, got response: '.$errstr);
+            throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'].' ('.$errstr.')', PhpXmlRpc::$xmlrpcerr['http_error']);
         }
 
         // be tolerant to usage of \n instead of \r\n to separate headers and data
         // (even though it is not valid http)
         $pos = strpos($data, "\r\n\r\n");
         if ($pos || is_int($pos)) {
-            $bd = $pos + 4;
+            $bd = $pos+4;
         } else {
             $pos = strpos($data, "\n\n");
             if ($pos || is_int($pos)) {
-                $bd = $pos + 2;
+                $bd = $pos+2;
             } else {
                 // No separation between response headers and body: fault?
                 // we could take some action here instead of going on...
@@ -133,10 +133,10 @@  discard block
 block discarded – undo
         }
         // be tolerant to line endings, and extra empty lines
         $ar = preg_split("/\r?\n/", trim(substr($data, 0, $pos)));
-        foreach($ar as $line) {
+        foreach ($ar as $line) {
             // take care of multi-line headers and cookies
             $arr = explode(':', $line, 2);
-            if (count($arr) > 1) {
+            if (count($arr)>1) {
                 $headerName = strtolower(trim($arr[0]));
                 /// @todo some other headers (the ones that allow a CSV list of values)
                 /// do allow many values to be passed using multiple header lines.
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
                         // glue together all received cookies, using a comma to separate them
                         // (same as php does with getallheaders())
                         if (isset($httpResponse['headers'][$headerName])) {
-                            $httpResponse['headers'][$headerName] .= ', ' . trim($cookie);
+                            $httpResponse['headers'][$headerName] .= ', '.trim($cookie);
                         } else {
                             $httpResponse['headers'][$headerName] = trim($cookie);
                         }
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
                 }
             } elseif (isset($headerName)) {
                 /// @todo version1 cookies might span multiple lines, thus breaking the parsing above
-                $httpResponse['headers'][$headerName] .= ' ' . trim($line);
+                $httpResponse['headers'][$headerName] .= ' '.trim($line);
             }
         }
 
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
             // Decode chunked encoding sent by http 1.1 servers
             if (isset($httpResponse['headers']['transfer-encoding']) && $httpResponse['headers']['transfer-encoding'] == 'chunked') {
                 if (!$data = Http::decodeChunked($data)) {
-                    error_log('XML-RPC: ' . __METHOD__ . ': errors occurred when trying to rebuild the chunked data received from server');
+                    error_log('XML-RPC: '.__METHOD__.': errors occurred when trying to rebuild the chunked data received from server');
                     throw new \Exception(PhpXmlRpc::$xmlrpcstr['dechunk_fail'], PhpXmlRpc::$xmlrpcerr['dechunk_fail']);
                 }
             }
@@ -221,19 +221,19 @@  discard block
 block discarded – undo
                         if ($httpResponse['headers']['content-encoding'] == 'deflate' && $degzdata = @gzuncompress($data)) {
                             $data = $degzdata;
                             if ($debug) {
-                                Logger::instance()->debugMessage("---INFLATED RESPONSE---[" . strlen($data) . " chars]---\n$data\n---END---");
+                                Logger::instance()->debugMessage("---INFLATED RESPONSE---[".strlen($data)." chars]---\n$data\n---END---");
                             }
                         } elseif ($httpResponse['headers']['content-encoding'] == 'gzip' && $degzdata = @gzinflate(substr($data, 10))) {
                             $data = $degzdata;
                             if ($debug) {
-                                Logger::instance()->debugMessage("---INFLATED RESPONSE---[" . strlen($data) . " chars]---\n$data\n---END---");
+                                Logger::instance()->debugMessage("---INFLATED RESPONSE---[".strlen($data)." chars]---\n$data\n---END---");
                             }
                         } else {
-                            error_log('XML-RPC: ' . __METHOD__ . ': errors occurred when trying to decode the deflated data received from server');
+                            error_log('XML-RPC: '.__METHOD__.': errors occurred when trying to decode the deflated data received from server');
                             throw new \Exception(PhpXmlRpc::$xmlrpcstr['decompress_fail'], PhpXmlRpc::$xmlrpcerr['decompress_fail']);
                         }
                     } else {
-                        error_log('XML-RPC: ' . __METHOD__ . ': the server sent deflated data. Your php install must have the Zlib extension compiled in to support this.');
+                        error_log('XML-RPC: '.__METHOD__.': the server sent deflated data. Your php install must have the Zlib extension compiled in to support this.');
                         throw new \Exception(PhpXmlRpc::$xmlrpcstr['cannot_decompress'], PhpXmlRpc::$xmlrpcerr['cannot_decompress']);
                     }
                 }

Please login to merge, or discard this patch.

src/Encoder.php 1 patch

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

@@ -75,7 +75,7 @@  discard block
 block discarded – undo
                 return $xmlrpcVal->scalarval();
             case 'array':
                 $arr = array();
-                foreach($xmlrpcVal as $value) {
+                foreach ($xmlrpcVal as $value) {
                     $arr[] = $this->decode($value, $options);
                 }
 
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
             case 'msg':
                 $paramCount = $xmlrpcVal->getNumParams();
                 $arr = array();
-                for ($i = 0; $i < $paramCount; $i++) {
+                for ($i = 0; $i<$paramCount; $i++) {
                     $arr[] = $this->decode($xmlrpcVal->getParam($i), $options);
                 }
 
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
                     $xmlrpcVal = new Value($phpVal->format('Ymd\TH:i:s'), Value::$xmlrpcStruct);
                 } else {
                     $arr = array();
-                    foreach($phpVal as $k => $v) {
+                    foreach ($phpVal as $k => $v) {
                         $arr[$k] = $this->encode($v, $options);
                     }
                     $xmlrpcVal = new Value($arr, Value::$xmlrpcStruct);
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
                 break;
             case 'resource':
                 if (in_array('extension_api', $options)) {
-                    $xmlrpcVal = new Value((int)$phpVal, Value::$xmlrpcInt);
+                    $xmlrpcVal = new Value((int) $phpVal, Value::$xmlrpcInt);
                 } else {
                     $xmlrpcVal = new Value();
                 }
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
                     if (extension_loaded('mbstring')) {
                         $xmlVal = mb_convert_encoding($xmlVal, 'UTF-8', $valEncoding);
                     } else {
-                        error_log('XML-RPC: ' . __METHOD__ . ': invalid charset encoding of xml text: ' . $valEncoding);
+                        error_log('XML-RPC: '.__METHOD__.': invalid charset encoding of xml text: '.$valEncoding);
                     }
                 }
             }
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
             return false;
         }
         xml_parser_free($parser);
-        if ($xmlRpcParser->_xh['isf'] > 1) {
+        if ($xmlRpcParser->_xh['isf']>1) {
             // test that $xmlrpc->_xh['value'] is an obj, too???
 
             error_log($xmlRpcParser->_xh['isf_reason']);
@@ -301,7 +301,7 @@  discard block
 block discarded – undo
                 return $r;
             case 'methodcall':
                 $req = new Request($xmlRpcParser->_xh['method']);
-                for ($i = 0; $i < count($xmlRpcParser->_xh['params']); $i++) {
+                for ($i = 0; $i<count($xmlRpcParser->_xh['params']); $i++) {
                     $req->addParam($xmlRpcParser->_xh['params'][$i]);
                 }
 

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

@@ -95,7 +95,7 @@  discard block
 block discarded – undo
         $val = reset($ar);
         $typ = key($ar);
 
-        return '<value>' . $this->serializedata($typ, $val) . "</value>\n";
+        return '<value>'.$this->serializedata($typ, $val)."</value>\n";
         //}
     }
 
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
         // preserve back compatibility
 
         if (is_array($b)) {
-            foreach($b as $id => $cont) {
+            foreach ($b as $id => $cont) {
                 $b[$id] = $cont->scalarval();
             }
         }
@@ -122,10 +122,10 @@  discard block
 block discarded – undo
         // add support for structures directly encoding php objects
         if (is_object($b)) {
             $t = get_object_vars($b);
-            foreach($t as $id => $cont) { {
+            foreach ($t as $id => $cont) { {
                 $t[$id] = $cont->scalarval();
             }
-            foreach($t as $id => $cont) {
+            foreach ($t as $id => $cont) {
                 @$b->$id = $cont;
             }
         }
@@ -160,17 +160,17 @@  discard block
 block discarded – undo
 /* Expose as global functions the ones which are now class methods */
 
 /// Wrong speling, but we are adamant on backwards compatibility!
-function xmlrpc_encode_entitites($data, $srcEncoding='', $destEncoding='')
+function xmlrpc_encode_entitites($data, $srcEncoding = '', $destEncoding = '')
 {
     return PhpXmlRpc\Helper\Charset::instance()->encodeEntitites($data, $srcEncoding, $destEncoding);
 }
 
-function iso8601_encode($timeT, $utc=0)
+function iso8601_encode($timeT, $utc = 0)
 {
     return PhpXmlRpc\Helper\Date::iso8601Encode($timeT, $utc);
 }
 
-function iso8601_decode($iDate, $utc=0)
+function iso8601_decode($iDate, $utc = 0)
 {
     return PhpXmlRpc\Helper\Date::iso8601Decode($iDate, $utc);
 }
@@ -180,25 +180,25 @@  discard block
 block discarded – undo
     return PhpXmlRpc\Helper\Http::decodeChunked($buffer);
 }
 
-function php_xmlrpc_decode($xmlrpcVal, $options=array())
+function php_xmlrpc_decode($xmlrpcVal, $options = array())
 {
     $encoder = new PhpXmlRpc\Encoder();
     return $encoder->decode($xmlrpcVal, $options);
 }
 
-function php_xmlrpc_encode($phpVal, $options=array())
+function php_xmlrpc_encode($phpVal, $options = array())
 {
     $encoder = new PhpXmlRpc\Encoder();
     return $encoder->encode($phpVal, $options);
 }
 
-function php_xmlrpc_decode_xml($xmlVal, $options=array())
+function php_xmlrpc_decode_xml($xmlVal, $options = array())
 {
     $encoder = new PhpXmlRpc\Encoder();
     return $encoder->decodeXml($xmlVal, $options);
 }
 
-function guess_encoding($httpHeader='', $xmlChunk='', $encodingPrefs=null)
+function guess_encoding($httpHeader = '', $xmlChunk = '', $encodingPrefs = null)
 {
     return PhpXmlRpc\Helper\XMLParser::guessEncoding($httpHeader, $xmlChunk, $encodingPrefs);
 }

Please login to merge, or discard this patch.

		@@ -66,7 +66,7 @@ discard block
		block discarded – undo
66	66	public function xmlrpc_se($parser, $name, $attrs, $acceptSingleVals = false)
67	67	{
68	68	// if invalid xmlrpc already detected, skip all processing
69		- if ($this->_xh['isf'] < 2) {
	69	+ if ($this->_xh['isf']<2) {
70	70	// check for correct element nesting
71	71	// top level element can only be of 2 types
72	72	/// @todo optimization creep: save this check into a bool variable, instead of using count() every time:
		@@ -171,7 +171,7 @@ discard block
		block discarded – undo
171	171	$this->_xh['isf'] = 1;
172	172	break;
173	173	case 'MEMBER':
174		- $this->_xh['valuestack'][count($this->_xh['valuestack']) - 1]['name'] = ''; // set member name to null, in case we do not find in the xml later on
	174	+ $this->_xh['valuestack'][count($this->_xh['valuestack'])-1]['name'] = ''; // set member name to null, in case we do not find in the xml later on
175	175	//$this->_xh['ac']='';
176	176	// Drop trough intentionally
177	177	case 'PARAM':
		@@ -223,7 +223,7 @@ discard block
		block discarded – undo
223	223	*/
224	224	public function xmlrpc_ee($parser, $name, $rebuildXmlrpcvals = true)
225	225	{
226		- if ($this->_xh['isf'] < 2) {
	226	+ if ($this->_xh['isf']<2) {
227	227	// push this element name from stack
228	228	// NB: if XML validates, correct opening/closing is guaranteed and
229	229	// we do not have to check for $name == $currElem.
		@@ -249,8 +249,8 @@ discard block
		block discarded – undo
249	249	// check if we are inside an array or struct:
250	250	// if value just built is inside an array, let's move it into array on the stack
251	251	$vscount = count($this->_xh['valuestack']);
252		- if ($vscount && $this->_xh['valuestack'][$vscount - 1]['type'] == 'ARRAY') {
253		- $this->_xh['valuestack'][$vscount - 1]['values'][] = $temp;
	252	+ if ($vscount && $this->_xh['valuestack'][$vscount-1]['type'] == 'ARRAY') {
	253	+ $this->_xh['valuestack'][$vscount-1]['values'][] = $temp;
254	254	} else {
255	255	$this->_xh['value'] = $temp;
256	256	}
		@@ -264,8 +264,8 @@ discard block
		block discarded – undo
264	264	// check if we are inside an array or struct:
265	265	// if value just built is inside an array, let's move it into array on the stack
266	266	$vscount = count($this->_xh['valuestack']);
267		- if ($vscount && $this->_xh['valuestack'][$vscount - 1]['type'] == 'ARRAY') {
268		- $this->_xh['valuestack'][$vscount - 1]['values'][] = $this->_xh['value'];
	267	+ if ($vscount && $this->_xh['valuestack'][$vscount-1]['type'] == 'ARRAY') {
	268	+ $this->_xh['valuestack'][$vscount-1]['values'][] = $this->_xh['value'];
269	269	}
270	270	}
271	271	break;
		@@ -285,7 +285,7 @@ discard block
		block discarded – undo
285	285	$this->_xh['value'] = $this->_xh['ac'];
286	286	} elseif ($name == 'DATETIME.ISO8601') {
287	287	if (!preg_match('/^[0-9]{8}T[0-9]{2}:[0-9]{2}:[0-9]{2}$/', $this->_xh['ac'])) {
288		- error_log('XML-RPC: ' . __METHOD__ . ': invalid value received in DATETIME: ' . $this->_xh['ac']);
	288	+ error_log('XML-RPC: '.__METHOD__.': invalid value received in DATETIME: '.$this->_xh['ac']);
289	289	}
290	290	$this->_xh['vt'] = Value::$xmlrpcDateTime;
291	291	$this->_xh['value'] = $this->_xh['ac'];
		@@ -304,7 +304,7 @@ discard block
		block discarded – undo
304	304	} else {
305	305	// log if receiving something strange, even though we set the value to false anyway
306	306	if ($this->_xh['ac'] != '0' && strcasecmp($this->_xh['ac'], 'false') != 0) {
307		- error_log('XML-RPC: ' . __METHOD__ . ': invalid value received in BOOLEAN: ' . $this->_xh['ac']);
	307	+ error_log('XML-RPC: '.__METHOD__.': invalid value received in BOOLEAN: '.$this->_xh['ac']);
308	308	}
309	309	$this->_xh['value'] = false;
310	310	}
		@@ -314,37 +314,37 @@ discard block
		block discarded – undo
314	314	// NOTE: regexp could be much stricter than this...
315	315	if (!preg_match('/^[+-eE0123456789 \t.]+$/', $this->_xh['ac'])) {
316	316	/// @todo: find a better way of throwing an error than this!
317		- error_log('XML-RPC: ' . __METHOD__ . ': non numeric value received in DOUBLE: ' . $this->_xh['ac']);
	317	+ error_log('XML-RPC: '.__METHOD__.': non numeric value received in DOUBLE: '.$this->_xh['ac']);
318	318	$this->_xh['value'] = 'ERROR_NON_NUMERIC_FOUND';
319	319	} else {
320	320	// it's ok, add it on
321		- $this->_xh['value'] = (double)$this->_xh['ac'];
	321	+ $this->_xh['value'] = (double) $this->_xh['ac'];
322	322	}
323	323	} else {
324	324	// we have an I4/I8/INT
325	325	// we must check that only 0123456789-<space> are characters here
326	326	if (!preg_match('/^[+-]?[0123456789 \t]+$/', $this->_xh['ac'])) {
327	327	/// @todo find a better way of throwing an error than this!
328		- error_log('XML-RPC: ' . __METHOD__ . ': non numeric value received in INT: ' . $this->_xh['ac']);
	328	+ error_log('XML-RPC: '.__METHOD__.': non numeric value received in INT: '.$this->_xh['ac']);
329	329	$this->_xh['value'] = 'ERROR_NON_NUMERIC_FOUND';
330	330	} else {
331	331	// it's ok, add it on
332		- $this->_xh['value'] = (int)$this->_xh['ac'];
	332	+ $this->_xh['value'] = (int) $this->_xh['ac'];
333	333	}
334	334	}
335	335	$this->_xh['lv'] = 3; // indicate we've found a value
336	336	break;
337	337	case 'NAME':
338		- $this->_xh['valuestack'][count($this->_xh['valuestack']) - 1]['name'] = $this->_xh['ac'];
	338	+ $this->_xh['valuestack'][count($this->_xh['valuestack'])-1]['name'] = $this->_xh['ac'];
339	339	break;
340	340	case 'MEMBER':
341	341	// add to array in the stack the last element built,
342	342	// unless no VALUE was found
343	343	if ($this->_xh['vt']) {
344	344	$vscount = count($this->_xh['valuestack']);
345		- $this->_xh['valuestack'][$vscount - 1]['values'][$this->_xh['valuestack'][$vscount - 1]['name']] = $this->_xh['value'];
	345	+ $this->_xh['valuestack'][$vscount-1]['values'][$this->_xh['valuestack'][$vscount-1]['name']] = $this->_xh['value'];
346	346	} else {
347		- error_log('XML-RPC: ' . __METHOD__ . ': missing VALUE inside STRUCT in received xml');
	347	+ error_log('XML-RPC: '.__METHOD__.': missing VALUE inside STRUCT in received xml');
348	348	}
349	349	break;
350	350	case 'DATA':
		@@ -367,7 +367,7 @@ discard block
		block discarded – undo
367	367	$this->_xh['params'][] = $this->_xh['value'];
368	368	$this->_xh['pt'][] = $this->_xh['vt'];
369	369	} else {
370		- error_log('XML-RPC: ' . __METHOD__ . ': missing VALUE inside PARAM in received xml');
	370	+ error_log('XML-RPC: '.__METHOD__.': missing VALUE inside PARAM in received xml');
371	371	}
372	372	break;
373	373	case 'METHODNAME':
		@@ -409,7 +409,7 @@ discard block
		block discarded – undo
409	409	public function xmlrpc_cd($parser, $data)
410	410	{
411	411	// skip processing if xml fault already detected
412		- if ($this->_xh['isf'] < 2) {
	412	+ if ($this->_xh['isf']<2) {
413	413	// "lookforvalue==3" means that we've found an entire value
414	414	// and should discard any further character data
415	415	if ($this->_xh['lv'] != 3) {
		@@ -425,7 +425,7 @@ discard block
		block discarded – undo
425	425	public function xmlrpc_dh($parser, $data)
426	426	{
427	427	// skip processing if xml fault already detected
428		- if ($this->_xh['isf'] < 2) {
	428	+ if ($this->_xh['isf']<2) {
429	429	if (substr($data, 0, 1) == '&' && substr($data, -1, 1) == ';') {
430	430	$this->_xh['ac'] .= $data;
431	431	}
		@@ -496,8 +496,8 @@ discard block
		block discarded – undo
496	496	// Details:
497	497	// SPACE: (#x20 \| #x9 \| #xD \| #xA)+ === [ \x9\xD\xA]+
498	498	// EQ: SPACE?=SPACE? === [ \x9\xD\xA]=[ \x9\xD\xA]
499		- if (preg_match('/^<\?xml\s+version\s=\s' . "((?:\"[a-zA-Z0-9_.:-]+\")\|(?:'[a-zA-Z0-9_.:-]+'))" .
500		- '\s+encoding\s=\s' . "((?:\"[A-Za-z][A-Za-z0-9._-]\")\|(?:'[A-Za-z][A-Za-z0-9._-]'))/",
	499	+ if (preg_match('/^<\?xml\s+version\s=\s'."((?:\"[a-zA-Z0-9_.:-]+\")\|(?:'[a-zA-Z0-9_.:-]+'))".
	500	+ '\s+encoding\s=\s'."((?:\"[A-Za-z][A-Za-z0-9._-]\")\|(?:'[A-Za-z][A-Za-z0-9._-]'))/",
501	501	$xmlChunk, $matches)) {
502	502	return strtoupper(substr($matches[2], 1, -1));
503	503	}
		@@ -515,7 +515,7 @@ discard block
		block discarded – undo
515	515	// NB: mb_detect likes to call it ascii, xml parser likes to call it US_ASCII...
516	516	// IANA also likes better US-ASCII, so go with it
517	517	if ($enc == 'ASCII') {
518		- $enc = 'US-' . $enc;
	518	+ $enc = 'US-'.$enc;
519	519	}
520	520
521	521	return $enc;
		@@ -550,8 +550,8 @@ discard block
		block discarded – undo
550	550	// Details:
551	551	// SPACE: (#x20 \| #x9 \| #xD \| #xA)+ === [ \x9\xD\xA]+
552	552	// EQ: SPACE?=SPACE? === [ \x9\xD\xA]=[ \x9\xD\xA]
553		- if (preg_match('/^<\?xml\s+version\s=\s' . "((?:\"[a-zA-Z0-9_.:-]+\")\|(?:'[a-zA-Z0-9_.:-]+'))" .
554		- '\s+encoding\s=\s' . "((?:\"[A-Za-z][A-Za-z0-9._-]\")\|(?:'[A-Za-z][A-Za-z0-9._-]'))/",
	553	+ if (preg_match('/^<\?xml\s+version\s=\s'."((?:\"[a-zA-Z0-9_.:-]+\")\|(?:'[a-zA-Z0-9_.:-]+'))".
	554	+ '\s+encoding\s=\s'."((?:\"[A-Za-z][A-Za-z0-9._-]\")\|(?:'[A-Za-z][A-Za-z0-9._-]'))/",
555	555	$xmlChunk, $matches)) {
556	556	return true;
557	557	}

		@@ -95,7 +95,7 @@ discard block
		block discarded – undo
95	95	$val = reset($ar);
96	96	$typ = key($ar);
97	97
98		- return '<value>' . $this->serializedata($typ, $val) . "</value>\n";
	98	+ return '<value>'.$this->serializedata($typ, $val)."</value>\n";
99	99	//}
100	100	}
101	101
		@@ -114,7 +114,7 @@ discard block
		block discarded – undo
114	114	// preserve back compatibility
115	115
116	116	if (is_array($b)) {
117		- foreach($b as $id => $cont) {
	117	+ foreach ($b as $id => $cont) {
118	118	$b[$id] = $cont->scalarval();
119	119	}
120	120	}
		@@ -122,10 +122,10 @@ discard block
		block discarded – undo
122	122	// add support for structures directly encoding php objects
123	123	if (is_object($b)) {
124	124	$t = get_object_vars($b);
125		- foreach($t as $id => $cont) { {
	125	+ foreach ($t as $id => $cont) { {
126	126	$t[$id] = $cont->scalarval();
127	127	}
128		- foreach($t as $id => $cont) {
	128	+ foreach ($t as $id => $cont) {
129	129	@$b->$id = $cont;
130	130	}
131	131	}
		@@ -160,17 +160,17 @@ discard block
		block discarded – undo
160	160	/* Expose as global functions the ones which are now class methods */
161	161
162	162	/// Wrong speling, but we are adamant on backwards compatibility!
163		-function xmlrpc_encode_entitites($data, $srcEncoding='', $destEncoding='')
	163	+function xmlrpc_encode_entitites($data, $srcEncoding = '', $destEncoding = '')
164	164	{
165	165	return PhpXmlRpc\Helper\Charset::instance()->encodeEntitites($data, $srcEncoding, $destEncoding);
166	166	}
167	167
168		-function iso8601_encode($timeT, $utc=0)
	168	+function iso8601_encode($timeT, $utc = 0)
169	169	{
170	170	return PhpXmlRpc\Helper\Date::iso8601Encode($timeT, $utc);
171	171	}
172	172
173		-function iso8601_decode($iDate, $utc=0)
	173	+function iso8601_decode($iDate, $utc = 0)
174	174	{
175	175	return PhpXmlRpc\Helper\Date::iso8601Decode($iDate, $utc);
176	176	}
		@@ -180,25 +180,25 @@ discard block
		block discarded – undo
180	180	return PhpXmlRpc\Helper\Http::decodeChunked($buffer);
181	181	}
182	182
183		-function php_xmlrpc_decode($xmlrpcVal, $options=array())
	183	+function php_xmlrpc_decode($xmlrpcVal, $options = array())
184	184	{
185	185	$encoder = new PhpXmlRpc\Encoder();
186	186	return $encoder->decode($xmlrpcVal, $options);
187	187	}
188	188
189		-function php_xmlrpc_encode($phpVal, $options=array())
	189	+function php_xmlrpc_encode($phpVal, $options = array())
190	190	{
191	191	$encoder = new PhpXmlRpc\Encoder();
192	192	return $encoder->encode($phpVal, $options);
193	193	}
194	194
195		-function php_xmlrpc_decode_xml($xmlVal, $options=array())
	195	+function php_xmlrpc_decode_xml($xmlVal, $options = array())
196	196	{
197	197	$encoder = new PhpXmlRpc\Encoder();
198	198	return $encoder->decodeXml($xmlVal, $options);
199	199	}
200	200
201		-function guess_encoding($httpHeader='', $xmlChunk='', $encodingPrefs=null)
	201	+function guess_encoding($httpHeader = '', $xmlChunk = '', $encodingPrefs = null)
202	202	{
203	203	return PhpXmlRpc\Helper\XMLParser::guessEncoding($httpHeader, $xmlChunk, $encodingPrefs);
204	204	}

		@@ -3,16 +3,16 @@ discard block
		block discarded – undo
3	3	<body>
4	4	<?php
5	5
6		-include_once __DIR__ . "/../vendor/autoload.php";
	6	+include_once __DIR__."/../vendor/autoload.php";
7	7
8	8	$req = new PhpXmlRpc\Request('examples.getStateName');
9	9
10	10	print "<h3>Testing value serialization</h3>\n";
11	11
12	12	$v = new PhpXmlRpc\Value(23, "int");
13		-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
	13	+print "<PRE>".htmlentities($v->serialize())."</PRE>";
14	14	$v = new PhpXmlRpc\Value("What are you saying? >> << &&");
15		-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
	15	+print "<PRE>".htmlentities($v->serialize())."</PRE>";
16	16
17	17	$v = new PhpXmlRpc\Value(
18	18	array(
		@@ -23,7 +23,7 @@ discard block
		block discarded – undo
23	23	"array"
24	24	);
25	25
26		-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
	26	+print "<PRE>".htmlentities($v->serialize())."</PRE>";
27	27
28	28	$v = new PhpXmlRpc\Value(
29	29	array(
		@@ -51,11 +51,11 @@ discard block
		block discarded – undo
51	51	"struct"
52	52	);
53	53
54		-print "<PRE>" . htmlentities($v->serialize()) . "</PRE>";
	54	+print "<PRE>".htmlentities($v->serialize())."</PRE>";
55	55
56	56	$w = new PhpXmlRpc\Value(array($v, new PhpXmlRpc\Value("That was the struct!")), "array");
57	57
58		-print "<PRE>" . htmlentities($w->serialize()) . "</PRE>";
	58	+print "<PRE>".htmlentities($w->serialize())."</PRE>";
59	59
60	60	$w = new PhpXmlRpc\Value("Mary had a little lamb,
61	61	Whose fleece was white as snow,
		@@ -67,26 +67,26 @@ discard block
		block discarded – undo
67	67	Ten thousand volts went down its back
68	68	And turned it into nylon", "base64"
69	69	);
70		-print "<PRE>" . htmlentities($w->serialize()) . "</PRE>";
71		-print "<PRE>Value of base64 string is: '" . $w->scalarval() . "'</PRE>";
	70	+print "<PRE>".htmlentities($w->serialize())."</PRE>";
	71	+print "<PRE>Value of base64 string is: '".$w->scalarval()."'</PRE>";
72	72
73	73	$req->method('');
74	74	$req->addParam(new PhpXmlRpc\Value("41", "int"));
75	75
76	76	print "<h3>Testing request serialization</h3>\n";
77	77	$op = $req->serialize();
78		-print "<PRE>" . htmlentities($op) . "</PRE>";
	78	+print "<PRE>".htmlentities($op)."</PRE>";
79	79
80	80	print "<h3>Testing ISO date format</h3><pre>\n";
81	81
82	82	$t = time();
83	83	$date = PhpXmlRpc\Helper\Date::iso8601Encode($t);
84	84	print "Now is $t --> $date\n";
85		-print "Or in UTC, that is " . PhpXmlRpc\Helper\Date::iso8601Encode($t, 1) . "\n";
	85	+print "Or in UTC, that is ".PhpXmlRpc\Helper\Date::iso8601Encode($t, 1)."\n";
86	86	$tb = PhpXmlRpc\Helper\Date::iso8601Decode($date);
87	87	print "That is to say $date --> $tb\n";
88		-print "Which comes out at " . PhpXmlRpc\Helper\Date::iso8601Encode($tb) . "\n";
89		-print "Which was the time in UTC at " . PhpXmlRpc\Helper\Date::iso8601Encode($date, 1) . "\n";
	88	+print "Which comes out at ".PhpXmlRpc\Helper\Date::iso8601Encode($tb)."\n";
	89	+print "Which was the time in UTC at ".PhpXmlRpc\Helper\Date::iso8601Encode($date, 1)."\n";
90	90
91	91	print "</pre>\n";
92	92

		@@ -22,12 +22,12 @@ discard block
		block discarded – undo
22	22
23	23	// read chunk-size, chunk-extension (if any) and crlf
24	24	// get the position of the linebreak
25		- $chunkEnd = strpos($buffer, "\r\n") + 2;
	25	+ $chunkEnd = strpos($buffer, "\r\n")+2;
26	26	$temp = substr($buffer, 0, $chunkEnd);
27	27	$chunkSize = hexdec(trim($temp));
28	28	$chunkStart = $chunkEnd;
29		- while ($chunkSize > 0) {
30		- $chunkEnd = strpos($buffer, "\r\n", $chunkStart + $chunkSize);
	29	+ while ($chunkSize>0) {
	30	+ $chunkEnd = strpos($buffer, "\r\n", $chunkStart+$chunkSize);
31	31
32	32	// just in case we got a broken connection
33	33	if ($chunkEnd == false) {
		@@ -39,19 +39,19 @@ discard block
		block discarded – undo
39	39	}
40	40
41	41	// read chunk-data and crlf
42		- $chunk = substr($buffer, $chunkStart, $chunkEnd - $chunkStart);
	42	+ $chunk = substr($buffer, $chunkStart, $chunkEnd-$chunkStart);
43	43	// append chunk-data to entity-body
44	44	$new .= $chunk;
45	45	// length := length + chunk-size
46	46	$length += strlen($chunk);
47	47	// read chunk-size and crlf
48		- $chunkStart = $chunkEnd + 2;
	48	+ $chunkStart = $chunkEnd+2;
49	49
50		- $chunkEnd = strpos($buffer, "\r\n", $chunkStart) + 2;
	50	+ $chunkEnd = strpos($buffer, "\r\n", $chunkStart)+2;
51	51	if ($chunkEnd == false) {
52	52	break; //just in case we got a broken connection
53	53	}
54		- $temp = substr($buffer, $chunkStart, $chunkEnd - $chunkStart);
	54	+ $temp = substr($buffer, $chunkStart, $chunkEnd-$chunkStart);
55	55	$chunkSize = hexdec(trim($temp));
56	56	$chunkStart = $chunkEnd;
57	57	}
		@@ -68,7 +68,7 @@ discard block
		block discarded – undo
68	68	* @return array with keys 'headers' and 'cookies'
69	69	* @throws \Exception
70	70	*/
71		- public function parseResponseHeaders(&$data, $headersProcessed = false, $debug=0)
	71	+ public function parseResponseHeaders(&$data, $headersProcessed = false, $debug = 0)
72	72	{
73	73	$httpResponse = array('raw_data' => $data, 'headers'=> array(), 'cookies' => array());
74	74
		@@ -78,11 +78,11 @@ discard block
		block discarded – undo
78	78	// (even though it is not valid http)
79	79	$pos = strpos($data, "\r\n\r\n");
80	80	if ($pos \|\| is_int($pos)) {
81		- $bd = $pos + 4;
	81	+ $bd = $pos+4;
82	82	} else {
83	83	$pos = strpos($data, "\n\n");
84	84	if ($pos \|\| is_int($pos)) {
85		- $bd = $pos + 2;
	85	+ $bd = $pos+2;
86	86	} else {
87	87	// No separation between response headers and body: fault?
88	88	$bd = 0;
		@@ -93,8 +93,8 @@ discard block
		block discarded – undo
93	93	// maybe we could take them into account, too?
94	94	$data = substr($data, $bd);
95	95	} else {
96		- error_log('XML-RPC: ' . __METHOD__ . ': HTTPS via proxy error, tunnel connection possibly failed');
97		- throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'] . ' (HTTPS via proxy error, tunnel connection possibly failed)', PhpXmlRpc::$xmlrpcerr['http_error']);
	96	+ error_log('XML-RPC: '.__METHOD__.': HTTPS via proxy error, tunnel connection possibly failed');
	97	+ throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'].' (HTTPS via proxy error, tunnel connection possibly failed)', PhpXmlRpc::$xmlrpcerr['http_error']);
98	98	}
99	99	}
100	100
		@@ -111,20 +111,20 @@ discard block
		block discarded – undo
111	111	$data = substr($data, $pos);
112	112	}
113	113	if (!preg_match('/^HTTP\/[0-9.]+ 200 /', $data)) {
114		- $errstr = substr($data, 0, strpos($data, "\n") - 1);
115		- error_log('XML-RPC: ' . __METHOD__ . ': HTTP error, got response: ' . $errstr);
116		- throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'] . ' (' . $errstr . ')', PhpXmlRpc::$xmlrpcerr['http_error']);
	114	+ $errstr = substr($data, 0, strpos($data, "\n")-1);
	115	+ error_log('XML-RPC: '.__METHOD__.': HTTP error, got response: '.$errstr);
	116	+ throw new \Exception(PhpXmlRpc::$xmlrpcstr['http_error'].' ('.$errstr.')', PhpXmlRpc::$xmlrpcerr['http_error']);
117	117	}
118	118
119	119	// be tolerant to usage of \n instead of \r\n to separate headers and data
120	120	// (even though it is not valid http)
121	121	$pos = strpos($data, "\r\n\r\n");
122	122	if ($pos \|\| is_int($pos)) {
123		- $bd = $pos + 4;
	123	+ $bd = $pos+4;
124	124	} else {
125	125	$pos = strpos($data, "\n\n");
126	126	if ($pos \|\| is_int($pos)) {
127		- $bd = $pos + 2;
	127	+ $bd = $pos+2;
128	128	} else {
129	129	// No separation between response headers and body: fault?
130	130	// we could take some action here instead of going on...
		@@ -133,10 +133,10 @@ discard block
		block discarded – undo
133	133	}
134	134	// be tolerant to line endings, and extra empty lines
135	135	$ar = preg_split("/\r?\n/", trim(substr($data, 0, $pos)));
136		- foreach($ar as $line) {
	136	+ foreach ($ar as $line) {
137	137	// take care of multi-line headers and cookies
138	138	$arr = explode(':', $line, 2);
139		- if (count($arr) > 1) {
	139	+ if (count($arr)>1) {
140	140	$headerName = strtolower(trim($arr[0]));
141	141	/// @todo some other headers (the ones that allow a CSV list of values)
142	142	/// do allow many values to be passed using multiple header lines.
		@@ -154,7 +154,7 @@ discard block
		block discarded – undo
154	154	// glue together all received cookies, using a comma to separate them
155	155	// (same as php does with getallheaders())
156	156	if (isset($httpResponse['headers'][$headerName])) {
157		- $httpResponse['headers'][$headerName] .= ', ' . trim($cookie);
	157	+ $httpResponse['headers'][$headerName] .= ', '.trim($cookie);
158	158	} else {
159	159	$httpResponse['headers'][$headerName] = trim($cookie);
160	160	}
		@@ -183,7 +183,7 @@ discard block
		block discarded – undo
183	183	}
184	184	} elseif (isset($headerName)) {
185	185	/// @todo version1 cookies might span multiple lines, thus breaking the parsing above
186		- $httpResponse['headers'][$headerName] .= ' ' . trim($line);
	186	+ $httpResponse['headers'][$headerName] .= ' '.trim($line);
187	187	}
188	188	}
189	189
		@@ -206,7 +206,7 @@ discard block
		block discarded – undo
206	206	// Decode chunked encoding sent by http 1.1 servers
207	207	if (isset($httpResponse['headers']['transfer-encoding']) && $httpResponse['headers']['transfer-encoding'] == 'chunked') {
208	208	if (!$data = Http::decodeChunked($data)) {
209		- error_log('XML-RPC: ' . __METHOD__ . ': errors occurred when trying to rebuild the chunked data received from server');
	209	+ error_log('XML-RPC: '.__METHOD__.': errors occurred when trying to rebuild the chunked data received from server');
210	210	throw new \Exception(PhpXmlRpc::$xmlrpcstr['dechunk_fail'], PhpXmlRpc::$xmlrpcerr['dechunk_fail']);
211	211	}
212	212	}
		@@ -221,19 +221,19 @@ discard block
		block discarded – undo
221	221	if ($httpResponse['headers']['content-encoding'] == 'deflate' && $degzdata = @gzuncompress($data)) {
222	222	$data = $degzdata;
223	223	if ($debug) {
224		- Logger::instance()->debugMessage("---INFLATED RESPONSE---[" . strlen($data) . " chars]---\n$data\n---END---");
	224	+ Logger::instance()->debugMessage("---INFLATED RESPONSE---[".strlen($data)." chars]---\n$data\n---END---");
225	225	}
226	226	} elseif ($httpResponse['headers']['content-encoding'] == 'gzip' && $degzdata = @gzinflate(substr($data, 10))) {
227	227	$data = $degzdata;
228	228	if ($debug) {
229		- Logger::instance()->debugMessage("---INFLATED RESPONSE---[" . strlen($data) . " chars]---\n$data\n---END---");
	229	+ Logger::instance()->debugMessage("---INFLATED RESPONSE---[".strlen($data)." chars]---\n$data\n---END---");
230	230	}
231	231	} else {
232		- error_log('XML-RPC: ' . __METHOD__ . ': errors occurred when trying to decode the deflated data received from server');
	232	+ error_log('XML-RPC: '.__METHOD__.': errors occurred when trying to decode the deflated data received from server');
233	233	throw new \Exception(PhpXmlRpc::$xmlrpcstr['decompress_fail'], PhpXmlRpc::$xmlrpcerr['decompress_fail']);
234	234	}
235	235	} else {
236		- error_log('XML-RPC: ' . __METHOD__ . ': the server sent deflated data. Your php install must have the Zlib extension compiled in to support this.');
	236	+ error_log('XML-RPC: '.__METHOD__.': the server sent deflated data. Your php install must have the Zlib extension compiled in to support this.');
237	237	throw new \Exception(PhpXmlRpc::$xmlrpcstr['cannot_decompress'], PhpXmlRpc::$xmlrpcerr['cannot_decompress']);
238	238	}
239	239	}

		@@ -75,7 +75,7 @@ discard block
		block discarded – undo
75	75	return $xmlrpcVal->scalarval();
76	76	case 'array':
77	77	$arr = array();
78		- foreach($xmlrpcVal as $value) {
	78	+ foreach ($xmlrpcVal as $value) {
79	79	$arr[] = $this->decode($value, $options);
80	80	}
81	81
		@@ -105,7 +105,7 @@ discard block
		block discarded – undo
105	105	case 'msg':
106	106	$paramCount = $xmlrpcVal->getNumParams();
107	107	$arr = array();
108		- for ($i = 0; $i < $paramCount; $i++) {
	108	+ for ($i = 0; $i<$paramCount; $i++) {
109	109	$arr[] = $this->decode($xmlrpcVal->getParam($i), $options);
110	110	}
111	111
		@@ -183,7 +183,7 @@ discard block
		block discarded – undo
183	183	$xmlrpcVal = new Value($phpVal->format('Ymd\TH:i:s'), Value::$xmlrpcStruct);
184	184	} else {
185	185	$arr = array();
186		- foreach($phpVal as $k => $v) {
	186	+ foreach ($phpVal as $k => $v) {
187	187	$arr[$k] = $this->encode($v, $options);
188	188	}
189	189	$xmlrpcVal = new Value($arr, Value::$xmlrpcStruct);
		@@ -205,7 +205,7 @@ discard block
		block discarded – undo
205	205	break;
206	206	case 'resource':
207	207	if (in_array('extension_api', $options)) {
208		- $xmlrpcVal = new Value((int)$phpVal, Value::$xmlrpcInt);
	208	+ $xmlrpcVal = new Value((int) $phpVal, Value::$xmlrpcInt);
209	209	} else {
210	210	$xmlrpcVal = new Value();
211	211	}
		@@ -248,7 +248,7 @@ discard block
		block discarded – undo
248	248	if (extension_loaded('mbstring')) {
249	249	$xmlVal = mb_convert_encoding($xmlVal, 'UTF-8', $valEncoding);
250	250	} else {
251		- error_log('XML-RPC: ' . __METHOD__ . ': invalid charset encoding of xml text: ' . $valEncoding);
	251	+ error_log('XML-RPC: '.__METHOD__.': invalid charset encoding of xml text: '.$valEncoding);
252	252	}
253	253	}
254	254	}
		@@ -280,7 +280,7 @@ discard block
		block discarded – undo
280	280	return false;
281	281	}
282	282	xml_parser_free($parser);
283		- if ($xmlRpcParser->_xh['isf'] > 1) {
	283	+ if ($xmlRpcParser->_xh['isf']>1) {
284	284	// test that $xmlrpc->_xh['value'] is an obj, too???
285	285
286	286	error_log($xmlRpcParser->_xh['isf_reason']);
		@@ -301,7 +301,7 @@ discard block
		block discarded – undo
301	301	return $r;
302	302	case 'methodcall':
303	303	$req = new Request($xmlRpcParser->_xh['method']);
304		- for ($i = 0; $i < count($xmlRpcParser->_xh['params']); $i++) {
	304	+ for ($i = 0; $i<count($xmlRpcParser->_xh['params']); $i++) {
305	305	$req->addParam($xmlRpcParser->_xh['params'][$i]);
306	306	}
307	307

gggeek / phpxmlrpc

Push — master ( 509dee...ac735c )

Status

Category

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

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

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

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

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