Why won't this for-each loop work?

[Mark Constant]

I have a xslt file and it keeps giving me an EOF error when it reaches
the point
<xsl:for-each select="lc:Ente rtainment/lc:$Hardware[lc:Genre =
$Genre]">
and
<xsl:for-each select="lc:Ente rtainment/lc:$Hardware">

My xslt file looks like this
<?xml version="1.0" encoding="ISO-8859-1"?>
<xsl:styleshe et version="1.0"
xmlns:xsl="http ://www.w3.org/1999/XSL/Transform"
xmlns:lc="http://mark/Homepage/">
<xsl:output method="html"/>
<xsl:param name="Hardware"/>
<xsl:param name="Genre"/>

<xsl:template match="/">
<xsl:choose>
<xsl:when test="$Genre = 'All'">
<xsl:call-template name="All" />
</xsl:when>
<xsl:otherwis e>
<xsl:call-template name="NotAll" />
</xsl:otherwise>
</xsl:choose>
</xsl:template>

<xsl:template name="All">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:$Hardware">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>

<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:$Hardware[lc:Genre =
$Genre]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>

</xsl:stylesheet>

My xml file looks like this
<Entertainmen t xmlns="http://mark/Homepage/">
<Movie>
<Title>Terminat or 1</Title>
<Description> A T100 machine comes back from the future to destroy
Sarah Connor who will give birth to a kid who will go on to save the
world.</Description>
<Genre>Action </Genre>
<Picture>images \movies\Termina tor1.jpg</Picture>
<Rating>5</Rating>
</Movie>
<Movie>
<Title>Terminat or 2</Title>
<Description>Th e T100 is back again only this time he is back to
save John Connor from a T1000</Description>
<Genre>Action </Genre>
<Picture>images \movies\Termina tor2.jpg</Picture>
<Rating>5</Rating>
</Movie>
<PS2>
<Title>Mortal Kombat Deadly Alliance</Title>
<Description>Qu an Chi and Shang Tsung join forces to destroy Lui
Kang and take over the wrold</Description>
<Genre>Fighting </Genre>
<Picture>images \ps2\MortalKomb at.jpg</Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>Sly Cooper</Title>
<Description> A racoon must travel around the world to get back
pieces of a book that has been in his family for
generations</Description>
<Genre>Platform er</Genre>
<Picture>images \ps2\SlyCooper</Picture>
<Rating>5</Rating>
</PS2>
</Entertainment>

I am passing my variable through a piece of Javascript code that looks
like this
xsltProc.addPar ameter("Hardwar e", Hardware);
xsltProc.addPar ameter("Genre", SelValue);

[Dimitre Novatchev]

"Mark Constant" <co*******@mi x-net.net> wrote in message
news:ce******** *************** **@posting.goog le.com...

I have a xslt file and it keeps giving me an EOF error when it reaches
the point
<xsl:for-each select="lc:Ente rtainment/lc:$Hardware[lc:Genre =
$Genre]">
and
<xsl:for-each select="lc:Ente rtainment/lc:$Hardware">

This is not legal.

lc:$Hardware is not a QName.

Probably you wanted:

lc:*[local-name() = $Hardware]

or (,which is the same):

*[name() = concat('lc:', $Hardware)]

=====
Cheers,

Dimitre Novatchev.
http://fxsl.sourceforge.net/ -- the home of FXSL

[Mark Constant]

I am still getting an error saying that Hardware is undefined. All I
want is a drop down list. If the user is on the PS2 page it will send
the parameter PS2 for Hardware and then whatever Genre they selected.
Here is what I changed my XSLT code to
<?xml version="1.0" encoding="ISO-8859-1"?>
<xsl:styleshe et version="1.0"
xmlns:xsl="http ://www.w3.org/1999/XSL/Transform"
xmlns:lc="http://mark/Homepage/">
<xsl:output method="html"/>
<xsl:param name="Hardware"/>
<xsl:param name="Genre"/>

<xsl:template match="/">
<xsl:choose>
<xsl:when test="$Genre = 'All'">
<xsl:call-template name="All" />
</xsl:when>
<xsl:otherwis e>
<xsl:call-template name="NotAll" />
</xsl:otherwise>
</xsl:choose>
</xsl:template>

<xsl:template name="All">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>

<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>
</xsl:stylesheet>

My Javascript code looks like this
function CallXML(dropdow n, hardware)
{
var myIndex = dropdown.select edIndex;
var SelValue = dropdown.option s[myIndex].value;
var xsltTree = new ActiveXObject(" Msxml2.FreeThre adedDOMDocument .4.0");
xsltTree.async = false;
xsltTree.load(" Entertainment.x slt")

var srcTree = new ActiveXObject(" Msxml2.DOMDocum ent.4.0");
srcTree.async = false;
srcTree.load("E ntertainment.xm l");

var xsltThread = new ActiveXObject(" Msxml2.XSLTempl ate.4.0");
xsltThread.styl esheet = xsltTree;
var xsltProc = xsltThread.crea teProcessor();
xsltProc.input = srcTree;
xsltProc.addPar ameter("Hardwar e", Hardware);
xsltProc.addPar ameter("Genre", SelValue);
xsltProc.transf orm();
GameTable.inner HTML = xsltProc.output ;
}

[Dimitre Novatchev]

"Mark Constant" <co*******@mi x-net.net> wrote in message
news:ce******** *************** ***@posting.goo gle.com...

I am still getting an error saying that Hardware is undefined.
This is not a Javascript group. The error is in your script -- see below.
All I
want is a drop down list. If the user is on the PS2 page it will send
the parameter PS2 for Hardware and then whatever Genre they selected.
Here is what I changed my XSLT code to
<?xml version="1.0" encoding="ISO-8859-1"?>
<xsl:styleshe et version="1.0"
xmlns:xsl="http ://www.w3.org/1999/XSL/Transform"
xmlns:lc="http://mark/Homepage/">
<xsl:output method="html"/>
<xsl:param name="Hardware"/>
<xsl:param name="Genre"/>

<xsl:template match="/">
<xsl:choose>
<xsl:when test="$Genre = 'All'">
<xsl:call-template name="All" />
</xsl:when>
<xsl:otherwis e>
<xsl:call-template name="NotAll" />
</xsl:otherwise>
</xsl:choose>
</xsl:template>

<xsl:template name="All">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>

<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>
</xsl:stylesheet>

My Javascript code looks like this
function CallXML(dropdow n, hardware)
{
var myIndex = dropdown.select edIndex;
var SelValue = dropdown.option s[myIndex].value;
var xsltTree = new ActiveXObject(" Msxml2.FreeThre adedDOMDocument .4.0");
xsltTree.async = false;
xsltTree.load(" Entertainment.x slt")

var srcTree = new ActiveXObject(" Msxml2.DOMDocum ent.4.0");
srcTree.async = false;
srcTree.load("E ntertainment.xm l");

var xsltThread = new ActiveXObject(" Msxml2.XSLTempl ate.4.0");
xsltThread.styl esheet = xsltTree;
var xsltProc = xsltThread.crea teProcessor();
xsltProc.input = srcTree;
xsltProc.addPar ameter("Hardwar e", Hardware);
You don't have a variable named "Hardware" defined in your Javascript code.
The line above will raise an error due to this.
xsltProc.addPar ameter("Genre", SelValue);
xsltProc.transf orm();
GameTable.inner HTML = xsltProc.output ;
}

[Mark Constant]

I fixed the Javascript error easy enough once I noticed my mistake.
Now I get an error that seems to be a FAQ that reads "Reference to
variable or parameter 'Hardware' must evaluate to a node set list. Now
from what I have been reading I have to add a function called
node-set() but where I need to add it is what I can't figure out. Here
is my XSLT code.
<?xml version="1.0" encoding="ISO-8859-1"?>
<xsl:styleshe et version="1.0"
xmlns:xsl="http ://www.w3.org/1999/XSL/Transform"
xmlns:lc="http://mark/Homepage/">
<xsl:output method="html"/>
<xsl:param name="Hardware"/>
<xsl:param name="Genre"/>

<xsl:template match="/">
<xsl:choose>
<xsl:when test="$Genre = 'All'">
<xsl:call-template name="All" />
</xsl:when>
<xsl:otherwis e>
<xsl:call-template name="NotAll" />
</xsl:otherwise>
</xsl:choose>
</xsl:template>

<xsl:template name="All">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>

<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>
</xsl:stylesheet>

[Dimitre Novatchev]

"Mark Constant" <co*******@mi x-net.net> wrote in message news:ce******** *************** ***@posting.goo gle.com...

I fixed the Javascript error easy enough once I noticed my mistake.
Now I get an error that seems to be a FAQ that reads "Reference to
variable or parameter 'Hardware' must evaluate to a node set list. Now
from what I have been reading I have to add a function called
node-set() but where I need to add it is what I can't figure out. Here
is my XSLT code.

using the xxx:node-set() extension function won't help.

From your code it can be seen that in different places you're using
the value of the "Hardware" parameter both as a simple value (string)
and as a node-set.

Most probably you're passing a string as the value of the "Hardware"
parameter.

However in the "NotAll" template you're using it as a node-set:
<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">
BTW the above hardly makes sense to me. What did you really want?
Please, say it with words.

Probably your problem is here and you have to correct it.

Anyway, should you continue to use the same parameter supposing it has
two different types at the same time, the solution is to use two
different parameters -- one string and one node-set.
=====
Cheers,

Dimitre Novatchev.
http://fxsl.sourceforge.net/ -- the home of FXSL



<?xml version="1.0" encoding="ISO-8859-1"?>
<xsl:styleshe et version="1.0"
xmlns:xsl="http ://www.w3.org/1999/XSL/Transform"
xmlns:lc="http://mark/Homepage/">
<xsl:output method="html"/>
<xsl:param name="Hardware"/>
<xsl:param name="Genre"/>

<xsl:template match="/">
<xsl:choose>
<xsl:when test="$Genre = 'All'">
<xsl:call-template name="All" />
</xsl:when>
<xsl:otherwis e>
<xsl:call-template name="NotAll" />
</xsl:otherwise>
</xsl:choose>
</xsl:template>

<xsl:template name="All">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>

<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>
</xsl:stylesheet>

[Mark Constant]

I can see now that the problem lies within the NotAll template. It
will take a little while to explain in words but here I go. I have an
xml file that supports three types of hardware PS2, XBOX, and Movie.
Now if the user is on the PS2 and selects that they want the genre
horror it will send two parameters. One being the Hardware (i.e. PS2)
and then the Genre which is horror. From there the template calls
NotAll and says for every instance of PS2 in the XML that has a Genre
of horror then display it. Here is my Xslt again.
<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">
<tr>
<th colspan="2" bgcolor="Green" ><xsl:value-of
select="lc:Titl e"/></th>
</tr>
<tr>
<td>
<xsl:element name="IMG">
<xsl:attribut e name="SRC">
<xsl:value-of select="lc:Pict ure"/>
</xsl:attribute>
</xsl:element>
</td>
<td>
<xsl:value-of select="lc:Desc ription"/>
</td>
</tr>
</xsl:for-each>
</table>
</xsl:template>

It is obvious that I need to change <xsl:for-each
select="lc:Ente rtainment/lc:*[local-name() = $Hardware[lc:Genre =
$Genre]]"> but what I need to change it to is what I am wondering.
Basically I want this line to say whatever Hardware the user pass
display the xml content only if the genre is horror. Here is an
example of my xml file.

<Entertainmen t xmlns="http://mark/Homepage/">
<PS2>
<Title>Grand Theft Auto 3</Title>
<Description> An open ended enviroment where you have to take on jobs
for different mob bosses to stay alive</Description>
<Genre>TPA</Genre>
<Picture>images \ps2\GrandTheft Auto3.gif</Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>Maximo </Title>
<Description>Af ter returning from a war you find out that you must
save the love of your life from your one time friend turned
nemesis</Description>
<Genre>Platform er</Genre>
<Picture>images \ps2\Maximo.gif </Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>Mortal Kombat Deadly Alliance</Title>
<Description>Qu an Chi and Shang Tsung join forces to destroy Lui
Kang and take over the world</Description>
<Genre>Fighting </Genre>
<Picture>images \ps2\MortalKomb at.gif</Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>Ratche t and Clank</Title>
<Description> In this action platformer you must play as a Cat and
Robot team that must work together to save the world from the evil
Chairman Drek</Description>
<Genre>Platform er</Genre>
<Picture>images \ps2\RatchetAnd Clank.gif</Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>Silent Hill 2</Title>
<Description>Af ter recieving a letter from a wife who you thought
was dead you travel to a huanted town to find out the
truth</Description>
<Genre>Horror </Genre>
<Picture>images \ps2\SilentHill 2.gif</Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>Sly Cooper</Title>
<Description> A racoon must travel around the world to get back
pieces of a book that has been in his family for
generations</Description>
<Genre>Platform er</Genre>
<Picture>images \ps2\SlyCooper. gif</Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>The Getaway</Title>
<Description>Yo u play as Mark Hammond who is taken back into the mob
life to save the life of his own child</Description>
<Genre>Platform er</Genre>
<Picture>images \ps2\TheGetaway .gif</Picture>
<Rating>5</Rating>
</PS2>
<PS2>
<Title>Tribes : Ariel Assualt</Title>
<Description>Th is game is a port of a old PC game where you can fly
using jetpacks but also use vehicles</Description>
<Genre>FPS</Genre>
<Picture>images \ps2\Tribes.gif </Picture>
<Rating>3</Rating>
</PS2>
<PS2>
<Title>Virtua Fighter 4: Evolution</Title>
<Description>Tr avel around the world to fight A.I. that is
programmed after the best Virtua Fighter arcade players</Description>
<Genre>Fighting </Genre>
<Picture>images \ps2\NoGif.gif</Picture>
<Rating>5</Rating>
</PS2>
</Entertainment>

[Dimitre Novatchev]

"Mark Constant" <co*******@mi x-net.net> wrote in message
news:ce******** *************** **@posting.goog le.com...

I can see now that the problem lies within the NotAll template. It
will take a little while to explain in words but here I go. I have an
xml file that supports three types of hardware PS2, XBOX, and Movie.
Now if the user is on the PS2 and selects that they want the genre
horror it will send two parameters. One being the Hardware (i.e. PS2)
and then the Genre which is horror. From there the template calls
NotAll and says for every instance of PS2 in the XML that has a Genre
of horror then display it. Here is my Xslt again.
<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">

You didn't understand the pattern -- one cannot use a string as a node-set.

Also the above doesnt make sense -- if $Hardware were a node-set it would
mean:

Select all children of lc:Entertainmen t that are in the namespace to which
the prefix "lc" is bound and that have local-name the same as the *value* of
the (first node of) the node=set of all nodes belonging to $Hardware, which
have an lc:Genre child with value $Genre

As in the previous template, once again, you have to write:

<xsl:for-each
select="lc:Ente rtainment/lc:*[local-name() = $Hardware][lc:Genre =
$Genre]]">

I think that using the XPath Visualizer may be very helpful in such cases as
it has been proven in practice to be an effective XPath learning tool.
=====
Cheers,

Dimitre Novatchev.
http://fxsl.sourceforge.net/ -- the home of FXSL

[Mark Constant]

"Dimitre Novatchev" <dn********@yah oo.com> wrote in message news:<br******* ****@ID-152440.news.uni-berlin.de>...

"Mark Constant" <co*******@mi x-net.net> wrote in message
news:ce******** *************** **@posting.goog le.com...

I can see now that the problem lies within the NotAll template. It
will take a little while to explain in words but here I go. I have an
xml file that supports three types of hardware PS2, XBOX, and Movie.
Now if the user is on the PS2 and selects that they want the genre
horror it will send two parameters. One being the Hardware (i.e. PS2)
and then the Genre which is horror. From there the template calls
NotAll and says for every instance of PS2 in the XML that has a Genre
of horror then display it. Here is my Xslt again.
<xsl:template name="NotAll">
<table border="0">
<xsl:for-each select="lc:Ente rtainment/lc:*[local-name() =
$Hardware[lc:Genre = $Genre]]">

You didn't understand the pattern -- one cannot use a string as a node-set.

Also the above doesnt make sense -- if $Hardware were a node-set it would
mean:

Select all children of lc:Entertainmen t that are in the namespace to which
the prefix "lc" is bound and that have local-name the same as the *value* of
the (first node of) the node=set of all nodes belonging to $Hardware, which
have an lc:Genre child with value $Genre

As in the previous template, once again, you have to write:

<xsl:for-each
select="lc:Ente rtainment/lc:*[local-name() = $Hardware][lc:Genre =
$Genre]]">

I think that using the XPath Visualizer may be very helpful in such cases as
it has been proven in practice to be an effective XPath learning tool.
=====
Cheers,

Dimitre Novatchev.
http://fxsl.sourceforge.net/ -- the home of FXSL

Dang your good at this stuff. Thanks once again. That worked perfectly.