mirror/monocles_chat_clean
1
0
Fork 1
Code Activity

Catch concurrent modification and abort

Browse source 
Someone added an attachment while we were sending or what?

(cherry picked from commit c4f751e2dd6b0f1bbb56704a3f95e76152989389)
This commit is contained in:
Stephen Paul Weber 2024-11-28 10:21:05 -05:00 • committed by Arne
parent d2ef9e3428
commit 475c01cbc6
1 changed files with 20 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/main/java/eu/siacs/conversations/ui/ConversationFragment.java
View file

@ -1524,6 +1524,7 @@ public class ConversationFragment extends XmppFragment
final Runnable next = new Runnable() { final Runnable next = new Runnable() {
@Override @Override
public void run() { public void run() {
try {
final Attachment attachment = i.next(); final Attachment attachment = i.next();
if (attachment.getType() == Attachment.Type.LOCATION) { if (attachment.getType() == Attachment.Type.LOCATION) {
attachLocationToConversation(conversation, attachment.getUri()); attachLocationToConversation(conversation, attachment.getUri());
@ -1540,6 +1541,10 @@ public class ConversationFragment extends XmppFragment
attachFileToConversation(conversation, attachment.getUri(), attachment.getMime(), i.hasNext() ? this : null); attachFileToConversation(conversation, attachment.getUri(), attachment.getMime(), i.hasNext() ? this : null);
} }
i.remove(); i.remove();
} catch (final java.util.ConcurrentModificationException e) {
// Abort, leave any unsent attachments alone for the user to try again
Toast.makeText(activity, "Sometimes went wrong with some attachments. Try again?", Toast.LENGTH_SHORT).show();
}
mediaPreviewAdapter.notifyDataSetChanged(); mediaPreviewAdapter.notifyDataSetChanged();
toggleInputMethod(); toggleInputMethod();
} }