Implementada comprobación de NIFs duplicados para pacientes y todos los

médicos.

1.sources/MyHealth/docroot/header.xhtml

@@ -33,7 +33,7 @@
 		</div>
 		<hr />
 		<div id="menuDiv">
-			<p:growl id="messages" sticky="false" showDetail="true" life="15000" />
+			<p:growl id="messages" globalOnly="true" sticky="false" showDetail="true" life="15000" />
 			<p:ajaxStatus style="width:32px; height:32px; position:fixed; right:32px; bottom:32px">
 				<f:facet name="start">
 					<i id="loginSpin" class="pi pi-spin pi-spinner" style="font-size: 3em"></i>

1.sources/MyHealth/docroot/profile/RegisterUser.xhtml

@@ -61,7 +61,7 @@
 				        */
 				    }
 				</h:outputScript>
-				<p:messages id="mesgs" showDetail="true" closable="true" autoupdate="true" />
+				<p:messages id="mesgs" globalOnly="true" showDetail="true" closable="true" autoupdate="true" />
 				<p:panel id="tipoUsuario" header="Especifique el tipo de usuario que desea registrarse">
 					<div class="ui-g ui-fluid">
 						<div class="ui-g-2 ui-md-2">
@@ -83,10 +83,12 @@
 							<p:outputLabel value="NIF:" for="nif" />
 						</div>
 						<div class="ui-g-4 ui-md-4">
-							<p:inputText id="nif" value="#{RegisterUser.nif}" validator="nifValidator" maxlength="20" required="true" requiredMessage="Por favor, indque su NIF" />
+							<p:inputText id="nif" value="#{RegisterUser.nif}" validator="nifValidator" maxlength="20" required="true" requiredMessage="Por favor, indque su NIF">
+								<p:ajax event="blur" update="nifmsg" listener="#{RegisterUser.hadleNIFValueChange}" />
+							</p:inputText>
 						</div>
 						<div class="ui-g-6 ui-md-6">
-							<p:message for="nif" display="text" showDetail="true" showSummary="true" />
+							<p:message id="nifmsg" for="nif" />
 						</div>
 
 						<div class="ui-g-2 ui-md-2">
@@ -114,12 +116,12 @@
 						</div>
 						<div class="ui-g-4 ui-md-4">
 							<p:inputText id="email" value="#{RegisterUser.email}" required="true" maxlength="120" validator="emailValidator"
-								validatorMessage="La dirección #{RegisterUser.email} no es válida" requiredMessage="Por favor, especifique su correo electrónico">
+								validatorMessage="La dirección de correo electrónico #{RegisterUser.email} no es válida" requiredMessage="Por favor, especifique su correo electrónico">
 								<f:validateRegex pattern="[\w\.-]*[a-zA-Z0-9_]@[\w\.-]*[a-zA-Z0-9]\.[a-zA-Z][a-zA-Z\.]*[a-zA-Z]" />
 							</p:inputText>
 						</div>
 						<div class="ui-g-6 ui-md-6">
-							<p:message for="email" display="text" showDetail="true" showSummary="true" />
+							<p:message for="email" />
 						</div>
 						<div class="ui-g-2 ui-md-2">
 							<p:outputLabel value="Contraseña" for="password" />
@@ -148,8 +150,8 @@
 									<p:outputLabel value="Centro:" for="selPHC" />
 								</div>
 								<div class="ui-g-4 ui-md-4">
-									<p:autoComplete id="selPHC" dropdown="true" value="#{RegisterUser.primaryHealthCareCenter}" completeMethod="#{RegisterUser.completePrimaryHealCareCenter}" var="phc"
+									<p:autoComplete id="selPHC" dropdown="true" required="true" value="#{RegisterUser.primaryHealthCareCenter}" completeMethod="#{RegisterUser.completePrimaryHealCareCenter}"
-										itemLabel="#{phc.displayName}" itemValue="#{phc}" forceSelection="true" requiredMessage="Por favor, selecciona un nuevo centro de antención primaria">
+										var="phc" itemLabel="#{phc.displayName}" itemValue="#{phc}" forceSelection="true" requiredMessage="Por favor, selecciona un nuevo centro de antención primaria">
 										<o:converter converterId="omnifaces.ListConverter" list="#{RegisterUser.phcList}" />
 										<p:column headerText="Nombre">
 											<h:outputText value="#{phc.name}" />
@@ -169,7 +171,7 @@
 									<p:outputLabel value="Especialidad médica:" for="selMS" />
 								</div>
 								<div class="ui-g-4 ui-md-4">
-									<p:autoComplete id="selMS" dropdown="true" value="#{RegisterUser.medicalSpecialty}" completeMethod="#{RegisterUser.completeMedicalSpecialty}" var="ms"
+									<p:autoComplete id="selMS" dropdown="true" required="true" value="#{RegisterUser.medicalSpecialty}" completeMethod="#{RegisterUser.completeMedicalSpecialty}" var="ms"
 										itemLabel="#{ms.displayName}" itemValue="#{ms}" forceSelection="true" requiredMessage="Por favor, selecciona una especialidad médica">
 										<o:converter converterId="omnifaces.ListConverter" list="#{RegisterUser.medicalSpecialtiesList}" />
 										<p:column headerText="Nombre">

1.sources/MyHealth/docroot/profile/UpdateProfile.xhtml

@@ -41,10 +41,12 @@
 						<p:outputLabel value="NIF:" for="nif" />
 					</div>
 					<div class="ui-g-4 ui-md-4">
-						<p:inputText id="nif" value="#{UpdateProfile.nif}" validator="nifValidator" required="true"  maxlength="50" requiredMessage="Por favor, indque su NIF" />
+						<p:inputText id="nif" value="#{UpdateProfile.nif}" validator="nifValidator" required="true" maxlength="50" requiredMessage="Por favor, indque su NIF">
+							<p:ajax event="blur" update="nifmsg" listener="#{UpdateProfile.hadleNIFValueChange}" />
+						</p:inputText>
 					</div>
 					<div class="ui-g-6 ui-md-6">
-						<p:message for="nif" display="text" />
+						<p:message id="nifmsg" for="nif" display="text" />
 					</div>
 
 					<div class="ui-g-2 ui-md-2">
@@ -71,8 +73,8 @@
 						<p:outputLabel value="Correo electrónico:" for="email" />
 					</div>
 					<div class="ui-g-4 ui-md-4">
-						<p:inputText id="email" value="#{UpdateProfile.email}" required="true" maxlength="120" validator="emailValidator" validatorMessage="La dirección #{RegisterUser.email} no es válida"
+						<p:inputText id="email" value="#{UpdateProfile.email}" required="true" maxlength="120" validator="emailValidator"
-							requiredMessage="Por favor, especifique su correo electrónico">
+							validatorMessage="La dirección #{RegisterUser.email} no es válida" requiredMessage="Por favor, especifique su correo electrónico">
 							<f:validateRegex pattern="[\w\.-]*[a-zA-Z0-9_]@[\w\.-]*[a-zA-Z0-9]\.[a-zA-Z][a-zA-Z\.]*[a-zA-Z]" />
 						</p:inputText>
 					</div>
@@ -103,7 +105,7 @@
 						<p:outputLabel value="Verificación de contraseña:" for="passwordRepeat" />
 					</div>
 					<div class="ui-g-4">
-						<p:password id="passwordRepeat" value="#{UpdateProfile.password}" required="false"  maxlength="100" requiredMessage="Por favor, escriba la verificación de su contraseña" />
+						<p:password id="passwordRepeat" value="#{UpdateProfile.password}" required="false" maxlength="100" requiredMessage="Por favor, escriba la verificación de su contraseña" />
 					</div>
 					<div class="ui-g-6">
 						<p:message for="passwordRepeat" />

1.sources/MyHealth/src/ejb/systemAdmin/SystemAdminFacadeBean.java

@@ -5,15 +5,15 @@ import javax.ejb.Stateless;
 import javax.persistence.EntityManager;
 import javax.persistence.PersistenceContext;
 
+import TO.FamilyDoctorTO;
 import TO.LoggedUserTO;
+import TO.PatientTO;
+import TO.SpecialistDoctorTO;
 import common.Constants;
 import common.HashUtils;
 import common.UserType;
 import ejb.common.CommonFacadeLocal;
 import jpa.AdministratorJPA;
-import jpa.FamilyDoctorJPA;
-import jpa.PatientJPA;
-import jpa.SpecialistDoctorJPA;
 
 /**
  * 
@@ -61,20 +61,20 @@ public class SystemAdminFacadeBean implements SystemAdminFacadeRemote {
 		if (userCode.startsWith(Constants.PERSONAL_IDENTIFICATION_CODE_PREFIX)) {
 			// Si el identificador de usuario es de tipo paciente, intentamos realizar el
 			// login.
-			PatientJPA pat = this.commonServices.findPatientByCode(userCode);
+			PatientTO pat = this.commonServices.findPatientByCode(userCode);
 			if (pat != null) {
 				usr = new LoggedUserTO(String.valueOf(pat.getId()), pat.getName(), pat.getPassword(), UserType.PATIENT);
 			}
 		} else if (userCode.startsWith(Constants.PROFESSIONAL_NUMBER_PREFIX)) {
 			// Si el identificador de usuario es de tipo profesional, intentamos realizar el
 			// login, primero como médico de familia, después como especialista
-			FamilyDoctorJPA fd = this.commonServices.findFamilyDoctorByCode(userCode);
+			FamilyDoctorTO fd = this.commonServices.findFamilyDoctorByCode(userCode);
 
 			if (fd != null) {
 				usr = new LoggedUserTO(String.valueOf(fd.getId()), fd.getName(), fd.getPassword(), UserType.FAMILY_DOCTOR);
 			} else {
 				// No era un código de médico de familia, intenamos logearlo como especialista
-				SpecialistDoctorJPA sd = this.commonServices.findSpecialistDoctorByCode(userCode);
+				SpecialistDoctorTO sd = this.commonServices.findSpecialistDoctorByCode(userCode);
 
 				if (sd != null) {
 					usr = new LoggedUserTO(String.valueOf(sd.getId()), sd.getName(), sd.getPassword(), UserType.SPECIALIST_DOCTOR);

1.sources/MyHealth/src/managedbean/common/ManagedBeanBase.java

@@ -95,25 +95,29 @@ public class ManagedBeanBase {
 	protected void addFacesMessageKeep(FacesMessage.Severity severity, String summary, String detail) {
 		FacesContext context = FacesContext.getCurrentInstance();
 
-		this.addFacesMessage(FacesContext.getCurrentInstance(), severity, summary, detail);
+		this.addFacesMessage(FacesContext.getCurrentInstance(), null, severity, summary, detail);
 
 		context.getExternalContext().getFlash().setKeepMessages(true);
 	}
 
 	protected void addFacesMessage(FacesMessage.Severity severity, String summary, String detail) {
-		this.addFacesMessage(FacesContext.getCurrentInstance(), severity, summary, detail);
+		this.addFacesMessage(FacesContext.getCurrentInstance(), null, severity, summary, detail);
 	}
 	
-	protected void addFacesMessage(FacesContext context, FacesMessage.Severity severity, String summary, String detail) {
+	protected void addFacesMessage(String clientId, FacesMessage.Severity severity, String summary, String detail) {
-		this.addFacesMessage(context, new FacesMessage(severity, summary, detail));
+		this.addFacesMessage(FacesContext.getCurrentInstance(), clientId, severity, summary, detail);
 	}
 
-	protected void addFacesMessage(FacesMessage facesMsg) {
+	protected void addFacesMessage(FacesContext context, String clientId, FacesMessage.Severity severity, String summary, String detail) {
-		this.addFacesMessage(FacesContext.getCurrentInstance(), facesMsg);
+		this.addFacesMessage(context, clientId, new FacesMessage(severity, summary, detail));
 	}
 	
-	protected void addFacesMessage(FacesContext context, FacesMessage facesMsg) {
+	protected void addFacesMessage(FacesMessage facesMsg, String clientId) {
-		context.addMessage(null, facesMsg);
+		this.addFacesMessage(FacesContext.getCurrentInstance(), clientId, facesMsg);
+	}
+	
+	protected void addFacesMessage(FacesContext context, String clientId, FacesMessage facesMsg) {
+		context.addMessage(clientId, facesMsg);
 	}
 
 	protected void manageException(Exception ex) {

1.sources/MyHealth/src/managedbean/common/ValidationUtils.java

@@ -1,5 +1,11 @@
 package managedbean.common;
 
+import TO.FamilyDoctorTO;
+import TO.PatientTO;
+import TO.SpecialistDoctorTO;
+import common.UserType;
+import ejb.common.CommonFacadeRemote;
+
 /***
  * 
  * @author Marcos García Núñez (mgarcianun@uoc.edu)
@@ -47,4 +53,51 @@ public class ValidationUtils {
 		return false;
 	}
 
+	/**
+	 * Comprueba si un NIF existe para un tipo de usuario. * No se permite que el
+	 * mismo NIF esté registrado como médico de familia y como especialista. * Se
+	 * permite que el mismo NIF esté registrado como paciente y como médico (de
+	 * familia o especialista)
+	 * 
+	 * @param remoteSvc Servicio Remoto para recuperar datos.
+	 * @param userType  Tipo de usuario a comprobar
+	 * @param nif       NIF a comprobar.
+	 * @param id        Parámetro opcional, id del usuario actual (si encuentra el
+	 *                  mismo nif y el usuario coincide no se devuelve la
+	 *                  coincidencia)
+	 * @return true si el NIF ya estaba registrado para un usuario diferente.
+	 */
+	public static boolean checkIfNifAlreadyRegistered(CommonFacadeRemote remoteSvc, UserType userType, String nif, Integer id) {
+		boolean nifExists = false;
+
+		switch (userType) {
+		case ADMINISTRATOR:
+			break;
+		case FAMILY_DOCTOR:
+		case SPECIALIST_DOCTOR:
+			FamilyDoctorTO fd = remoteSvc.findFamilyDoctorByNif(nif);
+
+			if (fd == null) {
+				// Si el nif no está en uso para un médico de familia, reivsamos si está en uso
+				// para los especialistas
+				SpecialistDoctorTO sd = remoteSvc.findSpecialistDoctorByNif(nif);
+
+				if (sd != null && (id == null || fd.getId() != id))
+					nifExists = true;
+			} else if (id == null || fd.getId() != id)
+				// Si se trata de un usuario diferente, entonces está repetido
+				nifExists = true;
+
+			break;
+		case PATIENT:
+			PatientTO pat = remoteSvc.findPatientByNif(nif);
+
+			// Si se trata de un usuario diferente, entonces está repetido
+			if (pat != null && (id == null || pat.getId() != id))
+				nifExists = true;
+			break;
+		}
+
+		return nifExists;
+	}
 }

1.sources/MyHealth/src/managedbean/common/homeMBean.java

@@ -36,7 +36,7 @@ public class homeMBean extends ManagedBeanBase implements Serializable {
 		FacesMessage message = SessionUtils.getMessage();
 
 		if (message != null) {
-			this.addFacesMessage(message);
+			this.addFacesMessage(message, null);
 		}
 	}
 

1.sources/MyHealth/src/managedbean/profile/RegisterUserMBean.java

@@ -128,6 +128,11 @@ public class RegisterUserMBean extends ManagedBeanBase implements Serializable {
 		return this.medicalSpecialitiesList;
 	}
 	
+	public void hadleNIFValueChange() {
+		if (ValidationUtils.checkIfNifAlreadyRegistered(this.getRemoteManagerCommon(), this.userType, this.nif, null) == true)
+			this.addFacesMessage("frmRegisterUser:nif", FacesMessage.SEVERITY_WARN, "NIF duplicado", "El nif indicado pertenece a otro usuario previamente registrado");
+	}
+
 	public boolean isPatient() {
 		return (this.userType == UserType.PATIENT);
 	}
@@ -199,6 +204,10 @@ public class RegisterUserMBean extends ManagedBeanBase implements Serializable {
 			this.addFacesMessage(FacesMessage.SEVERITY_WARN, "El NIF indicado no es válido", "Por favor, especifique un NIF válido.");
 			error++;
 		}
+		if (ValidationUtils.checkIfNifAlreadyRegistered(this.getRemoteManagerCommon(), this.userType, this.nif, null) == true) {
+			this.addFacesMessage("frmRegisterUser:nif", FacesMessage.SEVERITY_WARN, "NIF duplicado", "El nif indicado pertenece a otro usuario previamente registrado");
+			error++;
+		}
 
 		if (error == 0) {
 			try {

1.sources/MyHealth/src/managedbean/profile/UpdateProfileMBean.java

@@ -172,6 +172,11 @@ public class UpdateProfileMBean extends ManagedBeanBase implements Serializable
 		return this.medicalSpecialitiesList;
 	}
 
+	public void hadleNIFValueChange() {
+		if (ValidationUtils.checkIfNifAlreadyRegistered(this.getRemoteManagerCommon(), this.userType, this.nif, this.id) == true)
+			this.addFacesMessage("frmUpdateProfile:nif", FacesMessage.SEVERITY_WARN, "NIF duplicado", "El nif indicado pertenece a otro usuario previamente registrado");
+	}
+	
 	public List<FamilyDoctorTO> getFamilyDoctorList() {
 		return familyDoctorList;
 	}
@@ -261,6 +266,10 @@ public class UpdateProfileMBean extends ManagedBeanBase implements Serializable
 			this.addFacesMessage(FacesMessage.SEVERITY_WARN, "El NIF indicado no es válido", "Por favor, especifique un NIF válido.");
 			error++;
 		}
+		if (ValidationUtils.checkIfNifAlreadyRegistered(this.getRemoteManagerCommon(), this.userType, this.nif, this.id) == true) {
+			this.addFacesMessage("frmUpdateProfile:nif", FacesMessage.SEVERITY_WARN, "NIF duplicado", "El nif indicado pertenece a otro usuario previamente registrado");
+			error++;
+		}		
 		if (changePassword == true) {
 			// el usuario queire cambiar el password Comprobamos que el password
 			// especificado coincide con el guardado