[tryton-commits] changeset in tryton:5.0 Recursively destroy records before propa...

[Nicolas Évrard]

changeset 15d0f7fceefb in tryton:5.0
details: https://hg.tryton.org/tryton?cmd=changeset&node=15d0f7fceefb
description:
        Recursively destroy records before propagating the signal

        issue10686
        review371591003
        (grafted from b1b613ba6059248938777afd16e58eb64b6d18de)
diffstat:

 tryton/gui/window/view_form/model/group.py |  4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diffs (15 lines):

diff -r ee43feb8166c -r 15d0f7fceefb tryton/gui/window/view_form/model/group.py
--- a/tryton/gui/window/view_form/model/group.py        Wed Sep 08 10:12:38 
2021 +0200
+++ b/tryton/gui/window/view_form/model/group.py        Thu Sep 02 14:50:23 
2021 +0200
@@ -129,8 +129,10 @@
         # Use reversed order to minimize the cursor reposition as the cursor
         # has more chances to be on top of the list.
         for record in reversed(self[:]):
+            # Destroy record before propagating the signal to recursively
+            # destroy also the underlying records
+            record.destroy()
             self.signal('group-list-changed', ('record-removed', record))
-            record.destroy()
             self.pop()
         self.__id2record = {}
         self.record_removed, self.record_deleted = [], []