64 odpc organisaties koppelen aan publicaties

ODPC.Server/Features/Organisaties/MijnOrganisaties/MijnOrganisatiesController.cs

@@ -0,0 +1,36 @@
+using System.Runtime.CompilerServices;
+using System.Text.Json.Nodes;
+using Microsoft.AspNetCore.Mvc;
+using ODPC.Apis.Odrc;
+
+namespace ODPC.Features.Organisaties.MijnOrganisaties
+{
+    public class MijnOrganisatiesController(IOdrcClientFactory clientFactory, IGebruikerWaardelijstItemsService waardelijstItemsService) : ControllerBase
+    {
+        [HttpGet("api/v1/mijn-organisaties")]
+        public async IAsyncEnumerable<JsonObject> Get([EnumeratorCancellation] CancellationToken token)
+        {
+            var organisaties = await waardelijstItemsService.GetAsync(token);
+
+            if (organisaties.Count == 0) yield break;
+
+            using var client = clientFactory.Create("Eigen organisaties ophalen");
+            var url = "api/v1/organisaties";
+
+            // omdat we zelf moeten filteren obv van de waardelijstitems waar de gebruiker toegang toe heeft,
+            // kunnen we geen paginering gebruiker. we lopen door alle pagina's van de ODRC
+            while (!string.IsNullOrWhiteSpace(url))
+            {
+                var page = await client.GetFromJsonAsync<PagedResponseModel<JsonObject>>(url, token) ?? new() { Results = [] };
+                foreach (var item in page.Results)
+                {
+                    if (item["uuid"]?.GetValue<string>() is string uuid && organisaties.Contains(uuid))
+                    {
+                        yield return item;
+                    }
+                }
+                url = page?.Next;
+            }
+        }
+    }
+}

ODPC.Server/Features/Publicaties/Publicatie.cs

@@ -3,6 +3,8 @@
     public class Publicatie
     {
         public Guid Uuid { get; set; }
+        public string? Publisher { get; set; }
+        public string? Verantwoordelijke { get; set; }
         public string? OfficieleTitel { get; set; }
         public string? VerkorteTitel { get; set; }
         public string? Omschrijving { get; set; }

ODPC.Server/Features/Publicaties/PublicatieBijwerken/PublicatieBijwerkenController.cs

@@ -8,9 +8,15 @@ public class PublicatieBijwerkenController(IGebruikerWaardelijstItemsService waa
         [HttpPut("api/v1/publicaties/{uuid}")]
         public async Task<IActionResult> Put(Guid uuid, Publicatie publicatie, CancellationToken token)
         {
-            var categorieen = await waardelijstItemsService.GetAsync(token);
+            var waardelijstItems = await waardelijstItemsService.GetAsync(token);
 
-            if (publicatie.InformatieCategorieen != null && publicatie.InformatieCategorieen.Any(c => !categorieen.Contains(c)))
+            if (publicatie.Publisher != null && !waardelijstItems.Contains(publicatie.Publisher))
+            {
+                ModelState.AddModelError(nameof(publicatie.Publisher), "Gebruiker is niet geautoriseerd voor deze organisatie");
+                return BadRequest(ModelState);
+            }
+
+            if (publicatie.InformatieCategorieen != null && publicatie.InformatieCategorieen.Any(c => !waardelijstItems.Contains(c)))
             {
                 ModelState.AddModelError(nameof(publicatie.InformatieCategorieen), "Gebruiker is niet geautoriseerd voor deze informatiecategorieën");
                 return BadRequest(ModelState);

ODPC.Server/Features/Publicaties/PublicatieRegistreren/PublicatieRegistrerenController.cs

@@ -11,9 +11,15 @@ public class PublicatieRegistrerenController(IOdrcClientFactory clientFactory, I
         [HttpPost("api/v1/publicaties")]
         public async Task<IActionResult> Post(Publicatie publicatie, CancellationToken token)
         {
-            var categorieen = await waardelijstItemsService.GetAsync(token);
+            var waardelijstItems = await waardelijstItemsService.GetAsync(token);
 
-            if (publicatie.InformatieCategorieen != null && publicatie.InformatieCategorieen.Any(c => !categorieen.Contains(c)))
+            if (publicatie.Publisher != null && !waardelijstItems.Contains(publicatie.Publisher))
+            {
+                ModelState.AddModelError(nameof(publicatie.Publisher), "Gebruiker is niet geautoriseerd voor deze organisatie");
+                return BadRequest(ModelState);
+            }
+
+            if (publicatie.InformatieCategorieen != null && publicatie.InformatieCategorieen.Any(c => !waardelijstItems.Contains(c)))
             {
                 ModelState.AddModelError(nameof(publicatie.InformatieCategorieen), "Gebruiker is niet geautoriseerd voor deze informatiecategorieën");
                 return BadRequest(ModelState);
@@ -25,9 +31,6 @@ public async Task<IActionResult> Post(Publicatie publicatie, CancellationToken t
             response.EnsureSuccessStatusCode();
             var viewModel = await response.Content.ReadFromJsonAsync<Publicatie>(token);
 
-            // TODO deze regel kan eraf als deze story is geimplementeerd: https://github.com/GeneriekPublicatiePlatformWoo/registratie-component/issues/48
-            viewModel!.InformatieCategorieen = publicatie.InformatieCategorieen;
-
             // TODO deze regel kan eraf als deze story is geimplementeerd: https://github.com/GeneriekPublicatiePlatformWoo/registratie-component/issues/49
             viewModel!.Status = publicatie.Status;
 

ODPC.Server/Features/Publicaties/PublicatiesMock.cs

@@ -7,6 +7,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Openbaarheid en Verantwoording: De Impact van de Wet open overheid op Bestuurlijke Transparantie",
                 VerkorteTitel = "De Impact van de Wet open overheid op Bestuurlijke Transparantie",
                 Omschrijving = "",
@@ -18,6 +19,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Inzicht voor Iedereen: Toepassing en Resultaten van de Wet open overheid",
                 VerkorteTitel = "Toepassing en Resultaten van de Wet open overheid",
                 Omschrijving = "",
@@ -28,6 +30,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Open Overheid: Transparantie als Standaard in Bestuurlijk Nederland",
                 VerkorteTitel = "Transparantie als Standaard in Bestuurlijk Nederland",
                 Omschrijving = "",
@@ -38,6 +41,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "De Wet open overheid: Een Nieuwe Norm voor Openbare Informatie",
                 VerkorteTitel = "Een Nieuwe Norm voor Openbare Informatie",
                 Omschrijving = "",
@@ -48,6 +52,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Inzicht in Overheid: De Toekomst van Transparantie met de Woo",
                 VerkorteTitel = "De Toekomst van Transparantie met de Woo",
                 Omschrijving = "",
@@ -58,6 +63,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Toegang tot Informatie: De Praktische Uitwerking van de Woo",
                 VerkorteTitel = "De Praktische Uitwerking van de Woo",
                 Omschrijving = "",
@@ -68,6 +74,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Openbaarheid en Verantwoording: De Impact van de Wet open overheid op Bestuurlijke Transparantie",
                 VerkorteTitel = "Openbaarheid en Verantwoording",
                 Omschrijving = "",
@@ -78,6 +85,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Verantwoording en Openheid: Hoe de Woo de Overheid Hervormt",
                 VerkorteTitel = "Hoe de Woo de Overheid Hervormt",
                 Omschrijving = "",
@@ -88,6 +96,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "De Wet open overheid in de Praktijk: Successen en Uitdagingen",
                 VerkorteTitel = "Successen en Uitdagingen",
                 Omschrijving = "",
@@ -98,6 +107,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Publieke Toegang tot Overheidsinformatie: Het Effect van de Woo",
                 VerkorteTitel = "Het Effect van de Woo",
                 Omschrijving = "",
@@ -108,6 +118,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "De Kracht van Openbaarheid: Verantwoord Bestuur door de Wet open overheid",
                 VerkorteTitel = "Verantwoord Bestuur door de Wet open overheid",
                 Omschrijving = "",
@@ -118,6 +129,7 @@ public static class PublicatiesMock
             new()
             {
                 Uuid = Guid.NewGuid(),
+                Publisher = "",
                 OfficieleTitel = "Transparantie als Fundament: De Woo en de Weg naar Open Overheid",
                 VerkorteTitel = "De Woo en de Weg naar Open Overheid",
                 Omschrijving = "",

odpc.client/src/assets/main.scss

@@ -117,7 +117,7 @@ details {
   }
 }
 
-.checkbox {
+.input-option {
   margin-block-end: var(--spacing-small);
 
   label {

odpc.client/src/components/checkbox-group/CheckboxGroup.vue → odpc.client/src/components/option-group/OptionGroup.vue

@@ -4,14 +4,13 @@
     ref="groupRef"
     :aria-labelledby="`label-${instanceId}`"
     :aria-required="required ? true : undefined"
-    @change="required && setCustomValidity()"
-    @invalid.capture="required && onInvalid()"
+    @change="setCustomValidity"
   >
     <summary :id="`label-${instanceId}`">{{ title }} {{ required ? "*" : "" }}</summary>
 
-    <p v-if="required" class="error" :id="`description-${instanceId}`">Kies minimaal één optie.</p>
+    <p v-if="required" class="error" :id="`description-${instanceId}`">{{ getMessage(type) }}</p>
 
-    <div class="checkbox check-all">
+    <div v-if="type === `checkbox`" class="input-option check-all">
       <label
         ><input
           type="checkbox"
@@ -24,10 +23,10 @@
       </label>
     </div>
 
-    <div class="checkbox" v-for="({ uuid, naam }, key) in options" :key="key">
+    <div class="input-option" v-for="({ uuid, naam }, key) in options" :key="key">
       <label
         ><input
-          type="checkbox"
+          :type="type"
           :value="uuid"
           v-model="model"
           :aria-describedby="`description-${instanceId}`"
@@ -39,42 +38,38 @@
 </template>
 
 <script setup lang="ts">
-import { computed, getCurrentInstance } from "vue";
-import { useCheckboxGroup } from "./use-checkbox-group";
+import { computed, getCurrentInstance, useModel } from "vue";
+import { useOptionGroup } from "./use-option-group";
 import type { OptionProps } from "./types";
 
 const instanceId = getCurrentInstance()?.uid;
 
-const { groupRef, setCustomValidity, onInvalid } = useCheckboxGroup();
-
-const props = defineProps<{
-  title: string;
-  options: OptionProps[];
-  modelValue: string[];
-  required?: boolean;
-}>();
+const { groupRef, setCustomValidity, getMessage } = useOptionGroup();
 
-const emit = defineEmits<{
-  (e: "update:modelValue", payload: string[]): void;
-}>();
+const props = withDefaults(
+  defineProps<{
+    type?: string;
+    title: string;
+    options: OptionProps[];
+    modelValue: string | string[];
+    required?: boolean;
+  }>(),
+  {
+    type: "checkbox"
+  }
+);
 
-const model = computed({
-  get: () => props.modelValue,
-  set: (value) => emit("update:modelValue", value)
-});
+const model = useModel(props, "modelValue");
 
-const itemIds = computed(() => props.options.map((option) => option.uuid));
+const uuids = computed(() => props.options.map((option) => option.uuid));
 
-const allSelected = computed(
-  () =>
-    !!itemIds.value?.length &&
-    model.value?.filter((id) => itemIds.value.includes(id)).length === itemIds.value?.length
-);
+const allSelected = computed(() => uuids.value.every((uuid) => model.value.includes(uuid)));
 
 const toggleAll = () => {
-  model.value = allSelected.value
-    ? model.value?.filter((id) => !itemIds.value.includes(id)) || []
-    : [...new Set([...(model.value || []), ...itemIds.value])];
+  model.value =
+    Array.isArray(model.value) && allSelected.value
+      ? model.value.filter((uuid) => !uuids.value.includes(uuid))
+      : [...new Set([...model.value, ...uuids.value])];
 };
 </script>
 

odpc.client/src/components/option-group/use-option-group.ts

@@ -0,0 +1,32 @@
+import { computed, ref, watchEffect } from "vue";
+
+export const useOptionGroup = () => {
+  const groupRef = ref<HTMLElement>();
+
+  const required = computed(() => groupRef.value?.hasAttribute("aria-required"));
+
+  const getMessage = (type: string) =>
+    type === "radio" ? "Kies één optie." : "Kies minimaal één optie.";
+
+  const isAnyChecked = (options: NodeListOf<HTMLInputElement>) =>
+    Array.from(options).some((option) => option.checked);
+
+  const setCustomValidity = () => {
+    if (!required.value) return;
+
+    const options = (groupRef.value?.querySelectorAll("[type='checkbox'], [type='radio']") ||
+      []) as NodeListOf<HTMLInputElement>;
+
+    options.forEach((option) =>
+      option.setCustomValidity(!isAnyChecked(options) ? getMessage(option.type) : "")
+    );
+  };
+
+  watchEffect(() => setCustomValidity());
+
+  return {
+    groupRef,
+    setCustomValidity,
+    getMessage
+  };
+};

odpc.client/src/directives/form-invalid-handler.ts

@@ -0,0 +1,22 @@
+export default {
+  mounted(el: HTMLFormElement) {
+    const onInvalid = () => {
+      const invalidInputs = (el.querySelectorAll(":user-invalid") ||
+        []) as NodeListOf<HTMLInputElement>;
+
+      invalidInputs.forEach((input) => {
+        const details = input.closest("details");
+        details && (details.open = true);
+      });
+    };
+
+    el.addEventListener("invalid", onInvalid, true);
+    el.onInvalidHandler = onInvalid;
+  },
+  unmounted(el: HTMLFormElement) {
+    if (el.onInvalidHandler) {
+      el.removeEventListener("invalid", el.onInvalidHandler, true);
+      delete el.onInvalidHandler;
+    }
+  }
+};

odpc.client/src/features/gebruikersgroep/GebruikersgroepDetails.vue

@@ -1,7 +1,7 @@
 <template>
   <simple-spinner v-show="loadingGebruikersgroep"></simple-spinner>
 
-  <form v-show="!loadingGebruikersgroep" @submit.prevent="submit" ref="formRef">
+  <form v-show="!loadingGebruikersgroep" @submit.prevent="submit" v-form-invalid-handler>
     <section>
       <alert-inline v-if="gebruikersgroepError"
         >Er is iets misgegaan bij het ophalen van de gebruikersgroep...</alert-inline