Opened 12 years ago

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

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by Bill Meek <keemllib@…>

Attachment: version.txt​ added

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

Changed 12 years ago by Bill Meek <keemllib@…>

Attachment: channelIconsSG.patch.0​ added

comment:1 Changed 12 years ago by stuartm

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

Changed 12 years ago by Bill Meek <keemllib@…>

Attachment: channelIconsSG.patch.1​ added

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

comment:2 Changed 12 years ago by Bill Meek <keemllib@…>

Fixed by [d34c696], please close.

comment:3 Changed 12 years ago by Kenni Lund [kenni a kelu dot dk]

Resolution: → Fixed
Status: accepted → closed

Fixed in 85462fbc670eb3efc4d3d66cf4708541a51199f1 and d34c696d2308bc1318b2704781bc15c0b4319b8c

Note: See TracTickets for help on using tickets.