Funktion wird nicht ausgeführt

mentle · 21. Februar 2016

Hallo,

im folgenden Code steckt eine Formulareingabe. Das Absenden soll nur funktionieren, wenn alle Formulareinträge gemacht wurden.

PHP

$change_form_html = '		<form action="datenaendern.php" method="post" id="datensatzaendern">		<p>Datensatz Nr.: ' . $daten->bild[(int)$zahl-1]->bildnr . '<p>		<label>Bildtitel<input id="titel" type="text" name="titel" value="' . $daten->bild[(int)$zahl-1]->bildtitel . '"></label><br>		<label>Beschreibung<textarea rows="5" id="remark" type="text"  name="remark">' . $daten->bild[(int)$zahl-1]->remark . '</textarea></label><br>		<label>Länge<input id="laenge" type="number" name="laenge" value="' . $daten->bild[(int)$zahl-1]->laenge . '"></label><br>		<label>Beite<input id="breite" type="number" name="breite" value="' . $daten->bild[(int)$zahl-1]->breite . '"></label><br>		<label>Rahmen<input id="rahmen" type="text" name="rahmen" value="' . $daten->bild[(int)$zahl-1]->rahmen . '"></label><br>		<label>Verfügbarkeit<input id="preis" type="text" name="preis" value="' . $daten->bild[(int)$zahl-1]->avail . '"></label><br>		<input type="hidden" name="filenumber" value="' . $daten->bild[(int)$zahl-1]->bildnr .'">		<input type="hidden" name="datname" value="' . $datname . '">		<br>		<button onclick="return checkFormAenderung()" class="internbutton" >Datensatz ändern</button>		<br>		</form>		';echo $change_form_html;

Die Funktion dazu:

JavaScript

<script type="text/JavaScript">
	function checkFormAenderung()
	{
    var strFehler = "";
  		if(document.forms[1].titel.value == "")
  			{
    		strFehler += "Feld Bildtitel ist leer\n";
    		}
    	if(document.forms[1].remark.value == "")
    		{
    		strFehler += "Feld Bemerkungen ist leer\n";
    		}
    	if(document.forms[1].laenge.value == "")
  			{
    		strFehler += "Feld Länge ist leer\n";
    		}
  		if(document.forms[1].breite.value == "")
  			{
    		strFehler += "Feld Breite ist leer\n";
    		}
    	if(document.forms[1].rahmen.value == "")
  			{
    		strFehler += "Feld Rahmen ist leer\n";
    		}
  		if(document.forms[1].preis.value == "")
  			{
    		strFehler += "Feld Verfügbarkeit ist leer\n";
    		}
  		if(strFehler.length>0)
  			{
    		alert("Festgestellte Probleme: \n\n"+strFehler);
			return(false);
			}
	}
	</script>

Alles anzeigen

Die Funktion wird nur leider nicht ausgeführt, d.h. es ist möglich, die Daten auch unvollständig abzuspeichern.
Erkennt jemand, warum?

Basi · 21. Februar 2016

Sprichst du das richtige Formular an? Denk daran, dass ab 0 gezählt wird. Das erste Formular der Seite wird also mittels document.forms[0] angesprochen.

mentle · 21. Februar 2016

ja, ist das zweite Formular auf der Seite...

Basi · 21. Februar 2016

Mit exakt dem Code, den du oben gepostet hast (den PHP Teil mal entfernt), funktioniert alles wie von dir gewünscht.

Da muss also noch mehr Code sein.

cottton · 21. Februar 2016

Vom eigtl Thema abgesehen: PHP output maskieren. Egal was/woher. Ob $_POST, $_GET oder aus der eigenen Datenbank - output -> maskieren.
zB mit (am weitesten verbreitet) htmlspecialchars()
bsp:

PHP

function escapeHtml($value){
	return htmlspecialchars($value);
}
$change_form_html = '
        <form action="datenaendern.php" method="post" id="datensatzaendern">
        <p>Datensatz Nr.: ' . escapeHtml($daten->bild[(int)$zahl-1]->bildnr) . '<p>
        <label>Bildtitel<input id="titel" type="text" name="titel" value="' . escapeHtml($daten->bild[(int)$zahl-1]->bildtitel) . '"></label><br>
        <label>Beschreibung<textarea rows="5" id="remark" type="text"  name="remark">' . escapeHtml($daten->bild[(int)$zahl-1]->remark) . '</textarea></label><br>
        <label>Länge<input id="laenge" type="number" name="laenge" value="' . escapeHtml($daten->bild[(int)$zahl-1]->laenge) . '"></label><br>
        <label>Beite<input id="breite" type="number" name="breite" value="' . escapeHtml($daten->bild[(int)$zahl-1]->breite) . '"></label><br>
        <label>Rahmen<input id="rahmen" type="text" name="rahmen" value="' . escapeHtml($daten->bild[(int)$zahl-1]->rahmen) . '"></label><br>
        <label>Verfügbarkeit<input id="preis" type="text" name="preis" value="' . escapeHtml($daten->bild[(int)$zahl-1]->avail) . '"></label><br>
        <input type="hidden" name="filenumber" value="' . escapeHtml($daten->bild[(int)$zahl-1]->bildnr) .'">
        <input type="hidden" name="datname" value="' . escapeHtml($datname) . '">
        <br>
        <button onclick="return checkFormAenderung()" class="internbutton" >Datensatz ändern</button>
        <br>
        </form>
        ';
echo $change_form_html;

Alles anzeigen

Auch bei JS (was jetzt hier wohl nicht zutrifft)
Bsp:

JavaScript

function escapeHtml(value){
	return document.createTextNode(value);
}
function checkFormAenderung()
{
// ...
		if(document.forms[1].titel.value != "") { // != macht jetzt keinen sinn, soll nur dem bsp dienen
		strFehler += "du hast " + escapeHtml(document.forms[1].titel.value) + " eingegeben";
	}
// ...
}

Alles anzeigen

und weiter im Programm

Funktion wird nicht ausgeführt

Jetzt mitmachen!

Teilen