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





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit b92d17adc364169dde05440c209a217c35dadf26 by Thorsten Zachmann to branch master.</pre>
 <br />









<p>- Commit</p>


<br />
<p>On March 9th, 2013, 5:32 p.m. UTC, Thorsten Zachmann wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Calligra and Matus Uzak.</div>
<div>By Thorsten Zachmann.</div>


<p style="color: grey;"><i>Updated March 9, 2013, 5:32 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;">When testing roundtrip for headers/footers in words it happens that the header/footers are no longer shown after roundtrip. Problem is that the style used for the page style gets it's name change but is expected to be standard.
This patch adds the possibility to have same style names if the family is different.

This also fixes an import problem for doc files when the char and paragrpah style has the same name then the wrong style was used.</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;">cstester and roundtrip testing </pre>
  </td>
 </tr>
</table>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=316412">316412</a>


</div>


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

 <li>filters/libmso/ODrawToOdf.cpp <span style="color: grey">(ac589a9)</span></li>

 <li>filters/libmsooxml/MsooXmlUtils.cpp <span style="color: grey">(d45089a)</span></li>

 <li>filters/sheets/xlsx/XlsxXmlWorksheetReader.cpp <span style="color: grey">(a4e2b5f)</span></li>

 <li>filters/words/docx/DocxXmlDocumentReader.h <span style="color: grey">(97916ee)</span></li>

 <li>filters/words/docx/DocxXmlDocumentReader.cpp <span style="color: grey">(df81026)</span></li>

 <li>filters/words/msword-odf/paragraph.cpp <span style="color: grey">(3308d8d)</span></li>

 <li>filters/words/msword-odf/tablehandler.cpp <span style="color: grey">(285f4f7)</span></li>

 <li>filters/words/msword-odf/texthandler.cpp <span style="color: grey">(4473736)</span></li>

 <li>libs/koreport/renderer/odtframe/KoOdtFrameReportDocument.cpp <span style="color: grey">(51c5958)</span></li>

 <li>libs/kotext/styles/KoStyleManager.cpp <span style="color: grey">(c2a64e9)</span></li>

 <li>libs/odf/KoGenStyle.cpp <span style="color: grey">(11b1888)</span></li>

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

 <li>libs/odf/KoGenStyles.cpp <span style="color: grey">(7a92453)</span></li>

 <li>libs/odf/tests/TestKoGenStyles.cpp <span style="color: grey">(5818091)</span></li>

</ul>

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







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








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