Javascript not working in I.E., but works fine in FF

Question

I have the following javascript function:

function updatePrintCost()
{
    var qty = parseFloat(document.getElementById('qty').value);
    var pp = parseFloat(document.getElementById('pp').value);

    var finalPrice = qty * pp;

    if (!isNaN(finalPrice))
        fp.value = "$" + finalPrice.toFixed(2);
    else
        fp.value = "Error";
}

it's called by the following HTML code:

<table>
    <tr>
        <td style='border:none;'>Quantity:</td>
        <td style='border:none;'><input type='text' id='qty' name='quantity' onChange="updatePrintCost()" style='width:50px;' /></td>
        <td style='border:none;'>Print Price:</td>
        <td style='border:none;'><input type='text' name='printPrice' id='pp' onChange="updatePrintCost()" style='width:50px;' /></td>
        <td style='border:none;'> = </td>
        <td style='border:none;'><input type='finalPrice' id='fp' placeholder='0.00' style='width:75px;' /></td>
    </tr>
</table>

I put some alert tests in there, and the function IS being called, but fails out when it comes to setting the value of fp.

Can anybody see why this won't work?

also, where are you initializing `fp`? I don't see it being declared. — Shmiddty, Oct 04 '12 at 16:55
@Shmiddty https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/isNaN — Ian, Oct 04 '12 at 17:08

score 4 · Accepted Answer · edited May 23 '17 at 11:48

4

Where is your declaration of fp? You need:

var fp = document.getElementById('fp');

Or:

if (!isNaN(finalPrice)) 
       document.getElementById('fp').value = "$" + finalPrice.toFixed(2); 
else 
       document.getElementById('fp').value = "Error";

You also have a weird html input in type finalPrice rather than a regular type.

See this if you want to know why it was working in Firefox. This is a cross browser solution. The way you were doing it works in only newer browsers.

edited May 23 '17 at 11:48

Community

1
1

answered Oct 04 '12 at 16:54

fanfavorite

5,128
1
31
58

Javascript doesn't know that fp is the element unless you tell it. You can also do what I am posting in a few seconds. – fanfavorite Oct 04 '12 at 16:57

roelandvanbeek · Answer 2 · 2012-10-04T18:00:45.883

1

Not answering your question, but it seems to me that <input type='finalPrice' ...> is not want you meant. You probably mean <input type='text' name='finalPrice' ...>, right?

edited Oct 04 '12 at 18:00

answered Oct 04 '12 at 16:56

roelandvanbeek

659
8
20

score 1 · Answer 3 · answered Oct 12 '12 at 18:23

1

it is not fp,value. change it to f.p.value it'll work

answered Oct 12 '12 at 18:23

shreyas

11
1

Javascript not working in I.E., but works fine in FF

3 Answers3