dma61 / VBPUOdsk

3 stars 0 forks source link

Maximum in 'financialTransaction'? #54

Closed AnkeAPG closed 5 months ago

AnkeAPG commented 6 months ago

Festina Finance heeft een verschil geconstateerd tussen de handleiding en de voorbeeld-jsons (april-versie) m.b.t. 'financialTransaction'. In de jsons staat een maximum van 99 genoemd, in de handleiding is er geen maximum. Kan dit maximum verwijderd worden uit de jsons?

Voorbeeld: https://github.com/dma61/VBPUOdsk/blob/80f95e5ed2a60c3d4e0830d03905a1f782135502/VBPUO-Bericht_3.Pensioenprojectie(0001c)/VBPUO-001.00-Bericht_3.Pensioenprojectie(0001c).json#L446 en https://github.com/dma61/VBPUOdsk/blob/80f95e5ed2a60c3d4e0830d03905a1f782135502/VBPUO-Bericht_3.Pensioenprojectie(0001c)/VBPUO-001.00-Bericht_3.Pensioenprojectie(0001c).json#L602.

dma61 commented 6 months ago

M.i. het zelfde probleem als https://github.com/dma61/VBPUOdsk/issues/50. Zo ja, dit issue svp sluiten.

AnkeAPG commented 6 months ago

Dit is een soortgelijk issue als #50, alleen lopen json en handleiding hier uiteen. Handleiding heeft geen maximum. Er zou in mijn ogen voor dit item ook geen maximum moeten zijn inderdaad.

dma61 commented 5 months ago

Maximumaantal gaat net als bij cohort (zie issue #50) naar 99999.

JulenMN commented 4 months ago

@dma61 er zijn twee "financialTransaction" entiteiten in Berichtenstructuur 2 (1c), eentje op schema niveau en eentje op cohort niveau. Echter staat er in het schema bij de eerste een maxContains van 99999 en bij de tweede van 99. Maar als die eerste op 99999 staat kan ik me niet voorstellen dat er een reden is om die tweede daar niet ook op te zetten. Is deze misschien over het hoofd gezien?

Zie onderstaande code:

    "financialTransaction": {
      "type": "array",
      "uniqueItems": true,
      "allOf": [
        {
          "minContains": 1,
          "maxContains": 99999,
          "contains": {
            "$ref": "#/definitions/financialTransactionPayment"

    "financialTransaction": {
      "type": "array",
      "uniqueItems": true,
      "allOf": [
        {
          "minContains": 1,
          "maxContains": 99,
          "contains": {
            "$ref": "#/definitions/financialTransactionPayment_1"