Bug 11329 - CLI : "save playlist" command with unicode characters incorrect
: CLI : "save playlist" command with unicode characters incorrect
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: CLI
: 7.3.3
: PC Windows Home Server
: -- normal (vote)
: 7.4.0
Assigned To: Michael Herger
: charset_issues
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-12 01:43 UTC by pulp_136
Modified: 2009-10-05 14:37 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description pulp_136 2009-03-12 01:43:07 UTC
The command:
playlist save Λίστα

returns:
playlist save ?I?I?I±

The saved playlist appears with incorrect characters also on the web interface.

If I save a playlist with unicode characters through the web interface everything is OK. The webif and the cli can read it and play it correctly. So, the problem is specifically with the "playlist save" cli command.

SqueezeCenter 7.3.3 on WHS
CLI Client on WinXP Home

This seems very similar (the wrong characters too) to the Bug 8132 which was just resolved. Maybe it's a similar solution.

At your disposal for any additional information.

pulp
Comment 1 Michael Herger 2009-03-12 05:15:21 UTC
Are you using Windows telnet client to issue that CLI command? Or are you running the command programmatically from an application?
Comment 2 pulp_136 2009-03-12 05:57:39 UTC
(In reply to comment #1)
> Are you using Windows telnet client to issue that CLI command? Or are you
> running the command programmatically from an application?

Programmatically from a vb.net application. I'm not using any encoding before sending the string. 

Similary, I don't use any encoding when sending other strings e.g. to play a file or add it to the playlist. With the difference that all other commands work, even when greek characters are included.
Comment 3 Andy Grundman 2009-03-20 14:40:29 UTC
Would URI-escaping help here?
Comment 4 pulp_136 2009-04-14 00:24:33 UTC
(In reply to comment #3)
> Would URI-escaping help here?

Andy, does your question target me (the app programmer) or the CLI developers?
Comment 5 Michael Herger 2009-04-14 00:30:14 UTC
> > Would URI-escaping help here?
> 
> Andy, does your question target me (the app programmer) or the CLI developers?

I can't speak for Andy, but I would indeed have asked you to run this test. Because URI escaping is the way queries should be submitted by design.
Comment 6 pulp_136 2009-04-14 00:40:05 UTC
(In reply to comment #5)
> > > Would URI-escaping help here?
> > 
> > Andy, does your question target me (the app programmer) or the CLI developers?
> 
> I can't speak for Andy, but I would indeed have asked you to run this test.
> Because URI escaping is the way queries should be submitted by design.

OK Michael, I will.
Comment 7 pulp_136 2009-04-14 22:36:11 UTC
(In reply to comment #3)
> Would URI-escaping help here?

Sending the string URI-escaped makes the playlists appear correctly both in the webif and the CLI (playlists x y command). Saving the first playlist that starts with a unicode character is OK. Every other attempt works too, but the new playlist replaces the first one. This is only for playlists saved uri-escaped through the cli that also start with a unicode character. Playlists either saved through the webif or have a non-unicode first character are not affected.

I hope it's clear, at your desposal for any clarification.
pulp
Comment 8 Michael Herger 2009-04-15 02:49:58 UTC
> I hope it's clear, at your desposal for any clarification.

I'm sorry, I'm not sure. IMHO it works as advertised if you user URI-escaping (which is what the CLI is expecting). Anything which wouldn't work?
Comment 9 Michael Herger 2009-04-22 03:49:27 UTC
pulp - do you still see issues when using URI escaping?
Comment 10 Michael Herger 2009-06-08 11:15:30 UTC
pulp - could you please try the latest 7.4 nightly build?
Comment 11 pulp_136 2009-06-11 04:26:40 UTC
(In reply to comment #10)
> pulp - could you please try the latest 7.4 nightly build?

I will today and report.
Sorry for my long absence.
Comment 12 pulp_136 2009-06-13 11:18:35 UTC
(In reply to comment #10)
> pulp - could you please try the latest 7.4 nightly build?

It works with the latest 7.4 (WHS).

Any chance you add the fix to the 7.3.3 too, since 7.4 is unusable for me? (it only scans half of my library, and hangs quite a lot).

thanks
pulp
Comment 13 Michael Herger 2009-06-14 23:39:12 UTC
> It works with the latest 7.4 (WHS).

Great, thanks for the feedback!

> Any chance you add the fix to the 7.3.3 too, since 7.4 is unusable for me? (it
> only scans half of my library, and hangs quite a lot).

No, no backporting, as most of the scanning code in 7.4 has been rewritten. Please test with the latest 7.4 nightlies and report any bug you see (unless they've been reported already). Thanks!
Comment 14 pulp_136 2009-06-15 02:54:18 UTC
> Great, thanks for the feedback!

Thanks for your efforts.


> No, no backporting, as most of the scanning code in 7.4 has been rewritten.

I'm a little confused by it. It was my understanding that no new features will make it to the 7.3.3 beta, but bug fixes? I remember reading that the 7.3.3 "stable" is to be released soon and I expected that you are working mainly on fixing bugs and especially the already resolved ones will make it there.


> Please test with the latest 7.4 nightlies and report any bug you see (unless
> they've been reported already). Thanks!

I'm back to 7.3.3 for good, but I will report my experiences with 7.4 in the forum.

thanks again
pulp
Comment 15 James Richardson 2009-10-05 14:37:10 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.