Yogesh(Open ERP) has proposed merging 
lp:~openerp-dev/openerp-web/trunk-bug-871625-ysa into lp:openerp-web.

Requested reviews:
  OpenERP R&D Web Team (openerp-dev-web)
Related bugs:
  Bug #871625 in OpenERP Web: "no need to import default value"
  https://bugs.launchpad.net/openerp-web/+bug/871625

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-871625-ysa/+merge/79073
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-871625-ysa/+merge/79073
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-bug-871625-ysa.
=== modified file 'addons/web/static/src/js/data_import.js'
--- addons/web/static/src/js/data_import.js	2011-09-26 10:07:44 +0000
+++ addons/web/static/src/js/data_import.js	2011-10-12 08:08:24 +0000
@@ -49,12 +49,18 @@
             });
         };
         this.ready  = $.Deferred.queue().then(function () {
-            self.required_fields = _(self.fields).chain()
+            self.required_default_fields = _(self.fields).chain()
                 .filter(function (field) { return field.required; })
-                .pluck('name')
+                .pluck('id')
                 .value();
-            convert_fields(self);
-            self.all_fields.sort();
+            dataset.default_get(self.required_default_fields || [], function(res) {
+                self.required_fields = _(self.fields).chain()
+                    .filter(function (field) {if( !(field.id in res)) return field.required; })
+                    .pluck('name')
+                    .value();
+                convert_fields(self);
+                self.all_fields.sort();
+            });
         });
     },
     start: function() {

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to