From 9171872ec4b9106ca8a03d3a216757ec868aa181 Mon Sep 17 00:00:00 2001
From: Oliver Booth <me@olivr.me>
Date: Mon, 8 May 2023 16:00:00 +0100
Subject: [PATCH] refactor: offer cls-compliant attribute ctors

---
 .../ExecutionChecks/RequireAvatarNameAttribute.cs  | 14 ++++++++++++++
 .../ExecutionChecks/RequireUserIdAttribute.cs      | 14 ++++++++++++++
 .../ExecutionChecks/RequireUserNameAttribute.cs    | 14 ++++++++++++++
 3 files changed, 42 insertions(+)

diff --git a/VpSharp.Commands/Attributes/ExecutionChecks/RequireAvatarNameAttribute.cs b/VpSharp.Commands/Attributes/ExecutionChecks/RequireAvatarNameAttribute.cs
index 5b851a6..c395675 100644
--- a/VpSharp.Commands/Attributes/ExecutionChecks/RequireAvatarNameAttribute.cs
+++ b/VpSharp.Commands/Attributes/ExecutionChecks/RequireAvatarNameAttribute.cs
@@ -1,15 +1,29 @@
 namespace VpSharp.Commands.Attributes.ExecutionChecks;
 
+#pragma warning disable CA1019 // Define accessors for attribute arguments
+
 /// <summary>
 ///     Specifies that this command can only be run by bots.
 /// </summary>
 public sealed class RequireAvatarNameAttribute : PreExecutionCheckAttribute
 {
+    /// <summary>
+    ///     Initializes a new instance of the <see cref="RequireAvatarNameAttribute" /> class.
+    /// </summary>
+    /// <param name="names">An enumerable collection of allowed user names.</param>
+    /// <exception cref="ArgumentNullException"><paramref name="names" /> is <see langword="null" />.</exception>
+    public RequireAvatarNameAttribute(IEnumerable<string> names)
+    {
+        ArgumentNullException.ThrowIfNull(names);
+        Names = names.ToArray();
+    }
+
     /// <summary>
     ///     Initializes a new instance of the <see cref="RequireAvatarNameAttribute" /> class.
     /// </summary>
     /// <param name="names">An array of allowed user names.</param>
     /// <exception cref="ArgumentNullException"><paramref name="names" /> is <see langword="null" />.</exception>
+    [CLSCompliant(false)]
     public RequireAvatarNameAttribute(params string[] names)
     {
         ArgumentNullException.ThrowIfNull(names);
diff --git a/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserIdAttribute.cs b/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserIdAttribute.cs
index 57541b1..cfeefd6 100644
--- a/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserIdAttribute.cs
+++ b/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserIdAttribute.cs
@@ -1,15 +1,29 @@
 namespace VpSharp.Commands.Attributes.ExecutionChecks;
 
+#pragma warning disable CA1019 // Define accessors for attribute arguments
+
 /// <summary>
 ///     Specifies that this command can only be run by bots.
 /// </summary>
 public sealed class RequireUserIdAttribute : PreExecutionCheckAttribute
 {
+    /// <summary>
+    ///     Initializes a new instance of the <see cref="RequireUserIdAttribute" /> class.
+    /// </summary>
+    /// <param name="userIds">An enumerable collection of allowed user IDs.</param>
+    /// <exception cref="ArgumentNullException"><paramref name="userIds" /> is <see langword="null" />.</exception>
+    public RequireUserIdAttribute(IEnumerable<int> userIds)
+    {
+        ArgumentNullException.ThrowIfNull(userIds);
+        UserIds = userIds.ToArray();
+    }
+
     /// <summary>
     ///     Initializes a new instance of the <see cref="RequireUserIdAttribute" /> class.
     /// </summary>
     /// <param name="userIds">An array of allowed user IDs.</param>
     /// <exception cref="ArgumentNullException"><paramref name="userIds" /> is <see langword="null" />.</exception>
+    [CLSCompliant(false)]
     public RequireUserIdAttribute(params int[] userIds)
     {
         ArgumentNullException.ThrowIfNull(userIds);
diff --git a/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserNameAttribute.cs b/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserNameAttribute.cs
index 5ccb619..7847a1f 100644
--- a/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserNameAttribute.cs
+++ b/VpSharp.Commands/Attributes/ExecutionChecks/RequireUserNameAttribute.cs
@@ -1,15 +1,29 @@
 namespace VpSharp.Commands.Attributes.ExecutionChecks;
 
+#pragma warning disable CA1019 // Define accessors for attribute arguments
+
 /// <summary>
 ///     Specifies that this command can only be run by bots.
 /// </summary>
 public sealed class RequireUserNameAttribute : PreExecutionCheckAttribute
 {
+    /// <summary>
+    ///     Initializes a new instance of the <see cref="RequireUserNameAttribute" /> class.
+    /// </summary>
+    /// <param name="names">An enumerable collection of allowed user names.</param>
+    /// <exception cref="ArgumentNullException"><paramref name="names" /> is <see langword="null" />.</exception>
+    public RequireUserNameAttribute(IEnumerable<string> names)
+    {
+        ArgumentNullException.ThrowIfNull(names);
+        Names = names.ToArray();
+    }
+
     /// <summary>
     ///     Initializes a new instance of the <see cref="RequireUserNameAttribute" /> class.
     /// </summary>
     /// <param name="names">An array of allowed user names.</param>
     /// <exception cref="ArgumentNullException"><paramref name="names" /> is <see langword="null" />.</exception>
+    [CLSCompliant(false)]
     public RequireUserNameAttribute(params string[] names)
     {
         ArgumentNullException.ThrowIfNull(names);