xmltv/grab/is tv_grab_is,1.30,1.31

Geoff <[email protected]>
Newsgroups gmane.comp.tv.xmltv.cvs
Message-ID <[email protected]>
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>


------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://p.sf.net/sfu/Zoho
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.