<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/100742/">http://git.reviewboard.kde.org/r/100742/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 15th, 2011, 9:03 a.m., <b>Ralf Engels</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 filtering out unknown Albums was "0ffe80bbd8fe074dea64abb985a413885a1ab765"
Prune out any genres with empty names or the name "Unknown" when importing the Jamendo database.
This gets rid of the "Unknown" genre at the top of the lsit that freezes Amarok when you try to expand it.
Some albums might be filtered out by this, but a) these were not accessible anyway and b) if the album has not been tagged with even a single genre by the
BUG: 213165
Can you test that your patch does not cause these problems to re-appear?</pre>
</blockquote>
<p>On May 16th, 2011, 8:10 p.m., <b>Wolfgang Lorenz</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;">This patch DOES cause the freeze to reappear, it was just uploaded as an attachment to Bug 267065. Before reverting patch "0ffe80bbd8fe074dea64abb985a413885a1ab765", the SQL queries should be reworked to prevent these freezes.</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;">Wolfgang, do you want to discard this request or do you want to rework it?</pre>
<br />
<p>- Ralf</p>
<br />
<p>On February 25th, 2011, 1:13 p.m., Wolfgang Lorenz 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 Amarok.</div>
<div>By Wolfgang Lorenz.</div>
<p style="color: grey;"><i>Updated Feb. 25, 2011, 1:13 p.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;">The Amarok Jamendo-Plugin ignores albums, that come without a genre. As these are rather many, this seems not advisable (see Bug 235184).
This patch simply "unignores" albums without genre. The problem here is, it makes Amarok freeze, when it comes to showing the contents of the "Unknown genre"-Section, because the SQL query takes a very long time - more than an hour on my computer (see Bug 267065).
The second thing, this patch does is to write out the SQL-queries in debug mode.
Note: To search the database you still might need to apply patch https://git.reviewboard.kde.org/r/100723/ (for Bug 233170)
Also Note: At the moment, this patch is really only for testing purposes.</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="https://bugs.kde.org/show_bug.cgi?id=235184">235184</a>,
<a href="https://bugs.kde.org/show_bug.cgi?id=267065">267065</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/services/ServiceSqlQueryMaker.cpp <span style="color: grey">(2cb1242)</span></li>
<li>src/services/jamendo/JamendoXmlParser.cpp <span style="color: grey">(b8735e8)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100742/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>