catch unknown scotsman error

2 files changed, 9 insertions, 26 deletions

diff --git a/core/views/logbooks.py b/core/views/logbooks.py
index 00ac4e4..4abe966 100644
--- a/core/views/logbooks.py
+++ b/core/views/logbooks.py
@@ -115,16 +115,13 @@ class Expeditions_jsonListView(ListView):
 
 
 def person(request, first_name='', last_name='', ):
-    this_person = Person.objects.get(first_name = first_name, last_name = last_name)
+    try:
+        this_person = Person.objects.get(first_name = first_name, last_name = last_name)
+        return render(request,'person.html', {'person': this_person, })
+    except:
+        message = f'Person not found - possibly Scottish? (We have a name parser issue with Mc, Mac etc.)'
+        return render(request, 'errors/generic.html', {'message': message})   
     
-    # This is for removing the reference to the user's profile, in case they set it to the wrong person
-    if request.method == 'GET':
-        if request.GET.get('clear_profile')=='True':
-            this_person.user=None
-            this_person.save()
-            return HttpResponseRedirect(reverse('profiles_select_profile'))
-    
-    return render(request,'person.html', {'person': this_person, })
 
 
 def get_person_chronology(personexpedition):
diff --git a/core/views/other.py b/core/views/other.py
index 569264b..b179f82 100644
--- a/core/views/other.py
+++ b/core/views/other.py
@@ -134,14 +134,12 @@ def ajax_QM_number(request):
 
     return HttpResponse(res)
 
-
-#print(" -  newFile() is next in troggle/core/views/other.py")
-
 @login_required_if_public
 def newFile(request, pslug = None):
+    ''' not known quite what this was for or where it fits in - original 2006 troggle idea never finished?
+    '''
     if pslug:
         previousfile = LogbookEntry.objects.get(slug = pslug, date = previousdate, expedition = expedition)
-        #assert previousfile.filename
     if request.method == 'POST': # If the form has been submitted...
         tripForm = TripForm(request.POST) # A form bound to the POST data
         personTripFormSet = PersonTripFormSet(request.POST)
@@ -175,16 +173,4 @@ def newFile(request, pslug = None):
 
     return render(request, 'editfile.html', {'fileForm': fileform, })
 
-@login_required_if_public
-def deleteFile(request, expeditionyear, date = None, slug = None):
-    expedition = Expedition.objects.get(year=expeditionyear)
-    previousdate = datetime.date(*[int(x) for x in date.split("-")])
-    previouslbe = LogbookEntry.objects.get(slug = slug, date = previousdate, expedition = expedition)
-    delLogbookEntry(previouslbe)
-    return HttpResponseRedirect(reverse('expedition', args=[expedition.year])) # Redirect after POST
-
-def delFile(f):
-    for pt in lbe.persontrip_set.all():
-        pt.delete()
-    lbe.delete()
-    os.remove(lbe.filename)
+