lux-lookup-combobox interpretiert noch gültige Einträge als ungültig

IHK-GfI / lux-components

Other

16 stars 5 forks source link

lux-lookup-combobox interpretiert noch gültige Einträge als ungültig #431

Closed taubec closed 7 months ago

taubec commented 7 months ago

Ist bei einem Schlüsseltabelleneintrag ein GültigBis eingetragen, welches noch nicht erreicht ist, interpretiert die Komponente lux-lookup-combobox-ac diesen trotzdem als ungültig.

Umgebung: ENTW Schlüsseltabelle: 1096 (Verwendungszweck Telekom) Schlüssel: 001077 - gültig bis zum 5.3.2024, als ungültig interpretiert am 28.2.2024.

Folgender Code in der Datei lux-lookup-component.ts dürfte so auch nicht funktionieren: if (entry.gueltigkeitBis) { isUngueltig = Date.now() > +entry.gueltigkeitBis;

Wobei das Datum 99.99.9999 anscheinend vorher schon als null oder undefined zurück geliefert wird. falscheUngueltigInterpretation

taubec commented 7 months ago

Die Markierung in der Combobox folgte anhand dieser Konfiguration: behandlungUngueltige: LuxBehandlungsOptionenUngueltige = LuxBehandlungsOptionenUngueltige.ausgrauen;

Noch mal eine kurze Erklärung zum erwähnten Code-Schnippsel: Date.now() liefert Millisekunden, entry.gueltigkeitBis dürfte vom Format aber yyyymmtt sein. Als Zahl interpretiert auf jeden Fall immer kleiner.

thodic77 commented 7 months ago

Der Fehler wurde behoben.

Test

In der Datei "src\app\demo\components-overview\lookup-examples\mock-result.ts" verschiedene Daten (Format yyyymmdd) für gueltigkeitVon und gueltigkeitBis eintragen.
Die Demo bauen und starten.
Die Seite "Lookup-Combobox-Authentic" öffnen.
Prüfen, ob die ungültigen Einträge korrekt in rot dargestellt werden.

In der aktuellen Demo sind z.B. Bellux (wegen Gültigkeit in der Vergangenheit) und Niederlande (wegen Gültigkeit in der Zukunft) rot dargestellt.

thodic77 commented 7 months ago

@DSeifer: Bitte testen.

DSeifer commented 7 months ago

Unter Windows 10 Enterprise 22H2 (19045.4046) und MacOS 14.3.1 mit den jeweils aktuellen Browserversionen getestet:

MacOS: Safari -> OK Windows: Edge -> OK