YesWiki / yeswiki

YesWiki is a wiki system written in PHP, including extensions making collaboration more simple (databases, maps, easy editing, bootstrap themes,...).
https://yeswiki.net
GNU Affero General Public License v3.0
107 stars 55 forks source link

When you create a user in YesWiki, a password is created but you do n… #1178

Closed GayLaurent closed 1 month ago

GayLaurent commented 2 months ago

…ot have the means to simply notify the user.

Description of pull request / Description de la demande d'ajout

Implement issue #1177

mrflos commented 2 months ago

si je checkout la branche et me rend sur la page wiki ParametresUtilisateur en étant non connecté, j'obtiens

Une erreur inattendue s'est produite. Veuillez contacter l'administrateur du site et lui communiquer l'erreur suivante :
YesWiki\Core\Service\UserManager::getLastUserLink(): Argument #1 ($user) must be of type YesWiki\Core\Entity\User, null given, called in /Users/mrflos/Developpements/yeswiki/tools/login/actions/UserSettingsAction.php on line 65 in /Users/mrflos/Developpements/yeswiki/includes/services/UserManager.php on line 252
mrflos commented 2 months ago

J'ai corrigé la dernière erreur, mais en voulant créer un user de test par l'admin des users j'ai une erreur 500 :

{"notCreated":["toto"],"error":"YesWiki\\Core\\Controller\\UserController::create(): Return value must be of type ?YesWiki\\Core\\Entity\\User, array returned"}

Je pense peut etre qu'il vaudrait mieux que la methode UserController::create() continue d'envoyer juste un objet User, ca parait plus propre et surtout ca evite de casser des trucs ailleurs. Peut etre juste ajouter le recoverPasswordLink dans l'objet User ?

mrflos commented 1 month ago

@GayLaurent c'est pret a merger a ton avis ?
Pour les considérations d'ux je ferai tester @gatienbataille et @furax37 .