From 32423169af0a4dcd38398e4ad4f630faccba4ff8 Mon Sep 17 00:00:00 2001
From: Jeremy Kersten <jke@odoo.com>
Date: Wed, 28 Jun 2017 14:36:08 +0200
Subject: [PATCH 1/2] [FIX] web: namespace the event for Apps Widget.

With the new client web from v9, we start the new action before to destroy the old one.

When we switch from Apps (store) to Update and vice-versa, iframe was not loaded correctly.

In case of the Apps widget, we are binding event of the new iframe during the start,
and when it is ready, we destroy the old ifram and so unbind the listener of 'message.apps'.

Using a uniq id, we ensure to bind new event, and only unbind old event during the destroy.
---
 addons/web/static/src/js/apps.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/addons/web/static/src/js/apps.js b/addons/web/static/src/js/apps.js
index 156fe9e10b7..69993dcc0ef 100644
--- a/addons/web/static/src/js/apps.js
+++ b/addons/web/static/src/js/apps.js
@@ -62,7 +62,7 @@ var Apps = Widget.extend({
     },
 
     destroy: function() {
-        $(window).off("message.apps");
+        $(window).off("message." + this.uniq);
         if (this.$ifr) {
             this.$ifr.remove();
             this.$ifr = null;
@@ -114,10 +114,12 @@ var Apps = Widget.extend({
 
                 var qs = (session.debug ? 'debug&' : '') + 'db=' + client.dbname;
                 var u = client.origin + '/apps/embed/client?' + qs;
-                var css = {width: '100%', height: '400px'};
+
+                var css = {width: '100%', height: '750px'};
                 self.$ifr = $('<iframe>').attr('src', u);
 
-                $(window).on("message.apps", self.proxy('_on_message'));
+                self.uniq = _.uniqueId('apps');
+                $(window).on("message." + self.uniq, self.proxy('_on_message'));
 
                 self.on('message:ready', self, function(m) {
                     var w = this.$ifr[0].contentWindow;

From 4d4ff7c5e7982285f2137e6f1a1ca8d2bc569d27 Mon Sep 17 00:00:00 2001
From: Richard Mathot <rim@odoo.com>
Date: Wed, 28 Jun 2017 17:38:32 +0200
Subject: [PATCH 2/2] [IMP] doc: mixing attrs and states is iffy

---
 doc/reference/views.rst | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/doc/reference/views.rst b/doc/reference/views.rst
index 4622716ff7e..054cfbf0be8 100644
--- a/doc/reference/views.rst
+++ b/doc/reference/views.rst
@@ -234,6 +234,11 @@ Possible children elements of the list view are:
 
         Makes the button ``invisible`` if the record is *not* in one of the
         listed states
+
+        .. danger::
+
+            Using ``states`` in combination with ``attrs`` may lead to
+            unexpected results as domains are combined with a logical AND.
     ``context``
         merged into the view's context when performing the button's Odoo call
     ``confirm``