[Mono-docs-list] More monodocer

[Jonathan Pryor]

On Sun, 2006-10-08 at 21:21 -0400, Jonathan Pryor wrote:
> On Sun, 2006-10-08 at 18:18 -0400, Joshua Tauberer wrote:
> > monodocs2html:
> > 
> > Include inherited members in the member listing for a type, if the
> > type's ancestor types are documented in the same set.
> Index: DocTest/html.expected/Mono.DocTest.Generic/MyList`1.html
> ===================================================================
> --- DocTest/html.expected/Mono.DocTest.Generic/MyList`1.html    (revision 66435)
> +++ DocTest/html.expected/Mono.DocTest.Generic/MyList`1.html    (working copy)
> @@ -98,7 +98,7 @@
>                <td>
>                  <b>
>                    <a href="#M:Mono.DocTest.Generic.MyList`1.GetHelper`2">GetHelper&lt;U,V&gt;</a>
> -                </b>()<nobr>:<a href="../Mono.DocTest.Generic/MyList`1+Helper`2.html">MyList&lt;T&gt;.Helper&lt;U,V&gt;</a></nobr><div>To be added.</div></td>
> +                </b>()<nobr> : <a href="../Mono.DocTest.Generic/MyList`1+Helper`2.html">MyList&lt;T&gt;.Helper&lt;U,V&gt;</a></nobr><div>To be added.</div></td>
>              </tr>
>              <tr valign="top">
>                <td>
> @@ -129,7 +129,7 @@
>                </td>
>                <td>
>                  <b>
> -                  <a href="#M:Mono.DocTest.Generic.MyList`1.UseHelper`2(Mono.DocTest.Generic.MyList{`1}.Helper{``1,``2})">UseHelper&lt;U,V&gt;</a>
> +                  <a href="#M:Mono.DocTest.Generic.MyList`1.UseHelper`2(Mono.DocTest.Generic.MyList{T}.Helper{``1,``2})">UseHelper&lt;U,V&gt;</a>
>                  </b>(<a href="../Mono.DocTest.Generic/MyList`1+Helper`2.html">MyList&lt;T&gt;.Helper&lt;U,V&gt;</a>)<div>To be added.</div></td>
>              </tr>
>            </table>
> 
> Your changes broke something: {T} should continue to be {`1} within the
> href="..." text.

I imagine that this change was created due to this change in
stylesheet.xsl:

@@ -1182,11 +1241,31 @@
                        <xsl:sort select="Parameters/Parameter/@Type"/>
                        
                        <xsl:variable name="linkid">
+                               <xsl:if test="not(parent::Members/@FullName = $TypeFullName)">
+                                       <xsl:call-template name="GetLinkTarget">
+                                               <xsl:with-param name="type" select="parent::Members/@FullName" />
+                                               <xsl:with-param name="local-suffix"/>
+                                               <xsl:with-param name="remote"/>
+                                               <xsl:with-param name="empty-if-this" select="1"/>
+                                       </xsl:call-template>
+                               </xsl:if>
+
+                               <xsl:text>#</xsl:text>
                                <xsl:call-template name="GetLinkId" >
-                                       <xsl:with-param name="type" select="../.." />
+                                       <xsl:with-param name="type" select="parent::Members" />
                                        <xsl:with-param name="member" select="." />
                                </xsl:call-template>
                        </xsl:variable>

I'm not too familiar with the extended select syntax, but I imagine that
"../.." isn't the same as "paremt::Members".  At least, this is the only
reason I can think of for GetLinkId to return the wrong string,
especially since I don't see any other related change.

 - Jon