Review Request 120312: Port .cpp files from KDELibs4Support
Jakub Caban
kuba at whyblack.pl
Tue Sep 30 20:45:39 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120312/
-----------------------------------------------------------
(Updated Sept. 30, 2014, 8:45 p.m.)
Review request for KDevelop.
Changes
-------
Fix TODOS: one seems to be obsolete and other one was caused by QProgressDialog::close() emitting cancel for some reason.
Repository: kdev-upload
Description
-------
Mostly simple KUrl->QUrl changes with just a few harder corners like porting to QDialog or KUrl::relativePath(...). Removing KDELibs4support from CMakeList.txt is impossible now as some weird dependency forces profile dialog to link against it. Probably something with .ui files? I will investigate when we'll have this part done.
Diffs (updated)
-----
uploadprofilemodel.cpp 16847b2
uploadprojectmodel.h 8379810
uploadprojectmodel.cpp 30db851
uploadprofiledlg.cpp ff234bd
uploadprofileitem.h b17187a
uploadprofileitem.cpp ee4610e
uploadpreferences.ui ac4c679
uploadprofiledlg.h d363190
uploadjob.cpp c58db45
uploadpreferences.cpp 0949c92
profilesfiletree.cpp 2c19df0
tests/CMakeLists.txt f4bc1a8
tests/permissions.cpp 8735c8d
CMakeLists.txt 3428c80
kdevuploadplugin.cpp 5261978
profilesfiletree.h c595f32
Diff: https://git.reviewboard.kde.org/r/120312/diff/
Testing
-------
Code compiles and works with 2 regressions:
- Directory chooser for profile settings always opens user home folder instead of remote view
- upload job always reports 'Upload cancelled' even after succesfull upload
Thanks,
Jakub Caban
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140930/e8ccb711/attachment.html>
More information about the KDevelop-devel
mailing list