[Libreoffice-commits] core.git: Branch 'aoo/trunk' - svgio/source

[Armin Le Grand]

 svgio/source/svgreader/svgnode.cxx |   33 ++++++++++++++++++++++++++++++++-
 1 file changed, 32 insertions(+), 1 deletion(-)

New commits:
commit 2610028a8a1465006059c504a5230b67f550406c
Author: Armin Le Grand <alg at apache.org>
Date:   Fri Jun 14 17:06:00 2013 +0000

    i122522 apply needed order to CSS styles to presentation styles

diff --git a/svgio/source/svgreader/svgnode.cxx b/svgio/source/svgreader/svgnode.cxx
index 4a6f266..438d3c3 100644
--- a/svgio/source/svgreader/svgnode.cxx
+++ b/svgio/source/svgreader/svgnode.cxx
@@ -185,12 +185,43 @@ namespace svgio
         void SvgNode::parseAttributes(const com::sun::star::uno::Reference< com::sun::star::xml::sax::XAttributeList >& xAttribs)
         {
             const sal_uInt32 nAttributes(xAttribs->getLength());
+            // #122522# SVG defines that 'In general, this means that the presentation attributes have
+            // lower priority than other CSS style rules specified in author style sheets or ‘style’
+            // attributes.' in http://www.w3.org/TR/SVG/styling.html#UsingPresentationAttributes
+            // (6.4 Specifying properties using the presentation attributes SVG 1.1). That means that
+            // e.g. font-size will appear as presentation attribute and CSS style attribute. In these
+            // cases, CSS style attributes need to have precedence. To do so it is possible to create
+            // a proirity system for all properties of a shape, but it will also work to parse the
+            // presentation attributes of type 'style' last, so they will overwrite the less-prioritized
+            // already interpreted ones. Thus, remember SVGTokenStyle entries and parse them last.
+            // To make this work it is required that parseAttribute is only called by parseAttributes
+            // which is the case.
+            std::vector< sal_uInt32 > aSVGTokenStyleIndexes;
 
             for(sal_uInt32 a(0); a < nAttributes; a++)
             {
                 const ::rtl::OUString aTokenName(xAttribs->getNameByIndex(a));
+                const SVGToken aSVGToken(StrToSVGToken(aTokenName));
 
-                parseAttribute(aTokenName, StrToSVGToken(aTokenName), xAttribs->getValueByIndex(a));
+                if(SVGTokenStyle == aSVGToken)
+                {
+                    // #122522# remember SVGTokenStyle entry
+                    aSVGTokenStyleIndexes.push_back(a);
+                }
+                else
+                {
+                    parseAttribute(aTokenName, StrToSVGToken(aTokenName), xAttribs->getValueByIndex(a));
+                }
+            }
+
+            // #122522# parse SVGTokenStyle entries last to override already interpreted
+            // 'presentation attributes' of potenially the same type
+            for(sal_uInt32 b(0); b < aSVGTokenStyleIndexes.size(); b++)
+            {
+                const sal_uInt32 nSVGTokenStyleIndex(aSVGTokenStyleIndexes[b]);
+                const ::rtl::OUString aTokenName(xAttribs->getNameByIndex(nSVGTokenStyleIndex));
+
+                parseAttribute(aTokenName, SVGTokenStyle, xAttribs->getValueByIndex(nSVGTokenStyleIndex));
             }
         }