PHP - If Statement Always Testing True

I have a statement

$account = $_SESSION['account_id'];
$postedby = $account_id;

/*both echo out to the same number*/

if($account == $postedby){
 echo "DELETE BUTTON";
}

i echo'ed out both $account and $postedby and both return 1 but the if statement fails to read as true.

HELP!!!

thanks in advance!

Hey here is my code im trying to redirect  the index.php to index.php?id=1
the way i have done it doesn't look really good is there a better way to do it please?

im trying to redirect to id=1 because otherwise i wont be able to echo out anything...

id 1 is default category so is there a way to tell it if  $id is empty echo from id 1?

<?php   

        $id = $_GET['id']; 
if($id == ''){
echo "<meta http-equiv=Refresh content=0;url=index.php?id=1>";

}
        $result = mysql_query("SELECT * FROM pages WHERE id='$id' ",$connect);
        while($row = mysql_fetch_assoc($result))
             {
        echo $row['anything1']; 
        echo $row['anything2']; 
} 
?>

I have an IF statement that is constantly returning TRUE, even when it shouldn't be. Basically it checks if an array is empty.

Code: [Select]
<?php

      $error = array();
      

      //No blanks
      if ($street_addr == "" || $city == "" || $state == "" || $zip == "" || $phone1 == "" || 
            $phone2 == "" || $phone3 == "" || $county == "")
               $error[] = "You left something blank!";
      
      //validate zip
      if(!preg_match("/^[0-9]{5}$/", $zip)) 
          $error[] = "The ZIP code must be a 5-digit number.";
          
      //validate phone number
      if (!is_numeric($phone1) || !is_numeric($phone2) || !is_numeric($phone3))
         $error[] = "Phone can only contain digits";
         
      if (strlen($phone1) < 3 || strlen($phone2) < 3 || strlen($phone3) < 4)
         $error[] = "Invalid phone number";

      $count_errors = count($error);
         
      if ($count_errors > 0){
      
         echo "ERROR:<br />";
         
         foreach ($error as $err){
         
            echo "-$err<br />";
         
         }
         
      } else { 
         
          //Other code
      }

?>

So the first part of the IF statement is always executing.

$count_errors DOES return 0 when printed out.

Any help would be greatly appreciated. Thanks!

Hi.

I have a simple change password form, when checked and verified and submitted the user is directed to a different web page. 
I'm getting all the true statements echoed out, but it doesn't go to the go to the next web page.
// Thank you Page
$insertGoTo = "changepass.php";
header(sprintf("Location: %s", $insertGoTo));

My thought was ounce you get through the while statement without errors you should be able to move to the next check of the actual database... which it does, and it updates the database with the new values.  It also echos out Continue 6....
echo "True - Continue 6 GO TO HEADER PAGE";
// Thank you Page
$insertGoTo = "changepass.php";
header(sprintf("Location: %s", $insertGoTo));

What am I missing here? 


Code:

<?php
session_start();
?>
<?php
$submit = $_POST['submit'];
// Form Data
$email = $_POST['email'];
$password_old = $_POST['password_old'];
$password_new = $_POST['password_new'];
$password_new_con = $_POST['password_new_con'];
$errorcount = 0;

// Edit anything inbetween the " " for the display error message
$errormsg['Email'] = "Email Entered is not in our database"; 
$errormsg['OldPass'] = "Old Password Entered is Incorrect. Please check your Email";
$errormsg['NewPass'] = "New Password must be between 6 and 32 characters";
$errormsg['NewPassCon'] = "New Passwords do not match.";
$errormsg['SecCode'] = "Security Code is Invalid";
$errormsg['dbPass'] = "Invalide Email or activation code. 
Please Contact <a href='mailto:webmaster@fusionfashionhair.com?subject=Fusion Fashion Hair - 
Member Activation Error%20Request'>Admin</a>";
$errormsg['NoErr'] = "No Errors, Continue";
$errormsg['PlaceHold'] = "";

$errortrack[] = $errormsg['PlaceHold'];

if ($_POST[submit]){
if ($errorstop = "go") {
$errorstop="go";
while ($errorstop<>"stop") {
//Check for security code
if ($_SESSION[key]==$_POST[user_code]){ 
echo "True - Continue 0";
echo "<p>----------</p>";
$_SESSION[key]='';
} else {
echo "False - Stop 0";
$errortrack[] = $errormsg['SecCode'];
$errorcount++;
$errorstop="stop";
}
// check for existance
if (!checkEmail($email)) {
echo "False - Stop 1";
$errortrack[] = $errormsg['Email'];
$errorcount++;
$errorstop="stop";
} else {
echo "True - Continue 1";
echo "<p>----------</p>";
}
// check for existance
if (strlen($password_old)>5) {
echo "True - Continue 2";
echo "<p>----------</p>";
} else {
echo "False - Stop 2";
$errortrack[] = $errormsg['OldPass'];
$errorcount++;
$errorstop="stop";
}
// check for existance
if (strlen($password_new)>32||strlen($password_new)<6) {
echo "False - Stop 3";
$errortrack[] = $errormsg['NewPass'];
$errorcount++;
$errorstop="stop";
} else {
echo "True - Continue 3";
echo "<p>----------</p>";
}

// check for existance
if ($password_new_con==$password_new) {
echo "True - Continue 4";
echo "<p>----------</p>";
$errorstop="stop";//Get Out of loop
} else {
echo "False - Stop 4";
$errortrack[] = $errormsg['NewPassCon'];
$errorcount++;
$errorstop="stop";
}
}//End While Loop


// Check database
require('dbConfig.php');
// Encrypts old password to check with Database Encryped Password
$password_old = md5($password_old);

$check = mysql_query("SELECT * FROM {$usertable} WHERE email='$email' AND password='$password_old'");
$checknum = mysql_num_rows($check);
if ($checknum==1) {
echo "True - Continue 5 Set password";
echo "<p>----------</p>";
// Encrypts new password
$password = md5($password_new);
//run a query to update the account
$acti = mysql_query("UPDATE {$usertable} SET password='$password' WHERE email='$email'");
} else {
echo "False - Stop 5";
$errortrack[] = $errormsg['dbPass'];
$errorcount++;
$errorstop="stop";
}//End if checknum

echo "True - Continue 6 GO TO HEADER PAGE";
// Thank you Page
$insertGoTo = "changepass.php";
header(sprintf("Location: %s", $insertGoTo));
} else {
while($errorcount>=0) {
// Test display all error messages
echo "<p>----------</p>";
echo "<p>Error Count = '$errorcount'</p>";
}
die ("PLEASE FILL IN ALL FIELDS");
}
}
?>
<?php

// LINUX PLATFORM OPTION 3

// checkEmail function checks standard email format same as preg_match()
// checkEmail function checks DSN records using checkdnsrr  Use list() to seperate name and domain using split function
// checkdnsrr ONLY WORKS on LINUX PLATFORM
// Check to see if domain and username is active
// uses fsockopen() to check if its in use using port 25
function checkEmail($email) {
// checks proper syntax
if(preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/" , $email)) {

// gets domain name
list($username,$domain)=split('@',$email);
// checks for if MX records in the DNS
if(!checkdnsrr($domain, 'MX')) {
return false;
}
return true;
}
return false;
}
?>

$looponce = 1;
foreach ($this->info as $k => $v) {
if ( (($k == 'subject') && ($v['required'])) && (!$this->settings['customSubject'])) {
for ($i = 0; $i <= 0; $i++) {
$output[] = $this->display_errors('error_system_subject');
}
}
if ( (($k == 'name') && (!$v['required'])) || ((!array_key_exists("name", $this->info)) && ($looponce == 1)) ) {
$output[] = $this->display_errors('error_system_name');
$looponce++;
}
}

That is my loop that i check things in. What i'm more interested in is the name if statement.

If currently checks for a name key in an array(below) and makes sure that it is set to required. If it's not set to required, print out a system error and die()'s.

I'm looking for ways to remove the need for program errors and just change them to what is needed to run.

What i know how to do is, get into the inner array, what i don't know is how to edit the data and put it back exactly as it was given to me. The inner array data can be put back in any order, but the outer array must be in exact order as it was given.


above code $this->info = $formdata;

$formdata = array(
'name' => array('name'=>"Full Name", 'required'=>false, 'type'=>'text'), # This needs to be required=>true, but i can't trust the user, which is why i have the error.
'telephone' => array('name'=>"Telephone", 'required'=>false, 'type'=>'phone'),
);


Any help is greatly appreciated, also am i doing the foreach loop in the code above in an efficient manner or is there another way?