Review Request 108800: don't load the package if the servicetype doesn't exist

Giorgos Tsiapaliokas terietor at gmail.com
Wed Feb 6 11:18:17 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108800/
-----------------------------------------------------------

Review request for Plasma.


Description
-------

Make plasmate to not accept packages which doesn't specify the servicetype.
Without the servicetype plasmate can't choose the appropriate editor for the package.

With this patch plasmate is using a kmessagebox in order to show the error,
should it be a kmessagewidget?


Diffs
-----

  plasmate/mainwindow.cpp 53c4502 

Diff: http://git.reviewboard.kde.org/r/108800/diff/


Testing
-------


Thanks,

Giorgos Tsiapaliokas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130206/f2df1df3/attachment.html>


More information about the Plasma-devel mailing list