From feaf38aa39eb867b359fa5ceab6fe7d3886fc782 Mon Sep 17 00:00:00 2001
From: Philip Sargent <philip.sargent@gmail.com>
Date: Tue, 23 Jul 2024 09:39:21 +0200
Subject: [PATCH] bug fix

---
 core/views/logbooks.py   | 4 +++-
 templates/logreport.html | 6 +++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/core/views/logbooks.py b/core/views/logbooks.py
index 2c15a1e..5d9dca2 100644
--- a/core/views/logbooks.py
+++ b/core/views/logbooks.py
@@ -223,9 +223,11 @@ def personexpedition(request, slug="", year=""):
 def logentrydelete(request, year):
     """This only gets called by a POST from the logreport page
     
-    The function in memory of James Waite who managed to make so many duplicate  logbook entries
+    This function is dedicated to James Waite who managed to make so many duplicate  logbook entries
     that we needed a sopecial mechanism to delete them.
     """
+    for i in request.POST:
+        print(f" - '{i}'  {request.POST[i]}")
     eslug = request.POST["entry_slug"] 
     entry = LogbookEntry.objects.get(slug=eslug)
     # OK we delete it formt he db and then re-save logbook.html file
diff --git a/templates/logreport.html b/templates/logreport.html
index ed18da2..bdb0eca 100644
--- a/templates/logreport.html
+++ b/templates/logreport.html
@@ -21,7 +21,6 @@
 <p>(Hover mouse over the date to see the slug for the entry.)
 {% if logged_in %}<font color="red">Logged in as expoadmin</font>{% endif %}
 
-<input  type="hidden" value="{{entry.slug}}" name="entry_slug">
 
 <table class="expeditionlogbooks">
 <tr><th>Date</th><th>Logged trips and diary entries</th><th>Cave</th><th>Text..</th><th>Words</th><th>Author</th><th>Who else</th>
@@ -33,7 +32,7 @@
     <tr>
         <td title="{{entry.slug}}">{{date.grouper|date:"D d M Y"}}</td>
         <td>
-           <a href="{{ entry.get_absolute_url }}">{{entry.title|truncatechars:30|safe|striptags}}</a>&nbsp;
+           <a href="{{ entry.get_absolute_url }}">{{entry.title|truncatechars:25|safe|striptags}}</a>&nbsp;
         </td>
         <td>
               {% if entry.cave %}
@@ -60,10 +59,11 @@
                  
         </td> 
 {% if logged_in %}
-<form name="import" method="post" action="/logentrydelete/{{year}}">{% csrf_token %}
+<form name="delete_entry" method="post" action="/logentrydelete/{{year}}">{% csrf_token %}
 <td>
 {{entry.slug}}</td>
 <td>
+<input  type="hidden" value="{{entry.slug}}" name="entry_slug">
 <input type="submit"  value="delete"></td>
 </form>
 {% endif %}