D15743: [WIP] Fix removal of external scripts

[Amish Naidu]

amhndu updated this revision to Diff 42320.
amhndu added a subscriber: flherne.
amhndu added a comment.


  Refactored external script to store the config label inside the model
  as discussed with @flherne on irc
  Inserting/Deleting/Updating scripts now works as expected.

REPOSITORY
  R32 KDevelop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15743?vs=42296&id=42320

BRANCH
  script-remove-bug

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

AFFECTED FILES
  plugins/externalscript/externalscriptitem.cpp
  plugins/externalscript/externalscriptitem.h
  plugins/externalscript/externalscriptplugin.cpp
  plugins/externalscript/externalscriptplugin.h
  plugins/externalscript/externalscriptview.cpp

To: amhndu, #kdevelop
Cc: flherne, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180925/d5512c56/attachment.html>