Crash when submitting review

[ntmlod]

2017-12-07 14:06:22,928 Trac[main] ERROR: Exception caught while post-processing request:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/trac/web/main.py", line 243, in dispatch
    self._post_process_request(req)
  File "/usr/lib/python2.7/site-packages/trac/web/main.py", line 340, in _post_process_request
    f.post_process_request(req, *(None,)*extra_arg_count)
  File "build/bdist.linux-x86_64/egg/coderev/web_ui.py", line 80, in post_process_request
    changeset = data['changeset']
TypeError: 'NoneType' object has no attribute '__getitem__'
2017-12-07 14:06:22,928 Trac[main] ERROR: Internal Server Error:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/trac/web/main.py", line 497, in _dispatch_request
    dispatcher.dispatch(req)
  File "/usr/lib/python2.7/site-packages/trac/web/main.py", line 224, in dispatch
    self._post_process_request(req, *resp)
  File "/usr/lib/python2.7/site-packages/trac/web/main.py", line 338, in _post_process_request
    resp = f.post_process_request(req, *resp)
  File "build/bdist.linux-x86_64/egg/coderev/web_ui.py", line 90, in post_process_request
    self._update_tickets(changeset, review, status_changed)
  File "build/bdist.linux-x86_64/egg/coderev/web_ui.py", line 156, in _update_tickets
    changes = self._get_ticket_changes(tkt, status)
  File "build/bdist.linux-x86_64/egg/coderev/web_ui.py", line 204, in _get_ticket_changes
    field, value = group.split('=', 1)
ValueError: need more than 1 value to unpack

[Jun Omae]

In 17015:

CodeReviewerPlugin 1.0.0dev: fix post_process_request crashing when internal error is raised (refs #13344)

[Jun Omae]

In 17016:

CodeReviewerPlugin 1.0.0dev: fix ValueError raising when configuration is invalid (refs #13344)

[Jun Omae]

The original exception is leaded by invalid configuration (e.g. missing = in [codereviewer] passed).

Please close this ticket if the issue is gone with the latest version.

[ntmlod]

Yeah my trac.ini settings were wrong, anyway your fix was able to go through.

Thanks for the patch.