View GitHub Repository phpbu

Schedule Inspection
Subscribe
Completed
Push — master ( e61662...273c09 )
by Sebastian
06:26
created
Status
Category
Toggle Patches
src/Backup/Sync/AmazonS3.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch 
@@ -126,7 +126,7 @@  discard block
 
 block discarded – undo
126 126 
     {
127 127 
         foreach (['key', 'secret', 'bucket', 'region', 'path'] as $option) {
128 128 
             if (!Arr::isSetAndNotEmptyString($config, $option)) {
129 
-                throw new Exception('AWS S3 ' . $option . ' is mandatory');
129 
+                throw new Exception('AWS S3 '.$option.' is mandatory');
130 130 
             }
131 131 
         }
132 132 
     }
 
@@ -150,11 +150,11 @@  discard block
 
 block discarded – undo
150 150 
     public function simulate(Target $target, Result $result)
151 151 
     {
152 152 
         $result->debug(
153 
-            'sync backup to Amazon S3' . PHP_EOL
154 
-            . '  region:   ' . $this->region . PHP_EOL
155 
-            . '  key:      ' . $this->key . PHP_EOL
156 
-            . '  secret:    ********' . PHP_EOL
157 
-            . '  location: ' . $this->bucket
153 
+            'sync backup to Amazon S3'.PHP_EOL
154 
+            . '  region:   '.$this->region.PHP_EOL
155 
+            . '  key:      '.$this->key.PHP_EOL
156 
+            . '  secret:    ********'.PHP_EOL
157 
+            . '  location: '.$this->bucket
158 158 
         );
159 159 
     }
160 160
Please login to merge, or discard this patch.
src/Backup/Cleaner/Capacity.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -130,7 +130,7 @@
 
 block discarded – undo
130 130 
         // should the currently created backup be deleted as well?
131 131 
         if ($this->deleteTarget) {
132 132 
             $file          = $target->toFile();
133 
-            $index         = date('YmdHis', $file->getMTime()) . '-' . count($files) . '-' . $file->getPathname();
133 
+            $index         = date('YmdHis', $file->getMTime()).'-'.count($files).'-'.$file->getPathname();
134 134 
             $files[$index] = $file;
135 135 
         }
136 136 
         return $files;
Please login to merge, or discard this patch.
src/Log/ResultFormatter/FormData.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -41,7 +41,7 @@
 
 block discarded – undo
41 41 
         $messages = [];
42 42 
         /** @var \Exception $e */
43 43 
         foreach ($errors as $e) {
44 
-            $messages[] = $e->getMessage() . ' in file:' . $e->getFile() . ' at line' . $e->getLine();
44 
+            $messages[] = $e->getMessage().' in file:'.$e->getFile().' at line'.$e->getLine();
45 45 
         }
46 46 
         return implode(';', $messages);
47 47 
     }
Please login to merge, or discard this patch.
src/Log/ResultFormatter/Template.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch 
@@ -45,7 +45,7 @@  discard block
 
 block discarded – undo
45 45 
     private function loadBodyFromTemplate(string $template) : string
46 46 
     {
47 47 
         if (!file_exists($template)) {
48 
-            throw new Exception('template not found: ' . $template);
48 
+            throw new Exception('template not found: '.$template);
49 49 
         }
50 50 
         return file_get_contents($template);
51 51 
     }
 
@@ -115,7 +115,7 @@  discard block
 
 block discarded – undo
115 115 
     {
116 116 
         $subTemplate = '';
117 117 
         $match       = [];
118 
-        if (preg_match('#%%' . $loop . '%%([\w\W\s]*)%%' . $loop . '%%#im', $template, $match)) {
118 
+        if (preg_match('#%%'.$loop.'%%([\w\W\s]*)%%'.$loop.'%%#im', $template, $match)) {
119 119 
             $subTemplate = $match[1];
120 120 
         }
121 121 
         return $subTemplate;
 
@@ -133,7 +133,7 @@  discard block
 
 block discarded – undo
133 133 
         $markup = '';
134 134 
         /* @var $e \Exception */
135 135 
         foreach ($errors as $e) {
136 
-            $data   = [
136 
+            $data = [
137 137 
                 'class'   => get_class($e),
138 138 
                 'message' => $e->getMessage(),
139 139 
                 'file'    => $e->getFile(),
 
@@ -156,7 +156,7 @@  discard block
 
 block discarded – undo
156 156 
         $markup = '';
157 157 
         /* @var $b \phpbu\App\Result\Backup */
158 158 
         foreach ($backups as $b) {
159 
-            $data   = [
159 
+            $data = [
160 160 
                 'name'           => $b->getName(),
161 161 
                 'status'         => $b->allOk() ? 0 : 1,
162 162 
                 'checkCount'     => $b->checkCount(),
 
@@ -186,7 +186,7 @@  discard block
 
 block discarded – undo
186 186 
     private function renderTemplate(string $template, array $data) : string
187 187 
     {
188 188 
         foreach ($data as $name => $value) {
189 
-            $template = str_replace('%' . $name . '%', $value, $template);
189 
+            $template = str_replace('%'.$name.'%', $value, $template);
190 190 
         }
191 191 
         return $template;
192 192 
     }
 
@@ -199,6 +199,6 @@  discard block
 
 block discarded – undo
199 199 
      */
200 200 
     private function renderLoop(string $loop, string $markup)
201 201 
     {
202 
-        $this->template = preg_replace('#%%' . $loop . '%%[\w\W\s]*%%' . $loop . '%%#im', $markup, $this->template);
202 
+        $this->template = preg_replace('#%%'.$loop.'%%[\w\W\s]*%%'.$loop.'%%#im', $markup, $this->template);
203 203 
     }
204 204 
 }
Please login to merge, or discard this patch.
src/Log/Webhook.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -153,7 +153,7 @@  discard block
 
 block discarded – undo
153 153 
     private function buildGetUri(array $data) : string
154 154 
     {
155 155 
         $glue = strpos($this->uri, '?') !== false ? '&' : '?';
156 
-        return $this->uri . $glue . http_build_query($data);
156 
+        return $this->uri.$glue.http_build_query($data);
157 157 
     }
158 158
159 159 
     /**
 
@@ -170,7 +170,7 @@  discard block
 
 block discarded – undo
170 170 
         }
171 171
172 172 
         if (!isset($this->availableFormatter[$this->contentType])) {
173 
-            throw new Exception('no default formatter for content-type: ' . $this->contentType);
173 
+            throw new Exception('no default formatter for content-type: '.$this->contentType);
174 174 
         }
175 175 
         $class = $this->availableFormatter[$this->contentType];
176 176 
         return new $class();
 
@@ -214,12 +214,12 @@  discard block
 
 block discarded – undo
214 214 
         ];
215 215
216 216 
         if (!empty($body)) {
217 
-            $headers[]                  = 'Content-Type: ' . $this->contentType;
217 
+            $headers[]                  = 'Content-Type: '.$this->contentType;
218 218 
             $options['http']['content'] = $body;
219 219 
         }
220 220
221 221 
         if (!empty($this->username)) {
222 
-            $headers[] = 'Authorization: Basic ' . base64_encode($this->username . ':' . $this->password);
222 
+            $headers[] = 'Authorization: Basic '.base64_encode($this->username.':'.$this->password);
223 223 
         }
224 224
225 225 
         $options['http']['header'] = implode("\r\n", $headers);
 
@@ -228,7 +228,7 @@  discard block
 
 block discarded – undo
228 228 
         try {
229 229 
             file_get_contents($uri, false, $context);
230 230 
         } catch (\Throwable $t) {
231 
-            throw new Exception('could not reach webhook: ' . $this->uri);
231 
+            throw new Exception('could not reach webhook: '.$this->uri);
232 232 
         }
233 233 
     }
234 234 
 }
Please login to merge, or discard this patch.
src/Log/Mail.php 2 patches
Indentation   +57 added lines, -57 removed lines patch added patch discarded remove patch 
@@ -177,12 +177,12 @@  discard block
 
 block discarded – undo
177 177 
             $info    = $this->getInfoHtml($result);
178 178 
             $footer  = $this->getFooterHtml();
179 179 
             $body    = '<html><body '. TPL::getSnippet('sBody') . '>'
180 
-                     . $header
181 
-                     . $status
182 
-                     . $errors
183 
-                     . $info
184 
-                     . $footer
185 
-                     . '</body></html>';
180 
+                        . $header
181 
+                        . $status
182 
+                        . $errors
183 
+                        . $info
184 
+                        . $footer
185 
+                        . '</body></html>';
186 186 
             $sent    = null;
187 187 
             $state   = $allGood ? 'OK' : ($result->backupOkButSkipsOrFails() ? 'WARNING' : 'ERROR');
188 188 
 
@@ -310,7 +310,7 @@  discard block
 
 block discarded – undo
310 310
311 311 
         if ($username && $password) {
312 312 
             $transport->setUsername($username)
313 
-                      ->setPassword($password);
313 
+                        ->setPassword($password);
314 314 
         }
315 315 
         if ($encryption) {
316 316 
             $transport->setEncryption($encryption);
 
@@ -343,7 +343,7 @@  discard block
 
 block discarded – undo
343 343 
     protected function getHeaderHtml()
344 344 
     {
345 345 
         return '<table ' . TPL::getSnippet('sTableContent') . '><tr><td ' . TPL::getSnippet('sTableContentCol') . '>' .
346 
-               '<table ' . TPL::getSnippet('sTableHeader') . '><tr><td>PHPBU - backup report</td></tr></table>';
346 
+                '<table ' . TPL::getSnippet('sTableHeader') . '><tr><td>PHPBU - backup report</td></tr></table>';
347 347 
     }
348 348
349 349 
     /**
 
@@ -381,11 +381,11 @@  discard block
 
 block discarded – undo
381 381 
             Str::appendPluralS('cleanup', $this->numCleanups)
382 382 
         );
383 383 
         $html = '<table ' . sprintf(TPL::getSnippet('sTableStatus'), $color) .'>' .
384 
-                 '<tr><td>' .
385 
-                  '<span ' . TPL::getSnippet('sTableStatusText') . '>' . date('Y-m-d H:i') . '</span>' .
386 
-                  '<h1 ' . TPL::getSnippet('sTableStatusHead') . '>' . $status . '</h1>' .
387 
-                  '<span ' . TPL::getSnippet('sTableStatusText') . '>' . $info . '</span>' .
388 
-                 '</td></tr>' .
384 
+                    '<tr><td>' .
385 
+                    '<span ' . TPL::getSnippet('sTableStatusText') . '>' . date('Y-m-d H:i') . '</span>' .
386 
+                    '<h1 ' . TPL::getSnippet('sTableStatusHead') . '>' . $status . '</h1>' .
387 
+                    '<span ' . TPL::getSnippet('sTableStatusText') . '>' . $info . '</span>' .
388 
+                    '</td></tr>' .
389 389 
                 '</table>';
390 390
391 391 
         return $html;
 
@@ -446,65 +446,65 @@  discard block
 
 block discarded – undo
446 446 
                     $status = 'FAILURE';
447 447 
                 }
448 448 
                 $html .= '<tr>' .
449 
-                          '<td ' . sprintf(TPL::getSnippet('sTableBackupStatusColumn'), $color) . ' colspan="4">' .
450 
-                          sprintf('backup <em>%s</em>', $backup->getName()) .
451 
-                          ' <span ' . TPL::getSnippet('sTableBackupStatusText') . '>' . $status .'</span>'.
452 
-                          '</td>' .
453 
-                         '</tr>' .
454 
-                         '<tr>' .
455 
-                          '<td ' . TPL::getSnippet('sRowHead') . '>&nbsp;</td>' .
456 
-                          '<td ' . TPL::getSnippet('sRowHead') . ' align="right">executed</td>' .
457 
-                          '<td ' . TPL::getSnippet('sRowHead') . ' align="right">skipped</td>' .
458 
-                          '<td ' . TPL::getSnippet('sRowHead') . ' align="right">failed</td>' .
459 
-                         '</tr>';
449 
+                            '<td ' . sprintf(TPL::getSnippet('sTableBackupStatusColumn'), $color) . ' colspan="4">' .
450 
+                            sprintf('backup <em>%s</em>', $backup->getName()) .
451 
+                            ' <span ' . TPL::getSnippet('sTableBackupStatusText') . '>' . $status .'</span>'.
452 
+                            '</td>' .
453 
+                            '</tr>' .
454 
+                            '<tr>' .
455 
+                            '<td ' . TPL::getSnippet('sRowHead') . '>&nbsp;</td>' .
456 
+                            '<td ' . TPL::getSnippet('sRowHead') . ' align="right">executed</td>' .
457 
+                            '<td ' . TPL::getSnippet('sRowHead') . ' align="right">skipped</td>' .
458 
+                            '<td ' . TPL::getSnippet('sRowHead') . ' align="right">failed</td>' .
459 
+                            '</tr>';
460 460
461 461 
                 $html .= '<tr>' .
462 
-                          '<td ' . TPL::getSnippet('sRowCheck') . '>checks</td>' .
463 
-                          '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">' .
462 
+                            '<td ' . TPL::getSnippet('sRowCheck') . '>checks</td>' .
463 
+                            '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">' .
464 464 
                             $backup->checkCount() . '
465 465 
                            </td>' .
466 
-                          '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">
466 
+                            '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">
467 467 
                             &nbsp;
468 468 
                            </td>' .
469 
-                          '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">' .
469 
+                            '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">' .
470 470 
                             $backup->checkCountFailed() .
471 
-                          '</td>' .
472 
-                         '</tr>' .
473 
-                         '<tr>' .
474 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . '>crypts</td>' .
475 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
471 
+                            '</td>' .
472 
+                            '</tr>' .
473 
+                            '<tr>' .
474 
+                            '<td ' . TPL::getSnippet('sRowCrypt') . '>crypts</td>' .
475 
+                            '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
476 476 
                             $backup->cryptCount() .
477 
-                          '</td>' .
478 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
477 
+                            '</td>' .
478 
+                            '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
479 479 
                             $backup->cryptCountSkipped() .
480 
-                          '</td>' .
481 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
480 
+                            '</td>' .
481 
+                            '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
482 482 
                             $backup->cryptCountFailed() .
483 
-                          '</td>' .
484 
-                         '</tr>' .
485 
-                         '<tr>' .
486 
-                          '<td ' . TPL::getSnippet('sRowSync') . '>syncs</td>' .
487 
-                          '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
483 
+                            '</td>' .
484 
+                            '</tr>' .
485 
+                            '<tr>' .
486 
+                            '<td ' . TPL::getSnippet('sRowSync') . '>syncs</td>' .
487 
+                            '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
488 488 
                             $backup->syncCount() . '</td>' .
489 
-                          '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
489 
+                            '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
490 490 
                             $backup->syncCountSkipped() .
491 
-                          '</td>' .
492 
-                          '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
491 
+                            '</td>' .
492 
+                            '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
493 493 
                             $backup->syncCountFailed() .
494 
-                          '</td>' .
495 
-                         '</tr>' .
496 
-                         '<tr>' .
497 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . '>cleanups</td>' .
498 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
494 
+                            '</td>' .
495 
+                            '</tr>' .
496 
+                            '<tr>' .
497 
+                            '<td ' . TPL::getSnippet('sRowCleanup') . '>cleanups</td>' .
498 
+                            '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
499 499 
                             $backup->cleanupCount() .
500 
-                          '</td>' .
501 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
500 
+                            '</td>' .
501 
+                            '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
502 502 
                             $backup->cleanupCountSkipped() .
503 
-                          '</td>' .
504 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
503 
+                            '</td>' .
504 
+                            '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
505 505 
                             $backup->cleanupCountFailed() .
506 
-                          '</td>' .
507 
-                         '</tr>';
506 
+                            '</td>' .
507 
+                            '</tr>';
508 508
509 509 
             }
510 510 
             $html .= '</table>';
 
@@ -520,6 +520,6 @@  discard block
 
 block discarded – undo
520 520 
     protected function getFooterHtml()
521 521 
     {
522 522 
         return '<p ' . TPL::getSnippet('sStats') . '>' . PHP_Timer::resourceUsage() . '</p>' .
523 
-               '</td></tr></table>';
523 
+                '</td></tr></table>';
524 524 
     }
525 525 
 }
Please login to merge, or discard this patch.
Spacing   +74 added lines, -74 removed lines patch added patch discarded remove patch 
@@ -148,8 +148,8 @@  discard block
 
 block discarded – undo
148 148 
         $mails                 = $options['recipients'];
149 149 
         $server                = gethostname();
150 150 
         $this->sendOnlyOnError = Str::toBoolean(Arr::getValue($options, 'sendOnlyOnError'), false);
151 
-        $this->subject         = Arr::getValue($options, 'subject', 'PHPBU backup report from ' . $server);
152 
-        $this->senderMail      = Arr::getValue($options, 'sender.mail', 'phpbu@' . $server);
151 
+        $this->subject         = Arr::getValue($options, 'subject', 'PHPBU backup report from '.$server);
152 
+        $this->senderMail      = Arr::getValue($options, 'sender.mail', 'phpbu@'.$server);
153 153 
         $this->senderName      = Arr::getValue($options, 'sender.name');
154 154 
         $this->transportType   = Arr::getValue($options, 'transport', 'mail');
155 155 
         $this->recipients      = array_map('trim', explode(';', $mails));
 
@@ -176,7 +176,7 @@  discard block
 
 block discarded – undo
176 176 
             $errors  = $this->getErrorHtml($result);
177 177 
             $info    = $this->getInfoHtml($result);
178 178 
             $footer  = $this->getFooterHtml();
179 
-            $body    = '<html><body '. TPL::getSnippet('sBody') . '>'
179 
+            $body    = '<html><body '.TPL::getSnippet('sBody').'>'
180 180 
                      . $header
181 181 
                      . $status
182 182 
                      . $errors
 
@@ -189,7 +189,7 @@  discard block
 
 block discarded – undo
189 189 
             try {
190 190 
                 /** @var \Swift_Message $message */
191 191 
                 $message = new Swift_Message();
192 
-                $message->setSubject($this->subject . ' [' . $state . ']')
192 
+                $message->setSubject($this->subject.' ['.$state.']')
193 193 
                         ->setFrom($this->senderMail, $this->senderName)
194 194 
                         ->setTo($this->recipients)
195 195 
                         ->setBody($body, 'text/html');
 
@@ -328,9 +328,9 @@  discard block
 
 block discarded – undo
328 328 
     {
329 329 
         if (isset($options['sendmail.path'])) {
330 330 
             $path    = $options['sendmail.path'];
331 
-            $options = isset($options['sendmail.options']) ? ' ' . $options['sendmail.options'] : '';
331 
+            $options = isset($options['sendmail.options']) ? ' '.$options['sendmail.options'] : '';
332 332 
             /* @var $transport \Swift_SendmailTransport */
333 
-            return new \Swift_SendmailTransport($path . $options);
333 
+            return new \Swift_SendmailTransport($path.$options);
334 334 
         }
335 335 
         return new \Swift_SendmailTransport();
336 336 
     }
 
@@ -342,8 +342,8 @@  discard block
 
 block discarded – undo
342 342 
      */
343 343 
     protected function getHeaderHtml()
344 344 
     {
345 
-        return '<table ' . TPL::getSnippet('sTableContent') . '><tr><td ' . TPL::getSnippet('sTableContentCol') . '>' .
346 
-               '<table ' . TPL::getSnippet('sTableHeader') . '><tr><td>PHPBU - backup report</td></tr></table>';
345 
+        return '<table '.TPL::getSnippet('sTableContent').'><tr><td '.TPL::getSnippet('sTableContentCol').'>'.
346 
+               '<table '.TPL::getSnippet('sTableHeader').'><tr><td>PHPBU - backup report</td></tr></table>';
347 347 
     }
348 348
349 349 
     /**
 
@@ -380,12 +380,12 @@  discard block
 
 block discarded – undo
380 380 
             $this->numCleanups,
381 381 
             Str::appendPluralS('cleanup', $this->numCleanups)
382 382 
         );
383 
-        $html = '<table ' . sprintf(TPL::getSnippet('sTableStatus'), $color) .'>' .
384 
-                 '<tr><td>' .
385 
-                  '<span ' . TPL::getSnippet('sTableStatusText') . '>' . date('Y-m-d H:i') . '</span>' .
386 
-                  '<h1 ' . TPL::getSnippet('sTableStatusHead') . '>' . $status . '</h1>' .
387 
-                  '<span ' . TPL::getSnippet('sTableStatusText') . '>' . $info . '</span>' .
388 
-                 '</td></tr>' .
383 
+        $html = '<table '.sprintf(TPL::getSnippet('sTableStatus'), $color).'>'.
384 
+                 '<tr><td>'.
385 
+                  '<span '.TPL::getSnippet('sTableStatusText').'>'.date('Y-m-d H:i').'</span>'.
386 
+                  '<h1 '.TPL::getSnippet('sTableStatusHead').'>'.$status.'</h1>'.
387 
+                  '<span '.TPL::getSnippet('sTableStatusText').'>'.$info.'</span>'.
388 
+                 '</td></tr>'.
389 389 
                 '</table>';
390 390
391 391 
         return $html;
 
@@ -402,17 +402,17 @@  discard block
 
 block discarded – undo
402 402 
         $html   = '';
403 403 
         $errors = $result->getErrors();
404 404 
         if (count($errors)) {
405 
-            $html .= '<table ' . TPL::getSnippet('sTableError') . '>';
405 
+            $html .= '<table '.TPL::getSnippet('sTableError').'>';
406 406 
             /* @var $e Exception */
407 407 
             foreach ($errors as $e) {
408 
-                $html .= '<tr><td ' . TPL::getSnippet('sTableErrorCol') . '>' .
408 
+                $html .= '<tr><td '.TPL::getSnippet('sTableErrorCol').'>'.
409 409 
                     sprintf(
410 410 
                         "Exception '%s' with message '%s' in %s:%d",
411 411 
                         get_class($e),
412 412 
                         $e->getMessage(),
413 413 
                         $e->getFile(),
414 414 
                         $e->getLine()
415 
-                    ) .
415 
+                    ).
416 416 
                     '</td></tr>';
417 417
418 418 
             }
 
@@ -432,78 +432,78 @@  discard block
 
 block discarded – undo
432 432 
         $html    = '';
433 433 
         $backups = $result->getBackups();
434 434 
         if (count($backups)) {
435 
-            $html .= '<table ' . TPL::getSnippet('sTableBackup') . '>';
435 
+            $html .= '<table '.TPL::getSnippet('sTableBackup').'>';
436 436 
             /** @var \phpbu\App\Result\Backup $backup */
437 437 
             foreach ($backups as $backup) {
438 438 
                 if ($backup->allOk()) {
439 439 
                     $color  = TPL::getSnippet('cStatusOK');
440 440 
                     $status = 'OK';
441 
-                } elseif($backup->okButSkipsOrFails()) {
441 
+                } elseif ($backup->okButSkipsOrFails()) {
442 442 
                     $color  = TPL::getSnippet('cStatusWARN');
443 443 
                     $status = 'WARNING';
444 444 
                 } else {
445 445 
                     $color  = TPL::getSnippet('cStatusFAIL');
446 446 
                     $status = 'FAILURE';
447 447 
                 }
448 
-                $html .= '<tr>' .
449 
-                          '<td ' . sprintf(TPL::getSnippet('sTableBackupStatusColumn'), $color) . ' colspan="4">' .
450 
-                          sprintf('backup <em>%s</em>', $backup->getName()) .
451 
-                          ' <span ' . TPL::getSnippet('sTableBackupStatusText') . '>' . $status .'</span>'.
452 
-                          '</td>' .
453 
-                         '</tr>' .
454 
-                         '<tr>' .
455 
-                          '<td ' . TPL::getSnippet('sRowHead') . '>&nbsp;</td>' .
456 
-                          '<td ' . TPL::getSnippet('sRowHead') . ' align="right">executed</td>' .
457 
-                          '<td ' . TPL::getSnippet('sRowHead') . ' align="right">skipped</td>' .
458 
-                          '<td ' . TPL::getSnippet('sRowHead') . ' align="right">failed</td>' .
448 
+                $html .= '<tr>'.
449 
+                          '<td '.sprintf(TPL::getSnippet('sTableBackupStatusColumn'), $color).' colspan="4">'.
450 
+                          sprintf('backup <em>%s</em>', $backup->getName()).
451 
+                          ' <span '.TPL::getSnippet('sTableBackupStatusText').'>'.$status.'</span>'.
452 
+                          '</td>'.
453 
+                         '</tr>'.
454 
+                         '<tr>'.
455 
+                          '<td '.TPL::getSnippet('sRowHead').'>&nbsp;</td>'.
456 
+                          '<td '.TPL::getSnippet('sRowHead').' align="right">executed</td>'.
457 
+                          '<td '.TPL::getSnippet('sRowHead').' align="right">skipped</td>'.
458 
+                          '<td '.TPL::getSnippet('sRowHead').' align="right">failed</td>'.
459 459 
                          '</tr>';
460 460
461 
-                $html .= '<tr>' .
462 
-                          '<td ' . TPL::getSnippet('sRowCheck') . '>checks</td>' .
463 
-                          '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">' .
464 
-                            $backup->checkCount() . '
461 
+                $html .= '<tr>'.
462 
+                          '<td '.TPL::getSnippet('sRowCheck').'>checks</td>'.
463 
+                          '<td '.TPL::getSnippet('sRowCheck').' align="right">'.
464 
+                            $backup->checkCount().'
465 465 
                            </td>' .
466 
-                          '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">
466 
+                          '<td '.TPL::getSnippet('sRowCheck').' align="right">
467 467 
                             &nbsp;
468 468 
                            </td>' .
469 
-                          '<td ' . TPL::getSnippet('sRowCheck') . ' align="right">' .
470 
-                            $backup->checkCountFailed() .
471 
-                          '</td>' .
472 
-                         '</tr>' .
473 
-                         '<tr>' .
474 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . '>crypts</td>' .
475 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
476 
-                            $backup->cryptCount() .
477 
-                          '</td>' .
478 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
479 
-                            $backup->cryptCountSkipped() .
480 
-                          '</td>' .
481 
-                          '<td ' . TPL::getSnippet('sRowCrypt') . ' align="right">' .
482 
-                            $backup->cryptCountFailed() .
483 
-                          '</td>' .
484 
-                         '</tr>' .
485 
-                         '<tr>' .
486 
-                          '<td ' . TPL::getSnippet('sRowSync') . '>syncs</td>' .
487 
-                          '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
488 
-                            $backup->syncCount() . '</td>' .
489 
-                          '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
490 
-                            $backup->syncCountSkipped() .
491 
-                          '</td>' .
492 
-                          '<td ' . TPL::getSnippet('sRowSync') . ' align="right">' .
493 
-                            $backup->syncCountFailed() .
494 
-                          '</td>' .
495 
-                         '</tr>' .
496 
-                         '<tr>' .
497 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . '>cleanups</td>' .
498 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
499 
-                            $backup->cleanupCount() .
500 
-                          '</td>' .
501 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
502 
-                            $backup->cleanupCountSkipped() .
503 
-                          '</td>' .
504 
-                          '<td ' . TPL::getSnippet('sRowCleanup') . ' align="right">' .
505 
-                            $backup->cleanupCountFailed() .
506 
-                          '</td>' .
469 
+                          '<td '.TPL::getSnippet('sRowCheck').' align="right">'.
470 
+                            $backup->checkCountFailed().
471 
+                          '</td>'.
472 
+                         '</tr>'.
473 
+                         '<tr>'.
474 
+                          '<td '.TPL::getSnippet('sRowCrypt').'>crypts</td>'.
475 
+                          '<td '.TPL::getSnippet('sRowCrypt').' align="right">'.
476 
+                            $backup->cryptCount().
477 
+                          '</td>'.
478 
+                          '<td '.TPL::getSnippet('sRowCrypt').' align="right">'.
479 
+                            $backup->cryptCountSkipped().
480 
+                          '</td>'.
481 
+                          '<td '.TPL::getSnippet('sRowCrypt').' align="right">'.
482 
+                            $backup->cryptCountFailed().
483 
+                          '</td>'.
484 
+                         '</tr>'.
485 
+                         '<tr>'.
486 
+                          '<td '.TPL::getSnippet('sRowSync').'>syncs</td>'.
487 
+                          '<td '.TPL::getSnippet('sRowSync').' align="right">'.
488 
+                            $backup->syncCount().'</td>'.
489 
+                          '<td '.TPL::getSnippet('sRowSync').' align="right">'.
490 
+                            $backup->syncCountSkipped().
491 
+                          '</td>'.
492 
+                          '<td '.TPL::getSnippet('sRowSync').' align="right">'.
493 
+                            $backup->syncCountFailed().
494 
+                          '</td>'.
495 
+                         '</tr>'.
496 
+                         '<tr>'.
497 
+                          '<td '.TPL::getSnippet('sRowCleanup').'>cleanups</td>'.
498 
+                          '<td '.TPL::getSnippet('sRowCleanup').' align="right">'.
499 
+                            $backup->cleanupCount().
500 
+                          '</td>'.
501 
+                          '<td '.TPL::getSnippet('sRowCleanup').' align="right">'.
502 
+                            $backup->cleanupCountSkipped().
503 
+                          '</td>'.
504 
+                          '<td '.TPL::getSnippet('sRowCleanup').' align="right">'.
505 
+                            $backup->cleanupCountFailed().
506 
+                          '</td>'.
507 507 
                          '</tr>';
508 508
509 509 
             }
 
@@ -519,7 +519,7 @@  discard block
 
 block discarded – undo
519 519 
      */
520 520 
     protected function getFooterHtml()
521 521 
     {
522 
-        return '<p ' . TPL::getSnippet('sStats') . '>' . PHP_Timer::resourceUsage() . '</p>' .
522 
+        return '<p '.TPL::getSnippet('sStats').'>'.PHP_Timer::resourceUsage().'</p>'.
523 523 
                '</td></tr></table>';
524 524 
     }
525 525 
 }
Please login to merge, or discard this patch.
src/Cli/Executable/Tar.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -260,7 +260,7 @@
 
 block discarded – undo
260 260 
         $tar->addOptionIfNotEmpty('--force-local', $this->local, false);
261 261 
         $tar->addOptionIfNotEmpty('--ignore-failed-read', $this->ignoreFailedRead, false);
262 262 
         $tar->addOptionIfNotEmpty('--use-compress-program', $this->compressProgram);
263 
-        $tar->addOption('-' . (empty($this->compressProgram) ? $this->compression : '') . $create);
263 
+        $tar->addOption('-'.(empty($this->compressProgram) ? $this->compression : '').$create);
264 264
265 265 
         if ($this->isThrottled()) {
266 266 
             $pv = new Cmd('pv');
Please login to merge, or discard this patch.