Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

How can I remove white spaces from end of code?


wvmlt

Recommended Posts

Posted

I get a parse error for the last line of this code

 

<?php

/*

 $Id: account_edit_process.php,v 1.2 2002/11/28 23:39:44 wilt Exp $



 osCommerce, Open Source E-Commerce Solutions

 http://www.oscommerce.com



 Copyright (c) 2002 osCommerce



 Released under the GNU General Public License

*/



 require('includes/application_top.php');



 if (!tep_session_is_registered('customer_id')) {

   $navigation->set_snapshot(array('mode' => 'SSL', 'page' => FILENAME_ACCOUNT_EDIT));

   tep_redirect(tep_href_link(FILENAME_LOGIN, '', 'SSL'));

 }



 if ($HTTP_POST_VARS['action'] != 'process') {

   tep_redirect(tep_href_link(FILENAME_ACCOUNT_EDIT, '', 'SSL'));

 }



 $gender = tep_db_prepare_input($HTTP_POST_VARS['gender']);

 $firstname = tep_db_prepare_input($HTTP_POST_VARS['firstname']);

 $lastname = tep_db_prepare_input($HTTP_POST_VARS['lastname']);

 $dob = tep_db_prepare_input($HTTP_POST_VARS['dob']);

 $email_address = tep_db_prepare_input($HTTP_POST_VARS['email_address']);

 $telephone = tep_db_prepare_input($HTTP_POST_VARS['telephone']);

 $fax = tep_db_prepare_input($HTTP_POST_VARS['fax']);

 $newsletter = tep_db_prepare_input($HTTP_POST_VARS['newsletter']);

 $password = tep_db_prepare_input($HTTP_POST_VARS['password']);

 $confirmation = tep_db_prepare_input($HTTP_POST_VARS['confirmation']);

 $street_address = tep_db_prepare_input($HTTP_POST_VARS['street_address']);

 $company = tep_db_prepare_input($HTTP_POST_VARS['company']);

 $suburb = tep_db_prepare_input($HTTP_POST_VARS['suburb']);

 $postcode = tep_db_prepare_input($HTTP_POST_VARS['postcode']);

 $city = tep_db_prepare_input($HTTP_POST_VARS['city']);

 $zone_id = tep_db_prepare_input($HTTP_POST_VARS['zone_id']);

 $state = tep_db_prepare_input($HTTP_POST_VARS['state']);

 $country = tep_db_prepare_input($HTTP_POST_VARS['country']);



 $error = false; // reset error flag



 if (ACCOUNT_GENDER == 'true') {

   if (($gender == 'm') || ($gender == 'f')) {

     $entry_gender_error = false;

   } else {

     $error = true;

     $entry_gender_error = true;

   }

 }



 if (strlen($firstname) < ENTRY_FIRST_NAME_MIN_LENGTH) {

   $error = true;

   $entry_firstname_error = true;

 } else {

   $entry_firstname_error = false;

 }



 if (strlen($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($dob), 4, 2), substr(tep_date_raw($dob), 6, 2), substr(tep_date_raw($dob), 0, 4))) {

     $entry_date_of_birth_error = false;

   } else {

     $error = true;

     $entry_date_of_birth_error = true;

   }

 }



 if (strlen($email_address) < ENTRY_EMAIL_ADDRESS_MIN_LENGTH) {

   $error = true;

   $entry_email_address_error = true;

 } else {

   $entry_email_address_error = false;

 }



 if (!tep_validate_email($email_address)) {

   $error = true;

   $entry_email_address_check_error = true;

 } else {

   $entry_email_address_check_error = false;

 }



 if (strlen($street_address) < ENTRY_STREET_ADDRESS_MIN_LENGTH) {

   $error = true;

   $entry_street_address_error = true;

 } else {

   $entry_street_address_error = false;

 }



 if (strlen($postcode) < ENTRY_POSTCODE_MIN_LENGTH) {

   $error = true;

   $entry_post_code_error = true;

 } else {

   $entry_post_code_error = false;

 }



 if (strlen($city) < ENTRY_CITY_MIN_LENGTH) {

   $error = true;

   $entry_city_error = true;

 } else {

   $entry_city_error = false;

 }



 if (!is_numeric($country)) {

   $error = true;

   $entry_country_error = true;

 } else {

   $entry_country_error = false;

 }



 if (ACCOUNT_STATE == 'true') {

   if ($entry_country_error) {

     $entry_state_error = true;

   } else {

     $zone_id = 0;

     $entry_state_error = false;

     $country_check_query = tep_db_query("select count(*) as total from " . TABLE_ZONES . " where zone_country_id = '" . tep_db_input($country) . "'");

     $country_check = tep_db_fetch_array($country_check_query);

     if ($entry_state_has_zones = ($country_check['total'] > 0)) {

       $match_zone_query = tep_db_query("select zone_id from " . TABLE_ZONES . " where zone_country_id = '" . tep_db_input($country) . "' and zone_name = '" . tep_db_input($state) . "'");

       if (tep_db_num_rows($match_zone_query) == 1) {

         $match_zone = tep_db_fetch_array($match_zone_query);

         $zone_id = $match_zone['zone_id'];

       } else {

         $match_zone_query = tep_db_query("select zone_id from " . TABLE_ZONES . " where zone_country_id = '" . tep_db_input($country) . "' and zone_code = '" . tep_db_input($state) . "'");

         if (tep_db_num_rows($match_zone_query) == 1) {

           $match_zone = tep_db_fetch_array($match_zone_query);

           $zone_id = $match_zone['zone_id'];

         } else {

           $error = true;

           $entry_state_error = true;

         }

       }

     } elseif (strlen($state) < ENTRY_STATE_MIN_LENGTH) {

       $error = true;

       $entry_state_error = true;

     }

   }

 }



 if (strlen($telephone) < ENTRY_TELEPHONE_MIN_LENGTH) {

   $error = true;

   $entry_telephone_error = true;

 } else {

   $entry_telephone_error = false;

 }



 if (strlen($password) < ENTRY_PASSWORD_MIN_LENGTH) {

   $error = true;

   $entry_password_error = true;

 } else {

   $entry_password_error = false;

 }



 if ($password != $confirmation) {

   $error = true;

   $entry_password_error = true;

 }



 $check_email_query = tep_db_query("select count(*) as total from " . TABLE_CUSTOMERS . " where customers_email_address = '" . tep_db_input($email_address) . "' and customers_id != '" . tep_db_input($customer_id) . "'");

 $check_email = tep_db_fetch_array($check_email_query);

 if ($check_email['total'] > 0) {

   $error = true;

   $entry_email_address_exists = true;

 } else {

   $entry_email_address_exists = false;

 }



 if ($error == true) {

   $processed = true;



   include(DIR_WS_LANGUAGES . $language . '/' . FILENAME_ACCOUNT_EDIT_PROCESS);



   $breadcrumb->add(NAVBAR_TITLE_1, tep_href_link(FILENAME_ACCOUNT, '', 'SSL'));

   $breadcrumb->add(NAVBAR_TITLE_2, tep_href_link(FILENAME_ACCOUNT_EDIT, '', 'SSL'));

?>

<!doctype html public "-//W3C//DTD HTML 4.01 Transitional//EN">

<html <?php echo HTML_PARAMS; ?>>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=<?php echo CHARSET; ?>">

<?php

// BOF: WebMakers.com Changed: Header Tag Controller v1.0

// Replaced by header_tags.php

if ( file_exists(DIR_WS_INCLUDES . 'header_tags.php') ) {

 require(DIR_WS_INCLUDES . 'header_tags.php');

} else {

?>

 <title><?php echo TITLE ?></title>

<?php

}

// EOF: WebMakers.com Changed: Header Tag Controller v1.0

?>

<base href="<?php echo (getenv('HTTPS') == 'on' ? HTTPS_SERVER : HTTP_SERVER) . DIR_WS_CATALOG; ?>">

<link rel="stylesheet" type="text/css" href="<? echo THEMA_STYLE;?>">

<?php require('includes/form_check.js.php'); ?>

</head>

<body marginwidth="0" marginheight="0" topmargin="0" bottommargin="0" leftmargin="0" rightmargin="0">

<!-- header //-->

<?php require(DIR_WS_CLASSES . 'thema/' . SITE_THEMA . '/thema_header.php'); ?>

<!-- header_eof //-->



<!-- body //-->

<center>

<table border="0" width="<?php echo SITE_WIDTH; ?>" cellspacing="0" cellpadding="13" class="centerTable">

 <tr>

   <td width="<?php echo BOX_WIDTH; ?>" valign="top" class="leftColumn"><table border="0" width="<?php echo BOX_WIDTH; ?>" cellspacing="0" cellpadding="2">

<!-- left_navigation //-->

<?php require(DIR_WS_CLASSES . 'thema/' . SITE_THEMA . '/thema_column_left.php'); ?>

<!-- left_navigation_eof //-->

   </table></td>

<!-- body_text //-->

   <td width="100%" valign="top"><?php echo tep_draw_form('account_edit', tep_href_link(FILENAME_ACCOUNT_EDIT_PROCESS, '', 'SSL'), 'post', 'onSubmit="return check_form();"') . tep_draw_hidden_field('action', 'process'); ?><table border="0" width="100%" cellspacing="0" cellpadding="0">

     <tr>

       <td><table border="0" width="100%" cellspacing="0" cellpadding="0">

<?php if (DISPLAY_HEADER_IMAGE =='true') { ?>

           <td class="pageHeading"><?php echo HEADING_TITLE; ?></td>

           <td class="pageHeading" align="right"><?php echo tep_image(DIR_WS_IMAGES . 'table_background_account.gif', HEADING_TITLE, HEADING_IMAGE_WIDTH, HEADING_IMAGE_HEIGHT); ?></td>

         </tr>

       </table></td>

     </tr>

     <tr>

       <td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>

     </tr>

     <tr>

       <td><?php include(DIR_WS_MODULES . 'account_details.php'); ?></td>

     </tr>

     <tr>

       <td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>

     </tr>

     <tr>

       <td><table border="0" width="100%" cellspacing="0" cellpadding="2">

         <tr>

           <td class="main"><?php echo '<a href="' . tep_href_link(FILENAME_ACCOUNT, '', 'SSL') . '">' . tep_image_button('button_back.gif', IMAGE_BUTTON_BACK) . '</a>'; ?></td>

           <td class="main" align="right"><?php echo tep_image_submit('button_continue.gif', IMAGE_BUTTON_CONTINUE); ?></td>

         </tr>

       </table></td>

     </tr>

   </table></form></td>

<!-- body_text_eof //-->

   <td width="<?php echo BOX_WIDTH; ?>" valign="top"><table border="0" width="<?php echo BOX_WIDTH; ?>" cellspacing="0" cellpadding="2">

<!-- right_navigation //-->

<?php include(DIR_WS_INCLUDES . 'column_right.php'); ?>

<!-- right_navigation_eof //-->

   </table></td>

 </tr>

</table>

<!-- body_eof //-->



<!-- footer //-->

<?php include(DIR_WS_INCLUDES . 'footer.php'); ?>

<!-- footer_eof //-->

<br>

</body>

</html>

<?php

 } else {

   $sql_data_array = array('customers_firstname' => $firstname,

                           'customers_lastname' => $lastname,

                           'customers_email_address' => $email_address,

                           'customers_telephone' => $telephone,

                           'customers_fax' => $fax,

                           'customers_newsletter' => $newsletter,

                           'customers_password' => crypt_password($password));



   if (ACCOUNT_GENDER == 'true') $sql_data_array['customers_gender'] = $gender;

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



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



   $sql_data_array = array('entry_street_address' => $street_address,

                           'entry_firstname' => $firstname,

                           'entry_lastname' => $lastname,

                           'entry_postcode' => $postcode,

                           'entry_city' => $city,

                           'entry_country_id' => $country);



   if (ACCOUNT_GENDER == 'true') $sql_data_array['entry_gender'] = $gender;

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

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

   if (ACCOUNT_STATE == 'true') {

     if ($zone_id > 0) {

       $sql_data_array['entry_zone_id'] = $zone_id;

       $sql_data_array['entry_state'] = '';

     } else {

       $sql_data_array['entry_zone_id'] = '0';

       $sql_data_array['entry_state'] = $state;

     }

   }



   tep_db_perform(TABLE_ADDRESS_BOOK, $sql_data_array, 'update', "customers_id = '" . tep_db_input($customer_id) . "' and address_book_id = '" . tep_db_input($customer_default_address_id) . "'");



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



   $customer_country_id = $country;

   $customer_zone_id = $zone_id;



   tep_redirect(tep_href_link(FILENAME_ACCOUNT, '', 'SSL'));

 }



require(DIR_WS_INCLUDES . 'application_bottom.php');

?>

 

How do I remove the white spaces following the last "?>" ??

Keith

 

 

What the hell was I thinkin'?

Posted

Reason is not at end of file. You have unclused 'if' operator:

 

 if ($error == true) {  

   $processed = true;

 

i think it must be

 

 if ($error == true) {  

   $processed = true;  

 }

There is always more than one way to do it.

And always Keep It Simple, Stupid.

Posted

UNCLOSED i mean :)

There is always more than one way to do it.

And always Keep It Simple, Stupid.

Posted

no, that is wrong. it is closed after all the html. notice...

</html> 

<?php 

 } else { 

   $sql_data_array

 

so you probably do have extra spaces somewhere. a good text editor will be able to make visible the spaces and tabs.

 
Posted

Don't sure, but Zend shows that

  if ($error == true) {

line is have unmatched bracked.

</html> 

<?php 

 } else { 

   $sql_data_array

match to

<?php if (DISPLAY_HEADER_IMAGE =='true') { ?>

There is always more than one way to do it.

And always Keep It Simple, Stupid.

Posted
Reason is not at end of file. You have unclused 'if' operator:

 

Quote:

if ($error == true) {

$processed = true;

 

 

i think it must be

 

Quote:

if ($error == true) {

$processed = true;

}

 

After putting in the "}" i get the following error

 

Warning: Cannot modify header information - headers already sent by (output started at /home/virtual/site118/fst/var/www/html/account_edit_process.php:188) in /home/virtual/site118/fst/var/www/html/includes/functions/general.php on line 23

Keith

 

 

What the hell was I thinkin'?

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...