----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106909/#review20530 -----------------------------------------------------------
This review has been submitted with commit 9c0021e98c0c6a808ffa82449d355a64ecef4552 by Dawit Alemayehu to branch KDE/4.9. - Commit Hook On Oct. 16, 2012, 10:59 a.m., Gordian Klein wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106909/ > ----------------------------------------------------------- > > (Updated Oct. 16, 2012, 10:59 a.m.) > > > Review request for KDE Runtime. > > > Description > ------- > > Hi, this patch fixes an issue where kio_smb wouldnt save the credentials in > kwallet wether one checks the "save password" box or not. > > In slavebase.h is the following is stated: > "Starting with KDE 4.7, calling openPasswordDialog will no longer store > passwords automatically for you. This was done to avoid accidental storage > of incorrect or invalid password information." > > So i added a call to cacheAuthentication() to store the password permanently. > > > Diffs > ----- > > kioslave/smb/kio_smb_auth.cpp 1010ffc > > Diff: http://git.reviewboard.kde.org/r/106909/diff/ > > > Testing > ------- > > Used dolphin to access a samba share and checked the "save password" dialog. > Verified that the password gets stored in KWallet. > > > Thanks, > > Gordian Klein > >
