<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/107353/">http://git.reviewboard.kde.org/r/107353/</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;">I agree with the move. Have you checked with the author, David Faure, whether he had any reason to keep it in libs/main? </pre>
<br />
<p>- Boudewijn</p>
<br />
<p>On November 17th, 2012, 2:58 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.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated Nov. 17, 2012, 2:58 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;">Did you know that Calligra also installs a fileproperties dialog plugin, to show the general meta data and the author info?
I did not, until I wondered where those pages in the fileproperties dialog plugin came from (e.g. select "Properties" on a ODF file in Dolphin and select the "General" or "Last saved by" tabs).
This plugin better lives next to its cousin, the thumbnailer plugin, in the tools part of the sources tree (even if developing tools and user tools ideally would be in separate tree branches in the sources).
So the attached patch moves all files belonging to this plugin into the new dir "tools/properties" (any better name proposal?). Additionally it makes it also work again, by teaching KoDocumentInfoDlg to also deal with a KoDocumentInfo object without an attached KoDocument, like in the case of the fileproperties plugin, and turning off the entries in the "General" tab which are redundant in the fileproperties dialog (path & type).
Okay to backport to 2.6?
Setting the Authors profile does not work correctly currently, but that is a separate problem I will look next into. The dialog shows at least any set Author info correctly.</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>libs/main/CMakeLists.txt <span style="color: grey">(ae68a2b)</span></li>
<li>libs/main/KoDocInfoPropsFactory.cpp <span style="color: grey">(c1d89b9)</span></li>
<li>libs/main/KoDocumentInfoDlg.cpp <span style="color: grey">(920c05c)</span></li>
<li>libs/main/koDocumentInfoAboutWidget.ui <span style="color: grey">(a5133ed)</span></li>
<li>libs/main/kodocinfopropspage.desktop <span style="color: grey">(5acfbc0)</span></li>
<li>tools/CMakeLists.txt <span style="color: grey">(b0eb900)</span></li>
<li>tools/properties/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>tools/properties/KoDocInfoPropsFactory.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>tools/properties/calligradocinfopropspage.desktop <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107353/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>