1
0
mirror of https://github.com/moparisthebest/k-9 synced 2024-08-13 17:03:48 -04:00

Fix a number of assumptions about accounts in MessageLists.

Fixes Issue 1079
This commit is contained in:
Jesse Vincent 2010-01-24 22:26:45 +00:00
parent 9218a1f15e
commit 13395ff83c

View File

@ -787,7 +787,7 @@ public class MessageList
private void onDelete(MessageInfoHolder holder, int position) private void onDelete(MessageInfoHolder holder, int position)
{ {
mAdapter.removeMessage(holder); mAdapter.removeMessage(holder);
mController.deleteMessages(mAccount, holder.message.getFolder().getName(), new Message[] { holder.message }, null); mController.deleteMessages(holder.account, holder.message.getFolder().getName(), new Message[] { holder.message }, null);
mListView.setSelection(position); mListView.setSelection(position);
} }
@ -795,7 +795,7 @@ public class MessageList
private void onMove(MessageInfoHolder holder) private void onMove(MessageInfoHolder holder)
{ {
if (mController.isMoveCapable(mAccount) == false) if (mController.isMoveCapable(holder.account) == false)
{ {
return; return;
} }
@ -809,7 +809,7 @@ public class MessageList
Intent intent = new Intent(this, ChooseFolder.class); Intent intent = new Intent(this, ChooseFolder.class);
intent.putExtra(ChooseFolder.EXTRA_ACCOUNT, mAccount); intent.putExtra(ChooseFolder.EXTRA_ACCOUNT, holder.account);
intent.putExtra(ChooseFolder.EXTRA_CUR_FOLDER, holder.folder.name); intent.putExtra(ChooseFolder.EXTRA_CUR_FOLDER, holder.folder.name);
intent.putExtra(ChooseFolder.EXTRA_MESSAGE_UID, holder.message.getUid()); intent.putExtra(ChooseFolder.EXTRA_MESSAGE_UID, holder.message.getUid());
startActivityForResult(intent, ACTIVITY_CHOOSE_FOLDER_MOVE); startActivityForResult(intent, ACTIVITY_CHOOSE_FOLDER_MOVE);
@ -817,7 +817,7 @@ public class MessageList
private void onCopy(MessageInfoHolder holder) private void onCopy(MessageInfoHolder holder)
{ {
if (mController.isCopyCapable(mAccount) == false) if (mController.isCopyCapable(holder.account) == false)
{ {
return; return;
} }
@ -831,7 +831,7 @@ public class MessageList
Intent intent = new Intent(this, ChooseFolder.class); Intent intent = new Intent(this, ChooseFolder.class);
intent.putExtra(ChooseFolder.EXTRA_ACCOUNT, mAccount); intent.putExtra(ChooseFolder.EXTRA_ACCOUNT, holder.account);
intent.putExtra(ChooseFolder.EXTRA_CUR_FOLDER, holder.folder.name); intent.putExtra(ChooseFolder.EXTRA_CUR_FOLDER, holder.folder.name);
intent.putExtra(ChooseFolder.EXTRA_MESSAGE_UID, holder.message.getUid()); intent.putExtra(ChooseFolder.EXTRA_MESSAGE_UID, holder.message.getUid());
startActivityForResult(intent, ACTIVITY_CHOOSE_FOLDER_COPY); startActivityForResult(intent, ACTIVITY_CHOOSE_FOLDER_COPY);
@ -877,19 +877,19 @@ public class MessageList
private void onMoveChosen(MessageInfoHolder holder, String folderName) private void onMoveChosen(MessageInfoHolder holder, String folderName)
{ {
if (mController.isMoveCapable(mAccount) == true && folderName != null) if (mController.isMoveCapable(holder.account) == true && folderName != null)
{ {
mAdapter.removeMessage(holder); mAdapter.removeMessage(holder);
mController.moveMessage(mAccount, holder.message.getFolder().getName(), holder.message, folderName, null); mController.moveMessage(holder.account, holder.message.getFolder().getName(), holder.message, folderName, null);
} }
} }
private void onCopyChosen(MessageInfoHolder holder, String folderName) private void onCopyChosen(MessageInfoHolder holder, String folderName)
{ {
if (mController.isCopyCapable(mAccount) == true && folderName != null) if (mController.isCopyCapable(holder.account) == true && folderName != null)
{ {
mController.copyMessage(mAccount, mController.copyMessage(holder.account,
holder.message.getFolder().getName(), holder.message, folderName, null); holder.message.getFolder().getName(), holder.message, folderName, null);
} }
} }
@ -897,17 +897,17 @@ public class MessageList
private void onReply(MessageInfoHolder holder) private void onReply(MessageInfoHolder holder)
{ {
MessageCompose.actionReply(this, mAccount, holder.message, false); MessageCompose.actionReply(this, holder.account, holder.message, false);
} }
private void onReplyAll(MessageInfoHolder holder) private void onReplyAll(MessageInfoHolder holder)
{ {
MessageCompose.actionReply(this, mAccount, holder.message, true); MessageCompose.actionReply(this, holder.account, holder.message, true);
} }
private void onForward(MessageInfoHolder holder) private void onForward(MessageInfoHolder holder)
{ {
MessageCompose.actionForward(this, mAccount, holder.message); MessageCompose.actionForward(this, holder.account, holder.message);
} }
private void onMarkAllAsRead(final Account account, final String folder) private void onMarkAllAsRead(final Account account, final String folder)
@ -1393,12 +1393,12 @@ public class MessageList
menu.findItem(R.id.flag).setTitle(R.string.unflag_action); menu.findItem(R.id.flag).setTitle(R.string.unflag_action);
} }
if (mController.isCopyCapable(mAccount) == false) if (mController.isCopyCapable(message.account) == false)
{ {
menu.findItem(R.id.copy).setVisible(false); menu.findItem(R.id.copy).setVisible(false);
} }
if (mController.isMoveCapable(mAccount) == false) if (mController.isMoveCapable(message.account) == false)
{ {
menu.findItem(R.id.move).setVisible(false); menu.findItem(R.id.move).setVisible(false);
} }