Project

General

Profile

Bug #7261

Username is incorrect (profile settings)

Added by Jeantina Jansen almost 5 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Category:
Users/Groups
Target version:
Start date:
08/09/2014
Due date:
% Done:

100%

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

Description

When the user wants to edit the password or add a photo at 'Profile settings', she gets an error message.
It says that the username is incorrect en you only can use letters, numbers and _.-
In the screenshot you can see that she is using her e-mailadress, which means that the @ cannot be used.
But in the Chamilo settings is the option 'Use the email as username' enabled.

What's going wrong?

The Chamilo LMS version is: 1.9.6 but it's also when we use 1.9.8
The URL to this platform is: http://platform.blooming-nl.com/main/auth/profile.php


Files

Screenshot.png (174 KB) Screenshot.png Screenshot Jeantina Jansen, 08/09/2014 11:22
profile.php (31.3 KB) profile.php Yoselyn Castillo, 15/09/2014 17:08

Associated revisions

Revision 85ad24fa (diff)
Added by Yoselyn Castillo almost 5 years ago

Updating field when login is email -refs #7261

History

#1

Updated by Julio Montoya almost 5 years ago

  • Status changed from New to Assigned
  • Assignee set to Yoselyn Castillo

Hi Yoselyn,

Could you take a look?

#2

Updated by Yoselyn Castillo almost 5 years ago

Checking this..

#3

Updated by Yoselyn Castillo almost 5 years ago

┬ĘBug confirmed. Checking code.

#4

Updated by Yoselyn Castillo almost 5 years ago

  • % Done changed from 0 to 70

Change main/auth/profile.php for which is in this pull request:
https://github.com/chamilo/chamilo-lms/pull/350
Let me know your test result.

#5

Updated by Jeantina Jansen almost 5 years ago

The problem is not solved yet. I get the following error message:

Fatal error: Call to undefined function api_get_password_checker_js() in /var/www/platform.blooming-nl.com/main/auth/profile.php on line 24

Yoselyn Castillo wrote:

Change main/auth/profile.php for which is in this pull request:
https://github.com/chamilo/chamilo-lms/pull/350
Let me know your test result.

#6

Updated by Yoselyn Castillo almost 5 years ago

Please copy this attached file in main/auth/profile.php in your local installation.
Describe me in which scenario the error occurs.

#7

Updated by Jeantina Jansen almost 5 years ago

I still see the same error as in the beginning: The username is incorrect en you only can use letters, numbers and _.-

I have added the user in the userlist and then I log in as that user.
Then I change the Profile settings (password) and that's when I get the error message.

#8

Updated by Yoselyn Castillo almost 5 years ago

But you have tried with that profile.php? I have reproduced your bug, but it is fixed with those changes.

#9

Updated by Jeantina Jansen almost 5 years ago

Yes, I've uploaded the file to my localhost Chamilo LMS 1.9.8.

#10

Updated by Yoselyn Castillo almost 5 years ago

  • Assignee changed from Yoselyn Castillo to Julio Montoya

Julio Could you check the change?. I have try it and for me is solved when applied, but it is not in comment above.

#11

Updated by Yannick Warnier almost 5 years ago

  • Category set to Users/Groups
  • Status changed from Assigned to Needs testing
  • Assignee changed from Julio Montoya to Yannick Warnier
  • Target version set to 1.9.10
  • % Done changed from 70 to 90

Jeantina, you get the error because the api_get_password_checker_js() function has been added to main_api.lib.php in a different (unrelated) task than this one.

Please try to use your own main/auth/profile.php and only apply the changes you can see here: https://github.com/chamilo/chamilo-lms/pull/350/files

This should do it. In the meantime, I have included Yoselyn's change to Chamilo as, together with this other change, it seems to fix the issue.

#12

Updated by Yannick Warnier almost 5 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee changed from Yannick Warnier to Yoselyn Castillo
  • % Done changed from 90 to 100

Marking as resolved. Feel free to report again here after checking with a full update of the code.

#13

Updated by Jeantina Jansen over 4 years ago

Yannick, I've edited profile.php, but it's still not working. This time I've tested this online (chamilolms198.helixworks.eu).
I don't know what you mean with the function api_get_password_checker_js(). Do I need to get some changes in this file?

Also available in: Atom PDF