Zoals het al in de titel staat is de onderstaande regel goed of niet goed om de id uit te database te halen en te kunnen gebruiken voor je query?


echo "<td>" . "<a href='EditFactuur.php?edit=$row[0]&edit=$row[1]?>'><img src='../img/icons/icon-edit.png' height='16' width='16'></a>" . "</td>";
- Waarom twee dezelfde editparameters in de URL?
- Wat doet doe ?> na $row[1]?
- Verder horen variabelen buiten quotes

echo "
<td><a href='EditFactuur.php?edit={$row[0]}&edit2={$row[1]}'>
<img src='../img/icons/icon-edit.png' height='16' width='16'></a></td>
";
Wat Aar zegt: dit kan haast niet kloppen. Een van de twee "edit" waarden gaat op deze manier verloren.

De naamgeving van de parameters is ook niet erg omschrijvend. Het PHP-bestand heet al "edit...", waarom zou je je parameters dan ook zo noemen?

Daarnaast, $row[0] en $row[1] is ook redelijk nietszeggend.

En tot slot snap ik de charme van het echo'en van statische lappen HTML niet.

In deze opzichten is het codefragment van Rick hierboven (die alle toelichting ontbeert) ook niet echt een verbetering.

<td><a href='EditFactuur.php?edit=<?php echo $row[0];?>}&edit2=<?php echo $row[1]; ?>'>
<img src='../img/icons/icon-edit.png' height='16' width='16'></a></td>
Hartelijk dank heren

Aar c ik had een typ foutje gemaakt het het inderdaad moeten zijn wat jij ook hebt.

en de echo voor <td> heb ik neergezet omdat het in de while loop staat
en de echo voor <td> heb ik neergezet omdat het in de while loop staat

Dit werkt prima:
<?php
// een of andere loop
for ($i = 0; $i < 10; $i++) {
    ?><tr>
        <td>Dit is rij <?php echo $i ?></td>
    </tr><?php
}
?>

Je kunt PHP-blokken haast te allen tijde sluiten en weer openen.
Ik had nog een vraagje bij de volgende query

SELECT upload.UploadId, upload.ID, klanten.ID , upload.status, upload.name, klanten.Voornaam, klanten.Achternaam, klanten.Email,klanten.Bedrijfsnaam,klanten.Telefoon FROM klanten INNER JOIN upload ON klanten.ID=upload.ID WHERE upload.ID='1' AND upload.UploadId = '12'

krijg ik de volgende fout "De huidige selectie bevat geen unieke kolom. Functies zoals rasterbewerkingen, checkboxen, Bewerken, Kopiëren en Verwijderen, zijn niet beschikbaar."

terwijl ik bij de beide tabellen een primary key gedefineerd heb.
Je moet een Unieke key dan aanmaken. Maar dit gaat over PHPMyAdmin, en die is toch al veeleisend.
Aar C op 20/03/2015 18:27:30

Je moet een Unieke key dan aanmaken. Maar dit gaat over PHPMyAdmin, en die is toch al veeleisend.



Is het dan zo dat ik bijv de ID verander van Primary key naar Unieke key


[size=xsmall]Toevoeging op 20/03/2015 18:37:40:[/size]

Murti Aydogdu op 20/03/2015 18:36:02

[quote="Aar C op 20/03/2015 18:27:30"]
Je moet een Unieke key dan aanmaken. Maar dit gaat over PHPMyAdmin, en die is toch al veeleisend.



Is het dan zo dat ik bijv de ID verander van Primary key naar Unieke key

[/quote]

IK heb het al dank voor de moeite



[size=xsmall]Toevoeging op 20/03/2015 18:39:26:[/size]

Murti Aydogdu op 20/03/2015 18:36:02

[quote="Aar C op 20/03/2015 18:27:30"]
Je moet een Unieke key dan aanmaken. Maar dit gaat over PHPMyAdmin, en die is toch al veeleisend.



Is het dan zo dat ik bijv de ID verander van Primary key naar Unieke key


Veranderd maar krijg de zelde melding :(

[size=xsmall]Toevoeging op 20/03/2015 18:37:40:[/size]

Murti Aydogdu op 20/03/2015 18:36:02

[quote="Aar C op 20/03/2015 18:27:30"]
Je moet een Unieke key dan aanmaken. Maar dit gaat over PHPMyAdmin, en die is toch al veeleisend.



Is het dan zo dat ik bijv de ID verander van Primary key naar Unieke key

[/quote]

IK heb het al dank voor de moeite


[/quote]

Reageren