addon.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
-<addon id="plugin.video.vidfltr" name="VIDFLTR Music Videos" version="0.9.7" provider-name="bassdart">
+<addon id="plugin.video.vidfltr" name="VIDFLTR Music Videos" version="0.9.5" provider-name="bassdart">
   <requires>
     <import addon="xbmc.python" version="2.1.0"/>
     <import addon="script.module.kodi-six" />
@@ -23,22 +23,7 @@
     <website>https://vidfltr.slashproc.org/</website>
     <email></email>
     <source>https://kodi.slashproc.org/repo/plugin.video.vidfltr/</source>
-    <assets>
+    <news>0.9.5
-        <icon>resources/img/kodi/icon.png</icon>
-        <fanart>resources/img/kodi/fanart.jpg</fanart>
-        <banner>resources/img/kodi/banner.jpg</banner>
-        <clearlogo>resources/img/kodi/clearlogo.png</clearlogo>
-        <screenshot></screenshot>
-    </assets>
-    <news>0.9.7
-- Vimeo Add-on is broken, play with Youtube-DL until it's fixed
-0.9.6
-- add display of official/unofficial count in main artist and country folders
-- fix pager enumeration in main artist and country if Video Selection isn't set to show all videos
-- don't list artists with known country also in Unknown
-- add page counter for main artists, countries and sorted folders
-- various other fixes and cleanups
-0.9.5
 - fix display of quotation marks in video lists
 - add pagination for artists on main page and countries
 - fix Musly (similar music) sort order and filter out dupes

changelog.txt

@@ -1,11 +1,3 @@
-0.9.7
-- Vimeo Add-on is broken, play with Youtube-DL until it's fixed
-0.9.6
-- add display of official/unofficial count in main artist and country folders
-- fix pager enumeration in main artist and country if Video Selection isn't set to show all videos
-- don't list artists with known country also in Unknown
-- add page counter for main artists, countries and sorted folders
-- various other fixes and cleanups
 0.9.5
 - fix display of quotation marks in video lists
 - add pagination for artists on main page and countries

default.py

@@ -18,12 +18,11 @@ addon = xbmcaddon.Addon(id=addonID)
 pluginhandle = int(sys.argv[1])
 translation = addon.getLocalizedString
 addonDir = xbmc.translatePath(addon.getAddonInfo('path'))
-defaultFanart = os.path.join(addonDir, 'resources/img/kodi/noicon.png')
+defaultFanart = os.path.join(addonDir, 'resources/images/noicon.png')
-icon = os.path.join(addonDir, 'resources/img/kodi/icon.png')
+#fanart = os.path.join(addonDir, 'noicon.png')
-clearlogo = os.path.join(addonDir, 'resources/img/kodi/clearlogo.png')
+# don't use special folder icons as long as there is'nt a nice icon for every style
-fanart = os.path.join(addonDir, 'resources/img/kodi/fanart.png')
+fanart = 'DefaultFolder.png'
-banner = os.path.join(addonDir, 'resources/img/kodi/banner.png')
+icon = os.path.join(addonDir, 'noicon.png')
-poster = os.path.join(addonDir, 'resources/img/kodi/poster.png')
 addon_work_folder = xbmc.translatePath("special://profile/addon_data/" + addonID)
 jsonVideos = xbmc.translatePath("special://profile/addon_data/" + addonID + "/videos.json")
 jsonArtists = xbmc.translatePath("special://profile/addon_data/" + addonID + "/artists.json")
@@ -33,14 +32,8 @@ maxFileAge = maxFileAge * 60
 mediatype = addon.getSetting("mediatype")
 # show only official, fanmade or all videos?
 videoselection = str(addon.getSetting("videoselection")).lower()
-if videoselection != "2":
-    # videoselection is set to official or fanmade
 # despite this selection show all in similar playlists and more from?
 relatedselection = str(addon.getSetting("relatedselection")).lower()
-else:
-    # if videoselection is set to show all videos relatedselection should
-    # also be true if the setting was set to false beforehand
-    relatedselection = "true"
 playLocalFile = str(addon.getSetting("playLocalFile")).lower()
 filesinlists = int(addon.getSetting("filesinlists"))
 useYTDL = addon.getSetting("useytdl")
@@ -254,14 +247,6 @@ def play(url):
                         idVideo = entry['purl']
                         ytdl = YDStreamExtractor.getVideoInfo(idVideo)
                         playback_url = ytdl.streamURL()
-                    else:
-                        if "vimeo" in entry['provider']:
-                            # quick hard coded workaround
-                            xbmc.log(msg="vimeo addon is broken, playing with ytdl", level=xbmc.LOGNOTICE)
-                            import YDStreamExtractor
-                            idVideo = entry['purl']
-                            ytdl = YDStreamExtractor.getVideoInfo(idVideo)
-                            playback_url = ytdl.streamURL()
                     else:
                         # without youtube_dl
                         xbmc.log(msg="without ytdl", level=xbmc.LOGDEBUG)
@@ -280,7 +265,6 @@ def play(url):
         
 #    doesn't work       
 #    item.setProperty('StartOffset', '56.4')
-#    Bug? It maybe works with StartPercent, see https://forum.kodi.tv/showthread.php?tid=129188&pid=2443114#pid2443114
 #    item.setProperty("VolumeAmplification", "7")
 #    xbmc.log(msg=item.getProperty('VolumeAmplification'), level=xbmc.LOGNOTICE)
 
@@ -518,8 +502,6 @@ def sortArtists(channel=""):
             for artist in channels:
                 if artist in [unidecode(name.upper()) for name in entry['artists']]:
                     if entry not in result:
-                        # limit selection based on videoselection setting if show all isn't activated
-                        if relatedselection != "true" and videoselection != "2":
                         if videoselection == "0":
                             if "true" in entry['official'].lower():
                                 result.append(entry)
@@ -557,16 +539,20 @@ def sortTitlesBy(limit, sort, start):
     start = int(start)
     end = start + filesinlists
     nextstart = end + 1
-    fanart = getFanart(channel)
 
-    # limit selection if "all" isn't activated in videoselection
+    fanart = getFanart(channel)
-    if videoselection != "2"  and channel != "related" and channel != "relartists":
-        data = limitselection(data)
 
     # limit to hits
     if hit != "" and hit == "true":
         for entry in data:
             if entry['hit'] == "true":
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
         data = result
         result = []
@@ -576,6 +562,13 @@ def sortTitlesBy(limit, sort, start):
         for entry in data:
             # played often at provider
             if int(entry['pcount']) >= pcounthigh:
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
         data = result
         result = []
@@ -584,6 +577,13 @@ def sortTitlesBy(limit, sort, start):
     if mycount != "" and mycount == "acount" and sort != "count":
         for entry in data:
             if int(entry['acount']) >= acounthigh:
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
         data = result
         result = []
@@ -592,6 +592,13 @@ def sortTitlesBy(limit, sort, start):
     if mycount != "" and mycount == "comments" and sort != "count":
         for entry in data:
             if int(entry['comments']) >= ccounthigh:
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
         data = result
         result = []
@@ -600,7 +607,15 @@ def sortTitlesBy(limit, sort, start):
     if mycount != "" and mycount == "likes" and sort != "count":
         for entry in data:
             if int(entry['likes']) >= likecounthigh:
+                if videoselection == "0":
+                    if "true" in entry['official']:
                         result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
+                    result.append(entry)
+#                    xbmc.log(msg=str(entry['pcount']), level=xbmc.LOGNOTICE)
         data = result
         result = []
 
@@ -608,6 +623,13 @@ def sortTitlesBy(limit, sort, start):
     if mycount != "" and mycount == "dislikes" and sort != "count":
         for entry in data:
             if int(entry['dislikes']) >= dislikecounthigh:
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
         data = result
         result = []
@@ -616,6 +638,13 @@ def sortTitlesBy(limit, sort, start):
     if mycount != "" and mycount == "controversial":
         for entry in data:
             if int(entry['controversial']) == int(1):
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
         data = result
         result = []
@@ -624,6 +653,13 @@ def sortTitlesBy(limit, sort, start):
     if channel != "" and channel != "all" and channel != "year" and channel != "related":
         for entry in data:
             if entry['style'] == channel:
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
     # or limit to last year
     # hm, either style or year, not both?
@@ -638,6 +674,13 @@ def sortTitlesBy(limit, sort, start):
                 import time
                 dateadded = datetime.fromtimestamp(time.mktime(time.strptime(entry['dateadded'], '%Y-%m-%d %H:%M:%S')))
             if dateadded >= lastyear:
+                if videoselection == "0":
+                    if "true" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "1":
+                    if "false" in entry['official']:
+                        result.append(entry)
+                elif videoselection == "2":
                     result.append(entry)
     # related tracks (generated with musly)
     elif channel != "" and channel == "related":
@@ -649,6 +692,7 @@ def sortTitlesBy(limit, sort, start):
                 slugs.extend(entry['slug'].split(", "))
                 # and add all related slugs
                 slugs.extend(entry['related'].split(", "))
+                #xbmc.log(msg=str(slugs), level=xbmc.LOGNOTICE)
         for entry in data:
             # add all related slugs
             if entry['slug'] in slugs:
@@ -657,6 +701,14 @@ def sortTitlesBy(limit, sort, start):
                 # main slug will be filtered out.
                 # In this case we add it again later, after sorting the result
                 if not filter(lambda result: result['title'] == entry['title'], result):
+                    if videoselection != "2" and relatedselection != "true":
+                        if videoselection == "0":
+                            if "true" in entry['official']:
+                                result.append(entry)
+                        elif videoselection == "1":
+                            if "false" in entry['official']:
+                                result.append(entry)
+                    else:
                         result.append(entry)
         # slugs are sorted newest first because that's how they occur in the json
         # so we have to sort the result to the order in related aka the musly order (with prepended main slug)
@@ -670,9 +722,6 @@ def sortTitlesBy(limit, sort, start):
             for entry in data:
                 if entry['slug'] == start:
                     result.insert(0, entry)
-        if relatedselection != "true":
-            result = limitselection(result)
-
         maximum = len(result)
         # related is a list with max 21 entries, so we have to set start always to 0
         start = 0
@@ -689,15 +738,27 @@ def sortTitlesBy(limit, sort, start):
             xbmc.log(msg=str(artists), level=xbmc.LOGNOTICE)
             # add all entries with artist in artists
             if entry['artists'] in artists:
+                if videoselection != "2" and relatedselection != "true":
+                    if videoselection == "0":
+                        if "true" in entry['official']:
+                            result.append(entry)
+                    elif videoselection == "1":
+                        if "false" in entry['official']:
+                            result.append(entry)
+                else:
                     result.append(entry)
-        xbmc.log(msg=str(results), level=xbmc.LOGNOTICE)
-        if relatedselection != "true":
-            result = limitselection(result)
         start = 0
 
-    # sorted lists (by date and by numbers. But NOT controversial, which is a fixed list)
+    # unfiltered sorted lists (by date and by numbers. But NOT controversial, which is a fixed list)
     else:
         for entry in data:
+            if videoselection == "0":
+                if "true" in entry['official']:
+                    result.append(entry)
+            elif videoselection == "1":
+                if "false" in entry['official']:
+                    result.append(entry)
+            elif videoselection == "2":
                 result.append(entry)
     
     if sort != "" and sort == "random":
@@ -716,16 +777,9 @@ def sortTitlesBy(limit, sort, start):
             modus = channel
         else:
             modus = 'all'
-
-        if mycount != "" and mycount == "controversial":
-            # controversial is either 0 or 1 so it does not makes sense to sort upon it
-            # instead sort videos considered controversial by dislikes
-            # change controversial in videos.json to a float?
-            result = sorted(result, key = lambda i: (-1*float(i['dislikes']), i['title']))
-        else:
-            # If count would'nt be negated one had to use reverse=true
-            # but then the second sorting, by title would be reversed also
         # sort by negated count first and then by title in lexical order.
+        # If count would'nt be negated one had to use reverse=true
+        # But then the second sorting, by title would be reversed also
         result = sorted(result, key = lambda i: (-1*float(i[mycount]), i['title']))
         # itemgetter, should be faster (hm, but does'nt work: ValueError: could not convert string to float: pcount)
         # importing "operator" for implementing itemgetter 
@@ -876,26 +930,13 @@ def getFanart(channel):
         fanart = icon
     return fanart
 
-def limitselection(data):
-    xbmc.log(msg="limit by videoselection", level=xbmc.DEBUG)
-    result = []
-    for entry in data:
-        if videoselection == "0":
-            if "true" in entry['official']:
-                result.append(entry)
-        elif videoselection == "1":
-            if "false" in entry['official']:
-                result.append(entry)
-        elif videoselection == "2":
-            result.append(entry)
-    return result
 
 def addDir(name, url, mode, iconimage, total=0):
     u = sys.argv[0] + "?url=" + urllib.quote_plus(url.encode('utf-8')) + "&mode=" + str(mode)
 #    xbmc.log(msg=u, level=xbmc.LOGNOTICE)
     ok = True
 #    if (xbmcversion < 17):
-    liz = xbmcgui.ListItem(name, thumbnailImage=iconimage)
+    liz = xbmcgui.ListItem(name, iconImage=icon, thumbnailImage=iconimage)
 #    else:
 #        With offscreen=true large lists (=folder) load much faster (needs >= krypton)
 #        But at the end, the differences are minimal in VIDFLTR, so just drop it :-)
@@ -911,12 +952,7 @@ def addDir(name, url, mode, iconimage, total=0):
     if iconimage:
         liz.setProperty("fanart_image", iconimage)
     else:
-        liz.setProperty("fanart_image", fanart)
+        liz.setProperty("fanart_image", defaultFanart)
-
-    if iconimage:
-        liz.setArt({ 'fanart' : iconimage })
-    else:
-        liz.setArt({ 'fanart' : fanart })
     ok = xbmcplugin.addDirectoryItem(handle=int(sys.argv[1]), url=u, listitem=liz, totalItems=total, isFolder=True)
     return ok
 
@@ -954,7 +990,7 @@ def addVideo(entry, mycount="playcount"):
         if playLocalFile == "true":
             description = "Provider: " + entry['provider'] + "\nprovider playcount: " + entry['pcount'] + "\nlikes: " + entry['likes'] + "\ndislikes: " + entry['dislikes'] + "\ncomments: " + entry['comments']
         else:
-            description = "Provider: " + entry['provider'] + ""
+            description = ""
         if "false" in entry['official'].lower():
             description = (
                 "Unofficial Video by " + director + "\n" + description)

resources/language/resource.language.de_de/strings.po

@@ -81,8 +81,8 @@ msgid "Hits, grouped by style"
 msgstr "Hits, gruppiert nach Musikstil"
 
 msgctxt "#30022"
-msgid "Genre"
+msgid "Style (not genre)"
-msgstr "Genre"
+msgstr "Stil (nicht Genre)"
 
 msgctxt "#30025"
 msgid "Artists"

resources/language/resource.language.en_gb/strings.po

@@ -81,7 +81,7 @@ msgid "Hits, grouped by style"
 msgstr ""
 
 msgctxt "#30022"
-msgid "Genre"
+msgid "Style (not genre)"
 msgstr ""
 
 msgctxt "#30025"