Test Adapter OpenLigaDB

[Saschag]

@myssv Dank Dir!

[myssv]

So, nachdem endlich wieder Daten fließen, ist mir ein weiterer Punkt aufgefallen:

In der 2. Bundesliga werden die Torschützen nicht angezeigt

In der 1. Liga werden sie angezeigt.

[OliverIO]

@myssv
ich habe das geprüft. leider sind die openligadb daten unvollständig erfasst.
die tore an sich gibt es zwar, aber ohne torschütze und ohne minutenangaben.
daher auch die (null), daher macht die anzeige der tore und minuten in der bl2 aktuell keinen sinn.

hier die originaldaten von openligadb direkt
https://www.openligadb.de/Ergebnisse/4363/26

das null werde ich in der anzeige korrigieren

[OliverIO]

Neue Version 1.2.0 (latest Repository)

• Anpassen der Anzeige der Tore, falls bei openligadb die Daten nicht komplett erfasst wurden (ohne Spielername, ohne Torminute)
• Die widgets wurden verbessert, falls in manchen Fällen der Status nicht korrekt abgerufen werden konnte.

Über eine Rückmeldung von Tests würde ich mich freuen, damit ich diese Version auch in das stable Repo übernehmen lassen kann.

[myssv]

Die neue Version wird mir noch nicht als Update angeboten.

Kann ich das irgendwie manuell anstoßen?

[myssv]

Keine "null" mehr zu sehen

[OliverIO]

@myssv
ja so ist es dann korrekt (von meiner Seite zumindest).
Allerdings macht das so nicht viel Sinn.

Gern kannst du die Daten bei openligadb verbessern
Dann zeigt es die auch an.
Der aktuelle Erfasser dort macht das wohl nicht vollständig.

[OliverIO]

Hinweis für alle, die nur aus dem Verwahrort stable aktualisieren.
Da irgenwie in letzter Zeit die Aktualisierung des stable Repositorys nicht funktioniert,
werde ich nur noch latest aktualisieren.
https://github.com/ioBroker/ioBroker.repositories/pull/738

oweitman created this issue in ioBroker/ioBroker.repositories

closed update openligadb to v1.1.0 #738

[arteck]

@OliverIO wie kommst du auf sowas ?? ausserdem ganz sachte mit latest... man kann es nur ganz oder garnicht umstellen..und latest ist nicht immer freigegeben bzw nur zum Testen

[apollon77]

@OliverIO das kann nicht die Lösung sein. Einfach im PR mal nett nachfragen hätte hier geholfen.

Auch wenn ich gerade nicht so schnell dazu komme neue Adapter zu reviewen schaue ich das stable updates schnell durchgehen. In dem Fall hatte ich wohl nen Knick in der Pupille und dachte Adapter wäre neu. Sorry.

Stable nicht mehr updaten und User ins latest repo zu treiben kann hier bitte nicht die Lösung sein!

[OliverIO]

@apollon77
Der Hinweis:
https://github.com/oweitman/ioBroker.openligadb/issues/3#issuecomment-638009916

Darüber hinaus habe ich weitere PR gesehen, die ähnlich alt waren und ebenfalls keine Fehler angezeigt haben

oder man sollte definieren, über welche Schiene die relevante Kommunikation (github,forum,?) laufen sollte.

Nicht-Kommunikation ist für mich ein Motivationskiller. Aber evtl. bin ich hier ein Einzelfall

GermanBluefox created this issue in oweitman/ioBroker.openligadb

closed Update stable version in repo from 1.0.0 to 1.2.0 #3

[apollon77]

@OliverIO Eine ANtwort auf den Post sieht glaube ich keiner (also ich nicht) da das ein Bot postet! AM besten dann nochmal im "repository" PR "pingen"
Ich habe versucht die Adapter Requests alle korrekt zu flaggen.

Neue Adapter im Latest bekommen einen Code-Review und der fehlt leider aus zeitgründen bei einigen noch ... Nur "Stable" Updates brauchen das nicht - das hab ich beid einem siehe oben falsch gesehen.

[Stephan Schleich]

@OliverIO Die Torschützenliste wird bei mit in der Version 1.2.0 nicht geupdated.

[OliverIO]

@Stephan-Schleich
Mach einmal ein Beispiel bitte. Dann kann ich mit den Original Daten von openlogadb vergleichen.
Gerade bei den Torschützen wird nicht immer sauber erfasst. Ich kann nur Daten ausgeben die bei openLigadb auch eingegeben Wurden

[Stephan Schleich]

@OliverIO

[OliverIO]

@Stephan-Schleich
Es ist so wie ich vermutet habe. Die Daten passen, allerdings werden die Daten nicht sauber bei openligadb erfasst.
Beispiel Lewandowski kommt mit 3 Namenskombinationen vor, Summe ist 32

• Lewandowski, Robert 16
• Lewandowski 8
• Lewandowski, Robert 8

Das widget versucht im Hintergrund bereits einige Namensnormalisierungen durchzuführen (Details stecken hier in diesem thread irgendwo weiter oben), wenn aber bei openligadb die Tore 3 unterschiedlichen Einträgen zugeordnet wird, kann das widget leider nicht zaubern.

[Stephan Schleich]

@OliverIO alles klar, danke für die Info

[Vandura]

@myssv Hallo, ich habe als Neuling mal eine Frage, Wieso sehe ich auf meinem Monitor im VIS alle Daten, aber auf meinem Tablet oder Handy ist die Vis Leer....nur die Navigationsbar ist zu sehen..????

[OliverIO]

@Vandura
Das liegt leider an der vis App.
Diese lädt eine ganze Reihen von Adapter vor. Viele Adapter ua auch openliga sind nicht dabei.
Daher geht das hier nur über die view in view Technik.
Dh du erstellst eine 2. view mit einem iframe. In diesen trägst du den link zum ersten view ein.
Dann geht es.

[Vandura]

@OliverIO Upps.....kannst Du mir das eventuell etwas näher erklären, dass Übersteigt meine frischen Kenntnisse wohl noch etwas....