forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thorsten Scherler <thors...@apache.org>
Subject Re: Error transforming Table IDs
Date Mon, 25 Jul 2005 13:54:31 GMT
On Mon, 2005-07-25 at 15:44 +0200, Ferdinand Soethe wrote:
> Looks like an oversight. When (in file
> main\webapp\skins\common\xslt\html\document2html.xsl)
> 
> <xsl:template match="table">
>     <xsl:apply-templates select="@id"/>
>     <table cellpadding="4" cellspacing="1" class="ForrestTable">
>       <xsl:copy-of select="@cellspacing | @cellpadding | @border |
>       @class | @bgcolor"/>
>       <xsl:apply-templates/>
>     </table>
>   </xsl:template>
> 
> id is used to apply a templates that creates the local anchor (to make
> ids addressable with local anchors) the id-attribute is 'used up' in
> the process instead of also being copied over in
> 
> <xsl:copy-of select="@cellspacing | @cellpadding | @border | @class | @bgcolor"/>
> 
> Adding @id to the list of copied attributes fixes the problem.
> 
> <xsl:copy-of select="@cellspacing | @cellpadding | @border | @class | @bgcolor |@id"/>
> 
> In my view this fix should be applied to all elements that use 'up id' in
> this way. Any reason not to do that?
> 

I am more for 
<xsl:copy-of select="@*"/>

That will be better for maintainment. I could not see anything in the
code that would speak against that and the fix you suggested. 

salu2
thorsten
> 
> --
> Ferdinand Soethe
> 
-- 
thorsten

"Together we stand, divided we fall!" 
Hey you (Pink Floyd)


Mime
View raw message