Project

General

Profile

Actions
Copy link

Bug #5480

closed

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

Added by Pierre TEISSONNIERE about 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Pierre TEISSONNIERE
Category:
Administration
Target version:
Silverpeas - Version 5.13.5
Start date:
04/16/2014
Due date:
% Done:

100%

Estimated time:
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 ?

Actions
Copy link
 #1

Updated by Pierre TEISSONNIERE about 7 years ago

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

Actions
Copy link
 #3

Updated by Miguel Moquillon about 7 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

Actions
Copy link
 #4

Updated by Miguel Moquillon about 7 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

Actions
Copy link
 #5

Updated by Pierre TEISSONNIERE almost 7 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".

Actions
Copy link
 #6

Updated by Pierre TEISSONNIERE almost 7 years ago

  • Status changed from Re-opened to Resolved
  • % Done changed from 90 to 100
Actions
Copy link
 #7

Updated by Miguel Moquillon almost 7 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF