VNG-Realisatie / gemma-zaken

Samen ontwikkelen van API's voor Zaakgericht werken

https://vng-realisatie.github.io/gemma-zaken/

Other

41 stars 26 forks source link

Scope geforceerd-bijwerken in Documenten API is overbodig vanaf 1.4 #2390

Open joeribekker opened 5 months ago

joeribekker commented 5 months ago

Bug

In this ticket https://github.com/VNG-Realisatie/gemma-zaken/issues/2241 it was decided to remove the status-check to prevent the update of "definitive" documents.

It was however already possible to update documents, using a special scope "geforceerd-bijwerken". It was added via this user story: https://github.com/VNG-Realisatie/gemma-zaken/issues/1859 with the same reasoning.

I still think this is the proper way to do it. Changing a definitive document should not be possible except for "special cases", for which we have this scope.

Anyway, either bring back the status check or remove the permission scope.

HenriKorver commented 5 months ago

Het lijkt erop dat bij het uitwerken van ticket #2241 het bestaan van de special scope "geforceerd-bijwerken" over het hoofd is gezien.

@michielverhoef en @johannesbattjes: Is dit inderdaad over het hoofd gezien of was er een speciale reden om het zo te doen?

MNIJ commented 4 months ago

De scope geforceerd_bijwerken is volgens mij bedoeld voor handelingen die buiten normaal gebruik vallen. Denk bijvoorbeeld aan foutherstel. Bewerkingen op definitieve documenten vallen wel in normaal gebruik, zoals toevoegen van ondertekening, verzenddatum, enzovoorts. Daarvoor zou deze scope dus niet nodig moeten zijn. Maar dat wil niet zeggen dat deze scope helemaal niet meer nodig is. Het lijkt me verstandig om deze te behouden, maar dan alleen voor het bewerken van gearchiveerde documenten. Dat trekt het ook gelijk met de werking bij zaken, waar geforceerd-bijwerken nodig is voor het bewerken van gesloten zaken. (de vraag was aan @johannesbattjes, ik heb dit antwoord met hem afgestemd)