Add audio file type handling using constants for better maintainability

Author: Daniel Neto

objects/Encoder.php

@@ -38,6 +38,8 @@ class Encoder extends ObjectYPT
     const LOG_TYPE_ERROR = 'error';
     const LOG_TYPE_INFO = 'info';
 
+    const AUDIO_EXTENSIONS = ['mp3', 'aac', 'm4a', 'ogg', 'opus', 'flac', 'wav', 'wma'];
+
     protected $id;
     protected $fileURI;
     protected $filename;
@@ -794,6 +796,7 @@ private static function setDownloaded($queue_id, $filePath)
         _error_log($msg);
         $encoder->setStatus(Encoder::STATUS_DOWNLOADED);
         $encoder->setStatus_obs($msg);
+        $encoder->setDownloadedFileName($filePath);
         return $encoder->save();
     }
 

view/queue.php

@@ -54,7 +54,8 @@
     $e->setTitle($path_parts['filename']);
     $e->setPriority($s->getPriority());
 
-    if (strtolower(pathinfo($_POST['fileURI'], PATHINFO_EXTENSION)) === 'mp3') {
+    $fileExtension = strtolower(pathinfo($_POST['fileURI'], PATHINFO_EXTENSION));
+    if (in_array($fileExtension, Encoder::AUDIO_EXTENSIONS)) {
         $e->setFormats_idFromOrder(89); //Audio to MP4
     }else if (!empty($_POST['audioOnly']) && $_POST['audioOnly'] !== 'false') {
         if (!empty($_POST['spectrum']) && $_POST['spectrum'] !== 'false') {

view/upload.php

@@ -23,7 +23,7 @@
     //echo "Success: file extension OK\n";
     //chack if is an audio
     $type = "video";
-    if (strcasecmp($extension, 'mp3') == 0 || strcasecmp($extension, 'wav') == 0) {
+    if (in_array(strtolower($extension), Encoder::AUDIO_EXTENSIONS)) {
         $type = 'audio';
     }