From e33b9a722e0604641e29b7c2ae017fd72cc019ae Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Wed, 16 Nov 2011 11:04:16 +0100 Subject: [PATCH] [FIX] some badly implemented function fields return empty strings (for e.g. dates, instead of False) treat empty strings as 'false' except for char fields bzr revid: xmo@openerp.com-20111116100416-0713i9va49ffbzlh --- addons/web/static/src/js/formats.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/addons/web/static/src/js/formats.js b/addons/web/static/src/js/formats.js index 23d8a2c24e0..3151b34636b 100644 --- a/addons/web/static/src/js/formats.js +++ b/addons/web/static/src/js/formats.js @@ -76,7 +76,13 @@ openerp.web.format_value = function (value, descriptor, value_if_empty) { if (typeof value === 'number' && isNaN(value)) { value = false; } + //noinspection FallthroughInSwitchStatementJS switch (value) { + case '': + if (descriptor.type === 'char') { + return ''; + } + console.warn('Field', descriptor, 'had an empty string as value, treating as false...'); case false: case Infinity: case -Infinity: