[jira] [Commented] (CB-6450) [WP8] XHRHelper incompatibility with Sencha Touch
[ https://issues.apache.org/jira/browse/CB-6450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13980295#comment-13980295 ] Jesse MacFadyen commented on CB-6450: - Can you try this? if (responseCode == '200') { alias.responseText = responseText; // MY CHANGE HERE alias.responseXML = DOMParser.parseFromString(responseText, "text/xml"); // END } > [WP8] XHRHelper incompatibility with Sencha Touch > - > > Key: CB-6450 > URL: https://issues.apache.org/jira/browse/CB-6450 > Project: Apache Cordova > Issue Type: Bug > Components: WP8 >Affects Versions: 3.4.0 > Environment: WP8, IE10 >Reporter: Marco Spinola Durante >Assignee: Jesse MacFadyen >Priority: Critical > > My app uses Cordova 3.4.0 & Sencha Touch successfully on iOS and Android. I > am extending support to WP8, but the app wont start correctly. > The problem lies in the XHRHelper (probably in the injected JS code), because > if I load the app in the IE10 browser everything works fine. > From Sencha Touch point of view loading of XML fails, although the reader > returns 200, I can see that the response.responseXML field is empty. > I tried to edit the injected JS script and change as follows: > {code:javascript}var funk = function () { >window.__onXHRLocalCallback = function (responseCode, responseText) { > alias.status = responseCode; > if (responseCode == '200') { > alias.responseText = responseText; > // MY CHANGE HERE > alias.responseXML = responseText; > // END > } > else { > alias.onerror && alias.onerror(responseCode); > } > alias.changeReadyState(XHRShim.DONE); >} >alias.changeReadyState(XHRShim.LOADING); >window.external.Notify('XHRLOCAL/' + resolvedUrl); > } > {code} > but it didn't help. -- This message was sent by Atlassian JIRA (v6.2#6252)
[jira] [Commented] (CB-6450) [WP8] XHRHelper incompatibility with Sencha Touch
[ https://issues.apache.org/jira/browse/CB-6450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13980818#comment-13980818 ] Marco Spinola Durante commented on CB-6450: --- the change gives me an unhanded exception: {code} An exception of type 'System.SystemException' occurred in Microsoft.Phone.Interop.ni.dll but was not handled in user code {code} at this line in XHRHelper.cs {code} Browser.InvokeScript("__onXHRLocalCallback", new string[] { "200", text }); {code} > [WP8] XHRHelper incompatibility with Sencha Touch > - > > Key: CB-6450 > URL: https://issues.apache.org/jira/browse/CB-6450 > Project: Apache Cordova > Issue Type: Bug > Components: WP8 >Affects Versions: 3.4.0 > Environment: WP8, IE10 >Reporter: Marco Spinola Durante >Assignee: Jesse MacFadyen >Priority: Critical > > My app uses Cordova 3.4.0 & Sencha Touch successfully on iOS and Android. I > am extending support to WP8, but the app wont start correctly. > The problem lies in the XHRHelper (probably in the injected JS code), because > if I load the app in the IE10 browser everything works fine. > From Sencha Touch point of view loading of XML fails, although the reader > returns 200, I can see that the response.responseXML field is empty. > I tried to edit the injected JS script and change as follows: > {code:javascript}var funk = function () { >window.__onXHRLocalCallback = function (responseCode, responseText) { > alias.status = responseCode; > if (responseCode == '200') { > alias.responseText = responseText; > // MY CHANGE HERE > alias.responseXML = responseText; > // END > } > else { > alias.onerror && alias.onerror(responseCode); > } > alias.changeReadyState(XHRShim.DONE); >} >alias.changeReadyState(XHRShim.LOADING); >window.external.Notify('XHRLOCAL/' + resolvedUrl); > } > {code} > but it didn't help. -- This message was sent by Atlassian JIRA (v6.2#6252)
[jira] [Commented] (CB-6450) [WP8] XHRHelper incompatibility with Sencha Touch
[ https://issues.apache.org/jira/browse/CB-6450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13981563#comment-13981563 ] Jesse MacFadyen commented on CB-6450: - Okay, my mistake. {code} if (responseCode == '200') { alias.responseText = responseText; // MY CHANGE HERE var dp = new DOMParser(); alias.responseXML = dp.parseFromString(xhr.responseText, "text/xml"); // END } {code} I will be formalizing this soon, as it should be a getter, but that should get you through till the update. > [WP8] XHRHelper incompatibility with Sencha Touch > - > > Key: CB-6450 > URL: https://issues.apache.org/jira/browse/CB-6450 > Project: Apache Cordova > Issue Type: Bug > Components: WP8 >Affects Versions: 3.4.0 > Environment: WP8, IE10 >Reporter: Marco Spinola Durante >Assignee: Jesse MacFadyen >Priority: Critical > > My app uses Cordova 3.4.0 & Sencha Touch successfully on iOS and Android. I > am extending support to WP8, but the app wont start correctly. > The problem lies in the XHRHelper (probably in the injected JS code), because > if I load the app in the IE10 browser everything works fine. > From Sencha Touch point of view loading of XML fails, although the reader > returns 200, I can see that the response.responseXML field is empty. > I tried to edit the injected JS script and change as follows: > {code:javascript}var funk = function () { >window.__onXHRLocalCallback = function (responseCode, responseText) { > alias.status = responseCode; > if (responseCode == '200') { > alias.responseText = responseText; > // MY CHANGE HERE > alias.responseXML = responseText; > // END > } > else { > alias.onerror && alias.onerror(responseCode); > } > alias.changeReadyState(XHRShim.DONE); >} >alias.changeReadyState(XHRShim.LOADING); >window.external.Notify('XHRLOCAL/' + resolvedUrl); > } > {code} > but it didn't help. -- This message was sent by Atlassian JIRA (v6.2#6252)
[jira] [Commented] (CB-6450) [WP8] XHRHelper incompatibility with Sencha Touch
[ https://issues.apache.org/jira/browse/CB-6450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13981590#comment-13981590 ] ASF subversion and git services commented on CB-6450: - Commit 67715e0294c362a0ee9585ec8165d47a95fc4964 in cordova-wp8's branch refs/heads/master from [~purplecabbage] [ https://git-wip-us.apache.org/repos/asf?p=cordova-wp8.git;h=67715e0 ] CB-6450 added support for local XHR.responseXML getter > [WP8] XHRHelper incompatibility with Sencha Touch > - > > Key: CB-6450 > URL: https://issues.apache.org/jira/browse/CB-6450 > Project: Apache Cordova > Issue Type: Bug > Components: WP8 >Affects Versions: 3.4.0 > Environment: WP8, IE10 >Reporter: Marco Spinola Durante >Assignee: Jesse MacFadyen >Priority: Critical > > My app uses Cordova 3.4.0 & Sencha Touch successfully on iOS and Android. I > am extending support to WP8, but the app wont start correctly. > The problem lies in the XHRHelper (probably in the injected JS code), because > if I load the app in the IE10 browser everything works fine. > From Sencha Touch point of view loading of XML fails, although the reader > returns 200, I can see that the response.responseXML field is empty. > I tried to edit the injected JS script and change as follows: > {code:javascript}var funk = function () { >window.__onXHRLocalCallback = function (responseCode, responseText) { > alias.status = responseCode; > if (responseCode == '200') { > alias.responseText = responseText; > // MY CHANGE HERE > alias.responseXML = responseText; > // END > } > else { > alias.onerror && alias.onerror(responseCode); > } > alias.changeReadyState(XHRShim.DONE); >} >alias.changeReadyState(XHRShim.LOADING); >window.external.Notify('XHRLOCAL/' + resolvedUrl); > } > {code} > but it didn't help. -- This message was sent by Atlassian JIRA (v6.2#6252)
[jira] [Commented] (CB-6450) [WP8] XHRHelper incompatibility with Sencha Touch
[ https://issues.apache.org/jira/browse/CB-6450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13981584#comment-13981584 ] Jesse MacFadyen commented on CB-6450: - My upcoming commit will be the following change : {code} if (responseCode == '200') { alias.responseText = responseText; Object.defineProperty(alias, 'responseXML', { get: function () { return new DOMParser().parseFromString(this.responseText, 'text/xml'); } }); } {code} I have verified this with the following : {code} function loadMeSomeXHR() { var xhr = new XMLHttpRequest(); xhr.open("GET", "../config.xml", true); xhr.onload = function () { console.log("loaded!"); try { var responseXML = xhr.responseXML; console.log("responseXML: origin= " + responseXML.getElementsByTagName('access')[0].getAttribute('origin')); } catch (e) { console.log(e); } } xhr.send(); } {code} > [WP8] XHRHelper incompatibility with Sencha Touch > - > > Key: CB-6450 > URL: https://issues.apache.org/jira/browse/CB-6450 > Project: Apache Cordova > Issue Type: Bug > Components: WP8 >Affects Versions: 3.4.0 > Environment: WP8, IE10 >Reporter: Marco Spinola Durante >Assignee: Jesse MacFadyen >Priority: Critical > > My app uses Cordova 3.4.0 & Sencha Touch successfully on iOS and Android. I > am extending support to WP8, but the app wont start correctly. > The problem lies in the XHRHelper (probably in the injected JS code), because > if I load the app in the IE10 browser everything works fine. > From Sencha Touch point of view loading of XML fails, although the reader > returns 200, I can see that the response.responseXML field is empty. > I tried to edit the injected JS script and change as follows: > {code:javascript}var funk = function () { >window.__onXHRLocalCallback = function (responseCode, responseText) { > alias.status = responseCode; > if (responseCode == '200') { > alias.responseText = responseText; > // MY CHANGE HERE > alias.responseXML = responseText; > // END > } > else { > alias.onerror && alias.onerror(responseCode); > } > alias.changeReadyState(XHRShim.DONE); >} >alias.changeReadyState(XHRShim.LOADING); >window.external.Notify('XHRLOCAL/' + resolvedUrl); > } > {code} > but it didn't help. -- This message was sent by Atlassian JIRA (v6.2#6252)
[jira] [Commented] (CB-6450) [WP8] XHRHelper incompatibility with Sencha Touch
[ https://issues.apache.org/jira/browse/CB-6450?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14095829#comment-14095829 ] VinÃcius Gonçalves commented on CB-6450: This line: {code} Object.defineProperty(alias, 'responseXML', { get: function () { return new DOMParser().parseFromString(this.responseText, 'text/xml'); } }); {code} produce an error when I try to load a local json file in a Ajax Sencha Touch request, when I using WP8. > [WP8] XHRHelper incompatibility with Sencha Touch > - > > Key: CB-6450 > URL: https://issues.apache.org/jira/browse/CB-6450 > Project: Apache Cordova > Issue Type: Bug > Components: WP8 >Affects Versions: 3.4.0 > Environment: WP8, IE10 >Reporter: Marco Spinola Durante >Assignee: Jesse MacFadyen >Priority: Critical > Fix For: 3.5.0 > > > My app uses Cordova 3.4.0 & Sencha Touch successfully on iOS and Android. I > am extending support to WP8, but the app wont start correctly. > The problem lies in the XHRHelper (probably in the injected JS code), because > if I load the app in the IE10 browser everything works fine. > From Sencha Touch point of view loading of XML fails, although the reader > returns 200, I can see that the response.responseXML field is empty. > I tried to edit the injected JS script and change as follows: > {code:javascript}var funk = function () { >window.__onXHRLocalCallback = function (responseCode, responseText) { > alias.status = responseCode; > if (responseCode == '200') { > alias.responseText = responseText; > // MY CHANGE HERE > alias.responseXML = responseText; > // END > } > else { > alias.onerror && alias.onerror(responseCode); > } > alias.changeReadyState(XHRShim.DONE); >} >alias.changeReadyState(XHRShim.LOADING); >window.external.Notify('XHRLOCAL/' + resolvedUrl); > } > {code} > but it didn't help. -- This message was sent by Atlassian JIRA (v6.2#6252)