This code works in all other browsers except in IE

Question

This works fine in all browsers except IE can anyone explain why so I can fix it. I am displaying the index of a javascript object based on the index of the selected index of my dropdown list

 $(document).ready(function () {
var pdata = [{ Name: "Apples", Price: 1.99 },{ Name: "Bananas", Price: 2.45 } ];

    $('#produceTMPL').tmpl(pdata).appendTo('#produceList');

      $(document).ready(function () {


      $('#add1').click(function () {
        var selected = $('#produceList option:selected').index();

        item = pdata[selected];

        console.log(selected);
        $('#cart').append('<p>' + item.Name + ', ' + item.Price + '</p>');




    });  
    });

HTML:

     <div>
  <select id="produceList">
  <option>make a selection</option>
  </select>

How does this "not work" in IE? Which version of IE? What *does* it do? Do you see any errors in the console? — gen_Eric, Jan 10 '14 at 23:23
Why do you have a `$(document).ready(function () {` inside of a `$(document).ready(function () {`? — gen_Eric, Jan 10 '14 at 23:24
do you really need 2 document ready calls there? Also, older versions of IE will probably cry if you try to run console — Kai Qing, Jan 10 '14 at 23:24
Try to open the console, then reload your page and see if it works. `console.log` won't work in IE unless the console is open. — gen_Eric, Jan 10 '14 at 23:25
This answer might help: http://stackoverflow.com/a/11147595/1363726 — eivers88, Jan 10 '14 at 23:31
@ Rocket Hazmat I removed the console.log and removed the extra $(document).ready(function(){}); but it still says undefined in my html — chrisp54, Jan 10 '14 at 23:32

score 1 · Accepted Answer · answered Jan 10 '14 at 23:40

1

item is a protected property of the window object in IE. Just rename your variable, or declare it properly (using var) in your function.

answered Jan 10 '14 at 23:40

Teemu

22,918
7
53
106

Nagendra Rao · Answer 2 · 2014-01-10T23:36:40.027

0

console.log throws an error in IE, so to be on the safer side DO NOT use console.log with out checking if browser supports it or not.

try{
    if(console && console.log) console.log('message')   
}catch(e){}

OR

 if ('undefined' !== typeof console){console.log(message); }

edited Jan 10 '14 at 23:36

answered Jan 10 '14 at 23:25

Nagendra Rao

7,016
5
54
92

IE supports `console.log`. The console (dev tools) just need to be opened for it to do so. You need to open the dev tools, then reload your page. – gen_Eric Jan 10 '14 at 23:26
That's why I suggested to open it then reload the page. – gen_Eric Jan 10 '14 at 23:30
but in IE it says undefined on my html – chrisp54 Jan 10 '14 at 23:32
@RocketHazmat I know, but what I am trying to say is no body has it open in the first, at which time, console.log is not supported, and if u want to avoid that situation, check before blindly using the function. – Nagendra Rao Jan 10 '14 at 23:33
@Rao: Fair enough :-) – gen_Eric Jan 10 '14 at 23:34
@ Rocket Hazmat I removed the console.log and removed the extra $(document).ready(function(){}); but it still says undefined in my html – chrisp54 – chrisp54 Jan 10 '14 at 23:40
try renaming `item`, as suggested by Teemu – Nagendra Rao Jan 10 '14 at 23:41

This code works in all other browsers except in IE

2 Answers2