Feature #34597

Disable forgot password forms

Added by Marco Huber almost 8 years ago. Updated about 1 year ago.

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

100%

PHP Version:
Tags:
Complexity:
no-brainer
Sprint Focus:

Description

If there are two login forms on a page (usecases described here: http://forge.typo3.org/issues/26910) the forgot feature does not work because the last generated forgot_hash overwrites all previous generated hashes in the session. This is already discussed here http://forge.typo3.org/issues/26910 and I aggree that "it's nonsense and a sign of bad design if there are two forgot password forms on the same page" (http://forge.typo3.org/issues/26910#note-5).

So it would be nice to disable the forgot feature for one of the two login forms, perhaps by introducing two new Typoscript options "disableForgot" and "disableChangePassword" which simply disable the execution of the showForgot and changePassword functions.

I will create a patch for this later and send it to gerrit...


Related issues

Related to TYPO3 Core - Bug #26910: forgot does not work with more than one login form on one page New 2011-05-19
Related to TYPO3 Core - Feature #50138: Show forgot password if conf showForgotPassword = 1 Closed 2013-07-18

Associated revisions

Revision bd32759e (diff)
Added by Nicolas Forgeot about 6 years ago

[FEATURE] Show forgot password only if conf showForgotPassword = 1

Do not show forgot password form in felogin, if the forgot
password link for the login form has been disabled.

Resolves: #50138
Resolves: #34597
Releases: 6.2
Change-Id: I40b490cb09dc339067ff4ff0bc0d0afe0464a456
Reviewed-on: https://review.typo3.org/22541
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Marco Huber
Tested-by: Marco Huber
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

History

#1 Updated by Wouter Wolters over 6 years ago

  • Status changed from New to Needs Feedback

Seems this request is related to #50138 ? If this is correct can you test the patch that is pending?

#2 Updated by Gerrit Code Review about 6 years ago

  • Status changed from Needs Feedback to Under Review

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

#3 Updated by Nicolas Forgeot about 6 years ago

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

#4 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF