Project

General

Profile

Bug #1956

Username and email required at password reminder

Added by Jose Manuel Abuin Mosquera about 9 years ago. Updated almost 9 years ago.

Status:
Rejected - Abandoned
Priority:
Normal
Category:
-
Target version:
Start date:
08/09/2010
Due date:
% Done:

0%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

In our post at the chamilo forum http://www.chamilo.org/es/node/725 we said that there is no need to ask for both username and email at the password reminder, we think that just the email is needed and, as Yannick told us, we report it as a bug.

Our chamilo version is 1.8.7.1 on Debian lenny


Related issues

Is duplicate of Chamilo LMS - Bug #1754: "Forgot your password ?" page has a logical problemBug resolved23/07/2010

Actions

Associated revisions

Revision a285f485 (diff)
Added by Yannick Warnier over 2 years ago

Add note about text-to-speech being broken in 1.11.4 - refs #1956

Revision 4970a875 (diff)
Added by Yannick Warnier almost 2 years ago

Minor - Add comment about Google Translate API key for text2speech feature - refs #1956

Revision 6063fb05 (diff)
Added by Yannick Warnier almost 2 years ago

Remove Pediaphon Text2Speech feature + add message to advise about Google API setting requirement - refs #1956

History

#1

Updated by Jose Manuel Abuin Mosquera about 9 years ago

I forgot to write that we solved this "bug" by commentig lines 48,55 and adding line $form->addRule('email', '<div class="required">'.get_lang('ThisFieldIsRequired'), 'required'); after line 55.

We also changed the query at line 69 just like this:
$query = " SELECT user_id AS uid, lastname AS lastName, firstname AS firstName,
username AS loginName, password, email, status AS status,
official_code, phone, picture_uri, creator_id
FROM ".$tbl_user."
WHERE ( email = '".Database::escape_string($email)."') ";

This works fine for us.

#2

Updated by Yannick Warnier almost 9 years ago

  • Target version set to 1.8.8 alpha

Thanks for that. We'll be looking at it soon.

#3

Updated by Marco Villegas almost 9 years ago

  • Assignee set to Marco Villegas

taking a look

#4

Updated by Marco Villegas almost 9 years ago

  • Status changed from New to Rejected - Abandoned

Also available in: Atom PDF