<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/107534/">http://git.reviewboard.kde.org/r/107534/</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 5a71ddb36ccdd828a4ca30d1835a470dc2f13cdc by Friedrich W. H. Kossebau to branch calligra/2.6.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On November 30th, 2012, 2:11 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 and C. Boemann.</div>
<div>By Friedrich W. H. Kossebau.</div>


<p style="color: grey;"><i>Updated Nov. 30, 2012, 2:11 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;">There is some serious 3rd-party user of Calligra who sets KoRdfSemanticItems to text ranges, but updates the content itself, because the content is not just text, but e.g. charts. He does so because the current KoSemanticStylesheet assumes there is just text used to display the content of the item.
The inner systen of the KoRdf* classes assume there is always at least one stylesheet set for an item, this assumption is assured with some asserts(). Removing this assumption will result in quite some changes which I feel I do not oversee, as still not too familiar with all that code.

A simple fix I came now up with and which has been tested successfully with the 3rd-party code is to set the template string of the stylesheet to a special string, which the stylesheet detects and if present then will not do any changes to the text belonging to the semantic item.

Not nice, but efficient for 2.6 at least.

Should that string be documented in the API dox?
Any better idea for the string?</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/rdf/KoSemanticStylesheet.cpp <span style="color: grey">(ed0072c)</span></li>

</ul>

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




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








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