Project

General

Profile

Actions

Bug #24733

closed

Label with comma in required form fields causes problems in popup

Added by Jochen Weiland over 13 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2011-01-23
Due date:
% Done:

0%

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

Description

When using the standard form content element there is a problem with using commas in the label when the option required is set.

When the required fields are not filled out, a popup appears "you must fill out these fields.." with a list of the labels of the unfilled fields. However, this list is aborted after the first comma.

Example:

first name, last name | *name=input,40
E-Mail | *email=input,40
Adresse | address=textarea,40,3

When submitting the empty form the popup just says "first name" and omitting last name, e-mail

This problem has already been in TYPO3 versions prior to 4.5.

(issue imported from #M17223)

Actions #1

Updated by Georg Ringer over 13 years ago

possible security issue

Actions #2

Updated by Helmut Hummel over 12 years ago

  • Category deleted (Communication)
  • Target version deleted (0)
Actions #3

Updated by Christian Kuhn over 12 years ago

The implied security issue will be solved with #25052 ... this issue here is the a usual bug. I suggest to move this issue to core as soon as #25052 was solved and released.

Possible solution: str_replace comma with its html entity.

Actions #4

Updated by Marcus Krause over 10 years ago

  • Project changed from 1716 to TYPO3 Core

no longer a security issue - only bug

Actions #5

Updated by Mathias Schreiber over 9 years ago

  • Category set to Form Framework
  • Target version set to 7.2 (Frontend)
  • Is Regression set to No
Actions #6

Updated by Benni Mack almost 9 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)
Actions #7

Updated by Björn Jacob almost 9 years ago

  • Category changed from Form Framework to 1602
Actions #8

Updated by Björn Jacob almost 9 years ago

  • Category changed from 1602 to Form Framework
Actions #9

Updated by Susanne Moog over 8 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #10

Updated by Björn Jacob over 8 years ago

  • Status changed from New to Needs Feedback
  • Assignee set to Jochen Weiland
  • Target version deleted (7.5)

This issue belongs to the old form wizard which is not part of TYPO3 7.x anymore.

@Jochen Fröhlich: since we've got the new form wizard I think we should concentrate on making this one better. What do you think? Is this issue still important for you? Do you still use the old form wizard?

Actions #11

Updated by Jochen Weiland over 8 years ago

No, this issue is not important (we just stumbled on it during a seminar) and we have not sites that are affected by this bug. So just leave it as is and move forward to version 7.

Best regards
Jochen

Actions #12

Updated by Björn Jacob over 8 years ago

  • Status changed from Needs Feedback to Closed
  • Assignee deleted (Jochen Weiland)

Okay, thanks Jochen. I'm closing this issue and we're concentrating on the new stuff :)

Actions

Also available in: Atom PDF