gggeek / phpxmlrpc

pakefile.php

Braces +7 added lines, -6 removed lines

@@ -27,8 +27,9 @@ 
 
     public static function libVersion()
     {
-        if (self::$libVersion == null)
-            throw new \Exception('Missing library version argument');
+        if (self::$libVersion == null) {
+                    throw new \Exception('Missing library version argument');
+        }
         return self::$libVersion;
     }
 
@@ -59,9 +60,10 @@ 
 
     public static function getOpts($args=array(), $cliOpts=array())
     {
-        if (count($args) > 0)
-        //    throw new \Exception('Missing library version argument');
+        if (count($args) > 0) {
+                //    throw new \Exception('Missing library version argument');
             self::$libVersion = $args[0];
+        }
 
         foreach (self::$tools as $name => $binary) {
             if (isset($cliOpts[$name])) {
@@ -338,8 +340,7 @@ 
             $cmd = Builder::tool('zip');
             $extra = '-9 -r';
             pake_sh("$cmd $distFile $extra ".basename(Builder::distDir()));
-        }
-        else {
+        } else {
             $finder = pakeFinder::type('any')->pattern(basename(Builder::distDir()).'/**');
             // see https://bugs.php.net/bug.php?id=58852
             $pharFile = str_replace(Builder::libVersion(), '_LIBVERSION_', $distFile);

debugger/common.php

Indentation +9 added lines, -9 removed lines

@@ -1,14 +1,14 @@
 <?php
 /**
- * @author Gaetano Giunta
- * @copyright (C) 2005-2015 G. Giunta
- * @license code licensed under the BSD License: see file license.txt
- *
- * Parses GET/POST variables
- *
- * @todo switch params for http compression from 0,1,2 to values to be used directly
- * @todo do some more sanitization of received parameters
- */
+     * @author Gaetano Giunta
+     * @copyright (C) 2005-2015 G. Giunta
+     * @license code licensed under the BSD License: see file license.txt
+     *
+     * Parses GET/POST variables
+     *
+     * @todo switch params for http compression from 0,1,2 to values to be used directly
+     * @todo do some more sanitization of received parameters
+     */
 
 // work around magic quotes
 if (get_magic_quotes_gpc()) {

debugger/action.php

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * @author Gaetano Giunta
- * @copyright (C) 2005-2015 G. Giunta
- * @license code licensed under the BSD License: see file license.txt
- *
- * @todo switch params for http compression from 0,1,2 to values to be used directly
- * @todo use ob_start to catch debug info and echo it AFTER method call results?
- * @todo be smarter in creating client stub for proxy/auth cases: only set appropriate property of client obj
- **/
+     * @author Gaetano Giunta
+     * @copyright (C) 2005-2015 G. Giunta
+     * @license code licensed under the BSD License: see file license.txt
+     *
+     * @todo switch params for http compression from 0,1,2 to values to be used directly
+     * @todo use ob_start to catch debug info and echo it AFTER method call results?
+     * @todo be smarter in creating client stub for proxy/auth cases: only set appropriate property of client obj
+     **/
 
 header('Content-Type: text/html; charset=utf-8');
 

Braces +3 added lines, -1 removed lines

@@ -367,7 +367,9 @@
                                 echo "<code>OUT:&nbsp;" . htmlspecialchars($ret->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding) . "<br />IN: (";
                                 if ($x->count() > 1) {
                                     foreach($x as $k => $y) {
-                                        if ($k == 0) continue;
+                                        if ($k == 0) {
+                                            continue;
+                                        }
                                         echo htmlspecialchars($y->scalarval(), ENT_COMPAT, \PhpXmlRpc\PhpXmlRpc::$xmlrpc_internalencoding);
                                         if ($wstype != 1) {
                                             $type = $y->scalarval();

tests/4LocalhostMultiTest.php

Braces +7 added lines, -8 removed lines

@@ -24,8 +24,9 @@ 
         {
             if(strpos($method, 'test') === 0 && !in_array($method, $unsafeMethods))
             {
-                if (!isset(self::$failed_tests[$method]))
-                    $this->$method();
+                if (!isset(self::$failed_tests[$method])) {
+                                    $this->$method();
+                }
             }
             /*if ($this->_failed)
             {
@@ -76,9 +77,9 @@ 
         {
             $this->client->setProxy($this->args['PROXYSERVER'], $this->args['PROXYPORT']);
             $this->_runtests();
+        } else {
+                    $this->markTestSkipped('PROXY definition missing: cannot test proxy');
         }
-        else
-            $this->markTestSkipped('PROXY definition missing: cannot test proxy');
     }
 
     function testHttp11()
@@ -132,8 +133,7 @@ 
         {
             $this->markTestSkipped('CURL missing: cannot test http 1.1 w. proxy');
             return;
-        }
-        else if ($this->args['PROXYSERVER'] == '')
+        } else if ($this->args['PROXYSERVER'] == '')
         {
             $this->markTestSkipped('PROXY definition missing: cannot test proxy w. http 1.1');
             return;
@@ -170,8 +170,7 @@ 
         {
             $this->markTestSkipped('CURL missing: cannot test https functionality');
             return;
-        }
-        else if ($this->args['PROXYSERVER'] == '')
+        } else if ($this->args['PROXYSERVER'] == '')
         {
             $this->markTestSkipped('PROXY definition missing: cannot test proxy w. http 1.1');
             return;

tests/benchmark.php

Indentation +9 added lines, -9 removed lines

@@ -1,14 +1,14 @@
 <?php
 /**
- * Benchmarking suite for the PHP-XMLRPC lib.
- *
- * @author Gaetano Giunta
- * @copyright (c) 2005-2015 G. Giunta
- * @license code licensed under the BSD License: see file license.txt
- *
- * @todo add a test for response ok in call testing
- * @todo add support for --help option to give users the list of supported parameters
- **/
+     * Benchmarking suite for the PHP-XMLRPC lib.
+     *
+     * @author Gaetano Giunta
+     * @copyright (c) 2005-2015 G. Giunta
+     * @license code licensed under the BSD License: see file license.txt
+     *
+     * @todo add a test for response ok in call testing
+     * @todo add support for --help option to give users the list of supported parameters
+     **/
 
 use PhpXmlRpc\PhpXmlRpc;
 use PhpXmlRpc\Value;

tests/phpunit_coverage.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Used to serve back the server-side code coverage results to phpunit-selenium
- *
- * @copyright (C) 2007-2015 G. Giunta
- * @license code licensed under the BSD License: see file license.txt
- **/
+     * Used to serve back the server-side code coverage results to phpunit-selenium
+     *
+     * @copyright (C) 2007-2015 G. Giunta
+     * @license code licensed under the BSD License: see file license.txt
+     **/
 
 $coverageFile = realpath(__DIR__ . "/../vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/SeleniumCommon/phpunit_coverage.php");
 

tests/3LocalhostTest.php

Braces +6 added lines, -4 removed lines

@@ -96,14 +96,16 @@
 
         $this->coverageScriptUrl = 'http://' . $this->args['LOCALSERVER'] . '/' . str_replace( '/demo/server/server.php', 'tests/phpunit_coverage.php', $this->args['URI'] );
 
-        if ($this->args['DEBUG'] == 1)
-            ob_start();
+        if ($this->args['DEBUG'] == 1) {
+                    ob_start();
+        }
     }
 
     protected function tearDown()
     {
-        if ($this->args['DEBUG'] != 1)
-            return;
+        if ($this->args['DEBUG'] != 1) {
+                    return;
+        }
         $out = ob_get_clean();
         $status = $this->getStatus();
         if ($status == PHPUnit_Runner_BaseTestRunner::STATUS_ERROR

tests/2InvalidHostTest.php

Braces +6 added lines, -4 removed lines

@@ -20,14 +20,16 @@
         $this->client = new xmlrpc_client('/NOTEXIST.php', $this->args['LOCALSERVER'], 80);
         $this->client->setDebug($this->args['DEBUG']);
 
-        if ($this->args['DEBUG'] == 1)
-            ob_start();
+        if ($this->args['DEBUG'] == 1) {
+                    ob_start();
+        }
     }
 
     protected function tearDown()
     {
-        if ($this->args['DEBUG'] != 1)
-            return;
+        if ($this->args['DEBUG'] != 1) {
+                    return;
+        }
         $out = ob_get_clean();
         $status = $this->getStatus();
         if ($status == PHPUnit_Runner_BaseTestRunner::STATUS_ERROR

tests/1ParsingBugsTest.php

Indentation +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 /**
- * NB: do not let your IDE fool you. The correct encoding for this file is NOT UTF8.
- */
+     * NB: do not let your IDE fool you. The correct encoding for this file is NOT UTF8.
+     */
 include_once __DIR__ . '/../lib/xmlrpc.inc';
 include_once __DIR__ . '/../lib/xmlrpcs.inc';
 

Braces +6 added lines, -4 removed lines

@@ -17,14 +17,16 @@
     protected function setUp()
     {
         $this->args = argParser::getArgs();
-        if ($this->args['DEBUG'] == 1)
-            ob_start();
+        if ($this->args['DEBUG'] == 1) {
+                    ob_start();
+        }
     }
 
     protected function tearDown()
     {
-        if ($this->args['DEBUG'] != 1)
-            return;
+        if ($this->args['DEBUG'] != 1) {
+                    return;
+        }
         $out = ob_get_clean();
         $status = $this->getStatus();
         if ($status == PHPUnit_Runner_BaseTestRunner::STATUS_ERROR