<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/100629/">http://git.reviewboard.kde.org/r/100629/</a>
     </td>
    </tr>
   </table>
   <br />


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 Inge Wallin.</div>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Updated to fix nearly all issues found by Zagge.

This update merges the KoEmbeddedFileSaver into KoEmbeddedDocumentSaver.  It also removes many small changes to make the updated diff as small as possible against master.

The combined saver should be called KoEmbeddedObjectSaver, but that can be done after commit of the main code, according to Zagge.  I agree with that.

There is still one issue, though: The resulting saved file does not load into OOwriter again.  I have been racking my head why that is, but I can't find the reason.  The XML inside ODF seems correct. Perhaps somebody else can help me here?</pre>
  </td>
 </tr>
</table>


<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;">This patch implements saving in the KoUnavailShape.

The method it uses is to introduce a new class, KoEmbeddedFileSaver, in koodf. This class is a bit like the KoEmbeddedObjectSaver class, and is created and instantiated in the same places. It is also part of the KoShapeSavingContext.

As a bonus, this patch also implements saving for the Vector shape, since I used that for testing in early stages.</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;">Tested with a simple odt file with an embedded spreadsheet.</pre>
  </td>
 </tr>
</table>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>libs/flake/KoDrag.cpp <span style="color: grey">(8ed2656)</span></li>

 <li>libs/flake/KoDragOdfSaveHelper.cpp <span style="color: grey">(727e6c5)</span></li>

 <li>libs/flake/KoShapeSavingContext.h <span style="color: grey">(a06e040)</span></li>

 <li>libs/flake/KoShapeSavingContext.cpp <span style="color: grey">(fd8fb41)</span></li>

 <li>libs/flake/KoUnavailShape.cpp <span style="color: grey">(3f10acf)</span></li>

 <li>libs/kopageapp/KoPAOdfPageSaveHelper.h <span style="color: grey">(d8f064a)</span></li>

 <li>libs/kopageapp/KoPAOdfPageSaveHelper.cpp <span style="color: grey">(c60d0d8)</span></li>

 <li>libs/kopageapp/KoPAPastePage.cpp <span style="color: grey">(01bafe0)</span></li>

 <li>libs/kopageapp/KoPASavingContext.h <span style="color: grey">(c678e2c)</span></li>

 <li>libs/kopageapp/KoPASavingContext.cpp <span style="color: grey">(52524bf)</span></li>

 <li>libs/kotext/KoTextDrag.cpp <span style="color: grey">(f30cb5c)</span></li>

 <li>libs/kotext/KoTextOdfSaveHelper.h <span style="color: grey">(155aecd)</span></li>

 <li>libs/kotext/KoTextOdfSaveHelper.cpp <span style="color: grey">(d012557)</span></li>

 <li>libs/kotext/KoTextShapeSavingContext.h <span style="color: grey">(4d27bc4)</span></li>

 <li>libs/kotext/KoTextShapeSavingContext.cpp <span style="color: grey">(d5ade20)</span></li>

 <li>libs/kotext/opendocument/tests/TestChangeLoading.cpp <span style="color: grey">(ac42ddd)</span></li>

 <li>libs/kotext/opendocument/tests/TestLoading.cpp <span style="color: grey">(e6a790a)</span></li>

 <li>libs/main/KoDocument.cpp <span style="color: grey">(956e179)</span></li>

 <li>libs/odf/CMakeLists.txt <span style="color: grey">(1a6a131)</span></li>

 <li>libs/odf/KoEmbeddedDocumentSaver.h <span style="color: grey">(11cf075)</span></li>

 <li>libs/odf/KoEmbeddedDocumentSaver.cpp <span style="color: grey">(a8e13d8)</span></li>

 <li>libs/odf/KoOdfDocument.h <span style="color: grey">(c242776)</span></li>

 <li>libs/odf/KoOdfLoadingContext.h <span style="color: grey">(023b92f)</span></li>

 <li>libs/odf/KoOdfLoadingContext.cpp <span style="color: grey">(ce53992)</span></li>

 <li>libs/odf/KoOdfManifest.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>libs/odf/KoOdfManifest.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>plugins/vectorshape/VectorShape.cpp <span style="color: grey">(9965d68)</span></li>

 <li>tables/Cell.cpp <span style="color: grey">(7aa8306)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/100629/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>




  </div>
 </body>
</html>