Project

General

Profile

Actions

Bug #2967

closed

CB 1.7.1 Lost username request generates password reset process

Added by nant about 13 years ago. Updated almost 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
08 November 2011
Due date:
% Done:

100%

Estimated time:
1:00 h


Files

2967.patch (805 Bytes) 2967.patch krileon, 02 February 2012 17:52
Actions #1

Updated by nant about 13 years ago

Note: A side issue is that the module seems to use a different language string:

Beat: no, the string exists, and issue in login module has been described in original bug. Just add URLs to the bug report
Beat: this is not an important enough reason for 1.7.2 release
Nicholas G. Antimisiaris: url already in the bug
Nicholas G. Antimisiaris: what is the language string for that case?
Beat: _UE_USERNAME_OR_EMAIL
Beat: he wrote it :)
Beat: string exists, just it's the wrong one which is displayed
Nicholas G. Antimisiaris: ah so bug also makes system not use this language string?
Beat: you can make it a separate issue, but it will just add overhead in documenting fixes
Nicholas G. Antimisiaris: issue 1 - wrong language string is displayed/used
issue 2 - password is sent in case of username reminder
Beat: 2 fixes are needed by the first bug report. One in cb and one in cb login module, that's already clearly described
Beat: opening 2nd ticket will flag it as duplicate
Nicholas G. Antimisiaris: ok - will add a note to ticket just in case
Beat: and just add work
Beat: thanks

Actions #2

Updated by krileon almost 13 years ago

  • File 2967.patch 2967.patch added
  • Status changed from New to Resolved
  • Assignee set to beat
  • % Done changed from 0 to 100

The $usernameExists in comprofiler.php didn't match what was in comprofiler.html.php. Fixed $usernameExists in comprofiler.php and now works as it should.

Actions #3

Updated by beat almost 13 years ago

  • Target version set to CB 1.8
Actions #4

Updated by beat almost 13 years ago

  • Status changed from Resolved to Closed
  • Estimated time set to 1:00 h

Fixed in r1723 as suggested. Thank you Kyle.

Actions

Also available in: Atom PDF