[PATCH] parser: Use 'objects.create' instead of 'save'

Stephen Finucane stephen at that.guru
Wed Oct 31 08:56:55 AEDT 2018


As noted in the Django documentation [1], this lets us do things in one
step.

[1] https://docs.djangoproject.com/en/dev/topics/db/queries/#creating-objects

Signed-off-by: Stephen Finucane <stephen at that.guru>
---
 patchwork/parser.py | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/patchwork/parser.py b/patchwork/parser.py
index 6b7189cb..d6fa8437 100644
--- a/patchwork/parser.py
+++ b/patchwork/parser.py
@@ -1034,12 +1034,12 @@ def parse_mail(mail, list_id=None):
         # - there is an existing series, but it already has a patch with this
         #   number in it
         if not series or Patch.objects.filter(series=series, number=x).count():
-            series = Series(project=project,
-                            date=date,
-                            submitter=author,
-                            version=version,
-                            total=n)
-            series.save()
+            series = Series.objects.create(
+                project=project,
+                date=date,
+                submitter=author,
+                version=version,
+                total=n)
 
             # NOTE(stephenfin) We must save references for series. We
             # do this to handle the case where a later patch is
@@ -1109,12 +1109,12 @@ def parse_mail(mail, list_id=None):
                     msgid=msgid, series__project=project).first().series
 
             if not series:
-                series = Series(project=project,
-                                date=date,
-                                submitter=author,
-                                version=version,
-                                total=n)
-                series.save()
+                series = Series.objects.create(
+                    project=project,
+                    date=date,
+                    submitter=author,
+                    version=version,
+                    total=n)
 
                 # we don't save the in-reply-to or references fields
                 # for a cover letter, as they can't refer to the same
-- 
2.19.1



More information about the Patchwork mailing list