No se aconseja mezclar en un mismo flujo código PHP / Javascript. Porque PHP trabaja del lado del servidor y Javascript del lado del cliente.
Esto lo puedes resolver de una forma más limpia, poniendo una clase .css
en el elemento, según el valor. Eso lo haremos en esta línea:
$color=$usuario['reposicion'] == 0 ? 'red' : 'black';
Luego, crearemos un <span class="$color">
que tomará red
o black
según el caso y la regla .css
se encargará de aplicar el color. En el .css
aprovechamos también para poner una regla llamada row-center
para centrar las filas, y quitamos el style
que aplicas directamente sobre los elementos, lo cual es otra mala práctica.
En caso de que los valores sean cambiados por el usuario (en el cliente), y se requiera una actualización dinámica, también se puede hacer. En este caso ya desde Javascript, escuchando ese cambio y asignado al elemento una clase diferente. No parece ser el caso, pero de serlo, usar una clase .css
te servirá también para ese caso de uso.
Veamos un ejemplo.
.css
En primer lugar, tendrías que crear un archivo de estilo con las siguientes reglas:
.red {
color: #ee2500
}
.black {
color: #333
}
.row-center {
text-align: center
}
PHP
En PHP introduciremos algunas mejoras que den claridad y simplicidad al código usando sprintf
, es algo parecido a implementar plantillas. De paso evitaremos ids repetidos en los elementos, lo cual es otra mala práctica que puede llevar a errores si se usará en un futuro getElementById
del lado del cliente.
<table>
<thead>
<th>id</th>
<th>No. Devolucion</th>
<th>Reposicion</th>
</thead>
<tbody>
<?php
#AQUÍ EL CÓDIGO QUE DEFINE $rol_user Y DEMÁS ...
if($rol_user==='admin' || $rol_user==='administrativo'){
$td='<td>%s</td>';
$anchor='<a id="enlace%s" href="repo.php?repos=%s"><span class="%s">%s</span></a>';
$rows='';
foreach($usuarios as $usuario) {
$rows.='<tr class="row-center">';
$rows.=sprintf($td,$usuario['id']);
$rows.=sprintf($td,$usuario['Devolucion']);
$color=$usuario['reposicion'] == 0 ? 'red' : 'black';
$a=sprintf($anchor,$usuario['id'],$usuario['reposicion'],$color,$usuario['reposicion']);
$rows.=sprintf($td,$a);
$rows.='</tr>';
}
echo $rows;
}
#AQUÍ LO QUE SIGUE DE TU CÓDIGO
?>
</tbody>
</table>
El código creará las filas como se puede ver en el snippet siguiente. Si observas el valor de la fila 2 está en rojo, porque es 0
.
.red {
color: #ee2500
}
.black {
color: #333
}
.row-center {
text-align: center
}
<table>
<thead>
<th>id</th>
<th>No. Devolucion</th>
<th>Reposicion</th>
</thead>
<tbody>
<tr class="row-center">
<td>1</td>
<td>Devolución 1</td>
<td><a id="enlace1" href="repo.php?repos=1"><span class="black">1</span></a></td>
</tr>
<tr class="row-center">
<td>2</td>
<td>Devolución 2</td>
<td><a id="enlace2" href="repo.php?repos=0"><span class="red">0</span></a></td>
</tr>
<tr class="row-center">
<td>3</td>
<td>Devolución 3</td>
<td><a id="enlace3" href="repo.php?repos=3"><span class="black">3</span></a></td>
</tr>
</tbody>
</table>