Bug #18901

XHTML Validation Problems on Forgot Password Form

Added by Jeff Segars almost 13 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Should have
Category:
felogin
Target version:
-
Start date:
2008-06-03
Due date:
% Done:

100%

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

Description

The forgot password form has a minor XHTML validation problem when using the default template.

Both the label "for" attribute and the input ID are tied to ###FORGOT_EMAIL### (as is the input "name" attribute"). This marker evaluates as "tx_felogin_pi1[forgot_email]", but the brackets are not allowed in the ID or "for" attribute.

Changing the ID to forgot-email or something like that should clear up the issue, although it may introduce some small compatibility problems for people hooking CSS or Javascript onto the existing ID.

(issue imported from #M8600)

#1

Updated by Jigal van Hemert over 9 years ago

  • Assignee changed from Steffen Kamper to Jigal van Hemert
  • Target version deleted (0)
  • TYPO3 Version set to 4.5
#2

Updated by Jigal van Hemert about 9 years ago

  • Status changed from Accepted to Under Review

A patch is available for review: https://review.typo3.org/7514

(Somehow Gerrit didn't update this issue)

#3

Updated by Gerrit Code Review about 9 years ago

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

#4

Updated by Gerrit Code Review about 9 years ago

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

#5

Updated by Gerrit Code Review about 9 years ago

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

#6

Updated by Gerrit Code Review about 9 years ago

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

#7

Updated by Jigal van Hemert about 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#8

Updated by Gerrit Code Review about 9 years ago

  • Status changed from Resolved to Under Review

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

#9

Updated by Wouter Wolters almost 9 years ago

Merged in 4.7.0rc2

#10

Updated by Ernesto Baschny almost 9 years ago

  • Status changed from Under Review to Closed
  • TYPO3 Version changed from 4.5 to 4.7

Also available in: Atom PDF