Ticket #199 (closed Fehlermeldung: fixed)

Opened 2 months ago

Last modified 2 months ago

Zunahme an backslashes in Blogbeschreibung

Reported by: jeena Assigned to: jeena
Priority: critical Milestone: Jlog 1.1.2
Component: Admincenter Version: 1.1.1
Keywords: Cc:

Description

Wie man am Demoblog sieht werden jedes mal Slashes hinzugefügt wenn man die Einstellungen speichert:

Dies ist nur ein Testblog.<br /><a href=\\\"/projects/jlog/demo/admin/\\\">Admincenter</a><br />Passwort: <strong>test</strong>

Attachments

Change History

07/19/2008 04:04:56 PM changed by robertb

  • status changed from new to closed.
  • resolution set to invalid.

(follow-up: ↓ 3 ) 09/19/2008 10:53:40 PM changed by jeena

  • status changed from new to closed.
  • resolution set to fixed.

Warum hast du es zu "invalid" geändert? Ich hab doch sogar ein konkretes Beispiel gebracht.

(in reply to: ↑ 2 ) 09/20/2008 12:56:10 AM changed by robertb

Replying to jeena:

Warum hast du es zu "invalid" geändert? Ich hab doch sogar ein konkretes Beispiel gebracht. Das war nicht ich. Habe ich einen PGP-Key von dir, dann schicke ein neues Passwort zu.

09/20/2008 10:16:41 AM changed by driehle

  • status changed from closed to reopened.
  • resolution deleted.

Scheint ein Bug von Track zu sein, ich öffne das Ticket mal wieder, da es ja noch keiner im SVN behoben hat...

09/20/2008 12:37:15 PM changed by driehle

  • milestone set to Jlog 1.1.2.

09/20/2008 02:13:52 PM changed by jeena

Jein, war kein richtiger "Bug", das lag wie dennis schon vermutete daran dass das gelöschter spam war, wo aber die änderungen an sich noch in der DB mit der gleichen ID herumschwirrten. Ich hab das hoffentlich jetzt einigermaßen bereinigt und bei der gelegenheit auch älteren Spam rausgemacht.

09/22/2008 09:10:48 AM changed by jeena

  • owner set to jeena.
  • status changed from reopened to new.

Ich habe jetzt in der settings.class.php in der get_value()-Methode beim return stripslashes() benutzt, da wir ja beim erstellen tes Textes für die settings.inc.php auch addslashes() auf jede Konstante anwenden. Ich hoffe dass das nicht plötzlich irgendwo anders zu Problemen führt, kann mir jetzt aber nicht denken wo.

Ich kann es jetzt grad noch nicht einchecken, mach das aber bald.

09/22/2008 12:08:44 PM changed by jeena

  • status changed from new to closed.
  • resolution set to fixed.

fixed mit [1760]

09/28/2008 09:26:31 AM changed by jeena

  • status changed from closed to reopened.
  • resolution deleted.

Ich hab jetzt erst gesehen dass jemand das da aus einem bestimmten grund rausgemacht hat, damit es unter Windows funktioniert. Mangels windows kann ich das jetzt aber leider gar nicht testen :-( hat einer von euch Windows installiert?

09/30/2008 03:19:12 PM changed by driehle

Ich nutze Windows, und ich war es auch, der das Stripslashes an der anderen Stelle da rausgenommen hatte, da dies unter Windows Probleme bereitete.

Abgsehen davon gehört dort wo du mit [1760] stripslashes eingefügt hast, ebendieses nicht hin. Die Daten sollten immer im Rohformat im Hauptspeicher gehalten werden, also auch ohne irgendwelche zusätzliche Backslashes. Somit darf beim Auslesen eines Arrays niemals ein stripslashes auftauchen. Der Fehler muss irgendwo an der Stelle liegen, wo die Daten in das Array geschrieben werden - dort wird wohl ein addslashes zu viel sein, oder ein magic_quotes nicht richtig behandelt sein.

09/30/2008 05:27:29 PM changed by driehle

  • status changed from reopened to closed.
  • resolution set to fixed.

Ich habe in [1761] ff. die Settings-Klasse mal etwas überarbeitet und auch das Verfahren zum Speichern der settings.inc.php gründlich aufgeräumt.

Das von dir beschriebene Verhalten kann ich nun mit Revision 1764 aus trunk nicht mehr feststellen, weder unter Linux, noch unter Windows, weder mit magic_quotes aktiviert, noch mit magic_quotes deaktiviert.

Somit schließe ich dieses Ticket.

Add/Change #199 (Zunahme an backslashes in Blogbeschreibung)




Action