Project

General

Profile

Actions

Bug #22127

closed

FORM: validateForm fails if a label contains a comma

Added by Daniel Nienhaus about 14 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2010-02-16
Due date:
% Done:

0%

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

Description

If a standard-mailform contains several required fields with commas in the label validateForms() in t3lib/jsfunc.validateform.js is not able to process them all or display the correct error-message.
This is due to a change in the FORM method in typo3/sysext/cms/tslib/class.tslib_content.php. Up to and including Version 4.2.9 the parameters ($fieldlist-array) for validateForms() where passed through rawurlencode(), now the raw data is passed, including unencoded commas. validateForms() uses commas to separate the field to check from the field-label to display on error, and does not like unencoded commas in the label.

(issue imported from #M13556)


Files

patch13556.diff (2.02 KB) patch13556.diff Administrator Admin, 2010-05-21 13:15
Actions #1

Updated by Georg Nebel almost 14 years ago

A possible solution might be to use '|' instead of comma as a separator.

Actions #2

Updated by Chris topher almost 14 years ago

Hi Georg,

I just noticed your patch, because I read the Bugs-Newsgroup.
That was just luck.

When you have a patch for an issue, you should instead post it as an RFC to Core List. Then it will be integrated in the next releases.
Check http://typo3.org/teams/core/core-mailinglist-rules

Actions #3

Updated by Georg Nebel almost 14 years ago

Hi Christopher,
Thanks for the info.

Patch posted as RFC.

Actions #4

Updated by Susanne Moog over 12 years ago

  • Status changed from Needs Feedback to Accepted
  • Target version deleted (0)
  • TYPO3 Version set to 4.4
Actions #5

Updated by Maik Matthias about 11 years ago

This bug still exists in Version 4.5.x and is very annoying.

Actions #6

Updated by Mathias Schreiber over 9 years ago

  • Status changed from Accepted to Closed
  • Is Regression set to No
Actions

Also available in: Atom PDF