From fae1c0666dc1508a449f5783d113abbdc9588606 Mon Sep 17 00:00:00 2001
From: Damien De Paoli <ddp@depaoli.id.au>
Date: Thu, 20 Jan 2022 10:16:38 +1100
Subject: [PATCH] more back-forward button cleanups

---
 person.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/person.py b/person.py
index 8dcb578..8dd84d6 100644
--- a/person.py
+++ b/person.py
@@ -116,9 +116,7 @@ def new_person():
     form = PersonForm(request.form)
     page_title='Create new Person'
 
-    if 'surname' not in request.form:
-        return render_template("person.html", person=None, form=form, page_title=page_title )
-    else:
+    if request.method='POST':
         person = Person( tag=request.form["tag"], surname=request.form["surname"], firstname=request.form["firstname"] )
         try:
             db.session.add(person)
@@ -127,7 +125,9 @@ def new_person():
             return redirect( url_for( 'person', id=person.id) )
         except SQLAlchemyError as e:
             st.SetMessage( f"<b>Failed to add Person:</b>&nbsp;{e.orig}", "danger" )
-            return render_template("person.html", person=person, form=form, page_title = page_title)
+            return redirect( url_for( '/persons') )
+    else:
+        return render_template("person.html", person=None, form=form, page_title=page_title )
 
 ################################################################################
 # /person/<id> -> GET/POST(save or delete) -> shows/edits/delets a single person
@@ -175,7 +175,7 @@ def person(id):
             return redirect( url_for( 'person', id=person.id) )
         except SQLAlchemyError as e:
             st.SetMessage( f"<b>Failed to modify Person:</b>&nbsp;{e}", "danger" )
-            return render_template("person.html", form=form, page_title=page_title)
+            return redirect( url_for( 'persons' ) )
     else:
         person = Person.query.get(id)
         if not person: