Closed GoogleCodeExporter closed 8 years ago
I whipped up a quick patch for this one, but can't test it until later.
Original comment by brian.wa...@gmail.com
on 28 Dec 2010 at 3:56
Attachments:
Should we also test for enough slides to fill the set total slide time?
change the if statement in your patch from:
if ( len( self.slide_playlist ) == 0 and self.settings[ "trivia_unwatched_only"
] and len( self.watched ) > 0 ):
to
if ( ( len( self.slide_playlist ) == 0 or ( len(
self.slide.playlist)*self.settings[ "trivia_slide_time" ] )/60 < self.settings[
"trivia_total_time" ] ) and self.settings[ "trivia_unwatched_only" ] and len(
self.watched ) > 0 ):
Original comment by giftie61@gmail.com
on 28 Dec 2010 at 6:54
We should also do this for the local trailer scraper.
Original comment by giftie61@gmail.com
on 28 Dec 2010 at 7:03
I tested your alteration and the script gets into an endless loop if there
aren't sufficient trivia slides for the time limit
Original comment by brian.wa...@gmail.com
on 2 Jan 2011 at 1:03
submitted this patch to SVN
Original comment by brian.wa...@gmail.com
on 2 Jan 2011 at 3:07
Original issue reported on code.google.com by
brian.wa...@gmail.com
on 28 Dec 2010 at 3:33