nystudio107 / craft-transcoder

src/controllers/DefaultController.php

Spacing +10 added lines, -10 removed lines

@@ -31,7 +31,7 @@ 
     // Protected Properties
     // =========================================================================
 
-    protected array|bool|int $allowAnonymous = [
+    protected array | bool | int $allowAnonymous = [
         'download-file',
         'progress',
     ];
@@ -66,7 +66,7 @@ 
         $filePath = parse_url($url, PHP_URL_PATH);
         // Remove any relative paths
         if (!PathHelper::ensurePathIsContained($filePath)) {
-            throw new BadRequestHttpException('Invalid resource path: ' . $filePath);
+            throw new BadRequestHttpException('Invalid resource path: '.$filePath);
         }
         // Only work for `allowedFileExtensions` file extensions
         $extension = strtolower(pathinfo($filePath, PATHINFO_EXTENSION));
@@ -75,7 +75,7 @@ 
             throw new AssetDisallowedExtensionException("File “{$filePath}” cannot be downloaded because “{$extension}” is not allowed.");
         }
 
-        $filePath = $_SERVER['DOCUMENT_ROOT'] . $filePath;
+        $filePath = $_SERVER['DOCUMENT_ROOT'].$filePath;
         Craft::$app->getResponse()->sendFile(
             $filePath,
             null,
@@ -99,7 +99,7 @@ 
     public function actionProgress($filename): Response
     {
         $result = [];
-        $progressFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . $filename . '.progress';
+        $progressFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.$filename.'.progress';
         if (file_exists($progressFile)) {
             $content = @file_get_contents($progressFile);
             if ($content) {
@@ -110,12 +110,12 @@ 
 
                     // rawDuration is in 00:00:00.00 format. This converts it to seconds.
                     $ar = array_reverse(explode(':', $rawDuration));
-                    $duration = (float)$ar[0];
+                    $duration = (float) $ar[0];
                     if (!empty($ar[1])) {
-                        $duration += (int)$ar[1] * 60;
+                        $duration += (int) $ar[1] * 60;
                     }
                     if (!empty($ar[2])) {
-                        $duration += (int)$ar[2] * 60 * 60;
+                        $duration += (int) $ar[2] * 60 * 60;
                     }
                 } else {
                     $duration = 'unknown'; // with GIF as input, duration is unknown
@@ -132,12 +132,12 @@ 
 
                 //rawTime is in 00:00:00.00 format. This converts it to seconds.
                 $ar = array_reverse(explode(':', $rawTime));
-                $time = (float)$ar[0];
+                $time = (float) $ar[0];
                 if (!empty($ar[1])) {
-                    $time += (int)$ar[1] * 60;
+                    $time += (int) $ar[1] * 60;
                 }
                 if (!empty($ar[2])) {
-                    $time += (int)$ar[2] * 60 * 60;
+                    $time += (int) $ar[2] * 60 * 60;
                 }
 
                 //calculate the progress

src/services/Transcode.php

Spacing +71 added lines, -71 removed lines

@@ -98,7 +98,7 @@ 
      * @return string       URL of the transcoded video or ""
      * @throws InvalidConfigException
      */
-    public function getVideoUrl(string|Asset $filePath, array $videoOptions, bool $generate = true): string
+    public function getVideoUrl(string | Asset $filePath, array $videoOptions, bool $generate = true): string
     {
         $result = '';
         $settings = Transcoder::$plugin->getSettings();
@@ -126,20 +126,20 @@ 
 
             // Build the basic command for ffmpeg
             $ffmpegCmd = $settings['ffmpegPath']
-                . ' -i ' . escapeshellarg($filePath)
-                . ' -vcodec ' . $thisEncoder['videoCodec']
-                . ' ' . $thisEncoder['videoCodecOptions']
+                . ' -i '.escapeshellarg($filePath)
+                . ' -vcodec '.$thisEncoder['videoCodec']
+                . ' '.$thisEncoder['videoCodecOptions']
                 . ' -bufsize 1000k'
-                . ' -threads ' . $thisEncoder['threads'];
+                . ' -threads '.$thisEncoder['threads'];
 
             // Set the framerate if desired
             if (!empty($videoOptions['videoFrameRate'])) {
-                $ffmpegCmd .= ' -r ' . $videoOptions['videoFrameRate'];
+                $ffmpegCmd .= ' -r '.$videoOptions['videoFrameRate'];
             }
 
             // Set the bitrate if desired
             if (!empty($videoOptions['videoBitRate'])) {
-                $ffmpegCmd .= ' -b:v ' . $videoOptions['videoBitRate'] . ' -maxrate ' . $videoOptions['videoBitRate'];
+                $ffmpegCmd .= ' -b:v '.$videoOptions['videoBitRate'].' -maxrate '.$videoOptions['videoBitRate'];
             }
 
             // Adjust the scaling if desired
@@ -157,17 +157,17 @@ 
                 $ffmpegCmd .= ' -c:a copy';
             } else {
                 // Do audio transcoding based on the settings
-                $ffmpegCmd .= ' -acodec ' . $thisEncoder['audioCodec'];
+                $ffmpegCmd .= ' -acodec '.$thisEncoder['audioCodec'];
                 if (!empty($videoOptions['audioBitRate'])) {
-                    $ffmpegCmd .= ' -b:a ' . $videoOptions['audioBitRate'];
+                    $ffmpegCmd .= ' -b:a '.$videoOptions['audioBitRate'];
                 }
                 if (!empty($videoOptions['audioSampleRate'])) {
-                    $ffmpegCmd .= ' -ar ' . $videoOptions['audioSampleRate'];
+                    $ffmpegCmd .= ' -ar '.$videoOptions['audioSampleRate'];
                 }
                 if (!empty($videoOptions['audioChannels'])) {
-                    $ffmpegCmd .= ' -ac ' . $videoOptions['audioChannels'];
+                    $ffmpegCmd .= ' -ac '.$videoOptions['audioChannels'];
                 }
-                $ffmpegCmd .= ' ' . $thisEncoder['audioCodecOptions'];
+                $ffmpegCmd .= ' '.$thisEncoder['audioCodecOptions'];
             }
 
             // Create the directory if it isn't there already
@@ -182,17 +182,17 @@ 
             $destVideoFile = $this->getFilename($filePath, $videoOptions);
 
             // File to store the video encoding progress in
-            $progressFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . $destVideoFile . '.progress';
+            $progressFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.$destVideoFile.'.progress';
 
             // Assemble the destination path and final ffmpeg command
             $destVideoPath .= $destVideoFile;
             $ffmpegCmd .= ' -f '
                 . $thisEncoder['fileFormat']
-                . ' -y ' . escapeshellarg($destVideoPath)
-                . ' 1> ' . $progressFile . ' 2>&1 & echo $!';
+                . ' -y '.escapeshellarg($destVideoPath)
+                . ' 1> '.$progressFile.' 2>&1 & echo $!';
 
             // Make sure there isn't a lockfile for this video already
-            $lockFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . $destVideoFile . '.lock';
+            $lockFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.$destVideoFile.'.lock';
             $oldPid = @file_get_contents($lockFile);
             if ($oldPid !== false) {
                 // See if the process is running, and empty result means the process is still running
@@ -210,14 +210,14 @@ 
             if (file_exists($destVideoPath) && (@filemtime($destVideoPath) >= @filemtime($filePath))) {
                 $url = $settings['transcoderUrls']['video'] ?? $settings['transcoderUrls']['default'];
                 $url .= $subfolder;
-                $result = App::parseEnv($url) . $destVideoFile;
+                $result = App::parseEnv($url).$destVideoFile;
             // skip encoding
             } elseif (!$generate) {
                 $result = '';
             } else {
                 // Kick off the transcoding
                 $pid = $this->executeShellCommand($ffmpegCmd);
-                Craft::info($ffmpegCmd . "\nffmpeg PID: " . $pid, __METHOD__);
+                Craft::info($ffmpegCmd."\nffmpeg PID: ".$pid, __METHOD__);
 
                 // Create a lockfile in tmp
                 file_put_contents($lockFile, $pid);
@@ -239,7 +239,7 @@ 
      * @return string|false|null URL or path of the video thumbnail
      * @throws InvalidConfigException
      */
-    public function getVideoThumbnailUrl(Asset|string $filePath, array $thumbnailOptions, bool $generate = true, bool $asPath = false): string|false|null
+    public function getVideoThumbnailUrl(Asset | string $filePath, array $thumbnailOptions, bool $generate = true, bool $asPath = false): string | false | null
     {
         $result = null;
         $settings = Transcoder::$plugin->getSettings();
@@ -261,7 +261,7 @@ 
 
             // Build the basic command for ffmpeg
             $ffmpegCmd = $settings['ffmpegPath']
-                . ' -i ' . escapeshellarg($filePath)
+                . ' -i '.escapeshellarg($filePath)
                 . ' -vcodec mjpeg'
                 . ' -vframes 1';
 
@@ -274,7 +274,7 @@ 
             // Set the timecode to get the thumbnail from if desired
             if (!empty($thumbnailOptions['timeInSecs'])) {
                 $timeCode = gmdate('H:i:s', $thumbnailOptions['timeInSecs']);
-                $ffmpegCmd .= ' -ss ' . $timeCode . '.00';
+                $ffmpegCmd .= ' -ss '.$timeCode.'.00';
             }
 
             // Create the directory if it isn't there already
@@ -290,7 +290,7 @@ 
 
             // Assemble the destination path and final ffmpeg command
             $destThumbnailPath .= $destThumbnailFile;
-            $ffmpegCmd .= ' -f image2 -y ' . escapeshellarg($destThumbnailPath) . ' >/dev/null 2>/dev/null &';
+            $ffmpegCmd .= ' -f image2 -y '.escapeshellarg($destThumbnailPath).' >/dev/null 2>/dev/null &';
 
             // If the thumbnail file already exists, return it.  Otherwise, generate it and return it
             if (!file_exists($destThumbnailPath)) {
@@ -302,7 +302,7 @@ 
                 // if ffmpeg fails which we can't check because the process is ran in the background
                     // don't return the future path of the image or else we can't check this in the front end
                 } else {
-                    Craft::info('Thumbnail does not exist, but not asked to generate it: ' . $filePath, __METHOD__);
+                    Craft::info('Thumbnail does not exist, but not asked to generate it: '.$filePath, __METHOD__);
 
                     // The file doesn't exist, and we weren't asked to generate it
                 }
@@ -314,7 +314,7 @@ 
             } else {
                 $url = $settings['transcoderUrls']['thumbnail'] ?? $settings['transcoderUrls']['default'];
                 $url .= $subfolder;
-                $result = App::parseEnv($url) . $destThumbnailFile;
+                $result = App::parseEnv($url).$destThumbnailFile;
             }
         }
 
@@ -331,7 +331,7 @@ 
      * @return string       URL of the transcoded audio file or ""
      * @throws InvalidConfigException
      */
-    public function getAudioUrl(Asset|string $filePath, array $audioOptions): string
+    public function getAudioUrl(Asset | string $filePath, array $audioOptions): string
     {
         $result = '';
         $settings = Transcoder::$plugin->getSettings();
@@ -359,33 +359,33 @@ 
 
             // Build the basic command for ffmpeg
             $ffmpegCmd = $settings['ffmpegPath']
-                . ' -i ' . escapeshellarg($filePath)
-                . ' -acodec ' . $thisEncoder['audioCodec']
-                . ' ' . $thisEncoder['audioCodecOptions']
+                . ' -i '.escapeshellarg($filePath)
+                . ' -acodec '.$thisEncoder['audioCodec']
+                . ' '.$thisEncoder['audioCodecOptions']
                 . ' -bufsize 1000k'
                 . ' -vn'
-                . ' -threads ' . $thisEncoder['threads'];
+                . ' -threads '.$thisEncoder['threads'];
 
             // Set the bitrate if desired
             if (!empty($audioOptions['audioBitRate'])) {
-                $ffmpegCmd .= ' -b:a ' . $audioOptions['audioBitRate'];
+                $ffmpegCmd .= ' -b:a '.$audioOptions['audioBitRate'];
             }
             // Set the sample rate if desired
             if (!empty($audioOptions['audioSampleRate'])) {
-                $ffmpegCmd .= ' -ar ' . $audioOptions['audioSampleRate'];
+                $ffmpegCmd .= ' -ar '.$audioOptions['audioSampleRate'];
             }
             // Set the audio channels if desired
             if (!empty($audioOptions['audioChannels'])) {
-                $ffmpegCmd .= ' -ac ' . $audioOptions['audioChannels'];
+                $ffmpegCmd .= ' -ac '.$audioOptions['audioChannels'];
             }
-            $ffmpegCmd .= ' ' . $thisEncoder['audioCodecOptions'];
+            $ffmpegCmd .= ' '.$thisEncoder['audioCodecOptions'];
 
             if (!empty($audioOptions['seekInSecs'])) {
-                $ffmpegCmd .= ' -ss ' . $audioOptions['seekInSecs'];
+                $ffmpegCmd .= ' -ss '.$audioOptions['seekInSecs'];
             }
 
             if (!empty($audioOptions['timeInSecs'])) {
-                $ffmpegCmd .= ' -t ' . $audioOptions['timeInSecs'];
+                $ffmpegCmd .= ' -t '.$audioOptions['timeInSecs'];
             }
 
             // Create the directory if it isn't there already
@@ -400,7 +400,7 @@ 
             $destAudioFile = $this->getFilename($filePath, $audioOptions);
 
             // File to store the audio encoding progress in
-            $progressFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . $destAudioFile . '.progress';
+            $progressFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.$destAudioFile.'.progress';
 
             // Assemble the destination path and final ffmpeg command
             $destAudioPath .= $destAudioFile;
@@ -415,16 +415,16 @@ 
             // Add the file format
             $ffmpegCmd .= ' -f '
                 . $thisEncoder['fileFormat']
-                . ' -y ' . escapeshellarg($destAudioPath);
+                . ' -y '.escapeshellarg($destAudioPath);
             // Handle the `synchronous` setting
             $synchronous = false;
             if (!empty($audioOptions['synchronous'])) {
                 $synchronous = $audioOptions['synchronous'];
             }
             if (!$synchronous) {
-                $ffmpegCmd .= ' 1> ' . $progressFile . ' 2>&1 & echo $!';
+                $ffmpegCmd .= ' 1> '.$progressFile.' 2>&1 & echo $!';
                 // Make sure there isn't a lockfile for this audio file already
-                $lockFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . $destAudioFile . '.lock';
+                $lockFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.$destAudioFile.'.lock';
                 $oldPid = @file_get_contents($lockFile);
                 if ($oldPid !== false) {
                     // See if the process is running, and empty result means the process is still running
@@ -443,7 +443,7 @@ 
             if (file_exists($destAudioPath) && (@filemtime($destAudioPath) >= @filemtime($filePath))) {
                 $url = $settings['transcoderUrls']['audio'] ?? $settings['transcoderUrls']['default'];
                 $url .= $subfolder;
-                $result = App::parseEnv($url) . $destAudioFile;
+                $result = App::parseEnv($url).$destAudioFile;
             } else {
                 // Kick off the transcoding
                 $pid = $this->executeShellCommand($ffmpegCmd);
@@ -452,9 +452,9 @@ 
                     Craft::info($ffmpegCmd, __METHOD__);
                     $url = $settings['transcoderUrls']['audio'] ?? $settings['transcoderUrls']['default'];
                     $url .= $subfolder;
-                    $result = App::parseEnv($url) . $destAudioFile;
+                    $result = App::parseEnv($url).$destAudioFile;
                 } else {
-                    Craft::info($ffmpegCmd . "\nffmpeg PID: " . $pid, __METHOD__);
+                    Craft::info($ffmpegCmd."\nffmpeg PID: ".$pid, __METHOD__);
                     // Create a lockfile in tmp
                     file_put_contents($lockFile, $pid);
                 }
@@ -473,7 +473,7 @@ 
      * @return null|array
      * @throws InvalidConfigException
      */
-    public function getFileInfo(Asset|string $filePath, bool $summary = false): ?array
+    public function getFileInfo(Asset | string $filePath, bool $summary = false): ?array
     {
         $result = null;
         $settings = Transcoder::$plugin->getSettings();
@@ -483,8 +483,8 @@ 
             // Build the basic command for ffprobe
             $ffprobeOptions = $settings['ffprobeOptions'];
             $ffprobeCmd = $settings['ffprobePath']
-                . ' ' . $ffprobeOptions
-                . ' ' . escapeshellarg($filePath);
+                . ' '.$ffprobeOptions
+                . ' '.escapeshellarg($filePath);
 
             $shellOutput = $this->executeShellCommand($ffprobeCmd);
             Craft::info($ffprobeCmd, __METHOD__);
@@ -530,7 +530,7 @@ 
                     && (str_contains($summaryResult['videoFrameRate'], '/'))
                 ) {
                     $parts = explode('/', $summaryResult['videoFrameRate']);
-                    $summaryResult['videoFrameRate'] = (float)$parts[0] / (float)$parts[1];
+                    $summaryResult['videoFrameRate'] = (float) $parts[0] / (float) $parts[1];
                 }
                 $result = $summaryResult;
             }
@@ -548,7 +548,7 @@ 
      * @return string
      * @throws InvalidConfigException
      */
-    public function getVideoFilename(Asset|string $filePath, array $videoOptions): string
+    public function getVideoFilename(Asset | string $filePath, array $videoOptions): string
     {
         $settings = Transcoder::$plugin->getSettings();
         $videoOptions = $this->coalesceOptions('defaultVideoOptions', $videoOptions);
@@ -571,7 +571,7 @@ 
      * @return string
      * @throws InvalidConfigException
      */
-    public function getAudioFilename(Asset|string $filePath, array $audioOptions): string
+    public function getAudioFilename(Asset | string $filePath, array $audioOptions): string
     {
         $settings = Transcoder::$plugin->getSettings();
         $audioOptions = $this->coalesceOptions('defaultAudioOptions', $audioOptions);
@@ -594,7 +594,7 @@ 
      * @return string
      * @throws InvalidConfigException
      */
-    public function getGifFilename(Asset|string $filePath, array $gifOptions): string
+    public function getGifFilename(Asset | string $filePath, array $gifOptions): string
     {
         $settings = Transcoder::$plugin->getSettings();
         $gifOptions = $this->coalesceOptions('defaultGifOptions', $gifOptions);
@@ -616,7 +616,7 @@ 
      * @return null|false|string
      * @throws InvalidConfigException
      */
-    public function handleGetAssetThumbPath(DefineAssetThumbUrlEvent $event): null|false|string
+    public function handleGetAssetThumbPath(DefineAssetThumbUrlEvent $event): null | false | string
     {
         $options = [
             'width' => $event->width,
@@ -638,7 +638,7 @@ 
      * @throws InvalidConfigException
      */
 
-    public function getGifUrl(Asset|string $filePath, array $gifOptions): string|false|null
+    public function getGifUrl(Asset | string $filePath, array $gifOptions): string | false | null
     {
         $result = '';
         $settings = Transcoder::$plugin->getSettings();
@@ -668,9 +668,9 @@ 
             // Build the basic command for ffmpeg
             $ffmpegCmd = $settings['ffmpegPath']
                 . ' -f gif'
-                . ' -i ' . escapeshellarg($filePath)
-                . ' -vcodec ' . $thisEncoder['videoCodec']
-                . ' ' . $thisEncoder['videoCodecOptions'];
+                . ' -i '.escapeshellarg($filePath)
+                . ' -vcodec '.$thisEncoder['videoCodec']
+                . ' '.$thisEncoder['videoCodecOptions'];
 
 
             // Create the directory if it isn't there already
@@ -685,16 +685,16 @@ 
             $destVideoFile = $this->getFilename($filePath, $gifOptions);
 
             // File to store the video encoding progress in
-            $progressFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . $destVideoFile . '.progress';
+            $progressFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.$destVideoFile.'.progress';
 
             // Assemble the destination path and final ffmpeg command
             $destVideoPath .= $destVideoFile;
             $ffmpegCmd .= ' '
-                . ' -y ' . escapeshellarg($destVideoPath)
-                . ' 1> ' . $progressFile . ' 2>&1 & echo $!';
+                . ' -y '.escapeshellarg($destVideoPath)
+                . ' 1> '.$progressFile.' 2>&1 & echo $!';
 
             // Make sure there isn't a lockfile for this video already
-            $lockFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . $destVideoFile . '.lock';
+            $lockFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.$destVideoFile.'.lock';
             $oldPid = @file_get_contents($lockFile);
             if ($oldPid !== false) {
                 // See if the process is running, and empty result means the process is still running
@@ -712,11 +712,11 @@ 
             if (file_exists($destVideoPath) && (@filemtime($destVideoPath) >= @filemtime($filePath))) {
                 $url = $settings['transcoderUrls']['gif'] ?? $settings['transcoderUrls']['default'];
                 $url .= $subfolder;
-                $result = App::parseEnv($url) . $destVideoFile;
+                $result = App::parseEnv($url).$destVideoFile;
             } else {
                 // Kick off the transcoding
                 $pid = $this->executeShellCommand($ffmpegCmd);
-                Craft::info($ffmpegCmd . "\nffmpeg PID: " . $pid, __METHOD__);
+                Craft::info($ffmpegCmd."\nffmpeg PID: ".$pid, __METHOD__);
 
                 // Create a lockfile in tmp
                 file_put_contents($lockFile, $pid);
@@ -735,7 +735,7 @@ 
      * @return string
      * @throws InvalidConfigException
      */
-    protected function getFilename(Asset|string $filePath, array $options): string
+    protected function getFilename(Asset | string $filePath, array $options): string
     {
         $settings = Transcoder::$plugin->getSettings();
         $filePath = $this->getAssetPath($filePath);
@@ -758,16 +758,16 @@ 
                     $suffix = self::SUFFIX_MAP[$key];
                 }
                 if (is_bool($value)) {
-                    $value = $value ? $key : 'no' . $key;
+                    $value = $value ? $key : 'no'.$key;
                 }
                 if (!in_array($key, self::EXCLUDE_PARAMS, true)) {
-                    $fileName .= '_' . $value . $suffix;
+                    $fileName .= '_'.$value.$suffix;
                 }
             }
         }
         // See if we should use a hash instead
         if ($settings['useHashedNames']) {
-            $fileName = $pathParts['filename'] . md5($fileName);
+            $fileName = $pathParts['filename'].md5($fileName);
         }
         $fileName .= $options['fileSuffix'];
 
@@ -782,7 +782,7 @@ 
      * @return string
      * @throws InvalidConfigException
      */
-    protected function getAssetPath(Asset|string $filePath): string
+    protected function getAssetPath(Asset | string $filePath): string
     {
         // If we're passed an Asset, extract the path from it
         if (($filePath instanceof Asset)) {
@@ -808,7 +808,7 @@ 
                     }
                     $folderPath .= '' === $folderPath ? '' : DIRECTORY_SEPARATOR;
 
-                    $filePath = $sourcePath . $folderPath . $asset->filename;
+                    $filePath = $sourcePath.$folderPath.$asset->filename;
                 } else {
                     // Otherwise, get a URL
                     $filePath = $asset->getUrl() ?? '';
@@ -816,7 +816,7 @@ 
             }
         }
 
-        $filePath = (string)App::parseEnv($filePath);
+        $filePath = (string) App::parseEnv($filePath);
 
         // Make sure that $filePath is either an existing file, or a valid URL
         if (!file_exists($filePath)) {
@@ -850,16 +850,16 @@ 
                     case 'letterbox':
                         $letterboxColor = '';
                         if (!empty($options['letterboxColor'])) {
-                            $letterboxColor = ':color=' . $options['letterboxColor'];
+                            $letterboxColor = ':color='.$options['letterboxColor'];
                         }
                         $aspectRatio = ':force_original_aspect_ratio=decrease'
-                            . ',pad=' . $options['width'] . ':' . $options['height'] . ':(ow-iw)/2:(oh-ih)/2'
+                            . ',pad='.$options['width'].':'.$options['height'].':(ow-iw)/2:(oh-ih)/2'
                             . $letterboxColor;
                         break;
                     // Scale to the appropriate aspect ratio, cropping
                     case 'crop':
                         $aspectRatio = ':force_original_aspect_ratio=increase'
-                            . ',crop=' . $options['width'] . ':' . $options['height'];
+                            . ',crop='.$options['width'].':'.$options['height'];
                         break;
                     // No aspect ratio scaling at all
                     default:
@@ -873,7 +873,7 @@ 
                 $sharpen = ',unsharp=5:5:1.0:5:5:0.0';
             }
             $ffmpegCmd .= ' -vf "scale='
-                . $options['width'] . ':' . $options['height']
+                . $options['width'].':'.$options['height']
                 . $aspectRatio
                 . $sharpen
                 . '"';