Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

Strange error on login.php


Cowardino

Recommended Posts

Hi

 

Regarding the page login.php

 

In Firefox the login button is working well, however when trying with IE the button does not work. Just like there is no link from the button.

Usually on OsCommerce the link from the login button is showing in the bottom of the browser, but in this case the link is not visible in the bottom of the browser when hovering over the button.

 

Just to check I uploaded the login.php from another OSC site I have and in the case the new OSC site's login works well. But I can not use the older login.php file as it is with another design.

 

I am not that php savvy, but I did try to compare the two files, but nothing seems to differ reg the button codes.

 

Does any of you have any suggestion?

 

Best

Jens

Link to comment
Share on other sites

Here is the login.php that came with this OSC, and this one does not have a functioning login button/link in IE, but ok in FF

 

<?php

 require('includes/application_top.php');

// redirect the customer to a friendly cookie-must-be-enabled page if cookies are disabled (or the session has not started)
 if ($session_started == false) {
   tep_redirect(tep_href_link(FILENAME_COOKIE_USAGE));
 }

 require(DIR_WS_LANGUAGES . $language . '/' . FILENAME_LOGIN);

 $error = false;
 if (isset($HTTP_GET_VARS['action']) && ($HTTP_GET_VARS['action'] == 'process')) {
   $email_address = tep_db_prepare_input($HTTP_POST_VARS['email_address']);
   $password = tep_db_prepare_input($HTTP_POST_VARS['password']);

// Check if email exists
   $check_customer_query = tep_db_query("select customers_id, customers_firstname, customers_password, customers_email_address, customers_default_address_id from " . TABLE_CUSTOMERS . " where customers_email_address = '" . tep_db_input($email_address) . "'");
   if (!tep_db_num_rows($check_customer_query)) {
     $error = true;
   } else {
     $check_customer = tep_db_fetch_array($check_customer_query);
// Check that password is good
     if (!tep_validate_password($password, $check_customer['customers_password'])) {
       $error = true;
     } else {
       if (SESSION_RECREATE == 'True') {
         tep_session_recreate();
       }

       $check_country_query = tep_db_query("select entry_country_id, entry_zone_id from " . TABLE_ADDRESS_BOOK . " where customers_id = '" . (int)$check_customer['customers_id'] . "' and address_book_id = '" . (int)$check_customer['customers_default_address_id'] . "'");
       $check_country = tep_db_fetch_array($check_country_query);

       $customer_id = $check_customer['customers_id'];
       $customer_default_address_id = $check_customer['customers_default_address_id'];
       $customer_first_name = $check_customer['customers_firstname'];
       $customer_country_id = $check_country['entry_country_id'];
       $customer_zone_id = $check_country['entry_zone_id'];
       tep_session_register('customer_id');
       tep_session_register('customer_default_address_id');
       tep_session_register('customer_first_name');
       tep_session_register('customer_country_id');
       tep_session_register('customer_zone_id');

       tep_db_query("update " . TABLE_CUSTOMERS_INFO . " set customers_info_date_of_last_logon = now(), customers_info_number_of_logons = customers_info_number_of_logons+1 where customers_info_id = '" . (int)$customer_id . "'");

// restore cart contents
       $cart->restore_contents();

       if (sizeof($navigation->snapshot) > 0) {
         $origin_href = tep_href_link($navigation->snapshot['page'], tep_array_to_string($navigation->snapshot['get'], array(tep_session_name())), $navigation->snapshot['mode']);
         $navigation->clear_snapshot();
         tep_redirect($origin_href);
       } else {
         tep_redirect(tep_href_link(FILENAME_DEFAULT));
       }
     }
   }
 }

 if ($error == true) {
   $messageStack->add('login', TEXT_LOGIN_ERROR);
 }

 $breadcrumb->add(NAVBAR_TITLE, tep_href_link(FILENAME_LOGIN, '', '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; ?>">
<title><?php echo TITLE; ?></title>
<base href="<?php echo (($request_type == 'SSL') ? HTTPS_SERVER : HTTP_SERVER) . DIR_WS_CATALOG; ?>">
<link rel="stylesheet" type="text/css" href="stylesheet.css">
<script language="javascript"><!--
function session_win() {
 window.open("<?php echo tep_href_link(FILENAME_INFO_SHOPPING_CART); ?>","info_shopping_cart","height=460,width=430,toolbar=no,statusbar=no,scrollbars=yes").focus();
}
//--></script>
</head>
<body>
<!-- header //-->
<?php require(DIR_WS_INCLUDES . 'header.php'); ?>
<!-- header_eof //-->

<!-- body //-->
<table width="1004" border="0" cellspacing="0" cellpadding="0" align="center" style="background-image:url(images/centertabbble.gif)">
 <tr>
   <td>
<table border="0" width="1004" align="center" cellspacing="0" cellpadding="0" style="background-image:url(images/h001.jpg); background-position:top; background-repeat:no-repeat  ">
 <tr>
   <td width="<?php echo BOX_WIDTH; ?>" valign="top">
<!-- left_navigation //-->
<?php require(DIR_WS_INCLUDES . 'column_left.php'); ?>
<!-- left_navigation_eof //-->    </td>
   <td width="100%" valign="top"><table width="100%"  border="0" cellspacing="0" cellpadding="0">
     <tr>
       <td> </td>
     </tr>
     <tr>
       <!-- body_text //-->
       <td height="100%" valign="top"><?php echo tep_draw_form('login', tep_href_link(FILENAME_LOGIN, 'action=process', 'SSL')); ?>
           <table border="0" width="100%" cellspacing="0" cellpadding="0">
             <tr>
               <td><table border="0" width="100%" cellspacing="0" cellpadding="0">
                   <tr>
                     <td class="pageHeading"><table width="100%"  border="0" cellspacing="0" cellpadding="0">
             <tr>
               <td height="20"><table width="100%"  border="0" cellspacing="0" cellpadding="0" style="margin-left:0px ">
                 <tr align="left" valign="middle">
                   <td width="26"><img src="images/lupa.jpg" width="21" height="18"></td>
                   <td width="140" class="searchz"><strong><?php echo BOX_HEADING_SEARCH ?></strong></td>
                   <td width="140"><?php include(DIR_WS_BOXES . 'manufacturers.php'); ?></td>
                   <td width="10"></td>
                   <td width="185"><?php include(DIR_WS_BOXES . 'search.php'); ?>
                   </td>
                   <td align="center"><?php echo '<a href="' . tep_href_link(FILENAME_ADVANCED_SEARCH) . '">' . tep_image_button('adw.jpg') . '</a>'; ?></td>
                 </tr>
               </table></td>
             </tr>
             <tr>
               <td height="21"> </td>
             </tr>
           </table><?php echo HEADING_TITLE; ?></td>
                   </tr>
               </table></td>
             </tr>
             <tr>
               <td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
             </tr>
             <?php
 if ($messageStack->size('login') > 0) {
?>
             <tr>
               <td><?php echo $messageStack->output('login'); ?></td>
             </tr>
             <tr>
               <td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
             </tr>
             <?php
 }

 if ($cart->count_contents() > 0) {
?>
             <tr>
               <td class="smallText"><?php echo TEXT_VISITORS_CART; ?></td>
             </tr>
             <tr>
               <td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
             </tr>
             <?php
 }
?>
             <tr>
               <td><table border="0" width="100%" cellspacing="0" cellpadding="2">
                   <tr>
                     <td class="main" width="50%" valign="top"><b><?php echo HEADING_NEW_CUSTOMER; ?></b></td>
                     <td class="main" width="50%" valign="top"><b><?php echo HEADING_RETURNING_CUSTOMER; ?></b></td>
                   </tr>
                   <tr>
                     <td width="50%" height="100%" valign="top"><table border="0" width="100%" height="100%" cellspacing="1" cellpadding="2" class="infoBox">
                         <tr class="infoBoxContents">
                           <td><table border="0" width="100%" height="100%" cellspacing="0" cellpadding="2">
                               <tr>
                                 <td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
                               </tr>
                               <tr>
                                 <td class="main" valign="top"><?php echo TEXT_NEW_CUSTOMER . '<br><br>' . TEXT_NEW_CUSTOMER_INTRODUCTION; ?></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 width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
                                       <td align="right"><?php echo '<a href="' . tep_href_link(FILENAME_CREATE_ACCOUNT, '', 'SSL') . '">' . tep_image_button('button_continue.gif', IMAGE_BUTTON_CONTINUE) . '</a>'; ?></td>
                                       <td width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
                                     </tr>
                                 </table></td>
                               </tr>
                           </table></td>
                         </tr>
                     </table></td>
                     <td width="50%" height="100%" valign="top"><table border="0" width="100%" height="100%" cellspacing="1" cellpadding="2" class="infoBox">
                         <tr class="infoBoxContents">
                           <td>
						<table border="0" width="100%" height="100%" cellspacing="0" cellpadding="2">
                               <tr>
                                 <td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
                               </tr>
                               <tr>
                                 <td class="main" colspan="2"><?php echo TEXT_RETURNING_CUSTOMER; ?></td>
                               </tr>
                               <tr>
                                 <td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
                               </tr>
                               <tr>
                                 <td class="main"><b><?php echo ENTRY_EMAIL_ADDRESS; ?></b></td>
                                 <td class="main"><?php echo tep_draw_input_field('email_address'); ?></td>
                               </tr>
                               <tr>
                                 <td class="main"><b><?php echo ENTRY_PASSWORD; ?></b></td>
                                 <td class="main"><?php echo tep_draw_password_field('password'); ?></td>
                               </tr>
                               <tr>
                                 <td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
                               </tr>
                               <tr>
                                 <td class="smallText" colspan="2"><?php echo '<a href="' . tep_href_link(FILENAME_PASSWORD_FORGOTTEN, '', 'SSL') . '">' . TEXT_PASSWORD_FORGOTTEN . '</a>'; ?></td>
                               </tr>
                               <tr>
                                 <td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>
                               </tr>
                               <tr>
                                 <td colspan="2"><table border="0" width="100%" cellspacing="0" cellpadding="2">
                                     <tr>
                                       <td width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
                                       <td align="right"><?php echo tep_image_submit('button_login.gif', IMAGE_BUTTON_LOGIN); ?></td>
                                       <td width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
                                     </tr>
                                 </table></td>
                               </tr>
                           </table></td>
                         </tr>
                     </table></td>
                   </tr>
               </table></td>
             </tr>
         </table></td>
       <!-- body_text_eof //-->
     </tr>
     <tr>
       <td> </td>
     </tr>
   </table></td>
   <td width="<?php echo BOX_WIDTH2; ?>" align="left" valign="top"><?php require(DIR_WS_INCLUDES . 'column_right.php'); ?></td>
 </tr>
</table>
</td>
 </tr>
</table>

<!-- body_eof //-->

<!-- footer //-->
<?php require(DIR_WS_INCLUDES . 'footer.php'); ?>
<!-- footer_eof //-->
<br>
</body>
</html>
<?php require(DIR_WS_INCLUDES . 'application_bottom.php'); ?>

 

 

 

Here is the code from an old login.php of mine that i tried instead of the file above, and this one works well in both IE anf FF:

 

<?php

 require('includes/application_top.php');

// redirect the customer to a friendly cookie-must-be-enabled page if cookies are disabled (or the session has not started)
 if ($session_started == false) {
   tep_redirect(tep_href_link(FILENAME_COOKIE_USAGE));
 }

 require(DIR_WS_LANGUAGES . $language . '/' . FILENAME_LOGIN);

 $error = false;
 if (isset($HTTP_GET_VARS['action']) && ($HTTP_GET_VARS['action'] == 'process')) {
   $email_address = tep_db_prepare_input($HTTP_POST_VARS['email_address']);
   $password = tep_db_prepare_input($HTTP_POST_VARS['password']);

// Check if email exists
   $check_customer_query = tep_db_query("select customers_id, customers_firstname, customers_password, customers_email_address, customers_default_address_id from " . TABLE_CUSTOMERS . " where customers_email_address = '" . tep_db_input($email_address) . "'");
   if (!tep_db_num_rows($check_customer_query)) {
     $error = true;
   } else {
     $check_customer = tep_db_fetch_array($check_customer_query);
// Check that password is good
     if (!tep_validate_password($password, $check_customer['customers_password'])) {
       $error = true;
     } else {
       if (SESSION_RECREATE == 'True') {
         tep_session_recreate();
       }

       $check_country_query = tep_db_query("select entry_country_id, entry_zone_id from " . TABLE_ADDRESS_BOOK . " where customers_id = '" . (int)$check_customer['customers_id'] . "' and address_book_id = '" . (int)$check_customer['customers_default_address_id'] . "'");
       $check_country = tep_db_fetch_array($check_country_query);

       $customer_id = $check_customer['customers_id'];
       $customer_default_address_id = $check_customer['customers_default_address_id'];
       $customer_first_name = $check_customer['customers_firstname'];
       $customer_country_id = $check_country['entry_country_id'];
       $customer_zone_id = $check_country['entry_zone_id'];
       tep_session_register('customer_id');
       tep_session_register('customer_default_address_id');
       tep_session_register('customer_first_name');
       tep_session_register('customer_country_id');
       tep_session_register('customer_zone_id');

       tep_db_query("update " . TABLE_CUSTOMERS_INFO . " set customers_info_date_of_last_logon = now(), customers_info_number_of_logons = customers_info_number_of_logons+1 where customers_info_id = '" . (int)$customer_id . "'");

// restore cart contents
       $cart->restore_contents();

       if (sizeof($navigation->snapshot) > 0) {
         $origin_href = tep_href_link($navigation->snapshot['page'], tep_array_to_string($navigation->snapshot['get'], array(tep_session_name())), $navigation->snapshot['mode']);
         $navigation->clear_snapshot();
         tep_redirect($origin_href);
       } else {
         tep_redirect(tep_href_link(FILENAME_DEFAULT));
       }
     }
   }
 }

 if ($error == true) {
   $messageStack->add('login', TEXT_LOGIN_ERROR);
 }

 $breadcrumb->add(NAVBAR_TITLE, tep_href_link(FILENAME_LOGIN, '', '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; ?>">
<title><?php echo TITLE; ?></title>
<base href="<?php echo (($request_type == 'SSL') ? HTTPS_SERVER : HTTP_SERVER) . DIR_WS_CATALOG; ?>">
<link rel="stylesheet" type="text/css" href="stylesheet.css">
<script language="javascript"><!--
function session_win() {
 window.open("<?php echo tep_href_link(FILENAME_INFO_SHOPPING_CART); ?>","info_shopping_cart","height=500,width=445,toolbar=no,statusbar=no,scrollbars=yes").focus();
}
//--></script>
</head>
<body marginwidth="0" marginheight="0" topmargin="0" bottommargin="0" leftmargin="0" rightmargin="0">
<!-- header //-->
<?php require(DIR_WS_INCLUDES . 'header.php'); ?>
<!-- header_eof //-->

<!-- body //-->
<table border="0" width="100%" cellspacing="0" cellpadding="0">
 <tr>
   <td class="col_left">
<!-- left_navigation //-->
<?php require(DIR_WS_INCLUDES . 'column_left.php'); ?>
<!-- left_navigation_eof //-->
  </td>
<!-- body_text //-->
   <td width="100%" class="col_center"><?php echo tep_draw_form('login', tep_href_link(FILENAME_LOGIN, 'action=process', 'SSL')); ?><table border="0" width="100%" cellspacing="0" cellpadding="0">
     <tr>
       <td>

<? tep_draw_heading_top();?>

<? new contentBoxHeading_ProdNew($info_box_contents);?>

<? tep_draw_heading_top_1();?>

<?php
 if ($messageStack->size('login') > 0) {
?>

	<table cellpadding="0" cellspacing="0" border="0">
		<tr><td><?php echo $messageStack->output('login'); ?></td></tr>
		<tr><td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td></tr>
	</table>
<?php
 }

 if ($cart->count_contents() > 0) {
?>
     <table cellpadding="0" cellspacing="0" border="0">
  	<tr><td class="smallText padd_1"><?php echo TEXT_VISITORS_CART; ?></td></tr>
	<tr><td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td></tr>
  </table>
<?php
 }
?>


	<table border="0" cellspacing="3" cellpadding="2">
         <tr>
           <td class="main" width="50%" valign="top"><b><?php echo HEADING_NEW_CUSTOMER; ?></b></td>
           <td class="main" width="50%" valign="top"><b><?php echo HEADING_RETURNING_CUSTOMER; ?></b></td>
         </tr>
         <tr>
           <td width="50%" height="100%" valign="top">

		<table border="0" width="100%" height="100%" cellspacing="1" cellpadding="2" class="infoBox">
             <tr class="infoBoxContents">
               <td><table border="0" width="100%" height="100%" cellspacing="4" cellpadding="2">
                 <tr><td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td></tr>
                 <tr><td class="main" valign="top"><?php echo TEXT_NEW_CUSTOMER . '<br><br>' . TEXT_NEW_CUSTOMER_INTRODUCTION; ?></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 width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
						<td align="right"><?php echo '<a href="' . tep_href_link(FILENAME_CREATE_ACCOUNT, '', 'SSL') . '">' . tep_image_button('button_continue.gif', IMAGE_BUTTON_CONTINUE) . '</a>'; ?><br style="line-height:1px;"><br style="line-height:5px;"></td>
						<td width="10"><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
					  </tr>
					</table></td>
				  </tr>
				</table>
			</td></tr>
           </table>

		</td>
           <td width="50%" height="100%" valign="top">

		<table border="0" width="100%" height="100%" cellspacing="1" cellpadding="2" class="infoBox">
             <tr class="infoBoxContents">
               <td><table border="0" width="100%" height="100%" cellspacing="4" cellpadding="2">
                 <tr><td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td></tr>
                 <tr><td class="main" colspan="2"><?php echo TEXT_RETURNING_CUSTOMER; ?></td></tr>
                 <tr><td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td> </tr>
                 <tr><td class="main"><b><?php echo ENTRY_EMAIL_ADDRESS; ?></b><br style="line-height:1px;"><br style="line-height:5px;"><?php echo tep_draw_input_field('email_address'); ?></td></tr>
                 <tr><td class="main"><b><?php echo ENTRY_PASSWORD; ?></b><br style="line-height:1px;"><br style="line-height:5px;"><?php echo tep_draw_password_field('password'); ?></td></tr>
                 <tr><td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td></tr>
                 <tr><td class="smallText" colspan="2"><?php echo '<a href="' . tep_href_link(FILENAME_PASSWORD_FORGOTTEN, '', 'SSL') . '">' . TEXT_PASSWORD_FORGOTTEN . '</a>'; ?></td></tr>
                 <tr><td colspan="2"><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td></tr>
                 <tr><td colspan="2">
				<table border="0" width="100%" cellspacing="0" cellpadding="2">
                     <tr>
                       <td><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
                       <td align="right" class="bg_input"><?php echo tep_image_submit('button_sign_in1.gif', IMAGE_BUTTON_LOGIN); ?><br style="line-height:1px;"><br style="line-height:5px;"></td>
                       <td><?php echo tep_draw_separator('pixel_trans.gif', '10', '1'); ?></td>
                     </tr>
                   </table>
				</td></tr>
               </table></td>
             </tr>
           </table>
		</td>
         </tr>
       </table>

<? tep_draw_heading_bottom_1();?>

<? tep_draw_heading_bottom();?>

	</td>
     </tr>
   </table></form></td>
<!-- body_text_eof //-->
   <td class="col_right">
<!-- right_navigation //-->
<?php require(DIR_WS_INCLUDES . 'column_right.php'); ?>
<!-- right_navigation_eof //-->
   </td>
 </tr>
</table>
<!-- body_eof //-->

<!-- footer //-->
<?php require(DIR_WS_INCLUDES . 'footer.php'); ?>
<!-- footer_eof //--></body>
</html>
<?php require(DIR_WS_INCLUDES . 'application_bottom.php'); ?>

 

I would appreciate any help I can get on this.

 

Best

Jens

Link to comment
Share on other sites

Just found out, by trial and error, that removing this piece of code (search bar) The login button also works in IE

 

<table width="100%"  border="0" cellspacing="0" cellpadding="0" style="margin-left:0px ">
                 <tr align="left" valign="middle">
                   <td width="26"><img src="images/lupa.jpg" width="21" height="18"></td>
                   <td width="140" class="searchz"><strong><?php echo BOX_HEADING_SEARCH ?></strong></td>
                   <td width="140"><?php include(DIR_WS_BOXES . 'manufacturers.php'); ?></td>
                   <td width="10"></td>
                   <td width="185"><?php include(DIR_WS_BOXES . 'search.php'); ?>
                   </td>
                   <td align="center"><?php echo '<a href="' . tep_href_link(FILENAME_ADVANCED_SEARCH) . '">' . tep_image_button('adw.jpg') . '</a>'; ?></td>
                 </tr>
</table>

 

Now the question remains: what exactly in the code above is creating the problem?

 

Best

Jens

Link to comment
Share on other sites

The login button is a "form".

 

Look in the HTML source,

 

Count the <form name=...> tags.

 

Count the </form> tags.

 

They should be equal.

 

My guess is you need to change your original code from this:

 

</table></td>
<!-- body_text_eof //-->

to this:

 

</table></form></td>
<!-- body_text_eof //-->

If I suggest you edit any file(s) make a backup first - I'm not perfect and neither are you.

 

"Given enough impetus a parallelogramatically shaped projectile can egress a circular orifice."

- Me -

 

"Headers already sent" - The definitive help

 

"Cannot redeclare ..." - How to find/fix it

 

SSL Implementation Help

 

Like this post? "Like" it again over there >

Link to comment
Share on other sites

Archived

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

×
×
  • Create New...