search

Ston3s / plugin.video.xstream

Video Addon for XBMC
GNU General Public License v3.0
9 stars 8 forks source link

movies an series are now listed with imdb rating and year of manufacture #84

Closed gu3nter closed 8 years ago

gu3nter commented 8 years ago

Hi, würde gerne meine Erweiterung der Film-/Serienanzeige mit in den Code integrieren. Die Imdb Punkte und das Herstellungsjahr werden nun im Film-/Serienverzeichnis zusätzlich mit angezeigt.

ghost commented 8 years ago

Damit funktioniert der MetaHandler aber nicht mehr (da er "(7.5) FIilm 123 (2016)" nicht bei bei z.b IMDB finden wird^^). Was dann wiederrum dazuführt das keine Cover und co geladen werden. Er würde noch funktionieren wenn du bei "setTVShowTitle" nur den Filmnamen "sNameOrig" (wieso hast du den sName überhaupt umbenannt? ^^) übergeben würdest. Dann würde in der Zeile der Name stehen und der Metahandler könnte mit dem Richtigen Namen den FIlm ermitteln.

Was Zusatzinformationen angeht werden diese nur ans Objekt gebunden so das man sie z.b im Kontexmenü über INFO (oder je nach ansicht und Skin auch auf der rechten Seite) sieht. Dadurch wird die Liste nicht überladen. Der Metahandler funktioniert wie gewünscht und jeder hat was er möchte :)

Aber zusammenfassend bin ich persönlich gegen die Anzeigen Änderung. Aber mal gucken was die anderen sagen.

gu3nter commented 8 years ago

ok das seh ich ein mit dem Metahandler (Punkte und Jahr in der Namensliste). Wie könnte ich denn die imdb Punkte in der Info Anzeige einfügen? hab schon unter anderem oGuiElement.setImdbRating(sImdb) probiert, da kommt aber dann leider immer eine Fehlermeldung.

ghost commented 8 years ago

Das untersützen XBMC leider nicht. Was untersützt wird ist die IMDB anzugeben. Dadurch kann der Metahandler dann alle Informationen (wie Cover und co) aus IMDB laden. Nur leider zeigt HDFilme.TV die ID nirgends wo an :/

Ston3s commented 8 years ago

Sorry das war ein Versehen in meinem mobilen Client, wird wieder rückgängig gemacht!

Ston3s commented 8 years ago

Nochmals sorry für das durcheinander, jetzt müsste wieder alles passen. Da sowieso ein rebase vor der beta1 vollzogen werden sollte sind die vielen Commits dann auch kein Problem mehr.

Also zu der beschriebenen Funktion kann ich nur sagen das es klasse wäre wenn noch mehr Informationen bereitgestellt weren würden aber natürlich nur wen die Funktion nicht eingeschränkt wird!

Ston3s commented 8 years ago

Kommt da noch was hinterher?!