D22250: get rid of blockSignals

[Marco Martin]

mart created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
mart requested review of this revision.

REVISION SUMMARY
  Don't call blockSignals, but instead, to avoid an infinite signal chain,
  check KCoreConfigSkeleton::configChanged wasn't emitted by ourselves writing on the 
  config.
  Plasma::Applet::configChanged() slot was never invoked when config was written from 
  QML, which is wrong.

TEST PLAN
  Config still loads, plasma which makes heavy use of this still loads and saves 
  configs correctly, no infinite signal/slot recursions.
  Now if somebody connects to KCoreConfigSkeleton::configChanged from outside of
  this class, it wil lreceive correct signals when somebody writes to the config from
  QML.

REPOSITORY
  R296 KDeclarative

BRANCH
  blocksignals

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

AFFECTED FILES
  src/kdeclarative/configpropertymap.cpp

To: mart
Cc: kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190703/1f067685/attachment-0001.html>