Form e Cicli

[gorn]

Salve a tutti,
Ho un problema che proprio non riesco a spiegarmi.
La situazione è la seguente: in una pagina .php vado a costruire dinamicamente una lista di utenti prelevati con una select. Il ciclo che si occupa di farlo è un while. Potendo dare la possibilità di cancellare/inserire degli utenti di questa lista i dati vengono racchiusi in un form.
Tutto funziona correttamente esclusa la prima riga. Mi spiego meglio: sia in inserimento che in eliminazione per il primo dato (visualizzato) il form non compare mentre per gli altri si (non lo avrei mai notato senza l'ispeziona elemento di chrome ). Ho provato a creare il form fuori da ciclo while ma così facendo il form non compare per nessun dato.
Spero di essermi spiegato anche se la problematica è davvero particolare e rimango a disposizione delle anime pie che mi vorranno aiutare per ulteriori spiegazioni.
Ciao a tutti.

[satifal]

Originariamente inviato da gorn
Spero di essermi spiegato...

Per spiegare ti sei spiegato, ma senza codice si dovrebbe spolverare la sfera di cristallo

[gorn]

Allego la parte di creazione della lista (quindi il ciclo while).

$Ag = mysql_query("SELECT IDUtenteMaster, IDUtenteDetail, cognome, nome FROM agenti, utenti WHERE IDUtenteDetail = id AND IDUtenteMaster = $id");
$tabella = "agenti";
//creo dinamicamente la tabella con i dati trovati
while ($row = mysql_fetch_assoc($Ag)) {
   $ret .= "<tr class=\"$td_class\">";
   $ret .= "<td width=\"16px\" class=buttons>";
   $ret .= "<form action=\"?q=sistema/utenti/eliminaAgenti\" method=post>";
   $ret .= html_par("IDUtenteMaster", $id);
   $ret .= html_par("tabella", $tabella);
   $ret .= html_par("IDUtenteDetail", $row["IDUtenteDetail"]);
   $ret .= "<input alt=\"elimina agente\" type=\"image\" onclick=\"return askConfirm();\"    src=\"images/icons/delete.gif\">";
   $ret .= "</form>";
   $ret .= "</td>";
   $ret .= "<td nowrap width=\"86px\">"."&nbsp".$row["cognome"]."&nbsp".$row["nome"]."</td>";
   $ret .= "</tr>";
   $td_class = ($td_class == "light" ? "dark" : "light");
}
   $ret .= "</table>"; 

Questa porzione di codice viene usata per cancellare degli elementi dalla lista. Funziona correttamente per tutti gli elementi tranne il primo (della lista)
Il form richiama una funzione presente nella stessa pagina attraverso un case ( case EliminaAgenti -> funzione di eliminazione)

function eliminaAgente_utente()
{
  global $utente;
  if (!check_login() || $utente->ruolo < R_ADMIN)
  {
    return ACCESS_DENIED;
  }

  $master = $_POST['IDUtenteMaster'];
  $detail = $_POST['IDUtenteDetail'];
  if ($id == "-1")
  {
   append_msg("Eliminzione record non riuscita.");
  }
  else
  {
  $strSql = "DELETE FROM agenti WHERE IDUtenteMaster= $master AND     IDUtenteDetail= $detail";
  if (ExecuteQuery($strSql))
  {
   append_msg("Eliminzione record riuscita.");
  }
  else
 {
  $msg = "Eliminzione utente non riuscita.";
  append_msg($msg);
 }
 }
 return OK;
} 

Spero che cosi possa essere tutto più chiaro.

[satifal]

Prova a visualizzare da browser l'html della pagina contenente l'elenco e controlla che i valori presenti nel primo elemento della lista siano corretti.

[gorn]

I dati della prima riga sono corretti e facendo un po di debug anche i valori che andrebbero poi passati alla funzione sono corretti.
Il problema è che manca la riga

<form action=\"?q=sistema/utenti/eliminaAgenti\" method=post> 

(scoperto sfruttando l'ispeziona elemento di chrome... su tutti gli altri elementi compare ma sul primo non c'è verso di farglielo inserire)
e quindi non andrò mai a chiamare la funzione di eliminazione.
Magari ho commesso uno stupidissimo errore ma sinceramente non riesco a trovarlo.

Comunque vi ringrazio per l'aiuto che gia mi state dando.

[satifal]

Alla prima iterazione $td_class non è valorizzato, magari il problema nasce da lì. Prova a valorizzare $td_class prima del while:

...
$td_class = "light";

while ($row = mysql_fetch_assoc($Ag)) { 
... 

[gorn]

Scusa... non mi ero accorto che non avevo inserito queste due righe nel codice di prima

$ret .= "<table><td></td><td nowrap class=\"label\">Agenti associati</td><tr>";
$td_class = "light"; 

Sono a corto di idee...
La cosa strana è che l'html generato presenta tutti i campi all'interno del form correttamente valorizzati ma mancano le righe del form stesso (<form ecc...> e </form>)

[satifal]

Originariamente inviato da gorn
Scusa... non mi ero accorto che non avevo inserito queste due righe nel codice di prima

$ret .= "<table><td></td><td nowrap class=\"label\">Agenti associati</td><tr>";
$td_class = "light"; 

Sono a corto di idee...
La cosa strana è che l'html generato presenta tutti i campi all'interno del form correttamente valorizzati ma mancano le righe del form stesso (<form ecc...> e </form>)

Apri 2 tag <tr>:

...
$ret .= "<table><td></td><td nowrap class=\"label\">Agenti associati</td><tr>"; 
... 

il <tr> alla fine della riga precedente e poi nel while ne apri un altro anche se non credo sia questo il problema.
Se come dici c'è tutto il codice html tranne il tag form aperto e chiuso è strano.
Prova a postare TUTTO lo script, anche la parte prima del while e magari anche l'html generato (da browser "visualizza sorgente pagina")

[gorn]

Ho aggiunto il </tr> che mi ero completamente scordato ma non è cambiato nulla (come immaginavo)
Visualizzando il sorgente pagina ho notato che li la form compare anche per il primo elemento mentre se uso l'ispeziona elemento non la rileva e comunque poi in esecuzione il primo elemento non risponde ai comandi mentre gli altri si.
Di seguito posto il pezzo di sorgente pagina incriminato

<div id="InsAgenti"><table>
<td></td><td nowrap class="label">Agenti associati</td><tr></tr>
<tr class="light"><td width="16px" class=buttons>
<form action="?q=sistema/utenti/eliminaAgenti" method=post>
<input type=hidden name="IDUtenteMaster" value="42">
<input type=hidden name="tabella" value="nc_utenti_agenti">
<input type=hidden name="IDUtenteDetail" value="9">
<input alt="elimina agente" type="image" onclick="return askConfirm();" src="images/icons/delete.gif">
</form></td>
<td nowrap width="86px">blablalba</td>
</tr>
<tr class="dark"><td width="16px" class=buttons>
<form action="?q=sistema/utenti/eliminaAgenti" method=post>
<input type=hidden name="IDUtenteMaster" value="42">
<input type=hidden name="tabella" value="nc_utenti_agenti">
<input type=hidden name="IDUtenteDetail" value="15">
<input alt="elimina agente" type="image" onclick="return askConfirm();" src="images/icons/delete.gif">
</form></td>
<td nowrap width="86px">blablalbu</td>
</tr>
<tr class="light"><td width="16px" class=buttons>
<form action="?q=sistema/utenti/eliminaAgenti" method=post>
<input type=hidden name="IDUtenteMaster" value="42">
<input type=hidden name="tabella" value="nc_utenti_agenti">
<input type=hidden name="IDUtenteDetail" value="28">
<input alt="elimina agente" type="image" onclick="return askConfirm();" src="images/icons/delete.gif">
</form></td><td nowrap width="86px">blalbalble</td>
</tr>
</table></div>

Per quanto riguarda il codice l'unica cosa che manca è la seguente

if ($utente->ruolo >= R_CS)
{
 $ret .= "<div id=\"InsAgenti\">";
}else{
 $ret .= "<div id=\"InsAgentiNo\">";
} 

che controlla se visualizzare o meno la div contenente la lista a seconda dei diritti dell'utente.
Spero in un vostro aiuto dato che io non riesco a vedere dove si nasconde la stupidata che ho fatto.

[satifal]

Prova a stampare immediatamente prima di eseguirla, anche con un semplice echo, la query di cancellazione e vedi se è corretta, magari eseguendola a mano.
Inoltre aggiungi un mysql_error() nel punto in cui esegui la query così vedi se c'è qualche errore. Qualcosa del tipo:

...
echo $query;
$result = mysql_query($query) or die ("Error: ".mysql_error());
...