-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[HDfilme] Wrong metadata for series #62
Comments
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? |
|
#46 "Das wäre richtiger aber beim metahandler ist zum laden der Seasoninfo die imdbID Pflicht und der Seaonview zeigt weniger Infos in der Regel." |
Ok, danke für die detaillierten Infos. Gibt es eventuell eine Möglichkeit/Workaround wie die Serien an den Metahandler übergeben werden jedoch die Staffel-Cover angezeigt werden |
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. |
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 ;) |
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. |
|
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? |
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. |
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. |
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. |
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. |
Wurde das Problem weiter verfolgt? |
nope |
Lynx187 hatte ja weiter oben vorgeschlagen wie man hier vorgehen könnte. |
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? |
Würde es an der Stelle nicht auch mehr Sinn machen das Cover von den einzelnen Staffeln zu laden?
The text was updated successfully, but these errors were encountered: