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

[Cédric Krier]

changeset 4f6a9cbd14ad in sao:5.0
details: https://hg.tryton.org/sao?cmd=changeset;node=4f6a9cbd14ad
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
        (grafted from fb90082bd482bc2dd32c28a5d8e0e6682ca0ccfa)
diffstat:

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

diffs (25 lines):

diff -r 8e9eada53048 -r 4f6a9cbd14ad 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
@@ -908,7 +908,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 8e9eada53048 -r 4f6a9cbd14ad 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
@@ -625,7 +625,7 @@
                                     ids: record_ids
                                 };
                                 Sao.Action.exec_action(exec_action, data,
-                                    screen.context());
+                                    jQuery.extend({}, screen.group._context));
                             });
                         }.bind(this))
                         .appendTo(menu);