<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/102492/">http://git.reviewboard.kde.org/r/102492/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looks good, commited</pre>
<br />
<p>- Robin</p>
<br />
<p>On August 30th, 2011, 6:20 p.m., Elias Probst 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 ownCloud.</div>
<div>By Elias Probst.</div>
<p style="color: grey;"><i>Updated Aug. 30, 2011, 6:20 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;">There is a bug in the filelisting (files/index.php) which causes troubles when listing files which don't have an extension, e.g.
Test123 vs. Test123.txt
The filenames of such files were only shown when hovering them.
The reason for this behaviour was the hackish approach on splitting the filename by using substr() and strrpos().
The bugfix uses now PHP's builtin pathinfo() to do this job.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>files/index.php <span style="color: grey">(0595463)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102492/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/102492/s/244/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/08/30/filename-only-shown-on-hover_400x100.png" style="border: 1px black solid;" alt="Filename for files without extension is only shown on hover" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>