Skip to content

Commit

Permalink
fix: or-2334 use dedicated property for IsAuthentieke verenging
Browse files Browse the repository at this point in the history
  • Loading branch information
emalfroy committed Dec 24, 2024
1 parent 388eec4 commit 5ce5abf
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
namespace AssociationRegistry.Admin.AddressSync.MessageHandling.Sqs.AddressSync;

using AssociationRegistry.Messages;
using Messages;
using Wolverine;

public class TeSynchroniserenLocatieAdresMessageHandler(IMessageBus messageBus)
Expand Down
2 changes: 1 addition & 1 deletion src/AssociationRegistry/Vereniging/Vereniging.cs
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,7 @@ public void MarkeerAlsDubbelVan(VCode isDubbelVan)
{
Throw<VerenigingKanGeenDubbelWordenVanZichzelf>.If(isDubbelVan.Equals(VCode));

Throw<AuthentiekeVerenigingKanNietAlsDubbelGemarkeerdWorden>.If(State.CorresponderendeVCodes.Any());
Throw<AuthentiekeVerenigingKanNietAlsDubbelGemarkeerdWorden>.If(State.IsAuthentiekeVereniging);

AddEvent(VerenigingWerdGemarkeerdAlsDubbelVan.With(VCode, isDubbelVan));
}
Expand Down
1 change: 1 addition & 0 deletions src/AssociationRegistry/Vereniging/VerenigingState.cs
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ public string Identity
public List<string> HandledIdempotenceKeys { get; set; } = new();
public bool IsVerwijderd { get; set; }
public bool IsDubbel => VerenigingStatus == VerenigingStatus.Dubbel;
public bool IsAuthentiekeVereniging => CorresponderendeVCodes.Length != 0;
public string[] CorresponderendeVCodes { get; set; } = [];
public VerenigingStatus VerenigingStatus { get; set; }

Check warning on line 48 in src/AssociationRegistry/Vereniging/VerenigingState.cs

View workflow job for this annotation

GitHub Actions / Build PowerBI Export Scheduled Task / build-image

Non-nullable property 'VerenigingStatus' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.

Check warning on line 48 in src/AssociationRegistry/Vereniging/VerenigingState.cs

View workflow job for this annotation

GitHub Actions / Build ACM Api / build-image

Non-nullable property 'VerenigingStatus' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.

Check warning on line 48 in src/AssociationRegistry/Vereniging/VerenigingState.cs

View workflow job for this annotation

GitHub Actions / Build Admin Api / build-image

Non-nullable property 'VerenigingStatus' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
public VerenigingStatus VorigeVerenigingStatus { get; set; }

Check warning on line 49 in src/AssociationRegistry/Vereniging/VerenigingState.cs

View workflow job for this annotation

GitHub Actions / Build PowerBI Export Scheduled Task / build-image

Non-nullable property 'VorigeVerenigingStatus' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
Expand Down

0 comments on commit 5ce5abf

Please sign in to comment.