divs, innerHTML and unknown runtime errors

[rwalrus]

Getting an "unknown runtime error code 0" in IE6 on the last two lines
of the function below. Works fine in Firefox 1.5, I can't figure out
what IE is complaining about. Both "subtotal" and "totaldiv" are divs
with the respective ids, both contain only "$0" initially.

function updateprice(pac kageName) {
document.orderF orm.Subpackage. value = packageName;
var tempsub =
getSelectedRadi oValue(document .orderForm.subs cription);
var sub1 = tempsub.split(" :");
var suba = sub1[0];
var subb = sub1[1];
totalprice = document.orderF orm.seats.value * suba;
shippingprice = document.orderF orm.shipping.va lue
document.getEle mentById("total ").value = (parseInt(total price) +
parseInt(shippi ngprice));
document.orderF orm.subtotal.va lue = totalprice;
document.getEle mentById('subto tal').innerHTML = '$' + totalprice;
document.getEle mentById("total div").innerHTM L = "$" +
(parseInt(total price) + parseInt(shippi ngprice))
}

The named "subtotal" child of orderForm is getting set correctly, so
it's something about setting the innerHTML in the divs. Looked through
the groups/googled, most of what I found referenced altering table
structure, which I'm not doing.

Any thoughts?

[UnaCoder]

Switch to double quotes on referencing subtotal and the string, see if
that helps

[marss]

rwalrus wrote:

var subb = sub1[1];
totalprice = document.orderF orm.seats.value * suba;

You multiply two string values. Convert them to integers or floats
before multiplication.
(parseInt, parseFloat)

totalprice = parseInt(docume nt.orderForm.se ats.value) * parseInt(suba);

[marss]

rwalrus wrote:

totalprice = document.orderF orm.seats.value * suba;

You multiply two string values. You should convert it to desired type
with "parseInt" or "parseFloat " functions.
Eg.:
totalprice = parseInt(docume nt.orderForm.se ats.value) * parseInt(suba);

Maybe you problem is here.

[RobG]

marss wrote:

rwalrus wrote:

var subb = sub1[1];
totalprice = document.orderF orm.seats.value * suba;

You multiply two string values. Convert them to integers or floats
before multiplication.
(parseInt, parseFloat)

That is unnecessary, strings are converted to numbers (if they can be)
as a part of the multiplication process. If they can't, the result is
NaN, which is not the error that the OP reported.

totalprice = parseInt(docume nt.orderForm.se ats.value) * parseInt(suba);

parseInt should never be used without a radix, particularly when
processing string input that may contain leading zeros.
--
Rob

[RobG]

UnaCoder wrote:

Switch to double quotes on referencing subtotal and the string, see if
that helps

It shouldn't. Double and single quotes are interchangeable , provided
they are nested correctly. There is no thing in the posted code to
indicate that they aren't (nested correctly).


--
Rob

[RobG]

rwalrus wrote:

Getting an "unknown runtime error code 0" in IE6 on the last two lines
of the function below. Works fine in Firefox 1.5, I can't figure out
what IE is complaining about. Both "subtotal" and "totaldiv" are divs
with the respective ids, both contain only "$0" initially.

You have a form element named 'subtotal' and a div with the same ID. IE
doesn't like that, change one of them.

[...]

--
Rob

[VK]

rwalrus wrote:

document.orderF orm.subtotal.va lue = totalprice;
document.getEle mentById('subto tal').innerHTML = '$' + totalprice;

.... which suggests that you have a form element and a DIV named
"subtotal" (I guess the same with "total"). You cannot do that in IE,
because it implements a broken namespace schema.

Go through you page (and any others if needed) and make sure that *no
one* ID on the page repeats an ID *or* NAME of a form element. So say
if you have:
<input type="text" name="subtotal" id="subtotalID" >
you cannot use neither "subtotal" nor "subtotalID " anywhere else: nor
for div's, nor for iframe's, nor for nothing.

Correct this and it will work (unless of course you have other errors
in your script :-)

[Thomas 'PointedEars' Lahn]

marss wrote:

rwalrus wrote:

var subb = sub1[1];
totalprice = document.orderF orm.seats.value * suba;

You multiply two string values. Convert them to integers or floats
before multiplication.
(parseInt, parseFloat)

As Rob said.

Furthermore, there is only one external numerical type in the corresponding
ECMAScript implementations as yet: `number', an IEEE-754 double-precision
floating-point number value, no matter its string representation (such as
with window.alert()) . In those implementations , unsigned 32-bit Integers
are used internally only. parseInt() and parseFloat() do not convert a
string value to an integer value or a float value, respectively. Both
merely parse a string in a certain way and return the corresponding
`number' value.
PointedEars

[rwalrus]

VK wrote:

rwalrus wrote:

document.orderF orm.subtotal.va lue = totalprice;
document.getEle mentById('subto tal').innerHTML = '$' + totalprice;

... which suggests that you have a form element and a DIV named
"subtotal" (I guess the same with "total"). You cannot do that in IE,
because it implements a broken namespace schema.

Go through you page (and any others if needed) and make sure that *no
one* ID on the page repeats an ID *or* NAME of a form element. So say
if you have:
<input type="text" name="subtotal" id="subtotalID" >
you cannot use neither "subtotal" nor "subtotalID " anywhere else: nor
for div's, nor for iframe's, nor for nothing.

Correct this and it will work (unless of course you have other errors
in your script :-)

That fixed it. Thanks for the help. Most of the underlying code and
markup I'm trying to maintain is not mine, and pretty hackwork, so a
lot of these bizzare errors keep cropping up. I appreciate the help.

--
Ryan W