[comic] Destroy download dialog as time of applet
ClosedPublic

Authored by anthonyfieroni on Dec 3 2017, 2:04 PM.

Details

Test Plan

When DownloadDialog is not destroyed KNS3 engine not store registry of installed/upgradable stuff.

Diff Detail

Repository
R114 Plasma Addons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
anthonyfieroni created this revision.Dec 3 2017, 2:04 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 3 2017, 2:04 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
anthonyfieroni requested review of this revision.Dec 3 2017, 2:04 PM
apol added a subscriber: apol.Dec 3 2017, 2:53 PM

QScopedPointer won't be checking if the object has been destroyed. Are you sure it will never happen?
If so, +1.

davidedmundson accepted this revision.Dec 4 2017, 9:35 AM

Personally I'd have just set a parent when creating the new KNS3::DownloadDialog.

If someone ever did change this code so it deletes on close, the old code would have handled it, this won't.

But this is also fine.

This revision is now accepted and ready to land.Dec 4 2017, 9:35 AM

Take ownership of DownloadDialog.

Oh i revert patch, DownloadDialog expect Widget as its parent.

Ability to detect delete on close.

This revision was automatically updated to reflect the committed changes.