Method _add_transaction could be called with None kwargs

[ced, 2016-10-21 16:50:18]

https://codereview.appspot.com/317760043/diff/1/celery_tryton.py File celery_tryton.py (right): https://codereview.appspot.com/317760043/diff/1/celery_tryton.py#newcode107 celery_tryton.py:107: kwargs = kwargs.copy() I would test here is kwargs is not None and create an empty dict.

[ced, 2016-10-21 16:52:50]

Also I think the description of the issue should be more complete. It should not explain what issue was raised but more what was the problem like _add_transaction could be called with None etc.

[sergio.morillo, 2016-10-21 21:32:25]

Fix AttributeError when calling kwargs.copy()

[ced, 2016-10-22 17:37:37]

LGTM but a better commit message should be written.

[sergio.morillo, 2016-10-24 06:59:16]

Method _add_transaction could be called with None kwargs

[sergio.morillo, 2016-10-24 07:01:52]

On 2016/10/22 17:37:37, ced wrote: > LGTM but a better commit message should be written. Sorry, I have just seen the review was committed. Thanks.