Diff comments:

> diff --git a/lib/lp/bugs/subscribers/bugactivity.py 
> b/lib/lp/bugs/subscribers/bugactivity.py
> index c036186..d1d1bc8 100644
> --- a/lib/lp/bugs/subscribers/bugactivity.py
> +++ b/lib/lp/bugs/subscribers/bugactivity.py
> @@ -66,6 +66,10 @@ def what_changed(object_modified_event):
>      after = object_modified_event.object
>      fields = object_modified_event.edited_fields
>      changes = {}
> +
> +    if not fields:

* so I'm happy to look into it, if that's the case. What do you think?

> +        return changes
> +
>      for fieldname in fields:
>          # XXX 2011-01-21 gmb bug=705955:
>          #     Sometimes, something (webservice, I'm looking at you


-- 
https://code.launchpad.net/~ines-almeida/launchpad/+git/launchpad/+merge/445031
Your team Launchpad code reviewers is requested to review the proposed merge of 
~ines-almeida/launchpad:add-bug-webhooks/fix-failing-tests into 
launchpad:master.


_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : launchpad-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to