[Differential] [Commented On] D2618: [kdevplatform] fix plugins loading

[antonanikin (Anton Anikin)]

antonanikin added a comment.


  > The *EnabledByDefault* property needs  to be read somewhere.
  
  Hi, Kevin!
  
  https://techbase.kde.org/Development/Tutorials/Kate/KTextEditor_Plugins says:
  
  > X-KDE-PluginInfo-EnabledByDefault: whether the plugin is enabled by default. Applied when clicking on "Defaults" button on the plugin selector.
  
  I grep sources of `kdevplatform` and `kdevelop` and found that all plugins are "EnabledByDefault" except `KDevManPage`. It seems that this setting is not used anywhere except in removed code (I can't find something other). I think that we сan safely remove this code without negative consequences, since `EnabledByDefault` field is only used for setting the initial value of appropriate plugin's property.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

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

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