• 
      

    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.