Javascript не работает в IE

[РЕДАКТИРОВАТЬ] Извините. Я ввел ошибку в своем первом сообщении, не внимательно изучив, как вы строите свой URL. Не стоило убирать параметр id. Я обновил код, и теперь он должен работать.

Попробуйте вместо этого:

function drop(ctl,id)
{
   var value = ctl.options[ctl.selectedIndex].value;

   if (value == "Ficha de pediatria"){
       window.top.location.href = "print.jsp?id = "+id+"&type=2";
   }
   else if (value == "Ficha normal"){
       window.top.location.href = "print.jsp?id = "+id+"&type=1";
   }
}

<select id = "select1" name = "select1" onChange = "drop(this,id);return false;">
<option>Imprimir:</option>
<option>Ficha de pediatria</option>
<option>Ficha normal</option>
</select>

[РЕДАКТИРОВАТЬ] Небольшое объяснение ...

Я думаю, проблема в том, как он получает доступ к DOM. Я не думаю, что IE имеет свойство value для выбора. Я думаю, вам нужно добраться до этого с помощью выбранной опции. Кроме того, я не уверен, что в глобальном пространстве имен есть свойство top, но вы сможете получить к нему доступ через window.top, чтобы задать местоположение. Наконец, я немного улучшил это, указав this в качестве аргумента, вы можете пропустить поиск элемента и ссылаться на него непосредственно из элемента управления, переданного в качестве аргумента.

Я вижу две возможные причины.

1 - способ объявления функции. Я никогда не видел такого, хотя думаю, что это работает.

Возможно, попробуйте следующее и посмотрите, не работает ли это по-прежнему:

function drop(id)
{
    // same code
}

2 - возврат false в IE не всегда ведет себя должным образом (поверьте мне, это зависит от компьютера) Поэтому вместо того, чтобы возвращать false, попробуйте:

onChange = "drop(id);event.returnValue=false;return false;"

Если возможно, создайте такой метод:

function CrossBrowserFalse()
{
     if (IE) // use whatever you want to detect IE
     {
         event.returnValue = false;
     }

     return false;
}

а затем в своих методах вы можете использовать:

onChange = "drop(id);return CrossBrowserFalse();"

... да, IE иногда бывает странно (часто)

Если эти два метода не работают, по крайней мере, убедитесь, что ваша функция перетаскивания вызывается, поместив туда несколько предупреждений или точек останова, если ваша IDE поддерживает ее для javascript.

Дело не в том, что IE «не имеет .value» элемента <select>, дело в том, что вы не указали никаких значений для ваших элементов <option>. Firefox, Safari и т. д. скорее всего, защитят вас от этой ошибки. Тем не менее, ваш элемент должен быть построен как:

<select ...>
<option value = "Imprimir">Imprimir:</option>
<option value = "Ficha de pediatria">Ficha de pediatria</option>
<option value = "Ficha normal">Ficha normal</option>
</select>

... и вы увидите более стандартное поведение x-браузера для свойства .value <select>.

IE не нравится, как вы захватываете значение select

document.getElementById("select1").value

Это говорит: «дайте мне текст, который находится в атрибуте value выбранного параметра в элементе select с идентификатором select1. У ваших параметров нет никаких значений. Когда Firefox и Safari сталкиваются с этой двусмысленностью, они возвращают текст выбранного вариант. Когда Internet Explorer обнаруживает эту двусмысленность, он возвращает пустую строку. Ваш javascript будет работать как есть, если вы сделаете что-то вроде

<select id = "select1" name = "select1" onChange = "drop(this,id);return false;">
    <option value = "Imprimir:">Imprimir:</option>
    <option value = "Ficha de pediatria">Ficha de pediatria</option>
    <option value = "Ficha normal">Ficha normal</option>
</select>

Если это невозможно, вы захотите получить текст выбранной опции.

var theSelect = document.getElementById("select1");
var theValue  = theSelect.options[theSelect.selectedIndex].text

Наконец, хотя это и не ваш прямой вопрос, жестко запрограммированный select1 - не лучший способ добраться до списка выбора. Рассмотрите возможность использования этого ключевого слова

function foo(theSelect){
    alert(theSelect.options);
}

...

<select id = "whatever" onchange = "foo(this);">
...
</select>

Это немного более общий характер, и вы сможете использовать свою функцию для выбора с любым идентификатором.