Commit (STABLE): Re: commit (head): fix 8230

Backport: fix 8230 (tomas)

CVS:
----------------------------------------------------------------------
CVS: Enter Log. Lines beginning with `CVS:' are removed automatically
CVS:
CVS: Committing in .
CVS:
CVS: Modified Files:
CVS: Tag: ABI-2-2-0-STABLE
CVS: ./src/text/fmt/xp/fl_FrameLayout.cpp
CVS: ./src/text/fmt/xp/fl_TableLayout.cpp
CVS:
----------------------------------------------------------------------

On Sun, 2005-04-10 at 09:48 +0100, Tomas Frydrych wrote:
>
> This fixes 8230 (both for textboxes and tables). The bug was caused by
> the frame and table layouts calling directly PD_Document::getAttrProp()
> to retrieved the associated AP -- this is a reall no-no, as that
> function does not parse the revisions attribute; all container layouts
> need to use fl_ContainerLayout::getAP() to obtain their AP.
>
> Quick grep through fmt/xp suggests that there will be some more bugs of
> this type; in particular the new fp_EmbedRun and
> fp_MathRun::_getLayoutPropFromObject() bypass the revisioning mechanism,
> but I will need to familarise myself with that code bit more before
> daring to change anything.
>
> Tomas
>
>
>
Received on Thu Apr 14 01:58:37 2005