diff options
| author | Olivier Dony <odo@openerp.com> | 2016-03-08 11:31:06 +0100 |
|---|---|---|
| committer | Olivier Dony <odo@openerp.com> | 2016-03-08 16:46:25 +0100 |
| commit | ed7567d3f55b12181693ae27c0019aad849b7a80 (patch) | |
| tree | 23d7bf85a3fba6a1906c850980da5445a735703e /addons/web/static/src/js/widgets/debug_manager.js | |
| parent | c4429aa4993a8a38680ddf20f67d1b7797a7f171 (diff) | |
[REM] sql_db: legacy useless tests and duplicate exception block
- The test for %f/%d is useless, eveybody has been using %s for
ages, and the psycopg2 error is pretty explicit if you don't.
It can introduce a small but useless delay for long queries.
- The duplicate except block was redundant with the generic `except
Exception` block, with only a different log prefix.
+ the logging for the type testing of `params` was useless too,
raising a error is enough (though that particular test is still
relatively useful because the psycopg2 error isn't very clear
when you mess up the parameters)
Diffstat (limited to 'addons/web/static/src/js/widgets/debug_manager.js')
0 files changed, 0 insertions, 0 deletions
