From Michael Stahl <mstahl@redhat.com>:
Michael Stahl has uploaded a new change for review.
Change subject: fdo#38056: ODF import: fix page style attributes:
......................................................................
fdo#38056: ODF import: fix page style attributes:
PageMasterImportPropMapper: in the case of a single "fo:border"
and "style:border-line-width" attribute, it is possible that the border
is imported wrongly, like this:
1. pAllBorderProperty is set from the imported value, with name "TopBorder"
2. individual pNewBorder[i] are created as copies from pAllBorderProperty,
   one of which also with name "TopBorder"
3. pNewBorder[i] is updated with widths from pBorderWidths[i]
4. the individual pNewBorder[i] are added to the property vector
5. the property vector is sorted by property name
6. the properites are applied in order; if the pNewBorder[TOP]
   happens to precede the pAllBorderProperty (which is indeterminate
   as they both have name "TopBorder"), then the pAllBorderProperty
   will overwrite the border widths computed in step 3.
Thus, nerf the various pAllFoo properties so they do not override
the individual Foo properties later on.
Change-Id: I87755f1184d59da2aa72ac053e6f77d7295d6958
(cherry picked from commit 7f9928bfa561ccb6ed4e2baacc7d6960bc1ce231)
---
M xmloff/source/style/PageMasterImportPropMapper.cxx
1 file changed, 51 insertions(+), 0 deletions(-)
  git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/34/334/1
--
To view, visit https://gerrit.libreoffice.org/334
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I87755f1184d59da2aa72ac053e6f77d7295d6958
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-5
Gerrit-Owner: Michael Stahl <mstahl@redhat.com>
Context
- [PATCH] Change in core[libreoffice-3-5]: fdo#38056: ODF import: fix	page style attributes: · Gerrit
 
   
 
  Privacy Policy |
  
Impressum (Legal Info) |
  
Copyright information: Unless otherwise specified, all text and images
  on this website are licensed under the
  
Creative Commons Attribution-Share Alike 3.0 License.
  This does not include the source code of LibreOffice, which is
  licensed under the Mozilla Public License (
MPLv2).
  "LibreOffice" and "The Document Foundation" are
  registered trademarks of their corresponding registered owners or are
  in actual use as trademarks in one or more countries. Their respective
  logos and icons are also subject to international copyright laws. Use
  thereof is explained in our 
trademark policy.