From 36990d46cf231dae0c194523f2c21eb04abce343 Mon Sep 17 00:00:00 2001
From: Ben Komalo <benkomalo@google.com>
Date: Mon, 18 Apr 2011 09:48:58 -0700
Subject: [PATCH] Fixup auto advance policy handling.

The phone code has gone stale and ignored auto advance policies - this
fixes it, though that code is likely to get merged anyways.

Makes it so that auto-advancing to the end of the list pops back out to
the message list

Bug: 4302999
Change-Id: I609747c0a19672b8aa56d2c0ab57533823899e47
---
 .../MessageListXLFragmentManager.java         | 18 +++++-------
 .../android/email/activity/MessageView.java   | 29 +++++++++++++++----
 2 files changed, 32 insertions(+), 15 deletions(-)

diff --git a/src/com/android/email/activity/MessageListXLFragmentManager.java b/src/com/android/email/activity/MessageListXLFragmentManager.java
index 0ff7cb8c5..21b91be2d 100644
--- a/src/com/android/email/activity/MessageListXLFragmentManager.java
+++ b/src/com/android/email/activity/MessageListXLFragmentManager.java
@@ -37,8 +37,6 @@ import java.util.Set;
  * A class manages what are showing on {@link MessageListXL} (i.e. account id, mailbox id, and
  * message id), and show/hide fragments accordingly.
  *
- * TODO Highlight selected message on message list
- *
  * TODO: Test it.  It's testable if we implement MockFragmentTransaction, which may be too early
  * to do so at this point.  (API may not be stable enough yet.)
  *
@@ -152,7 +150,14 @@ class MessageListXLFragmentManager implements
         if (Email.DEBUG_LIFECYCLE && Email.DEBUG) {
             Log.d(Logging.LOG_TAG, "MessageListXLFragmentManager#onMailboxNotFound()");
         }
-        // Shouldn't happen
+        // TODO: handle more gracefully.
+        Log.e(Logging.LOG_TAG, "unable to find mailbox for account " + accountId);
+    }
+
+    @Override
+    public void onMailboxNotFound() {
+        // TODO: handle more gracefully.
+        Log.e(Logging.LOG_TAG, "unable to find mailbox");
     }
 
     // MoveMessageToDialog$Callback
@@ -209,11 +214,6 @@ class MessageListXLFragmentManager implements
         }
     }
 
-    @Override
-    public void onMailboxNotFound() {
-        // TODO: What to do??
-    }
-
     @Override
     public void onEnterSelectionMode(boolean enter) {
     }
@@ -652,11 +652,9 @@ class MessageListXLFragmentManager implements
         switch (Preferences.getPreferences(mActivity).getAutoAdvanceDirection()) {
             case Preferences.AUTO_ADVANCE_NEWER:
                 if (moveToNewer()) return;
-                if (moveToOlder()) return;
                 break;
             case Preferences.AUTO_ADVANCE_OLDER:
                 if (moveToOlder()) return;
-                if (moveToNewer()) return;
                 break;
         }
         // Last message in the box or AUTO_ADVANCE_MESSAGE_LIST.  Go back to message list.
diff --git a/src/com/android/email/activity/MessageView.java b/src/com/android/email/activity/MessageView.java
index 87746956d..577814376 100644
--- a/src/com/android/email/activity/MessageView.java
+++ b/src/com/android/email/activity/MessageView.java
@@ -17,6 +17,7 @@
 package com.android.email.activity;
 
 import com.android.email.Email;
+import com.android.email.Preferences;
 import com.android.email.R;
 import com.android.emailcommon.Logging;
 import com.android.emailcommon.provider.EmailContent.Mailbox;
@@ -169,7 +170,8 @@ public class MessageView extends MessageViewBase implements View.OnClickListener
         // the delete triggers mCursorObserver in MessageOrderManager.
         // first move to older/newer before the actual delete
         long messageIdToDelete = mMessageId;
-        boolean moved = moveToOlder() || moveToNewer(); // TODO use "auto-advance" preference
+
+        boolean moved = autoAdvance();
         ActivityHelper.deleteMessage(this, messageIdToDelete);
         if (!moved) {
             // this generates a benign warning "Duplicate finish request" because
@@ -179,6 +181,22 @@ public class MessageView extends MessageViewBase implements View.OnClickListener
         }
     }
 
+    /**
+     * Auto-advances the message being shown according to the auto-advance policy set in preferences
+     * @return Whether or not a new message was selected. This will return false either if there are
+     *     no appropriate messages to advance to, or if the preferences indicate we should not
+     *     auto-advance
+     */
+    private boolean autoAdvance() {
+        switch (Preferences.getPreferences(this).getAutoAdvanceDirection()) {
+            case Preferences.AUTO_ADVANCE_NEWER:
+                return moveToNewer();
+            case Preferences.AUTO_ADVANCE_OLDER:
+                return moveToOlder();
+        }
+        return false;
+    }
+
     private boolean moveToOlder() {
         if (mOrderManager != null && mOrderManager.moveToOlder()) {
             mMessageId = mOrderManager.getCurrentMessageId();
@@ -258,7 +276,9 @@ public class MessageView extends MessageViewBase implements View.OnClickListener
 
     @Override
     public void onMessageSetUnread() {
-        finish();
+        if (!autoAdvance()) {
+            finish();
+        }
     }
 
     private void enableForwardReply(boolean enabled) {
@@ -319,9 +339,8 @@ public class MessageView extends MessageViewBase implements View.OnClickListener
 
     @Override
     public void onRespondedToInvite(int response) {
-        // TODO use "auto-advance" preference
-        if (!moveToOlder()) {
-            finish(); // if this is the last message, move up to message-list.
+        if (!autoAdvance()) {
+            finish();
         }
     }