Merge lp:~openerp-dev/openerp-web/7.0-opw-591158-vme into lp:openerp-web/7.0

[Vidhin Mehta (OpenERP)]

To Reproduct:
   Video: https://docs.google.com/file/d/0By4GWTWr6HDNTjhQNmJmUkVKU0U/edit

Unmerged revisions

3894. By Vidhin Mehta (OpenERP) on 2013-04-12

[FIX]handle_button should call after form view loaded.

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2013-04-11 16:32:38 +0000
+++ addons/web/static/src/js/view_form.js	2013-04-12 13:02:30 +0000
@@ -356,6 +356,7 @@
             }
             self.$el.add(self.$buttons).removeClass('oe_form_dirty');
             self.autofocus();
+            self.trigger('record_loaded');
         });
     },
     /**
@@ -3919,14 +3920,26 @@
         }
         var parent_form = this.o2m.view;
         var self = this;
-        this.ensure_saved().then(function () {
-            if (parent_form)
-                return parent_form.save();
-            else
-                return $.when();
-        }).done(function () {
+        var dirty = parent_form.$el.hasClass('oe_form_dirty');
+
+        // if parent_form and dirty found than, handle_button function must call after
+        // loading all record and initializing form view.
+        var call_handle_button = function(){
+            parent_form.off("record_loaded", self, call_handle_button);
             self.handle_button(name, id, callback);
-        });
+        };
+        parent_form.on("record_loaded", self, call_handle_button);
+
+         this.ensure_saved().then(function () {
+             if (parent_form)
+                 return parent_form.save();
+             else
+                 return $.when();
+         }).done(function () {
+            if(!parent_form || (parent_form && !dirty)){
+                call_handle_button();
+            }
+         });
     },
 
     _before_edit: function () {