Diff of /spamassassin/trunk/lib/Mail/SpamAssassin/EvalTests.pm

Parent Directory | Revision Log | Patch

--- spamassassin/trunk/lib/Mail/SpamAssassin/EvalTests.pm	2005/05/03 02:01:12	167847
+++ spamassassin/trunk/lib/Mail/SpamAssassin/EvalTests.pm	2005/05/03 02:51:18	167848
@@ -2456,14 +2456,16 @@ sub _check_attachments {
   if ($self->{mime_multipart_alternative}) {
     my $text;
     my $html;
-    for (my $i = 0; $i <= $part; $i++) {
+    # bug 4207: we want the size of the last parts
+    for (my $i = $part; $i >= 0; $i--) {
       next if !defined $part_bytes[$i];
       if (!defined($html) && $part_type[$i] eq 'text/html') {
 	$html = $part_bytes[$i];
       }
-      if (!defined($text) && $part_type[$i] eq 'text/plain') {
+      elsif (!defined($text) && $part_type[$i] eq 'text/plain') {
 	$text = $part_bytes[$i];
       }
+      last if (defined($html) && defined($text));
     }
     if (defined($text) && defined($html) && $html > 0) {
       $self->{mime_multipart_ratio} = ($text / $html);