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

Stephen Finucane stephen at that.guru
Wed Oct 31 21:51:47 AEDT 2018


On Wed, 2018-10-31 at 10:33 +1100, Daniel Axtens wrote:
> Acked-by: Daniel Axtens <dja at axtens.net>

Thanks. Applied.

> Stephen Finucane <stephen at 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 <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
> > 
> > _______________________________________________
> > Patchwork mailing list
> > Patchwork at lists.ozlabs.org
> > https://lists.ozlabs.org/listinfo/patchwork




More information about the Patchwork mailing list