Project

General

Profile

Actions
Copy link

Bug #24733

closed

Label with comma in required form fields causes problems in popup

Added by Jochen Weiland almost 14 years ago. Updated over 9 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
Copy link
 #1

Updated by Georg Ringer almost 14 years ago

possible security issue

Actions
Copy link
 #2

Updated by Helmut Hummel over 13 years ago

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

Updated by Christian Kuhn about 13 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
Copy link
 #4

Updated by Marcus Krause almost 11 years ago

  • Project changed from 1716 to TYPO3 Core

no longer a security issue - only bug

Actions
Copy link
 #5

Updated by Mathias Schreiber almost 10 years ago

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

Updated by Benni Mack over 9 years ago

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

Updated by Björn Jacob over 9 years ago

  • Category changed from Form Framework to 1602
Actions
Copy link
 #8

Updated by Björn Jacob over 9 years ago

  • Category changed from 1602 to Form Framework
Actions
Copy link
 #9

Updated by Susanne Moog over 9 years ago

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

Updated by Björn Jacob over 9 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
Copy link
 #11

Updated by Jochen Weiland over 9 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
Copy link
 #12

Updated by Björn Jacob over 9 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
Copy link

Also available in: Atom PDF