Bug 15328 - CLI artists query should respect Accept-Language header for the Various Artists item when called via Comet
: CLI artists query should respect Accept-Language header for the Various Artis...
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: CLI
: 7.5.0
: Other Other
: P1 normal (vote)
: 7.5.0
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-12-18 08:50 UTC by Andy Grundman
Modified: 2010-04-08 17:24 UTC (History)
3 users (show)

See Also:
Category: Bug


Attachments
add language override to requests (1.75 KB, patch)
2009-12-21 05:14 UTC, Michael Herger
Details | Diff
localize VA object in artists query (856 bytes, patch)
2009-12-22 02:09 UTC, Michael Herger
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Grundman 2009-12-18 08:50:48 UTC

    
Comment 1 Joerg Schwieder 2009-12-19 17:24:58 UTC
As there is now a bug for this...

This also affects the "Various Artists" item in the artists list, that one also is shown as set in the server settings instead of in the "Accept-Language" language.
Comment 2 Andy Grundman 2009-12-20 08:04:36 UTC
Yeah, really any CLI call that calls string() should respect Accept-Language.  This may be tricky for some cases, as right now the language override value is stored in $client.
Comment 3 Michael Herger 2009-12-21 05:14:06 UTC
Created attachment 6397 [details]
add language override to requests

Jörg - could you please test this?

Client language has priorty over override.
Comment 4 Joerg Schwieder 2009-12-21 15:49:40 UTC
"getstring" looks good with the patch.

"Vaious Artists" is still "Various Artists", though, instead of "Diverse Interpreten" (or "Diverse Artisten" how the scanner would call it :) )
Comment 5 SVN Bot 2009-12-22 00:43:09 UTC
 == Auto-comment from SVN commit #29668 to the slim repo by michael ==
 == https://svn.slimdevices.com/slim?view=revision&revision=29668 ==

Bug: 15328
Description: respect "Accept-Language" headers from JSON/RPC requests to override server language settings when no client is available.
Comment 6 Michael Herger 2009-12-22 02:09:08 UTC
Created attachment 6400 [details]
localize VA object in artists query

Jörg - would I need a more recent iPeng to test this? My 1.2.1 is EN only?
Comment 7 Joerg Schwieder 2009-12-22 02:27:04 UTC
Yes. I'll send you one later today.
1.2.1 uses the language setting of the server but that won't work with localization since iPhone automatically sets it's default language so future iPeng versions will use the client language.
Comment 8 SVN Bot 2009-12-22 05:45:23 UTC
 == Auto-comment from SVN commit #7825 to the network repo by michael ==
 == https://svn.slimdevices.com/network?view=revision&revision=7825 ==

Bug: 15328
Description: roll out workaround for dropped UUID in json issue
Comment 9 Michael Herger 2009-12-22 05:48:51 UTC
Oops... wrong bug. Ignore please.
Comment 10 SVN Bot 2009-12-23 05:17:34 UTC
 == Auto-comment from SVN commit #29676 to the slim repo by michael ==
 == https://svn.slimdevices.com/slim?view=revision&revision=29676 ==

Fixed Bug: 15328
Description: localize "Various Artists" item in artists query when language override is requested by the controlling client
Comment 11 Joerg Schwieder 2010-01-01 19:34:03 UTC
I'm not 100% sure whether this is another "ignoring" of AcceptLanguage or
https://bugs-archive.lyrion.org/show_bug.cgi?id=15333

But the "tooltips" for Love/Ban icons (the ones for the Repeat/Shuffle button replacements on the NowPlaying screens, not the context menus) are not localized, at least for Last.fm and Slacker. Slacker might not need to be localized since it's only available for the US, but Last.fm would be nice.

I use the tooltips for the accessibility (screen reader) support in iPeng and VoiceOver really reads stuff badly if it's in the wrong language.
Comment 12 Chris Owens 2010-04-08 17:24:07 UTC
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release.

If you are still seeing this issue, please let us know!