show toast if no application found to attach

Daniel Gultsch created 5 years ago

Change summary

src/main/java/eu/siacs/conversations/ui/ConversationFragment.java | 10 
src/main/res/values/strings.xml                                   |  1 
2 files changed, 9 insertions(+), 2 deletions(-)

Detailed changes

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

@@ -1048,10 +1048,11 @@ public class ConversationFragment extends XmppFragment implements EditMessage.Ke
 
         return binding.getRoot();
     }
+
     @Override
     public void onDestroyView() {
         super.onDestroyView();
-        Log.d(Config.LOGTAG,"ConversationFragment.onDestroyView()");
+        Log.d(Config.LOGTAG, "ConversationFragment.onDestroyView()");
         messageListAdapter.setOnContactPictureClicked(null);
         messageListAdapter.setOnContactPictureLongClicked(null);
         messageListAdapter.setOnQuoteListener(null);
@@ -1729,7 +1730,10 @@ public class ConversationFragment extends XmppFragment implements EditMessage.Ke
                 break;
         }
         final Context context = getActivity();
-        if (context != null && intent.resolveActivity(context.getPackageManager()) != null) {
+        if (context == null) {
+            return;
+        }
+        if (intent.resolveActivity(context.getPackageManager()) != null) {
             if (chooser) {
                 startActivityForResult(
                         Intent.createChooser(intent, getString(R.string.perform_action_with)),
@@ -1737,6 +1741,8 @@ public class ConversationFragment extends XmppFragment implements EditMessage.Ke
             } else {
                 startActivityForResult(intent, attachmentChoice);
             }
+        } else {
+            Toast.makeText(context, R.string.no_application_found, Toast.LENGTH_LONG).show();
         }
     }
 

src/main/res/values/strings.xml

@@ -949,4 +949,5 @@
     </plurals>
     <string name="failed_deliveries">Failed deliveries</string>
     <string name="more_options">More options</string>
+    <string name="no_application_found">No application found</string>
 </resources>