Ticket #199 (closed Fehlermeldung: fixed)
Zunahme an backslashes in Blogbeschreibung
| Reported by: | jeena | Owned by: | 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>
Change History
comment:1 Changed 4 years ago by robertb
- Status changed from new to closed
- Resolution set to invalid
comment:2 follow-up: ↓ 3 Changed 3 years ago 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.
comment:3 in reply to: ↑ 2 Changed 3 years ago 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.
comment:4 Changed 3 years ago by driehle
- Status changed from closed to reopened
- Resolution fixed deleted
Scheint ein Bug von Track zu sein, ich öffne das Ticket mal wieder, da es ja noch keiner im SVN behoben hat...
comment:6 Changed 3 years ago 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.
comment:7 Changed 3 years ago 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.
comment:8 Changed 3 years ago by jeena
- Status changed from new to closed
- Resolution set to fixed
fixed mit [1760]
comment:9 Changed 3 years ago by jeena
- Status changed from closed to reopened
- Resolution fixed 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?
comment:10 Changed 3 years ago 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.
comment:11 Changed 3 years ago 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.
