Projet

Général

Profil

Actions

Bug #5480

fermé

Domaine de création des utilisateurs dans le cas de l'autoregistration

Ajouté par Pierre TEISSONNIERE il y a plus de 10 ans. Mis à jour il y a plus de 10 ans.

Statut:
Closed
Priorité:
Normal
Catégorie:
Administration
Début:
16/04/2014
Echéance:
% réalisé:

100%

Temps estimé:
Navigateur:
Tous
Votre version de Silverpeas:
5.13.4
Système d'exploitation:
Votre base de données:
Toutes
Livraison en TEST:
Livraison en PROD:

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 ?

Mis à jour par Pierre TEISSONNIERE il y a plus de 10 ans

Il serait peut être mieux de mettre le nom de variable SELF_AUTHENTICATION_DOMAINID au lieu de SELF_AUTHENTICATION_DOMAIN

Mis à jour par Miguel Moquillon il y a plus de 10 ans

  • Statut changé de New à Resolved
  • Assigné à mis à Pierre TEISSONNIERE
  • % réalisé changé de 0 à 100

Le bogue a été corrigée par Pierre. En cours d'intégration

Mis à jour par Miguel Moquillon il y a plus de 10 ans

  • Projet changé de 46 à Silverpeas Core
  • Catégorie mis à Administration
  • Statut changé de Resolved à Closed
  • Version cible mis à Version 5.13.5

Validé et intégré en vue des versions correctives 5.13.5 et 5.14.1

Mis à jour par Pierre TEISSONNIERE il y a plus de 10 ans

  • Statut changé de Closed à Re-opened
  • % réalisé changé de 100 à 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".

Mis à jour par Pierre TEISSONNIERE il y a plus de 10 ans

  • Statut changé de Re-opened à Resolved
  • % réalisé changé de 90 à 100

Mis à jour par Miguel Moquillon il y a plus de 10 ans

  • Statut changé de Resolved à Closed
Actions

Formats disponibles : Atom PDF