Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

html output oscid error


yanarasod

Recommended Posts

hi, i have a error with & entity in the url, whereas it should be & , i found a contribution to correct this error, but am worried , is it going to change or damae anyother thing, this is the readme

 

 

Released under the GPL

2004/04/19, use for osC2.2MS2 only!

 

For better validation (validator.w3.org) the next two changes are made to "/includes/functions/html_output.php".

One change to "includes/functions/general.php" is required to keep all functionality.

 

You can copy the included version over yours, or make the changes below yourself, if you like.

 

1) "includes/functions/html_output.php"

-before (line 69-70):

code:

return $link;

}

 

-insert (line 68):

code:

$link = str_replace('&', '&', $link);

 

If doing the str_replace with the &'s in tep_href_link, you will not be able to add any products to the cart with cookies disabled.

This is because of the tep_redirect and '&' not working on a header redirect. Here is the fix:

 

 

2) "includes/functions/general.php"

- near the top find:

Code:

function tep_redirect($url) {

 

- replace with (change is addition of one line):

Code:

function tep_redirect($url) {

$url = str_replace('&', '&', $url);

 

 

3) "/includes/functions/html_output.php"

- change (line 121):

code:

$image_submit = '<input type="image" src="' . tep_output_string(DIR_WS_LANGUAGES . $language . '/images/buttons/' . $image) . '" border="0" alt="' . tep_output_string($alt) . '"';

 

- to:

code:

$image_submit = '<input type="image" src="' . tep_output_string(DIR_WS_LANGUAGES . $language . '/images/buttons/' . $image) . '" style="border-width:0px;border-collapse:collapse;" alt="' . tep_output_string($alt) . '"';

 

(border="0" is changed to style="border-width:0px;border-collapse:collapse;")

 

4) removed one <tr> from "includes/modules/upcoming_products" (file included)

 

 

 

 

 

 

 

if anyone can help it would be fine. thanks

Link to comment
Share on other sites

Archived

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

×
×
  • Create New...