rosell-dk / webp-convert

src/Convert/Converters/Wpc.php

Spacing +4 added lines, -4 removed lines

@@ -49,10 +49,10 @@ 
         parent::createOptions();
 
         $this->options2->addOptions(
-            new SensitiveStringOption('api-key', ''),   /* for communicating with wpc api v.1+ */
-            new SensitiveStringOption('secret', ''),    /* for communicating with wpc api v.0 */
+            new SensitiveStringOption('api-key', ''), /* for communicating with wpc api v.1+ */
+            new SensitiveStringOption('secret', ''), /* for communicating with wpc api v.0 */
             new SensitiveStringOption('api-url', ''),
-            new SensitiveStringOption('url', ''),       /* DO NOT USE. Only here to keep the protection */
+            new SensitiveStringOption('url', ''), /* DO NOT USE. Only here to keep the protection */
             new IntegerOption('api-version', 2, 0, 2),
             new BooleanOption('crypt-api-key-in-transfer', false)  /* new in api v.1 */
         );
@@ -68,7 +68,7 @@ 
             ['.', '/']
         );
 
-        for ($i=0; $i<22; $i++) {
+        for ($i = 0; $i < 22; $i++) {
             $salt .= $validCharsForSalt[array_rand($validCharsForSalt)];
         }
         return $salt;

src/Options/Options.php

Indentation +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
             $this->addOption($option);
         }
     }
-     /*
+        /*
      In some years, we can use the splat instead (requires PHP 5.6):
      @param  Option[]  ...$options  Array of options objects to add
     public function addOptions(...$options)

src/Convert/Converters/Cwebp.php

Indentation +1 added lines, -1 removed lines

@@ -482,7 +482,7 @@
         if ($this->options[$optionName]) {
             $this->logLn(
                 'Discovering binaries ' . $description . ' ' .
-                 '(to skip this step, disable the "' . $optionName . '" option)'
+                    '(to skip this step, disable the "' . $optionName . '" option)'
             );
         } else {
             $this->logLn(

Spacing +4 added lines, -4 removed lines

@@ -277,7 +277,7 @@ 
                 // otherwise encoding=auto would not work as expected
 
                 if ($options['encoding'] == 'lossless') {
-                    $cmdOptions[] ='-near_lossless ' . $options['near-lossless'];
+                    $cmdOptions[] = '-near_lossless ' . $options['near-lossless'];
                 } else {
                     $this->logLn(
                         'The near-lossless option ignored for lossy'
@@ -427,7 +427,7 @@ 
         } else {
             $this->log('. Result: ');
             if ($returnCode == 127) {
-                $this->logLn('*Exec failed* (the cwebp binary was not found at path: ' . $binary. ')');
+                $this->logLn('*Exec failed* (the cwebp binary was not found at path: ' . $binary . ')');
             } else {
                 if ($returnCode == 126) {
                     $this->logLn(
@@ -443,7 +443,7 @@ 
             }
             return $returnCode;
         }
-        return '';  // Will not happen. Just so phpstan doesn't complain
+        return ''; // Will not happen. Just so phpstan doesn't complain
     }
 
     /**
@@ -683,7 +683,7 @@ 
             'Binaries ordered by version number.'
         );
         foreach ($binaryVersions as $binary => $version) {
-            $this->logLn('- ' . $binary . ': (version: ' . $version .')');
+            $this->logLn('- ' . $binary . ': (version: ' . $version . ')');
         }
 
         // Execute!

src/Helpers/PathChecker.php

Indentation +16 added lines, -16 removed lines

@@ -15,22 +15,22 @@
 class PathChecker
 {
 
-     /**
-      * Check absolute file path to prevent attacks.
-      *
-      * - Prevents non printable characters
-      * - Prevents stream wrappers
-      * - Prevents directory traversal
-      *
-      * Preventing non printable characters is especially done to prevent the NUL character, which can be used
-      * to bypass other tests. See https://st-g.de/2011/04/doing-filename-checks-securely-in-PHP.
-      *
-      * Preventeng stream wrappers is especially done to protect against Phar Deserialization.
-      * See https://blog.ripstech.com/2018/new-php-exploitation-technique/
-      *
-      * @param  string  $absFilePath
-      * @return void
-      */
+        /**
+         * Check absolute file path to prevent attacks.
+         *
+         * - Prevents non printable characters
+         * - Prevents stream wrappers
+         * - Prevents directory traversal
+         *
+         * Preventing non printable characters is especially done to prevent the NUL character, which can be used
+         * to bypass other tests. See https://st-g.de/2011/04/doing-filename-checks-securely-in-PHP.
+         *
+         * Preventeng stream wrappers is especially done to protect against Phar Deserialization.
+         * See https://blog.ripstech.com/2018/new-php-exploitation-technique/
+         *
+         * @param  string  $absFilePath
+         * @return void
+         */
     public static function checkAbsolutePath($absFilePath, $text = 'file')
     {
         if (empty($absFilePath)) {