Refactoring

app/src/main/java/eu/faircode/email/AdapterMessage.java

@@ -2557,11 +2557,12 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                         private void onCopy() {
                             ClipboardManager clipboard =
                                     (ClipboardManager) context.getSystemService(Context.CLIPBOARD_SERVICE);
-                            if (clipboard != null) {
+                            if (clipboard == null)
-                                ClipData clip = ClipData.newPlainText(title, text);
+                                return;
-                                clipboard.setPrimaryClip(clip);
+
-                                ToastEx.makeText(context, R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
+                            ClipData clip = ClipData.newPlainText(title, text);
-                            }
+                            clipboard.setPrimaryClip(clip);
+                            ToastEx.makeText(context, R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
                         }
                     });
 
@@ -3718,13 +3719,17 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
         }
 
         private void onActionCopyNote(TupleMessageEx message) {
+            if (TextUtils.isEmpty(message.notes))
+                return;
+
             ClipboardManager clipboard =
                     (ClipboardManager) context.getSystemService(Context.CLIPBOARD_SERVICE);
-            if (clipboard != null && !TextUtils.isEmpty(message.notes)) {
+            if (clipboard == null)
-                ClipData clip = ClipData.newPlainText(context.getString(R.string.app_name), message.notes);
+                return;
-                clipboard.setPrimaryClip(clip);
+
-                ToastEx.makeText(context, R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
+            ClipData clip = ClipData.newPlainText(context.getString(R.string.app_name), message.notes);
-            }
+            clipboard.setPrimaryClip(clip);
+            ToastEx.makeText(context, R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
         }
 
         private void onShow(final TupleMessageEx message, boolean full) {

app/src/main/java/eu/faircode/email/FragmentCompose.java

@@ -655,15 +655,16 @@ public class FragmentCompose extends FragmentBase {
 
                 ClipboardManager clipboard =
                         (ClipboardManager) getContext().getSystemService(Context.CLIPBOARD_SERVICE);
-                if (clipboard != null) {
+                if (clipboard == null)
-                    ClipData clip = ClipData.newHtmlText(
+                    return;
-                            getContext().getString(R.string.title_edit_signature_text),
+
-                            HtmlHelper.getText(getContext(), identity.signature),
+                ClipData clip = ClipData.newHtmlText(
-                            identity.signature);
+                        getContext().getString(R.string.title_edit_signature_text),
-                    clipboard.setPrimaryClip(clip);
+                        HtmlHelper.getText(getContext(), identity.signature),
-
+                        identity.signature);
-                    ToastEx.makeText(getContext(), R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
+                clipboard.setPrimaryClip(clip);
-                }
+
+                ToastEx.makeText(getContext(), R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
             }
         });
 

app/src/main/java/eu/faircode/email/FragmentDialogOpenLink.java

@@ -211,12 +211,13 @@ public class FragmentDialogOpenLink extends FragmentDialogBase {
             public void onClick(View v) {
                 ClipboardManager clipboard =
                         (ClipboardManager) context.getSystemService(Context.CLIPBOARD_SERVICE);
-                if (clipboard != null) {
+                if (clipboard == null)
-                    ClipData clip = ClipData.newPlainText(title, etLink.getText().toString());
+                    return;
-                    clipboard.setPrimaryClip(clip);
 
-                    ToastEx.makeText(context, R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
+                ClipData clip = ClipData.newPlainText(title, etLink.getText().toString());
-                }
+                clipboard.setPrimaryClip(clip);
+
+                ToastEx.makeText(context, R.string.title_clipboard_copied, Toast.LENGTH_LONG).show();
             }
         });