Review Request: Fix DAV jobs bug

Grégory Oestreicher greg at kamago.net
Sun Feb 20 21:48:13 GMT 2011


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

(Updated Feb. 20, 2011, 9:48 p.m.)


Review request for kdelibs.


Changes
-------

Be consistent in qint64 usage, as suggested by Dawit Alemayehu, and use this type everywhere. This diff replaces the previous.


Summary
-------

Following commit 01db8ec6e9934a13ed8025949dab9dd43010bf53 the DAV jobs created with the DavJob::dav* utility functions ceased to work because they do not provide the body data length. This patch corrects this issue by using the same mechanisms as those used for HTTP POST requests.


Diffs (updated)
-----

  kio/kio/davjob.cpp 3b2a8da 
  kioslave/http/http.h e8fe695 
  kioslave/http/http.cpp 70960d6 

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


Testing
-------

Tested using the Akonadi Dav groupware resource and DaviCal as a backend server. No issues so far.


Thanks,

Grégory

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110220/bfe331e3/attachment.htm>


More information about the kde-core-devel mailing list