From 28afa319a4007a8fa017f41c3c1bed67bbd7ab2a Mon Sep 17 00:00:00 2001
From: Andy Byers <a.byers@bbk.ac.uk>
Date: Tue, 7 May 2024 20:32:37 +0100
Subject: [PATCH] How has this not triggered before?

---
 views.py | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/views.py b/views.py
index bac0b56..e34588e 100644
--- a/views.py
+++ b/views.py
@@ -366,20 +366,21 @@ def books_chapter(request, book_id, chapter_id=None):
             instance=chapter,
             items=logic.get_chapter_contributor_items(book),
         )
-        form.save(book=book)
-        form.save_m2m()
-        messages.add_message(
-            request,
-            messages.SUCCESS,
-            'Chapter Saved.',
-        )
+        if form.is_valid():
+            form.save(book=book)
+            form.save_m2m()
+            messages.add_message(
+                request,
+                messages.SUCCESS,
+                'Chapter Saved.',
+            )
 
-        return redirect(
-            reverse(
-                'books_edit_book',
-                kwargs={'book_id': book.pk},
+            return redirect(
+                reverse(
+                    'books_edit_book',
+                    kwargs={'book_id': book.pk},
+                )
             )
-        )
 
     template = 'books/chapter.html'
     context = {