Opened 13 years ago

Closed 13 years ago

#11865 closed Patch - Bug Fix (Fixed)

Patch: Services API GetChannelIcon doesn't respect Storage Group: ChannelIcons

Reported by: Bill Meek <keemlib@…> Owned by: stuartm
Priority: minor Milestone: 0.27.1
Component: MythTV - Services API - Backend Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

If <backend>:6544/Channel/GetChannelIcon?ChanId=n is used, and the backend has been configured to use the ChannelIcons StorageGroup and the user has correctly used used file names for their icons (not full path names,) then the backend log will look like: 

2013-09-17 18:55:40.199823 N [3204/8242] ProcessRequest mainserver.cpp:5326 (HandleSetVerbose) - Verbose mask changed, new mask is:  general upnp
2013-09-17 18:55:53.850059 I [3204/8227] HttpServer60 httprequest.cpp:1366 (ExtractMethodFromURL) - ExtractMethodFromURL(end) : GetChannelIcon : /Guide
2013-09-17 18:55:53.850097 I [3204/8227] HttpServer60 servicehost.cpp:294 (ProcessRequest) - ServiceHost::ProcessRequest: GetChannelIcon : GET /Guide/GetChannelIcon?ChanId=1051 HTTP/1.1
2013-09-17 18:55:53.850265 I [3204/8227] HttpServer60 httprequest.cpp:764 (FormatFileResponse) - HTTPRequest::FormatFileResponse() - cannot find file
2013-09-17 18:55:53.850313 I [3204/8227] HttpServer60 httprequest.cpp:244 (SendResponse) - HTTPRequest::SendResponse(xml/html) () :404 Not Found -> 192.168.1.200: 2

The patch allows the use of the ChannelIcons StorageGroup for users with and without full path names in their mythconverg channel table.

Attachments (3)

version.txt​ (890 bytes ) - added by Bill Meek <keemllib@…> 13 years ago.
--version ( v0.28-pre-140-g9d0b80b-dirty)
channelIconsSG.patch.0​ (980 bytes ) - added by Bill Meek <keemllib@…> 13 years ago.
channelIconsSG.patch.1​ (1.9 KB ) - added by Bill Meek <keemllib@…> 13 years ago.
Added LOG messages (with -v upnp & -loglevel debug,) proper Storage Group selection.

Download all attachments as: .zip

Change History (6)

by Bill Meek <keemllib@…>, 13 years ago

Attachment: version.txt​ added

--version ( v0.28-pre-140-g9d0b80b-dirty)

by Bill Meek <keemllib@…>, 13 years ago

Attachment: channelIconsSG.patch.0​ added

comment:1 by stuartm, 13 years ago

Milestone: unknown → 0.27.1
Owner: set to stuartm
Status: new → accepted

by Bill Meek <keemllib@…>, 13 years ago

Attachment: channelIconsSG.patch.1​ added

Added LOG messages (with -v upnp & -loglevel debug,) proper Storage Group selection.

comment:2 by Bill Meek <keemllib@…>, 13 years ago

Fixed by [d34c696], please close.

comment:3 by Kenni Lund [kenni a kelu dot dk], 13 years ago

Resolution: → Fixed
Status: accepted → closed

Fixed in 85462fbc670eb3efc4d3d66cf4708541a51199f1 and d34c696d2308bc1318b2704781bc15c0b4319b8c

Note: See TracTickets for help on using tickets.