Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 7 months ago

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

Download all attachments as: .zip

Change History (6)

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

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

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

comment:1 Changed 7 months ago by stuartm

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

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

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

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

Fixed by [d34c696], please close.

comment:3 Changed 6 months ago by kenni

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

Add Comment

Modify Ticket

Action
as closed .
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.