Close the db cursor created for updating the local_id.

Review Request #8713 — Created Feb. 3, 2017 and submitted — Latest diff uploaded

Information

Review Board
release-2.0.x
67967ad...

Reviewers

We compute a review request's local_id by running some custom SQL. In doing so,
we create a db cursor, which we weren't closing correctly. Eventually the
garbage collector would get around to it, but we really should be closing it
ourselves.

In Django 1.7, we can avoid this and make it slightly more safe by using the
cursor as a context manager. I've added a TODO comment to track that.

Ran unit tests.