Problem listening to JS events from Android App about jockeyjs HOT 5 CLOSED

I found the root of the problem as well as the solution. In the latest js of the library you included the following code:

if (iOS && UIWebView) {
Jockey.dispatcher = iOSDispatcher;
} else {
Jockey.dispatcher = nullDispatcher;
}

Unfortunately in Android the dispatcher used was the nullDispatcher so nothing was been send to the app. After almost going nuts why your sample application was working was due to the fact that actually in the library included in the sample project the code looked like this.

// if (iOS && UIWebView) {
Jockey.dispatcher = iOSDispatcher;
// } else {
// Jockey.dispatcher = nullDispatcher;
// }

So the iOSDispatcher was always been used. After looking at the code I am not sure why do you even need to distinguish between iOS and Android. The dispatch method inside the iOSDispatcher has the following line of code:

window.location.href = "jockey://" + type + "/" + envelope.id + "?" + encodeURIComponent(JSON.stringify(envelope));

This might cause issues but actually you already solved it in the Android library with this:
public boolean isJockeyScheme(URI uri) {
return uri.getScheme().equals("jockey") && !uri.getQuery().equals("");
}

So yo actually detect the redirection in the Android JockeyWebViewClient() so it is handled correctly. So I would recommend updating the JS library code as it is really confusing and it seems to be working correctly by using always the iOSDispatcher.

Again thanks for the work done.

from jockeyjs.

Wow @jatago, thanks for finding this. The original JS was written by me when I assumed there would need to be a different dispatcher for Android. Another fine coder came along and filled out the android work not needing a new dispatcher, and I guess the JS never got updated.

Would you mind submitting a PR? I'd love to have this change in there.

from jockeyjs.

Hi Tim!

Sure no problem ;) thanks to you guys for writing this library. Super
simple to use and extremely helpful!
On Jan 23, 2014 5:38 PM, "Tim Coulter" [email protected] wrote:

Wow @jatago https://github.com/jatago, thanks for finding this. The
original JS was written by when I assumed there would need to be a
different dispatcher for Android. Another fine coder came along and filled
out the android not needing a new dispatcher, and I guess the JS never got
updated.

Would you mind submitting a PR? I'd love to have this change in there.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/18#issuecomment-33141548
.

from jockeyjs.

This should be fixed by #20

from jockeyjs.

Awesome, thanks @paulpdaniels. Closing.

from jockeyjs.