search

Ston3s / plugin.video.xstream

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

[HDfilme] Wrong metadata for series #62

Open Ston3s opened 8 years ago

Ston3s commented 8 years ago
  1. Mir ist eben aufgefallen, dass für ganz viele Serien keine Episoden-Thumbnails geladen werden können. Liegt das eventuell am zusätzlichen Staffelangaben im Namen.
  2. Außerdem werden bei HDfilme keine Serien sondern einzelne Staffeln angezeigt. Würde es an der Stelle nicht auch mehr Sinn machen das Cover von den einzelnen Staffeln zu laden?
ghost commented 8 years ago

Wie meinen?

Beim erstellen der Serien Liste wird dem Eintrag alles mitgegeben was geht:

und der MediaType wird auch auf Episode gesetzt. HAst du evtl. ein Beispiel?

Ston3s commented 8 years ago
  1. Zum ersten Problem ist mir z.B. Navy CIS L.A. aufgefallen. Da werden für die Folgen nur das Seriencover angezeigt.
  2. Meine damit, dass die Serien bei HDfilme ja eigentlich als Staffeln zur verfügung stehen. Wenn ich jetzt z.B. nach Game of Thrones, House of Cards oder andere Serien mit mehreren verfügbaren Staffeln wird nicht das jeweilige Cover der Staffel angezeigt sondern das Bild zur gesamten Serie. Solange HDfilme es als Season bereitstellt wäre es auch schön wenn es als Season dargestellt würde...
ghost commented 8 years ago
  1. wird an der Schreibweise liegen vermute ich mal
  2. ich weiß das liegt dadran das die Einträge als "Serie" und nicht als "Staffeln" behandelt werden. Sieht dazu:

https://github.com/StoneOffStones/plugin.video.xstream/issues/46 Dort hatte ich Lynx187 gefragt ob man sie nicht eher als "Staffeln" listen sollte. Und sein Aussage dazu war:

"Das wäre richtiger aber beim metahandler ist zum laden der Seasoninfo die imdbID Pflicht und der Seaonview zeigt weniger Infos in der Regel."

Ston3s commented 8 years ago

Ok, danke für die detaillierten Infos. Hatte ich damals nicht in Zusammenhang gebracht.

Gibt es eventuell eine Möglichkeit/Workaround wie die Serien an den Metahandler übergeben werden jedoch die Staffel-Cover angezeigt werden

ghost commented 8 years ago

Nicht wirklich. Aktuell übergebe wir alle Infos die die Seite hergibt. Das einzige was man wie gesagt anders machen könnte hatte ich oben ja mal beschreiben. Wenn das für dich soweit ok ist kannst das eigentlich zu machen.

Ston3s commented 8 years ago

Können wir das mit der SeasonInfo in der Nightly ausprobieren. Könnten ja direkt sehen ob es Auswirkungen bzw. Probleme provoziert.

Commit kann in diesem Fall immer noch zurückgezogen werden ;)

Lynx187 commented 8 years ago

zu 1. in den xStreamSettings aktiviert, dass der metahandler Infos von der Seite überschreiben darf?

Für die Seasoninfo (die abgesehen von dem Bild so gut wie nichts beinhaltet) braucht man wie gesagt erst mal die (imdb)ID der Serie. Wenn man die nicht schon (rechtzeitig) von der Seite bekommt muss man 2 mal die Metadaten abfragen (Serie + Seasons). Und wenn man jetzt das ganze als Serie angezeigt bekommen will aber mit dem Staffelcover muss man in den Metdaten der Serie nur das Cover durch das von der Staffel ersetzen und das Ganze so ans GuiElement übergeben.

Ston3s commented 8 years ago
  1. ja
  2. Danke für die Info. @viper2k4: ist das von deiner Seite aus machbar?
ghost commented 8 years ago

Naja die IMDB-ID habe ich auf die schnelle nicht gefunden. Jedoch weiß ich nicht ob sich der Aufwand lohnt und die zusätzlichen Abfragen (würde die Abfragen pro Liste quasi verdoppeln) nur um das Bild zu ändern.

Die Frage ist eher wo das hingehört. Die Aufgabe ansich müsste aber der MetaHandler übernehmen oder?

Lynx187 commented 8 years ago

Der Metahandler kann die Serieninfo und Staffelinfo holen mehr nicht, dass eine Seite mal mehr mal weniger passend strukturiert ist ist ja nicht seine Schuld. Du kannst die nötige methode dafür aber direkt über das GuiElement nutzen (sieht man ja). xStream sollte dann für dieses GuiElement auch nicht nochmal automatisch nach den Infos suchen. Ich halte es aber auch mehr für nett aber nicht wichtig.

ghost commented 8 years ago

Ich hatte die Aufgabe nur Strukturell eher ausserhalb des SitePlugins gesehen ob jetzt der MetaHandler oder ne zwischen Instance (die wir hier ja nicht haben), ist dabei ja egal.

Ich habe das eher so geshene. Das SitePlugin liefert die Informationen (Es ist eine Staffel von Serie x mit Staffel Nr. Y) und nachgelagerte Instanzen kümmern sich dann um die Verarbeitung.

Ich persönlich finde es aber auch eher nett als nötig. Wir können das Issu gern offen lassen falls sich da jemand ransetzen möchte. Aber ich werde mir was HDFilme.TV angeht eher gedanken um das MultiHoster-Problem machen.

Lynx187 commented 8 years ago

Ok das wäre sinnvoll wenn man wirklich nur die Staffelinfos möchste ohne vorher die id zu haben. Aber das kommt ja selten vor deswegen wäre die einfache Variante das erst mal auf Plugineben zu machen.

Ston3s commented 8 years ago

Vielleicht ist es aus Entwickler-Sicht "nur" eher nett als sinnvoll aber ich glaube für den normalen User sind solche Sachen doch durchaus interessant.

User-Experience und selbsterklärendes Auftreten spielt eine nicht zu unterschätzende Rolle. Würde mich freuen wenn man es zumindest versuchen könnte...

Ston3s commented 8 years ago

Wurde das Problem weiter verfolgt?

ghost commented 8 years ago

nope

Ston3s commented 8 years ago

Lynx187 hatte ja weiter oben vorgeschlagen wie man hier vorgehen könnte. Wäre schön wenn das noch hinzukommen könnte... :)

Kukolorus commented 8 years ago

Wie wäre es, wenn man zum jetzigen Problem von Cloudflare hier weiter schreiben würde bzw. hier einen Thread aufmachen würde wo der Stand der Dinge besprochen wird?

Fernab vom Trubel momentan im Forum.

Wie sieht es denn aus? Wird man was machen können oder sieht es eher schlecht aus?

ghost commented 8 years ago

https://github.com/Lynx187/plugin.video.xstream/issues/142