----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120313/#review67189 -----------------------------------------------------------
kded/secretagent.cpp <https://git.reviewboard.kde.org/r/120313/#comment46886> Ok, your patch works. However, in case of processGetSecrets() it will show a password dialog *for every request* that needs secrets, which I do not think users are going to like. Anyway, we should warn the user that kwallet is not working or he/she will just cancel the password dialog and try again. Think that most kwallet users use it so they do not need to remember their passwords. Just showing a password dialog when kwallet fails is not a good usability action in my opinion. - Lamarque Souza On Sept. 22, 2014, 11:38 a.m., Jan Grulich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120313/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2014, 11:38 a.m.) > > > Review request for Network Management, Lukáš Tinkl and Lamarque Souza. > > > Repository: plasma-nm > > > Description > ------- > > In the secret agent, if any of functions for processing secrets fails to open > KWallet, then m_wallet will be always set to 0, which leads again to another > attempt to open KWallet. We should check if opening of KWallet failed before > and do not try to open it again, which should be done in my patch. > > BUG: https://bugs.kde.org/show_bug.cgi?id=339223 > > > Diffs > ----- > > kded/secretagent.h a438191 > kded/secretagent.cpp 77438ca > > Diff: https://git.reviewboard.kde.org/r/120313/diff/ > > > Testing > ------- > > > Thanks, > > Jan Grulich > >
_______________________________________________ kde-networkmanager mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-networkmanager
