From 23797b62eed483e7139ef22c66a57153fb307aa9 Mon Sep 17 00:00:00 2001
From: Bao-Long Nguyen-Trong <baolongnt@gmail.com>
Date: Wed, 20 May 2009 04:36:20 +0000
Subject: [PATCH] . Fixed issue 423:   . HTMLized version of plain text email
 is not loaded up as a MIME part of the message anymore. We use a custom
 seperate variable in the text body class.

---
 .../android/email/activity/MessageView.java   | 71 +++++++++++--------
 .../android/email/mail/store/LocalStore.java  | 35 +++++++--
 2 files changed, 68 insertions(+), 38 deletions(-)

diff --git a/src/com/android/email/activity/MessageView.java b/src/com/android/email/activity/MessageView.java
index a343e904d..ff43d9586 100644
--- a/src/com/android/email/activity/MessageView.java
+++ b/src/com/android/email/activity/MessageView.java
@@ -9,7 +9,6 @@ import java.io.OutputStream;
 import java.text.DateFormat;
 import java.util.ArrayList;
 import java.util.Map;
-import java.util.regex.Matcher;
 
 import org.apache.commons.io.IOUtils;
 
@@ -26,11 +25,7 @@ import android.net.Uri;
 import android.os.Bundle;
 import android.os.Environment;
 import android.os.Handler;
-import android.os.Process;
 import android.text.Spannable;
-import android.text.SpannableString;
-import android.text.SpannableStringBuilder;
-import android.text.util.Regex;
 import android.util.Config;
 import android.util.Log;
 import android.view.KeyEvent;
@@ -56,8 +51,6 @@ import com.android.email.MessagingController;
 import com.android.email.MessagingListener;
 import com.android.email.R;
 import com.android.email.Utility;
-import com.android.email.activity.FolderMessageList.FolderMessageListAdapter.FolderInfoHolder;
-import com.android.email.activity.FolderMessageList.FolderMessageListAdapter.MessageInfoHolder;
 import com.android.email.mail.Address;
 import com.android.email.mail.Flag;
 import com.android.email.mail.Message;
@@ -65,11 +58,10 @@ import com.android.email.mail.MessagingException;
 import com.android.email.mail.Multipart;
 import com.android.email.mail.Part;
 import com.android.email.mail.Message.RecipientType;
-import com.android.email.mail.internet.MimeHeader;
 import com.android.email.mail.internet.MimeUtility;
-import com.android.email.mail.store.LocalStore.LocalAttachmentBody;
 import com.android.email.mail.store.LocalStore.LocalAttachmentBodyPart;
 import com.android.email.mail.store.LocalStore.LocalMessage;
+import com.android.email.mail.store.LocalStore.LocalTextBody;
 import com.android.email.provider.AttachmentProvider;
 import java.util.concurrent.LinkedBlockingQueue;
 import java.util.concurrent.ThreadPoolExecutor;
@@ -1053,34 +1045,51 @@ public class MessageView extends Activity
 
         @Override
         public void loadMessageForViewBodyAvailable(Account account, String folder, String uid,
-                Message message) {
+            Message message) {
             Spannable markup;
             MessageView.this.mMessage = message;
             try {
-            Part part = MimeUtility.findFirstPartByMimeType(mMessage, "text/html");
-            if (part == null) {
-                part = MimeUtility.findFirstPartByMimeType(mMessage, "text/plain");
-            }
-            if (part != null) {
-                String text = MimeUtility.getTextFromPart(part);
-                  /*
-                   * TODO this should be smarter, change to regex for img, but consider how to
-                   * get background images and a million other things that HTML allows.
-                   */
-                 mHandler.showShowPictures(text.contains("<img"));
-                 mMessageContentView.loadDataWithBaseURL("email://", text, "text/html", "utf-8", null);
-              }
-             else
-                  mMessageContentView.loadUrl("file:///android_asset/empty.html");
-              renderAttachments(mMessage, 0);
-              }
-            catch (Exception e) {
-                    if (Config.LOGV) {
-                        Log.v(Email.LOG_TAG, "loadMessageForViewBodyAvailable", e);
+                String text;
+                Part part = MimeUtility.findFirstPartByMimeType(mMessage, "text/html");
+                if (part == null) {
+                    part = MimeUtility.findFirstPartByMimeType(mMessage, "text/plain");
+                    if (part == null) {
+                        text = null;
+                    }
+                    else {
+                        LocalTextBody body = (LocalTextBody)part.getBody();
+                        if (body == null) {
+                            text = null;
+                        }
+                        else {
+                            text = body.getBodyForDisplay();
+                        }
                     }
                 }
+                else {
+                    text = MimeUtility.getTextFromPart(part);
+                }
+
+                if (text != null) {
+                    /*
+                     * TODO this should be smarter, change to regex for img, but consider how to
+                     * get background images and a million other things that HTML allows.
+                     */
+                    mHandler.showShowPictures(text.contains("<img"));
+                    mMessageContentView.loadDataWithBaseURL("email://", text, "text/html", "utf-8", null);
+                }
+                else {
+                    mMessageContentView.loadUrl("file:///android_asset/empty.html");
+                }
+
+                renderAttachments(mMessage, 0);
             }
-  
+            catch (Exception e) {
+                     if (Config.LOGV) {
+                          Log.v(Email.LOG_TAG, "loadMessageForViewBodyAvailable", e);
+                }
+            }
+        }//loadMessageForViewBodyAvailable
 
 
         @Override
diff --git a/src/com/android/email/mail/store/LocalStore.java b/src/com/android/email/mail/store/LocalStore.java
index aa8656d05..e4a087e4f 100644
--- a/src/com/android/email/mail/store/LocalStore.java
+++ b/src/com/android/email/mail/store/LocalStore.java
@@ -802,15 +802,14 @@ public class LocalStore extends Store implements Serializable {
                         String htmlContent = cursor.getString(0);
                         String textContent = cursor.getString(1);
 
-                        if (htmlContent != null) {
-                            TextBody body = new TextBody(htmlContent);
-                            MimeBodyPart bp = new MimeBodyPart(body, "text/html");
+                        if (textContent != null) {
+                            LocalTextBody body = new LocalTextBody(textContent, htmlContent);
+                            MimeBodyPart bp = new MimeBodyPart(body, "text/plain");
                             mp.addBodyPart(bp);
                         }
-
-                        if (textContent != null) {
-                            TextBody body = new TextBody(textContent);
-                            MimeBodyPart bp = new MimeBodyPart(body, "text/plain");
+                        else {
+                            TextBody body = new TextBody(htmlContent);
+                            MimeBodyPart bp = new MimeBodyPart(body, "text/html");
                             mp.addBodyPart(bp);
                         }
                     }
@@ -1509,6 +1508,28 @@ public class LocalStore extends Store implements Serializable {
         }
     }
 
+    public class LocalTextBody extends TextBody {
+        private String mBodyForDisplay;
+
+        public LocalTextBody(String body) {
+            super(body);
+        }
+
+        public LocalTextBody(String body, String bodyForDisplay) throws MessagingException {
+            super(body);
+            this.mBodyForDisplay = bodyForDisplay;
+        }
+
+        public String getBodyForDisplay() {
+            return mBodyForDisplay;
+        }
+
+        public void setBodyForDisplay(String mBodyForDisplay) {
+            this.mBodyForDisplay = mBodyForDisplay;
+        }
+
+    }//LocalTextBody
+
     public class LocalMessage extends MimeMessage {
         private long mId;
         private int mAttachmentCount;