Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#2887 closed patch (fixed)

More updates for master_icon map

Reported by: skd5aner@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: 0.20
Severity: medium Keywords:
Cc: Ticket locked: no

Description

My first patch for the project, as trivial as it may be... I hope it helps.

This patch includes updates for several local stations in the Chicago area as well as a few national networks that had not been mapped yet. Also, the OLN (Outdoor Life Network) icon was pulling back incorrectly, I have fixed it to pull from the lyngsat site.

Please let me know if you have any questions.

Attachments (2)

master_iconmap.patch (8.2 KB) - added by skd5aner@… 17 years ago.
master_iconmap-v2.patch (13.1 KB) - added by skd5aner 17 years ago.

Download all attachments as: .zip

Change History (7)

Changed 17 years ago by skd5aner@…

Attachment: master_iconmap.patch added

comment:1 Changed 17 years ago by danielk

Resolution: fixed
Status: newclosed

(In [12551]) Fixes #2887. Updates master icon map for Chicago.

I did not add the PAX channel, it looks like that channel is now an ITV channel.

comment:2 Changed 17 years ago by skd5aner

Resolution: fixed
Status: closedreopened

I think a problem occured when trying to apply this patch to the new XML file: trunk/mythtv/contrib/master_iconmap/master_iconmap.xml

All the the network URLs that were committed with this patch currently look like this: <network>CBS-WBBM</network> <url>[ls-logo]/ww/wbbm_cbs2_chicago.jpg</url>

but they need to include a "/tv" between the [ls-logo] and the existing url... so that it would look like this: <network>CBS-WBBM</network> <url>[ls-logo]/tv/ww/wbbm_cbs2_chicago.jpg</url>

Otherwise it'll return a 404 error and not pull back the URL.

comment:3 Changed 17 years ago by skd5aner

I'm working on a patch to address the URL issues as we speak... should upload later today.

comment:4 Changed 17 years ago by danielk

Resolution: fixed
Status: reopenedclosed

(In [12552]) Fixes #2887. Fixes some links committed in [12551], the format was changed in the last few days and I forgot about the change. Also grabs an orange "I" for the WCPX.

Changed 17 years ago by skd5aner

Attachment: master_iconmap-v2.patch added

comment:5 Changed 17 years ago by skd5aner

oops, didn't realize Daniel already did this on his own. I did submit an update patch to fix the issues, but just in case, I also addresses some further changes.

I'll probably submit a new ticket and a patch to address the additional issues - not included the fixes already fixed for this ticket.

Thanks!

Note: See TracTickets for help on using tickets.