Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ugyldige tegn i 'veglenkesekvens': mangler eksplisitt formatering av double #112

Closed
monsendag opened this issue May 5, 2022 · 2 comments · Fixed by #113
Closed

Ugyldige tegn i 'veglenkesekvens': mangler eksplisitt formatering av double #112

monsendag opened this issue May 5, 2022 · 2 comments · Fixed by #113

Comments

@monsendag
Copy link
Contributor

monsendag commented May 5, 2022

Hei, vi får denne når vi henter enkelte attributter på vegobjekter. Bruker siste versjon (1.18.2) av Java-klienten.

Ugyldige tegn i 'veglenkesekvens' field: '3.17E-6@2070659'. Gyldig eksempel: 0.6-0.9@42364

Dette er sannsynligvis grunnet feil formatering når man bruker Double#toString(), f.eks ved konkatenering av double med string. Specen for double viser at formatet ved ren toString() avhenger av verdien, og ved veldig lave eller høye verdier risikerer man å få vitenskapelig notasjon, som vi ser i feilmeldinga. (3.17E-6).

Dette løses ved å bruke String.format() og spesifisere formatet. Jeg har gjort det alle steder jeg kunne finne formatering av posisjon i vår kodebase, men ser at det ikke er gjort alle steder i klientbiblioteket. F.eks RefLinkExtendAttribute#getValueAsString.

Har dere tid/mulighet til å lage en fiks, eventuelt tar dere inn en PR hvis jeg lager en?

@hansog
Copy link
Contributor

hansog commented May 30, 2022

Hvis du kan så gjerne lag en PR så tar jeg inn endringen :-)

monsendag added a commit to monsendag/nvdb-api-client that referenced this issue May 30, 2022
@monsendag
Copy link
Contributor Author

@hansog da er den klar :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants