Datepicker bugs in validation

[nickcuypers]

Niet-bestaande datums
Op het scherm waar ik nieuwe lidmaatschappen kan toevoegen, vul ik het volgende in:

Bij het invullen of opslaan krijg ik geen foutmelding.
Na het opslaan zie ik:

Ik had hier verwacht dat ik al rode tekst of zo zou krijgen bij het ingeven.
Als ik 32/02/2020 ingeef als geboortedatum (of een andere niet bestaande datum), dan krijg ik van de datepicker component achterliggend deze datum ook terug, maar visueel toont hij een andere datum dus visueel is er wel een soort validatie en correctie maar achterliggend niet. Correctie is dan weer niet wenselijkn eerder een validatie error of indicatie.

Onvolledige datums
Als je de datum manueel invult en niet via de datepicker, en dan bijv. enkel een dag invult, dan kan men opslaan, maar de datum is dan wel verkeerd.
Bijv. ik vul enkel als dag 12 in. Bij het opslaan maakt het systeem er 01/12/2001 van.

Na opslaan staat er:

Als ik 12/03 invul, maar geen jaartal, dan wordt er 12/03/2001 van gemaakt. Lijkt iets te hebben met jaartal 2001 😊.

Dus achterliggend maakt hij van gedeeltelijke datums zelf maar iets terwijl ik hier een validatie error zou verwachten.

[TriangleJuice]

Hey @nickcuypers, kan je eens checken of jullie wel de laatste versie van ACPaaS UI gebruiken, want als ik beide bugs probeer te reproduceren op de datepicker documentatiepagina dan krijg ik telkens wél een foutboodschap.

[nickcuypers]

@TriangleJuice oei hier heb ik geen melding van gekregen vandaar mijn late antwoord.
heb je een versienummer zodat we zeker zijn dat we over dezelfde versie spreken indien verder onderzoek vereist is?

[TriangleJuice]

Zeker, @nickcuypers! De huidige versie is v4.5.2 waar het probleem zich volgens mij niet voordoet.

[nickcuypers]

@TriangleJuice die versie gaat over de calendar zeker?
https://www.npmjs.com/package/@acpaas-ui/ngx-calendar
maar wij maken gebruik van de ngx-components package
https://www.npmjs.com/package/@acpaas-ui/ngx-components
waarbij we de genoemde issues hebben bij de datepicker

[TriangleJuice]

Hey @nickcuypers het klopt dat de werking van die versie met momenten niet helemaal juist was. Is het voor jullie veel moeite om te upgraden naar de nieuwe versie van ACPaaS UI? Welke versie van Angular gebruiken jullie?

[nickcuypers]

@TriangleJuice hangt ervan af wat de impact is, we zitten nl in de laatste fase van het onderweg project, momenteel gebruiken angular 8.2.14 met volgende packages van acpaas ui

[nickcuypers]

@TriangleJuice kan je hiervoor de impact nog aangeven?

[TriangleJuice]

Hey @nickcuypers normaal gezien is de enige impact dat jullie de ACPaaS UI import paths moeten veranderen. Maar veel hangt ook af van de versie van Angular die jullie gebruiken. Als dat v8 of hoger is, dan zou je nagenoeg geen problemen mogen hebben. Als je niet zeker bent, stuur me dan even de repo door, dan kan ik eventueel zelf mee kijken of het een grote impact gaat hebben of niet.

[nickcuypers]

@TriangleJuice als dat de enige impact is, kunnen we dit wel doen lijkt me, voor de zekerheid kan je het checken in volgende repo:
https://bitbucket.antwerpen.be/projects/OCMW/repos/onderweg_app_aspnetcore/browse

kan je zoiezo even verduidelijken welke package we dan moeten gebruiken, m.a.w. wat bedoel je met "upgraden naar de nieuwe versie van ACPaaS UI"?

[TriangleJuice]

Hey @nickcuypers Ik heb het vlug even bekeken en heb volgende zaken vastgesteld:

• Alle import { ... } from '@acpaas-ui/ngx-components/forms'; dienen jullie te vervangen door import { ... } from '@acpaas-ui/ngx-forms'; (is ook de enige die jullie moeten vervangen/importeren in package.json)
• Het enige mogelijke probleem dat ik zag is dat jullie meerdere (oude) versies van de embeddable widgets door elkaar gebruiken. Geen idee waarom, maar als jullie volledig kunnen upgraden naar @acpaas-ui/ngx-embeddable-widgets dan zou er geen probleem mogen zijn.

Sidenote: er zijn volgens mij enkele componenten in de ACPaaS UI bibliotheek die jullie nog niet gebruiken. Ik geef het maar mee om in de toekomst gemakkelijker te kunnen upgraden. De header is daar een voorbeeld van.

[nickcuypers]

hey @TriangleJuice , ik heb die aanpassing gedaan en alles werkt idd nog maar ik zie ook geen verschil met de problemen die ik aangaf in dit bugticket. ik krijg idd in sommige gevallen de error 'ongeldige datum' maar lang niet op alle momenten die in dit ticket zijn aangegeven.
Als ik bv maar een gedeeltelijke datum ingeef geeft hij enkel een error als de dag of maand of het jaar maar gedeeltelijk zijn ingegeven, maar als ik bv 01// ingeef krijg ik geen error en na opslaan maakt hij er zelf nog steeds een andere datum van.
ook gaat hij pas vanaf 13// een error geven wss omdat hij dat als maand rekent terwijl dat de plek is om een dag in te vullen dus structureel lijken er mij nog wat zaen mis te lopen en dit zijn maar enkele voorbeelden. Kan dit worden nagekeken?

[TriangleJuice]

Hey @nickcuypers Kan het zijn dat de datepickers waarvan je spreekt, deel uitmaken van de form renderer en dus niet 'rechtstreeks' uit ACPaaS UI komen? Kan je bevestigen dat deze foutieve data ook geen error geven in de datepicker onderaan deze pagina?

[nickcuypers]

@TriangleJuice die pagina lijkt nog een nieuwere versie te gebruiken want daar komen die errors idd niet voor.
maar het gaat wel degelijk om de aui-datepicker en dus niet een datepicker uit een form.
op dit moment hebben we volgende versie staan:

met volgende imports:

en volgende config:

[TriangleJuice]

@nickcuypers Ik denk dat je toch iets over het hoofd ziet, want deze versie is maar sinds enkele minuten verhoogd en heeft exact dezelfde configuratie als die van jou.