[tryton-commits] changeset in sao:default Use group _context to avoid leaking ses...

[Cédric Krier]

changeset fb90082bd482 in sao:default
details: https://hg.tryton.org/sao?cmd=changeset;node=fb90082bd482
description:
        Use group _context to avoid leaking session context

        When launching a new action, the context should not contain the session
        context because it is automatically appended to the evaluation.
        When creating a new group when the context model changed, it should not
        contain the session context neither.

        issue8312
        review267301002
diffstat:

 src/screen.js |  3 ++-
 src/tab.js    |  2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diffs (25 lines):

diff -r fe2a9a26fec4 -r fb90082bd482 src/screen.js
--- a/src/screen.js     Thu May 02 12:46:37 2019 +0200
+++ b/src/screen.js     Thu May 02 12:49:12 2019 +0200
@@ -978,7 +978,8 @@
                     this.context_screen.display(true);
                     return jQuery.when();
                 }
-                this.new_group(jQuery.extend(this.context,
+                this.new_group(jQuery.extend({},
+                    this.group._context,
                     this.context_screen.get_on_change_value()));
             }
 
diff -r fe2a9a26fec4 -r fb90082bd482 src/tab.js
--- a/src/tab.js        Thu May 02 12:46:37 2019 +0200
+++ b/src/tab.js        Thu May 02 12:49:12 2019 +0200
@@ -644,7 +644,7 @@
                                     ids: record_ids
                                 };
                                 Sao.Action.exec_action(exec_action, data,
-                                    screen.context);
+                                    jQuery.extend({}, screen.group._context));
                             });
                         }.bind(this))
                         .appendTo(menu);