Odoo Server (MOVED TO GITHUB)

Code review comment for lp:~openerp-community-testers/openobject-server/trunk-sequences-test-bug-1083724

  1. trunk-sequences-test-bug-1083724
  2. Merge into trunk
Revision history for this message
Vo Minh Thu (thu) wrote :

Hi!

Indeed, the

  with ... as raise_ctx:

seems nice.

But the diff seems messed up by tabs.

And I don't understand the try/finally. Don't you want to put all the code using the cursors in the try body?

review: Needs Information
Reply

« Back to merge proposal