From a8647d6ce5efb671fb8e9b7fb3e7b0d4a2bd1846 Mon Sep 17 00:00:00 2001
From: Ian Young <ian@iay.org.uk>
Date: Tue, 5 Jul 2011 16:16:14 +0000
Subject: [PATCH] Don't generate MDUI statistics for an empty class, to prevent
 division by zero.

---
 build/statistics_mdui.xsl | 50 ++++++++++++++++++++++-----------------
 1 file changed, 28 insertions(+), 22 deletions(-)

diff --git a/build/statistics_mdui.xsl b/build/statistics_mdui.xsl
index 1e0a51d9..2efce7c3 100644
--- a/build/statistics_mdui.xsl
+++ b/build/statistics_mdui.xsl
@@ -39,37 +39,43 @@
         <xsl:value-of select="$entityCount"/>
         <xsl:text>&#10;</xsl:text>
 
-        <xsl:variable name="uiInfoEntities" select="$entities[descendant::mdui:UIInfo]"/>
-        <xsl:variable name="uiInfoEntitiesCount" select="count($uiInfoEntities)"/>
-        <xsl:text>   mdui:UIInfo: </xsl:text>
-        <xsl:value-of select="$uiInfoEntitiesCount"/>
-        <xsl:text> (</xsl:text>
-        <xsl:value-of select="format-number($uiInfoEntitiesCount div $entityCount, '0.0%')"/>
-        <xsl:text>)&#10;</xsl:text>
+        <xsl:if test="$entityCount > 0">
+            <xsl:variable name="uiInfoEntities" select="$entities[descendant::mdui:UIInfo]"/>
+            <xsl:variable name="uiInfoEntitiesCount" select="count($uiInfoEntities)"/>
+            <xsl:text>   mdui:UIInfo: </xsl:text>
+            <xsl:value-of select="$uiInfoEntitiesCount"/>
+            <xsl:text> (</xsl:text>
+            <xsl:value-of select="format-number($uiInfoEntitiesCount div $entityCount, '0.0%')"/>
+            <xsl:text>)&#10;</xsl:text>
+        </xsl:if>
         
         <xsl:text>Identity providers: </xsl:text>
         <xsl:value-of select="$idpCount"/>
         <xsl:text>&#10;</xsl:text>
-        
-        <xsl:variable name="idp.uiinfo" select="$idps[descendant::mdui:UIInfo]"/>
-        <xsl:variable name="idp.uiinfo.count" select="count($idp.uiinfo)"/>
-        <xsl:text>   mdui:UIInfo: </xsl:text>
-        <xsl:value-of select="$idp.uiinfo.count"/>
-        <xsl:text> (</xsl:text>
-        <xsl:value-of select="format-number($idp.uiinfo.count div $idpCount, '0.0%')"/>
-        <xsl:text>)&#10;</xsl:text>
+
+        <xsl:if test="$idpCount > 0">
+            <xsl:variable name="idp.uiinfo" select="$idps[descendant::mdui:UIInfo]"/>
+            <xsl:variable name="idp.uiinfo.count" select="count($idp.uiinfo)"/>
+            <xsl:text>   mdui:UIInfo: </xsl:text>
+            <xsl:value-of select="$idp.uiinfo.count"/>
+            <xsl:text> (</xsl:text>
+            <xsl:value-of select="format-number($idp.uiinfo.count div $idpCount, '0.0%')"/>
+            <xsl:text>)&#10;</xsl:text>
+        </xsl:if>
 
         <xsl:text>Service providers: </xsl:text>
         <xsl:value-of select="$spCount"/>
         <xsl:text>&#10;</xsl:text>
                 
-        <xsl:variable name="sp.uiinfo" select="$sps[descendant::mdui:UIInfo]"/>
-        <xsl:variable name="sp.uiinfo.count" select="count($sp.uiinfo)"/>
-        <xsl:text>   mdui:UIInfo: </xsl:text>
-        <xsl:value-of select="$sp.uiinfo.count"/>
-        <xsl:text> (</xsl:text>
-        <xsl:value-of select="format-number($sp.uiinfo.count div $spCount, '0.0%')"/>
-        <xsl:text>)&#10;</xsl:text>
+        <xsl:if test="$spCount > 0">
+            <xsl:variable name="sp.uiinfo" select="$sps[descendant::mdui:UIInfo]"/>
+            <xsl:variable name="sp.uiinfo.count" select="count($sp.uiinfo)"/>
+            <xsl:text>   mdui:UIInfo: </xsl:text>
+            <xsl:value-of select="$sp.uiinfo.count"/>
+            <xsl:text> (</xsl:text>
+            <xsl:value-of select="format-number($sp.uiinfo.count div $spCount, '0.0%')"/>
+            <xsl:text>)&#10;</xsl:text>
+        </xsl:if>
 
     </xsl:template>