Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

script error on checkout_shipping


Guest

Recommended Posts

when testing on IE, i get the little yellow warning box in the lower left-hand corner that reads:

 

 

Problems with this web page may prevent it from being displayed properly.... etc.

 

Line: 40

Char: 1

Error: Expected '}'

 

 

 

lines 33-41 (line 40 is bold:

$order_total_modules = new order_total;

$order_total_modules->collect_posts();

$order_total_modules->pre_confirmation_check(); */

// if the customer is not logged on, redirect them to the login page

if (!tep_session_is_registered('customer_id')) {

$navigation->set_snapshot();

// tep_redirect(tep_href_link('create_account3.php', '', 'SSL'));

tep_redirect(tep_href_link(FILENAME_CREATE_ACCOUNT, '', 'SSL'));

}

 

simply inserting a } on line 40 produces an error.

Link to comment
Share on other sites

when testing on IE, i get the little yellow warning box in the lower left-hand corner that reads:

Problems with this web page may prevent it from being displayed properly.... etc.

 

Line: 40

Char: 1

Error: Expected '}'

lines 33-41 (line 40 is bold:

simply inserting a } on line 40 produces an error.

 

that's a JS error ... you need to open your source code (not page code) in IE, copy to an editor with line sizes and you'll see around where the error is ... not always in the line mentioned, but somewhere near

 

you can post the 10 lines before and after it here while marking the line in the error message bold

:-)

Monika

 

addicted to writing code ... can't get enough of databases either, LOL!

 

my toolbox: Textpad - Compare and Merge - phpMyAdmin - WS_FTP - Photoshop - How to search the forum

 

Interactive Media Award July 2007 ~ category E-Commerce

my advice on the forum is for free, PMs where you send me work are considered consultation which I charge for ...

Link to comment
Share on other sites

here is the code:

<script language="javascript"><!--

function check_form() {

var error = 0;

var error_message = "Errors have occured during the process of your form.\n\nPlease make the following corrections:\n\n";

var payment_value = null;

if (document.checkout_payment.payment.length) {

for (var i=0; i<document.checkout_payment.payment.length; i++) {

if (document.checkout_payment.payment.checked) {

payment_value = document.checkout_payment.payment.value;

}

}

} else if (document.checkout_payment.payment.checked) {

payment_value = document.checkout_payment.payment.value;

} else if (document.checkout_payment.payment.value) {

payment_value = document.checkout_payment.payment.value;

}

 

 

if (payment_value == null) {

error_message = error_message + "* Please select a payment method for your order.\n";

error = 1;

}

 

if (error == 1 && submitter != 1) {

if (error == 1) {

alert(error_message);

return false;

} else {

return true;

}

}

//--></script>

 

line 40 is bold

Link to comment
Share on other sites

catalog/includes/classes/payment.php line #119

No longer giving free advice. Please place deposit in meter slot provided.  Individual: [=] SME: [==] Corporation: [===]
If deposit does not fit one of the slots provided then you are asking too much! :P

Is your Osc dated try Phoenix  raising oscommerce from the ashes.

Link to comment
Share on other sites

catalog/includes/classes/payment.php line #119

 

 

thanks :)

 

yea try taking out the //

that didn't do anything, but putting a } in front of the // seems to have cleared the error:)

Link to comment
Share on other sites

You'll also find that javascript is improperly defined in osCommerce. All references use:

<script language="javascript">

 

when they should use:

<script language="javascript" type="text/javascript">

 

Vger

Link to comment
Share on other sites

  • 8 months later...
You'll also find that javascript is improperly defined in osCommerce. All references use:

<script language="javascript">

 

when they should use:

<script language="javascript" type="text/javascript">

 

Vger

does it hurt leaving them as default?

 

 

 

i am having a similar problem as posted in my 1st post:

 

IE error:

Done, but with errors on the page.

 

Line: 72

Char: 8

Code: 0

 

line 72:

/*fast easy checkout start

function check_password(field_name_1, field_name_2, field_size, message_1, message_2) {

if (form.elements[field_name_1] && (form.elements[field_name_1].type != "hidden")) {

var password = form.elements[field_name_1].value;

var confirmation = form.elements[field_name_2].value;

 

if (password == '' || password.length < field_size) {

error_message = error_message + "* " + message_1 + "\n";

if ($createaccount == 'Y') error = true;

} else if (password != confirmation) {

error_message = error_message + "* " + message_2 + "\n";

if ($createaccount == 'Y') error = true;

}

}

}

*/fast easy checkout end

line 72 in bold

Link to comment
Share on other sites

should been

 

*/

// fast easy checkout end

 

and to the same for the start make a new line and set it

 

// fast easy checkout start

 

So in the future if you decide to uncomment the code it will not break things.

 

does it hurt leaving them as default?

 

It may, depends on the browser. And it's not w3c compliant.

Link to comment
Share on other sites

Archived

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

×
×
  • Create New...