<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/105370/">http://git.reviewboard.kde.org/r/105370/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 28th, 2012, 4:16 a.m., <b>Thorsten Zachmann</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;">I'm not 100% sre but when looking at the desktop files of LO they contain both mimetypes the one with the prefix and also the other. Is there a reason to not have both? Also it seems that Calligra Words has the same problem while sheets only has the one with the prefix in the list.</pre>
</blockquote>
<p>On June 28th, 2012, 5:06 a.m., <b>Friedrich W. H. Kossebau</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;">I think it does not hurt to have both there, the official and (one of) the inofficial. E.g. some webservers might use an inofficial, and if the lookup mechanism does not translate it to the official mimetype, it is better to have it explicitely listed. But whatever ids are used, they need to be also matched by what the filter code checks in the end and what is listed for "X-KDE-Import=" in kpresenter_powerpoint_import.desktop :)
The mime-database (no idea starting with which version) has all the known inofficial as alias:
<alias type="application/powerpoint"/>
<alias type="application/mspowerpoint"/>
<alias type="application/x-mspowerpoint"/>
So if the minimal required version has that also, it should be fine to go just with the official. IMHO just the official one should be used in Calligra, any inofficial mimetype ids should be catched in other places. (no idea about OSX, that is why I left the plist template untouched).
(BTW, seems there is no entry for visio files even in the latest xdg shared-mime-info db :/ )
So shall I check and extend the patch for Words and others as well, with regard to the inofficial mimetypes for MS products?</pre>
</blockquote>
<p>On June 28th, 2012, 5:15 a.m., <b>Thorsten Zachmann</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;">I would say extending the patch would make sense. As a fast start we might use the officical onces and also backport that to 2.5</pre>
</blockquote>
<p>On June 28th, 2012, 6:20 a.m., <b>C. Boemann</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;">fixing this has my full support</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;">Seems that "application/mswords" is an exception within the MS mimetypes, perhaps because registered early. All other (well, the xls and ppt ones) are in the proper namespace: "application/vnd.ms-excel" and "application/vnd.ms-powerpoint", cmp. http://www.iana.org/assignments/media-types/application/index.html
Thus the rest of the code seems fine, nothing to extend the patch with.
So, green light for commit and backport as is?</pre>
<br />
<p>- Friedrich W. H.</p>
<br />
<p>On June 28th, 2012, 2:57 a.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 and Thorsten Zachmann.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated June 28, 2012, 2:57 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;">Ever wondered why Stage was not offered for old PPT files in Konqueror/Dolphin?
Reason is that stage.desktop only listed "application/mspowerpoint", but not "application/vnd.ms-powerpoint" as supported mimetype.
And the PowerPointImport import filter only accepts "application/vnd.ms-powerpoint" as input.
Attached patch fixes this, by changing the mimetype id in stage.desktop and removing the not supported mimetype "application/mspowerpoint" from stagepart.desktop and calligraactive.desktop
Okay to backport also to 2.5?</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>active/calligraactive.desktop <span style="color: grey">(dc79188)</span></li>
<li>stage/part/stage.desktop <span style="color: grey">(a823f9e)</span></li>
<li>stage/part/stagepart.desktop <span style="color: grey">(8f19f93)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105370/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>