Use an array to list supported content types for enclosures
authorAntonio Ospite <ospite@studenti.unina.it>
Sun, 11 Aug 2013 18:52:47 +0000 (20:52 +0200)
committerAntonio Ospite <ospite@studenti.unina.it>
Sun, 11 Aug 2013 23:21:44 +0000 (01:21 +0200)
tweeper.php

index bdba04d..e9345cc 100644 (file)
@@ -54,35 +54,40 @@ function get_url_info($url)
 
 function generate_enclosure($url)
 {
+  $supported_content_types = array(
+    "video/avi",
+    "image/gif",
+    "image/jpeg",
+    "audio/x-midi",
+    "audio/mpeg",
+    "video/mpeg",
+    "audio/vorbis",
+    "application/ogg",
+    "audio/webm",
+    "audio/aac",
+    "audio/mp4",
+    "audio/wav",
+    "video/mp4",
+    "video/ogg",
+  );
+
   $url_info = get_url_info($url);
 
-  switch($url_info['content_type']) {
-    case "video/avi":
-    case "image/gif":
-    case "image/jpeg":
-    case "audio/x-midi":
-    case "audio/mpeg":
-    case "video/mpeg":
-    case "audio/vorbis":
-    case "application/ogg":
-    case "audio/webm":
-    case "audio/aac":
-    case "audio/mp4":
-    case "audio/wav":
-    case "video/mp4":
-    case "video/ogg":
-      $dom = new DomDocument();
-      $enc = $dom->createElement('enclosure');
-      $enc->setAttribute( 'url', $url_info['url']);
-      $enc->setAttribute( 'length', $url_info['download_content_length']);
-      $enc->setAttribute( 'type', $url_info['content_type']);
-
-      $dom->appendChild($enc);
-
-      return $dom->saveXML($enc);
-    default:
-        return;
+  $supported = in_array($url_info['content_type'], $supported_content_types);
+  if (!$supported) {
+    error_log("Unsupported enclosure content type \"" . $url_info['content_type'] . "\" for URL: " . $url_info['url']);
+    return '';
   }
+
+  $dom = new DomDocument();
+  $enc = $dom->createElement('enclosure');
+  $enc->setAttribute( 'url', $url_info['url']);
+  $enc->setAttribute( 'length', $url_info['download_content_length']);
+  $enc->setAttribute( 'type', $url_info['content_type']);
+
+  $dom->appendChild($enc);
+
+  return $dom->saveXML($enc);
 }