From c4e9a8ae153185d013a08cdd32d4eb5c3e23d6eb Mon Sep 17 00:00:00 2001
From: Ian Young <ian@iay.org.uk>
Date: Tue, 14 Jun 2011 09:57:38 +0000
Subject: [PATCH] Bugzilla 686: don't use variable references in match
 templates

---
 build/check.xsl | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/build/check.xsl b/build/check.xsl
index c669b169..c5f44885 100644
--- a/build/check.xsl
+++ b/build/check.xsl
@@ -38,12 +38,17 @@
 	<!--
 		Check for entities with OrganizationName elements which don't correspond to
 		a canonical owner name.
+		
+		Care needs to be taken if moving this template to another ruleset file, as it
+		matches all EntityDescriptor elements (with OrganizationName descendants)
+		whether or not the test will fail.
 	-->
-	<xsl:template match="md:EntityDescriptor[md:Organization/md:OrganizationName]
-		[not(ukfxMembers:isOwnerName($members, md:Organization/md:OrganizationName))]">
-		<xsl:call-template name="error">
-			<xsl:with-param name="m">unknown owner name: <xsl:value-of select="md:Organization/md:OrganizationName"/></xsl:with-param>
-		</xsl:call-template>
+	<xsl:template match="md:EntityDescriptor[md:Organization/md:OrganizationName]">
+		<xsl:if test="not(ukfxMembers:isOwnerName($members, md:Organization/md:OrganizationName))">
+			<xsl:call-template name="error">
+				<xsl:with-param name="m">unknown owner name: <xsl:value-of select="md:Organization/md:OrganizationName"/></xsl:with-param>
+			</xsl:call-template>
+		</xsl:if>
 	</xsl:template>