[Differential] [Request, 31 lines] D2613: Fix some status notifier items not appearing

davidedmundson (David Edmundson) noreply at phabricator.kde.org
Sat Aug 27 23:21:46 UTC 2016


davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Set the initial values for all the things
  This is important as Plasma::DataModel has an unsolvable bug
  When it gets data with a new key it tries to update the  QAIM roleNames
  From QML this achieves absolutely nothing as there is no signal to tell
  QQmlDelegateModel to reload the roleNames in QQmlAdapatorModel
  No matter if the row changes or the model refreshes
  This means it does not re-evaluate what bindings exist (watchedRoleIds)
  and we get properties that don't bind for this or any future SNI.
  
  The main source of this problem is syncStatus can occur before
  refreshCallback; but whilst that's easy to guard against, there's still
  multiple if{} blocks that don't set various keys in refreshCallback()
  which would all need changing too. This seemed cleaner and easier to
  manage.
  
  BUG: 366283
  BUG: 367756

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

BRANCH
  master

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

AFFECTED FILES
  dataengines/statusnotifieritem/statusnotifieritemsource.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160827/ebf90cd2/attachment.html>


More information about the Plasma-devel mailing list