Opened 11 years ago

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

Download all attachments as: .zip

Change History (6)

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

Attachment: version.txt added

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

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

Attachment: channelIconsSG.patch.0 added

comment:1 Changed 11 years ago by stuartm

Milestone: unknown0.27.1
Owner: set to stuartm
Status: newaccepted

Changed 10 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 10 years ago by Bill Meek <keemllib@…>

Fixed by [d34c696], please close.

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

Resolution: Fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.