jasig-ssp IRC Logs-2013-08-09

[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

[14:35:47 CDT(-0500)] <dmccallum54> Ext.js to the rescue (maybe): http://docs.sencha.com/extjs/4.1.0/#!/api/Ext.Array-method-filter

[14:36:30 CDT(-0500)] <pspaude1> Quite stealing my next moves (smile)

[14:36:37 CDT(-0500)] <dmccallum54> heh

[14:37:37 CDT(-0500)] <js70> thats the lot of the intern. Boss Man working the system.(sad)

[14:38:16 CDT(-0500)] <dmccallum54> that's the key to moving up… figure out how to get google to do all your work for you

[14:55:13 CDT(-0500)] <pspaude1> Alright, looks like it works. Greped through the code and found one other spot in MAP where filter is being used, will check that out as well.

[14:55:17 CDT(-0500)] <pspaude1> Thanks for the help

[14:55:41 CDT(-0500)] <pspaude1> My google isn't as fast as yours and it links to useless results at the top (smile)

[15:26:43 CDT(-0500)] <dmccallum54> anybody running SSP in standalone mode for dev purposes?

[15:27:17 CDT(-0500)] <pspaude1> Yes

[15:27:26 CDT(-0500)] <dmccallum54> k. thx

[15:27:30 CDT(-0500)] <pspaude1> Or you mean without uportal?

[15:27:33 CDT(-0500)] <dmccallum54> yes

[15:27:37 CDT(-0500)] <dmccallum54> without uportal

[15:27:45 CDT(-0500)] <pspaude1> No, sorry.