D12477: Add unit test to see that :/ files can work

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Mon Apr 23 20:41:40 UTC 2018


kossebau added a comment.


  It passes only by pure chance as NotepadPart uses QFile, which understands such an url. So the value of that new autptest is questionable, as this is not testing any API promise.
  
  For `ReadOnlyPart::localFilePath()` it says: "Returns the local file path associated with this part. " So a :/ resource url is rather out of scope, at least by traditional meaning of "local file path". Any KParts plugins which pass the file path on to non-QFile-using code will fall flat on such qrc path.

REPOSITORY
  R306 KParts

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

To: svuorela, dfaure, kossebau, #frameworks
Cc: michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180423/1fcde5aa/attachment.html>


More information about the Kde-frameworks-devel mailing list