#834: Eliminar etiquetas y código redundante en JavaScript.

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

14 files changed, 18 insertions, 23 deletions

diff --git a/admin/WebConsole/asistentes/jscripts/asistentes.js b/admin/WebConsole/asistentes/jscripts/asistentes.js
index 383f19cc..5c257615 100644
--- a/admin/WebConsole/asistentes/jscripts/asistentes.js
+++ b/admin/WebConsole/asistentes/jscripts/asistentes.js
@@ -507,8 +507,6 @@ function getMaxCacheSize() {
 			maxSize = cacheSizeArray[i].value;
 	}
 	document.getElementById("maxcachesize").value = maxSize;
-	return
-
 }
 
 
@@ -516,8 +514,8 @@ function getMaxCacheSize() {
 function validaCache (freedisk) {
 	var form = document.fdatos;
 	var maxcachesize = parseInt(document.getElementById("maxcachesize").value);
-        if(form.part4.value == "CACHE" && form.check4.checked && form.size4.value == 0 ){
-	    return (freedisk - maxcachesize > 0 ? true : false);
+        if(form.part4.value === "CACHE" && form.check4.checked && form.size4.value == 0 ){
+	    return ((freedisk - maxcachesize) > 0);
 	}
 	return true;
 }
diff --git a/admin/WebConsole/comandos/EliminarImagenRepositorio.php b/admin/WebConsole/comandos/EliminarImagenRepositorio.php
index 438f3cea..7fbb046a 100644
--- a/admin/WebConsole/comandos/EliminarImagenRepositorio.php
+++ b/admin/WebConsole/comandos/EliminarImagenRepositorio.php
@@ -298,10 +298,10 @@ function confirmeliminar() {var mensaje="<?php echo $TbMsg[17];?>";if(confirm(me
 			<INPUT type="hidden" name="grupoid" value="<?php echo $grupoid ?>">
 			<?php if ($modov == "0"){?>
 				<INPUT type="hidden" name="modov" value="1">
-				<input type=button onclick=submit() value="<?php echo $TbMsg[28]; ?>"/>
+				<input type=button onclick="submit();" value="<?php echo $TbMsg[28]; ?>"/>
 			<?php }else{ ?>
 				<INPUT type="hidden" name="modov" value="0">
-				<input type=button onclick=submit() value="<?php echo $TbMsg[29]; ?>"/>
+				<input type=button onclick="submit();" value="<?php echo $TbMsg[29]; ?>"/>
 				<?php } ?>
 			</form>
 		</div>
@@ -617,7 +617,7 @@ function confirmeliminar() {var mensaje="<?php echo $TbMsg[17];?>";if(confirm(me
 		</TR>
 		<TR>
 			<TD></TD>
-		<TD align=center><A href=#><IMG border=0 src="../images/boton_confirmar_<?php echo $idioma ?>.gif" onclick="javascript:confirmeliminar()" ></A></TD>
+		<TD align=center><A href=#><IMG border=0 src="../images/boton_confirmar_<?php echo $idioma ?>.gif" onclick="confirmeliminar()" ></A></TD>
 		</TR>
 	</TABLE>
 </FORM>
diff --git a/admin/WebConsole/includes/opcionesbotones.php b/admin/WebConsole/includes/opcionesbotones.php
index dd7c47b0..930d0404 100644
--- a/admin/WebConsole/includes/opcionesbotones.php
+++ b/admin/WebConsole/includes/opcionesbotones.php
@@ -1,7 +1,7 @@
 <TABLE align=center>
 	<TR>
-		<TD><A href=#><IMG border=0 src="../images/boton_cancelar_<?php echo $idioma ?>.gif" onclick="javascript:cancelar()"></A></TD>
+		<TD><A href=#><IMG border=0 src="../images/boton_cancelar_<?php echo $idioma ?>.gif" onclick="cancelar()"></A></TD>
 		<TD width=20></TD>
-		<TD><A href=#><IMG border=0 src="../images/boton_confirmar_<?php echo $idioma ?>.gif" onclick="javascript:confirmar()" ></A></TD>
+		<TD><A href=#><IMG border=0 src="../images/boton_confirmar_<?php echo $idioma ?>.gif" onclick="confirmar()" ></A></TD>
 	</TR>
 </TABLE>
diff --git a/admin/WebConsole/jscripts/aulas.js b/admin/WebConsole/jscripts/aulas.js
index d333b179..b580c2fc 100644
--- a/admin/WebConsole/jscripts/aulas.js
+++ b/admin/WebConsole/jscripts/aulas.js
@@ -165,7 +165,6 @@ function eco_remoto(lit)
 	reset_contextual(-1,-1);
  	if(Ambito==null){
 		alert(TbMsg[6]);
-		 return;
 	}
 	else{
 		if(Ambito==AMBITO_ORDENADORES)
diff --git a/admin/WebConsole/jscripts/informacion_menus.js b/admin/WebConsole/jscripts/informacion_menus.js
index 68b86c6e..4f1777a7 100644
--- a/admin/WebConsole/jscripts/informacion_menus.js
+++ b/admin/WebConsole/jscripts/informacion_menus.js
@@ -58,7 +58,6 @@ function modificar_items(){
 		var descrierror=splitRetorno[1];
 		if (!resul){ // Ha habido algn error en la inserci�
 			alert(descrierror);
-			return
 		}
 	}
 }
@@ -79,7 +78,6 @@ function modificar_item(){
 		var descrierror=splitRetorno[1];
 		if (!resul){ // Ha habido algn error en la inserci�
 			alert(descrierror);
-			return
 		}
 	}
 }
diff --git a/admin/WebConsole/principal/colasacciones.php b/admin/WebConsole/principal/colasacciones.php
index 382d4165..1c6558d7 100644
--- a/admin/WebConsole/principal/colasacciones.php
+++ b/admin/WebConsole/principal/colasacciones.php
@@ -281,7 +281,7 @@
 	$HTMLCriterios.='<TABLE class="tabla_busquedas" align=center border="0">';
 	$HTMLCriterios.='	<TR>';
 	$HTMLCriterios.='		<TD>';	// Lupa
-	$HTMLCriterios.='			<A href="#busca"><IMG border=0 src="../images/iconos/busquedas.gif" onclick="javascript:fdatos.submit()" alt="Buscar"></A>';
+	$HTMLCriterios.= '			<A href="#busca"><IMG border=0 src="../images/iconos/busquedas.gif" onclick="fdatos.submit();" alt="Buscar"></A>';
 	$HTMLCriterios.='		</TD>';
 	$HTMLCriterios.='</TR>';
 	$HTMLCriterios.='</TABLE>';
diff --git a/admin/WebConsole/principal/colasreservas.php b/admin/WebConsole/principal/colasreservas.php
index 3d7595b3..c9ba198a 100644
--- a/admin/WebConsole/principal/colasreservas.php
+++ b/admin/WebConsole/principal/colasreservas.php
@@ -957,7 +957,7 @@ function CriteriosBusquedas(){
 		$HTMLCriterios.='<TR>'.chr(13);
 		$HTMLCriterios.='<TD>';
 
-		$HTMLCriterios.='<IMG src="../images/iconos/busquedas.gif" onclick="javascript:fdatos.submit()" style="cursor:hand" alt="Buscar">';
+		$HTMLCriterios.= '<IMG src="../images/iconos/busquedas.gif" onclick="fdatos.submit();" style="cursor:hand" alt="Buscar">';
 		$HTMLCriterios.='</TD>';
 		$HTMLCriterios.='<TD>';
 		$HTMLCriterios.='</TD>'.chr(13);
diff --git a/admin/WebConsole/principal/programacionesaulas.php b/admin/WebConsole/principal/programacionesaulas.php
index 9cc10751..d4918a4b 100644
--- a/admin/WebConsole/principal/programacionesaulas.php
+++ b/admin/WebConsole/principal/programacionesaulas.php
@@ -456,7 +456,7 @@ function CriteriosBusquedas(){
 		$HTMLCriterios.='<TR>'.chr(13);
 		$HTMLCriterios.='<TD>';
 
-		$HTMLCriterios.='<IMG src="../images/iconos/busquedas.gif" onclick="javascript:fdatos.submit()" style="cursor:hand" alt="Buscar">';
+		$HTMLCriterios.= '<IMG src="../images/iconos/busquedas.gif" onclick="fdatos.submit();" style="cursor:hand" alt="Buscar">';
 		$HTMLCriterios.='</TD>';
 		$HTMLCriterios.='<TD>';
 		$HTMLCriterios.='</TD>'.chr(13);
diff --git a/admin/WebConsole/principal/programacionesreservas.php b/admin/WebConsole/principal/programacionesreservas.php
index db1355cf..3b01d3af 100644
--- a/admin/WebConsole/principal/programacionesreservas.php
+++ b/admin/WebConsole/principal/programacionesreservas.php
@@ -423,7 +423,7 @@ function CriteriosBusquedas(){
 		$HTMLCriterios.='<TR>'.chr(13);
 		$HTMLCriterios.='<TD>';
 
-		$HTMLCriterios.='<IMG src="../images/iconos/busquedas.gif" onclick="javascript:fdatos.submit()" style="cursor:hand" alt="Buscar">';
+		$HTMLCriterios.= '<IMG src="../images/iconos/busquedas.gif" onclick="fdatos.submit();" style="cursor:hand" alt="Buscar">';
 		$HTMLCriterios.='</TD>';
 		$HTMLCriterios.='<TD>';
 		$HTMLCriterios.='</TD>'.chr(13);
diff --git a/admin/WebConsole/varios/accionmenu.php b/admin/WebConsole/varios/accionmenu.php
index b2f30900..fed455d0 100644
--- a/admin/WebConsole/varios/accionmenu.php
+++ b/admin/WebConsole/varios/accionmenu.php
@@ -151,7 +151,7 @@ else
 	<BR>	
 <TABLE align=center>
 	<TR>
-		<TD><A href="#botones"><IMG border=0 src="../images/boton_confirmar.gif" onclick="javascript:confirmar(<?php echo $op?>)" ></A></TD>
+		<TD><A href="#botones"><IMG border=0 src="../images/boton_confirmar.gif" onclick="confirmar(<?php echo $op?>)" ></A></TD>
 	</TR>
 </TABLE>
 	<BR>		
diff --git a/admin/WebConsole/varios/calendario_ventana.php b/admin/WebConsole/varios/calendario_ventana.php
index eec3d57d..b91c4e1d 100644
--- a/admin/WebConsole/varios/calendario_ventana.php
+++ b/admin/WebConsole/varios/calendario_ventana.php
@@ -175,8 +175,8 @@ echo '</TR>';
 
 echo '<table align=center border=0>';
 echo '<tr>';
-echo '<td onclick="javascript:mes_anterior();" style="cursor:pointer;COLOR: MediumBlue;FONT-FAMILY: Arial, Helvetica, sans-serif;FONT-SIZE:10" align=left><<</td>';
-echo '<td onclick="javascript:mes_siguiente()" style="cursor:pointer;COLOR: MediumBlue;FONT-FAMILY: Arial, Helvetica, sans-serif;FONT-SIZE:10" align=right>>></td>';
+echo '<td onclick="mes_anterior();" style="cursor:pointer;COLOR: MediumBlue;FONT-FAMILY: Arial, Helvetica, sans-serif;FONT-SIZE:10" align=left><<</td>';
+echo '<td onclick="mes_siguiente();" style="cursor:pointer;COLOR: MediumBlue;FONT-FAMILY: Arial, Helvetica, sans-serif;FONT-SIZE:10" align=right>>></td>';
 echo '</tr>';
 echo '<tr>';
 echo '<td colspan=2 valign=top width=100>'.$mialmanaque->MesAnno($mes_elegido,$anno_elegido).'</td>';
diff --git a/admin/WebConsole/varios/incluiracciones.php b/admin/WebConsole/varios/incluiracciones.php
index 8b04f4a2..43cf1abe 100644
--- a/admin/WebConsole/varios/incluiracciones.php
+++ b/admin/WebConsole/varios/incluiracciones.php
@@ -150,7 +150,7 @@ else
 	<BR>	
 <TABLE align=center>
 	<TR>
-		<TD><A href="#botones"><IMG border=0 src="../images/boton_confirmar.gif" onclick="javascript:confirmar(<?php echo $op?>)" ></A></TD>
+		<TD><A href="#botones"><IMG border=0 src="../images/boton_confirmar.gif" onclick="confirmar(<?php echo $op?>)" ></A></TD>
 	</TR>
 </TABLE>
 	<BR>		
diff --git a/admin/WebConsole/varios/incorporaordenadores.php b/admin/WebConsole/varios/incorporaordenadores.php
index 24b75cb3..103f7171 100644
--- a/admin/WebConsole/varios/incorporaordenadores.php
+++ b/admin/WebConsole/varios/incorporaordenadores.php
@@ -70,7 +70,7 @@ if(!empty($contenido)){ // Se ha introducido contenido en lugar de fichero
 	<TR>
 		<TD><IMG src="../images/boton_cancelar.gif" style="cursor:hand"  onclick=""></TD>
 		<TD width=20></TD>
-		<TD><IMG src="../images/boton_confirmar.gif" style="cursor:hand"  onclick="javascript:document.fdatos.submit();"></TD>
+		<TD><IMG src="../images/boton_confirmar.gif" style="cursor:hand"  onclick="document.fdatos.submit();"></TD>
 	</TR>
 </TABLE>
 <?php
diff --git a/admin/WebConsole/varios/programaciones.php b/admin/WebConsole/varios/programaciones.php
index 43b85b95..522614ca 100644
--- a/admin/WebConsole/varios/programaciones.php
+++ b/admin/WebConsole/varios/programaciones.php
@@ -208,8 +208,8 @@ $mialmanaque= new Almanaque("tabla_meses");
 								<TD>
 									<?php
 										$HTMLSELECT="";
-										$HTMLSELECT.='<SELECT onclick=consulta_programacion(); 
-										class=estilodesple id="lista_programaciones" size=2 style="height:100; width: 350">' ;
+										$HTMLSELECT.='<SELECT onclick="consulta_programacion();" 
+										class="estilodesple" id="lista_programaciones" size=2 style="height:100; width: 350">' ;
 										if ($numreg>0){
 											while (!$rs->EOF){
 												$HTMLSELECT.='<OPTION value="'.$rs->campos["idprogramacion"].'"';