This should test for the that case

<xsl:if test="umbraco.library:IsProtected($parent/@id, $parent/@path) = 0 or (umbraco.library:IsProtected($parent/@id, $parent/@path) = 1 and umbraco.library:IsLoggedOn() = 1)">

It's basically "If either this path isn't protected, or it is protected and the user is logged in)

Dan

Just realised I'm using a "parent" parameter, so here's the full xslt for context (it's for a subnav in this case)

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE xsl:Stylesheet [ <!ENTITY nbsp "&#x00A0;"> ]>
<xsl:stylesheet 
 version="1.0" 
 xmlns:xsl="http://www.w3.org/1999/XSL/Transform" 
 xmlns:msxml="urn:schemas-microsoft-com:xslt" xmlns:umbraco.library="urn:umbraco.library" exclude-result-prefixes="msxml 
umbraco.library">

<xsl:output method="html" omit-xml-declaration="yes"/>

<xsl:param name="currentPage"/>

<xsl:variable name="maxLevelForSitemap" select="6"/>

<xsl:template match="/">
    <xsl:variable name="parentNode" select="$currentPage/ancestor-or-self::node [@level=2]" />
      <xsl:if test="string($parentNode/@id) != ''">
           <h2><a href="{umbraco.library:NiceUrl($parentNode/@id)}"><xsl:value-of select="$parentNode/@nodeName"/></a></h2> 
           <xsl:call-template name="drawNodes"> 
                   <xsl:with-param name="parent" select="$parentNode"/>  
          </xsl:call-template>
        </xsl:if>
</xsl:template>


<xsl:template name="drawNodes">
 <xsl:param name="parent"/> 
 <xsl:if test="umbraco.library:IsProtected($parent/@id, $parent/@path) = 0 or (umbraco.library:IsProtected($parent/@id, $parent/@path) = 1 and umbraco.library:IsLoggedOn() = 1)">
       <ul class="subnav">
         <xsl:for-each select="$parent/node [string(./data [@alias='umbracoNaviHide']) != '1' and @level &lt;= $maxLevelForSitemap and @nodeTypeAlias!='NewsItem' and @nodeTypeAlias!='Event' and @nodeTypeAlias!='DateFolder']">   
             <li>
                    <xsl:if test="$currentPage/ancestor-or-self::node/@id = current()/@id">
                     <xsl:attribute name="class">currentli</xsl:attribute>
                 </xsl:if>  
                 <a href="{umbraco.library:NiceUrl(@id)}">
                   <xsl:if test="$currentPage/ancestor-or-self::node/@id = current()/@id">
                     <xsl:attribute name="class">current</xsl:attribute>
                   </xsl:if>
                   <xsl:value-of select="@nodeName"/></a>  
                  <xsl:if test="count(./node [string(./data [@alias='umbracoNaviHide']) != '1' and @level &lt;= $maxLevelForSitemap]) &gt; 0">   
                     <xsl:call-template name="drawNodes">    
                        <xsl:with-param name="parent" select="."/>    
                      </xsl:call-template>  
                  </xsl:if> 
              </li>
           </xsl:for-each>
     </ul>
   </xsl:if>
</xsl:template>
</xsl:stylesheet>

Dan

Thanks for your help Dan.  That seems to have nearly fixed the problem.  I have one remaining issue.  The navigation now seems to have security trimming, but will show the protected links for anyone who is logged on, regardless of their member group.  Is there a way to make sure that the security trimming is restricted to only those member groups that can actually access the page?

That worked - thanks for your help, Dan.

is working on a reply...