#643 Se mejora la validación del formulario de propiedades del ordenador

git-svn-id: https://opengnsys.es/svn/branches/version1.0@4223 a21b9725-9963-47de-94b9-378ad31fedc9

6 files changed, 38 insertions, 24 deletions

diff --git a/admin/WebConsole/idiomas/javascripts/cat/propiedades_ordenadores_cat.js b/admin/WebConsole/idiomas/javascripts/cat/propiedades_ordenadores_cat.js
index e27af69e..a58baa41 100644
--- a/admin/WebConsole/idiomas/javascripts/cat/propiedades_ordenadores_cat.js
+++ b/admin/WebConsole/idiomas/javascripts/cat/propiedades_ordenadores_cat.js
@@ -9,4 +9,5 @@ TbMsg[1]="Debe introducir una dirección IP";
 TbMsg[2]="Debe introducir a una dirección MAC";
 TbMsg[3]="";
 TbMsg[4]="";
-TbMsg[5]="Debe elegir un repositorio por defecto";
\ No newline at end of file
+TbMsg[5]="Debe elegir un repositorio por defecto";
+TbMsg[6]="Deben introducirse datos de validación: página de login y página de validación";
diff --git a/admin/WebConsole/idiomas/javascripts/eng/propiedades_ordenadores_eng.js b/admin/WebConsole/idiomas/javascripts/eng/propiedades_ordenadores_eng.js
index f255316b..a93a0279 100644
--- a/admin/WebConsole/idiomas/javascripts/eng/propiedades_ordenadores_eng.js
+++ b/admin/WebConsole/idiomas/javascripts/eng/propiedades_ordenadores_eng.js
@@ -9,4 +9,5 @@ TbMsg[1]="You must introduce an IP Address";
 TbMsg[2]="You must introduce a MAC Address";
 TbMsg[3]="You must link this computer with some of a hardware profile";
 TbMsg[4]="You must introduce a DHCP server";
-TbMsg[5]="You must introduce a OpenGnsys server";
\ No newline at end of file
+TbMsg[5]="You must introduce a OpenGnsys server";
+TbMsg[6]="Deben introducirse datos de validación: página de login y página de validación";
diff --git a/admin/WebConsole/idiomas/javascripts/esp/propiedades_ordenadores_esp.js b/admin/WebConsole/idiomas/javascripts/esp/propiedades_ordenadores_esp.js
index e27af69e..a58baa41 100644
--- a/admin/WebConsole/idiomas/javascripts/esp/propiedades_ordenadores_esp.js
+++ b/admin/WebConsole/idiomas/javascripts/esp/propiedades_ordenadores_esp.js
@@ -9,4 +9,5 @@ TbMsg[1]="Debe introducir una dirección IP";
 TbMsg[2]="Debe introducir a una dirección MAC";
 TbMsg[3]="";
 TbMsg[4]="";
-TbMsg[5]="Debe elegir un repositorio por defecto";
\ No newline at end of file
+TbMsg[5]="Debe elegir un repositorio por defecto";
+TbMsg[6]="Deben introducirse datos de validación: página de login y página de validación";
diff --git a/admin/WebConsole/jscripts/propiedades_ordenadores.js b/admin/WebConsole/jscripts/propiedades_ordenadores.js
index 22de9adf..21482f8f 100644
--- a/admin/WebConsole/jscripts/propiedades_ordenadores.js
+++ b/admin/WebConsole/jscripts/propiedades_ordenadores.js
@@ -74,26 +74,26 @@ function confirmar(op){
 //	Comprobar_datos 
 //________________________________________________________________________________________________________
 function comprobar_datos(){
-	if (document.fdatos.nombreordenador.value=="") {
-		alert(TbMsg[0]);
-		document.fdatos.nombreordenador.focus();
-		return(false);
+	function validate (field, validator, msgi) {
+		if (!validator (field.value)) {
+			alert(TbMsg[msgi]);
+			validation_highlight (field);
+			return false;
+		}
+	return true;
 	}
-	if (document.fdatos.ip.value=="") {
-		alert(TbMsg[1]);
-		document.fdatos.ip.focus();
-		return(false);
-	}
-	if (document.fdatos.mac.value=="") {
-		alert(TbMsg[2]);
-		document.fdatos.mac.focus();
-		return(false);
-	}
-	var  p=document.fdatos.idrepositorio.selectedIndex
-	 if (p==0){  
-         alert(TbMsg[5])
-         document.forms.fdatos.idrepositorio.focus()
-         return(false)
-	}
-	return(true);
+
+        var form = document.fdatos;
+        // Si se activa la validación, comprobar que se incluyen los datos adecuados.
+        if (form.validacion.options[form.validacion.selectedIndex].value == 1 && (form.paginalogin.value == '' || form.paginavalidacion.value == '')) {
+                alert(TbMsg[6]);
+                validation_highlight (document.fdatos.paginalogin);
+                validation_highlight (document.fdatos.paginavalidacion);
+                return(false);
+        }
+
+	return 	validate (form.nombreordenador, validate_notnull, 0) &&
+		validate (form.ip, validate_ipadress_notnull, 1) &&
+		validate (form.mac, validate_macaddress_notnull, 2) &&
+		validate (form.idrepositorio, validate_number_notnull, 5) ;
 }
diff --git a/admin/WebConsole/jscripts/validators.js b/admin/WebConsole/jscripts/validators.js
index 11ace4dc..b4f40801 100644
--- a/admin/WebConsole/jscripts/validators.js
+++ b/admin/WebConsole/jscripts/validators.js
@@ -60,6 +60,16 @@ function validate_ipadress_notnull(value) {
 	return validate_ipadress(value) && validate_notnull(value);
 }
 
+// Validar direccion MAC
+function validate_macaddress(value) {
+	var regex = '^([0-9a-fA-F]){12}$'
+	return validate_expr(value, regex);
+}
+
+function validate_macaddress_notnull(value) {
+	return validate_macaddress(value) && validate_notnull(value);
+}
+
 // Validar URL.
 function validate_url(value) {
 	var octet = '(25[0-5]|2[0-4][0-9]|1[0-9]{2}|[1-9][0-9]|[0-9])';
diff --git a/admin/WebConsole/propiedades/propiedades_ordenadores.php b/admin/WebConsole/propiedades/propiedades_ordenadores.php
index 300bfd02..8ba8f3b9 100644
--- a/admin/WebConsole/propiedades/propiedades_ordenadores.php
+++ b/admin/WebConsole/propiedades/propiedades_ordenadores.php
@@ -63,6 +63,7 @@ if  ($opcion!=$op_alta){
 <head>
 	<meta http-equiv="Content-Type" content="text/html;charset=UTF-8" />
 	<link rel="stylesheet" type="text/css" href="../estilos.css" />
+	<SCRIPT language="javascript" src="../jscripts/validators.js"></SCRIPT>
 	<script language="javascript" src="../jscripts/propiedades_ordenadores.js"></script>
 	<script language="javascript" src="../jscripts/opciones.js"></script>
 	<?php echo '<script language="javascript" src="../idiomas/javascripts/'.$idioma.'/propiedades_ordenadores_'.$idioma.'.js"></script>'?>