passing condition

[ddtpmyra]

Hello Bytes:

Here's a continuation of my dilema :)
Now it works fine but Im planning to put more functionality on my JS script so Im planning to have a 'header JS' to call the subs functions. The problem is that it do call the the sub function but it let the browser go to the next page even though the condition didn't pass to return true. Please help. Thanks in advance!

The button Form

<form method="post" name="recapform" action="tblevent_entry_recap_handler.php?id=<?php echo $_GET['id'];?> " onsubmit="return show_alert()">

header function

<script type="text/javascript">
function show_alert()
{
entity_alert();

}
</script>

sub function

<script type="text/javascript">
function entity_alert()
{
x=  parseInt(document.getElementById('Organization').value)+parseInt(document.getElementById('Agencies').value)
	+parseInt(document.getElementById('Education').value)+parseInt(document.getElementById('Medical').value)+parseInt(document.getElementById('Business').value)
	+parseInt(document.getElementById('Professional').value)+parseInt(document.getElementById('Government').value)+parseInt(document.getElementById('Residential').value)
	+parseInt(document.getElementById('ResidentialFacility').value);
	
y = parseInt(document.getElementById('ActualAttendees').value)

if (x>y)
  {
	alert ('Entity is over by:' + (x-y));
	return false;
  }
else
  {
	     return true;
  }

}
</script>

[acoder]

In show_alert(), add a return:

[i]return[/i] entity_alert();

[ddtpmyra]

It works but when I added more function underneath in only perform the most top functionalities ... am i doint this correctly?

header function

<script type="text/javascript">
function show_alert()
{
return entity_alert();
return age_alert();


}
</script>

sub functions

function entity_alert()
{
x=  parseInt(document.getElementById('Organization').value)+parseInt(document.getElementById('Agencies').value)
	+parseInt(document.getElementById('Education').value)+parseInt(document.getElementById('Medical').value)+parseInt(document.getElementById('Business').value)
	+parseInt(document.getElementById('Professional').value)+parseInt(document.getElementById('Government').value)+parseInt(document.getElementById('Residential').value)
	+parseInt(document.getElementById('ResidentialFacility').value);
	
y = parseInt(document.getElementById('ActualAttendees').value)

if (x>y)
  {
	alert ('Entity is over by:' + (x-y));
	document.getElementById('Organization').focus();
	return false;
  }
else
  {
	     return true;
  }

}
</script>


<!--- 2nd function -->
<script type="text/javascript">
function age_alert()
{
tAge=  parseInt(document.getElementById('Under18').value)+parseInt(document.getElementById('Eighteen34').value)+parseInt(document.getElementById('ThirtyFive54').value)+parseInt(document.getElementById('FiftyFour').value);
				
a = parseInt(document.getElementById('ActualAttendees').value)
			
if (tAge>a)
 {
alert ('Age Group is over by:' + (tAge-a));
document.getElementById('Under18').focus();
return false;
  }
else
 {
 return true;
  }
			
}
</script>

[acoder]

Once you return, it won't execute any more statements.

Try something like:

return (funA() && funB());

[ddtpmyra]

Once you return, it won't execute any more statements.

Try something like:

return (funA() && funB());

Here's what I did below but it only performs the first function. Anything do i have to do on sub functions?

Header

<script type="text/javascript">
function show_alert()
{
return (age_alert() && entity_alert());
}
</script>

sub two functions :
1).

<script type="text/javascript">
function entity_alert()
{
		tEntity=  parseInt(document.getElementById('Organization').value)+parseInt(document.getElementById('Agencies').value)
			+parseInt(document.getElementById('Education').value)+parseInt(document.getElementById('Medical').value)+parseInt(document.getElementById('Business').value)
			+parseInt(document.getElementById('Professional').value)+parseInt(document.getElementById('Government').value)+parseInt(document.getElementById('Residential').value)
			+parseInt(document.getElementById('ResidentialFacility').value);
			
		y = parseInt(document.getElementById('ActualAttendees').value);
		
	if (tEntity>y)
		  {
			alert ('Entity is over by:' + (tEntity-y));
				return false;
			  }
				else	
			  {
				 return true;
			  }
}
</script>

2).

<script type="text/javascript">
function age_alert()
{
		y = parseInt(document.getElementById('ActualAttendees').value);
		tAge=  parseInt(document.getElementById('Under18').value)+parseInt(document.getElementById('Eighteen34').value)+parseInt(document.getElementById('ThirtyFive54').value)+parseInt(document.getElementById('FiftyFour').value);
	if (tAge>y) {
				alert ('Age Group is over by:' + (tAge-y));
				return false;
			  	}
	
	else     	{
				 return true;
				}
}
</script>

[acoder]

If age_alert() returns false, it won't go to entity_alert() because there's no need.

[ddtpmyra]

If age_alert() returns false, it won't go to entity_alert() because there's no need.

Please tell me what the best way to handle this two conditions before it let the user to go to the next page?

[acoder]

Find a scenario when the first function should return true and the second false to see if it works.

[ddtpmyra]

I just redo it and make it simplier and it works perfectly fine. Thanks for all of your inputs ACECODE

	
if  ((a<=y) && (b<=y) && (c <=y) && (d<=y)  && (Ustatus=='Complete'))
 {   
return true; 
  }
else {   alert ('Please check your entry total and event status');
return false;
}