Passed
Branch master (f6a670)
by Sebastian
07:16 queued 02:32
created
src/OperationResult/Collection.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
      * @param int $code
37 37
      * @return $this
38 38
      */
39
-    public function makeError(string $message, int $code=0) : OperationResult
39
+    public function makeError(string $message, int $code = 0) : OperationResult
40 40
     {
41 41
         return $this->add('makeError', $message, $code);
42 42
     }
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
      * @param int $code
47 47
      * @return $this
48 48
      */
49
-    public function makeSuccess(string $message, int $code=0) : OperationResult
49
+    public function makeSuccess(string $message, int $code = 0) : OperationResult
50 50
     {
51 51
         return $this->add('makeSuccess', $message, $code);
52 52
     }
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
      * @param int $code
57 57
      * @return $this
58 58
      */
59
-    public function makeWarning(string $message, int $code=0) : OperationResult
59
+    public function makeWarning(string $message, int $code = 0) : OperationResult
60 60
     {
61 61
         return $this->add('makeWarning', $message, $code);
62 62
     }
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
      * @param int $code
67 67
      * @return $this
68 68
      */
69
-    public function makeNotice(string $message, int $code=0) : OperationResult
69
+    public function makeNotice(string $message, int $code = 0) : OperationResult
70 70
     {
71 71
         return $this->add('makeNotice', $message, $code);
72 72
     }
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
      * @param int $code
78 78
      * @return $this
79 79
      */
80
-    protected function add(string $method, string $message, int $code=0) : OperationResult
80
+    protected function add(string $method, string $message, int $code = 0) : OperationResult
81 81
     {
82 82
         $result = new OperationResult($this->subject);
83 83
         $result->$method($message, $code);
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
      */
96 96
     public function addResult(OperationResult $result) : OperationResult_Collection
97 97
     {
98
-        if($result instanceof OperationResult_Collection)
98
+        if ($result instanceof OperationResult_Collection)
99 99
         {
100 100
             return $this->importCollection($result);
101 101
         }
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
     {
116 116
         $results = $collection->getResults();
117 117
         
118
-        foreach($results as $result)
118
+        foreach ($results as $result)
119 119
         {
120 120
             $this->addResult($result);
121 121
         }
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
     
134 134
     public function isValid() : bool
135 135
     {
136
-        foreach($this->results as $result)
136
+        foreach ($this->results as $result)
137 137
         {
138
-            if(!$result->isValid())
138
+            if (!$result->isValid())
139 139
             {
140 140
                 return false;
141 141
             }
@@ -146,9 +146,9 @@  discard block
 block discarded – undo
146 146
     
147 147
     public function hasCode() : bool
148 148
     {
149
-        foreach($this->results as $result)
149
+        foreach ($this->results as $result)
150 150
         {
151
-            if($result->hasCode())
151
+            if ($result->hasCode())
152 152
             {
153 153
                 return true;
154 154
             }
@@ -159,9 +159,9 @@  discard block
 block discarded – undo
159 159
     
160 160
     public function getCode() : int
161 161
     {
162
-        foreach($this->results as $result)
162
+        foreach ($this->results as $result)
163 163
         {
164
-            if($result->hasCode())
164
+            if ($result->hasCode())
165 165
             {
166 166
                 return $result->getCode();
167 167
             }
@@ -170,13 +170,13 @@  discard block
 block discarded – undo
170 170
         return 0;
171 171
     }
172 172
     
173
-    public function getMessage(string $type='') : string
173
+    public function getMessage(string $type = '') : string
174 174
     {
175
-        foreach($this->results as $result)
175
+        foreach ($this->results as $result)
176 176
         {
177 177
             $msg = $result->getMessage($type);
178 178
             
179
-            if(!empty($msg))
179
+            if (!empty($msg))
180 180
             {
181 181
                 return $msg;
182 182
             }
@@ -187,9 +187,9 @@  discard block
 block discarded – undo
187 187
     
188 188
     public function containsCode(int $code) : bool
189 189
     {
190
-        foreach($this->results as $result)
190
+        foreach ($this->results as $result)
191 191
         {
192
-            if($result->getCode() === $code)
192
+            if ($result->getCode() === $code)
193 193
             {
194 194
                 return true;
195 195
             }
@@ -222,9 +222,9 @@  discard block
 block discarded – undo
222 222
     {
223 223
         $amount = 0;
224 224
         
225
-        foreach($this->results as $result)
225
+        foreach ($this->results as $result)
226 226
         {
227
-            if($result->isType($type))
227
+            if ($result->isType($type))
228 228
             {
229 229
                 $amount++;
230 230
             }
@@ -262,9 +262,9 @@  discard block
 block discarded – undo
262 262
     {
263 263
         $results = array();
264 264
         
265
-        foreach($this->results as $result)
265
+        foreach ($this->results as $result)
266 266
         {
267
-            if($result->isType($type))
267
+            if ($result->isType($type))
268 268
             {
269 269
                 $results[] = $result;
270 270
             }
@@ -275,9 +275,9 @@  discard block
 block discarded – undo
275 275
     
276 276
     public function isType(string $type) : bool
277 277
     {
278
-        foreach($this->results as $result)
278
+        foreach ($this->results as $result)
279 279
         {
280
-            if($result->isType($type))
280
+            if ($result->isType($type))
281 281
             {
282 282
                 return true;
283 283
             }
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
         $lines[] = 'Collection #'.$this->getID();
294 294
         $lines[] = 'Subject: '.get_class($this->subject);
295 295
         
296
-        foreach($this->results as $result)
296
+        foreach ($this->results as $result)
297 297
         {
298 298
             $lines[] = ' - '.$result->getType().' #'.$result->getCode().' "'.$result->getMessage($result->getType()).'"';
299 299
         }
Please login to merge, or discard this patch.
src/OperationResult.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
     * @param string $message Should not contain a date, just the system specific info.
145 145
     * @return $this
146 146
     */
147
-    public function makeSuccess(string $message, int $code=0) : OperationResult
147
+    public function makeSuccess(string $message, int $code = 0) : OperationResult
148 148
     {
149 149
         return $this->setMessage(self::TYPE_SUCCESS, $message, $code, true);
150 150
     }
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
     * @param string $message Should be as detailed as possible.
156 156
     * @return $this
157 157
     */
158
-    public function makeError(string $message, int $code=0) : OperationResult
158
+    public function makeError(string $message, int $code = 0) : OperationResult
159 159
     {
160 160
         return $this->setMessage(self::TYPE_ERROR, $message, $code, false);
161 161
     }
@@ -252,11 +252,11 @@  discard block
 block discarded – undo
252 252
         return $this->code;
253 253
     }
254 254
     
255
-    public function getMessage(string $type='') : string
255
+    public function getMessage(string $type = '') : string
256 256
     {
257
-        if(!empty($type))
257
+        if (!empty($type))
258 258
         {
259
-            if($this->type === $type)
259
+            if ($this->type === $type)
260 260
             {
261 261
                 return $this->message;
262 262
             }
@@ -280,11 +280,11 @@  discard block
 block discarded – undo
280 280
      *
281 281
      * @throws ConvertHelper_Exception
282 282
      */
283
-    public function makeException(Throwable $e, int $code=0, bool $withDeveloperInfo=false) : OperationResult
283
+    public function makeException(Throwable $e, int $code = 0, bool $withDeveloperInfo = false) : OperationResult
284 284
     {
285 285
         $info = parseThrowable($e);
286 286
 
287
-        if($code === 0)
287
+        if ($code === 0)
288 288
         {
289 289
             $code = $info->getCode();
290 290
         }
Please login to merge, or discard this patch.
src/Microtime.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -114,9 +114,9 @@  discard block
 block discarded – undo
114 114
      * @see Microtime::createFromString()
115 115
      * @see Microtime::createNow()
116 116
      */
117
-    public function __construct($datetime=self::DATETIME_NOW, ?DateTimeZone $timeZone=null)
117
+    public function __construct($datetime = self::DATETIME_NOW, ?DateTimeZone $timeZone = null)
118 118
     {
119
-        if($datetime instanceof Microtime_ParseResult)
119
+        if ($datetime instanceof Microtime_ParseResult)
120 120
         {
121 121
             $parsed = $datetime;
122 122
         }
@@ -148,9 +148,9 @@  discard block
 block discarded – undo
148 148
      * @return Microtime_ParseResult
149 149
      * @throws Microtime_Exception
150 150
      */
151
-    private function parseDate($datetime, ?DateTimeZone $timeZone=null) : Microtime_ParseResult
151
+    private function parseDate($datetime, ?DateTimeZone $timeZone = null) : Microtime_ParseResult
152 152
     {
153
-        if($datetime instanceof Microtime)
153
+        if ($datetime instanceof Microtime)
154 154
         {
155 155
             return new Microtime_ParseResult(
156 156
                 $datetime->getISODate(),
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
             );
159 159
         }
160 160
 
161
-        if($datetime instanceof DateTime)
161
+        if ($datetime instanceof DateTime)
162 162
         {
163 163
             return new Microtime_ParseResult(
164 164
                 $datetime->format(self::FORMAT_ISO),
@@ -166,17 +166,17 @@  discard block
 block discarded – undo
166 166
             );
167 167
         }
168 168
 
169
-        if($timeZone === null)
169
+        if ($timeZone === null)
170 170
         {
171 171
             $timeZone = new DateTimeZone(date_default_timezone_get());
172 172
         }
173 173
 
174
-        if(empty($datetime) || $datetime === self::DATETIME_NOW)
174
+        if (empty($datetime) || $datetime === self::DATETIME_NOW)
175 175
         {
176 176
             return self::parseNow($timeZone);
177 177
         }
178 178
 
179
-        if(is_string($datetime))
179
+        if (is_string($datetime))
180 180
         {
181 181
             return new Microtime_ParseResult(
182 182
                 $datetime,
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
     {
204 204
         $dateObj = DateTime::createFromFormat('0.u00 U', microtime(), new DateTimeZone('America/Denver'));
205 205
 
206
-        if($dateObj !== false)
206
+        if ($dateObj !== false)
207 207
         {
208 208
             $dateObj->setTimezone($timeZone);
209 209
 
@@ -227,9 +227,9 @@  discard block
 block discarded – undo
227 227
      * @return Microtime
228 228
      * @throws Microtime_Exception
229 229
      */
230
-    public static function createNow(?DateTimeZone $timeZone=null) : Microtime
230
+    public static function createNow(?DateTimeZone $timeZone = null) : Microtime
231 231
     {
232
-        if($timeZone === null)
232
+        if ($timeZone === null)
233 233
         {
234 234
             $timeZone = new DateTimeZone(date_default_timezone_get());
235 235
         }
@@ -246,9 +246,9 @@  discard block
 block discarded – undo
246 246
      * @return Microtime
247 247
      * @throws Microtime_Exception
248 248
      */
249
-    public static function createFromString(string $date, ?DateTimeZone $timeZone=null) : Microtime
249
+    public static function createFromString(string $date, ?DateTimeZone $timeZone = null) : Microtime
250 250
     {
251
-        if($timeZone === null)
251
+        if ($timeZone === null)
252 252
         {
253 253
             $timeZone = new DateTimeZone(date_default_timezone_get());
254 254
         }
Please login to merge, or discard this patch.
src/ConvertHelper/ThrowableInfo.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
      */
81 81
     protected function __construct($subject)
82 82
     {
83
-        if(is_array($subject))
83
+        if (is_array($subject))
84 84
         {
85 85
             $this->parseSerialized($subject);
86 86
         }
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
     */
140 140
     public function getPrevious() : ConvertHelper_ThrowableInfo
141 141
     {
142
-        if(isset($this->previous)) {
142
+        if (isset($this->previous)) {
143 143
             return $this->previous;
144 144
         }
145 145
         
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
     public function getFolderDepth() : int
259 259
     {
260 260
         $depth = $this->getOption('folder-depth');
261
-        if(!empty($depth)) {
261
+        if (!empty($depth)) {
262 262
             return $depth;
263 263
         }
264 264
         
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
 
316 316
         $this->setOptions($serialized[ConvertHelper_ThrowableInfo_Serializer::SERIALIZED_OPTIONS]);
317 317
         
318
-        foreach($serialized[ConvertHelper_ThrowableInfo_Serializer::SERIALIZED_CALLS] as $def)
318
+        foreach ($serialized[ConvertHelper_ThrowableInfo_Serializer::SERIALIZED_CALLS] as $def)
319 319
         {
320 320
             $this->calls[] = ConvertHelper_ThrowableInfo_Call::fromSerialized($this, $def);
321 321
         }
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
         $this->date = new Microtime();
327 327
         $this->class = get_class($e);
328 328
 
329
-        if($e instanceof BaseException)
329
+        if ($e instanceof BaseException)
330 330
         {
331 331
             $this->details = $e->getDetails();
332 332
         }
@@ -359,7 +359,7 @@  discard block
 block discarded – undo
359 359
      * @return string
360 360
      * @throws ConvertHelper_Exception
361 361
      */
362
-    public function renderErrorMessage(bool $withDeveloperInfo=false) : string
362
+    public function renderErrorMessage(bool $withDeveloperInfo = false) : string
363 363
     {
364 364
         return (new ConvertHelper_ThrowableInfo_MessageRenderer($this, $withDeveloperInfo))
365 365
             ->render();
@@ -435,7 +435,7 @@  discard block
 block discarded – undo
435 435
         }
436 436
         else
437 437
         {
438
-            $this->message = 'Original error code: [' . $code . ']. ' . $this->message;
438
+            $this->message = 'Original error code: ['.$code.']. '.$this->message;
439 439
         }
440 440
     }
441 441
 
Please login to merge, or discard this patch.
src/ConvertHelper/ThrowableInfo/MessageRenderer.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
             ->t('Message:')
39 39
             ->add($this->info->getMessage());
40 40
 
41
-        if($this->developerInfo)
41
+        if ($this->developerInfo)
42 42
         {
43 43
             $message
44 44
                 ->eol()
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
                 ->add($finalCall->toString());
47 47
         }
48 48
 
49
-        if($this->developerInfo && $this->info->hasDetails())
49
+        if ($this->developerInfo && $this->info->hasDetails())
50 50
         {
51 51
             $message
52 52
                 ->t('Developer details:')
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
                 ->add($this->info->getDetails());
55 55
         }
56 56
 
57
-        if($this->info->hasPrevious())
57
+        if ($this->info->hasPrevious())
58 58
         {
59 59
             $message
60 60
                 ->eol()
Please login to merge, or discard this patch.
src/ConvertHelper/ThrowableInfo/Serializer.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -39,13 +39,13 @@  discard block
 block discarded – undo
39 39
             self::SERIALIZED_PREVIOUS => null,
40 40
         );
41 41
 
42
-        if($info->hasPrevious())
42
+        if ($info->hasPrevious())
43 43
         {
44 44
             $result[self::SERIALIZED_PREVIOUS] = $info->getPrevious()->serialize();
45 45
         }
46 46
 
47 47
         $calls = $info->getCalls();
48
-        foreach($calls as $call)
48
+        foreach ($calls as $call)
49 49
         {
50 50
             $result[self::SERIALIZED_CALLS][] = $call->serialize();
51 51
         }
@@ -63,12 +63,12 @@  discard block
 block discarded – undo
63 63
         $data = self::validateSerializedData($serialized);
64 64
         $data[self::SERIALIZED_PREVIOUS] = self::unserializePrevious($data[self::SERIALIZED_PREVIOUS]);
65 65
 
66
-        if(!isset($data[self::SERIALIZED_CLASS]))
66
+        if (!isset($data[self::SERIALIZED_CLASS]))
67 67
         {
68 68
             $data[self::SERIALIZED_CLASS] = '';
69 69
         }
70 70
 
71
-        if(!isset($data[self::SERIALIZED_DETAILS]))
71
+        if (!isset($data[self::SERIALIZED_DETAILS]))
72 72
         {
73 73
             $data[self::SERIALIZED_DETAILS] = '';
74 74
         }
@@ -94,9 +94,9 @@  discard block
 block discarded – undo
94 94
             self::SERIALIZED_CALLS => 'array'
95 95
         );
96 96
 
97
-        foreach($keys as $key => $type)
97
+        foreach ($keys as $key => $type)
98 98
         {
99
-            if(!isset($serialized[$key]) || gettype($serialized[$key]) !== $type)
99
+            if (!isset($serialized[$key]) || gettype($serialized[$key]) !== $type)
100 100
             {
101 101
                 throw self::createTypeException($key, $type);
102 102
             }
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 
121 121
     private static function unserializePrevious(?array $previous) : ?ConvertHelper_ThrowableInfo
122 122
     {
123
-        if(!empty($previous))
123
+        if (!empty($previous))
124 124
         {
125 125
             return ConvertHelper_ThrowableInfo::fromSerialized($previous);
126 126
         }
Please login to merge, or discard this patch.
src/ConvertHelper/ThrowableInfo/StringConverter.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -19,8 +19,8 @@  discard block
 block discarded – undo
19 19
     public function toString() : string
20 20
     {
21 21
         return
22
-            $this->renderMessage() .
23
-            $this->renderCalls() .
22
+            $this->renderMessage().
23
+            $this->renderCalls().
24 24
             $this->renderPrevious();
25 25
     }
26 26
 
@@ -33,12 +33,12 @@  discard block
 block discarded – undo
33 33
 
34 34
         if ($this->info->hasCode())
35 35
         {
36
-            $string .= ' #' . $this->info->getCode();
36
+            $string .= ' #'.$this->info->getCode();
37 37
         }
38 38
 
39 39
         $string .=
40
-            ': ' .
41
-            $this->info->getMessage() .
40
+            ': '.
41
+            $this->info->getMessage().
42 42
             PHP_EOL;
43 43
 
44 44
         return $string;
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 
56 56
         foreach ($calls as $call)
57 57
         {
58
-            $string .= $call->toString() . PHP_EOL;
58
+            $string .= $call->toString().PHP_EOL;
59 59
         }
60 60
 
61 61
         return $string;
@@ -73,11 +73,11 @@  discard block
 block discarded – undo
73 73
         }
74 74
 
75 75
         return
76
-            PHP_EOL .
77
-            PHP_EOL .
78
-            'Previous error:' .
79
-            PHP_EOL .
80
-            PHP_EOL .
76
+            PHP_EOL.
77
+            PHP_EOL.
78
+            'Previous error:'.
79
+            PHP_EOL.
80
+            PHP_EOL.
81 81
             $this->info->getPrevious()->toString();
82 82
     }
83 83
 }
Please login to merge, or discard this patch.
src/HTMLTag.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
         return $this->name;
74 74
     }
75 75
 
76
-    public function setSelfClosing(bool $selfClosing=true) : HTMLTag
76
+    public function setSelfClosing(bool $selfClosing = true) : HTMLTag
77 77
     {
78 78
         $this->selfClosing = $selfClosing;
79 79
         return $this;
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
         return $this->selfClosing;
85 85
     }
86 86
 
87
-    public function setEmptyAllowed(bool $allowed=true) : HTMLTag
87
+    public function setEmptyAllowed(bool $allowed = true) : HTMLTag
88 88
     {
89 89
         $this->allowEmpty = $allowed;
90 90
         return $this;
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 
93 93
     public function isEmptyAllowed() : bool
94 94
     {
95
-        if($this->isSelfClosing())
95
+        if ($this->isSelfClosing())
96 96
         {
97 97
             return true;
98 98
         }
@@ -100,9 +100,9 @@  discard block
 block discarded – undo
100 100
         return $this->allowEmpty;
101 101
     }
102 102
 
103
-    public static function create(string $name, ?AttributeCollection $attributes=null) : HTMLTag
103
+    public static function create(string $name, ?AttributeCollection $attributes = null) : HTMLTag
104 104
     {
105
-        if($attributes === null)
105
+        if ($attributes === null)
106 106
         {
107 107
             $attributes = AttributeCollection::create();
108 108
         }
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 
129 129
     public function render() : string
130 130
     {
131
-        if(!$this->isEmptyAllowed() && $this->isEmpty())
131
+        if (!$this->isEmptyAllowed() && $this->isEmpty())
132 132
         {
133 133
             return '';
134 134
         }
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
 
142 142
     public static function getGlobalOptions() : GlobalOptions
143 143
     {
144
-        if(!isset(self::$globalOptions))
144
+        if (!isset(self::$globalOptions))
145 145
         {
146 146
             self::$globalOptions = new GlobalOptions();
147 147
         }
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
 
152 152
     public function getSelfClosingChar() : string
153 153
     {
154
-        if($this->selfClosing && self::getGlobalOptions()->getSelfCloseStyle() === self::SELF_CLOSE_STYLE_SLASH)
154
+        if ($this->selfClosing && self::getGlobalOptions()->getSelfCloseStyle() === self::SELF_CLOSE_STYLE_SLASH)
155 155
         {
156 156
             return '/';
157 157
         }
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
 
172 172
     public function renderClose() : string
173 173
     {
174
-        if($this->selfClosing)
174
+        if ($this->selfClosing)
175 175
         {
176 176
             return '';
177 177
         }
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
 
212 212
     public function renderContent() : string
213 213
     {
214
-        if($this->selfClosing)
214
+        if ($this->selfClosing)
215 215
         {
216 216
             return '';
217 217
         }
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
         return $this;
231 231
     }
232 232
 
233
-    public function prop(string $name, bool $enabled=true) : HTMLTag
233
+    public function prop(string $name, bool $enabled = true) : HTMLTag
234 234
     {
235 235
         $this->attributes->prop($name, $enabled);
236 236
         return $this;
Please login to merge, or discard this patch.
src/ZIPHelper.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     
33 33
     public const ERROR_OPENING_ZIP_FILE = 338003;
34 34
     
35
-    public const ERROR_CANNOT_SAVE_FILE_TO_DISK =338004;
35
+    public const ERROR_CANNOT_SAVE_FILE_TO_DISK = 338004;
36 36
     
37 37
     protected $options = array(
38 38
         'WriteThreshold' => 100
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     * 
83 83
     * @see FileHelper::ERROR_SOURCE_FILE_DOES_NOT_EXIST
84 84
     */
85
-    public function addFile(string $filePath, ?string $zipPath=null) : bool
85
+    public function addFile(string $filePath, ?string $zipPath = null) : bool
86 86
     {
87 87
         $this->open();
88 88
         
@@ -135,16 +135,16 @@  discard block
 block discarded – undo
135 135
      */
136 136
     protected function open() : void
137 137
     {
138
-        if($this->open) {
138
+        if ($this->open) {
139 139
             return;
140 140
         }
141 141
         
142
-        if(!isset($this->zip)) {
142
+        if (!isset($this->zip)) {
143 143
             $this->zip = new ZipArchive();
144 144
         }
145 145
         
146 146
         $flag = null;
147
-        if(!file_exists($this->file)) {
147
+        if (!file_exists($this->file)) {
148 148
             $flag = ZipArchive::CREATE;
149 149
         }
150 150
         
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
     {
188 188
         $this->fileTracker++;
189 189
 
190
-        if($this->options['WriteThreshold'] < 1) {
190
+        if ($this->options['WriteThreshold'] < 1) {
191 191
             return;
192 192
         }
193 193
         
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
     
201 201
     protected function close() : void
202 202
     {
203
-        if(!$this->open) {
203
+        if (!$this->open) {
204 204
             return;
205 205
         }
206 206
         
@@ -209,8 +209,8 @@  discard block
 block discarded – undo
209 209
             throw new ZIPHelper_Exception(
210 210
                 'Could not save ZIP file to disk',
211 211
                 sprintf(
212
-                    'Tried saving the ZIP file [%1$s], but the write failed. This can have several causes, ' .
213
-                    'including adding files that do not exist on disk, trying to create an empty zip, ' .
212
+                    'Tried saving the ZIP file [%1$s], but the write failed. This can have several causes, '.
213
+                    'including adding files that do not exist on disk, trying to create an empty zip, '.
214 214
                     'or trying to save to a directory that does not exist.',
215 215
                     $this->file
216 216
                 ),
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
     {
226 226
         $this->open();
227 227
         
228
-        if($this->countFiles() < 1) 
228
+        if ($this->countFiles() < 1) 
229 229
         {
230 230
             throw new ZIPHelper_Exception(
231 231
                 'No files in the zip file',
@@ -251,18 +251,18 @@  discard block
 block discarded – undo
251 251
      * @throws ZIPHelper_Exception
252 252
      * @return string The file name that was sent (useful in case none was specified).
253 253
      */
254
-    public function download(?string $fileName=null) : string
254
+    public function download(?string $fileName = null) : string
255 255
     {
256 256
         $this->save();
257 257
         
258
-        if(empty($fileName))
258
+        if (empty($fileName))
259 259
         {
260 260
             $fileName = basename($this->file);
261 261
         }
262 262
         
263 263
         header('Content-type: application/zip');
264
-        header('Content-Disposition: attachment; filename=' . $fileName);
265
-        header('Content-length: ' . filesize($this->file));
264
+        header('Content-Disposition: attachment; filename='.$fileName);
265
+        header('Content-length: '.filesize($this->file));
266 266
         header('Pragma: no-cache');
267 267
         header('Expires: 0');
268 268
         readfile($this->file);
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
      * @throws ZIPHelper_Exception
281 281
      * @see ZIPHelper::download()
282 282
      */
283
-    public function downloadAndDelete(?string $fileName=null) : ZIPHelper
283
+    public function downloadAndDelete(?string $fileName = null) : ZIPHelper
284 284
     {
285 285
         $this->download($fileName);
286 286
         
@@ -297,9 +297,9 @@  discard block
 block discarded – undo
297 297
     * @param string|NULL $outputFolder If no folder is specified, uses the target file's folder.
298 298
     * @return boolean
299 299
     */
300
-    public function extractAll(?string $outputFolder=null) : bool
300
+    public function extractAll(?string $outputFolder = null) : bool
301 301
     {
302
-        if(empty($outputFolder)) {
302
+        if (empty($outputFolder)) {
303 303
             $outputFolder = dirname($this->file);
304 304
         }
305 305
         
Please login to merge, or discard this patch.