Artikel ins Angebot übernehmen

[Jnnbo]

Hallo 1unitedpower,

1. Du hast die <label>-Elemente nun leider rausgeworfen, die solltest du wieder aufnehmen. Sie verliehen deinem Formular wesentlich sinnbringende Semantik.

Ok, habe ich nun geändert, ist es so richtig?

<section>
	<h2>Artikel</h2>
    <article>
    <h3>Alternativ zu Brother LC-1000 Tinten Spar-Set (4xBK,je 2xC,M,Y) 10 Stück</h3>
       <form method="post" action="moin123.php">
          <input type="radio" name="preis" id="000001-preis1" value="14.64" required>
          <label for="000001-preis1">14,64 €</label>
          <input type="radio" name="preis" id="000001-preis2" value="10.99">
          <label for="000001-preis2">10,99 €</label>
          <input type="radio" name="preis" id="000001-preis3" value="11.64">
          <label for="000001-preis3">11,64 €</label>
          <input type="hidden" value="000001" name="artikel_id">
          <input type="submit" name="wk" value="In den Warenkorb">
       </form>
	</article>
    
    <article>
    <h3>Original Brother TN-1050 Toner Black</h3>
        <form method="post" action="moin123.php">
          <input type="radio" name="preis" id="000002-preis1" value="31.61" required>
          <label for="000002-preis1">31,61 €</label>
          <input type="radio" name="preis" id="000002-preis2" value="32.40">
          <label for="000002-preis2">32,40 €</label>
          <input type="radio" name="preis" id="000002-preis3" value="33.20">
          <label for="000002-preis3">33,20 €</label>
          <input type="hidden" value="000002" name="artikel_id">
          <input type="submit" name="wk" value="In den Warenkorb">
       </form>
   </article>
</section>

2. Du hast mal erwähnt, dass der Preis ein Pflichtfeld sein soll. Du solltest also noch das required-Attribut vergeben. Es reicht aus, wenn jeweils ein Radiobutton innerhalb einer Radiobutton-Gruppe das required-Attribut besitzt, um die ganze Gruppe verpflichtend zu machen.

Habe ich auch eingefügt.

3. Die Fehler, die von deiner PHP-Überprüfung festgellt werden, sollten direkt an den betroffenen Feldern ausgegeben werden, und nicht irgendwo ihres Kontextes entrissen am Anfang oder Ende des Formulars ausgegeben werden.

Ja, das würde ich auch viel lieber machen, das würde dann bedeuten, ich muss diesen Block nach jedem Formular einfügen?

if(isset($_POST['wk'])){
	if(!empty($_POST['preis'])){
			echo "<pre>";
			print_r($_POST);
			echo "</pre>";
		 } 
		else {
		echo "<div class=\"alert alert-danger\">Bitte ein Preis wählen</div>";
	}
}