Skip to content

Commit

Permalink
refactor: or-1716 check only on status of state for corrigeer dubbel van
Browse files Browse the repository at this point in the history
  • Loading branch information
emalfroy committed Jan 3, 2025
1 parent 34153ec commit 683c345
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions src/AssociationRegistry/Vereniging/Vereniging.cs
Original file line number Diff line number Diff line change
Expand Up @@ -243,9 +243,9 @@ public void MarkeerAlsDubbelVan(VCode isDubbelVan)

public string CorrigeerMarkeringAlsDubbelVan()
{
Throw<VerenigingMoetGemarkeerdZijnAlsDubbelOmGecorrigeerdTeKunnenWorden>.If(!State.IsDubbel || State.IsDubbelVan == string.Empty);
Throw<VerenigingMoetGemarkeerdZijnAlsDubbelOmGecorrigeerdTeKunnenWorden>.If(!State.IsDubbel);

var vCodeAuthentiekeVereniging = VCode.Create(State.IsDubbelVan);
var vCodeAuthentiekeVereniging = VCode.Create(State.VCodeAuthentiekeVereniging);
AddEvent(MarkeringDubbeleVerengingWerdGecorrigeerd.With(VCode, vCodeAuthentiekeVereniging, State.VorigeVerenigingStatus));

return vCodeAuthentiekeVereniging;
Expand Down
6 changes: 3 additions & 3 deletions src/AssociationRegistry/Vereniging/VerenigingState.cs
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ public string Identity
public List<string> HandledIdempotenceKeys { get; set; } = new();
public bool IsVerwijderd { get; set; }
public bool IsDubbel => VerenigingStatus == VerenigingStatus.Dubbel;
public string IsDubbelVan { get; set; }
public string VCodeAuthentiekeVereniging { get; set; }
public bool IsAuthentiekeVereniging => CorresponderendeVCodes.Length != 0;
public string[] CorresponderendeVCodes { get; set; } = [];
public VerenigingStatus VerenigingStatus { get; set; }
Expand Down Expand Up @@ -685,7 +685,7 @@ public VerenigingState Apply(VerenigingWerdGemarkeerdAlsDubbelVan @event)
{
VorigeVerenigingStatus = VerenigingStatus,
VerenigingStatus = VerenigingStatus.Dubbel,
IsDubbelVan = @event.VCodeAuthentiekeVereniging,
VCodeAuthentiekeVereniging = @event.VCodeAuthentiekeVereniging,
};

public VerenigingState Apply(VerenigingAanvaarddeDubbeleVereniging @event)
Expand All @@ -698,7 +698,7 @@ public VerenigingState Apply(WeigeringDubbelDoorAuthentiekeVerenigingWerdVerwerk
=> this with
{
VerenigingStatus = VerenigingStatus.Parse(@event.VorigeStatus),
IsDubbelVan = string.Empty,
VCodeAuthentiekeVereniging = string.Empty,
};

public void ThrowIfVerwijderd()
Expand Down

0 comments on commit 683c345

Please sign in to comment.