search

remuslazar / EVPlugFinderPublic

Public Repository to hold the Issue Tracker for the private EVPlugFinder repo
13 stars 0 forks source link

Fehler: Filter "Jetzt geöffnet" #68

Closed Graefer closed 6 years ago

Graefer commented 7 years ago

Hi, offensichtlich hat der Filter "Jetzt geöffnet" ein Problem mit der Art und Weise, wie ein ganztags geschlossener Tag auf GE eingetragen wird (0.00 bis 0.00), denn das wird von der App als ganztags geöffnet interpretiert.

remuslazar commented 7 years ago

ich würde sagen, der Eintrag ist einfach buggy bei GE und sollte dort korrigiert werden. Man kann doch auch Tage „komplett geschlossen“ eintragen, dann braucht man auch kein Time Range. Wo tritt das Problem auf?

Graefer commented 7 years ago

Der Eintrag ist missverständlich - aber im Sinne der GE-Logik korrekt und auch nicht anders eintragbar. Das Problem betrifft alle Aldi- und Lidl-Ladesäulen.

remuslazar commented 7 years ago

@Graefer aber die Filterung macht nicht die App selbst sonder die GE API. Ich glaube, da ist noch ein Bug-Report offen im GE Forum, weil das nicht sauber funktioniert. Die App nutzt die Info lediglich um die Öffnungszeiten anzuzeigen.

Graefer commented 7 years ago

Tja, das mag so sein. Mir fällt aber auf, dass in der Ladesäulenbeschreibung der App die Ladesäule korrekt als geschlossen angegeben wird. Das wird dann offensichtlich nicht über das (fehlerhafte) open_now flag der API, sondern über eigenes parsen der eingetragenen Zeiten errechnet? Wenn Du Dir die Arbeit also schon gemacht hast, könnte man ja auch diesen Wert für den Filter nehmen?!

remuslazar commented 7 years ago

@Graefer genau, das habe ich mir auch schon überlegt, man könnte einfach ohne Filter fetchen und dann hinterher filtern. Das wäre pragmatisch, um das Problem zu lösen. Ich persönlich bin aber kein Freund davon, in diesem Fall, weil das ja nur ein „Hack“ wäre.

Es ist natürlich nicht ideal, dass offene GE API Bug-Reports so lange „liegen“ bleiben aber genau da sollte man ansetzen. Willst du dich mal in die Richtung engagieren?

Graefer commented 6 years ago

Ok, Fehler der GE-API scheint gelöst zu sein.