ADHOC/replicant-packages_apps_Email
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge "Fix NPE in move messages"

Browse Source
This commit is contained in:
Ben Komalo 2011-10-17 10:48:38 -07:00 committed by Android (Google) Code Review
commit 2e3d46cbde
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/com/android/email/activity/MessageListFragment.java
View File

@ -21,6 +21,7 @@ import android.app.ListFragment;
import android.app.LoaderManager; import android.app.LoaderManager;
import android.content.ClipData; import android.content.ClipData;
import android.content.ContentUris; import android.content.ContentUris;
import android.content.Context;
import android.content.Loader; import android.content.Loader;
import android.content.res.Configuration; import android.content.res.Configuration;
import android.content.res.Resources; import android.content.res.Resources;
@ -843,8 +844,14 @@ public class MessageListFragment extends ListFragment
@Override @Override
public void onMoveToMailboxSelected(long newMailboxId, long[] messageIds) { public void onMoveToMailboxSelected(long newMailboxId, long[] messageIds) {
final Context context = getActivity();
if (context == null) {
// Detached from activity. This callback was really delayed or a monkey was involved.
return;
}
mCallback.onAdvancingOpAccepted(Utility.toLongSet(messageIds)); mCallback.onAdvancingOpAccepted(Utility.toLongSet(messageIds));
ActivityHelper.moveMessages(getActivity(), newMailboxId, messageIds); ActivityHelper.moveMessages(context, newMailboxId, messageIds);
// Move is async, so we can't refresh now. Instead, just clear the selection. // Move is async, so we can't refresh now. Instead, just clear the selection.
onDeselectAll(); onDeselectAll();