fixed warning

[squirrelmail.git] / src / download.php

diff --git a/src/download.php b/src/download.php
index 16d93dcd506a07e2cd87bfcea542b47715f64c94..323c52172677e5cf8c2f9af6264a19dbebd8ec9d 100644 (file)
--- a/src/download.php
+++ b/src/download.php
@@ -36,7 +36,6 @@ if ( sqgetGlobalVar('passed_id', $temp, SQ_GET) ) {
 }  
 
 /* end globals */
-$mailbox = urldecode($mailbox);
 
 global $uid_support;
 
@@ -54,10 +53,8 @@ if ($ent_id) {
     
     if ($message->rfc822_header) {
        $subject = $message->rfc822_header->subject;
-       $charset = $header->content_type->properties['charset'];
     } else {
        $header = $message->header;
-       $charset = $header->getParameter('charset');
     }
     $type0 = $header->type0;
     $type1 = $header->type1;
@@ -66,7 +63,8 @@ if ($ent_id) {
     /* raw message */
     $type0 = 'message';
     $type1 = 'rfc822';
-    $encoding = "US-ASCII";
+    $encoding = 'US-ASCII';
+    $header = $message->header;
 }
 
 /*
@@ -85,7 +83,7 @@ $filename = '';
 if (is_object($message->header->disposition)) {
     $filename = $header->disposition->getProperty('filename');
     if (!$filename) {
-       $filename = $header->disposition->getProperty('name');
+        $filename = $header->disposition->getProperty('name');
     }
     if (!$filename) {
         $filename = $header->getParameter('name');
@@ -93,26 +91,27 @@ if (is_object($message->header->disposition)) {
 } else {
     $filename = $header->getParameter('name');
 }
+
 $filename = decodeHeader($filename);
 if (strlen($filename) < 1) {
     if ($type1 == 'plain' && $type0 == 'text') {
         $suffix = 'txt';
-       $filename = $subject . '.txt';
+        $filename = $subject . '.txt';
     } else if ($type1 == 'richtext' && $type0 == 'text') {
         $suffix = 'rtf';
-       $filename = $subject . '.rtf';
+        $filename = $subject . '.rtf';
     } else if ($type1 == 'postscript' && $type0 == 'application') {
         $suffix = 'ps';
-       $filename = $subject . '.ps';
+        $filename = $subject . '.ps';
     } else if ($type1 == 'rfc822' && $type0 == 'message') {
         $suffix = 'eml';
-       $filename = $subject . '.msg';
+        $filename = $subject . '.msg';
     } else {
         $suffix = $type1;
     }
 
     if (strlen($filename) < 1) {
-       $filename = 'untitled'.strip_tags($ent_id).$suffix;
+       $filename = 'untitled'.strip_tags($ent_id).'.'.$suffix;
     } else {
        $filename = "$filename.$suffix";
     }