Passed
Push — master ( 2d92e4...0d56ed )
by Darío
01:37
created
src/Validator/FormValidator.php 1 patch
Braces   +34 added lines, -26 removed lines patch added patch discarded remove patch
@@ -103,8 +103,9 @@  discard block
 block discarded – undo
103 103
      */
104 104
     public function getOption($key, $name)
105 105
     {
106
-        if (!array_key_exists($key, $this->options))
107
-            throw new \LogicException("The option '$key' does not exists");
106
+        if (!array_key_exists($key, $this->options)) {
107
+                    throw new \LogicException("The option '$key' does not exists");
108
+        }
108 109
 
109 110
         return array_key_exists($name, $this->options[$key]) ? $this->options[$key][$name] : null;
110 111
     }
@@ -158,8 +159,9 @@  discard block
 block discarded – undo
158 159
 
159 160
         foreach ($attribs as $key => $attributes)
160 161
         {
161
-            if (!array_key_exists($key, $attribs))
162
-                throw new \LogicException("The field '$key' does not exists!");
162
+            if (!array_key_exists($key, $attribs)) {
163
+                            throw new \LogicException("The field '$key' does not exists!");
164
+            }
163 165
 
164 166
             $label = (array_key_exists('label', array_keys($this->options))) ? $attributes["label"] : $key;
165 167
 
@@ -225,19 +227,21 @@  discard block
 block discarded – undo
225 227
 
226 228
                     case 'min':
227 229
 
228
-                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range']))
229
-                            $validator = new GreaterThan(['min' => $value, 'inclusive' => true]);
230
-                        else
231
-                            throw new \LogicException("The input type must be 'range' or 'number'");
230
+                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range'])) {
231
+                                                    $validator = new GreaterThan(['min' => $value, 'inclusive' => true]);
232
+                        } else {
233
+                                                    throw new \LogicException("The input type must be 'range' or 'number'");
234
+                        }
232 235
 
233 236
                         break;
234 237
 
235 238
                     case 'max':
236 239
 
237
-                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range']))
238
-                            $validator = new LessThan(['max' => $value, 'inclusive' => true]);
239
-                        else
240
-                            throw new \LogicException("The input type must be 'range' or 'number'");
240
+                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range'])) {
241
+                                                    $validator = new LessThan(['max' => $value, 'inclusive' => true]);
242
+                        } else {
243
+                                                    throw new \LogicException("The input type must be 'range' or 'number'");
244
+                        }
241 245
 
242 246
                         break;
243 247
 
@@ -245,10 +249,11 @@  discard block
 block discarded – undo
245 249
 
246 250
                         $baseValue = (array_key_exists('min', $all_attribs)) ? $all_attribs['min'] : 0;
247 251
 
248
-                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['range']))
249
-                            $validator = new Step(['baseValue' => $baseValue, 'step' => $value]);
250
-                        else
251
-                            throw new \LogicException("The input type must be 'range'");
252
+                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['range'])) {
253
+                                                    $validator = new Step(['baseValue' => $baseValue, 'step' => $value]);
254
+                        } else {
255
+                                                    throw new \LogicException("The input type must be 'range'");
256
+                        }
252 257
 
253 258
                         break;
254 259
                 }
@@ -272,8 +277,9 @@  discard block
 block discarded – undo
272 277
                 $validator = new NotEmpty();
273 278
                 $attribute = $this->formHandler->getAttribute($key, "value");
274 279
 
275
-                if (is_null($attribute))
276
-                    throw new \LogicException("Missing attribute 'value' in '" . $key . "'.");
280
+                if (is_null($attribute)) {
281
+                                    throw new \LogicException("Missing attribute 'value' in '" . $key . "'.");
282
+                }
277 283
 
278 284
                 $value = $attribute->getValue();
279 285
 
@@ -287,8 +293,9 @@  discard block
 block discarded – undo
287 293
                         {
288 294
                             $className = "\Zend\I18n\Validator\\" . $class;
289 295
 
290
-                            if (!class_exists($className))
291
-                                throw new \RuntimeException("The class '$userInputClass' or '$className' does not exists");
296
+                            if (!class_exists($className)) {
297
+                                                            throw new \RuntimeException("The class '$userInputClass' or '$className' does not exists");
298
+                            }
292 299
                         }
293 300
 
294 301
                         $validator = new $className($params);
@@ -326,21 +333,22 @@  discard block
 block discarded – undo
326 333
             $v->setTranslator($this->translator);
327 334
             $notEmpty = $v->isValid($val);
328 335
 
329
-            if (!$required && !$notEmpty)
330
-                return null;
336
+            if (!$required && !$notEmpty) {
337
+                            return null;
338
+            }
331 339
 
332 340
             $valid = $validator->isValid($val);
333 341
             $this->setValid($valid);
334 342
 
335 343
             if (!$valid)
336 344
             {
337
-                if (!in_array($key, array_keys($this->messages)))
338
-                    $this->messages[$key] = array();
345
+                if (!in_array($key, array_keys($this->messages))) {
346
+                                    $this->messages[$key] = array();
347
+                }
339 348
 
340 349
                 $this->messages[$key] = array_merge($this->messages[$key], $validator->getMessages());
341 350
             }
342
-        }
343
-        else {
351
+        } else {
344 352
             foreach ($form_value as $val)
345 353
             {
346 354
                 $this->_validate($validator, $val, $key, $required);
Please login to merge, or discard this patch.
src/Dom/Element/AbstractElement.php 1 patch
Braces   +12 added lines, -10 removed lines patch added patch discarded remove patch
@@ -108,8 +108,9 @@  discard block
 block discarded – undo
108 108
 
109 109
         foreach ($attribs[$label] as $attrib)
110 110
         {
111
-            if ($attrib->getName() == $name)
112
-                return $attrib;
111
+            if ($attrib->getName() == $name) {
112
+                            return $attrib;
113
+            }
113 114
         }
114 115
     }
115 116
 
@@ -129,12 +130,12 @@  discard block
 block discarded – undo
129 130
         if (array_key_exists($label, $attribs))
130 131
         {
131 132
 
132
-            if (!array_key_exists($name, $attribs))
133
-                $this->attributes[$label][] = new Attribute($name, $value);
134
-            else
135
-                $this->getAttribute($label, $name)->setValue($value);
136
-        }
137
-        else {
133
+            if (!array_key_exists($name, $attribs)) {
134
+                            $this->attributes[$label][] = new Attribute($name, $value);
135
+            } else {
136
+                            $this->getAttribute($label, $name)->setValue($value);
137
+            }
138
+        } else {
138 139
             $this->attributes[$label][] = new Attribute($name, $value);
139 140
         }
140 141
     }
@@ -166,8 +167,9 @@  discard block
 block discarded – undo
166 167
      */
167 168
     public function __construct($options)
168 169
     {
169
-        if (!array_key_exists('endTag', $options))
170
-            $this->setEndTag(false);
170
+        if (!array_key_exists('endTag', $options)) {
171
+                    $this->setEndTag(false);
172
+        }
171 173
 
172 174
         foreach ($options as $key => $value)
173 175
         {
Please login to merge, or discard this patch.
src/Network/Rest/Basic.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -29,8 +29,8 @@  discard block
 block discarded – undo
29 29
             $ht = $this->http;
30 30
 
31 31
             $this->http->writeStatus($ht::HTTP_UNAUTHORIZED);
32
-            header('WWW-Authenticate: Basic realm="'.$this->realm.'"');
33
-            die('Error ' . $ht::HTTP_UNAUTHORIZED .' (' . $this->http->getStatusText($ht::HTTP_UNAUTHORIZED) . ')!!');
32
+            header('WWW-Authenticate: Basic realm="' . $this->realm . '"');
33
+            die('Error ' . $ht::HTTP_UNAUTHORIZED . ' (' . $this->http->getStatusText($ht::HTTP_UNAUTHORIZED) . ')!!');
34 34
         }
35 35
     }
36 36
 
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
     public function response()
77 77
     {
78 78
         $status = http_response_code();
79
-        $this->response = 'Error ' . $status .' (' . $this->http->getStatusText($status) . ')!!';
79
+        $this->response = 'Error ' . $status . ' (' . $this->http->getStatusText($status) . ')!!';
80 80
         echo $this->response;
81 81
     }
82 82
 }
83 83
\ No newline at end of file
Please login to merge, or discard this patch.
src/Network/Rest/Digest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -29,8 +29,8 @@  discard block
 block discarded – undo
29 29
             $ht = $this->http;
30 30
 
31 31
             $this->http->writeStatus($ht::HTTP_UNAUTHORIZED);
32
-            header('WWW-Authenticate: Digest realm="'.$this->realm.'",qop="auth",nonce="'.uniqid().'",opaque="'.md5($this->realm).'"');
33
-            die('Error ' . $ht::HTTP_UNAUTHORIZED .' (' . $this->http->getStatusText($ht::HTTP_UNAUTHORIZED) . ')!!');
32
+            header('WWW-Authenticate: Digest realm="' . $this->realm . '",qop="auth",nonce="' . uniqid() . '",opaque="' . md5($this->realm) . '"');
33
+            die('Error ' . $ht::HTTP_UNAUTHORIZED . ' (' . $this->http->getStatusText($ht::HTTP_UNAUTHORIZED) . ')!!');
34 34
         }
35 35
     }
36 36
 
@@ -50,8 +50,8 @@  discard block
 block discarded – undo
50 50
         }
51 51
 
52 52
         $A1 = md5($data['username'] . ':' . $this->realm . ':' . $this->whiteList[$data['username']]);
53
-        $A2 = md5($_SERVER['REQUEST_METHOD'].':'.$data['uri']);
54
-        $valid_response = md5($A1.':'.$data['nonce'].':'.$data['nc'].':'.$data['cnonce'].':'.$data['qop'].':'.$A2);
53
+        $A2 = md5($_SERVER['REQUEST_METHOD'] . ':' . $data['uri']);
54
+        $valid_response = md5($A1 . ':' . $data['nonce'] . ':' . $data['nc'] . ':' . $data['cnonce'] . ':' . $data['qop'] . ':' . $A2);
55 55
 
56 56
         if ($data['response'] != $valid_response)
57 57
         {
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     public function response()
98 98
     {
99 99
         $status = http_response_code();
100
-        $this->response = 'Error ' . $status .' (' . $this->http->getStatusText($status) . ')!!';
100
+        $this->response = 'Error ' . $status . ' (' . $this->http->getStatusText($status) . ')!!';
101 101
         echo $this->response;
102 102
     }
103 103
 }
104 104
\ No newline at end of file
Please login to merge, or discard this patch.
src/Network/Rest/AbstractRest.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -156,8 +156,8 @@
 block discarded – undo
156 156
     {
157 157
         foreach ($options as $option => $value)
158 158
         {
159
-            if (property_exists(__CLASS__, strtolower($option)) && method_exists($this, 'set'.$option))
160
-            $this->{'set'.$option}($value);
159
+            if (property_exists(__CLASS__, strtolower($option)) && method_exists($this, 'set' . $option))
160
+            $this->{'set' . $option}($value);
161 161
         }
162 162
 
163 163
         # HTTP instance
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -127,8 +127,9 @@  discard block
 block discarded – undo
127 127
      */
128 128
     public function setWhiteList(array $whiteList)
129 129
     {
130
-        if (empty($whiteList))
131
-            throw new \RuntimeException("Empty whitelist!");
130
+        if (empty($whiteList)) {
131
+                    throw new \RuntimeException("Empty whitelist!");
132
+        }
132 133
 
133 134
         $this->whiteList = $whiteList;
134 135
     }
@@ -156,8 +157,9 @@  discard block
 block discarded – undo
156 157
     {
157 158
         foreach ($options as $option => $value)
158 159
         {
159
-            if (property_exists(__CLASS__, strtolower($option)) && method_exists($this, 'set'.$option))
160
-            $this->{'set'.$option}($value);
160
+            if (property_exists(__CLASS__, strtolower($option)) && method_exists($this, 'set'.$option)) {
161
+                        $this->{'set'.$option}($value);
162
+            }
161 163
         }
162 164
 
163 165
         # HTTP instance
Please login to merge, or discard this patch.
src/Network/Server.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -64,11 +64,13 @@
 block discarded – undo
64 64
      */
65 65
     public static function getClientIP()
66 66
     {
67
-        if (!empty($_SERVER['HTTP_CLIENT_IP']))
68
-            return $_SERVER['HTTP_CLIENT_IP'];
67
+        if (!empty($_SERVER['HTTP_CLIENT_IP'])) {
68
+                    return $_SERVER['HTTP_CLIENT_IP'];
69
+        }
69 70
 
70
-        if (!empty($_SERVER['HTTP_X_FORWARDED_FOR']))
71
-            return $_SERVER['HTTP_X_FORWARDED_FOR'];
71
+        if (!empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
72
+                    return $_SERVER['HTTP_X_FORWARDED_FOR'];
73
+        }
72 74
 
73 75
         return $_SERVER['REMOTE_ADDR'];
74 76
     }
Please login to merge, or discard this patch.
src/Pear/Mail.php 3 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -131,7 +131,7 @@
 block discarded – undo
131 131
         if (\PEAR::isError($mail))
132 132
         {
133 133
             $this->error(
134
-            	$mail->getCode(), $mail->getMessage()
134
+                $mail->getCode(), $mail->getMessage()
135 135
             );
136 136
 
137 137
             return false;
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
      */
115 115
     public function send($from, $to, $subject, $body)
116 116
     {
117
-        $headers = array (
117
+        $headers = array(
118 118
             'From'         => $from,
119 119
             'To'           => $to,
120 120
             'Subject'      => $subject,
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 
124 124
         $smtp = \Mail::factory(
125 125
             'smtp',
126
-            array ('host' => $this->host, 'auth' => false)
126
+            array('host' => $this->host, 'auth' => false)
127 127
         );
128 128
 
129 129
         $mail = $smtp->send($to, $headers, $body);
Please login to merge, or discard this patch.
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -74,18 +74,21 @@  discard block
 block discarded – undo
74 74
     {
75 75
         @include_once 'System.php';
76 76
 
77
-        if (!class_exists('System', false))
78
-            throw new \RuntimeException("PEAR is not installed in your system!");
77
+        if (!class_exists('System', false)) {
78
+                    throw new \RuntimeException("PEAR is not installed in your system!");
79
+        }
79 80
 
80 81
         include_once 'Net/SMTP.php';
81 82
 
82
-        if (!class_exists('\Net_SMTP', false))
83
-            throw new \RuntimeException("PEAR::Net_SMTP is not installed!");
83
+        if (!class_exists('\Net_SMTP', false)) {
84
+                    throw new \RuntimeException("PEAR::Net_SMTP is not installed!");
85
+        }
84 86
 
85 87
         @include_once 'Mail.php';
86 88
 
87
-        if (!class_exists('\Mail', false))
88
-            throw new \RuntimeException("PEAR::Mail is not installed!");
89
+        if (!class_exists('\Mail', false)) {
90
+                    throw new \RuntimeException("PEAR::Mail is not installed!");
91
+        }
89 92
     }
90 93
 
91 94
     /**
@@ -98,8 +101,9 @@  discard block
 block discarded – undo
98 101
      */
99 102
     protected function error($code, $message = null)
100 103
     {
101
-        if (!array_key_exists($code, $this->errors))
102
-            $this->errors[$message] = (is_null($message) && array_key_exists($code, $this->errors)) ? $this->errors[$code] : $message;
104
+        if (!array_key_exists($code, $this->errors)) {
105
+                    $this->errors[$message] = (is_null($message) && array_key_exists($code, $this->errors)) ? $this->errors[$code] : $message;
106
+        }
103 107
     }
104 108
 
105 109
     /**
Please login to merge, or discard this patch.
src/Util/ParamTrait.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -70,8 +70,9 @@
 block discarded – undo
70 70
     {
71 71
         $parameters = $this->getParams();
72 72
 
73
-        if (array_key_exists($param, $parameters))
74
-            return true;
73
+        if (array_key_exists($param, $parameters)) {
74
+                    return true;
75
+        }
75 76
 
76 77
         return false;
77 78
     }
Please login to merge, or discard this patch.
src/Exception/Storage.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@
 block discarded – undo
60 60
 
61 61
             if (!empty($string))
62 62
             {
63
-                $data   = json_decode($string, true);
63
+                $data = json_decode($string, true);
64 64
 
65 65
                 # json_encode can return TRUE, FALSE or NULL (http://php.net/manual/en/function.json-decode.php)
66 66
                 if (is_null($data) || $data === false)
Please login to merge, or discard this patch.
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -76,17 +76,20 @@
 block discarded – undo
76 76
             "object"  => serialize($exception)
77 77
         ];
78 78
 
79
-        if (!function_exists('mb_detect_encoding'))
80
-            throw new \RuntimeException("mbstring library is not installed!");
79
+        if (!function_exists('mb_detect_encoding')) {
80
+                    throw new \RuntimeException("mbstring library is not installed!");
81
+        }
81 82
 
82 83
         /*
83 84
          * Encodes to UTF8 all messages. It ensures JSON encoding.
84 85
          */
85
-        if (!mb_detect_encoding($data[$id]["message"], 'UTF-8', true))
86
-            $data[$id]["message"] = utf8_encode($data[$id]["message"]);
86
+        if (!mb_detect_encoding($data[$id]["message"], 'UTF-8', true)) {
87
+                    $data[$id]["message"] = utf8_encode($data[$id]["message"]);
88
+        }
87 89
 
88
-        if (!mb_detect_encoding($data[$id]["object"], 'UTF-8', true))
89
-            $data[$id]["object"] = utf8_decode($data[$id]["object"]);
90
+        if (!mb_detect_encoding($data[$id]["object"], 'UTF-8', true)) {
91
+                    $data[$id]["object"] = utf8_decode($data[$id]["object"]);
92
+        }
90 93
 
91 94
         if (($encoded_data = json_encode($data)) === false)
92 95
         {
Please login to merge, or discard this patch.