Hi,
yes you are right! Yesterday when i checked my code, i checked with an
old version of APE JSF! Thanks for the bug repport i'am going to fix it
on github ;-)
Regards.
mjcoder a écrit :
Thanks for the reply, Nicolas.
It seems the code contradicts your statements.
taken from line 181-190 of Source/Transport/Transport.XHRStreaming.js:
APE.Transport.XHRStreaming.browserSupport = function() {
if (Browser.Features.xhr && (Browser.Engine.webkit ||
Browser.Engine.gecko)) {
return true;
} else return 2;//No XHR Support, switch to JSONP
}
It checks for xhr AND that the browser is webkit or gecko else it uses
JSONP
I want IE to use Long Polling and Firefox/Safari/Chrome to use
XHRStreaming, but I believe this code is preventing that.
Am I missing something?
On Feb 22, 6:07 pm, Nicolas Trani <[email protected]> wrote:
Sorry, i made a mistake in my sentence. If browser support XHR request
but doesn't support XHRStreaming, it will use long polling. If it
doesn't support XHR request, JSONP is used.
Nicolas Trani wrote:
Hi Matt,
no it's not a bug, JSONP is only used if browser doesn't support XHR
request. If browser support XHR request but doesn't support XHRStreaming
feature it will fallback to JSONP.
Regards.
mjcoder wrote:
The documentation says that if the browser doesn't support a transport
method, it will failover to Long Polling:
http://www.ape-project.org/wiki/index.php/Tutorial:Use_different_tran...
This doesn't seem to be the case. Instead, it is failing over to
JSONP.
See Line 189 of Source/Transport/Transport.XHRStreaming.js
Is documentation just wrong or a bug?
Thanks,
Matt
--
Nicolas Trani - Web engineer
Weelya - Improve the web
32 rue du faubourg boutonnet
34090 Montpellier
Tel: 04 67 169 778 - Fax: 09 57 91 99
91http://www.weelya.comhttp://www.ape-project.org- Hide quoted text -
- Show quoted text -
--
You received this message because you are subscribed to the Google
Groups "APE Project" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/ape-project?hl=en
---
APE Project (Ajax Push Engine)
Official website : http://www.ape-project.org/
Git Hub : http://github.com/APE-Project/
