Acked-by: Daniel Axtens <d...@axtens.net> Stephen Finucane <step...@that.guru> writes:
> 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 <step...@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 > > _______________________________________________ > Patchwork mailing list > Patchwork@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/patchwork _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork
