Fix take photo option for choose media

Stephen Paul Weber created 11 months ago

Change summary

src/main/java/eu/siacs/conversations/ui/ConversationFragment.java | 11 
1 file changed, 9 insertions(+), 2 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/ui/ConversationFragment.java

@@ -1196,13 +1196,19 @@ public class ConversationFragment extends XmppFragment
             case REQUEST_START_VIDEO_CALL:
                 triggerRtpSession(RtpSessionActivity.ACTION_MAKE_VIDEO_CALL);
                 break;
-            case ATTACHMENT_CHOICE_CHOOSE_IMAGE:
+            case ATTACHMENT_CHOICE_CHOOSE_IMAGE: {
+                final Uri takePhotoUri = pendingTakePhotoUri.pop();
+                if (takePhotoUri != null && data.getData() == null && data.getClipData() == null) {
+                    mediaPreviewAdapter.addMediaPreviews(
+                            Attachment.of(getActivity(), takePhotoUri, Attachment.Type.IMAGE));
+                }
                 final List<Attachment> imageUris =
                         Attachment.extractAttachments(getActivity(), data, Attachment.Type.IMAGE);
                 mediaPreviewAdapter.addMediaPreviews(imageUris);
                 toggleInputMethod();
                 break;
-            case ATTACHMENT_CHOICE_TAKE_PHOTO:
+            }
+            case ATTACHMENT_CHOICE_TAKE_PHOTO: {
                 final Uri takePhotoUri = pendingTakePhotoUri.pop();
                 if (takePhotoUri != null) {
                     mediaPreviewAdapter.addMediaPreviews(
@@ -1212,6 +1218,7 @@ public class ConversationFragment extends XmppFragment
                     Log.d(Config.LOGTAG, "lost take photo uri. unable to to attach");
                 }
                 break;
+            }
             case ATTACHMENT_CHOICE_CHOOSE_FILE:
             case ATTACHMENT_CHOICE_RECORD_VIDEO:
             case ATTACHMENT_CHOICE_RECORD_VOICE: