Eldorado Wrote:I was considering it, but then thought it's probably best that I don't do that in case things such as the youtube addon changes, or you just would like to play it differently for whatever reason eg. with the urlresolver

I guess this is up for debate, generally I like to pass back unmolested data which may force you to take certain directions unless there is a specific reason for touching it, this could be a case where I do.. but I'm very 50/50 on it

Anyone want to chime in on the topic?

k_zeon Wrote:Eldorado, How would you call the search feature to update a single Movie. If on Apple TV2 , would it be select movie then hold menu to popup context menu...

k_zeon Wrote:What about creating a switch where when calling Get_Meta

ie
meta = metaget.get_meta('movie', CleanHTML(name), year=sYear, Trailer=??)

?? could be Youtube or Plain etc.... Just an idea

Otherwise when adding meta to the Dir creating routine in our addon , we add the Trailer as another dict.

Code:

addon.add_Metadirectory({'mode' : 'GetMovieSource', 'url' : url }, meta ,total_items=len(match),title=CleanHTML(name) + '  (' +  sYear + ')' , img=meta['cover_url'],trailer=resolvedYoutubeURL)

k_zeon Wrote:I got the Youtube trailers playing from the info screen.

No idea how to use the refresh button on the info screen. would be cool.

Must be set by a boolean true=active. thought maybe it was an infolabel

ie infolabel['refresh']=true

Eldorado Wrote:Haven't seen anything such as that, but then you would also need to step in and handle the code that the button executes..

I've been playing with context menus and testing single movie refresh, as it turns out my current implementation in metahandlers isn't so great I've revamped it a bit, still a bit messy.. once I'm happy with it, I'll push another version out

But the watched/unwatched status seems to work well

Now need to find t0mm0 and get him to add contextmenu support to his library!

k_zeon Wrote:Dragonwin has context support and he has even made a fork on t0mm0s module.
I did see it on his git hub before but not sure if its there now.
Maybe give him a text.

Eldorado Wrote:It would be nice to get to the point of using t0mm0's libraries with Icefilms, but I can only assume there will need to be major re-structuring of the code to get there.. the 1.1.0 release will likely just focus on the already included new features, then if anyone wants to take the challenge after that to rework it with t0mm0's libs

Can you post up the full debug log where you are getting that error? Basicalliy means it is trying to insert an exact duplicate row into the episode_meta table, and for some reason it did not find and delete the original one before trying to add the new

slyi Wrote:Sorry i have not been on in while, I need a couple of helper functions added to metahandler.py https://github.com/slyi/xbmc-metautils/blob/master/script.module.metahandler/lib/metahandler/metahandlers.py before i can implement it in icefilms. Can you please review. I still learning gitgui so excuse my rookie mistakes ! :-)