D18246: Fix premature firing of allInitialized signal

Dan Leinir Turthra Jensen noreply at phabricator.kde.org
Mon Jan 14 15:34:35 GMT 2019


leinir created this revision.
leinir added reviewers: apol, ngraham, Discover Software Store.
leinir added a project: Discover Software Store.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
leinir requested review of this revision.

REVISION SUMMARY
  In case this is in fact the first backend to be added, and also happens to be pre-filled, we still need for the rest of the backends to be added before trying to send out the initialized signal. To ensure this happens, this patch schedules it for the start of the next run of the event loop, and ensures that it only happens if there are no more backends to initialise.
  
  Specifically, this happens if e.g. the fwupd backend is the first to be initialised, and results in (amongst other things) that the application loads up and insists that there are no application backends.

REPOSITORY
  R134 Discover Software Store

REVISION DETAIL
  https://phabricator.kde.org/D18246

AFFECTED FILES
  libdiscover/resources/ResourcesModel.cpp

To: leinir, apol, ngraham, #discover_software_store
Cc: plasma-devel, sukalyanbanga, jraleigh, bam, GB_2, masilva, ragreen, ixoos, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190114/392bd749/attachment.html>


More information about the Plasma-devel mailing list