[Differential] [Updated] D2618: [kdevplatform] fix plugins loading

[kfunk (Kevin Funk)]

kfunk added a comment.


  Hmm, this appears to be the wrong fix, no?
  
  The *EnabledByDefault* property needs  to be read somewhere.
  
  What about:
  
    if (!isUserSelectable)
      return true;
    if (userEnabled) // the code from below
      return true;
    if (!isGlobalPlugin)
      return enabledByDefault.isNull() || enabledByDefault.toBool();
  
  Makes sense? Maybe I'm missing something...

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

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

To: antonanikin, #kdevelop, kfunk
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160829/7aa7dab1/attachment.html>