Project

General

Profile

Actions

Bug #45605

closed

Forms: Optimize and fix certain validation rules

Added by move:elevator no-lastname-given about 11 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Won't have this time
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2013-02-19
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

The following validations do not work properly:
  • e-mail address: there is no regex behind the test. It just looks for "@" surrounded by text.
  • mandatory for select-boxes: this validation ALWAYS succeeds because it is not possible to add an empty select-option. -> see #69396
  • length-validation (e.g. for simple text-inputs) with a given range doesn't work because the saved length is always 0. -> see #69024

Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #43924: Form - Email validation is case sensitiveClosed2012-12-12

Actions
Related to TYPO3 Core - Bug #69024: Forms: form wizard not working when trying to enter numbers for length validatorClosed2015-08-13

Actions
Related to TYPO3 Core - Task #69396: Forms: validation of SELECT elementClosed2015-08-28

Actions
Actions #1

Updated by Mathias Schreiber over 9 years ago

  • Target version set to 7.5
  • Is Regression set to No
Actions #2

Updated by Björn Jacob almost 9 years ago

  • Category changed from Form Framework to 1602
Actions #3

Updated by Björn Jacob almost 9 years ago

  • Category changed from 1602 to Form Framework
Actions #4

Updated by Björn Jacob over 8 years ago

  • Subject changed from Forms: Validations to Forms: Optimize and fix validation certain rules
Actions #5

Updated by Björn Jacob over 8 years ago

  • Description updated (diff)
Actions #6

Updated by Björn Jacob over 8 years ago

  • Subject changed from Forms: Optimize and fix validation certain rules to Forms: Optimize and fix certain validation rules
Actions #7

Updated by Björn Jacob over 8 years ago

  • Status changed from New to In Progress
Actions #8

Updated by Björn Jacob over 8 years ago

  • Description updated (diff)
Actions #9

Updated by Gerrit Code Review over 8 years ago

  • Status changed from In Progress to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42906

Actions #10

Updated by Björn Jacob over 8 years ago

  • Assignee set to move:elevator no-lastname-given

@move:elevator: Please check #69396. There I've posted the solution for having a mandatory SELECT object.

Actions #11

Updated by Björn Jacob over 8 years ago

  • Assignee deleted (move:elevator no-lastname-given)
Actions #12

Updated by Björn Jacob over 8 years ago

  • Target version changed from 7.5 to 7 LTS
  • TYPO3 Version changed from 6.0 to 7
Actions #13

Updated by Björn Jacob over 8 years ago

  • Status changed from Under Review to Closed
  • Priority changed from Must have to Won't have this time
  • Target version deleted (7 LTS)

The validator is okay since it does more than just looking for a @ sign, check code.

If you need a better/ deeper check just use the regex validator which is part of EXT:form as well.

Actions

Also available in: Atom PDF