VDVde / VDV463

Development related files of VDV463.
https://knowhow.vdv.de/documents/463/
4 stars 4 forks source link

Add Validations to JSON Schema Properties #36

Open Basler182 opened 3 weeks ago

Basler182 commented 3 weeks ago

The current JSON schemas lack validations for numerical properties. These validations ensure that the data conforms to expected value ranges. This aims to enhance the robustness and accuracy of the schema definitions.

maikwoehl-psi commented 2 weeks ago

Hallo @DohmenClaus,

wie lautet hierzu die Abstimmung?

Angesichts der letzten Diskussionen bin ich mir nicht sicher, ob die Einführung einer Restriktion einen "Breaking Change" darstellt. Ich bin gewillt zu sagen, dass dadurch zwar die Implementierung ggbf. angepasst werden muss, diese dafür dann aber stabiler ist. Für den Fall, dass irgendjemand Gleitkommazahlen statt Ganzzahlen von 0 bis 100 verwendet hat, ist das bestimmt eine sinnvolle Verbesserung.

Viele Grüße

Maik Wöhl

Basler182 commented 2 weeks ago

Vielen Dank für deinen Kommentar und die Anmerkungen, @maikwoehl-psi. Die Festlegung eines Wertebereichs durch Minimum und Maximum dient lediglich zur Begrenzung der Werte, ändert jedoch nicht den zugrunde liegenden Datentyp. Wenn ich nachfolgendes aber richtig verstehe:

Für den Fall, dass irgendjemand Gleitkommazahlen statt Ganzzahlen verwendet hat, ist das bestimmt eine sinnvolle Verbesserung.

Impliziert deine Nachricht, dass der zugrunde liegende Datentyp selbst von number auf integer geändert werden sollte. Wenn das korrekt ist und der aktuelle Datentyp in der Definition tatsächlich falsch hinterlegt ist, dann kann ich dies im PR zusätzlich korrigieren. Es wäre hilfreich, dies nochmals zu bestätigen, um die Implementierung entsprechend anzupassen und zu optimieren.