From 7f625b5ef4e3da5ed25a52ad63d915cd19d91359 Mon Sep 17 00:00:00 2001
From: cketti <cketti@googlemail.com>
Date: Sun, 20 Jun 2010 12:48:22 +0000
Subject: [PATCH] Added option to specify Reply-to address for an identity.
 Patch provided by fiouzy (Thanks!).

Fixes issue 1762
---
 res/layout/edit_identity.xml                 | 14 ++++++++++++++
 res/values/strings.xml                       |  2 ++
 src/com/fsck/k9/Account.java                 |  4 ++++
 src/com/fsck/k9/Identity.java                | 13 ++++++++++++-
 src/com/fsck/k9/activity/EditIdentity.java   | 13 +++++++++++++
 src/com/fsck/k9/activity/MessageCompose.java |  6 ++++++
 6 files changed, 51 insertions(+), 1 deletion(-)

diff --git a/res/layout/edit_identity.xml b/res/layout/edit_identity.xml
index c8a1d0adb..fd7c99601 100644
--- a/res/layout/edit_identity.xml
+++ b/res/layout/edit_identity.xml
@@ -49,6 +49,20 @@
             android:layout_width="fill_parent" 
             android:hint="@string/edit_identity_email_hint"
             />
+        <TextView
+            android:text="@string/edit_identity_reply_to_label"
+            android:layout_height="wrap_content"
+            android:layout_width="fill_parent"
+            android:textAppearance="?android:attr/textAppearanceSmall"
+            android:textColor="?android:attr/textColorPrimary" 
+            />
+        <EditText
+            android:id="@+id/reply_to"
+            android:singleLine="true"
+            android:layout_height="wrap_content"
+            android:layout_width="fill_parent" 
+            android:hint="@string/edit_identity_reply_to_hint"
+            />
             <!-- 
         <TextView
             android:text="@string/edit_identity_always_bcc_label"
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 2648a73b7..7590b8b90 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -609,6 +609,8 @@ Welcome to K-9 Mail setup.  K-9 is an open source mail client for Android origin
     <string name="edit_identity_name_hint">(Optional)</string>
     <string name="edit_identity_email_label">Email address</string>
     <string name="edit_identity_email_hint">(Required)</string>
+    <string name="edit_identity_reply_to_label">Reply-to address</string>
+    <string name="edit_identity_reply_to_hint">(Optional)</string>
     <string name="edit_identity_signature_label">Signature</string>
     <string name="edit_identity_signature_hint">(Optional)</string>
     
diff --git a/src/com/fsck/k9/Account.java b/src/com/fsck/k9/Account.java
index 12f4c097a..518709566 100644
--- a/src/com/fsck/k9/Account.java
+++ b/src/com/fsck/k9/Account.java
@@ -1044,6 +1044,7 @@ public class Account implements BaseAccount
             boolean signatureUse = prefs.getBoolean(mUuid  + ".signatureUse." + ident, true);
             String signature = prefs.getString(mUuid + ".signature." + ident, null);
             String description = prefs.getString(mUuid + ".description." + ident, null);
+            final String replyTo = prefs.getString(mUuid + ".replyTo." + ident, null);
             if (email != null)
             {
                 Identity identity = new Identity();
@@ -1052,6 +1053,7 @@ public class Account implements BaseAccount
                 identity.setSignatureUse(signatureUse);
                 identity.setSignature(signature);
                 identity.setDescription(description);
+                identity.setReplyTo(replyTo);
                 newIdentities.add(identity);
                 gotOne = true;
             }
@@ -1092,6 +1094,7 @@ public class Account implements BaseAccount
                 editor.remove(mUuid + ".signatureUse." + ident);
                 editor.remove(mUuid + ".signature." + ident);
                 editor.remove(mUuid + ".description." + ident);
+                editor.remove(mUuid + ".replyTo." + ident);
                 gotOne = true;
             }
             ident++;
@@ -1111,6 +1114,7 @@ public class Account implements BaseAccount
             editor.putBoolean(mUuid + ".signatureUse." + ident, identity.getSignatureUse());
             editor.putString(mUuid + ".signature." + ident, identity.getSignature());
             editor.putString(mUuid + ".description." + ident, identity.getDescription());
+            editor.putString(mUuid + ".replyTo." + ident, identity.getReplyTo());
             ident++;
         }
     }
diff --git a/src/com/fsck/k9/Identity.java b/src/com/fsck/k9/Identity.java
index 92d228701..bd8b20e04 100644
--- a/src/com/fsck/k9/Identity.java
+++ b/src/com/fsck/k9/Identity.java
@@ -9,6 +9,7 @@ public class Identity implements Serializable
     private String mEmail;
     private String mSignature;
     private boolean mSignatureUse;
+    private String replyTo;
 
     public synchronized String getName()
     {
@@ -60,9 +61,19 @@ public class Identity implements Serializable
         mDescription = description;
     }
 
+    public synchronized String getReplyTo()
+	{
+        return replyTo;
+    }
+
+    public synchronized void setReplyTo(String replyTo)
+	{
+        this.replyTo = replyTo;
+    }
+
     @Override
     public synchronized String toString()
     {
-        return "Account.Identity(description=" + mDescription + ", name=" + mName + ", email=" + mEmail + ", signature=" + mSignature;
+        return "Account.Identity(description=" + mDescription + ", name=" + mName + ", email=" + mEmail + ", replyTo=" + replyTo + ", signature=" + mSignature;
     }
 }
diff --git a/src/com/fsck/k9/activity/EditIdentity.java b/src/com/fsck/k9/activity/EditIdentity.java
index b3e85eccf..7ff7d5679 100644
--- a/src/com/fsck/k9/activity/EditIdentity.java
+++ b/src/com/fsck/k9/activity/EditIdentity.java
@@ -30,6 +30,7 @@ public class EditIdentity extends K9Activity
     private EditText mEmailView;
 //  private EditText mAlwaysBccView;
     private EditText mNameView;
+    private EditText mReplyTo;
 
     @Override
     public void onCreate(Bundle savedInstanceState)
@@ -66,6 +67,9 @@ public class EditIdentity extends K9Activity
         mEmailView = (EditText)findViewById(R.id.email);
         mEmailView.setText(mIdentity.getEmail());
 
+        mReplyTo = (EditText) findViewById(R.id.reply_to);
+        mReplyTo.setText(mIdentity.getReplyTo());
+
 //      mAccountAlwaysBcc = (EditText)findViewById(R.id.bcc);
 //      mAccountAlwaysBcc.setText(mIdentity.getAlwaysBcc());
 
@@ -115,6 +119,15 @@ public class EditIdentity extends K9Activity
         mIdentity.setSignatureUse(mSignatureUse.isChecked());
         mIdentity.setSignature(mSignatureView.getText().toString());
 
+        if (mReplyTo.getText().length() == 0)
+        {
+            mIdentity.setReplyTo(null);
+        }
+        else
+        {
+            mIdentity.setReplyTo(mReplyTo.getText().toString());
+        }
+
         List<Identity> identities = mAccount.getIdentities();
         if (mIdentityIndex == -1)
         {
diff --git a/src/com/fsck/k9/activity/MessageCompose.java b/src/com/fsck/k9/activity/MessageCompose.java
index f86d460fe..c26360819 100644
--- a/src/com/fsck/k9/activity/MessageCompose.java
+++ b/src/com/fsck/k9/activity/MessageCompose.java
@@ -707,6 +707,12 @@ public class MessageCompose extends K9Activity implements OnClickListener, OnFoc
         message.setSubject(mSubjectView.getText().toString());
         message.setHeader("X-User-Agent", getString(R.string.message_header_mua));
 
+        final String replyTo = mIdentity.getReplyTo();
+        if (replyTo != null)
+        {
+            message.setReplyTo(new Address[] { new Address(replyTo) });
+        }
+
         if (mInReplyTo != null)
         {
             message.setInReplyTo(mInReplyTo);