Modify

Opened 18 months ago

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

Download all attachments as: .zip

Change History (6)

Changed 18 months ago by Bill Meek <keemllib@…>

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

Changed 18 months ago by Bill Meek <keemllib@…>

comment:1 Changed 18 months ago by stuartm

  • Milestone changed from unknown to 0.27.1
  • Owner set to stuartm
  • Status changed from new to accepted

Changed 17 months ago by Bill Meek <keemllib@…>

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

comment:2 Changed 17 months ago by Bill Meek <keemllib@…>

Fixed by [d34c696], please close.

comment:3 Changed 17 months ago by kenni

  • Resolution set to Fixed
  • Status changed from accepted to closed

Add Comment

Modify Ticket

Action
as closed The owner will remain stuartm.
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.