Customer error in array

[Paycheck]

When I added this contribution to the customers.php page which allowed me to sort by name, date added etc. I am guessing this has messed up some logic in the admin/customers.php page and the way it looks at the array. Here is the old customers.php page that did not work.

 

<?php
/*
 $Id: customers.php,v 1.82 2003/06/30 13:54:14 dgw_ Exp $
 ####### WITH Customer Sort ################
 osCommerce, Open Source E-Commerce Solutions
 http://www.oscommerce.com

 Copyright (c) 2003 osCommerce

 Released under the GNU General Public License
*/

 require('includes/application_top.php');

 $action = (isset($HTTP_GET_VARS['action']) ? $HTTP_GET_VARS['action'] : '');

 $error = false;
 $processed = false;

 if (tep_not_null($action)) {
   switch ($action) {
     case 'update':
       $customers_id = tep_db_prepare_input($HTTP_GET_VARS['cID']);
       $customers_firstname = tep_db_prepare_input($HTTP_POST_VARS['customers_firstname']);
       $customers_lastname = tep_db_prepare_input($HTTP_POST_VARS['customers_lastname']);
       $customers_email_address = tep_db_prepare_input($HTTP_POST_VARS['customers_email_address']);
       $customers_telephone = tep_db_prepare_input($HTTP_POST_VARS['customers_telephone']);
       $customers_fax = tep_db_prepare_input($HTTP_POST_VARS['customers_fax']);
       $customers_newsletter = tep_db_prepare_input($HTTP_POST_VARS['customers_newsletter']);

       $customers_gender = tep_db_prepare_input($HTTP_POST_VARS['customers_gender']);
       $customers_dob = tep_db_prepare_input($HTTP_POST_VARS['customers_dob']);

       $default_address_id = tep_db_prepare_input($HTTP_POST_VARS['default_address_id']);
       $entry_street_address = tep_db_prepare_input($HTTP_POST_VARS['entry_street_address']);
       $entry_suburb = tep_db_prepare_input($HTTP_POST_VARS['entry_suburb']);
       $entry_postcode = tep_db_prepare_input($HTTP_POST_VARS['entry_postcode']);
       $entry_city = tep_db_prepare_input($HTTP_POST_VARS['entry_city']);
       $entry_country_id = tep_db_prepare_input($HTTP_POST_VARS['entry_country_id']);

       $entry_company = tep_db_prepare_input($HTTP_POST_VARS['entry_company']);
       $entry_state = tep_db_prepare_input($HTTP_POST_VARS['entry_state']);
       if (isset($HTTP_POST_VARS['entry_zone_id'])) $entry_zone_id = tep_db_prepare_input($HTTP_POST_VARS['entry_zone_id']);

       if (strlen($customers_firstname) < ENTRY_FIRST_NAME_MIN_LENGTH) {
         $error = true;
         $entry_firstname_error = true;
       } else {
         $entry_firstname_error = false;
       }

       if (strlen($customers_lastname) < ENTRY_LAST_NAME_MIN_LENGTH) {
         $error = true;
         $entry_lastname_error = true;
       } else {
         $entry_lastname_error = false;
       }

       if (ACCOUNT_DOB == 'true') {
         if (checkdate(substr(tep_date_raw($customers_dob), 4, 2), substr(tep_date_raw($customers_dob), 6, 2), substr(tep_date_raw($customers_dob), 0, 4))) {
           $entry_date_of_birth_error = false;
         } else {
           $error = true;
           $entry_date_of_birth_error = true;
         }
       }

       if (strlen($customers_email_address) < ENTRY_EMAIL_ADDRESS_MIN_LENGTH) {
         $error = true;
         $entry_email_address_error = true;
       } else {
         $entry_email_address_error = false;
       }

       if (!tep_validate_email($customers_email_address)) {
         $error = true;
         $entry_email_address_check_error = true;
       } else {
         $entry_email_address_check_error = false;
       }

       if (strlen($entry_street_address) < ENTRY_STREET_ADDRESS_MIN_LENGTH) {
         $error = true;
         $entry_street_address_error = true;
       } else {
         $entry_street_address_error = false;
       }

       if (strlen($entry_postcode) < ENTRY_POSTCODE_MIN_LENGTH) {
         $error = true;
         $entry_post_code_error = true;
       } else {
         $entry_post_code_error = false;
       }

       if (strlen($entry_city) < ENTRY_CITY_MIN_LENGTH) {
         $error = true;
         $entry_city_error = true;
       } else {
         $entry_city_error = false;
       }

       if ($entry_country_id == false) {
         $error = true;
         $entry_country_error = true;
       } else {
         $entry_country_error = false;
       }

       if (ACCOUNT_STATE == 'true') {
         if ($entry_country_error == true) {
           $entry_state_error = true;
         } else {
           $zone_id = 0;
           $entry_state_error = false;
           $check_query = tep_db_query("select count(*) as total from " . TABLE_ZONES . " where zone_country_id = '" . (int)$entry_country_id . "'");
           $check_value = tep_db_fetch_array($check_query);
           $entry_state_has_zones = ($check_value['total'] > 0);
           if ($entry_state_has_zones == true) {
             $zone_query = tep_db_query("select zone_id from " . TABLE_ZONES . " where zone_country_id = '" . (int)$entry_country_id . "' and zone_name = '" . tep_db_input($entry_state) . "'");
             if (tep_db_num_rows($zone_query) == 1) {
               $zone_values = tep_db_fetch_array($zone_query);
               $entry_zone_id = $zone_values['zone_id'];
             } else {
               $error = true;
               $entry_state_error = true;
             }
           } else {
             if ($entry_state == false) {
               $error = true;
               $entry_state_error = true;
             }
           }
        }
     }

     if (strlen($customers_telephone) < ENTRY_TELEPHONE_MIN_LENGTH) {
       $error = true;
       $entry_telephone_error = true;
     } else {
       $entry_telephone_error = false;
     }

     $check_email = tep_db_query("select customers_email_address from " . TABLE_CUSTOMERS . " where customers_email_address = '" . tep_db_input($customers_email_address) . "' and customers_id != '" . (int)$customers_id . "'");
     if (tep_db_num_rows($check_email)) {
       $error = true;
       $entry_email_address_exists = true;
     } else {
       $entry_email_address_exists = false;
     }

     if ($error == false) {

       $sql_data_array = array('customers_firstname' => $customers_firstname,
                               'customers_lastname' => $customers_lastname,
                               'customers_email_address' => $customers_email_address,
                               'customers_telephone' => $customers_telephone,
                               'customers_fax' => $customers_fax,
                               'customers_newsletter' => $customers_newsletter);

       if (ACCOUNT_GENDER == 'true') $sql_data_array['customers_gender'] = $customers_gender;
       if (ACCOUNT_DOB == 'true') $sql_data_array['customers_dob'] = tep_date_raw($customers_dob);

       tep_db_perform(TABLE_CUSTOMERS, $sql_data_array, 'update', "customers_id = '" . (int)$customers_id . "'");

       tep_db_query("update " . TABLE_CUSTOMERS_INFO . " set customers_info_date_account_last_modified = now() where customers_info_id = '" . (int)$customers_id . "'");

       if ($entry_zone_id > 0) $entry_state = '';

       $sql_data_array = array('entry_firstname' => $customers_firstname,
                               'entry_lastname' => $customers_lastname,
                               'entry_street_address' => $entry_street_address,
                               'entry_postcode' => $entry_postcode,
                               'entry_city' => $entry_city,
                               'entry_country_id' => $entry_country_id);

       if (ACCOUNT_COMPANY == 'true') $sql_data_array['entry_company'] = $entry_company;
       if (ACCOUNT_SUBURB == 'true') $sql_data_array['entry_suburb'] = $entry_suburb;

       if (ACCOUNT_STATE == 'true') {
         if ($entry_zone_id > 0) {
           $sql_data_array['entry_zone_id'] = $entry_zone_id;
           $sql_data_array['entry_state'] = '';
         } else {
           $sql_data_array['entry_zone_id'] = '0';
           $sql_data_array['entry_state'] = $entry_state;
         }
       }

       tep_db_perform(TABLE_ADDRESS_BOOK, $sql_data_array, 'update', "customers_id = '" . (int)$customers_id . "' and address_book_id = '" . (int)$default_address_id . "'");

       tep_redirect(tep_href_link(FILENAME_CUSTOMERS, tep_get_all_get_params(array('cID', 'action')) . 'cID=' . $customers_id));

       } else if ($error == true) {
         $cInfo = new objectInfo($HTTP_POST_VARS);
         $processed = true;
       }

       break;
     case 'deleteconfirm':
       $customers_id = tep_db_prepare_input($HTTP_GET_VARS['cID']);

       if (isset($HTTP_POST_VARS['delete_reviews']) && ($HTTP_POST_VARS['delete_reviews'] == 'on')) {
         $reviews_query = tep_db_query("select reviews_id from " . TABLE_REVIEWS . " where customers_id = '" . (int)$customers_id . "'");
         while ($reviews = tep_db_fetch_array($reviews_query)) {
           tep_db_query("delete from " . TABLE_REVIEWS_DESCRIPTION . " where reviews_id = '" . (int)$reviews['reviews_id'] . "'");
         }

 

Not sure what is happening and why but any help is appreciated. Again I installed a contribution that allows the customers to be sorted via date or name or join date. It is messing with the array however in admin/includes/object_info.php on lines 17 and 18 where the array is called.

 

JM