Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[3.5] Problem mit Umlauten bei E-Mail Adressen #8306

Closed
comolo opened this issue Apr 15, 2016 · 5 comments
Closed

[3.5] Problem mit Umlauten bei E-Mail Adressen #8306

comolo opened this issue Apr 15, 2016 · 5 comments
Assignees
Labels
Milestone

Comments

@comolo
Copy link

comolo commented Apr 15, 2016

Hallo zusammen,

ich bin auf folgenden Bug gestoßen:

Hintergrund:
Über ein eigenes Skript werden Daten automatisch in die Mitgliedertabelle tl_member importiert. Manche E-Mail-Adressen enthalten Umlaute und diese werden vom Skript in Punnycode konvertiert und dann in die Datenbank geschrieben.

Problem:
Wenn man die Mitgliederverwaltung im Backend aufruft, wird in der Übersicht die E-Mail-Adresse im Punnycode-Format angezeigt, so wie diese auch in der Datenbank steht. Zwar nicht schön, aber ok.
Wird nun das Mitglied aufgerufen, steht im Feld E-Mail der Inhalt wieder im Unicode-Format drin.
Das Problem: Man kann nun allerdings den Mitglieder-Datensatz nicht mehr abspeichern, da sich Contao beschwert, dass die E-Mail-Adresse Umlaute enthält.

Lösung:
Die Lösung wäre, dass Contao die E-Mail-Adressen vor dem Speichern in das Punnycode-Format konvertiert und diese bei der Ansicht wieder in Unicode ausgibt.

Beispiel:
In der Liste Darstellung im Punnycode Format
auswahl_003

Beim Bearbeiten: im Unicode Format
auswahl_002

Beim Speichern -> Fehler
auswahl_004

@asaage
Copy link

asaage commented Apr 15, 2016

klingt ja wild...
Ich kann mich erinnern, das vor einiger Zeit die akzeptierten Zeichen in E-mails erweitert wurden.
#7984
Hat allerdings nicht so viel mit deinem Problem zu tun...
Punnycode in der Datenbank halte ich für eine schlechte Idee - also fast so schlecht wie Umlaute in einer Mailadresse.

@comolo
Copy link
Author

comolo commented Apr 15, 2016

Bei der betroffenen Website läuft Contao 3.5.4. Wir prüfen mal, ob das Problem mit der Version 3.5.9 noch besteht.

Ich gebe dir Recht, es ist eleganter die E-Mail-Adressen nicht im Punnycode-Format abzuspeichern.
Aber erstmal wäre wichtig, dass das Problem gelöst wird.

@leofeyer
Copy link
Member

Punnycode in der Datenbank halte ich für eine schlechte Idee

Ich nicht. Ich halte ein standardisiertes Format für eine sehr gute Idee, weil es viele Probleme gelöst hat, die wir vorher hatten (unter dem Suchbegriff "Punycode" sollte es Issues geben).

Wird nun das Mitglied aufgerufen, steht im Feld E-Mail der Inhalt wieder im Unicode-Format drin.

Ich kann das initiale Problem nicht reproduzieren. Wenn ich in der Onlinedemo die E-Mail-Adresse von Donna Evans auf d.eväns@example.com ändere, wird sie nach dem Speicher nicht im Punycode-Format ausgegeben.

@asaage
Copy link

asaage commented Apr 15, 2016

okay - dasss punnycode implementiert ist wusste ich nicht. Aber ich habe im BE überhaupt keine Möglichkeit Umlaute in Mails zu verwenden - weder bei user noch bei member.
image

@leofeyer leofeyer added this to the 3.5.10 milestone Apr 15, 2016
@leofeyer leofeyer self-assigned this Apr 19, 2016
@leofeyer
Copy link
Member

Behoben in 9f1e039.

jsonn pushed a commit to jsonn/pkgsrc that referenced this issue Apr 23, 2016
### 4.1.3 (2016-04-22)

 * Use data URIs for the image preview in the back end.
 * Use DIRECTORY_SEPARATOR to convert kernel.cache_dir into a relative path (see #464).
 * Always trigger the "isVisibleElement" hook (see contao/core#8312).
 * Do not change all sessions when switching users (see contao/core#8158).
 * Do not allow to close fieldsets with empty required fields (see contao/core#8300).
 * Make the path related properties of the File class binary-safe (see contao/core#8295).
 * Correctly validate and decode IDNA e-mail addresses (see contao/core#8306).
 * Skip forward pages entirely in the book navigation module (see contao/core#5074).
 * Do not add the X-Priority header in the Email class (see contao/core#8298).
 * Determine the search index checksum in a more reliable way (see contao/core#7652).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants