wxGUI/main_window: fix terminating wxGUI Animation, Timeline, Temporal tool window after quitting GRASS

[tmszi]

Fixes #6400.

[petrasovaa]

It should at least test that the temporal module has been previously imported.

But the Timeline Tool is using OnClose method where it's already calling it, so I think we should investigate why it's not called from there.

[tmszi]

But the Timeline Tool is using OnClose method where it's already calling it, so I think we should investigate why it's not called from there.

Because wx.EVT_CLOSE event, OnClose() event handler is not triggered by SIGTERM signal.

[petrasovaa]

But the Timeline Tool is using OnClose method where it's already calling it, so I think we should investigate why it's not called from there.

Because wx.EVT_CLOSE event, OnClose() event handler is not triggered by SIGTERM signal.

Right, perhaps the tool should have the main Frame as a parent, that could help, no? Or is there a reason not to?

[tmszi]

Because wx.EVT_CLOSE event, OnClose() event handler is not triggered by SIGTERM signal.

Right, perhaps the tool should have the main Frame as a parent, that could help, no? Or is there a reason not to?

Yes you are right. If wxGUI tool frame class is initialized with param parent=self, and self.Destroy() method is called from the main window frame (line 2411)

grass/gui/wxpython/main_window/frame.py

Line 2411 in 46d62b4

self.Destroy()

wx.EVT_WINDOW_DESTROY event is triggered and we can catch it on the wxGUI tool frame class level. According my debug information problem is that self._quitGRASS() method from the main window frame class (which call SIGTERM signal) is triggered first than wx.EVT_WINDOW_DESTROY event handler on the wxGUI tool frame class level (with code which potential stopping tgis framework messenger and C-interface). We need to wait if tgis framework is stopped and then call the SIGTERM signal.