Changes between Initial Version and Version 1 of Ticket #20935, comment 13


Ignore:
Timestamp:
2021-07-12T18:51:00+02:00 (5 years ago)
Author:
Bjoeni

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #20935, comment 13

    initial v1  
    11Replying to [comment:10 Don-vip]:
    22> in #21101 I was able to reproduce it at the second split on a different way, after I chose the "remember my choice" option in the dialog displayed for the split of the first way (which ran OK). But the OP in #21101 says nothing about that.
    3 Ok, in that case for me it doesn't crash anymore. Part of the problem was that the DataSetListener was added but not removed when the dialog wasn't shown. Also I added an additional null check in case the selected node is removed while the dialog is opened. Both should fixed in v2 above, could you please have a look if it's still reproducible for you?
     3Ok, in that case for me it doesn't crash anymore. Part of the problem was that the DataSetListener was added but not removed when the dialog wasn't shown in the first place (cause then `setVisible(false)` was never called). Also I added an additional null check in case the selected node is removed while the dialog is opened. Both should fixed in v2 above, could you please have a look if it's still reproducible for you?