Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

At checkout, people get redirected to the advanced search page


grunionfab

Recommended Posts

Posted

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

Posted

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.

Posted
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.

Posted

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
 }
?>

Posted

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.

Posted

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.

Posted

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.

Posted

*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.

Archived

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

×
×
  • Create New...