New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix display name #13

Open

nworr wants to merge 1 commit into jelix:accounts from nworr:fix-display-name

Contributor

nworr commented Aug 22, 2024

Account::getDisplayName() method can return a simple space string if last name and first name are empty

see

authentication-module/modules/account/lib/Account.php

Line 48 in 871b339

return $this->data['firstname'].' '.$this->data['lastname'];

use trim to check if the displayName value is relevant


check if display name don't containe any space

cdd2e4d

nworr changed the base branch from master to accounts

August 22, 2024 14:51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment