Bugzilla – Bug 2386
status.html (partially?) not uf8 encoded
Last modified: 2008-09-15 14:39:24 UTC
_Some_ elements of status.html do not seem to be utf8 encoded. When I add the big5.mp3 from the test folder to my playlist, it shows up correctly in the playlist (as far as I can read the signs :-)), but garbage in the status panel. I think this is an issue I've seen before when using Handheld, but now with EN, Moser, NBMU, too.
is this really only the official release where you see it? anything like it in trunk or branch?
Nope, I didn't verify the version, assuming it was trunk. I'm experiencing this with the trunk version and the unicode files from the test folder.
in the case of EN, it appears status.html still uses browseid3 for one. it also uses artist and album, whereas the playlist is using itemobj.artist and itemobj.album. Perhaps this is a common pattern?
You're right, it's using album instead of itemobj.album.title that causes problems. Tested with EN and Moser.
I'm of the mind to standardise using itemobj values for web UI. (I've already converted several cases thus far) sound good to you, robert?
Please do.
Definitely. Although if it looks to be a lot of work I would hold off on it until we convert to looping in TT rather than in Pages.pm, since that will involve some rewriting anyway.
its not very tough. I've already got EN fixed. It is just a case of using tracklink.html for most of them. Of course, for Moser skin, it would need a custom version of tracklink.html to fit the standard. Since you use the same formatting for the "Now playing" line at the top, as your playlist, it should work out nicely. At least, that's what I'll do to get 6.2.1 looking ok (in fact, I'll just tweak the text parts of the link and convert to browsedb for anything that isn't)
quick fix committed to trunk at change 4889 modified version to fit 6.2.1 committed at change 4890 lots of room for optimisation left as the looping is brought in, but this problem should now be fixed.