Completed
Pull Request — master (#94)
by
unknown
05:56
created
src/Rules/Xml/Sitemap/ValidRule.php 2 patches
Doc Comments   +7 added lines patch added patch discarded remove patch
@@ -16,11 +16,18 @@
 block discarded – undo
16 16
 
17 17
     protected $contentTypes = array('text/xml', 'application/xml');
18 18
 
19
+    /**
20
+     * @param boolean $isIndex
21
+     */
19 22
     private function getSchema($isIndex)
20 23
     {
21 24
         return ($isIndex) ?  __DIR__ . '/' . self::INDEX : __DIR__ . '/' . self::SCHEMA;
22 25
     }
23 26
 
27
+    /**
28
+     * @param \Psr\Http\Message\StreamInterface $body
29
+     * @param string $filename
30
+     */
24 31
     private function validateBody($body, $filename, $isIndex = TRUE)
25 32
     {
26 33
         $dom = new \DOMDocument();
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -29,13 +29,13 @@  discard block
 block discarded – undo
29 29
     private function getSchema($isIndex)
30 30
     {
31 31
         if ($isIndex) {
32
-            return __DIR__ . '/' . self::INDEX;
32
+            return __DIR__.'/'.self::INDEX;
33 33
         }
34 34
 
35 35
         if ($this->strictMode) {
36
-            return __DIR__ . '/' . self::SCHEMA;
36
+            return __DIR__.'/'.self::SCHEMA;
37 37
         } else {
38
-            return __DIR__ . '/' . self::NON_STRICT_SCHEMA;
38
+            return __DIR__.'/'.self::NON_STRICT_SCHEMA;
39 39
         }
40 40
     }
41 41
 
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
         if (!$valid) {
50 50
             $lastError = libxml_get_last_error();
51 51
             throw new ValidationFailedException(
52
-                'The given sitemap file (' . $filename . ') did not validate against the sitemap schema (last error: ' . str_replace("\n", '', $lastError->message) . ').');
52
+                'The given sitemap file ('.$filename.') did not validate against the sitemap schema (last error: '.str_replace("\n", '', $lastError->message).').');
53 53
         }
54 54
     }
55 55
 
Please login to merge, or discard this patch.
src/Rules/Xml/XmlCheckRule.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,8 +21,8 @@
 block discarded – undo
21 21
         if (!$success) {
22 22
             $lastError = libxml_get_last_error();
23 23
 
24
-            throw new ValidationFailedException('The xml file ' . $response->getUri() . ' is not well formed (last error: ' .
25
-                str_replace("\n", '', $lastError->message) . ').');
24
+            throw new ValidationFailedException('The xml file '.$response->getUri().' is not well formed (last error: '.
25
+                str_replace("\n", '', $lastError->message).').');
26 26
         }
27 27
     }
28 28
 }
Please login to merge, or discard this patch.
src/Rules/Json/JsonPathExistsRule.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -65,17 +65,17 @@
 block discarded – undo
65 65
 
66 66
             if ($jsonValue === FALSE || (is_array($jsonValue) && empty($jsonValue))) {
67 67
                 $error = true;
68
-                $noCorrectJsonPaths[] = $path['pattern'] . ' (JSON Path not found)';
68
+                $noCorrectJsonPaths[] = $path['pattern'].' (JSON Path not found)';
69 69
             }
70 70
             if ($this->checkRelation($path['relation'], $path['value'], $count) === false) {
71 71
                 $error = true;
72
-                $noCorrectJsonPaths[] = $path['pattern'] . ' (number of JSONPaths is not correct corresponding to the given relation/value)';
72
+                $noCorrectJsonPaths[] = $path['pattern'].' (number of JSONPaths is not correct corresponding to the given relation/value)';
73 73
             }
74 74
         }
75 75
 
76 76
         if ($error === true) {
77 77
             $allNoCorrectJsonPaths = implode('", "', $noCorrectJsonPaths);
78
-            throw new ValidationFailedException('Disonances with JSON Paths "' . $allNoCorrectJsonPaths . '!');
78
+            throw new ValidationFailedException('Disonances with JSON Paths "'.$allNoCorrectJsonPaths.'!');
79 79
         }
80 80
     }
81 81
 }
Please login to merge, or discard this patch.
src/Rules/Html/CssSelectorExistsRule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
         if ($error === true) {
47 47
             $allNotFoundSelectors = implode('", "', $snotFoundSelectors);
48 48
 
49
-            throw new ValidationFailedException('CSS Selector "' . $allNotFoundSelectors . '" not found in DOM.');
49
+            throw new ValidationFailedException('CSS Selector "'.$allNotFoundSelectors.'" not found in DOM.');
50 50
         }
51 51
     }
52 52
 }
Please login to merge, or discard this patch.
src/Rules/Http/Header/GZipRule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
     {
24 24
         if (strpos($response->getContentType(), 'image') === false
25 25
             && strpos($response->getContentType(), 'pdf') === false
26
-            && strlen((string)$response->getBody()) >= $this->minFileSize
26
+            && strlen((string) $response->getBody()) >= $this->minFileSize
27 27
         ) {
28 28
             if (!$response->hasHeader('Content-Encoding') || $response->getHeader('Content-Encoding')[0] !== 'gzip') {
29 29
                 throw new ValidationFailedException('gzip compression not active');
Please login to merge, or discard this patch.
src/Rules/Json/JsonSchema/JsonSchemaRule.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -38,17 +38,17 @@  discard block
 block discarded – undo
38 38
     {
39 39
         $data = json_decode($response->getBody());
40 40
         if ($data === null) {
41
-            throw new ValidationFailedException("The given JSON data can not be validated (last error: '" . $this->json_errors[json_last_error()] . "').");
41
+            throw new ValidationFailedException("The given JSON data can not be validated (last error: '".$this->json_errors[json_last_error()]."').");
42 42
         }
43 43
         else {
44 44
             $error = false;
45 45
             $messageParts = array();
46 46
 
47 47
             foreach ($this->jsonSchemaFiles AS $jsonSchemaFile) {
48
-                $factory = new Factory( null, null, Constraint::CHECK_MODE_TYPE_CAST | Constraint::CHECK_MODE_COERCE );
48
+                $factory = new Factory(null, null, Constraint::CHECK_MODE_TYPE_CAST | Constraint::CHECK_MODE_COERCE);
49 49
                 $validator = new Validator($factory);
50 50
 
51
-                $jsonSchemaObject = (object)json_decode(file_get_contents($jsonSchemaFile['jsonschemafileurl']));
51
+                $jsonSchemaObject = (object) json_decode(file_get_contents($jsonSchemaFile['jsonschemafileurl']));
52 52
                 
53 53
                 $validator->check($data, $jsonSchemaObject);
54 54
 
@@ -56,14 +56,14 @@  discard block
 block discarded – undo
56 56
                     $error = true;
57 57
                     $errorMessage = '';
58 58
                     foreach ($validator->getErrors() as $error) {
59
-                        $errorMessage = $errorMessage .  sprintf("[%s] %s\n", $error['property'], $error['message']);
59
+                        $errorMessage = $errorMessage.sprintf("[%s] %s\n", $error['property'], $error['message']);
60 60
                     }
61
-                    $messageParts[] = $jsonSchemaFile['jsonschemafilename'] . ' - ' . $jsonSchemaFile['jsonschemafileurl'] . '(last error: ' . $errorMessage . ').';
61
+                    $messageParts[] = $jsonSchemaFile['jsonschemafilename'].' - '.$jsonSchemaFile['jsonschemafileurl'].'(last error: '.$errorMessage.').';
62 62
                 }
63 63
             }
64 64
 
65 65
             if ($error == true) {
66
-                $message = 'JSON file (' . (string)$response->getUri() . ')  does not validate against the following JSON Schema files: ' . implode(", ", $messageParts);
66
+                $message = 'JSON file ('.(string) $response->getUri().')  does not validate against the following JSON Schema files: '.implode(", ", $messageParts);
67 67
                 throw new ValidationFailedException($message);
68 68
             }
69 69
         }
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,8 +39,7 @@
 block discarded – undo
39 39
         $data = json_decode($response->getBody());
40 40
         if ($data === null) {
41 41
             throw new ValidationFailedException("The given JSON data can not be validated (last error: '" . $this->json_errors[json_last_error()] . "').");
42
-        }
43
-        else {
42
+        } else {
44 43
             $error = false;
45 44
             $messageParts = array();
46 45
 
Please login to merge, or discard this patch.
src/Rules/Xml/XmlValidXsdRule.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
         $dom = new \DOMDocument();
28 28
         @$dom->loadXML($body);
29 29
 
30
-        $filename = (string)$response->getUri();
30
+        $filename = (string) $response->getUri();
31 31
 
32 32
         $error = false;
33 33
         $messageParts = array();
@@ -39,12 +39,12 @@  discard block
 block discarded – undo
39 39
                 $error = true;
40 40
                 $lastError = libxml_get_last_error();
41 41
 
42
-                $messageParts[] = $xsdFile['xsdfilename'] . ' - ' . $xsdFile['xsdfileurl'] . '(last error: ' . str_replace("\n", '', $lastError->message) . ').';
42
+                $messageParts[] = $xsdFile['xsdfilename'].' - '.$xsdFile['xsdfileurl'].'(last error: '.str_replace("\n", '', $lastError->message).').';
43 43
             }
44 44
         }
45 45
 
46 46
         if ($error === true) {
47
-            $message = 'XML file (' . $filename . ')  does not validate against the following XSD files: ' . implode(", ", $messageParts);
47
+            $message = 'XML file ('.$filename.')  does not validate against the following XSD files: '.implode(", ", $messageParts);
48 48
             throw new ValidationFailedException($message);
49 49
         }
50 50
     }
Please login to merge, or discard this patch.