+- Kodi Community Forum (https://forum.kodi.tv)
+-- Forum: Support (https://forum.kodi.tv/forumdisplay.php?fid=33)
+--- Forum: Supplementary Tools for Kodi (https://forum.kodi.tv/forumdisplay.php?fid=116)
+---- Forum: Kodi Remote for Android Official Forum (https://forum.kodi.tv/forumdisplay.php?fid=129)
+---- Thread: Download Frodo compatible test version here (/showthread.php?tid=147741)
RE: Download Frodo compatible test version here - Tolriq - 2013-01-11
JSON rpc is a protocol if the data sent does not follow the protocol how do you want the client to understand ?
The specification is very clear about the fact that strings must be UTF-8 encoded, efficient json parsing needs a stream parser that analyse data as it comes, when some data should not be in the middle the parsers have no way to recover from that.
Xbmc should not be able to send data that does not follow the standard but it does if the data in it's database is not UTF-8 witch was possible until recent Frodo builds. (And will be in Frodo if you don't rescrape the bad data when updating from Eden).
This issue will be recurrent during update until all users rescrapes but well it was the chosen path by the Xbmc team.
RE: Download Frodo compatible test version here - jblance - 2013-01-13
(2013-01-11, 21:47)Nat! Wrote: I think the fetching code should be lenient with respect to encoding exceptions. If a tag is malformed in an album list, it shouldn't really abort the whole operation. After all a tag is just a tag.
Yeah - I've made a small change to the Jackson parser, so it just returns a space if the encoding is incorrect (instead of throwing an error)
This allows the remote to 'work' with bad data instead of failing with an error message. It works for me, will see if Tom is prepared to pull to his repo (as he seems to have the most current code)
RE: Download Frodo compatible test version here - Tolriq - 2013-01-13
Some of the needed tests to validate such an hack includes badly encoded utf8 just before an escaped " and as the last character of a string and some more I can't remember
RE: Download Frodo compatible test version here - jblance - 2013-01-13
Yeah I only addressed a small portion of the cases, though they seem to be the ones the are reported....
I didn't want to completely break the parser...
RE: Download Frodo compatible test version here - Tolriq - 2013-01-13
Lol yes I know but those 2 cases are user cases (I had lots of logs on this UTF8 problem :p)
And if your hack make the parser miss the " or the } then you'll completely break things with way more impact on the user
RE: Download Frodo compatible test version here - jblance - 2013-01-13
Ha, yeah I see your point.
The hack is just referenced by _finishstring and _handleUnexpectedValue functions, so hopefully does not mess with any important parsing...
But I could be mistaken....
RE: Download Frodo compatible test version here - freezy - 2013-01-15
A little status update.
Tombriden and I spent the whole evening together merging his changes into my master. It was a bit bitchy since our branches have diverged quite a bit. The intermediate result can be seen here:
https://github.com/freezy/android-xbmcremote/tree/tombridenmerge
We'll test it more thoroughly tomorrow and then post a build here.
On this opportunity I would like to thank tthomas48 for his patience and his agreement to basically throw his JSON-RPC integration away in favor of tombriden's branch. Cheers mate, looking forward seeing your commits in this and the next gen version of the remote! Of course, many thanks go also to tombriden. Let's bring this baby home before Frodo final ships!
Cheers,
-freezy.
RE: Download Frodo compatible test version here - spthomas - 2013-01-15
Looking forward to it, missing my Android remote.
RE: Download Frodo compatible test version here - theloon - 2013-01-15
Just moved from Eden w/XBMC remote 808 to Frodo w/remote on post #1 and everything I've tested works well! I was one of those that had java exception errors but after port forwarding and updating firmware on my Belkin router all functioned as it should and I am pleased as all heck. Thanks again for the great app!
RE: Download Frodo compatible test version here - freezy - 2013-01-16
Updated first post with test build.
RE: Download Frodo compatible test version here - corpgator - 2013-01-16
How do I enable and use json? I get no option when adding a new host. The app works fine as a remote, but if I try to open movies, music, etc, it just crashes.
Re: Download Frodo compatible test version here - freezy - 2013-01-16
Did you try uninstalling your previous version?
RE: Download Frodo compatible test version here - corpgator - 2013-01-16
No previous version since I wiped and rooted my android before installing this one. Running latest release candidate of frodo.
RE: Download Frodo compatible test version here - theloon - 2013-01-16
Remote works but crashes when opening movies also, theres no option for json, I'm on RC3 Frodo I'll test on Eden later today.
Re: Download Frodo compatible test version here - freezy - 2013-01-16
I'll need the log then.