Bug #113

bucket-note gets messed up by quotes on saving

Added by Andreas Unterkircher 1161 days ago. Updated 1159 days ago.

Status:Closed Start:11/30/2008
Priority:Normal Due date:
Assigned to:Andreas Unterkircher % Done:

100%

Category:internals Spent time: 1.00 hour
Target version:1.5

Description

Reported by Jörn Bredereck

Hallo Andreas,

ich bin gerade dabei Nephtys auszuprobieren und habe möglicherweise einen Bug (Version 1.4) gefunden:

Bei der Eingabe des freien Bucket-Textes (bucket_note) werden Zeilenumbrüche aus dem HTML-Textarea-Feld anscheinend nicht richtig geparsed. Bei jedem weitereren Editier-Vorgang wird darübrerhinaus ein "\" mehr als Escape-Zeichen hinzugefügt. Wenn man einen Bucket mehrmals bearbeitet sieht das Feld bucket_note dann z.B. so aus:

mysql> select bucket_note from nephthys_buckets;
+------------------------------------------------------------------------+
| bucket_note                                                            |
+------------------------------------------------------------------------+
|                                                                        |
| Hi Camilla\\\\\\\\r\\\\\\\\nwie findest Du das?\\\\\\\\r\\\\\\\\nGru� |
| Mach mal was                                                           |
| von gmx an juwi\\\\\\\\r\\\\\\\\nBernd                                 |
| test                                                                   |
|                                                                        |
| Hallo asdf\\\\\\\\\\\\\\\\r\\\\\\\\\\\\\\\\nasdf                       |
| asdf\\\\r\\\\nsss                                                      |
+------------------------------------------------------------------------+
8 rows in set (0.00 sec)

Der Text wird dann beim generieren der Benachrichtungs-E-Mails exakt so in die E-Mail geschrieben; d.h. anstatt die Zeilenumbrüche ("\n") in Zeilenumbrüche innerhalb der E-Mails umzusetzen erschtin in der E-Mail dann z.B.

asdf\\\\r\\\\nsss

Hier noch ein paar Daten zu meiner Umgebung:

root@fileshare:~# dpkg -l |grep apache
ii  apache2-mpm-prefork               2.2.8-1                     Traditional model for Apache HTTPD
ii  apache2-utils                     2.2.8-1                     utility programs for webservers
ii  apache2.2-common                  2.2.8-1                     Next generation, scalable, extendable web se
ii  libapache2-mod-php5               5.2.4-2ubuntu5.3            server-side, HTML-embedded scripting languag

root@fileshare:~# dpkg -l |grep php5
ii  libapache2-mod-php5               5.2.4-2ubuntu5.3            server-side, HTML-embedded scripting languag
ii  php5                              5.2.4-2ubuntu5.3            server-side, HTML-embedded scripting languag
ii  php5-cli                          5.2.4-2ubuntu5.3            command-line interpreter for the php5 script
ii  php5-common                       5.2.4-2ubuntu5.3            Common files for packages built from the php
ii  php5-mysql                        5.2.4-2ubuntu5.3            MySQL module for php5

Associated revisions

Revision de253b482760731a1ac542e28ca07059a2ed7764
Added by Andreas Unterkircher 1159 days ago

further rely on prepared SQL statements and no longer do quoting ourself, fixes #113

switch from ordinary MySQL to MySQLi interface, as it better support prepared statements

completely remove escape() function as it is no longer used

Signed-off-by: Andreas Unterkircher <>

History

Updated by Andreas Unterkircher 1159 days ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100

Updated by Andreas Unterkircher 1159 days ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF