From 5bb959448c9ed8700c4a897391e5f1e322cd2387 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Fri, 5 Apr 2019 14:12:56 +0200
Subject: [PATCH] Prevent crash

---
 .../eu/faircode/email/AdapterMessage.java     | 20 ++++++++++++-------
 .../main/java/eu/faircode/email/Helper.java   |  5 +++++
 2 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/AdapterMessage.java b/app/src/main/java/eu/faircode/email/AdapterMessage.java
index fc51009cec..d547ab1482 100644
--- a/app/src/main/java/eu/faircode/email/AdapterMessage.java
+++ b/app/src/main/java/eu/faircode/email/AdapterMessage.java
@@ -2431,13 +2431,18 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
 
                         @Override
                         public void onPageFinished(WebView view, String url) {
-                            PrintManager printManager = (PrintManager) context.getSystemService(Context.PRINT_SERVICE);
-                            String jobName = context.getString(R.string.app_name);
-                            if (!TextUtils.isEmpty(data.message.subject))
-                                jobName += " - " + data.message.subject;
-                            PrintDocumentAdapter adapter = printWebView.createPrintDocumentAdapter(jobName);
-                            printManager.print(jobName, adapter, new PrintAttributes.Builder().build());
-                            printWebView = null;
+                            try {
+                                PrintManager printManager = (PrintManager) context.getSystemService(Context.PRINT_SERVICE);
+                                String jobName = context.getString(R.string.app_name);
+                                if (!TextUtils.isEmpty(data.message.subject))
+                                    jobName += " - " + data.message.subject;
+                                PrintDocumentAdapter adapter = printWebView.createPrintDocumentAdapter(jobName);
+                                printManager.print(jobName, adapter, new PrintAttributes.Builder().build());
+                            } catch (Throwable ex) {
+                                Log.e(ex);
+                            } finally {
+                                printWebView = null;
+                            }
                         }
                     });
 
@@ -2557,6 +2562,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
 
             popupMenu.getMenu().findItem(R.id.menu_share).setEnabled(data.message.content);
             popupMenu.getMenu().findItem(R.id.menu_print).setEnabled(hasWebView && data.message.content);
+            popupMenu.getMenu().findItem(R.id.menu_print).setVisible(Helper.canPrint(context));
 
             popupMenu.getMenu().findItem(R.id.menu_show_headers).setChecked(show_headers);
             popupMenu.getMenu().findItem(R.id.menu_show_headers).setEnabled(data.message.uid != null);
diff --git a/app/src/main/java/eu/faircode/email/Helper.java b/app/src/main/java/eu/faircode/email/Helper.java
index adfc2d6b26..bd1aa7ec48 100644
--- a/app/src/main/java/eu/faircode/email/Helper.java
+++ b/app/src/main/java/eu/faircode/email/Helper.java
@@ -1035,6 +1035,11 @@ public class Helper {
             return false;
     }
 
+    static boolean canPrint(Context context) {
+        PackageManager pm = context.getPackageManager();
+        return pm.hasSystemFeature("android.software.print");
+    }
+
     public static String getFingerprint(Context context) {
         try {
             PackageManager pm = context.getPackageManager();