$.gevent.publish() is not work on IE8 about jquery.event.gevent HOT 4 CLOSED

It may be due to the version of jQuery you are using. However, I did not test ie8.

Sent from my ASUS Transformer Pad

-------- Original Message --------
From:biiterlove [email protected]
Sent:Wed, 15 Jan 2014 01:27:03 -0800
To:"mmikowski/jquery.event.gevent" [email protected]
Subject:[jquery.event.gevent] $.gevent.publish() is not work on IE8 (#1)

Hi!, I follow the SPA book (Really good book anyway)
And now I'm creating my model just similar to the one in the book.
But when I wrote the test for it I just found that the "gevent" lib its not work on IE

My test code look similar like this

var $t = $('

'); $.gevent.subscribe($t, 'login', function(event, name) { expect(name).to.be('Pongneng'); done(); });

And in my production code I call the publish mehtod

$.gevent.publish('login', ['Pongneng']);

It's caused my test to fail and give me this error: Object doesn't support this property or method
I still don't know what to do about it yet. I just leave it for now because the other browser is work fine.

—
Reply to this email directly or view it on GitHub.

from jquery.event.gevent.

I use jQuery 1.10.2. Is that a problem?
Anyway I found that it break on this line

data_list = Array.isArray( data ) ? data : [ data ];

IE8 don't have a method isArray (=_=")

I changed to

data_list = data instanceof Array ? data : [ data ];

and it work fine. (Not sure it's right or not ha ha)

from jquery.event.gevent.

Sorry about the delayed response. The instanceof looks correct at first glance. A little sniffing around shows your potential pain is much greater for IE 8:

http://whattheheadsaid.com/2010/10/cross-context-isarray-and-internet-explorer

from jquery.event.gevent.

The current release now uses the jQuery isArray method, which should fix the problem with IE 8.

from jquery.event.gevent.