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 "Zeichen@…" abfrägt und dabei keine Whitespaces erlaubt.

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

Siehe r5430

Version 0, edited 12 years ago by Simon Heinrich (next)

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.