Project

General

Profile

Actions
Copy link

Bug #18901

closed

XHTML Validation Problems on Forgot Password Form

Added by Jeff Segars over 16 years ago. Updated over 12 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Jigal van Hemert
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)

Actions
Copy link
 #1

Updated by Jigal van Hemert almost 13 years ago

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

Updated by Jigal van Hemert almost 13 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)

Actions
Copy link
 #3

Updated by Gerrit Code Review over 12 years ago

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

Actions
Copy link
 #4

Updated by Gerrit Code Review over 12 years ago

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

Actions
Copy link
 #5

Updated by Gerrit Code Review over 12 years ago

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

Actions
Copy link
 #6

Updated by Gerrit Code Review over 12 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

Actions
Copy link
 #7

Updated by Jigal van Hemert over 12 years ago

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

Updated by Gerrit Code Review over 12 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

Actions
Copy link
 #9

Updated by Wouter Wolters over 12 years ago

Merged in 4.7.0rc2

Actions
Copy link
 #10

Updated by Ernesto Baschny over 12 years ago

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

Also available in: Atom PDF