From 47b076b41ded071731c511b17bef279898fd3845 Mon Sep 17 00:00:00 2001
From: Sylvia van Os <sylvia@hackerchick.me>
Date: Sun, 16 Jun 2024 12:57:59 +0200
Subject: [PATCH] Support setting front or back image as thumbnail

---
 .../card_locker/LoyaltyCardEditActivity.java  | 63 ++++++++++++++-----
 app/src/main/res/values/strings.xml           |  2 +
 2 files changed, 51 insertions(+), 14 deletions(-)

diff --git a/app/src/main/java/protect/card_locker/LoyaltyCardEditActivity.java b/app/src/main/java/protect/card_locker/LoyaltyCardEditActivity.java
index 5d8e55032..bd1213120 100644
--- a/app/src/main/java/protect/card_locker/LoyaltyCardEditActivity.java
+++ b/app/src/main/java/protect/card_locker/LoyaltyCardEditActivity.java
@@ -681,16 +681,17 @@ public class LoyaltyCardEditActivity extends CatimaAppCompatActivity implements
 
                 if (bitmap != null) {
                     if (requestedFrontImage()) {
+                        mFrontImageRemoved = false;
                         mFrontImageUnsaved = true;
                         setCardImage(cardImageFront, Utils.resizeBitmap(bitmap, Utils.BITMAP_SIZE_BIG), true);
                     } else if (requestedBackImage()) {
+                        mBackImageRemoved = false;
                         mBackImageUnsaved = true;
                         setCardImage(cardImageBack, Utils.resizeBitmap(bitmap, Utils.BITMAP_SIZE_BIG), true);
                     } else {
+                        mIconRemoved = false;
                         mIconUnsaved = true;
-                        setCardImage(thumbnail, Utils.resizeBitmap(bitmap, Utils.BITMAP_SIZE_SMALL), false);
-                        thumbnail.setBackgroundColor(Color.TRANSPARENT);
-                        setColorFromIcon();
+                        setThumbnailImage(Utils.resizeBitmap(bitmap, Utils.BITMAP_SIZE_SMALL));
                     }
                     Log.d("cropper", "mRequestedImage: " + mRequestedImage);
                     mCropperFinishedType = mRequestedImage;
@@ -834,7 +835,7 @@ public class LoyaltyCardEditActivity extends CatimaAppCompatActivity implements
                     setCardImage(cardImageBack, Utils.retrieveCardImage(this, tempLoyaltyCard.id, ImageLocationType.back), true);
                 }
                 if (!mIconUnsaved && !croppedIcon() && !mIconRemoved) {
-                    setCardImage(thumbnail, Utils.retrieveCardImage(this, tempLoyaltyCard.id, ImageLocationType.icon), false);
+                    setThumbnailImage(Utils.retrieveCardImage(this, tempLoyaltyCard.id, ImageLocationType.icon));
                 }
             } else {
                 setTitle(R.string.addCardTitle);
@@ -847,7 +848,7 @@ public class LoyaltyCardEditActivity extends CatimaAppCompatActivity implements
                 setCardImage(cardImageBack, Utils.loadTempImage(this, TEMP_UNSAVED_BACK_IMAGE_NAME), true);
             }
             if (mIconUnsaved && !croppedIcon()) {
-                setCardImage(thumbnail, Utils.loadTempImage(this, TEMP_UNSAVED_ICON_NAME), false);
+                setThumbnailImage(Utils.loadTempImage(this, TEMP_UNSAVED_ICON_NAME));
             }
         }
 
@@ -998,6 +999,22 @@ public class LoyaltyCardEditActivity extends CatimaAppCompatActivity implements
         }
     }
 
+    protected void setThumbnailImage(@Nullable Bitmap bitmap) {
+        setCardImage(thumbnail, bitmap, false);
+
+        if (bitmap == null) {
+            generateIcon(storeFieldEdit.getText().toString().trim());
+        } else {
+            thumbnail.setBackgroundColor(Color.TRANSPARENT);
+            setColorFromIcon();
+        }
+
+        if (tempLoyaltyCard.headerColor != null) {
+            thumbnailEditIcon.setBackgroundColor(Utils.needsDarkForeground(tempLoyaltyCard.headerColor) ? Color.BLACK : Color.WHITE);
+            thumbnailEditIcon.setColorFilter(Utils.needsDarkForeground(tempLoyaltyCard.headerColor) ? Color.WHITE : Color.BLACK);
+        }
+    }
+
     protected void setCardImage(ImageView imageView, Bitmap bitmap, boolean applyFallback) {
         imageView.setTag(bitmap);
 
@@ -1325,6 +1342,28 @@ public class LoyaltyCardEditActivity extends CatimaAppCompatActivity implements
                 return null;
             });
 
+            if (v.getId() == R.id.thumbnail) {
+                if (cardImageFront.getTag() instanceof Bitmap) {
+                    cardOptions.put(getString(R.string.useFrontImage), () -> {
+                        mIconRemoved = false;
+                        mIconUnsaved = true;
+                        setThumbnailImage(Utils.resizeBitmap((Bitmap) cardImageFront.getTag(), Utils.BITMAP_SIZE_SMALL));
+
+                        return null;
+                    });
+                }
+
+                if (cardImageBack.getTag() instanceof Bitmap) {
+                    cardOptions.put(getString(R.string.useBackImage), () -> {
+                        mIconRemoved = false;
+                        mIconUnsaved = true;
+                        setThumbnailImage(Utils.resizeBitmap((Bitmap) cardImageBack.getTag(), Utils.BITMAP_SIZE_SMALL));
+
+                        return null;
+                    });
+                }
+            }
+
             int titleResource;
 
             if (v.getId() == R.id.frontImageHolder) {
@@ -1365,17 +1404,13 @@ public class LoyaltyCardEditActivity extends CatimaAppCompatActivity implements
     // We don't need to set or check the dialogId since it's only used for that single dialog
     @Override
     public void onColorSelected(int dialogId, int color) {
-        // Unset image if set
-        setCardImage(thumbnail, null, false);
-        mIconRemoved = true;
-        mIconUnsaved = false;
-
+        // Save new colour
         updateTempState(LoyaltyCardField.headerColor, color);
 
-        thumbnailEditIcon.setBackgroundColor(Utils.needsDarkForeground(color) ? Color.BLACK : Color.WHITE);
-        thumbnailEditIcon.setColorFilter(Utils.needsDarkForeground(color) ? Color.WHITE : Color.BLACK);
-
-        generateIcon(storeFieldEdit.getText().toString().trim());
+        // Unset image if set
+        mIconRemoved = true;
+        mIconUnsaved = false;
+        setThumbnailImage(null);
     }
 
     // ColorPickerDialogListener callback
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 60580b163..da2429652 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -349,4 +349,6 @@
     <string name="noCameraFoundGuideText">Your device does not seem to have a camera. If it does, try rebooting the device. Otherwise, use the "More options" button below to add a barcode another way.</string>
     <string name="importCancelled">Import cancelled</string>
     <string name="exportCancelled">Export cancelled</string>
+    <string name="useFrontImage">Use front image</string>
+    <string name="useBackImage">Use back image</string>
 </resources>