Modify

Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#88 closed Bug/Fehler (fixed)

xtc_validate_email.inc.php überprüfen

Reported by: Ronald Parcinski Owned by: H. H. H.
Priority: normal Milestone: modified-shop-2.0.0.0
Component: Admin Version: 2.0.0.0
Keywords: Cc:
Blocked By: Blocking:

Description

Wir müssen prüfen ob die Email Validierung noch dem aktuellen Stand entspricht. Vor allem der Regex mit den gültigen Domains.

Attachments (0)

Change History (12)

comment:1 by Ronald Parcinski, 12 years ago

Summary: xtc_validate_email.inc.php übeprüfenxtc_validate_email.inc.php überprüfen

comment:2 by Simon Heinrich, 12 years ago

Ich habe im Trunk das Verfahren etwas angepasst:

Für PHP > 5.2 gibt es eine neue Funktion, die wir hier nutzen sollten.
Für alle älteren Versionen, habe ich den REGEX dahingehend vereinfacht, dass er im Wesentlichen nur noch auf das Vorhandensein von @ und . und dabei keine Whitespaces erlaubt.

Den Rest sollte einfach die Confirmation-Mail oder die Funktion CheckDNS() abfangen.

Siehe r5430

Last edited 12 years ago by Simon Heinrich (previous) (diff)

comment:3 by Torsten Riemer, 12 years ago

Spricht etwas dagegen das auch nach /tags/modified-shop-1.06 zu übernehmen?
Ist das abwärtskompatibel zu PHP 5.0?

comment:4 by Simon Heinrich, 12 years ago

Es sollte noch beschlossen werden, ob man das vereinfachte REGEX (oder ein anderes) beibehalten möchte.

Der "else"-Zweig ist für PHP Versionen kleiner 5.2 gedacht...
In den älteren PHP-Versionen gibt es die Funktion "filter_var" nicht.

comment:5 by Torsten Riemer, 12 years ago

Prima, danke dir! Dann warten wir mal, was die anderen Entwickler hierzu sagen, bevor wir es nach /tags/modified-shop-1.06 übernehmen.

comment:6 by Torsten Riemer, 12 years ago

Ronald hat mich gebeten, dass ich r5430 erstmal wieder revoke, da wir damit zu viele E-Mail Adressen sperren würden. Wir werden das auf dem Team-Treff mal besprechen.

Revoked in r5434

comment:7 by Simon Heinrich, 12 years ago

Eigentlich werden dadurch weniger Emailadressen gesperrt, da das REGEX nun lockerer ist.

Warum kann man das nicht hier diskutieren?

comment:8 by Torsten Riemer, 12 years ago

Ja gerne. Ronald hatte nur vorgeschlagen, dass wir das Thema auch beim Treffen bereden können, was nicht heisst, dass wir hier nicht mehr schreiben. ;-)

comment:9 by Gerhard Waldemair, 12 years ago

Milestone: modified-shop-2.00modified-shop-2.10

comment:10 by Gerhard Waldemair, 12 years ago

Owner: changed from Gerhard Waldemair to H. H. H.
Status: newassigned

comment:11 by Ronald Parcinski, 12 years ago

Resolution: fixed
Status: assignedclosed

fixed in r6392

comment:12 by Torsten Riemer, 10 years ago

Milestone: modified-shop-2.10modified-shop-2.00
Version: 1.062.0

Modify Ticket

Action
as closed The owner will remain H. H. H..
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.