D19786: Fix WebDAV directory renaming if KeepAlive is off

Robert Hoffmann noreply at phabricator.kde.org
Thu Mar 21 16:54:05 GMT 2019


hoffmannrobert added a comment.


  In D19786#433580 <https://phabricator.kde.org/D19786#433580>, @sitter wrote:
  
  > setting server to keepalive=false? what is this? the 90's? :O
  
  
  Perhaps they are low on memory? I don't know, I can't change that.
  
  > wouldn't it be simpler to `finishDav` and recurse `rename` with the redirected url? the code inside the conditional block is basically a code copy of the code above, so why not just throw away the duplicated code and start the request 
  >  from the top? it'd solve this issue and prevent future issues at the cost of performance when/if renames are carried out without trailing slash but be 100% more reliable. or at the very least call `resetSessionSettings` before the new request is set?
  
  Calling davFinished() isn't possible here, it calls SlaveBase::finished() and that prints a warning "finished() called twice! Please fix...". I think, it would have to be dispatch()ed to change the d->m_state to d->InsideMethod.
  
  I changed it to call resetSessionSettings() now.

REPOSITORY
  R241 KIO

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

To: hoffmannrobert, #frameworks, #dolphin, dfaure, sitter
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190321/8e6d147e/attachment.html>


More information about the Kde-frameworks-devel mailing list