feat: or-2516 implement deleting a lidmaatschap

src/AssociationRegistry.Admin.Api/Verenigingen/Lidmaatschap/VerwijderLidmaatschap/VerwijderLidmaatschapController.cs

@@ -1,15 +1,19 @@
 namespace AssociationRegistry.Admin.Api.Verenigingen.Lidmaatschap.VerwijderLidmaatschap;
 
+using Acties.VerwijderLidmaatschap;
 using Asp.Versioning;
 using Be.Vlaanderen.Basisregisters.Api;
 using Be.Vlaanderen.Basisregisters.Api.Exceptions;
+using Framework;
 using Hosts.Configuration.ConfigurationBindings;
 using Infrastructure;
+using Infrastructure.Extensions;
 using Infrastructure.Middleware;
 using Infrastructure.Swagger.Annotations;
 using Infrastructure.Swagger.Examples;
 using Microsoft.AspNetCore.Mvc;
 using Swashbuckle.AspNetCore.Filters;
+using Vereniging;
 using Wolverine;
 using ProblemDetails = Be.Vlaanderen.Basisregisters.BasicApiProblem.ProblemDetails;
 using ValidationProblemDetails = Be.Vlaanderen.Basisregisters.BasicApiProblem.ValidationProblemDetails;
@@ -69,12 +73,15 @@ public async Task<IActionResult> VerwijderLidmaatschap(
         [FromServices] ICommandMetadataProvider metadataProvider,
         [FromHeader(Name = "If-Match")] string? ifMatch = null)
     {
-        // var metaData = metadataProvider.GetMetadata(IfMatchParser.ParseIfMatch(ifMatch));
-        // var envelope = new CommandEnvelope<VerwijderLidmaatschapCommand>(request.ToCommand(vCode, lidmaatschapId), metaData);
-        // var commandResult = await _messageBus.InvokeAsync<EntityCommandResult>(envelope);
-        //
-        // return this.AcceptedEntityCommand(_appSettings, WellKnownHeaderEntityNames.Lidmaatschappen, commandResult);
+        var metaData = metadataProvider.GetMetadata(IfMatchParser.ParseIfMatch(ifMatch));
+        var envelope = new CommandEnvelope<VerwijderLidmaatschapCommand>(
+            new VerwijderLidmaatschapCommand(VCode.Create(vCode), new LidmaatschapId(lidmaatschapId)),
+            metaData);
+        var commandResult = await _messageBus.InvokeAsync<CommandResult>(envelope);
 
-        return new EmptyResult();
+        Response.AddSequenceHeader(commandResult.Sequence);
+        Response.AddETagHeader(commandResult.Version);
+
+        return Accepted();
     }
 }

src/AssociationRegistry.Admin.Api/Verenigingen/Lidmaatschap/VoegLidmaatschapToe/RequestModels/VoegLidmaatschapToeRequest.cs

@@ -38,11 +38,11 @@ public class VoegLidmaatschapToeRequest
     public string Beschrijving { get; set; }
 
     public VoegLidmaatschapToeCommand ToCommand(string vCode) => new(
-        VCode: VCode.Create(vCode),
-        Lidmaatschap.Create(VCode.Create(AndereVereniging),
-                            new Geldigheidsperiode(new GeldigVan(Van), new GeldigTot(Tot)),
-                            Identificatie,
-                            Beschrijving
-        )
-    );
+        VCode.Create(vCode),
+        new VoegLidmaatschapToeCommand.ToeTeVoegenLidmaatschap(
+        VCode.Create(AndereVereniging),
+        new Geldigheidsperiode(new GeldigVan(Van), new GeldigTot(Tot)),
+        Identificatie,
+        Beschrijving
+    ));
 }

src/AssociationRegistry.Admin.ProjectionHost/Projections/Detail/BeheerVerenigingDetailProjection.cs

@@ -211,6 +211,9 @@ public async Task Project(IEvent<AdresHeeftGeenVerschillenMetAdressenregister> @
     public async Task Project(IEvent<LidmaatschapWerdToegevoegd> @event, IDocumentOperations ops)
         => await Update(@event, ops, BeheerVerenigingDetailProjector.Apply);
 
+    public async Task Project(IEvent<LidmaatschapWerdVerwijderd> @event, IDocumentOperations ops)
+        => await Update(@event, ops, BeheerVerenigingDetailProjector.Apply);
+
     private async Task SoftDelete(string? streamKey, IDocumentOperations ops)
         => ops.Delete<BeheerVerenigingDetailDocument>(streamKey);
 

src/AssociationRegistry.Admin.ProjectionHost/Projections/Detail/BeheerVerenigingDetailProjector.cs

@@ -50,7 +50,7 @@ public static BeheerVerenigingDetailDocument Create(IEvent<FeitelijkeVerenigingW
             HoofdactiviteitenVerenigingsloket = feitelijkeVerenigingWerdGeregistreerd.Data
                                                                                      .HoofdactiviteitenVerenigingsloket
                                                                                      .Select(BeheerVerenigingDetailMapper
-                                                                                         .MapHoofdactiviteitVerenigingsloket)
+                                                                                             .MapHoofdactiviteitVerenigingsloket)
                                                                                      .ToArray(),
             Werkingsgebieden = feitelijkeVerenigingWerdGeregistreerd.Data
                                                                     .Werkingsgebieden?
@@ -208,10 +208,9 @@ public static void Apply(
         BeheerVerenigingDetailDocument document)
     {
         document.Werkingsgebieden = werkingsgebiedenWerdenGewijzigd.Data.Werkingsgebieden
-           .Select(BeheerVerenigingDetailMapper.MapWerkingsgebied).ToArray();
+                                                                   .Select(BeheerVerenigingDetailMapper.MapWerkingsgebied).ToArray();
     }
 
-
     public static void Apply(
         IEvent<VertegenwoordigerWerdToegevoegd> vertegenwoordigerWerdToegevoegd,
         BeheerVerenigingDetailDocument document)
@@ -688,11 +687,19 @@ public static void Apply(
 
     public static void Apply(IEvent<LidmaatschapWerdToegevoegd> lidmaatschapWerdToegevoegd, BeheerVerenigingDetailDocument document)
     {
+        document.Lidmaatschappen = document.Lidmaatschappen
+                                           .Append(BeheerVerenigingDetailMapper.MapLidmaatschap(
+                                                       lidmaatschapWerdToegevoegd.Data.Lidmaatschap,
+                                                       document.VCode))
+                                           .OrderBy(l => l.LidmaatschapId)
+                                           .ToArray();
+    }
 
+    public static void Apply(IEvent<LidmaatschapWerdVerwijderd> lidmaatschapWerdToegevoegd, BeheerVerenigingDetailDocument document)
+    {
         document.Lidmaatschappen = document.Lidmaatschappen
-                                    .Append(BeheerVerenigingDetailMapper.MapLidmaatschap(lidmaatschapWerdToegevoegd.Data.Lidmaatschap,
-                                                                                    document.VCode))
-                                    .OrderBy(l => l.LidmaatschapId)
-                                    .ToArray();
+                                           .Where(l => l.LidmaatschapId != lidmaatschapWerdToegevoegd.Data.Lidmaatschap.LidmaatschapId)
+                                           .OrderBy(l => l.LidmaatschapId)
+                                           .ToArray();
     }
 }

src/AssociationRegistry/Acties/VerwijderLidmaatschap/VerwijderLidmaatschapCommand.cs

@@ -0,0 +1,5 @@
+namespace AssociationRegistry.Acties.VerwijderLidmaatschap;
+
+using Vereniging;
+
+public record VerwijderLidmaatschapCommand(VCode VCode, LidmaatschapId LidmaatschapId);

src/AssociationRegistry/Acties/VerwijderLidmaatschap/VerwijderLidmaatschapCommandHandler.cs

@@ -0,0 +1,30 @@
+namespace AssociationRegistry.Acties.VerwijderLidmaatschap;
+
+using Framework;
+using Vereniging;
+
+public class VerwijderLidmaatschapCommandHandler
+{
+    private readonly IVerenigingsRepository _verenigingRepository;
+
+    public VerwijderLidmaatschapCommandHandler(IVerenigingsRepository verenigingRepository)
+    {
+        _verenigingRepository = verenigingRepository;
+    }
+
+    public async Task<CommandResult> Handle(
+        CommandEnvelope<VerwijderLidmaatschapCommand> envelope,
+        CancellationToken cancellationToken = default)
+    {
+        var vereniging =
+            await _verenigingRepository.Load<VerenigingOfAnyKind>(
+                VCode.Create(envelope.Command.VCode),
+                envelope.Metadata.ExpectedVersion);
+
+        vereniging.VerwijderLidmaatschap(envelope.Command.LidmaatschapId);
+
+        var result = await _verenigingRepository.Save(vereniging, envelope.Metadata, cancellationToken);
+
+        return CommandResult.Create(VCode.Create(envelope.Command.VCode), result);
+    }
+}

src/AssociationRegistry/Acties/VoegLidmaatschapToe/VoegLidmaatschapToeCommand.cs

@@ -2,4 +2,14 @@
 
 using Vereniging;
 
-public record VoegLidmaatschapToeCommand(VCode VCode, Lidmaatschap Lidmaatschap);
+public record VoegLidmaatschapToeCommand(
+    VCode VCode,
+    VoegLidmaatschapToeCommand.ToeTeVoegenLidmaatschap Lidmaatschap)
+{
+    public record ToeTeVoegenLidmaatschap(
+        VCode AndereVereniging,
+        Geldigheidsperiode Geldigheidsperiode,
+        string Identificatie,
+        string Beschrijving);
+}
+

src/AssociationRegistry/Events/LidmaatschapWerdToegevoegd.cs

@@ -1,14 +1,13 @@
 namespace AssociationRegistry.Events;
 
 using Framework;
-using System.Runtime.Serialization;
 using Vereniging;
-using Vereniging.Bronnen;
 
-public record LidmaatschapWerdToegevoegd(Registratiedata.Lidmaatschap Lidmaatschap) : IEvent
+public record LidmaatschapWerdToegevoegd(string VCode, Registratiedata.Lidmaatschap Lidmaatschap) : IEvent
 {
-    public static LidmaatschapWerdToegevoegd With(Lidmaatschap lidmaatschap)
+    public static LidmaatschapWerdToegevoegd With(VCode vCode ,Lidmaatschap lidmaatschap)
         => new(
+            vCode,
             Registratiedata.Lidmaatschap.With(lidmaatschap)
         );
 }

src/AssociationRegistry/Events/LidmaatschapWerdVerwijderd.cs

@@ -0,0 +1,11 @@
+namespace AssociationRegistry.Events;
+
+using Framework;
+using Vereniging;
+
+public record LidmaatschapWerdVerwijderd(string VCode,
+                                         Registratiedata.Lidmaatschap Lidmaatschap) : IEvent
+{
+    public static LidmaatschapWerdVerwijderd With(VCode vCode, Lidmaatschap lidmaatschap)
+        => new(vCode, Registratiedata.Lidmaatschap.With(lidmaatschap));
+}

src/AssociationRegistry/Resources/ExceptionMessages.resx

@@ -190,6 +190,9 @@
   <data name="LidmaatschapMagNietVerwijzenNaarEigenVereniging">
     <value>Een lidmaatschap mag niet verwijzen naar de vereniging waarbij hij wordt toegevoegd.</value>
   </data>
+  <data name="LidmaatschapIsNietGekend">
+    <value>Lidmaatschap met lidmaatschapId '{0}' is niet gekend</value>
+  </data>
   <data name="VerenigingWerdVerwijderd">
     <value>Deze vereniging werd verwijderd.</value>
   </data>

src/AssociationRegistry/Vereniging/Lidmaatschappen/Exceptions/LidmaatschapIsNietGekend.cs

@@ -0,0 +1,17 @@
+namespace AssociationRegistry.Vereniging.Exceptions;
+
+using Be.Vlaanderen.Basisregisters.AggregateSource;
+using Resources;
+using System.Runtime.Serialization;
+
+[Serializable]
+public class LidmaatschapIsNietGekend : DomainException
+{
+    public LidmaatschapIsNietGekend(string lidmaatschapId) : base(string.Format(ExceptionMessages.LidmaatschapIsNietGekend, lidmaatschapId))
+    {
+    }
+
+    protected LidmaatschapIsNietGekend(SerializationInfo info, StreamingContext context) : base(info, context)
+    {
+    }
+}

src/AssociationRegistry/Vereniging/Lidmaatschappen/Lidmaatschap.cs

@@ -1,28 +1,27 @@
 namespace AssociationRegistry.Vereniging;
 
+using Acties.VoegLidmaatschapToe;
+
 public record Lidmaatschap
 {
-    public int LidmaatschapId { get; init; }
+    public LidmaatschapId LidmaatschapId { get; init; }
     public VCode AndereVereniging { get; init; }
     public Geldigheidsperiode Geldigheidsperiode { get; init; }
     public string Identificatie { get; init; }
     public string Beschrijving { get; init; }
 
-    private Lidmaatschap(VCode andereVereniging, Geldigheidsperiode geldigheidsperiode, string identificatie, string beschrijving)
-    {
-        AndereVereniging = andereVereniging;
-        Geldigheidsperiode = geldigheidsperiode;
-        Identificatie = identificatie;
-        Beschrijving = beschrijving;
-    }
-
-    public static Lidmaatschap Create(
+    private Lidmaatschap(
+        LidmaatschapId lidmaatschapId,
         VCode andereVereniging,
         Geldigheidsperiode geldigheidsperiode,
         string identificatie,
         string beschrijving)
     {
-        return new Lidmaatschap(andereVereniging, geldigheidsperiode, identificatie, beschrijving);
+        LidmaatschapId = lidmaatschapId;
+        AndereVereniging = andereVereniging;
+        Geldigheidsperiode = geldigheidsperiode;
+        Identificatie = identificatie;
+        Beschrijving = beschrijving;
     }
 
     public static Lidmaatschap Hydrate(
@@ -31,10 +30,7 @@ public static Lidmaatschap Hydrate(
         Geldigheidsperiode geldigheidsperiode,
         string identificatie,
         string beschrijving)
-        => new(andereVereniging, geldigheidsperiode, identificatie, beschrijving)
-        {
-            LidmaatschapId = lidmaatschapId
-        };
+        => new(new LidmaatschapId(lidmaatschapId), andereVereniging, geldigheidsperiode, identificatie, beschrijving);
 
     public virtual bool Equals(Lidmaatschap? other)
     {
@@ -58,4 +54,13 @@ public virtual bool Equals(Lidmaatschap? other)
 
     public override int GetHashCode()
         => HashCode.Combine(LidmaatschapId, AndereVereniging, Geldigheidsperiode, Identificatie, Beschrijving);
+
+    public static Lidmaatschap Create(
+        LidmaatschapId lidmaatschapId,
+        VoegLidmaatschapToeCommand.ToeTeVoegenLidmaatschap lidmaatschap)
+        => new(lidmaatschapId,
+               lidmaatschap.AndereVereniging,
+               lidmaatschap.Geldigheidsperiode,
+               lidmaatschap.Identificatie,
+               lidmaatschap.Beschrijving);
 }

src/AssociationRegistry/Vereniging/Lidmaatschappen/LidmaatschapId.cs

@@ -0,0 +1,16 @@
+namespace AssociationRegistry.Vereniging;
+
+using Be.Vlaanderen.Basisregisters.AggregateSource;
+
+public class LidmaatschapId : IntegerValueObject<LidmaatschapId>
+{
+    public static readonly LidmaatschapId InitialId = new(1);
+    public LidmaatschapId Next
+        => new LidmaatschapId(Value + 1);
+    public LidmaatschapId(int id) : base(id)
+    {
+    }
+
+    public static LidmaatschapId Max(LidmaatschapId id1, LidmaatschapId id2)
+        => new(Math.Max(id1.Value, id2.Value));
+}