Opened 10 years ago
Closed 9 years ago
Last modified 8 years ago
#12786 closed Patch - Bug Fix (fixed)
xmltv season parsing with total seasons supplied
| Reported by: | Owned by: | Peter Bennett | |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | MythTV - Mythfilldatabase | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
xmltv season parsing with total seasons number supplied
Per the xmltv dtd, the xmltv_ns episode numbering system allows for a season [aka series], and not just an episode or part, to have two sections, one before (the value) and one after (the total) a separating '/'. Currently, from a quick scan of the code, it appears that the code would not process the season number properly if that format (season/totalSeason) was provided by the grabber.
Completely untested patch to support such grabbers:
diff --git a/mythtv/programs/mythfilldatabase/xmltvparser.cpp b/mythtv/programs/mythfilldatabase/xmltvparser.cpp
index d6799ad..547cbda 100644
--- a/mythtv/programs/mythfilldatabase/xmltvparser.cpp
+++ b/mythtv/programs/mythfilldatabase/xmltvparser.cpp
@@ -462,6 +462,7 @@ ProgInfo *XMLTVParser::parseProgram(QDomElement &element)
totalepisodes = episode.section('/',1,1).trimmed();
episode = episode.section('/',0,0).trimmed();
season = episodenum.section('.',0,0).trimmed();
+ season = season.section('/',0,0).trimmed();
QString part(episodenum.section('.',2,2));
QString partnumber(part.section('/',0,0).trimmed());
QString parttotal(part.section('/',1,1).trimmed());
Change History (4)
Note:
See TracTickets
for help on using tickets.

I have tested this patch and it solves a problem with series id from xmltv.