Review Request 118314: KUrl -> QUrl

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Sun May 25 14:38:16 BST 2014


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

Review request for Dolphin.


Repository: kde-baseapps


Description
-------

Ported Dolphin from KUrl to QUrl.

Moved the "new name checking" from DolphinView::slotRoleEditingFinished to KFileItemModel::setData,
because we need the same url modifying code for both functions - avoid code duplication + porting mistakes.

RenameDialog:
Changed renameItems to renameItem, we do the url modifying there for all two cases -
avoid code duplication + porting mistakes.


Diffs
-----

  dolphin/src/dolphinapplication.cpp 8144da8 
  dolphin/src/dolphincontextmenu.h 180f917 
  dolphin/src/dolphincontextmenu.cpp 51351f0 
  dolphin/src/dolphinmainwindow.h 1192f6e 
  dolphin/src/dolphinmainwindow.cpp 049c440 
  dolphin/src/dolphinpart.h 9ab1e80 
  dolphin/src/dolphinpart.cpp 479b809 
  dolphin/src/dolphinpart_ext.h 5272df6 
  dolphin/src/dolphinpart_ext.cpp fb7a4d2 
  dolphin/src/dolphinviewcontainer.h a7a9969 
  dolphin/src/dolphinviewcontainer.cpp 7610625 
  dolphin/src/kitemviews/kfileitemmodel.h 62a283d 
  dolphin/src/kitemviews/kfileitemmodel.cpp 51bf546 
  dolphin/src/kitemviews/kfileitemmodelrolesupdater.cpp acb3e0f 
  dolphin/src/kitemviews/kstandarditemlistwidget.cpp 7a9f31a 
  dolphin/src/kitemviews/private/kdirectorycontentscounter.cpp cd448e2 
  dolphin/src/kitemviews/private/kfileitemclipboard.h 86eb8e9 
  dolphin/src/kitemviews/private/kfileitemclipboard.cpp ebf50e2 
  dolphin/src/kitemviews/private/kfileitemmodeldirlister.h da01d20 
  dolphin/src/panels/folders/folderspanel.h 14d8e87 
  dolphin/src/panels/folders/folderspanel.cpp 0b2ea38 
  dolphin/src/panels/folders/treeviewcontextmenu.cpp 2e59ae8 
  dolphin/src/panels/information/informationpanel.h c68b66e 
  dolphin/src/panels/information/informationpanel.cpp 4ad1276 
  dolphin/src/panels/information/informationpanelcontent.h 67fdf6c 
  dolphin/src/panels/information/informationpanelcontent.cpp 9dc59db 
  dolphin/src/panels/information/phononwidget.h b5aedfe 
  dolphin/src/panels/information/phononwidget.cpp 4b2cc28 
  dolphin/src/panels/panel.h a0b25d6 
  dolphin/src/panels/panel.cpp 14b7c02 
  dolphin/src/panels/places/placesitem.h 297f12d 
  dolphin/src/panels/places/placesitem.cpp 1729bbd 
  dolphin/src/panels/places/placesitemeditdialog.h bf34847 
  dolphin/src/panels/places/placesitemeditdialog.cpp 0a66ef7 
  dolphin/src/panels/places/placesitemmodel.h 4a374e5 
  dolphin/src/panels/places/placesitemmodel.cpp 6ba91c5 
  dolphin/src/panels/places/placespanel.h 16112e8 
  dolphin/src/panels/places/placespanel.cpp d3614c9 
  dolphin/src/panels/terminal/terminalpanel.h 987ee47 
  dolphin/src/panels/terminal/terminalpanel.cpp 5c72652 
  dolphin/src/search/dolphinsearchbox.h 53b12ff 
  dolphin/src/search/dolphinsearchbox.cpp df96f74 
  dolphin/src/search/filenamesearchprotocol.h f691f99 
  dolphin/src/search/filenamesearchprotocol.cpp b56a995 
  dolphin/src/settings/applyviewpropsjob.h 68fdcc4 
  dolphin/src/settings/applyviewpropsjob.cpp 9849216 
  dolphin/src/settings/dolphinsettingsdialog.h 2de1950 
  dolphin/src/settings/dolphinsettingsdialog.cpp 8df2996 
  dolphin/src/settings/general/behaviorsettingspage.h 7a9c2f0 
  dolphin/src/settings/general/behaviorsettingspage.cpp 7633b82 
  dolphin/src/settings/general/generalsettingspage.h 0d28664 
  dolphin/src/settings/general/generalsettingspage.cpp 8a8adf8 
  dolphin/src/settings/startup/startupsettingspage.h 29cdc63 
  dolphin/src/settings/startup/startupsettingspage.cpp 7f8c2d5 
  dolphin/src/settings/viewpropertiesdialog.cpp 593d1c4 
  dolphin/src/settings/viewpropsprogressinfo.h 6f8c763 
  dolphin/src/settings/viewpropsprogressinfo.cpp a4a45da 
  dolphin/src/statusbar/dolphinstatusbar.h 4d6dbb2 
  dolphin/src/statusbar/dolphinstatusbar.cpp 9f17c8e 
  dolphin/src/statusbar/spaceinfoobserver.h d2fb6eb 
  dolphin/src/statusbar/spaceinfoobserver.cpp 9125a93 
  dolphin/src/statusbar/statusbarspaceinfo.h 1065d9f 
  dolphin/src/statusbar/statusbarspaceinfo.cpp 3692947 
  dolphin/src/tests/kfileitemmodelbenchmark.cpp 66918b6 
  dolphin/src/tests/kfileitemmodeltest.cpp 48e72e8 
  dolphin/src/tests/testdir.h 0d3c5dd 
  dolphin/src/tests/testdir.cpp 8938e60 
  dolphin/src/views/dolphinnewfilemenuobserver.h 3b5014a 
  dolphin/src/views/dolphinremoteencoding.h b317cc1 
  dolphin/src/views/dolphinremoteencoding.cpp bf00e33 
  dolphin/src/views/dolphinview.h 4ba1029 
  dolphin/src/views/dolphinview.cpp 0e43dcd 
  dolphin/src/views/dolphinviewactionhandler.cpp 3955f25 
  dolphin/src/views/draganddrophelper.h eda5fc5 
  dolphin/src/views/draganddrophelper.cpp f8ae0ad 
  dolphin/src/views/renamedialog.h 29ef8bd 
  dolphin/src/views/renamedialog.cpp 3b94e01 
  dolphin/src/views/versioncontrol/versioncontrolobserver.h 2c07b06 
  dolphin/src/views/versioncontrol/versioncontrolobserver.cpp 9033e19 
  dolphin/src/views/viewmodecontroller.h f476595 
  dolphin/src/views/viewmodecontroller.cpp 26e1818 
  dolphin/src/views/viewproperties.h 69b507f 
  dolphin/src/views/viewproperties.cpp bcea062 

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


Testing
-------

KFileItemModelTest fails. (Expanded items)


Thanks,

Emmanuel Pescosta

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20140525/2024687e/attachment.htm>


More information about the kfm-devel mailing list