Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 11 Next »

[14:01:25 CDT(-0500)] <pspaude1> Hey Dan, EarlyAlertToolViewController is choking on line 84 in IE8.

[14:01:29 CDT(-0500)] <pspaude1> responseDispatcher.setSuccessCallback('earlyalert', me.bindEarlyAlertToView, me).apply(responseDispatcher, null);

[14:02:24 CDT(-0500)] <pspaude1> Its in the .apply says either object doesn't support property or method or Object not found

[14:02:41 CDT(-0500)] <pspaude1> Tried splitting up and it still says the same thing.

[14:03:17 CDT(-0500)] <dmccallum54> one sec pls

[14:04:24 CDT(-0500)] <pspaude1> No problem. Sorry to bother you, but you are the original author, so I thougt you might have more insight. It's fine in IE10/Chrome/FF etc.

[14:04:44 CDT(-0500)] <dmccallum54> tracked me down, eh

[14:04:52 CDT(-0500)] <pspaude1> Git blame (smile)

[14:15:18 CDT(-0500)] <pspaude1> Ok tried a left field move and put IE10 in 8 mode and it gave me a better description. It's not liking the null as a param to apply. I'll try working on it a bit.

[14:23:15 CDT(-0500)] <dmccallum54> pspaude1 no blinding insights so far… could try just passing an empty [] there… or i maybe IE is not liking the lack of a semi colon on line 53 of RequestDispatcher?

[14:25:39 CDT(-0500)] <dmccallum54> MDN says args can be null or undefined, but M$ docs suggest IE won't play nice with that (as it seems you've found): http://msdn.microsoft.com/en-us/library/ie/4zc42wh1(v=vs.94).aspx

[14:26:04 CDT(-0500)] <pspaude1> Yeah I was thinking of trying []. Got caught up in the proxy error on EA load.

[14:27:29 CDT(-0500)] <pspaude1> Ok it likes that

[14:27:45 CDT(-0500)] <pspaude1> But, now ResponseDispatcher line 96 object doesn't support proeprty or method filter

[14:29:21 CDT(-0500)] <dmccallum54> looked up the ECMA spec. IE is wrong. shocking.

[14:29:29 CDT(-0500)] <pspaude1> haha

[14:29:37 CDT(-0500)] <dmccallum54> not that being right helps us...

[14:30:01 CDT(-0500)] <pspaude1> nope, coaches might get mad when they can't view early alert details (smile)

[14:32:15 CDT(-0500)] <pspaude1> I think array.filter is not supported in IE8 argg

[14:32:37 CDT(-0500)] <dmccallum54> not totally sure how to read this but yea… requirements section of http://msdn.microsoft.com/en-us/library/ie/ff679973(v=vs.94).aspx

[14:32:47 CDT(-0500)] <dmccallum54> <sigh>

[14:32:53 CDT(-0500)] <dmccallum54> well

[14:33:10 CDT(-0500)] <dmccallum54> mind changing that to see if the filter function exists and if not fall back to a simple array loop?

[14:33:16 CDT(-0500)] <pspaude1> I'll see if I can rewrite it

[14:33:27 CDT(-0500)] <pspaude1> Yep, just what I was going to say.

[14:33:32 CDT(-0500)] <dmccallum54> k. thx/

[14:33:36 CDT(-0500)] <pspaude1> np

  • No labels