Guest Posted May 22, 2003 Posted May 22, 2003 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
Guest Posted May 22, 2003 Posted May 22, 2003 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
rseigel Posted May 22, 2003 Posted May 22, 2003 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:
Guest Posted May 22, 2003 Posted May 22, 2003 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:
rseigel Posted May 22, 2003 Posted May 22, 2003 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:
Guest Posted May 22, 2003 Posted May 22, 2003 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)
Guest Posted May 22, 2003 Posted May 22, 2003 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 ?>
rseigel Posted May 22, 2003 Posted May 22, 2003 If I meet you somewhere it'll be to shove my foot up your ass tough guy. :roll:
Guest Posted May 22, 2003 Posted May 22, 2003 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
Guest Posted May 22, 2003 Posted May 22, 2003 macro, i PM'd you a copy of the working file have fun :wink:
loxly Posted May 22, 2003 Posted May 22, 2003 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]
Recommended Posts
Archived
This topic is now archived and is closed to further replies.