Opened 16 years ago
Closed 16 years ago
Last modified 14 years ago
#8777 closed enhancement (fixed)
EIT enhancements for DishNet
| Reported by: | Owned by: | Stuart Auchterlonie | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.24 |
| Component: | MythTV - EIT | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | yes |
Attachments (8)
Change History (19)
by , 16 years ago
| Attachment: | bell_from_dish.diff added |
|---|
comment:1 by , 16 years ago
| Status: | new → assigned |
|---|
comment:2 by , 16 years ago
| Status: | assigned → infoneeded |
|---|
No point applying this as is. I'll wait for further patches that (that i assume are coming) that actually make some use of the changes.
Stuart
by , 16 years ago
| Attachment: | dish_descriptors1.diff added |
|---|
Define some of the descriptors we'll be using.
by , 16 years ago
| Attachment: | dish_descriptors3.diff added |
|---|
Start using some of the decoded descriptors. Ratings and stars will follow.
by , 16 years ago
| Attachment: | dish_descriptors4.diff added |
|---|
Move ratings from ProgEvent to DBEvent and add stars to DBEvent.
by , 16 years ago
| Attachment: | dish_descriptors5.diff added |
|---|
Final patch to eitfixup for Bell & Dish.
by , 16 years ago
| Attachment: | dish_descriptors6.diff added |
|---|
Set airdate correctly from EIT for Dish/Bell.
comment:3 by , 16 years ago
| Milestone: | unknown → 0.24 |
|---|---|
| Status: | infoneeded → assigned |
comment:4 by , 16 years ago
| Status: | assigned → infoneeded |
|---|
Mark,
I applied your patches locally but there were some compile issues (Two instances of DBEventEIT hadn't been updated for the stars argument). I realize that you may have been porting your patches without a trunk system to work on, but is there any chance you can test the updated combined patch that I added? It now compiles properly, and takes into account other changes made in trunk since, but as I do not have access to this EIT info I cannot test.
I also made some slight formatting changes (to avoid an empty set of if brackets, and because QStrings don't need to be initialized to be empty/non-NULL in Qt4). Otherwise this should be as originally written.
comment:5 by , 16 years ago
I'm having several people test this so it may be another week before I get good results.
- Mark.
comment:10 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | infoneeded → closed |
comment:11 by , 14 years ago
| Ticket locked: | set |
|---|

Separate DishNet fixup from Bell so we can distinguish between the two later.