Better error message when invalid arguments are supplied.
[moodle.git] / file.php
index 866942b..78c3ecf 100644 (file)
--- a/file.php
+++ b/file.php
         error("This script DEPENDS on PATH_INFO being available.  Read the README.");
     }
 
-    $args = get_slash_arguments();
+    if (! $args = get_slash_arguments()) {
+        error("No valid arguments supplied");
+    }
+
     $numargs = count($args);
     $courseid = (integer)$args[0];
 
@@ -39,7 +42,8 @@
         header("Expires: " . gmdate("D, d M Y H:i:s", time() + $lifetime) . " GMT");
         header("Cache-control: max_age = $lifetime"); // a day
         header("Pragma: ");
-        header("Content-Length: ".filesize($pathname));
+        header("Content-disposition: inline; filename=$filename");
+        header("Content-length: ".filesize($pathname));
         header("Content-type: $mimetype");
         readfile("$pathname");
     } else {