nolenfelten / xbmc-addons

Automatically exported from code.google.com/p/xbmc-addons
0 stars 0 forks source link

[cinema.experience] Reset watched trivia automatically if all watched #72

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Requested by ijourneaux on the XBMC forums:

Turns out that with a movie watching marathon over the holidays, the trivia 
slides had all been watched. Thus no Triva show. Reseting the viewed list got 
it too work again.

Perhaps if trivia is enabled and there are no trivia slides, the watched list 
should be reset automatically?

http://forum.xbmc.org/showpost.php?p=674883&postcount=340

Original issue reported on code.google.com by brian.wa...@gmail.com on 28 Dec 2010 at 3:33

GoogleCodeExporter commented 9 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:

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
We should also do this for the local trailer scraper.

Original comment by giftie61@gmail.com on 28 Dec 2010 at 7:03

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
submitted this patch to SVN

Original comment by brian.wa...@gmail.com on 2 Jan 2011 at 3:07