Text saving broken

[Pierre]

On Thursday 14 April 2011 10:20:23 Thorsten Zachmann wrote:
> Hello,
> 
> when I wanted to test some stuff I found out that saving of text is
> currently broken.
> 
> At the moment we save text like that:
> 
>         <draw:frame draw:style-name="gr2" draw:id="shape3" draw:layer=""
> svg:width="300.00000000000pt" svg:height="200.00000000000pt"
> svg:x="108.74767855096pt" svg:y="113.47540983607pt">
>           <draw:text-box text:style-name="P3" text:id="subitem1">This is a
> text</draw:text-box>
>         </draw:frame>
> 
> This is wrong as it should look like
> 
>         <draw:frame draw:style-name="gr2" draw:id="shape3" draw:layer=""
> svg:width="391.41463414634pt" svg:height="239.36675700090pt"
> svg:x="102.82926829268pt" svg:y="226.19060523939pt">
>           <draw:text-box>
>             <text:p text:style-name="P3" text:id="subitem1">This is a
> text</text:p>
>           </draw:text-box>
>         </draw:frame>
> 
> We now miss the <text:p> tag.
> 
> The change causing that is at least after commit
> 0e18ccbabb634a4769434452a960870549373691
> 
> which still works without problems.
> 
> Would  be nice if someone who had been working on text saving could have a
> look at that.
Fixed, was an unhappy consequence of the change tracking work.

@Ganesh : I don't know the change tracking specification, but I really don't 
like the way the code had to be modified to support it. Is there no way to have 
it in a less mixed-in way ? I mean, right now, you can't tell between the 
classical text related calls and the calls for change tracking. It was already 
messy and dirty enough... Moreover, do unit tests only cover the case where 
change tracking is enabled ? In any case, there is a real issue since the 
regressions I fixed here are major and should have been spotted far sooner... 
(I'll try to find some ways to improve our unit testing in that area anyway).
But, for instance, you often check a save format, between deltaxml and odf 1.2 : 
what are the differences ? Why are both needed ?
If you don't mind, a short explanation email would be helpful (or an IRC session 
if you prefer...)

 Pierre
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110415/99f712bb/attachment.sig>