From 9c05cb6af034882c15d16f07f5f7733e08b4c738 Mon Sep 17 00:00:00 2001
From: Philip Sargent <philip.sargent@gmail.com>
Date: Sat, 23 Sep 2023 00:48:35 +0300
Subject: rotatable images & text

---
 core/views/uploads.py | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

(limited to 'core/views/uploads.py')

diff --git a/core/views/uploads.py b/core/views/uploads.py
index dd9f60d..8bec24e 100644
--- a/core/views/uploads.py
+++ b/core/views/uploads.py
@@ -406,6 +406,16 @@ def expofilerename(request, filepath):
     Currently this just does files within wallets i.e. in /surveyscans/
     and it returns control to the original wallet edit page
     """
+    def is_rotatable(path):
+        """If file is a JPG but has no filename extension, then it must be renamed 
+        before it can be rotated.
+        """
+        print(f"{path}: '{Path(path).suffix.lower()}'")
+        if Path(path).suffix.lower() in [".png", ".jpg", ".jpeg"]:
+            return True
+        else:
+            return False
+    
     def rotate_image():
         wallet = str(Path(filepath).parent).lstrip("surveyscans/")
         cwd = settings.SCANS_ROOT / wallet
@@ -455,7 +465,9 @@ def expofilerename(request, filepath):
                 "filesize": filesize,
                 "files": files,
                 "walletpath": walletpath,
+                "wallet": wallet,
                 "notpics": notpics,
+                "rotatable": rotatable,
             },
         )
         
@@ -474,8 +486,11 @@ def expofilerename(request, filepath):
     else:
         filename = Path(filepath).name
         walletpath = Path(filepath).parent
+        wallet = Path(walletpath).name
         folder = actualpath.parent
         filesize = f"{actualpath.stat().st_size:,}"
+        rotatable= is_rotatable(filename)
+ 
 
     if not actualpath.is_relative_to(Path(settings.SCANS_ROOT)):
         message = f'\n Can only do rename within wallets (expofiles/surveyscans/) currently, sorry. "{actualpath}" '
@@ -494,10 +509,10 @@ def expofilerename(request, filepath):
                     for d in f.iterdir():
                         dirs.append(f"{f.name}/{d.name}")
                 if f.is_file():
-                    if Path(f.name).suffix.lower() in [".jpg", ".jpeg", ".png"]:
+                    if is_rotatable(f.name): # should allow all images here which can be thumsized, not just rotatables. e.g. PDF
                         files.append(f.name)
                     else:
-                         notpics.append(f.name)     
+                        notpics.append(f.name)     
         except FileNotFoundError:
             files.append(
                 "(Error. There should be at least one filename visible here. Try refresh.)"
@@ -537,6 +552,7 @@ def expofilerename(request, filepath):
                         "filesize": filesize,
                         "files": files,
                         "walletpath": walletpath,
+                        "wallet": wallet,
                         "notpics": notpics,
                         "rename_bad": rename_bad,
                     },
-- 
cgit v1.2.3