13
Editierung per Textarea
Textarea bekommt Daten über DB, diese sollen editiert und wieder abgespeichert werden
Hallo Freunde der Kunst,
ich habe mal wieder ein Problem, weiß aber diesmal wenigstens schonmal, woran es liegt
Ist auch wirklich dumm, ich komm nur einfach nicht drauf...naja, vielleicht landets ja in LSW Beiträgen des Jahres.
Wie oben kurz beschrieben, habe ich Texte in einer (MySql)Datenbank, welche zum Bearbeiten in eine Textarea geladen werden. So weit, so gut. Das nochmalige Abspeichern bereit mir aber irgendwie Probleme...Lösung sollte aber simpel sein, ich seh sie nur nicht.
Tabellenstruktur: id | inhalt | datum | datei
In der admin.php ist das erste Formular + Funktionsaufruf:
<code="php">
<?php
text_edit();
?>
nun die Funktion:
Was funktioniert nicht und warum?
Das Abspeichern, wie oben schon gesagt, aus folgendem Grund: Sobald ich auf "Ändern" klicke, ist natürlich die Variable $auswahl weg und damit die Datei, mit der sich die WHERE Klausel befassen kann.
Mein Problem: Wie bekomme ich das Formular dazu, die Variable $auswahl beizubehalten oder zumindest zu sagen "du speicherst nun den Text, der in der Textarea steht"?
Ich danke schonmal für eure Hilfe
Gruß
Mitchell
Hallo Freunde der Kunst,
ich habe mal wieder ein Problem, weiß aber diesmal wenigstens schonmal, woran es liegt
Ist auch wirklich dumm, ich komm nur einfach nicht drauf...naja, vielleicht landets ja in LSW Beiträgen des Jahres.Wie oben kurz beschrieben, habe ich Texte in einer (MySql)Datenbank, welche zum Bearbeiten in eine Textarea geladen werden. So weit, so gut. Das nochmalige Abspeichern bereit mir aber irgendwie Probleme...Lösung sollte aber simpel sein, ich seh sie nur nicht.
Tabellenstruktur: id | inhalt | datum | datei
In der admin.php ist das erste Formular + Funktionsaufruf:
<p>Wäle den zu bearbeitenden Text</p><br>
<form action="" method="post">
<select name="texte" size="1">
<option value="text1.php">Einleitung</option>
<option value="text2.php">Unterschriftenaktion</option>
<option value="text3.php">Datenschutz</option>
</select>
<input type="submit" value="Wählen">
</form><?php
text_edit();
?>
nun die Funktion:
$ausgabe = $_POST['ausgabe'];
$auswahl = $_POST['texte'];
$editieren = $_POST['edit'];
$text = $_POST['aenderung'];
$out_sql = mysql_query("SELECT inhalt, datum, datei FROM kf_texte WHERE file = '$auswahl' ORDER BY change_date DESC");
$text_out= mysql_fetch_array($out_sql);
echo "<form action='' name='aenderung' method='post'><input name='edit' type='submit' value='Ändern'><p><textarea name='ausgabe' cols='105' rows='30'>".html_entity_decode($text_out['inhalt'])."</textarea></p></form>";
// bis hier hin läufts :-)
if ($editieren == 'Ändern')
mysql_query("UPDATE tbl SET inhalt='$ausgabe', datum=NOW() WHERE datei= '$auswahl'"); Was funktioniert nicht und warum?
Das Abspeichern, wie oben schon gesagt, aus folgendem Grund: Sobald ich auf "Ändern" klicke, ist natürlich die Variable $auswahl weg und damit die Datei, mit der sich die WHERE Klausel befassen kann.
Mein Problem: Wie bekomme ich das Formular dazu, die Variable $auswahl beizubehalten oder zumindest zu sagen "du speicherst nun den Text, der in der Textarea steht"?
Ich danke schonmal für eure Hilfe
Gruß
Mitchell
Auf Facebook teilen
Auf Twitter teilen
Auf Reddit teilen
Auf Linkedin teilenBitte markiere auch die Kommentare, die zur Lösung des Beitrags beigetragen haben
Content-ID: 184870
Url: https://administrator.de/contentid/184870
Ausgedruckt am: 19.12.2024 um 12:12 Uhr
13 Kommentare
Neuester Kommentar
man könnte es mit einem Hidden Formular Feld durch reichen
btw. http://de.wikipedia.org/wiki/SQL-Injection
echo '<form .... ><input type="hidden" name="texte" value="'.$auswahl.'"> .... '; btw. http://de.wikipedia.org/wiki/SQL-Injection
Moin,
lg,
Slainte
.... type='submit' value='Ändern'> ....
if ($editieren == 'Ändern')
bau hier doch mal ein "echo $editieren" einif ($editieren == 'Ändern')
$ausgabe = $_POST['ausgabe'];
mysql_query("UPDATE tbl SET inhalt='$ausgabe' ....
Und hier wieder mein Liebliengs PHP-Thema SQL injection mysql_query("UPDATE tbl SET inhalt='$ausgabe' ....
lg,
Slainte
@nxclass: Hatte ich anfangs auch, bringt aber genau das selbe. Auch das hidden Formular ist natürlich beim Absenden des Formulares weg.
@slan...: Ich habe schon alle Variablen durchexerziert, aber wenns dich weiterbringt ^^. Die Ausgabe von $editeren bringt mir ein "'Ändern" (nach Abschicken des Editbuttons natürlich).
Zu euren Injektions...auf was bezieht ihr das, die Variablen? Keine Angst, vor dem Update wird $ausgabe natürlich noch durch htmlentities und mysql_real_escape geschickt
Mfg
Mitchell
@slan...: Ich habe schon alle Variablen durchexerziert, aber wenns dich weiterbringt ^^. Die Ausgabe von $editeren bringt mir ein "'Ändern" (nach Abschicken des Editbuttons natürlich).
Zu euren Injektions...auf was bezieht ihr das, die Variablen? Keine Angst, vor dem Update wird $ausgabe natürlich noch durch htmlentities und mysql_real_escape geschickt
Mfg
Mitchell
Auch das hidden Formular ist natürlich beim Absenden des Formulares weg
.. Zauberei.... schade das die OO Programmierung schon wieder aus der Mode ist.
Evtl. solltest Du deinen Code mal etwas umbauen, so dass erst die Daten verarbeitet werden und erst am Schluss die Ausgabe erfolgt.
.. Zauberei.
ich mag dich auch :-P
es soll am Schluss aber gar keine Ausgabe erfolgen, also was soll ich hier umbauen?
Mfg
Mitchell
es soll am Schluss aber gar keine Ausgabe erfolgen, also was soll ich hier umbauen?
// 1 - prüfen ob Formular Daten gesendet wurden und diese in der DB speichern
// 2 - prüfen ob Formulardaten aus der DB geholt werden müssen und Daten holen
// 3 - Formular mit den Daten darstellen (oder auch nicht)
Es geht doch darum, dass ich eine Variable erhalten will, da gibts vorher nichts zu prüfen. Auch nicht, ob die Formulardaten aus der DB geholt werden müssen, weil das ja definitiv der Fall ist.
Der User hat sein Dropdown, dort wählt er den Text zum Editieren. Sobald er diesen ausgewält hat, wird dieser angezeigt, also die Daten aus der DB genommen. Nun soll der editierte Text neu abgespeichert werden...hab ich mich falsch ausgedrückt am Anfang?
Außerdem hattest du doch schon vorgeschlagen, es mit einem hidden-field durchzureichen...was halt nicht klappt. Die Antwort ".. Zauberei." lässt mich irgendwie denken, dass du das wusstest :-P
Mfg
Mitchell
Der User hat sein Dropdown, dort wählt er den Text zum Editieren. Sobald er diesen ausgewält hat, wird dieser angezeigt, also die Daten aus der DB genommen. Nun soll der editierte Text neu abgespeichert werden...hab ich mich falsch ausgedrückt am Anfang?
Außerdem hattest du doch schon vorgeschlagen, es mit einem hidden-field durchzureichen...was halt nicht klappt. Die Antwort ".. Zauberei." lässt mich irgendwie denken, dass du das wusstest :-P
Mfg
Mitchell
... ok - zum Verständnis: Dein Script wird 3-mal durchlaufen
1
2
3
... ich glaube das Punkt 3 fehlt - und um im Punkt 3 die Daten speichern zu können müsst Du aus 2. die ID ($auswahl) in einem hidden Feld durch schleifen.
1
- Anzeige des "Wäle den zu bearbeitenden Text" - Formulars
2
- Auswahl empfangen
- Daten aus der DB holen
- und Anzeige des "Änderung" Formulars mit den Daten
3
- Empfangen der Daten
- Speichern der Daten
- Formular wieder anzeigen oder zurück zum "Wäle den zu bearbeitenden Text" -Formulars oder was auch immer
... ich glaube das Punkt 3 fehlt - und um im Punkt 3 die Daten speichern zu können müsst Du aus 2. die ID ($auswahl) in einem hidden Feld durch schleifen.
Punkt 3 fehlt, korrekt. Leider geht dein Plan so nicht ganz auf...$auswahl wird ja bei Punkt 2 schon durchgeschleift, zwar nicht in einem hidden Feld, aber es passiert . So, nun sind die Daten von $auswahl also da und mir wird auch ordentlich z. B. "text 2" angezeigt. Sobald ich aber nun auf "Ändern" klicke, ist die Variable $auswahl weg und das UPDATE funktioniert natürlich nicht mehr.
Nach dem Abschicken des geänderten Formulares gibt es logischerweise nur noch zwei Variablen, und zwar
- $edititeren | mit der ausgabe "Ändern"
und
- $ausgabe | mit dem geänderten Text
Mfg
Mitchell
. So, nun sind die Daten von $auswahl also da und mir wird auch ordentlich z. B. "text 2" angezeigt. Sobald ich aber nun auf "Ändern" klicke, ist die Variable $auswahl weg und das UPDATE funktioniert natürlich nicht mehr.Nach dem Abschicken des geänderten Formulares gibt es logischerweise nur noch zwei Variablen, und zwar
- $edititeren | mit der ausgabe "Ändern"
und
- $ausgabe | mit dem geänderten Text
Mfg
Mitchell
zwar nicht in einem hidden Feld, aber es passiert
... na wie auch immer - ist ja nicht so wichtigNach dem Abschicken des geänderten Formulares gibt es logischerweise nur noch zwei Variablen, und zwar
... dann solltest Du evtl doch mal dem Formular ein Hidden-Input Feld spendieren, und es mit der Variablen $auswahl füllen.( und wieso $auswahl - ist ist doch eher $datei ... ich bin echt verwirrt )
echo <<<__HTML__
<form action="" name="aenderung" method="post">
<input type="hidden" name="datei" value="$datei" />
<input type="submit" name="action" value="Ändern" />
<p> <textarea name="ausgabe" cols="105" rows="30">$text</textarea> </p>
</form>
__HTML__;BTW . da ich fasst am verzweifeln bin hab ich schon mal ein Script geschrieben dazu - mich wundert jetzt aber das deine SQL Statements
- auf unterschiedliche Tabellen verweisen
- $auswahl gegen unterschiedliche Datenfelder geprüft wird ( `datei` bzw `file` )
- das es offenbar eine Art Versions-Kontrolle gibt für die Texte, aber Du beim speichern diese aktualisiert anstatt eine neue Version anzulegen.
kopf > tisch | auuuuuuuuuuuuutttttttttsssssssscccccccchhhhhhhhhhh
Da könnt ich mir doch...danke für dein Script, seit gestern brauche ich es aber nicht mehr. Ich Depp habe dem falschen Formular das hidden Feld übergeben...dem mit dem Text-Select nämlich. So läufts dann endlich:
Tschuldige die Verwirrung, ich wars halt auch und konnte es nicht ändern ^^.
"auf unterschiedliche Tabellen verweisen" | wie meinst du das? Es werden 3 Tabellen verwenden (inhalt, datum, datei)
"$auswahl gegen unterschiedliche Datenfelder geprüft wird ( `datei` bzw `file` )" | ähm...nö, wo denn?
"das es offenbar eine Art Versions-Kontrolle gibt für die Texte, aber Du beim speichern diese aktualisiert anstatt eine neue Version anzulegen." | nein, eine Kontrolle findet nicht statt. Der Text wird so ausgegeben, wie er einst in der DB abgespeichert wurde. Beim Abspeichern gibts nur ein Update des eben editierten Textes, richtig.
Mfg
Mitchell
Da könnt ich mir doch...danke für dein Script, seit gestern brauche ich es aber nicht mehr
. Ich Depp habe dem falschen Formular das hidden Feld übergeben...dem mit dem Text-Select nämlich. So läufts dann endlich:$textwahl = $_POST['textauswahl'];
echo "<form action='panel.php' method='post'><input type='hidden' name='textauswahl' value=".$auswahl."><input class='panel_button' name='edit' type='submit' value='Ändern'><p><textarea name='ausgabe' cols='105' rows='30'>".$text_out['inhalt']."</textarea></p></form>";
if ($editieren == 'Ändern')
mysql_query("UPDATE tbl SET inhalt='$ausgabe', datum=NOW() WHERE datei = '$textwahl'"); Tschuldige die Verwirrung, ich wars halt auch und konnte es nicht ändern ^^.
"auf unterschiedliche Tabellen verweisen" | wie meinst du das? Es werden 3 Tabellen verwenden (inhalt, datum, datei)
"$auswahl gegen unterschiedliche Datenfelder geprüft wird ( `datei` bzw `file` )" | ähm...nö, wo denn?
"das es offenbar eine Art Versions-Kontrolle gibt für die Texte, aber Du beim speichern diese aktualisiert anstatt eine neue Version anzulegen." | nein, eine Kontrolle findet nicht statt. Der Text wird so ausgegeben, wie er einst in der DB abgespeichert wurde. Beim Abspeichern gibts nur ein Update des eben editierten Textes, richtig.
Mfg
Mitchell
So läufts dann endlich
... gut das freut mich - und ich hoffe Du versucht mal mehr mit Objekten und MVC zu Arbeiten. Am Anfang wird dir das sicherlich als viel unnötige coderei vorkommen, aber man hat im Endeffekt eine bessere Struktur im Code.
Ich habe bei Klassen ehrlich gesagt noch nicht den richtigen Durchblick (Aufbau und Sinn sind mir aber bekannt ^^)...brings mir zwar bei, aber leider momentan wenig Zeit dafür. MVC halte ich irgendwie nichts von...bei kleinen Projekten wie diesen sehe ich den Nutzen nicht wirklich.
Danke nochmal für die Gedult...noch einer, dem ich ein Bier schulde
Mfg
Mitchell
Danke nochmal für die Gedult...noch einer, dem ich ein Bier schulde
Mfg
Mitchell
