phmLabs / Smoke

src/Rules/CheckResult.php

Doc Comments +3 added lines, -2 removed lines

@@ -29,6 +29,7 @@ 
      * @param $status
      * @param $value
      * @param $message
+     * @param string $url
      */
     public function __construct($status, $message = '', $value = null, $url = null)
     {
@@ -47,7 +48,7 @@ 
     }
 
     /**
-     * @param array $attributes
+     * @param array $attribute
      */
     public function addAttribute(Attribute $attribute)
     {
@@ -71,7 +72,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getMessage()
     {

src/Rules/Xml/Rss/ValidRule.php

Spacing +6 added lines, -6 removed lines

@@ -21,12 +21,12 @@ 
 
     private function getSchema()
     {
-        return __DIR__ . '/' . self::SCHEMA;
+        return __DIR__.'/'.self::SCHEMA;
     }
 
     public function doValidation(ResponseInterface $response)
     {
-        $body = (string)$response->getBody();
+        $body = (string) $response->getBody();
 
         if (preg_match('/<rss/', $body)) {
             libxml_clear_errors();
@@ -35,8 +35,8 @@ 
             $lastError = libxml_get_last_error();
             if ($lastError) {
                 throw new ValidationFailedException(
-                    'The given xml file is not well formed (last error: ' .
-                    str_replace("\n", '', $lastError->message) . ').');
+                    'The given xml file is not well formed (last error: '.
+                    str_replace("\n", '', $lastError->message).').');
             }
             $valid = @$dom->schemaValidate($this->getSchema());
             if (!$valid) {
@@ -44,11 +44,11 @@ 
                 $lastErrorMessage = str_replace("\n", '', $lastError->message);
 
                 if ($response instanceof UriAwareResponse) {
-                    $toolUrl = sprintf(self::PUBLIC_SERVICE, urlencode((string)$response->getUri()));
+                    $toolUrl = sprintf(self::PUBLIC_SERVICE, urlencode((string) $response->getUri()));
                 } else {
                     $toolUrl = '';
                 }
-                return new CheckResult(CheckResult::STATUS_FAILURE, 'The given xml file is not a valid rss file (last error: ' . $lastErrorMessage . ').', null, $toolUrl);
+                return new CheckResult(CheckResult::STATUS_FAILURE, 'The given xml file is not a valid rss file (last error: '.$lastErrorMessage.').', null, $toolUrl);
             }
         }
         return new CheckResult(CheckResult::STATUS_SUCCESS, 'The given rss file is valid.');

src/Cli/Command/CustomCommand.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
         $configArray = $this->getConfigArray($configFile, true);
 
         if (is_string($configArray)) {
-            throw new \RuntimeException('Unable to load config file. Please check ' . $configFile);
+            throw new \RuntimeException('Unable to load config file. Please check '.$configFile);
         }
 
         $this->config = new Configuration(new Uri('http://www.example.com'), $dispatcher, $configArray);

src/Extensions/SmokeReporter/Reporter/HtmlReporter.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Symfony\Component\Console\Output\OutputInterface;
 use whm\Smoke\Config\Configuration;
-use whm\Smoke\Extensions\SmokeResponseRetriever\Retriever\Retriever;
 use whm\Smoke\Rules\CheckResult;
 
 

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
         $this->resultFile = $resultFile;
 
         if ($templateDir === null) {
-            $this->templateDir = __DIR__ . '/templates';
+            $this->templateDir = __DIR__.'/templates';
         } else {
             $this->templateDir = $templateDir;
         }
@@ -66,9 +66,9 @@ 
         }
 
         if (!file_put_contents($this->resultFile, $html)) {
-            $this->output->writeln("<error>HTML Reporter extension: Could not write result file to " . $this->resultFile ."</error>");
+            $this->output->writeln("<error>HTML Reporter extension: Could not write result file to ".$this->resultFile."</error>");
         } else {
-            $this->output->writeln("<info>HTML Reporter extension:</info> Result file written to " . $this->resultFile);
+            $this->output->writeln("<info>HTML Reporter extension:</info> Result file written to ".$this->resultFile);
         }
     }