insert funzt nicht bei mehrzeilern

Bohninger · 21.06.2011, 11:41

moin,

ich bastel gerade an einem forum/blog und bin zur zeit bei der zitier-funktion angelang. klappt auch soweit gut, aber nur wenn lediglich eine zeile eingetragen wurde. bei mehrzeilern wird nichts in das formular über-/eingetragen.

hier der code:

<input type="button" value="Zitieren" style="FONT-WEIGHT: bold;text-decoration: underline" onClick="insert('<div class=\'code\'>Zitat:<? echo nl2br($row->antwort.""); ?>', '</div>')">

wäre dufte wenn mir jemand weiter helfen kann.

nikosch · 21.06.2011, 11:50

Javascript kennt keine mehrzeiligen Strings. Jedenfalls nicht so. Schau mal in den Browserquellcode.

fab · 21.06.2011, 11:52

Dann schau mal, was nl2br genau macht:

Zitat:

Returns string with '<br />' or '<br>' inserted before all newlines (\r\n, \n\r, \n and \r).

Wenn du die Zeilenumbrüche anschließend entfernen willst, musst du das schon separat mit str_replace() o.ä. tun.

Nebenbei fehlt dir noch das escapen von HTML und Anführungszeichen in $row->antwort - überleg mal was passiert, wenn z.B. folgendes darin steht:

Code:

');alert('XSS')"/> <script type="text/javascript">alert('noch mehr XSS');</script>

Bohninger · 21.06.2011, 12:46

danke für die antworten, werde mich mal durchboxen

Bohninger · 21.06.2011, 16:23

habs geschafft. habe die umbrüche escaped:

PHP-Code:

 
          <?
          $row->antwort = preg_replace('/\r\n|\r|\n/', ' ', $row->antwort);
          ?>

HTML-Code:

 <input type="button" value="Zitieren" style="FONT-WEIGHT: bold;text-decoration: underline" onClick="insert('<div class=\'code\'>Zitat:<? echo $row->antwort; ?>', '</div>')">

nikosch · 21.06.2011, 16:27

Nö, haste nicht. Du hast sie ersetzt.

Beim Posten von Quellcode oder strukturierten Daten bitte bbCode verwenden ([php] für PHP, [html] für HTML, [code] für den Rest (Dumps, Queries..)) und Code geeignet einrücken.

Bohninger · 21.06.2011, 16:34

what ever, es funzt

nikosch · 21.06.2011, 16:37

In einem Zitat will man aber Absätze nicht durch Leerzeichen ersetzen?!

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
INSERT INTO funktioniert nicht	Schorschologe	Datenbanken	5	25.01.2011 16:02
[Erledigt] mehrfaches INSERT über PHP	drzwockel	PHP Tipps 2010	10	10.08.2010 11:03
Problem mit Mysql Left Join Ambfrage	pro-tech	Datenbanken	11	22.05.2010 10:59
Formular &gt; Insert into Befehl funzt nicht	Phalanx	Datenbanken	1	01.03.2009 20:13
Pathfinding Algorithmus optimieren? (Algorith. von Dijkstra)	Martin13	PHP Tipps 2007	19	04.09.2007 19:20
Verständnisprobleme bei Utf8 > Character Set > Collate	erna	Datenbanken	4	20.03.2007 14:08
Differenz zwischen 2 Zahlen	duderino	PHP Tipps 2006	19	10.08.2006 22:12
PRoblem mit Cookie bei PHPbb-Board		PHP Tipps 2007	8	12.12.2005 20:56
mysql dump einspielen (ohne phpmyadmin)		PHP Tipps 2005-2	6	18.06.2005 21:50
[Erledigt] fehler den ich noch nie gesehen habe		Datenbanken	7	09.06.2005 14:29
insert into funzt nicht ?	notyyy	Datenbanken	2	22.05.2005 02:04
insert into funzt nicht ?	notyyy	Datenbanken	4	16.05.2005 00:21
dynamisches MENÜ aus db lesen.probleme mit SUBid !!		PHP Tipps 2005	26	24.03.2005 18:53
dem verzweifeln nahe		PHP Tipps 2004	26	24.10.2004 17:28
HILFE: Column count doesn't match value count at row 1		Datenbanken	17	12.06.2004 16:45

21.06.2011, 11:41
Bohninger Neuer Benutzer Registriert seit: 21.06.2011 Beiträge: 4 PHP-Kenntnisse: Fortgeschritten	insert funzt nicht bei mehrzeilern moin, ich bastel gerade an einem forum/blog und bin zur zeit bei der zitier-funktion angelang. klappt auch soweit gut, aber nur wenn lediglich eine zeile eingetragen wurde. bei mehrzeilern wird nichts in das formular über-/eingetragen. hier der code: <input type="button" value="Zitieren" style="FONT-WEIGHT: bold;text-decoration: underline" onClick="insert('<div class=\'code\'>Zitat:<? echo nl2br($row->antwort.""); ?>', '</div>')"> wäre dufte wenn mir jemand weiter helfen kann.


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

21.06.2011, 11:50
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	Javascript kennt keine mehrzeiligen Strings. Jedenfalls nicht so. Schau mal in den Browserquellcode. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

21.06.2011, 12:46
Bohninger Neuer Benutzer Registriert seit: 21.06.2011 Beiträge: 4 PHP-Kenntnisse: Fortgeschritten	danke für die antworten, werde mich mal durchboxen

21.06.2011, 16:23
Bohninger Neuer Benutzer Registriert seit: 21.06.2011 Beiträge: 4 PHP-Kenntnisse: Fortgeschritten	habs geschafft. habe die umbrüche escaped: PHP-Code: `<? $row->antwort = preg_replace('/\r\n\|\r\|\n/', ' ', $row->antwort); ?>` HTML-Code: <input type="button" value="Zitieren" style="FONT-WEIGHT: bold;text-decoration: underline" onClick="insert('<div class=\'code\'>Zitat:<? echo $row->antwort; ?>', '</div>')"> Geändert von Bohninger (21.06.2011 um 16:32 Uhr).

21.06.2011, 16:27
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	Nö, haste nicht. Du hast sie ersetzt. Beim Posten von Quellcode oder strukturierten Daten bitte bbCode verwenden ([php] für PHP, [html] für HTML, [code] für den Rest (Dumps, Queries..)) und Code geeignet einrücken. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

21.06.2011, 16:34
Bohninger Neuer Benutzer Registriert seit: 21.06.2011 Beiträge: 4 PHP-Kenntnisse: Fortgeschritten	what ever, es funzt

21.06.2011, 16:37
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	In einem Zitat will man aber Absätze nicht durch Leerzeichen ersetzen?! __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --