Unified attachments filenames handling for message parts without a filename

Aleksander Machniak

2012-12-28 be72fb3597c21ca3aaa058adf41bb72d53d197c7

 program/steps/mail/func.inc

@@ -1598,10 +1598,7 @@
  $part  = $MESSAGE->mime_parts[$part];
  $table = new html_table(array('cols' => 3));

  $filename = $part->filename;
  if (empty($filename) && $attach_prop->mimetype == 'text/html') {
    $filename = rcube_label('htmlmessage');
  }
  $filename = rcmail_attachment_name($part);

  if (!empty($filename)) {
    $table->add('title', Q(rcube_label('filename')));
@@ -1614,7 +1611,6 @@

  return $table->show($attrib);
}



function rcmail_message_part_frame($attrib)
@@ -1841,6 +1837,30 @@
  return $name;
}

// return attachment filename, handle empty filename case
function rcmail_attachment_name($attachment)
{
    $filename = $attachment->filename;

    if ($filename === null || $filename === '') {
        if ($attachment->mimetype == 'text/html') {
            $filename = rcube_label('htmlmessage');
        }
        else {
            $ext      = rcube_mime::get_mime_extensions($attachment->mimetype);
            $ext      = array_shift($ext);
            $filename = rcube_label('messagepart') . ' ' . $attachment->mime_id;
            if ($ext) {
                $filename .= '.' . $ext;
            }
        }
    }

    $filename = preg_replace('[\r\n]', '', $filename);

    return $filename;
}

function rcmail_search_filter($attrib)
{
  global $OUTPUT, $CONFIG;