<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/107033/">http://git.reviewboard.kde.org/r/107033/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 25th, 2012, 10:09 a.m., <b>Daniele Elmo Domenichelli</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The patch looks ok, but testing it I found another related bug:
Steps to reproduce:
Method 1)
- Close all wallets
- Start ktp-accounts-kcm
- Click edit for one account (the kwallet dialog pops up)
- Insert kwallet password
- Close the edit dialog
- Close all wallets again
- Click edit for one account
Method 2)
- Close all wallets
- Start ktp-accounts-kcm
- Click edit for one account (the kwallet dialog pops up)
- Hit cancel on kwallet dialog (The edit dialog this time is shown with the password blank)
- Close the edit dialog
- Click edit for one account
This time kwallet dialog does not pop up but since the wallet is locked the edit dialog is not shown.
When you close the kcm module it causes a segfault.
On the other hand even if without the patch it shows the ugly useless version of the edit dialog, it doesn't seem to cause the segfault, so this patch must be causing it, so I'm not sure if we should ship it without fixing this bug...</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Just a note... I don't know what happens if kwallet is disabled, but this might be another issue</pre>
<br />
<p>- Daniele Elmo</p>
<br />
<p>On October 25th, 2012, 9:24 a.m., Dan Vrátil wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By Dan Vrátil.</div>
<p style="color: grey;"><i>Updated Oct. 25, 2012, 9:24 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Wait until KWallet is opened, then allow dialog to be shown. Otherwise you see just an empty dialog with "OK" and "Cancel" buttons.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=308079">308079</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/edit-account-dialog.h <span style="color: grey">(08a3481)</span></li>
<li>src/edit-account-dialog.cpp <span style="color: grey">(a747242)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107033/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>