Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

THEMEOSC


Guest

Recommended Posts

Posted

is it just me or theme osc from http://www.dcpcwebdevelopment.co.uk/themetest/ don't work when a user edit there account they get this error:

 

Parse error: parse error in /homepages/43/d42823247/htdocs/wsc42823248/themetest/account_edit_process.php on line 310

 

 

also there is a lot more errors when you purchase something. Is there a next version of themeosc, i like it every much just that everything seem to be not working

Posted

parse errors are generally caused by random white spaces in the php code

 

check for blank spaces at the start and end of your file

 

<?php

 

?>

 

also, the theme installation can be a little tricky and if you did a copy/paste of code from their instructions page, you definately picked up white spaces that are causing the error

Posted
parse errors are generally caused by random white spaces in the php code

 

Since when? :? You're actually referring to the good old headers already sent error. This is obviously something different.

 

Mike

 

Post 10 lines before and after line 310 of account_edit_process.php (be sure to point out which exact line is 310 as well) and we'll see if we can't find the problem. :wink:

Posted

since always if you pay attention

 

the sending headers is a definate sign of a blank space preceeding or trailing php code improperly

 

parse errors are generated by the same thing most of the time which can cause a line terminator to be incorrectly identified or line breaks to be ignored

 

just a tip :roll:

Posted

Whatever you say Jeff. Just can't admit you're wrong even when you are can you? :roll:

 

Just post the code I asked for Mike and someone (probably not Jeff - he'll be looking for whitespace) will spot the problem. :wink:

Posted

ron, as usual you are so enjoyable to read :roll:

 

Mike, better than posting the code, PM me and then you can send me the actual file and i will go through it and find the problem for you (despite other opinions) 8)

Posted

ok i'm going to post the whole file here

 

 

<?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');

?>

 

line 310 is a line after ?>

Posted

meet me sometime and say that... :wink:

Posted

Macro

 

in copying your file there are 2 things i notice immediately (one of which I was afraid would happen) - when stuff gets copy/pasted from here, inevitably white space characters get picked up and I found a couple at the end of your file but can't really be sure if they are in the file itself or because of the copy/paste

 

the second thing is since you are using the dpc theme contribution (which I dont have installed on this machine) I have to comment out those lines of code

 

so what is happening is I am not getting an error when i go to edit the account, but when i click the Continue button, the page is not refreshing properly, so i will continue to dig into this some more

Posted

macro, i PM'd you a copy of the working file

 

have fun :wink:

Posted

Corners everyone, the bell ending the round has rung.

 

Theme OSC is being revised, it most likely won't work on current snapshots. But hopefully will soon.

[no external urls in signatures please, kthanks]

Archived

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

×
×
  • Create New...