| @@ -53,7 +53,9 @@ | ||
| 53 | 53 | $max = false; | 
| 54 | 54 | $min = PHP_INT_MAX; | 
| 55 | 55 |          foreach ($matchedParts as $part) { | 
| 56 | - if ($part === null) continue; | |
| 56 | +            if ($part === null) { | |
| 57 | + continue; | |
| 58 | + } | |
| 57 | 59 |              if ($part >= $max) { | 
| 58 | 60 | $max = $part; | 
| 59 | 61 | } | 
| @@ -59,7 +59,7 @@ | ||
| 59 | 59 | $this->street1 = trim(array_shift($rows)); | 
| 60 | 60 | $this->business = trim(array_shift($rows)); | 
| 61 | 61 | //Easy | 
| 62 | -        }else if (count($rows) == 1) { | |
| 62 | +        } else if (count($rows) == 1) { | |
| 63 | 63 | $this->street1 = trim(array_shift($rows)); | 
| 64 | 64 | |
| 65 | 65 | // Not so easy | 
| @@ -3,8 +3,6 @@ | ||
| 3 | 3 | namespace Magium\Assertions\Element; | 
| 4 | 4 | |
| 5 | 5 | use Facebook\WebDriver\WebDriverBy; | 
| 6 | -use Magium\AbstractTestCase; | |
| 7 | -use Magium\InvalidTestTypeException; | |
| 8 | 6 | use Magium\WebDriver\ExpectedCondition; | 
| 9 | 7 | |
| 10 | 8 | class Clickable extends AbstractSelectorAssertion | 
| @@ -3,8 +3,6 @@ | ||
| 3 | 3 | namespace Magium\Assertions\Element; | 
| 4 | 4 | |
| 5 | 5 | use Facebook\WebDriver\WebDriverBy; | 
| 6 | -use Magium\AbstractTestCase; | |
| 7 | -use Magium\InvalidTestTypeException; | |
| 8 | 6 | use Magium\WebDriver\ExpectedCondition; | 
| 9 | 7 | |
| 10 | 8 | class NotClickable extends AbstractSelectorAssertion | 
| @@ -5,13 +5,9 @@ | ||
| 5 | 5 | use Guzzle\Http\Client; | 
| 6 | 6 | use Magium\Cli\Command\Test\TestSkeleton; | 
| 7 | 7 | use Magium\InvalidConfigurationException; | 
| 8 | -use Magium\NotFoundException; | |
| 9 | 8 | use Magium\Util\Api\ApiConfiguration; | 
| 10 | 9 | use Symfony\Component\Console\Command\Command; | 
| 11 | -use Symfony\Component\Console\Input\ArrayInput; | |
| 12 | -use Symfony\Component\Console\Input\InputArgument; | |
| 13 | 10 | use Symfony\Component\Console\Input\InputInterface; | 
| 14 | -use Symfony\Component\Console\Input\InputOption; | |
| 15 | 11 | use Symfony\Component\Console\Output\OutputInterface; | 
| 16 | 12 | |
| 17 | 13 | class ApiPing extends Command | 
| @@ -141,7 +141,9 @@ | ||
| 141 | 141 |      { | 
| 142 | 142 | $namespace = $class = ''; | 
| 143 | 143 | $content = file_get_contents($file); | 
| 144 | - if (!$content) return null; | |
| 144 | +        if (!$content) { | |
| 145 | + return null; | |
| 146 | + } | |
| 145 | 147 | $tokens = token_get_all($content); | 
| 146 | 148 | $count = count($tokens); | 
| 147 | 149 |          for ($i = 2; $i < $count; $i++) { | 
| @@ -49,10 +49,14 @@ | ||
| 49 | 49 | |
| 50 | 50 |          if ($operator === null) { | 
| 51 | 51 | $isTrue = in_array($comparison1, $this->reservedTrue); | 
| 52 | - if ($isTrue) return true; | |
| 52 | +            if ($isTrue) { | |
| 53 | + return true; | |
| 54 | + } | |
| 53 | 55 | |
| 54 | 56 | $isFalse = in_array($comparison1, $this->reservedFalse); | 
| 55 | - if ($isFalse) return false; | |
| 57 | +            if ($isFalse) { | |
| 58 | + return false; | |
| 59 | + } | |
| 56 | 60 | |
| 57 | 61 | return (boolean)$comparison1; | 
| 58 | 62 |          } else { | 
| @@ -5,7 +5,6 @@ | ||
| 5 | 5 | use Facebook\WebDriver\Exception\WebDriverException; | 
| 6 | 6 | use Facebook\WebDriver\Remote\DriverCommand; | 
| 7 | 7 | use Facebook\WebDriver\Remote\HttpCommandExecutor; | 
| 8 | -use Facebook\WebDriver\Remote\RemoteExecuteMethod; | |
| 9 | 8 | use Facebook\WebDriver\Remote\RemoteWebDriver; | 
| 10 | 9 | use Facebook\WebDriver\Remote\WebDriverCommand; | 
| 11 | 10 | use Facebook\WebDriver\WebDriverBy; | 
| @@ -71,7 +71,9 @@ | ||
| 71 | 71 | } | 
| 72 | 72 | } | 
| 73 | 73 | |
| 74 | - if (!is_dir($configurationDir)) return; | |
| 74 | +        if (!is_dir($configurationDir)) { | |
| 75 | + return; | |
| 76 | + } | |
| 75 | 77 | |
| 76 | 78 | $classes = $this->introspectClass($config); | 
| 77 | 79 | |