dfaure requested changes to this revision.
dfaure added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> kio_sftp.cpp:836
>          // Handle user canceled or dialog failed to open...
> -        if (!openPasswordDialog(info, errMsg)) {
> +        if (openPasswordDialogV2(info, errMsg) != 0) {
>            qCDebug(KIO_SFTP_LOG) << "User canceled password/retry dialog";

This should store the return value from openPasswordDialogV2 and use it in the 
error call 3 lines below, that's the whole point of the port to V2.

Same for the calls above, if possible (the patch doesn't show what the error 
handling is).

REPOSITORY
  R320 KIO Extras

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

To: marten, #plasma, dfaure
Cc: plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, lukas

Reply via email to