app.py

@@ -331,7 +331,6 @@ def admin():
         photos=photos,
         accent_color=config["appearance"]["accent_color"],
         config=config,
-        nonce=g.csp_nonce,
     )
 
 @app.route("/admin/logout")
@@ -462,22 +461,17 @@ def admin_upload():
         file_path = os.path.join(app.config["UPLOAD_FOLDER"], filename)
         file.save(file_path)
 
-        # Extract EXIF data with error handling
+        # Extract EXIF data
-        exif = {}
+        exif = None
         exifraw = None
-        width = height = 0
+        with Image.open(file_path) as img:
-        try:
+            exifraw = img.info["exif"]
-            with Image.open(file_path) as img:
+            width, height = img.size
-                width, height = img.size
+            exif = {
-                if hasattr(img, '_getexif') and img._getexif() is not None:
+                ExifTags.TAGS[k]: v
-                    exifraw = img.info.get("exif")
+                for k, v in img._getexif().items()
-                    exif = {
+                if k in ExifTags.TAGS
-                        ExifTags.TAGS[k]: v
+            }
-                        for k, v in img._getexif().items()
-                        if k in ExifTags.TAGS
-                    }
-        except Exception as e:
-            logger.warning(f"Error reading EXIF data for {filename}: {str(e)}")
 
         # Generate a unique key for the image
         unique_key = hashlib.sha256(
@@ -495,25 +489,25 @@ def admin_upload():
         # Generate thumbnails
         generate_thumbnails(filename)
 
-        # Handle exposure time with error handling
+        # Get image dimensions
-        try:
+        with Image.open(file_path) as img:
-            exposure_time = exif.get("ExposureTime", 0)
+            width, height = img.size
-            if isinstance(exposure_time, tuple):
-                exposure_fraction = f"{exposure_time[0]}/{exposure_time[1]}"
-            else:
-                exposure_fraction = f"1/{int(1/float(exposure_time))}" if exposure_time else "0"
-        except (TypeError, ZeroDivisionError):
-            exposure_fraction = "0"
 
-        # Create database entry with safe defaults
+        exposure_time = exif["ExposureTime"]
+        if isinstance(exposure_time, tuple):
+            exposure_fraction = f"{exposure_time[0]}/{exposure_time[1]}"
+        else:
+            exposure_fraction = f"1/{int(1/float(exposure_time))}"
+
+        # Create database entry
         db_session = DBSession()
         new_photo = Photo(
             input_filename=filename,
             thumbnail_filename=f"{os.path.splitext(filename)[0]}/256_{filename}",
             focal_length=str(
-                exif.get("FocalLengthIn35mmFilm", exif.get("FocalLength", "0"))
+                exif.get("FocalLengthIn35mmFilm", exif.get("FocalLength", ""))
             ),
-            aperture=str(exif.get("FNumber", "0")),
+            aperture=str(exif.get("FNumber", "")),
             shutter_speed=exposure_fraction,
             date_taken=datetime.strptime(
                 str(exif.get("DateTime", "1970:01:01 00:00:00")), "%Y:%m:%d %H:%M:%S"

templates/admin.html

@@ -203,8 +203,8 @@
                     <td class="editable" data-field="iso">{{ photo.iso }}</td>
                     <td>{{ photo.width }}x{{ photo.height }}</td>
                     <td>
-                        <button class="save-btn">Save</button>
+                        <button onclick="saveChanges(this)">Save</button>
-                        <button class="delete-btn">Delete</button>
+                        <button onclick="deletePhoto(this)" class="delete-btn">Delete</button>
                     </td>
                 </tr>
                 {% endfor %}
@@ -241,11 +241,8 @@
                             <input type="text" id="about.location" name="about.location" value="{{ config.about.location }}">
                         </div>
                         <div class="form-group">
-                            <label for="about.profile_image">Profile Image:</label>
+                            <label for="about.profile_image">Profile Image Path:</label>
-                            <div style="display: flex; align-items: center; gap: 1rem;">
+                            <input type="text" id="about.profile_image" name="about.profile_image" value="{{ config.about.profile_image }}">
-                                <img id="profile-preview" src="/static/profile.jpeg" alt="Profile" style="width: 100px; height: 100px; object-fit: cover; border-radius: 50%;">
-                                <input type="file" class="profile-image-upload" accept="image/jpeg,image/png" style="flex: 1;">
-                            </div>
                         </div>
                         <div class="form-group">
                             <label for="about.bio">Bio (Markdown):</label>
@@ -261,7 +258,7 @@
             </div>
         </div>
     </div>
-    <script nonce="{{ nonce }}">
+    <script nonce="{{ g.csp_nonce }}">
         function makeEditable(element) {
             const value = element.textContent;
             const input = document.createElement('input');
@@ -284,8 +281,7 @@
             });
         });
 
-        function saveChanges(event) {
+        function saveChanges(button) {
-            const button = event.target;
             const row = button.closest('tr');
             const photoId = row.dataset.id;
             const updatedData = {};
@@ -315,9 +311,8 @@
             });
         }
 
-        function deletePhoto(event) {
+        function deletePhoto(button) {
             if (confirm('Are you sure you want to delete this photo?')) {
-                const button = event.target;
                 const row = button.closest('tr');
                 const photoId = row.dataset.id;
 
@@ -340,50 +335,16 @@
             }
         }
 
-        document.querySelectorAll('.delete-btn').forEach(button => {
-            button.addEventListener('click', (event) => deletePhoto(event));
-        });
-
-        document.querySelectorAll('.save-btn').forEach(button => {
-            button.addEventListener('click', (event) => saveChanges(event));
-        });
-
-        document.querySelectorAll('.profile-image-upload').forEach(input => {
-            input.addEventListener('change', async (e) => {
-                const file = e.target.files[0];
-                if (!file) return;
-
-                const formData = new FormData();
-                formData.append('profile_image', file);
-
-                try {
-                    const response = await fetch('/admin/upload_profile', {
-                        method: 'POST',
-                        body: formData
-                    });
-
-                    const result = await response.json();
-                    if (result.success) {
-                        document.getElementById('profile-preview').src = '/static/profile.jpeg?' + new Date().getTime();
-                    } else {
-                        alert('Error uploading profile image: ' + result.error);
-                    }
-                } catch (error) {
-                    alert('Error uploading profile image: ' + error);
-                }
-            });
-        });
-
         document.getElementById('configForm').addEventListener('submit', async (e) => {
             e.preventDefault();
-
+            
             const formData = {};
-            const inputs = e.target.querySelectorAll('input:not([type="file"]), textarea');
+            const inputs = e.target.querySelectorAll('input, textarea');
-
+            
             inputs.forEach(input => {
                 formData[input.name] = input.value;
             });
-
+            
             try {
                 const response = await fetch('/admin/update_config', {
                     method: 'POST',
@@ -392,9 +353,9 @@
                     },
                     body: JSON.stringify(formData)
                 });
-
+                
                 const result = await response.json();
-
+                
                 if (result.success) {
                     alert('Configuration saved successfully');
                     location.reload();