Project

General

Profile

Actions

Bug #5480

closed

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

Added by Pierre TEISSONNIERE over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Category:
Administration
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 #1

Updated by Pierre TEISSONNIERE over 7 years ago

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

Actions #3

Updated by Miguel Moquillon over 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 #4

Updated by Miguel Moquillon over 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 #5

Updated by Pierre TEISSONNIERE over 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 #6

Updated by Pierre TEISSONNIERE over 7 years ago

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

Updated by Miguel Moquillon over 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF