[Announce] vdr-plugin-tvscraper 1.2

MarkusE · August 23, 2023 at 3:20 PM

Hi,

es hängt an

Code

if (seriesId > 0)

seriesId kann auch kleiner 0 sein. Sauberer wäre

Code

if (call.type == tSeries) {
...
}
else if (call.type == tMovie) {
...
}

so ist das auch in skindesigner implementiert, dashab habe ich den Fehler auch nicht bemerkt.

Könntet ihr das in SkinNopacity und skinflatplus ändern?

Notfalls auch "seriesId != 0" testen, anstelle den call.type zu prüfen.

~ Markus

MegaV0lt · August 23, 2023 at 3:26 PM

Ich baue das gerne um. Sollte ja nicht schwer das zu ersetzen. Schau morgen wenn ich zu hause bin ob ich es hin bekomme.

kamel5 · August 23, 2023 at 4:03 PM

Quote from MarkusE

Notfalls auch "seriesId != 0" testen, anstelle den call.type zu prüfen.

Ich habe bisher auf call.type und "seriesId > 0" geprüft und habe es jetzt auf "seriesId != 0" geändert und damit funktioniert es.

Heißt das jetzt aber, ich müsste nicht auf "seriesId != 0" prüfen, weil es das bei diesem call.type nicht geben kann, oder kann auch eine "0" zurückgegeben werden.

Grüße

kamel5

MarkusE · August 23, 2023 at 4:41 PM

Falls "GetEventType"

Code

call.type == tSeries

zurückgibt, dann ist es eine Serie (definitiv) und

Code

pScraper->Service("GetSeries", &series)

liefert Detailinformationen. Eine weitere Prüfung auf seriesId ist nicht notwendig und sollte eigentlich auch nicht erfolgen. Weche seriesId s verwendet werden, ist ein Implementierungsdetail von tvscraper, auf das sich Verwender nicht verlassen können.

Analog bei

Code

call.type == tMovie

kamel5 · August 23, 2023 at 4:53 PM

Danke für die Erklärung.

Dann lässt sich das ja im Skin noch vereinfachen.

Grüße

kamel5

MarkusE · August 23, 2023 at 10:09 PM

Im git ist ein Update, in dem fehlende Sprachen enthalten sind

MegaV0lt · August 24, 2023 at 12:13 PM

So. Hab auch ein wenig gebastelt und ein neues Release gemacht: https://github.com/MegaV0lt/vdr-p…eases/tag/0.6.7

Code

2023-08-24: Version 0.6.7
- [fix] Adaptation to tvscraper commit a4287558
- [update] Prevents *magick from occupying the segfaults (From SkinNopacity)

Hab ein wenig bei SkinNopacity ab geschaut

kamel5 · September 2, 2023 at 1:27 PM

MarkusE , ich habe da mal eine Frage:

Bei meiner aktuellen Arbeit am skindesigner ist mir aufgefallen, das es da auch noch in der Kategorie Schauspieler, Namen ohne Bilder gibt. Ich könnte das zwar jetzt auch wieder im skindesigner, wie bei anderen Skins auch, realisieren, andererseits wäre es nicht schöner, sowas zentral zu machen.

Also, ließe sich das mit vertretbarem Aufwand als Option in tvscraper nachrüsten, z.B. in dem in cMovie und dort wo noch Actors zurückgegeben werden, eine zusätzliche Option, z.B. "actors_all", eingeführt wird, die den alten Zustand default mäßig bereit stellt, und zusätzlich Actors nur dann, wenn sie auch ein Bild haben. Dann ließe sich das in den Skins nochmals vereinfachen.

Grüße

kamel5

MarkusE · September 2, 2023 at 7:50 PM

kamel5 , wenn ich das in bestehende Services einbaue, wäre das eine incompatible Änderung.

Also 2 neue Services. Die dann in scraper2vdr fehlen. Ich vermute mal, du möchtest auch noch scraper2vdr unterstützen (?).

kamel5 · September 2, 2023 at 8:25 PM

Quote from MarkusE

Ich vermute mal, du möchtest auch noch scraper2vdr unterstützen (?).

Ja, das soll schon noch funktionieren.

Manchmal hilft eine 2. Sicht um die Konsequenzen zu verstehen.

Ich habe erst einmal skindesigner angepasst, ich muss da aber leider im Moment 2 mal die ganzen Bilder abfragen,

deshalb die Idee. Vieleicht sollte ich da später noch mal eine Überarbeitung machen.

Grüße

kamel5

MarkusE · September 7, 2023 at 5:44 PM

v1.2.3 ist im git verfügbar.

Damit wird ein Index in der Datenbank korrigiert -> Performanceverbesserung bei der Anzeige von Detaildaten von Spielfilmen.

Bei Client - Server Systemen:

1.) Server updaten und restarten. Danach auf Update von /var/cache/vdr/plugins/tvscraper/tvscraper2.db warten oder erzwingen (durch Start einer Aufnahme). Der fertige Update wird im Syslog mit "tvscraper: access to /var/cache/vdr/plugins/tvscraper/tvscraper2.db finished" protokolliert.

2.) Client updaten und restarten.

~ Markus

MarkusE · November 11, 2023 at 3:07 PM

v1.2.4 ist im git verfügbar.

Neues plugin Service Interface, das andere Plugins zum Anzeigen des Grundes, aus dem Autotimer erstellt wurden, verwenden können.
Ändern des Verhaltens von "GetPosterBannerV2": Insbesondere wird jetzt der korrekte type zurückgegeben. Anmerkung: Das kann dazu führen, dass in manchen Skins weniger Bilder dargestellt werden. Bitte verwendet gegebenenfalls skindesigner 1.2.22 (oder neuer) bzw. SkinNopacity 1.1.16 (oder neuer).

~ Markus

rüsseltier · November 11, 2023 at 4:26 PM

Habe von der 1.2.3 folgendes im syslog gefunden:

Code

Nov 11 16:17:16 vdr vdr[2511]: [2511] tvscraper: cTVScraperConfig::loadPlugins cannot open plugin dir /usr/lib/tvscraper/plugins

Bin ich alleine damit?

Muss dazu sagen, dass ich live schon mal rausgepurged hatte und jetzt wieder installiert habe. Paket ist aus dem yaVDR PPA.

Es läuft auch, wollte es nur mal melden: es scheint ja mutmaßlich nur vdr in der Pfadangabe zu fehlen.

MarkusE · November 12, 2023 at 8:40 AM

Also, mit live hat das nichts zu tun.

Wenn Du kein tvscraper Plugin nutzt, kannst Du die Meldung einfach ignorieren. Infos zu tvscraper Plugins findest Du in der README.md.

Definiert wird das Verzeichnis in Make.config:

Code

PREFIX = /usr
PLGDEST = $(PREFIX)/lib/tvscraper/plugins

Der Wert ist also korrekt. Ich ergänze jetzt mal im Makefile unter "install-plugins: plugins" ein 'mkdir -p "$(PLGDEST)"' . Damit sollte die Meldung weg sein.

~ Markus

rüsseltier · November 12, 2023 at 9:53 AM

Quote from MarkusE

Also, mit live hat das nichts zu tun.
Wenn Du kein tvscraper Plugin nutzt, kannst Du die Meldung einfach ignorieren. Infos zu tvscraper Plugins findest Du in der README.md.

Die Meldung kommt mit installiertem tvscraper- und live-Plugin.

MarkusE · November 12, 2023 at 10:09 AM

Also, die kurze und einfache Antwort: Ignoriere die Meldung. Wenn ich mal wieder ein neues Update herausgebe, und dieses Update in yaVDR einzieht, wird sie auch nicht mehr da sein.

Die lange Antwort:

live ist ein VDR Plugin, so wie tvscraper ein VDR Plugin ist.
Die Meldung bezieht sich auf tvscraper Plugins (also nicht auf live, da live ein VDR Plugin ist, und nicht ein tvscraper Plugin).

Die Meldung kommt auch, wenn du live nicht verwendest. Und ja, sie kommt auch wenn Du live, softhddevice, und noch 5 andere VDR Plugins verwendest. Wie gesagt, die Meldung hat nichts mit VDR Plugins zu tun.

~ Markus

rüsseltier · November 12, 2023 at 10:23 AM

OK, alles klar.

wmautner · November 12, 2023 at 10:25 AM

Testhalber das Verzeichnis einfach mal anlegen? Und dann nachsehen, ob / was da drin landet?

rüsseltier · November 12, 2023 at 10:35 AM

Quote from wmautner

Testhalber das Verzeichnis einfach mal anlegen? Und dann nachsehen, ob / was da drin landet?

Es geht da um optionale Plugins für das tvscraper-Plugin. War mir bei meinen vorangegangenen Postings auch nicht so bewusst.

Dachte tvscraper kann irgendwas nicht finden, was es zum reibungslosen Funktionieren benötigt.

MarkusE · November 12, 2023 at 10:56 AM

Ihr könnt das Verzeichnis anlegen. Dann verschwindet die Meldung. Sonst ändert sich nichts.

Participate now!