Opened 14 years ago
Closed 14 years ago
Last modified 14 years ago
#8763 closed patch (fixed)
UPnP server does not work with Windows 7
Reported by: | Owned by: | beirdo | |
---|---|---|---|
Priority: | minor | Milestone: | 0.24 |
Component: | MythTV - UPnP | Version: | Master Head |
Severity: | medium | Keywords: | UPnP Windows Media Player |
Cc: | Ticket locked: | no |
Description
This patch adds code to detect the UPnP client and then uses that client identification to modify it's output to Windows Media Player 12 (Only on Windows 7).
This fixes the following:
- Recorded Programs are listed under RecordedTV in WMP12
- Entries are only list once (instead of 6 or more times)
Attachments (2)
Change History (8)
Changed 14 years ago by
Attachment: | upnp-wmp12.patch added |
---|
comment:1 Changed 14 years ago by
Owner: | changed from dblain to beirdo |
---|---|
Status: | new → assigned |
comment:2 Changed 14 years ago by
Status: | assigned → accepted |
---|
Thanks for the patch. I'm working through it now. However, in the future, please be SURE to not use tabs, but rather 4 spaces for indentation. I've cleaned this one up though.
Changed 14 years ago by
Attachment: | 0001-Reworked-patch-from-8763-WMP12-support.patch added |
---|
comment:3 Changed 14 years ago by
Status: | accepted → infoneeded |
---|
Please double-check the sections where you had removed the check for ContainerID 13 for WMP, I put that check back in, but I'd like confirmation on that before this is committed. I also redid the DetermineClient? section a bit into an easier to maintain chunk. I do not have WMP12 (or any other version) on the network, so I can not test this.
Please check my patch against trunk, and let me know if this still does what you need.
comment:4 Changed 14 years ago by
Ok, point taken about the spaces. I though I'd got them all! Sorry for that!
I believe the ContainerID was being used to determine if it was actually a WMP client, as the previous code did not have any other way of identifying clients, that is why I removed it. However, I have checked your code against WMP12 and WMP11 and it works fine, so I am happy for it to be committed as you have it. I believe it is a bit belt & braces, but it doesn't matter so if you could commit it that would be very nice of you!
Thanks, Matt.
comment:5 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | infoneeded → closed |
comment:6 Changed 14 years ago by
Milestone: | unknown → 0.24 |
---|
Patch for UPnP wmp12 support