From c0c89da4b8cd041c5332215ddc2dd3fcce7edccf Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Mon, 13 Feb 2012 09:20:47 +0100 Subject: [PATCH] [FIX] rendering of readonly URI fields don't call FieldChar's set_value as this nukes the URI widget's content (link in this case). Either that, or empty the content and add the link back via DOM manipulations lp bug: https://launchpad.net/bugs/930761 fixed bzr revid: xmo@openerp.com-20120213082047-931ea10a8lrb18wf --- addons/web/static/src/js/view_page.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/addons/web/static/src/js/view_page.js b/addons/web/static/src/js/view_page.js index 8a8148f8cae..46bd1cf2485 100644 --- a/addons/web/static/src/js/view_page.js +++ b/addons/web/static/src/js/view_page.js @@ -92,17 +92,22 @@ openerp.web.page = function (openerp) { openerp.web.page.FieldURIReadonly = openerp.web.page.FieldCharReadonly.extend({ template: 'FieldURI.readonly', scheme: null, + format_value: function (value) { + return value; + }, set_value: function (value) { - var displayed = this._super.apply(this, arguments); this.$element.find('a') - .attr('href', this.scheme + ':' + displayed) - .text(displayed); + .attr('href', this.scheme + ':' + value) + .text(this.format_value(value)); } }); openerp.web.page.FieldEmailReadonly = openerp.web.page.FieldURIReadonly.extend({ scheme: 'mailto' }); openerp.web.page.FieldUrlReadonly = openerp.web.page.FieldURIReadonly.extend({ + format_value: function (value) { + return value.slice(2); + }, set_value: function (value) { var s = /(\w+):(.+)/.exec(value); if (!s || !(s[1] === 'http' || s[1] === 'https')) { return; }