From 7e09de761b220aaa0889315af52fd083f2c1e055 Mon Sep 17 00:00:00 2001
From: Colomban Wendling <cwendling@hypra.fr>
Date: Thu, 13 Sep 2018 11:56:31 +0200
Subject: [PATCH] Simplify code renaming a parameter

Rely on XSLT keeping the last attribute in case of duplicates.
---
 strip-frames.xsl | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/strip-frames.xsl b/strip-frames.xsl
index 99ebf7b..4a4e0bf 100644
--- a/strip-frames.xsl
+++ b/strip-frames.xsl
@@ -61,18 +61,11 @@
                      text:h[draw:frame]">
   <xsl:comment>Removed outer <xsl:value-of select="name()"/></xsl:comment>
   <xsl:copy>
-    <!-- FIXME: there's probably a better way to do that -->
-    <xsl:for-each select="@*">
-      <xsl:choose>
-        <xsl:when test="../@text:style-name = .">
-          <xsl:attribute name="text:style-name">
-            <xsl:value-of select="concat('Placeholder', .)"/>
-          </xsl:attribute>
-        </xsl:when>
-        <xsl:otherwise>
-          <xsl:apply-templates select="."/>
-        </xsl:otherwise>
-      </xsl:choose>
+    <xsl:apply-templates select="@*"/>
+    <xsl:for-each select="@text:style-name">
+      <xsl:attribute name="{name(.)}">
+        <xsl:value-of select="concat('Placeholder', .)"/>
+      </xsl:attribute>
     </xsl:for-each>
   </xsl:copy>
   <xsl:apply-templates/>
-- 
GitLab