I get this error

theprofession

Can anyone help me out? It does everything its supposed to do, but when i get to the process.php that happens.

Warning: Missing argument 4 for updateuserfield() in /home/content/n/o/i/noirphantom/html/lounge/include/database.php on line 160

Warning: Cannot modify header information - headers already sent by (output started at /home/content/n/o/i/noirphantom/html/lounge/include/database.php:160) in /home/content/n/o/i/noirphantom/html/lounge/process.php on line 186

process.php

 /**
    * procEditAccount - Attempts to edit the user's account
    * information, including the password, which must be verified
    * before a change is made.
    */
   function procEditAccount(){
      global $session, $form;
      /* Account edit attempt */
      $retval = $session->editAccount($_POST['curpass'], $_POST['newpass'], $_POST['email'], $_POST['bio']);

  /* Account edit successful */
  if($retval){
     $_SESSION['useredit'] = true;
   line 186->      header("Location: ".$session->referrer);
      }
      /* Error found with form */
      else{
         $_SESSION['value_array'] = $_POST;
         $_SESSION['error_array'] = $form->getErrorArray();
         header("Location: ".$session->referrer);
      }
   }

Database.php



 /**
    * updateUserField - Updates a field, specified by the field
    * parameter, in the user's row of the database.
    */
 line 160 ->  function updateUserField($username, $field, $value, $bio){
      $q = "UPDATE ".TBL_USERS." SET ".$field." = '$value' WHERE username = '$username'";
      return mysql_query($q, $this->connection);
   }

   /**
    * getUserInfo - Returns the result array from a mysql
    * query asking for all information stored regarding
    * the given username. If query fails, NULL is returned.
    */
   function getUserInfo($username){
      $q = "SELECT * FROM ".TBL_USERS." WHERE username = '$username'";
      $result = mysql_query($q, $this->connection);
      /* Error occurred, return given name by default */
      if(!$result || (mysql_numrows($result) < 1)){
         return NULL;
      }
      /* Return result array */
      $dbarray = mysql_fetch_array($result);
      return $dbarray;
   }

   /**
    * getNumMembers - Returns the number of signed-up users
    * of the website, banned members not included. The first
    * time the function is called on page load, the database
    * is queried, on subsequent calls, the stored result
    * is returned. This is to improve efficiency, effectively
    * not querying the database when no call is made.
    */
   function getNumMembers(){
      if($this->num_members < 0){
         $q = "SELECT * FROM ".TBL_USERS;
         $result = mysql_query($q, $this->connection);
         $this->num_members = mysql_numrows($result);
      }
      return $this->num_members;
   }

bradgrafelman

Hmm.. what doe the function declaration look like for the editAccount() function?

theprofession

function updateUserField($username, $field, $value, $bio)

thats line 160

bradgrafelman

Sorry - edited my post after I re-read yours.

theprofession

bradgrafelman wrote:
Hmm.. what doe the function declaration look like for the editAccount() function?

I only see that in the process.php and I posted it.



   function procEditAccount(){
      global $session, $form;
      /* Account edit attempt */
      $retval = $session->editAccount($_POST['curpass'], $_POST['newpass'], $_POST['email'], $_POST['bio']);

  /* Account edit successful */
  if($retval){
     $_SESSION['useredit'] = true;
     header("Location: ".$session->referrer);
  }
  /* Error found with form */
  else{
     $_SESSION['value_array'] = $_POST;
     $_SESSION['error_array'] = $form->getErrorArray();
     header("Location: ".$session->referrer);
  }
   }

bradgrafelman

Right, but I wanted to see the part of your code where you define editAccount(), not just use it.

See, the error message talks about a misuse of the updateUserField() function, but in the code you've provided thus far, that function isn't actually ever used.

theprofession


 /**
    * updateUserField - Updates a field, specified by the field
    * parameter, in the user's row of the database.
    */
   function updateUserField($username, $field, $value, $bio){
      $q = "UPDATE ".TBL_USERS." SET ".$field." = '$value' WHERE username = '$username'";
      return mysql_query($q, $this->connection);
   }

bradgrafelman

That's the function declaration for updateUserField() - we need to see the one for editAccount(), e.g.

function editAccount() {
    // blah blah...

// perhaps a call to updateUserField() in here somewhere?

}

theprofession

i dont see one for that, thats all i got from the database one and the first function i posted was the only i seen that had editaccount in it.

theprofession

theprofession wrote:
i dont see one for that, thats all i got from the database one and the first function i posted was the only i seen that had editaccount in it.

sry bout that i found it, i didnt check all the included files.

 /**
    * editAccount - Attempts to edit the user's account information
    * including the password, which it first makes sure is correct
    * if entered, if so and the new password is in the right
    * format, the change is made. All other fields are changed
    * automatically.
    */
   function editAccount($subcurpass, $subnewpass, $subemail, $bio){
      global $database, $form;  //The database and form object
      /* New password entered */
      if($subnewpass){
         /* Current Password error checking */
         $field = "curpass";  //Use field name for current password
         if(!$subcurpass){
            $form->setError($field, "* Current Password not entered");
         }
         else{
            /* Check if password too short or is not alphanumeric */
            $subcurpass = stripslashes($subcurpass);
            if(strlen($subcurpass) < 4 ||
               !eregi("^([0-9a-z])+$", ($subcurpass = trim($subcurpass)))){
               $form->setError($field, "* Current Password incorrect");
            }
            /* Password entered is incorrect */
            if($database->confirmUserPass($this->username,md5($subcurpass)) != 0){
               $form->setError($field, "* Current Password incorrect");
            }
         }

bradgrafelman

Hrm... if you're sure you showed us the right line numbers in your first post, I'm at a loss as to where the error is...

theprofession

ok the one i just showed you is from my sessions.php where it calls to the edit the profile