change td class

[[deleted]]

I'm using the following function to affect the class of a row.

function selectClass(){
$result = sql_query("SELECT menuid,menuname,url,parent,menulabel FROM tblmenu WHERE parent = 0 AND module_id = '1' AND view= '1' ORDER BY menuname ASC");
$counter = 0;
while ($row = sql_fetch_array($result)){
$menulabel = $row['menulabel'];
$mod = '.$mod.';
if ($menulabel == "$mod"){
echo '<td width=" " valign="top" class="td_select">
<a id="a_root'.$counter.'" border="0" style="cursor:hand; text-align:top;">
<a class="td_root" onMouseover="this.style.color=\'black\'" onMouseout="this.style.color=\'white\'" onMouseout="this.style.color=\'white\'" href="'.$row[2].'">'.$row[1].'</a>
</td>';
}else{
echo '<td width=" " valign="top" class="root_td">
<a id="a_root'.$counter.'" border="0" style="cursor:hand; text-align:top; ">
<a class="td_root" onMouseover="this.style.color=\'black\'" onMouseout="this.style.color=\'white\'" onMouseout="this.style.color=\'white\'" href="'.$row[2].'">'.$row[1].'</a>
</td>';
}
}
return;
}

The idea is that the class of the row is changed if the name of the page (the mod) displayed is the same as the menulabel. Everything that echos out is the td_root class.

Any suggestions?

[zhabala]

What do you think, that this statement is doing ?

$mod = '.$mod.'; 

You should try to display values stored in $mod and in $menulabel variables, to see, what is compared.

Zdenek

[[deleted]]

I removed $mod = '.$mod.';

The problem seems to be that the value for $mod in the URL is not getting captured. All I'm trying to say is that if $menulabel (a value in tblmenu) is the same as the value $mod in the URL then echo specific code to have the row background color be different than non-selected.

If there is a more elegant way of doing this I would love to see it.

[zhabala]

$class = ( $row['menulabel'] == $GLOBAL["mod"] ? 'td_root' : 'td_select' );
echo '<td width=" " valign="top" class="'.$class.'"> 
<a id="a_root_'.$counter.'" border="0" style="cursor:hand; text-align:top;"> <a class="'.$class.'" onMouseover="this.style.color='black'" onMouseout="this.style.color='white'" onMouseout="this.style.color='white'" href="'.$row[2].'">'.$row[1].'</a> </td>';

You are not passing the $mod into the function, then you have to access it in the global array or you have to define it as a global variable by keyword global.

[[deleted]]

Thanks for the suggestion. Your code is much cleaner. However, Globals are off on my ISP so your method will not work.

[zhabala]

Then you are able to use

$POST["mod"] or $GET["mod"] or $_REQUEST["mod"] and
it has to work.

Zdenek

[[deleted]]

That worked! Thanks.