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.

Sign up for GitHub

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

Jump to bottom

(vrz-001c) Verzoek resource mist validatie op uniciteit van bronorganisatie en identificatie #1696

Closed
stevenbal opened this issue Oct 9, 2020 · 1 comment · May be fixed by VNG-Realisatie/verzoeken-api#14
Closed

(vrz-001c) Verzoek resource mist validatie op uniciteit van bronorganisatie en identificatie #1696

stevenbal opened this issue Oct 9, 2020 · 1 comment · May be fixed by VNG-Realisatie/verzoeken-api#14
Assignees
@stevenbal
Labels
bug Iets werkt niet zoals bedoeld

Comments

@stevenbal
Copy link
Collaborator

stevenbal commented Oct 9, 2020 •
edited
Loading

Bug

Bij het deels bewerken van een Verzoek, wordt er niet goed gevalideerd of de combinatie bronorganisatie en identificatie al bestaat, waardoor er een 500 error teruggegeven wordt als de combinatie al in gebruik is:

django.db.utils.IntegrityError: duplicate key value violates unique constraint "datamodel_verzoek_bronorganisatie_identificatie_ac11b65c_uniq"
DETAIL:  Key (bronorganisatie, identificatie)=(000000000, uniek) already exists.

Als de bronorganisatie aangepast wordt naar een bronorganisatie waarbinnen de identificatie al bestaat, krijg ik deze foutmelding
@stevenbal stevenbal added the bug Iets werkt niet zoals bedoeld label Oct 9, 2020
@stevenbal stevenbal self-assigned this Oct 9, 2020
@stevenbal stevenbal changed the title (vrc-001c) Verzoek resource mist validatie op uniciteit van bronorganisatie en identificatie (vrz-001c) Verzoek resource mist validatie op uniciteit van bronorganisatie en identificatie Oct 9, 2020
@stevenbal stevenbal mentioned this issue Oct 9, 2020
Merged
stevenbal added a commit to VNG-Realisatie/verzoeken-api that referenced this issue Oct 9, 2020
@stevenbal
✅ Add test for unique together validation for Verzoek bronorganisatie…
4326003 
… and identificatie

fixes: VNG-Realisatie/gemma-zaken#1696
@stevenbal stevenbal mentioned this issue Oct 9, 2020
Open
stevenbal added a commit to VNG-Realisatie/verzoeken-api that referenced this issue Dec 23, 2020
@stevenbal
✅ Add test for unique together validation for Verzoek bronorganisatie…
6ec5d13 
… and identificatie

fixes: VNG-Realisatie/gemma-zaken#1696
stevenbal added a commit to VNG-Realisatie/verzoeken-api that referenced this issue Dec 23, 2020
@stevenbal
✅ Add test for unique together validation for Verzoek bronorganisatie…
4f3d8d0 
… and identificatie

fixes: VNG-Realisatie/gemma-zaken#1696
@michielverhoef
Copy link
Collaborator

Zo te zien al opgelost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stevenbal stevenbal

Labels
bug Iets werkt niet zoals bedoeld
Projects
None yet
Milestone
No milestone
2 participants
@stevenbal @michielverhoef