Tankerkönig Adapter Komplett umbau

[Issi]

@liv-in-sky
hast du das durchgeführt ? https://forum.iobroker.net/topic/60140/tankerkönig-adapter-komplett-umbau/106

[liv-in-sky]

@issi
erstmal danke
ok - die beschreibung funktionierte nicht ganz - habe dann rabatt eingeschaltet, hinzugefügt und dann gespeichert - dann wieder rabatt rausgenommen - jetzt ist nur noch das als fehler übrig

Spoiler

tankerkoenig.0
2023-01-02 19:42:07.438 error [ Adapter V:3.3.2 createObjects ] Error creating all states: TypeError: Cannot read properties of null (reading 'length')

[Issi]

@liv-in-sky
hast du da irgendein Feld leer, z. B. Hausnummer?

[Steff]

@Issi,

ich bin heute auf die Version 3.2.2 gegangen.
Ist echt super geworden - danke für die vielen neuen Funktionen!

Gruß Steffen

[Issi]

@steff
danke, wieso die 3.2.2? Du kannst doch schon auf die 3.3.2 gehen

[liv-in-sky]

@issi sagte in Tankerkönig Adapter Komplett umbau:

@liv-in-sky
hast du da irgendein Feld leer, z. B. Hausnummer?

tausend dank - genau das war es

[Steff]

@issi,

ups, Tippfehler ich meinte eigentlich die 3.3.2

[Issi]

@liv-in-sky
ja das Problem ist da immer die Tankstellen liefern manchmal keine Hausnummer, sondern einfach nur "" und das für zu einem Error bei der Prüfung, manche haben aber auch ein " " das ist dann okay.
Ich habe in der nächsten Version eine Prüfung eingebaut, die das abfangen soll, mal schauen, ob das funktioniert

[liv-in-sky]

@issi
nur zur info
bei mir standen in 2 stationen die hausnummern bei der strasse dabei - habe sie dort gelöscht und ins feld kopiert

[Issi]

@liv-in-sky
ja das meine ich, die wird nicht separat geliefert, sondern mit der Straße

[Issi]

@liv-in-sky
ich kann sie da nicht rausparsen da es ja zigtausend verschiedene Hausnummern gibt

[liv-in-sky]

@issi

mit regex sollten doch zahlen rausgefiltert werden können - und wenn danach noch ein buchstabe kommt sollte das auch funktionieren - rein theoretisch

[Issi]

@liv-in-sky
schwierig es gibt auch solche Adressen Gewerbegebiet Am Thie/B6 da gibt es gar keine Hausnummer
ich werde das im Adapter abfangen und eine Warnmeldung ausgeben lassen, das man die config prüfen soll

[liv-in-sky]

@issi

ja - denke du hast recht - alles zu finden ist wohl schwer - aber eigentlich ist es den aufwand nicht wert - solange der adapter erkennt, dass die hausnummer fehlt und das abfängt, sollte das doch passen

[Issi]

@Flexer
die neue Version 3.3.3 sollte bis morgen früh auf lates verfügbar sein
Bitte prüfen, ob der Adapter die Daten jetzt immer abfragt
jetzt schon installierbar über NPM und GitHub

[Segway]

@issi
Irgendwas spezielles zu beachten wenn update von 3.3.1 auf 3.3.3 ?

[Issi]

@segway
https://github.com/iobroker-community-adapters/ioBroker.tankerkoenig/blob/master/docs/guide/migration_3.3.x.md
wenn du das schon gemacht hast brauchst du nichts zu beachten

[Segway]

@issi
Mal eine Frage / Idee ?
Mit cheapest holst du ja due günstigste Tankstellen heraus. Aber was nun wenn es mehrere mit gleichen Preis gibt ? Da holt er sich anscheinend die erste angefangen von Station 0 und gibt dann die weiteren nicht mehr raus, auch wenn Sie den gleichen Preis hat.
Könnte ja auch eine sein, die näher dran ist oder auf dem Arbeitsweg liegt ?

Kriegt man da einen json raus, der quasi ALLE cheapest rausholt also mehr als eine wenn es die gibt ?

Dann kann ja jeder entscheiden was er damit macht ?

Nur mal si meine Gedanken.

[Issi]

@segway
es gibt unter tankerkoenig.0.stations.cheapest.e5.cheapest_stationseine Json mit den Namen aller Stationen, die gerade am billigsten sind
falls das nicht reicht bitte ein Issue aufmachen mit einem Beispiel wie die json aussehen sollen

[Segway]

@issi sagte in Tankerkönig Adapter Komplett umbau:

@segway
es gibt unter tankerkoenig.0.stations.cheapest.e5.cheapest_stationseine Json mit den Namen aller Stationen, die gerade am billigsten sind
falls das nicht reicht bitte ein Issue aufmachen mit einem Beispiel wie die json aussehen sollen

Okay das schaue ich mir mal an. Evtl kann ich die Tankstelle mal auswerten mit Grafana wie oft die am günstigsten ist usw - quasi ein Balkendiagramm der letzten Tage für jede Tankstelle. Mal sehen ob ich das verarbeiten kann.

Übrigens hier mal wieder das altbekannte Problem bei Update direkt von Github

changed 1 package in 7m39 packages are looking for funding run `npm fund` for details