Bug #31711: Multiple FORM Elements on the same page - TYPO3 Core - TYPO3 Forge

Actions

Bug #31711

closed

Multiple FORM Elements on the same page

Added by Niki Mueller about 13 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Must have

Assignee:

-

Category:

Form Framework

Target version:

-

Start date:

2011-11-09

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

4.6

PHP Version:

5.3

Tags:

Complexity:

medium

Is Regression:

Sprint Focus:

Description

If you have more than one FORM Element on the same page it is not possible to send all of them. Only the first one works.
If you want to send the second one, for example, the rules for the validation are not resetted.
The Problem is that the instance of the Tx_Form_System_Validate is used as singleton as you can see in the setRules function in the form/Classes/Domain/Factory/Typoscript.php class (Line: 361).
If you clean the rules right after this Line it works with several forms on the same page.
My suggestion is to reset the rules array in the constructor of the Tx_Form_System_Validate class.

Best regards,

Niki Müller

Related issues 1 (0 open — 1 closed)

Actions

#1

Updated by Niki Mueller about 13 years ago

I worked out a better solution for this Bug.
The following patch changes the arrays $rules, $messages and $errors in the Tx_Form_System_Validate class, using the current
form-prefix as key, allowing for validation of several different forms on one page.

Actions

#2

Updated by Mr. Hudson about 13 years ago

Status changed from New to Under Review

Patch set 1 of change If4842a77fd7bfb3da62dbc3d79de01be88c708fd has been pushed to the review server.
It is available at http://review.typo3.org/6624

Actions

#3

Updated by Mr. Hudson about 13 years ago

Patch set 2 of change If4842a77fd7bfb3da62dbc3d79de01be88c708fd has been pushed to the review server.
It is available at http://review.typo3.org/6624

Actions

#4

Updated by Niki Mueller about 13 years ago

Uploaded new Patch-Set because of redundant whitespaces.

Actions

#5

Updated by Mr. Hudson about 13 years ago

Patch set 3 of change If4842a77fd7bfb3da62dbc3d79de01be88c708fd has been pushed to the review server.
It is available at http://review.typo3.org/6624

Actions

#6

Updated by Niki Mueller about 13 years ago

Removed unused function getRules();

Actions

#7

Updated by Mr. Hudson about 13 years ago

Patch set 4 of change If4842a77fd7bfb3da62dbc3d79de01be88c708fd has been pushed to the review server.
It is available at http://review.typo3.org/6624

Actions

#8

Updated by Xavier Perseguers almost 13 years ago

Target version changed from 4.6.1 to 4.6.2

Actions

#9

Updated by Gerrit Code Review almost 13 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6624

Actions

#10

Updated by Gerrit Code Review almost 13 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6624

Actions

#11

Updated by Gerrit Code Review almost 13 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6624

Actions

#12

Updated by Xavier Perseguers almost 13 years ago

Target version deleted (~~4.6.2~~)

Actions

#13

Updated by Gerrit Code Review almost 13 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/8920

Actions

#14

Updated by Niki Mueller almost 13 years ago

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

Applied in changeset 04b87281e596bb73b9e80e63652ceaa49ac3ec4a.

Actions

#15

Updated by Benni Mack about 6 years ago

Status changed from Resolved to Closed

Actions

Also available in: Atom PDF