Merge pull request #33 from realpython/flask-connexion-rest-part-3

Flask connexion rest part 3

Author: dbader

flask-connexion-rest-part-3/models.py

@@ -14,7 +14,6 @@ class Person(db.Model):
     notes = db.relationship(
         "Note",
         backref="person",
-        lazy="joined",
         cascade="all, delete, delete-orphan",
         single_parent=True,
         order_by="desc(Note.timestamp)",

flask-connexion-rest-part-3/notes.py

@@ -50,7 +50,7 @@ def read_one(person_id, note_id):
 
     # Otherwise, nope, didn't find that note
     else:
-        abort(404, "Note not found for Id: {note_id}".format(note_id=note_id))
+        abort(404, f"Note not found for Id: {note_id}")
 
 
 def create(person_id, note):
@@ -66,10 +66,7 @@ def create(person_id, note):
 
     # Was a person found?
     if person is None:
-        abort(
-            404,
-            "Person not found for Id: {person_id}".format(person_id=person_id),
-        )
+        abort(404, f"Person not found for Id: {person_id}")
 
     # Create a note schema instance
     schema = NoteSchema()
@@ -123,7 +120,7 @@ def update(person_id, note_id, note):
 
     # Otherwise, nope, didn't find that note
     else:
-        abort(404, "Note not found for Id: {note_id}".format(note_id=note_id))
+        abort(404, f"Note not found for Id: {note_id}")
 
 
 def delete(person_id, note_id):
@@ -151,4 +148,4 @@ def delete(person_id, note_id):
 
     # Otherwise, nope, didn't find that note
     else:
-        abort(404, "Note not found for Id: {note_id}".format(note_id=note_id))
+        abort(404, f"Note not found for Id: {note_id}")

flask-connexion-rest-part-3/people.py

@@ -49,10 +49,7 @@ def read_one(person_id):
 
     # Otherwise, nope, didn't find that person
     else:
-        abort(
-            404,
-            "Person not found for Id: {person_id}".format(person_id=person_id),
-        )
+        abort(404, f"Person not found for Id: {person_id}")
 
 
 def create(person):
@@ -90,12 +87,7 @@ def create(person):
 
     # Otherwise, nope, person exists already
     else:
-        abort(
-            409,
-            "Person {fname} {lname} exists already".format(
-                fname=fname, lname=lname
-            ),
-        )
+        abort(409, f"Person {fname} {lname} exists already")
 
 
 def update(person_id, person):
@@ -132,10 +124,7 @@ def update(person_id, person):
 
     # Otherwise, nope, didn't find that person
     else:
-        abort(
-            404,
-            "Person not found for Id: {person_id}".format(person_id=person_id),
-        )
+        abort(404, f"Person not found for Id: {person_id}")
 
 
 def delete(person_id):
@@ -152,13 +141,8 @@ def delete(person_id):
     if person is not None:
         db.session.delete(person)
         db.session.commit()
-        return make_response(
-            "Person {person_id} deleted".format(person_id=person_id), 200
-        )
+        return make_response(f"Person {person_id} deleted", 200)
 
     # Otherwise, nope, didn't find that person
     else:
-        abort(
-            404,
-            "Person not found for Id: {person_id}".format(person_id=person_id),
-        )
+        abort(404, f"Person not found for Id: {person_id}")