search

adlnet / xAPIWrapper

Wrapper to simplify communication to an LRS
https://adlnet.gov/projects/xapi/
Apache License 2.0
219 stars 114 forks source link

using this API with IE10 #89

Closed shigil closed 5 years ago

shigil commented 7 years ago

Hi

I ran into a problem while using this API with IE10. The API was throwing invalid state error in IE 10 while trying to fetch statements from SCORM CLOUD. All works well in IE 11 and other browsers.

Debugging further I realized that the issue was actually with the xhr.withCredentials in the ADL.XHR_request function. Moving it after xhr.open fixed the issue. Is it possible to raise a pull request to include it in the code?

xhr.open(method, url, callback != null); xhr.withCredentials = withCredentials;

or if(typeof(withCredentials) != "undefined") // if Boolean is not passed for withCredentials xhr.withCredentials = withCredentials; xhr.open(method, url, callback != null);

brian-learningpool commented 6 years ago

This is fixed in https://github.com/adlnet/xAPIWrapper/pull/112.