[RERelease] filmstarts

Ähm... Wäre dieser Thread der richtige "Dienstweg", um Bugs, Features und sonstiges BlaBla zum @LORE's Filmstarts AddOn abzusetzen? Wenn nicht, bitte ich um einen Fingerzeig in die richtige Richtung.

Ich nutze Filmstarts aus dem Kodinerds Repo unter Kodi 17.3 mit LibreELEC auf einem Pi. Bin nicht sicher, ob meine Punkte hier überhaupt AddOn oder KODI-spezifisch sind.

• Die Listen (z.B. "demnächst im Kino") sind scheinbar nicht sortiert. Ich kann im rechten Menü zwar zwischen Auf- und Absteigend wählen, aber nicht das Feld, nach dem Sortiert werden soll - z.B. der Titel. Die Listen scheinen mir durcheinander zu sein.
• Die Listen sind zerteilt. Ich weiß nicht, wer sowas unter Kodi braucht. Ich würde es vorziehen, wenn die vollständige Liste von A-Z auf einmal angezeigt werden würde.
• Die Ansicht-Einstellungen (Poster, Shift, Wall, ...) scheinen nicht für das gesamte AddOn, sondern immer nur für einen Teil der Liste zu gelten.

Zitat von L0RE

Das Plugin bildet nur das ab was die webseite hat. Wenn du da eine Liste von A-z Findest, kann ich das gerne machen

Ah, verstehe. Ich war so naiv und dachte da gibt es ein API oder so. Du parst im Grunde die HTML-Seite?

Wäre es theoretisch möglich, dass das AddOn gleich alle verfügbaren Einzelseiten abruft, parsed und zusammenwürfelt?

Zitat von L0RE

bedeutet aber viele viele Seiten aufrufe, Würde das ganze Langsam machen.

Hab mir dein Code gerade mal angeschaut. Müsste mal aufgeräumt und kommentiert werden.
Mal im Ernst: Neben der XML-Datei sollte auch im Code infos zur Lizenz, dem Autor (incl. Kontakt) und Projektwebsite (source-repository, bug-tracker, etc) stehen.
Nebenbei fällt mir auf, dass nicht alle Infos aus der XML-Datei auch in Kodi (im default Skin) angezeigt werden. Das ist schade.

Aber ich sehe schon was du meinst. Allerdings versteh ich nicht ganz, wie du da die HTML-Seiten ziehst. Kommt die HTML-Seite mit allen eingebundenen Grafiken und Kladaradatsch dazu? Könntest du nicht nur die HTML-Seite ziehen? Geht schneller. Nächste Ausbaustufe (weniger banal): je Seite ein Thread. Was meinst du?

Zitat von L0RE

Meine Größte schwäche ist schreiben (Legasteniker), drum fällt bei mir Doku sehr spärlich aus, ich füll immer das aus was ich muss. Aber wenn du dazu lust hast, ich bin immer um Hilfe danbar.

Bitte versteh mich nicht falsch. Deine Arbeit weiß ich zu schätzen und mir ist sehr bewusst, dass das eine Freizeitgeschichte ist. Gerade bei so einem offensichtlich blöden Anbieter wie FILMSTARTS ist es um so bemerkenswerter, dass überhaupt jemand hierzu ein AddOn macht. Das hat der Anbieter gar nicht verdient!
Aber FOSS funktioniert auf die Dauer nur, wenn sich mehrere Leute daran beteiligen. Das geht nur wenn dokumentiert wird. Die Art wie du deinen Code formatierst und nicht kommentierst ist für einige Contributoren sicher ein Hinderrungsgrund, weil gerade ein Einsteiger, deinen Code gar nicht versteht. Dokumentation ist IMO ein Mindeststandard der auch in der FOSS Welt gilt. Ohne Doku fehlt die "Lust".

Zitat von L0RE

Danach hat man nen TEXT File was dem enspricht was du im Browser bei Quellcode ANzeigen hast
Danach gehts ans Zerstückeln des HTMLCodes, bis man die Infos hat die man will

Mhm... Ok, warum meinst du dann, dass es lange dauern würde, auch die anderen Teilseiten gleichzeitig zu holen? Was genau frist da die Performance?

Dokumentation bringt immer etwas, auch wenn nur einer da ist. Du dokumentierst nicht primär für andere, sondern für dich.

Ich finde dein Source-Repository nicht.

Der "Sources" Link auf diesen beiden Seiten ist tot.

https://kodi.tv/addon/plugins-video-add-ons/filmstarts
http://kodi.wiki/view/Add-on:Filmstarts

Was ist hiermit? https://github.com/AddonScriptorD…o.filmstarts_de
Sieht tot und inaktiv aus.

Das kodinerd-repo auf GitHub kenne ich natürlich
https://github.com/kodinerds/repo…ideo.filmstarts
Soweit ich verstanden habe, dient das aber nur zum Einsammeln der einzelnen AddOns und nicht zur AddOn-Entwicklung. Oder soll ich PR's gegen dieses GitHub.Repo machen?

btw: Solche Fragen würden nicht aufkommen, wenn der Code korrekt dokumentiert wäre.

Zitat von MoonKid

btw: Solche Fragen würden nicht aufkommen, wenn der Code korrekt dokumentiert wäre.

Btw: Solche Geschichten gehören in die addon.xml und nicht in den Code.

Zitat von BJ1

Btw: Solche Geschichten gehören in die addon.xml und nicht in den Code.

Kann man ja gerne paralell auch machen, obwohl Redundanzen nie gut sind. Es liese sich die addon.xml aber auch aus dem python-code generieren.
Problem taucht aber auf, wenn der code (das py-file) von seinem Paket getrennt wird. Dann hab ich keine addon.xml.
Also @L0RE, gegen welches code-repository soll ich nun PullRequest machen?

Lieber @LORE,
ist das code repository https://github.com/kodinerds/repo…ideo.filmstarts tatsächlich das offizielle, gegen das man auch PullRequest's machen kann, die du dann ggf. mergest?

Hi LORE
benutze dein super nützliches FilmstartsAddon schon seit Jahren.
Leider funktioniert seit geraumer Zeit die Trailerauswahl über "Filmstarts/Kino/Starttermin nach Wochen" nicht mehr richtig. Es wird pro Woche nur noch die letzte Filmneuerscheinung aus der Liste angezeigt.
Wäre nett wenn du das wieder reparieren könntest. Gerade die Auswahl über "Starttermin nach Wochen" war besonders praktisch.
Wünsch dir noch schöne Weihnachten.

Zitat von L0RE

@beaker Neue Version ist Online sollte jetzt gehen

Viele Grüße

Cool - das ging aber schnell. Ja es funktioniert wieder. Dankeschön.

@L0RE
ich schon wieder!
Leider funktioniert seit kurzer Zeit schon wieder die Trailerauswahl über
"Filmstarts/Kino/Starttermin nach Wochen" nicht mehr richtig. Es wird
eine leere Liste angezeigt.
Genauso bei:
"Filmstarts/Kino/NeustartsDerWocheImKino"
"Filmstarts/Kino/AktuelleKinoFilme"
"Filmstarts/Kino/DieBestbewertetenKinoFilme"
"Filmstarts/Kino/KinderFilmeImKino"
Wäre nett wenn du das nochmal reparieren könntest. Gerade die Auswahl über "Starttermin nach Wochen" war besonders praktisch.
Wünsch dir noch schöne Weihnachten.

@beaker

In Vertretung von @L0RE - Ist erledigt, habe mir erlaubt das Addon komplett umzuschreiben.
Die vorher eingebaute Abfrage nach Datum funktioniert in dieser Form leider NICHT mehr, da nicht immer (wie vorher) 7 Tage zwischen den Terminen liegen.

Ich frage jetzt direkt die Datum-Übersicht ab, aus dieser kannst Du das Datum auswählen !

Zusätzlich habe ich noch die alte Lösung im Ordner integriert (siehe Addon-settings), Diese funktioniert jedoch nur 100 prozentig, wenn das Datum auch mit dem Datum in der Liste übereinstimmt (einfach mal ausprobieren) !

Siehe dazu auch: https://www.kodinerds.net/index.php/Thre…2549#post502549

Gruß

@realvito @'LORE' - hab das Addon heute erst durch die Aktualisierung entdeckt, echt geniales Addon! Schön schnell, sehr aufgeräumt und super umgesetzt! DANKE!!!