phpcs clean-up

gggeek · Jul 3, 2015 · bb2bc96 · bb2bc96
1 parent 0c20707
commit bb2bc96
Show file tree

Hide file tree

Showing 37 changed files with 78 additions and 41 deletions.
diff --git a/lib/WebDriver/ClassLoader.php b/lib/WebDriver/ClassLoader.php
@@ -55,7 +55,7 @@ public static function autoload($class)
     {
         try {
             self::loadClass($class);
-        } catch (Exception $e) {
+        } catch (\Exception $e) {
         }
     }
 }

diff --git a/lib/WebDriver/Container.php b/lib/WebDriver/Container.php
@@ -71,7 +71,7 @@ public function element($using = null, $value = null)
                 sprintf(
                     "Element not found with %s, %s\n\n%s",
                     $locatorJson['using'],
-		    $locatorJson['value'],
+                    $locatorJson['value'],
                     $e->getMessage()
                 ),
                 $e
@@ -81,7 +81,8 @@ public function element($using = null, $value = null)
         $element = $this->webDriverElement($result['value']);
 
         if ($element === null) {
-            throw WebDriverException::factory(WebDriverException::NO_SUCH_ELEMENT,
+            throw WebDriverException::factory(
+                WebDriverException::NO_SUCH_ELEMENT,
                 sprintf(
                     "Element not found with %s, %s\n",
                     $locatorJson['using'],
@@ -119,9 +120,12 @@ public function elements($using = null, $value = null)
             return array();
         }
 
-        return array_filter(array_map(
-            array($this, 'webDriverElement'), $result['value']
-        ));
+        return array_filter(
+            array_map(
+                array($this, 'webDriverElement'),
+                $result['value']
+            )
+        );
     }
 
     /**
@@ -147,12 +151,14 @@ private function parseArgs($method, $argv)
 
             case 1:
                 $arg = $argv[0];
+
                 if (is_array($arg)) {
                     $using = $arg['using'];
                     $value = $arg['value'];
                     break;
                 }
 
+                // fall through
             default:
                 throw WebDriverException::factory(
                     WebDriverException::JSON_PARAMETERS_EXPECTED,

diff --git a/lib/WebDriver/Exception/CurlExec.php b/lib/WebDriver/Exception/CurlExec.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class CurlExec extends BaseException {
+final class CurlExec extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/ElementIsNotSelectable.php b/lib/WebDriver/Exception/ElementIsNotSelectable.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class ElementIsNotSelectable extends BaseException {
+final class ElementIsNotSelectable extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/ElementNotVisible.php b/lib/WebDriver/Exception/ElementNotVisible.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class ElementNotVisible extends BaseException {
+final class ElementNotVisible extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/IMEEngineActivationFailed.php b/lib/WebDriver/Exception/IMEEngineActivationFailed.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class IMEEngineActivationFailed extends BaseException {
+final class IMEEngineActivationFailed extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/IMENotAvailable.php b/lib/WebDriver/Exception/IMENotAvailable.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class IMENotAvailable extends BaseException {
+final class IMENotAvailable extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/InvalidCookieDomain.php b/lib/WebDriver/Exception/InvalidCookieDomain.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class InvalidCookieDomain extends BaseException {
+final class InvalidCookieDomain extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/InvalidElementCoordinates.php b/lib/WebDriver/Exception/InvalidElementCoordinates.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class InvalidElementCoordinates extends BaseException {
+final class InvalidElementCoordinates extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/InvalidElementState.php b/lib/WebDriver/Exception/InvalidElementState.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class InvalidElementState extends BaseException {
+final class InvalidElementState extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/InvalidRequest.php b/lib/WebDriver/Exception/InvalidRequest.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class InvalidRequest extends BaseException {
+final class InvalidRequest extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/InvalidSelector.php b/lib/WebDriver/Exception/InvalidSelector.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class InvalidSelector extends BaseException {
+final class InvalidSelector extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/JavaScriptError.php b/lib/WebDriver/Exception/JavaScriptError.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class JavaScriptError extends BaseException {
+final class JavaScriptError extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/JsonParameterExpected.php b/lib/WebDriver/Exception/JsonParameterExpected.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class JsonParameterExpected extends BaseException {
+final class JsonParameterExpected extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/MoveTargetOutOfBounds.php b/lib/WebDriver/Exception/MoveTargetOutOfBounds.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class MoveTargetOutOfBounds extends BaseException {
+final class MoveTargetOutOfBounds extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/NoAlertOpenError.php b/lib/WebDriver/Exception/NoAlertOpenError.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class NoAlertOpenError extends BaseException {
+final class NoAlertOpenError extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/NoParametersExpected.php b/lib/WebDriver/Exception/NoParametersExpected.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class NoParametersExpected extends BaseException {
+final class NoParametersExpected extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/NoSuchDriver.php b/lib/WebDriver/Exception/NoSuchDriver.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class NoSuchDriver extends BaseException {
+final class NoSuchDriver extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/NoSuchElement.php b/lib/WebDriver/Exception/NoSuchElement.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class NoSuchElement extends BaseException {
+final class NoSuchElement extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/NoSuchFrame.php b/lib/WebDriver/Exception/NoSuchFrame.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class NoSuchFrame extends BaseException {
+final class NoSuchFrame extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/NoSuchWindow.php b/lib/WebDriver/Exception/NoSuchWindow.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class NoSuchWindow extends BaseException {
+final class NoSuchWindow extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/ObsoleteCommand.php b/lib/WebDriver/Exception/ObsoleteCommand.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class ObsoleteCommand extends BaseException {
+final class ObsoleteCommand extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/ScriptTimeout.php b/lib/WebDriver/Exception/ScriptTimeout.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class ScriptTimeout extends BaseException {
+final class ScriptTimeout extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/SessionNotCreated.php b/lib/WebDriver/Exception/SessionNotCreated.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class SessionNotCreated extends BaseException {
+final class SessionNotCreated extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/StaleElementReference.php b/lib/WebDriver/Exception/StaleElementReference.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class StaleElementReference extends BaseException {
+final class StaleElementReference extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/Timeout.php b/lib/WebDriver/Exception/Timeout.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class Timeout extends BaseException {
+final class Timeout extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/UnableToSetCookie.php b/lib/WebDriver/Exception/UnableToSetCookie.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class UnableToSetCookie extends BaseException {
+final class UnableToSetCookie extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/UnexpectedAlertOpen.php b/lib/WebDriver/Exception/UnexpectedAlertOpen.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class UnexpectedAlertOpen extends BaseException {
+final class UnexpectedAlertOpen extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/UnexpectedParameters.php b/lib/WebDriver/Exception/UnexpectedParameters.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class UnexpectedParameters extends BaseException {
+final class UnexpectedParameters extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/UnknownCommand.php b/lib/WebDriver/Exception/UnknownCommand.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class UnknownCommand extends BaseException {
+final class UnknownCommand extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/UnknownError.php b/lib/WebDriver/Exception/UnknownError.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class UnknownError extends BaseException {
+final class UnknownError extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/UnknownLocatorStrategy.php b/lib/WebDriver/Exception/UnknownLocatorStrategy.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class UnknownLocatorStrategy extends BaseException {
+final class UnknownLocatorStrategy extends BaseException
+{
 }
diff --git a/lib/WebDriver/Exception/XPathLookupError.php b/lib/WebDriver/Exception/XPathLookupError.php
@@ -29,5 +29,6 @@
  *
  * @package WebDriver
  */
-final class XPathLookupError extends BaseException {
+final class XPathLookupError extends BaseException
+{
 }
diff --git a/lib/WebDriver/SauceLabs/SauceRest.php b/lib/WebDriver/SauceLabs/SauceRest.php
@@ -74,7 +74,7 @@ protected function execute($requestMethod, $url, $parameters = null)
             CURLOPT_SSL_VERIFYPEER => false,
             CURLOPT_SSL_VERIFYHOST => false,
 
-            CURLOPT_HTTPHEADER => Array ('Expect:'),
+            CURLOPT_HTTPHEADER => array('Expect:'),
         );
 
         $url = 'https://saucelabs.com/rest/v1/' . $url;

diff --git a/lib/WebDriver/ServiceFactory.php b/lib/WebDriver/ServiceFactory.php
@@ -64,7 +64,7 @@ private function __construct()
      *
      * @return \WebDriver\ServiceFactory
      */
-    static public function getInstance()
+    public static function getInstance()
     {
         if (!self::$instance) {
             self::$instance = new self;

diff --git a/lib/WebDriver/Session.php b/lib/WebDriver/Session.php
@@ -138,7 +138,7 @@ public function open($url)
      */
     public function capabilities()
     {
-        if ( ! isset($this->capabilities)) {
+        if (! isset($this->capabilities)) {
             $result = $this->curl('GET', '');
 
             $this->capabilities = $result['value'];

diff --git a/lib/WebDriver/WebDriver.php b/lib/WebDriver/WebDriver.php
@@ -65,7 +65,7 @@ public function session($requiredCapabilities = Browser::FIREFOX, $desiredCapabi
         );
 
         // optional
-        if ( ! empty($requiredCapabilities)) {
+        if (! empty($requiredCapabilities)) {
             $parameters['requiredCapabilities'] = $requiredCapabilities;
         }