problema con checkbox funzionante solamente al secondo "tentativo"

[ali7.p]

Ho un probelma con un codice javascript associato a un checkbox

Se inizialmente il checkbox non è selzionato nessun problema, funziona tutto a meraviglia, se invece ha l'attributo checked settato a checked:
-la visualizzazione iniziale è corretta
-quando deseleziono il checkbox i capi day[day], day[month] e day[year] vengono correttamente modificati, mentre time e type[] rimangono come se il checkbox fosse selezionato
- se riseleziono la visualizzazione è esatta
- se deseleziono nuovamente la visualizzazione è esatta


Ecco il codice, spero che possiate aiutarmi!!

function NoScad(){

	ElCheck=document.getElementById('senzaScad');
	if(ElCheck.checked==true){
		vDay="00";
		vMonth="00";
		vYear="0000";
		incombente=1;
		stato=true;
		ora="00:00";
		tipo="hidden";
	}
	else{
		var data = new Date();
		vDay=data.getDate();
		if (vDay < 10) vDay = "0" + vDay;
		vMonth=data.getMonth()+1;
		if (vMonth < 10) vMonth = "0" + vMonth;
		vYear=data.getFullYear();
		incombente=0;
		stato=false;
		tipo="text";
	}

	arrCaselleInput = document.getElementsByTagName("input");
	for (i = 0; i < arrCaselleInput.length; i++)
	{
		switch (arrCaselleInput[i].name) 
		{  
		    case "day[day]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vDay;
			
		      break;
		    case "day[month]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vMonth;
			
		      break;
			case "day[year]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vYear;
			
		      break;
			case "time":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = ora;
			
		      break;
		}
	}
	
	// se selezionato senza scadenza automaticamente incombente
	arrSelect = parent.document.getElementsByTagName("SELECT");
	for (i = 0; i < arrSelect.length; i++)
	{
		switch (arrSelect[i].name) 
		{  
		    case "type[]":
				arrSelect[i].disabled = stato;
				arrSelect[i].value = incombente;
				
		    break;

		}
	}

	
}

 

[micio86]

Prova

Prova a farti degli alert nei vari cicli e nei vari if per capire i valori che assegni alle variabili e anche quelli di default. Solitamente faccio così anche xkè i valori possono anche cambiare tra i vari browsers. Negli input se checkati fai sempre <input.... checked="checked" />
Tieni conto ke la funzione getElementByTagName ti becca tutti gli input nella pagina mentre getElementByName becca quelli con lo stesso nome (ammesso dal w3c).

Fai un alert(ElCheck.checked) e il primo if fallo così: if(ElCheck.checked) {......

 

[ali7.p]

con if(ElCheck.checked){ non cambia niente

ma l'alert dove devo metterlo? scusa, ma javascript non è proprio il "mio" linguaggio, non ci capisco molto

 

[micio86]

Beh

mettilo subito prima dell'if

 

[ali7.p]

alert o non allert non cambia niente (e l'alert non fa niente, no mi stampa nessun messaggio)

 

[micio86]

allora

allora significa ke non gli arriva nulla!!! ecco perchè non funge

Controlla ke ci sia l'oggetto con id ElCheck nella pagina. l'alert ti deve dare o true o false!

 

[ali7.p]

ok, allora... adesso l'alert funziona e il risultato è giusto.
Parto con il checkbox selzionato, quando lo deselesiono l'alert mi dà false ma la select type[] mi rimane sempre disabilitata.
Riselezione, l'alert è true ed è tutto ok, deseleziono l'alert è false e adesso è tutto giusto, con il slect type[] attivo :incazz2:

 

[micio86]

Il problema sarà sicuramente qui:

// se selezionato senza scadenza automaticamente incombente
	arrSelect = parent.document.getElementsByTagName("SELECT");
	for (i = 0; i < arrSelect.length; i++)
	{
                alert(arrSelect[i].name);//Fai qui un alert
		switch (arrSelect[i].name) 
		{  
		    case "type[]":
				arrSelect[i].disabled = stato;
				arrSelect[i].value = incombente;
				
		    break;

		}
	}

Sostituiscilo come te l'ho scritto e vedi cosa ti dà l'alert

Poi prova a sostituire il valore di stato scambiando true con false e disabled lo chiami enabled.

function NoScad(){

	ElCheck=document.getElementById('senzaScad');
	if(ElCheck.checked){
		vDay="00";
		vMonth="00";
		vYear="0000";
		incombente=1;
		stato=false;
		ora="00:00";
		tipo="hidden";
	}
	else{
		var data = new Date();
		vDay=data.getDate();
		if (vDay < 10) vDay = "0" + vDay;
		vMonth=data.getMonth()+1;
		if (vMonth < 10) vMonth = "0" + vMonth;
		vYear=data.getFullYear();
		incombente=0;
		stato=true;
		tipo="text";
	}

	arrCaselleInput = document.getElementsByTagName("input");
	for (i = 0; i < arrCaselleInput.length; i++)
	{
		switch (arrCaselleInput[i].name) 
		{  
		    case "day[day]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vDay;
			
		      break;
		    case "day[month]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vMonth;
			
		      break;
			case "day[year]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vYear;
			
		      break;
			case "time":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = ora;
			
		      break;
		}
	}
	
	// se selezionato senza scadenza automaticamente incombente
	arrSelect = parent.document.getElementsByTagName("SELECT");
	for (i = 0; i < arrSelect.length; i++)
	{
                alert(arrSelect[i].name);
		switch (arrSelect[i].name) 
		{  
		    case "type[]":
				arrSelect[i].enabled = stato;
				arrSelect[i].value = incombente;
				
		    break;

		}
	}

	
}

 

[ali7.p]

facendo così se inizialmente il checkbox è settato la slòect rimane semple disabilitata; se inizialmente il checkbox non è selezionato la select rimane sempre attiva

 

[micio86]

Non capisco a me cambia correttamente...

Uso questo codice e credo ke faccia quello ke vuoi:

<script type="text/javascript">
function NoScad(){

	ElCheck=document.getElementById('senzaScad');
	if(ElCheck.checked){
		vDay="00";
		vMonth="00";
		vYear="0000";
		incombente=1;
		stato=false;
		ora="00:00";
		tipo="hidden";
	}
	else{
		var data = new Date();
		vDay=data.getDate();
		if (vDay < 10) vDay = "0" + vDay;
		vMonth=data.getMonth()+1;
		if (vMonth < 10) vMonth = "0" + vMonth;
		vYear=data.getFullYear();
		incombente=0;
		stato=true;
		tipo="text";
	}

	arrCaselleInput = document.getElementsByTagName("input");
	for (i = 0; i < arrCaselleInput.length; i++)
	{
		switch (arrCaselleInput[i].name) 
		{  
		    case "day[day]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vDay;
			
		      break;
		    case "day[month]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vMonth;
			
		      break;
			case "day[year]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vYear;
			
		      break;
			case "time":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = ora;
			
		      break;
		}
	}
	
	// se selezionato senza scadenza automaticamente incombente
	arrSelect = parent.document.getElementsByTagName("select");
	for (i = 0; i < arrSelect.length; i++)
	{
		switch (arrSelect[i].name) 
		{  
		    case "type[]":
				arrSelect[i].enabled = stato;
				arrSelect[i].value = incombente;
				
		    break;

		}
	}

	
}
</script>
<input type="checkbox" name="senzaScad" id="senzaScad" value="1" onchange="NoScad();" />
<input type="text" name="day[day]" id="day[day]" />
<input type="text" name="day[month]" id="day[month]" />
<input type="text" name="day[year]" id="day[year]" />
<input type="text" name="time" id="time" />

<select name="type[]" id="type[]">
<option value="">Seleziona</option>
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
<option value="5">5</option>
<option value="6">6</option>
</select>

 

[ali7.p]

uhm... intanto, come ti avevo fatto notare prima quando avevo provato la soluzione con enable al posto di disable, la select rimane sempre attiva e invece io voglio che sia disabilitata quando il checkbox è settato...

Io comunque il problema ce l'ho all'inverso, quando inizialmente la checkbox è settata... partendo dalla select non settata funziona tutto bene il codice che avevo postato.
rimetti disable e nel codice della select disabled="true" e vedi che funziona al seocndo tentativo...

 

[micio86]

Bah

Non kiedermi perchè ma ho risolto così:

<script type="text/javascript">
function NoScad(bool) {
	if(bool) {
		vDay="00";
		vMonth="00";
		vYear="0000";
		incombente=1;
		ora="00:00";
		tipo="hidden";
	}
	else {
		var data = new Date();
		vDay=data.getDate();
		if (vDay < 10) vDay = "0" + vDay;
		vMonth=data.getMonth()+1;
		if (vMonth < 10) vMonth = "0" + vMonth;
		vYear=data.getFullYear();
		incombente=0;
		tipo="text";
	}
	// se selezionato senza scadenza automaticamente incombente
	document.frm.type.disabled=bool;
	arrCaselleInput = document.getElementsByTagName("input");

	for (i = 0; i < arrCaselleInput.length; i++) {
		switch (arrCaselleInput[i].name) 
		{  
		    case "day[day]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vDay;
		      break;
		    case "day[month]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vMonth;
		      break;
			case "day[year]":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = vYear;
		      break;
			case "time":
			arrCaselleInput[i].type = tipo;
			arrCaselleInput[i].value = ora;
		      break;
		}
	}
}
</script>
<form name="frm">
<input type="checkbox" name="senzaScad" id="senzaScad" value="1" checked="checked" onclick="NoScad(this.checked);" />
<input type="text" name="day[day]" id="day[day]" />
<input type="text" name="day[month]" id="day[month]" />
<input type="text" name="day[year]" id="day[year]" />
<input type="text" name="time" id="time" />

<select name="type" id="type" disabled>
<option value="">Seleziona</option>
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
<option value="5">5</option>
<option value="6">6</option>
</select>
</form>

 

[ali7.p]

grazie!!
sei un 0