php error

[alin19]

i don't get it, what i have wrong here?

 

if ((isset ($_POST['inreg_contract']))=='Inreg Contract')
{

$query_Sel="SELECT `activ` AS activ FROM `conturi` WHERE  `Id_bo`='$id_bo'";

	if ($r=mysql_query($query_Sel))
		{
		$row = mysql_fetch_array($r);
			$activ=$row['activ'];
		}			

if (isset ($activ)=='Da')
	{
	echo <<<EOF
	<script language="javascript" type="text/javascript">
	<!--


	alert  ('acest client are deja un cont activ, verificati datele');

	-->
	</script>

EOF;
echo $activ;
	}

 

 

and it outputs: Nu and that alert message, but it should'n outpun nothing because $activ is not "Da"

[2levelsabove]

You are missing a closing brace

 

[alin19]

where?

this also doesn't work

 

if ((isset ($activ))=='Da')

 

 

 

[rhodesa]

Remove the isset() from around $activ

[paul2463]

try this one

if ((isset ($activ))&&($activ == 'Da'))

 

I think yours fails because "isset($activ)" will return true before any comparison is carried out so therefore you get the alert message

[kenrbnsn]

This line

<?php
if ((isset ($_POST['inreg_contract']))=='Inreg Contract')
?>

doesn't make any sense. You probably want:

<?php
if (isset ($_POST['inreg_contract']) && $_POST['inreg_contract'] =='Inreg Contract')
?>

 

Ken

[alin19]

try this one

if ((isset ($activ))&&($activ == 'Da'))

 

I think yours fails because "isset($activ)" will return true before any comparison is carried out so therefore you get the alert message

 

this work's but i don't get it, i've used that code before and it work's

[alin19]

This line

<?php
if ((isset ($_POST['inreg_contract']))=='Inreg Contract')
?>

 

 

Ken

 

this works just fine,

[kenrbnsn]

You may think it works fine, but it's not doing what you think it is supposed to do.

 

That condition will never evaluate to "true", since the isset() function will return either "true" or "false", which will never be equal to "Inreg Contract".

 

Ken

[paul2463]

if you are doing more than a simple IF statement you have to breakdown exactly what you want in the case I answered

if ((isset ($activ))=='Da') //your code
if ((isset ($activ))&&($activ == 'Da')) //my code

in your code the first thing that happens is it checks to see if $active is set, if its true, then nothing else gets checked because there is no conditional statement for another check

in my code it checks to see if $active is set (true) then because of the extra conditional "&&" it also checks to see if its equal to "Da"