Project

General

Profile

Actions

Feature #50138

closed

Show forgot password if conf showForgotPassword = 1

Added by Nicolas Forgeot over 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
felogin
Target version:
-
Start date:
2013-07-18
Due date:
% Done:

100%

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

Description

Sometimes 2 plugins felogin are in the same page : one in the header and the other in content's page of managing account.

If user uses the link forgot password, the 2 plugins show the form. Even if conf showForgotPassword is false.

It would be good to do a test before showing the form.

\typo3\sysext\felogin\Classes\Controller\FrontendLoginController.php l171

if ($this->piVars['forgot']) {
    $content .= $this->showForgot();
}

To :

if ($this->piVars['forgot'] && $this->conf['showForgotPassword']) {
    $content .= $this->showForgot();
}


Files

2plugins.png (66 KB) 2plugins.png Nicolas Forgeot, 2013-07-18 09:54
2plugins-forms.png (78.6 KB) 2plugins-forms.png Nicolas Forgeot, 2013-07-18 09:54
2plugins-with_patch.png (68.1 KB) 2plugins-with_patch.png Nicolas Forgeot, 2013-07-18 09:54

Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Feature #34597: Disable forgot password formsClosed2012-03-06

Actions
Related to TYPO3 Core - Bug #59821: link to forgot password page does'nt work in 6.2Closed2014-06-23

Actions
Has duplicate TYPO3 Core - Bug #18868: felogin: forgot-password email sent twiceClosed2008-05-29

Actions
Actions #1

Updated by Alexander Opitz over 10 years ago

  • Target version deleted (next-patchlevel)

I would like to mark this as duplicate of #34597

Can you make a patch and send it to gerrit?

Actions #2

Updated by Alexander Opitz over 10 years ago

  • % Done changed from 50 to 0
Actions #3

Updated by Nicolas Forgeot over 10 years ago

I hope it's good place / name for the patch : https://review.typo3.org/22535

Actions #4

Updated by Gerrit Code Review over 10 years ago

  • Status changed from New to Under Review

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

Actions #5

Updated by Gerrit Code Review over 10 years ago

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

Actions #6

Updated by Gerrit Code Review over 10 years ago

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

Actions #7

Updated by Nicolas Forgeot over 10 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF