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: | 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)
Change History (7)
Changed 17 years ago by
Attachment: | master_iconmap.patch added |
---|
comment:1 Changed 17 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 Changed 17 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
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
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
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Changed 17 years ago by
Attachment: | master_iconmap-v2.patch added |
---|
comment:5 Changed 17 years ago by
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!
(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.