Bug #43924

Form - Email validation is case sensitive

Added by d.ros no-lastname-given over 6 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2012-12-12
Due date:
% Done:

100%

TYPO3 Version:
4.7
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

At the moment in TYPO3 4.7 an 6.0 the email field validation is case sensitive, which is an highly unwanted feature. It is not security relevant but is disturbing some actions with inexperienced form users.

Just make a constant switch for that or make it just caseinsensitive at all.

Cheers


Related issues

Related to TYPO3 Core - Bug #45605: Forms: Optimize and fix certain validation rules Closed 2013-02-19

Associated revisions

Revision ae7e35c1 (diff)
Added by Patrick Broens over 6 years ago

[BUGFIX] Form email validation is case sensitive

Currently the FORM email validation is using a regular expression which
checks the email address. This check also inclused case sensitivity.

This patch fixes this by using the email address validator which comes
with the extension core.

Change-Id: Ib1449198f66f023a01c9791e74eb2d634abd77df
Resolves: #43924
Releases: 6.0, 6.1, 6.2
Reviewed-on: https://review.typo3.org/20421
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

Revision afd5d770 (diff)
Added by Patrick Broens over 6 years ago

[BUGFIX] Form email validation is case sensitive

Currently the FORM email validation is using a regular expression which
checks the email address. This check also inclused case sensitivity.

This patch fixes this by using the email address validator which comes
with the extension core.

Change-Id: Ib1449198f66f023a01c9791e74eb2d634abd77df
Resolves: #43924
Releases: 6.0, 6.1, 6.2
Reviewed-on: https://review.typo3.org/20524
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

Revision 4a35407c (diff)
Added by Patrick Broens over 6 years ago

[BUGFIX] Form email validation is case sensitive

Currently the FORM email validation is using a regular expression which
checks the email address. This check also inclused case sensitivity.

This patch fixes this by using the email address validator which comes
with the extension core.

Change-Id: Ib1449198f66f023a01c9791e74eb2d634abd77df
Resolves: #43924
Releases: 6.0, 6.1, 6.2
Reviewed-on: https://review.typo3.org/20525
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

Revision 0c422a7c (diff)
Added by Stefan Neufeind about 6 years ago

[BUGFIX] Form email validation is case sensitive

Form email-validation is using a regular expression which is
case-sensitive.

Backporting the changes from 6.x would be a too large change.
But adding case-invalid check for the regex is okay.

Change-Id: Ib1449198f66f023a01c9791e74eb2d634abd77df
Resolves: #43924
Releases: 4.7
Reviewed-on: https://review.typo3.org/22117
Reviewed-by: Ernesto Baschny
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Felix Nagel over 6 years ago

Should use default TYPO3 email validator (t3lib_div).

#2 Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20421

#3 Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20421

#4 Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20421

#5 Updated by Gerrit Code Review over 6 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20421

#6 Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/20524

#7 Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/20525

#8 Updated by Patrick Broens over 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#9 Updated by Rik Willems about 6 years ago

Is there a chance this will be changed in 4.5 and 4.7 too?

Cheers! Rik

#10 Updated by Gerrit Code Review about 6 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/22117

#11 Updated by Stefan Neufeind about 6 years ago

Backporting the change from 6.x (switching the actual check) would be too large for a stable release (4.5/4.7 - only security-fixes) imho. But let's see if adding a case-insentive flag to the preg finds approval. https://review.typo3.org/22117

#12 Updated by Gerrit Code Review about 6 years ago

Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/22117

#13 Updated by Stefan Neufeind about 6 years ago

Form was introduced with 4.7, so nothing to think about for 4.5.

#14 Updated by Stefan Neufeind about 6 years ago

  • Status changed from Under Review to Resolved

#15 Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF