Merge lp:~camptocamp/geospatial-addons/fix-widget-early-get-value-20121218 into lp:~geospatial-addons-core-editors/geospatial-addons/6.1

[Yannick Vaucher @ Camptocamp]

Make sure we don't call too early the get_value of a widget inserted after a geoengine widget geo_edit_map

[Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount)]

Hello,

Well, I'm not sure this is the best way to do it. But effectively I do not see any other way to fix this without patching the web client.

That's odd as start function should be called only when all widget are loaded. At that time it is not problematic because the context it is not really used yet. But that's a workaround that should be reverted when date widget/web client is fixed.

review: Approve

=== modified file 'base_geoengine/static/src/js/geoengine_view.js'
--- base_geoengine/static/src/js/geoengine_view.js	2012-12-12 15:44:48 +0000
+++ base_geoengine/static/src/js/geoengine_view.js	2012-12-18 13:08:11 +0000
@@ -600,7 +600,10 @@
                 return;
             }
             var self = this;
-            var rdataset = new openerp.web.DataSetStatic(self, self.view.model, self.build_context());
+            // We blacklist all other fields in order to avoid calling get_value inside the build_context on field widget which aren't started yet
+            var blacklist = this.view.fields_order.slice();
+            delete blacklist[this.name];
+            var rdataset = new openerp.web.DataSetStatic(self, self.view.model, self.build_context(blacklist));
             rdataset.call("get_edit_info_for_geo_column", [self.name, rdataset.get_context()], false, 0).then(function(result) {
                 var layers = self.create_edit_layers(self, result);
                 self.geo_type = result.geo_type;