From: Geoff <bil...@us...> - 2014-10-20 10:03:51
|
Update of /cvsroot/xmltv/xmltv/grab/is In directory sfp-cvs-1.v30.ch3.sourceforge.com:/tmp/cvs-serv19486 Modified Files: tv_grab_is Log Message: (major) fix broken grabber (many programmes missing from output xml) Index: tv_grab_is =================================================================== RCS file: /cvsroot/xmltv/xmltv/grab/is/tv_grab_is,v retrieving revision 1.30 retrieving revision 1.31 diff -C2 -d -r1.30 -r1.31 *** tv_grab_is 9 May 2014 15:35:51 -0000 1.30 --- tv_grab_is 20 Oct 2014 10:03:49 -0000 1.31 *************** *** 639,642 **** --- 639,650 ---- # NOTE: For Stod 2 et al, we ignore $fromdate and $num_days and always load the next week. + # [honir] I don't know what this piece of code was intending to do: + # <xsl:apply-templates select="event[not(reference_number/\@value=following::event/reference_number/\@value)]" /> + # Specifically + # event[not(reference_number/\@value=following::event/reference_number/\@value)] + # ignores nodes where the same 'reference_number' occurs on any following node. But the 'reference_number' is not unique to a showing, i.e. a repeat + # showing of the programme will have the same 'reference_number'; however this code will ignore all matching 'event' except the last! + # I.e. we only grab the last repeated showing of a programme. This can't be what was intended! + # my $xsl_transform = <<"EOF"; <xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0"> *************** *** 646,650 **** <!-- XML needs a toplevel node, so we wrap the programmes in a channel_wrapper element. The code will then remove it --> <channel_wrapper> ! <xsl:apply-templates select="event[not(reference_number/\@value=following::event/reference_number/\@value)]" /> </channel_wrapper> </xsl:template> --- 654,658 ---- <!-- XML needs a toplevel node, so we wrap the programmes in a channel_wrapper element. The code will then remove it --> <channel_wrapper> ! <xsl:apply-templates select="event"/> </channel_wrapper> </xsl:template> |