Review Request 128510: KNewFileMenu: fix copying template file from embedded resource

Wolfgang Bauer wbauer at tmo.at
Sat Jul 23 19:09:47 UTC 2016


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

Review request for KDE Frameworks, Chinmoy Ranjan Pradhan and David Faure.


Repository: kio


Description
-------

KIO::storedTransferJob apparently doesn't support copying "files" from embedded Qt resources.
Therefore creating a new file via the "Create New" context menu results in an empty file in that case.

This patch fixes it by reading the source file into a QByteArray buffer first.


Diffs
-----

  src/filewidgets/knewfilemenu.cpp 4908e4c 

Diff: https://git.reviewboard.kde.org/r/128510/diff/


Testing
-------

Use "Create New"->"Text File" or "Create New"->HTML File" (in dolphin or the folderview plasmoid).
Before only an empty file was created, now the file has the intended content, and therefore the mimetype is also recognized correctly if it has no extension.


Thanks,

Wolfgang Bauer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160723/92edc7f6/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list