Review Request 114658: Fixes the data loss caused when user closes kwalletmanager with unsaved changes

Arjun Ak arjunak234 at gmail.com
Sun Jun 22 07:38:27 UTC 2014



> On Dec. 26, 2013, 3:45 a.m., Valentin Rusu wrote:
> > Ship It!
> 
> Christoph Feck wrote:
>     What is the status of this patch? Arjun probably has no commit rights.

I can still reproduce this. Hasn't anyone committed this yet?


- Arjun


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114658/#review46154
-----------------------------------------------------------


On Dec. 25, 2013, 6:31 p.m., Arjun Ak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114658/
> -----------------------------------------------------------
> 
> (Updated Dec. 25, 2013, 6:31 p.m.)
> 
> 
> Review request for KDE Utils.
> 
> 
> Bugs: 329165
>     http://bugs.kde.org/show_bug.cgi?id=329165
> 
> 
> Repository: kwalletmanager
> 
> 
> Description
> -------
> 
> Patch to fix Bug 329165(Closing the app leads to loss of unsaved data)
> 
> 
> Diffs
> -----
> 
>   src/manager/kwalleteditor.h f3d08f3 
>   src/manager/kwalleteditor.cpp 6f07b7d 
>   src/manager/kwalletmanager.h 7489077 
>   src/manager/kwalletmanager.cpp 355fda7 
>   src/manager/kwalletmanagerwidget.h df1e4fa 
>   src/manager/kwalletmanagerwidget.cpp c855c91 
>   src/manager/kwalletmanagerwidgetitem.h ad2451e 
>   src/manager/kwalletmanagerwidgetitem.cpp 79bb476 
>   src/manager/walletcontrolwidget.h fb94999 
>   src/manager/walletcontrolwidget.cpp 948cc60 
> 
> Diff: https://git.reviewboard.kde.org/r/114658/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Arjun Ak
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20140622/e0f6f571/attachment.html>


More information about the Kde-utils-devel mailing list