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