<table><tr><td style="">davidedmundson added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24735">View Revision</a></tr></table><br /><div><div><p>Urgh, so the order of events is:</p>
<p>KCMKScreen::~KCMKScreen() <-- deletes the config and exposed properties<br />
KCModuleQML::~KCModuleQML() <-- tears down the UI, which now references dangling things.</p>
<p>I think thats's a sign of a more generic that it would be good if we could solve, maybe an explicit method in KCModuleQML we could call from our destructor. It sounds similar to that language translation crash.</p>
<p>For 5.17, change to .get() to address Roman's comment and then +1 from me.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R104 KScreen</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24735">https://phabricator.kde.org/D24735</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, romangg, davidedmundson<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>