Bug #5480
closedDomaine de création des utilisateurs dans le cas de l'autoregistration
100%
Description
Je préfère le dire dès le début, j'ai trouvé ça en regardant le code. Modifier le code est une chose mais je ne suis pas encore un pro de Git. Je vous laisse voir si ce que je dis est correct.
Dans RegisterHandler.java, ligne 50 :
service.registerUser(firstName, lastName, email, "0");
Le domainId est donc fixé à 0 et non pas récupéré de authenticationSettings.properties - justRegisteredDomainId
Il faudrait ajouter à RegistrationSettings.java :
private static String SELF_AUTHENTICATION_DOMAIN = "justRegisteredDomainId"; public String userSelfRegistrationDomainId() { return settings.getString(SELF_AUTHENTICATION_DOMAIN,"0"); }
et à RegisterHandler.java :
Ajouter
String domainId = settings.userSelfRegistrationDomainId();
et modifier la ligne 50 :
service.registerUser(firstName, lastName, email, domainId);
Est-ce que ça vous convient ?
Updated by Pierre TEISSONNIERE about 11 years ago
Il serait peut être mieux de mettre le nom de variable SELF_AUTHENTICATION_DOMAINID au lieu de SELF_AUTHENTICATION_DOMAIN
Updated by Pierre TEISSONNIERE about 11 years ago
ça doit être ça :
https://github.com/Silverpeas/Silverpeas-Core/pull/495
Updated by Miguel Moquillon about 11 years ago
- Status changed from New to Resolved
- Assignee set to Pierre TEISSONNIERE
- % Done changed from 0 to 100
Le bogue a été corrigée par Pierre. En cours d'intégration
Updated by Miguel Moquillon about 11 years ago
- Project changed from 46 to Silverpeas Core
- Category set to Administration
- Status changed from Resolved to Closed
- Target version set to Version 5.13.5
Validé et intégré en vue des versions correctives 5.13.5 et 5.14.1
Updated by Pierre TEISSONNIERE about 11 years ago
- Status changed from Closed to Re-opened
- % Done changed from 100 to 90
Lorsque l'utilisateur s'autoenregistre à partir d'un réseau social, la paramètre justRegisteredDomainId n'est pas pris en compte mais forcé à "0".
Updated by Pierre TEISSONNIERE about 11 years ago
- Status changed from Re-opened to Resolved
- % Done changed from 90 to 100
Traité par PR :
https://github.com/Silverpeas/Silverpeas-Core/pull/498
Updated by Miguel Moquillon about 11 years ago
- Status changed from Resolved to Closed