grunionfab Posted May 17, 2007 Posted May 17, 2007 So when you are on the shipping/cc info page and click continue it redirects you to the Advances search page. It was doign the same thing when you selected anything from the manufacturers box and the submit link button. I dont know what I did to mess it up but it must have stopped working around the beginning of the month because i have not gotten any orders since then.... (i was wondering what was oging on....) I think since then I have added Links Manager and have messed with the AJAX attribute sorter..... Please help, i really dont know what i did wrong! http://grunionfabrication.com
abra123cadabra Posted May 17, 2007 Posted May 17, 2007 On that page you have two opening form tags (one for the search in the header and one for the order) but there is only one closing form tag. From what I've seen, it's missing in the search infobox in your header. Another important thing to fix is your create_account2.php. I tried to get through with a standard testing address but it kept on giving me errors about fields not containing enough characters. I tried with a tick in the "use billing for shipping address" but I got stuck there. Only when I entered the same address into the the shipping section, I could continue. abra The First Law of E-Commerce: If the user can't find the product, the user can't buy the product. Feedback and suggestions on my shop welcome. Note: My advice is based on my own experience or on something I read in these forums. No guarantee it'll work for you! Make sure that you always BACKUP the database and the files you are going to change so that you can rollback to a working version if things go wrong.
grunionfab Posted May 17, 2007 Author Posted May 17, 2007 On that page you have two opening form tags (one for the search in the header and one for the order) but there is only one closing form tag. From what I've seen, it's missing in the search infobox in your header. Another important thing to fix is your create_account2.php. I tried to get through with a standard testing address but it kept on giving me errors about fields not containing enough characters. I tried with a tick in the "use billing for shipping address" but I got stuck there. Only when I entered the same address into the the shipping section, I could continue. abra thanks for looking, i was just trying the create_account1.php page, I usually use creat_account1.php which i just changed back to. You can try it now and that page should work at least. I will try to fix what you said about the tags.
grunionfab Posted May 17, 2007 Author Posted May 17, 2007 This is my header code. where is the problem? <?php /* $Id: header.php,v 1.42 2003/06/10 18:20:38 hpdl Exp $ osCommerce, Open Source E-Commerce Solutions [url="http://www.oscommerce.com"]http://www.oscommerce.com[/url] Copyright © 2003 osCommerce Released under the GNU General Public License */ // check if the 'install' directory exists, and warn of its existence if (WARN_INSTALL_EXISTENCE == 'true') { if (file_exists(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/install')) { $messageStack->add('header', WARNING_INSTALL_DIRECTORY_EXISTS, 'warning'); } } // check if the configure.php file is writeable if (WARN_CONFIG_WRITEABLE == 'true') { if ( (file_exists(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/includes/configure.php')) && (is_writeable(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/includes/configure.php')) ) { $messageStack->add('header', WARNING_CONFIG_FILE_WRITEABLE, 'warning'); } } // check if the session folder is writeable if (WARN_SESSION_DIRECTORY_NOT_WRITEABLE == 'true') { if (STORE_SESSIONS == '') { if (!is_dir(tep_session_save_path())) { $messageStack->add('header', WARNING_SESSION_DIRECTORY_NON_EXISTENT, 'warning'); } elseif (!is_writeable(tep_session_save_path())) { $messageStack->add('header', WARNING_SESSION_DIRECTORY_NOT_WRITEABLE, 'warning'); } } } // check session.auto_start is disabled if ( (function_exists('ini_get')) && (WARN_SESSION_AUTO_START == 'true') ) { if (ini_get('session.auto_start') == '1') { $messageStack->add('header', WARNING_SESSION_AUTO_START, 'warning'); } } if ( (WARN_DOWNLOAD_DIRECTORY_NOT_READABLE == 'true') && (DOWNLOAD_ENABLED == 'true') ) { if (!is_dir(DIR_FS_DOWNLOAD)) { $messageStack->add('header', WARNING_DOWNLOAD_DIRECTORY_NON_EXISTENT, 'warning'); } } if ($messageStack->size('header') > 0) { echo $messageStack->output('header'); } ?> <table border="0" width="100%" cellspacing="0" cellpadding="0"> <tr class="header"> <td align="center" td valign="middle"><?php echo '<a href="' . tep_href_link(FILENAME_DEFAULT) . '">' . tep_image(DIR_WS_IMAGES . 'oscommerce.gif', 'Grunion Fabrication') . '</a>'; ?></td> <td align="right" valign="bottom"> </td> </tr> </table> <table width="100%" height="21" border="0" cellpadding="1" cellspacing="0"> <tr class="headerNavigation"> <td width="60%" height="21" class="headerNavigation"><table border="0" width="50%" cellspacing="0" cellpadding="0" align="left"> <tr> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_form('advanced_search', tep_href_link(FILENAME_ADVANCED_SEARCH_RESULT, '', 'NONSSL', false), 'get"') ?></td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_hidden_field('search_in_description','1') . tep_draw_input_field('keywords', '', 'size="10" maxlength="30" style="width: ' . (BOX_WIDTH+50) . 'px"') ?></td> <td class="headerNavigation" valign="top" colspan="3"> </td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo BOX_ADVSEARCH_CAT ?></td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_pull_down_menu('categories_id', tep_get_categories(array(array('id' => '', 'text' => BOX_ADVSEARCH_ALLCAT)))) ?></td> <td class="headerNavigation" valign="top" colspan="3"> </td> <td class="headerNavigation" valign="middle" colspan="3"><center> <?php echo tep_image_submit('button_search.gif', BOX_HEADING_ADVSEARCH); ?> </center></td> </tr> </table> <td width="40%" align="right" class="headerNavigation"><?php if (tep_session_is_registered('customer_id')) { ?><a href="<?php echo tep_href_link(FILENAME_LOGOFF, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_LOGOFF; ?></a> | <a href="<?php if (!tep_session_is_registered('createaccount')) {echo tep_href_link(FILENAME_ACCOUNT, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_MY_ACCOUNT; } else {echo tep_href_link('account_password_new.php', '', 'SSL'); ?><?php echo 'Create Account'; } ?></a> | <?php } else { ?><a href="<?php echo tep_href_link(FILENAME_LOGIN, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_LOGIN; ?></a> | <?php } ?><a href="<?php echo tep_href_link(FILENAME_SHOPPING_CART); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CART_CONTENTS; ?></a> | <a href="<?php echo tep_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a> </td> </tr> </table> <table align="center" border="0" width="100%" cellspacing="0" cellpadding="1"> <tr> <td align="center" class=""><?php // // START advanced search header script insertion // // ?> <?php if ( (!strstr($_SERVER['PHP_SELF'],'checkout.php')) and (!strstr($_SERVER['PHP_SELF'],'create_account.php')) and (!strstr($_SERVER['PHP_SELF'],'shopping_cart.php')) and (!strstr($_SERVER['PHP_SELF'],'advanced_search.php')) and !tep_session_is_registered('customer_id') ) { ?></td> </tr> </table> </table> </form> </td> </tr> </table> </td> </tr> </table> <?php } ?> <?php // // END advanced search header script insertion // // ?> <?php if (isset($HTTP_GET_VARS['error_message']) && tep_not_null($HTTP_GET_VARS['error_message'])) { ?> <table border="0" width="100%" cellspacing="0" cellpadding="2"> <tr class="headerError"> <td class="headerError"><?php echo htmlspecialchars(urldecode($HTTP_GET_VARS['error_message'])); ?></td> </tr> </table> <?php } if (isset($HTTP_GET_VARS['info_message']) && tep_not_null($HTTP_GET_VARS['info_message'])) { ?> <table border="0" width="100%" cellspacing="0" cellpadding="2"> <tr class="headerInfo"> <td class="headerInfo"><?php echo htmlspecialchars($HTTP_GET_VARS['info_message']); ?></td> </tr> </table> <?php } ?>
abra123cadabra Posted May 17, 2007 Posted May 17, 2007 Replace this: <td width="60%" height="21" class="headerNavigation"><table border="0" width="50%" cellspacing="0" cellpadding="0" align="left"> <tr> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_form('advanced_search', tep_href_link(FILENAME_ADVANCED_SEARCH_RESULT, '', 'NONSSL', false), 'get"') ?></td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_hidden_field('search_in_description','1') . tep_draw_input_field('keywords', '', 'size="10" maxlength="30" style="width: ' . (BOX_WIDTH+50) . 'px"') ?></td> <td class="headerNavigation" valign="top" colspan="3"> </td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo BOX_ADVSEARCH_CAT ?></td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_pull_down_menu('categories_id', tep_get_categories(array(array('id' => '', 'text' => BOX_ADVSEARCH_ALLCAT)))) ?></td> <td class="headerNavigation" valign="top" colspan="3"> </td> <td class="headerNavigation" valign="middle" colspan="3"><center> <?php echo tep_image_submit('button_search.gif', BOX_HEADING_ADVSEARCH); ?> </center></td> </tr> </table> with this: <td width="60%" height="21" class="headerNavigation"><?php echo tep_draw_form('advanced_search', tep_href_link(FILENAME_ADVANCED_SEARCH_RESULT, '', 'NONSSL', false), 'get"') ?><table border="0" width="50%" cellspacing="0" cellpadding="0" align="left"> <tr> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_hidden_field('search_in_description','1') . tep_draw_input_field('keywords', '', 'size="10" maxlength="30" style="width: ' . (BOX_WIDTH+50) . 'px"') ?></td> <td class="headerNavigation" valign="top" colspan="3"> </td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo BOX_ADVSEARCH_CAT ?></td> <td class="headerNavigation" valign="middle" colspan="3"><?php echo tep_draw_pull_down_menu('categories_id', tep_get_categories(array(array('id' => '', 'text' => BOX_ADVSEARCH_ALLCAT)))) ?></td> <td class="headerNavigation" valign="top" colspan="3"> </td> <td class="headerNavigation" valign="middle" colspan="3"><center> <?php echo tep_image_submit('button_search.gif', BOX_HEADING_ADVSEARCH); ?> </center></td> </tr> </table></form></td> abra The First Law of E-Commerce: If the user can't find the product, the user can't buy the product. Feedback and suggestions on my shop welcome. Note: My advice is based on my own experience or on something I read in these forums. No guarantee it'll work for you! Make sure that you always BACKUP the database and the files you are going to change so that you can rollback to a working version if things go wrong.
grunionfab Posted May 17, 2007 Author Posted May 17, 2007 you are WONDERFUL! Thank you so much for taking the time to help! I Really appreciate it!
abra123cadabra Posted May 17, 2007 Posted May 17, 2007 Only, that this fixes the checkout pages but will break the normal catalog pages. Try what happens if you remove this condition in your header: // // START advanced search header script insertion // // ?> <?php if ( (!strstr($_SERVER['PHP_SELF'],'checkout.php')) and (!strstr($_SERVER['PHP_SELF'],'create_account.php')) and (!strstr($_SERVER['PHP_SELF'],'shopping_cart.php')) and (!strstr($_SERVER['PHP_SELF'],'advanced_search.php')) and !tep_session_is_registered('customer_id') ) { ?></td> </tr> </table> </table> </form> </td> </tr> </table> </td> </tr> </table> <?php } ?> <?php // // END advanced search header script insertion // // ?> Is this a template you are using? The whole file has lots of nesting errors that you should fix to make sure that most browsers render your pages more or less as intended. abra The First Law of E-Commerce: If the user can't find the product, the user can't buy the product. Feedback and suggestions on my shop welcome. Note: My advice is based on my own experience or on something I read in these forums. No guarantee it'll work for you! Make sure that you always BACKUP the database and the files you are going to change so that you can rollback to a working version if things go wrong.
grunionfab Posted May 17, 2007 Author Posted May 17, 2007 What does that condition do? No i am not using a template.
abra123cadabra Posted May 17, 2007 Posted May 17, 2007 That condition seems to be part of an attempt to hide the search box while on the checkout pages. What it does is, that while not on the checkout pages, it adds the missing closing form tag plus a lot of other tags which I couldn't figure out where they belong to. So unless you want to hide the search during checkout, just remove the whole part quoted in my previous post. With the change that fixed the original problem, this condition is not necessary and would add not needed tags. abra The First Law of E-Commerce: If the user can't find the product, the user can't buy the product. Feedback and suggestions on my shop welcome. Note: My advice is based on my own experience or on something I read in these forums. No guarantee it'll work for you! Make sure that you always BACKUP the database and the files you are going to change so that you can rollback to a working version if things go wrong.
grunionfab Posted May 17, 2007 Author Posted May 17, 2007 Thank you, I removed it and dont really care if it is there during checkout.... Thanks
abra123cadabra Posted May 17, 2007 Posted May 17, 2007 *lol* Glad I could help. abra The First Law of E-Commerce: If the user can't find the product, the user can't buy the product. Feedback and suggestions on my shop welcome. Note: My advice is based on my own experience or on something I read in these forums. No guarantee it'll work for you! Make sure that you always BACKUP the database and the files you are going to change so that you can rollback to a working version if things go wrong.
Recommended Posts
Archived
This topic is now archived and is closed to further replies.