<div dir="auto">Pushed it by mistake. Is there any way to revert it? </div><br><div class="gmail_quote"><div dir="ltr">On Wed 2 May, 2018 7:24 pm Fam Zheng, <<a href="mailto:famz@redhat.com">famz@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, 05/02 12:01, Paolo Bonzini wrote:<br>
> On 01/05/2018 09:35, Shubham Jain wrote:<br>
> > ---<br>
> >  api/models.py | 24 ++++++++++++++----------<br>
> >  api/rest.py   |  5 +----<br>
> >  2 files changed, 15 insertions(+), 14 deletions(-)<br>
> > <br>
> > diff --git a/api/models.py b/api/models.py<br>
> > index 4cc2b74..13ec44e 100644<br>
> > --- a/api/models.py<br>
> > +++ b/api/models.py<br>
> > @@ -245,12 +245,12 @@ class MessageManager(models.Manager):<br>
> >              self.delete_subthread(r)<br>
> >          msg.delete()<br>
> >  <br>
> > -    def add_message_from_mbox(self, mbox, user, project_name=None):<br>
> > +    def add_message_from_mbox(self, mbox_blob, user, project_name=None):<br>
> >  <br>
> >          def find_message_projects(m):<br>
> >              return [p for p in Project.objects.all() if p.recognizes(m)]<br>
> >  <br>
> > -        m = MboxMessage(mbox)<br>
> > +        m = MboxMessage(mbox_blob)<br>
> >          msgid = m.get_message_id()<br>
> >          if project_name:<br>
> >              projects = [Project.object.get(name=project_name)]<br>
> > @@ -272,7 +272,7 @@ class MessageManager(models.Manager):<br>
> >              msg.project = p<br>
> >              if self.filter(message_id=msgid, project__name=<a href="http://p.name" rel="noreferrer noreferrer" target="_blank">p.name</a>).first():<br>
> >                  raise self.DuplicateMessageError(msgid)<br>
> > -            msg.save_mbox(mbox)<br>
> > +            msg.save_mbox(mbox_blob)<br>
> >              msg.save()<br>
> >              emit_event("MessageAdded", message=msg)<br>
> >              self.update_series(msg)<br>
> > @@ -318,20 +318,24 @@ class Message(models.Model):<br>
> >      num_patches = models.IntegerField(null=False, default=-1, blank=True)<br>
> >  <br>
> >      objects = MessageManager()<br>
> > +    def _get_mbox_blob(self):<br>
> > +        return self.get_mbox()    <br>
> >  <br>
> > -    def save_mbox(self, mbox):<br>
> > -        save_blob(mbox, self.message_id)<br>
> > +    mbox = property(_get_mbox_blob)<br>
> <br>
> This can be simply "mbox = property(get_mbox)", right?  There should be<br>
> no need to add _get_mbox_blob.  After this is changed, we can look at<br>
> adding a setter for the mbox property.<br>
> <br>
> Fam, could you please review this patch?  It's more your area. :)<br>
<br>
Looks good to me, except the long subject could be broken into a proper commit<br>
message. :)<br>
<br>
Fam<br>
</blockquote></div>