Fehler in Trunk 1075

[WebBird]

Och, Leudde, nu werdet doch nich immer gleich so emotional. Man kann sich ja mal aufregen, mach ich auch, aber letztlich helfen tut es dann doch nie. Als Developer kann man schon mal etwas die Nerven verlieren, wenn sich auf einmal die Anforderungen und Meldungen vermehren wie die Tribbles. Aber sich dann zu streiten, hilft auch nicht weiter. Notfalls einfach 'nen Tag drüber schlafen... hoppla, ich meinte natürlich 'ne Nacht

[WebBird]

bin mir ziemlich sicher, dass das schon immer so war.

Hab's grad nochmal mit 2.7 getestet, ist dort definitiv auch so. (Egal ob man auf [Zurück] klickt oder den [Back]-Button des Browsers benutzt.)

[erpe0812]

Och, Leudde, nu werdet doch nich immer gleich so emotional.

[WebBird]

Bezog sich auf die Seite 1.

[Luisehahne]

Es war bereits in allen Versionen davor. Was ich auch als störend empfinde, in den Gruppenrechten, der Button zurücksetzen sollte auch auf die Module angewandt werden.

Ist auch noch nie so gewesen.

Wenns noch mehrere Vorschläge für Verbesserungen gibt, immer ran damit. Auch wenn man mal ein bisschen hinterfragen muss. Es wird sich mit allem beschäftigt. Ob das Ein ode rAndere sinnvoll ist umszusetzen wird dann jeweils im Dev-Team ausdiskustiert.

Ich bin immer dafür, Fehler die ein Anwender machen könnte, vorher abzufangen. Ich entscheide das aber nicht alleine. Dafpr haben wir ein Team was demokratisch entscheidet.

Dietmar

[susigross]

Noch ein sicher alter Bug, über den ich soeben erstmalig gestolpert bin:
Backend - Einstellungen - SMTP-Passwort
Wenn das Passwort ein "<" enthält, werden nur die Zeichen davor im Feld wbmailer_smtp_passw ord der settings-Tabelle gespeichert.
Da Passwörter mit solchen Zeichen in der Praxis vorkommen, wäre es schon schön, wenn WB in der Lage wäre die auch zu speichern.

[aldus]

Noch ein sicher alter Bug, über den ich soeben erstmalig gestolpert bin:
Backend - Einstellungen - SMTP-Passwort
Wenn das Passwort ein "<" enthält, werden nur die Zeichen davor im Feld wbmailer_smtp_passw ord der settings-Tabelle gespeichert.
Da Passwörter mit solchen Zeichen in der Praxis vorkommen, wäre es schon schön, wenn WB in der Lage wäre die auch zu speichern.

Hm ... ok, habe das mal im DevForum mit reingestellt ...
Der Grund dahinter ist die Funktion "strip_tags", die einfach nach dem "<" alles abschneidet.
Testweise kannst Du ja in Zeile 141 den "wbmailer_smtp_passw ord" mit in das $allow_tags_in_fields array mit aufnehmen:

Code: [Select]

<?php
$allow_tags_in_fields = array(&#39;website_header&#39;, &#39;website_footer&#39;,&#39;wbmailer_smtp_password&#39;);
?>

Hm ... jetzt werden alle Zeichen übernommen ... habe allerdings keine Ahnung wie SMTP mit den Entities, sprich
mit "&lt;", dann in Passwörtern umgeht. Könnte einen //Seiteneffekt// haben ...

Gruß
Aldus

[susigross]

Hallo Aldus,

man muss wohl außerdem die Zeile 48 der index.php wie folgt ändern:

Code: [Select]

   if ($setting_name == 'wbmailer_smtp_password') {
        $setting_value = $setting['value'];
    } else {
        $setting_value = htmlspecialchars($setting['value']);
    }
Ein Passwort darf nun mal nicht verändert werden, damit es funktioniert.
An der Stelle ist nun mal auch wirklich kein böser Userinput zu erwarten, da ja das Feld nur der Admin schreiben darf.

[aldus]

Hallo Frank

Hm ... da würde ich lieber einen Einzeiler draus machen, dann verbiege ich die Zeilennummern nicht so
derbe und bin das Klammerngeraffele los:

Code: [Select]

<?php
$setting_value = ( $setting_name != &#39;wbmailer_smtp_password&#39; ) ? htmlspecialchars($setting[&#39;value&#39;]) : $setting[&#39;value&#39;];
?>

Hm ... ich schau noch mal

Gruß
Aldus

[WebBird]

Boah, Aldus... So fiese Sachen machst Du?!? Sowas in eine Zeile?!?