<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/107676/">http://git.reviewboard.kde.org/r/107676/</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;">Ship It!</pre>
<br />
<p>- mojtaba</p>
<br />
<p>On December 12th, 2012, 2:49 p.m., Friedrich W. H. Kossebau 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 Calligra, Inge Wallin and mojtaba shahi.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated Dec. 12, 2012, 2:49 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;">Currently the suffix is ignored and always .xhtml is used. While .xhtml might be the correct suffix for the content which is currently created by the export filter, still the user should be more right. If she/he wants something else, it should be delivered.
Attached patch changes the behaviour to use the suffix of the filename passed to the export filter for all files that are generated.
It's not the optimal patch. I had no real idea how to inject the user-choosen suffix just for the case of one file, as with (x)html export. Perhaps you have a better idea.
But at least it works :)
Okay to commit to 2.6 and port to master?</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">export as html to a file with custom suffix uses that suffix
export to epub uses xhtml as before internally</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>filters/words/epub/FileCollector.h <span style="color: grey">(e705026)</span></li>
<li>filters/words/epub/FileCollector.cpp <span style="color: grey">(b25bfa6)</span></li>
<li>filters/words/epub/OdtHtmlConverter.cpp <span style="color: grey">(ac8dcc4)</span></li>
<li>filters/words/epub/exporthtml.cpp <span style="color: grey">(d657490)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107676/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>