IE 8: Object Expected for Every Javascript Function

Question

Moz say's everything is ok! IE say's object expected everywhere..

for example this my make a box function (in all.js file);

function kutuyap(Eid,iduzan,text,yer,ekle){

var div;
 if (document.createElement && (div = document.createElement('div'))) {
  div.name = div.id = Eid+iduzan;
  document.getElementById(yer).appendChild(div);

  }
 //$('#'+yer).append("<div id="+Eid+iduzan+"></div>")


 $('#'+Eid+iduzan).addClass("minikutu");
 $('#'+Eid+iduzan).html("&nbsp;"+text+'<span id='+Eid+'y'+iduzan+' class="yokedici">X</span>');
    $("#"+Eid+'y'+iduzan).attr("onclick","kutusil('"+Eid+"y"+iduzan+"','"+iduzan+"','"+ekle+"');");
 $('#'+ekle).val($('#'+ekle).val()+Eid+'-');

}

and after that i call function like this;

HTML;

  <select name="Mturs" class="inputs" id="Mturs">

    <option value="0" selected="selected">Choise One</option>
    <option value="4">Pop</option>
    <option value="3">Pop-Rock </option>
    <option value="5">Rock (Yabancı)</option>

  </select>

<input name="secMtur" id="secMtur" value="" type="hidden">

 <script>
 $('#Mturs').live('change', function() {

 $('#Mturs :selected').each(function (i) {

          if ( $('#Mturs :selected').val() != 0 ) {

 secMturde=$('#secMtur').val().indexOf($('#Mturs :selected').val()+'-');

splitter=$('#secMtur').val().split("-")
if(splitter.length<=12){

if (secMturde<0) {
 kutuyap($('#Mturs :selected').val(),'mtur',$(this).html(),'divmtur','secMtur');
 }else{
   alert("Choisen before")
 }

 }else{
     alert("Max limit is 12 !")
   }

}
});
});
 </script>

sory for my realy bad english..

edit: and i have this tags;

<meta http-equiv="X-UA-Compatible" content="IE=EmulateIE7" />
<script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.3.1/jquery.min.js">
</script>
<script type="text/javascript" src="alljs.js"></script>

What do your <script> tags look like, where you're importing jQuery? — Nick Craver
– Nick Craver, Commented Apr 7, 2010 at 17:49
Does your code really look like that? You should definitely go and fix your indentation. — Tomalak
– Tomalak, Commented Apr 7, 2010 at 17:52
i was try lastest before that;  i use now google apis; <script type="text/javascript" src="ajax.googleapis.com/ajax/libs/jquery/1.3.1/jquery.min.js"> </script> — Yetkin EREN
– Yetkin EREN, Commented Apr 7, 2010 at 17:57

bobince · Accepted Answer · 2010-04-07 18:36:13Z

1

$("#"+Eid+'y'+iduzan).attr("onclick","kutusil('"+Eid+"y"+iduzan+"','"+iduzan+"','"+ekle+"');");

Don't use attr to set event handlers like onclick, it won't generally work in IE.

It's also a really bad idea to be creating JavaScript code from strings in that way, too. Especially as if any of those variables can include a ', ", \, &, < or U+2018/2019 character you've got trouble.

Instead use inline function expressions, picking up the variables you already have from the outer function using a closure:

$("#"+Eid+'y'+iduzan).click(function() {
    kutusil(Eid+'y'+iduzan, iduzan, ekle);
});

Again:

$('#'+Eid+iduzan).html("&nbsp;"+text+...

If text can contain </& you've potentially got cross-site-scripting problems there. Don't stick HTML strings together from text content. Use .text(text) to write plain text to an element.

if (document.createElement && (div = document.createElement('div'))) {

There is no need for this checking. document.createElement exists in every browser (it is DOM Level 1 Core) and can never return null.

div.name

There's no such property on divs.

answered Apr 7, 2010 at 18:36

bobince

538k111 gold badges675 silver badges846 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Yetkin EREN Over a year ago

ok i made a test page with my code and IE says nothing(no more object expected error) also do nothing and change my code like this.. but nothing change.. yetkineren.com/testpage.html

bobince Over a year ago

You appear to be using jQuery 1.3. live change events don't work in IE under this version of jQuery. Either change to 1.4, or replace .live('change', ... with a normal .change(... binding. (You don't appear to be doing anything that requires live binding.)

Jacques Perrault · Accepted Answer · 2011-01-05 16:32:30Z

bobince's answer is excellent; do pay very careful attention to XSS vulnerabilities.

I would add only one other tidbit that caused me some consternation early on, and is a best practice that you should consider adopting:

Do not use primitives or core attribute/function names in your script unless you are purposefully doing so (e.g in the case of overriding).

Take this example:

01:     $.getJSON('/doSomething.php?r=' + new Date().getTime(),function(json){
02:         var someinfo = "";
03:         $.each(json.somearray, function(i,data){
04:             someinfo = someinfo + data.class + " ";
05:         });
            ...

Mozilla is perfectly fine with the above code; Internet Explorer, however, chokes on the ".class" reference (line 04).

This issue can be difficult to identify since the error prevents IE from parsing other javascript on the page; the result is that IE throws an error on all (custom) function calls, rather than just the function where the actual problem is.

The solution, as previously mentioned, is to replace ".class" with some other string

 04:                someinfo = someinfo + data.classification + " ";

Happy coding!

chris14679 · Accepted Answer · 2011-08-08 07:50:24Z

0

Try changing this:

src="ajax.goog...

to this:

src="http://ajax.goog...

Usually the http:// is necessary, this might be why your scripts were not loaded.

answered Aug 8, 2011 at 7:50

chris14679

1,6741 gold badge11 silver badges3 bronze badges

Collectives™ on Stack Overflow

IE 8: Object Expected for Every Javascript Function

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related