Revert "[Feature] Components V2 (#3065)" (#3100)

This reverts commit ba78e0c.

Author: Misha-133

src/Discord.Net.Core/Entities/Interactions/IDiscordInteraction.cs

@@ -132,7 +132,7 @@ public interface IDiscordInteraction : ISnowflakeEntity
         ///     A task that represents an asynchronous send operation for delivering the message.
         /// </returns>
         Task RespondAsync(string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false, AllowedMentions allowedMentions = null,
-            MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null, MessageFlags flags = MessageFlags.None);
+            MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
 
         /// <summary>
         ///     Responds to this interaction with a file attachment.
@@ -154,8 +154,7 @@ Task RespondAsync(string text = null, Embed[] embeds = null, bool isTTS = false,
         /// </returns>
 #if NETCOREAPP3_0_OR_GREATER
         async Task RespondWithFileAsync(Stream fileStream, string fileName, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None)
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null)
         {
             using (var file = new FileAttachment(fileStream, fileName))
             {
@@ -164,8 +163,7 @@ async Task RespondWithFileAsync(Stream fileStream, string fileName, string text
         }
 #else
         Task RespondWithFileAsync(Stream fileStream, string fileName, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null
-            , MessageFlags flags = MessageFlags.None);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
 #endif
         /// <summary>
         ///     Responds to this interaction with a file attachment.
@@ -187,8 +185,7 @@ Task RespondWithFileAsync(Stream fileStream, string fileName, string text = null
         /// </returns>
 #if NETCOREAPP3_0_OR_GREATER
         async Task RespondWithFileAsync(string filePath, string fileName = null, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None)
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null)
         {
             using (var file = new FileAttachment(filePath, fileName))
             {
@@ -197,8 +194,7 @@ async Task RespondWithFileAsync(string filePath, string fileName = null, string
         }
 #else
         Task RespondWithFileAsync(string filePath, string fileName = null, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
 #endif
         /// <summary>
         ///     Responds to this interaction with a file attachment.
@@ -219,13 +215,11 @@ Task RespondWithFileAsync(string filePath, string fileName = null, string text =
         /// </returns>
 #if NETCOREAPP3_0_OR_GREATER
         Task RespondWithFileAsync(FileAttachment attachment, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None)
-            => RespondWithFilesAsync([attachment], text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll, flags);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null)
+            => RespondWithFilesAsync(new FileAttachment[] { attachment }, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll);
 #else
         Task RespondWithFileAsync(FileAttachment attachment, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
 #endif
         /// <summary>
         ///     Responds to this interaction with a collection of file attachments.
@@ -245,9 +239,7 @@ Task RespondWithFileAsync(FileAttachment attachment, string text = null, Embed[]
         ///     contains the sent message.
         /// </returns>
         Task RespondWithFilesAsync(IEnumerable<FileAttachment> attachments, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None);
-
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
         /// <summary>
         ///     Sends a followup message for this interaction.
         /// </summary>
@@ -265,9 +257,7 @@ Task RespondWithFilesAsync(IEnumerable<FileAttachment> attachments, string text
         ///     contains the sent message.
         /// </returns>
         Task<IUserMessage> FollowupAsync(string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-             AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-             MessageFlags flags = MessageFlags.None);
-
+             AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
         /// <summary>
         ///     Sends a followup message for this interaction.
         /// </summary>
@@ -288,19 +278,17 @@ Task<IUserMessage> FollowupAsync(string text = null, Embed[] embeds = null, bool
         /// </returns>
 #if NETCOREAPP3_0_OR_GREATER
         async Task<IUserMessage> FollowupWithFileAsync(Stream fileStream, string fileName, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null, MessageFlags flags = MessageFlags.None)
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null)
         {
             using (var file = new FileAttachment(fileStream, fileName))
             {
-                return await FollowupWithFileAsync(file, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll, flags).ConfigureAwait(false);
+                return await FollowupWithFileAsync(file, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll).ConfigureAwait(false);
             }
         }
 #else
         Task<IUserMessage> FollowupWithFileAsync(Stream fileStream, string fileName, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
 #endif
-
         /// <summary>
         ///     Sends a followup message for this interaction.
         /// </summary>
@@ -321,17 +309,16 @@ Task<IUserMessage> FollowupWithFileAsync(Stream fileStream, string fileName, str
         /// </returns>
 #if NETCOREAPP3_0_OR_GREATER
         async Task<IUserMessage> FollowupWithFileAsync(string filePath, string fileName = null, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null,
-            MessageFlags flags = MessageFlags.None)
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null)
         {
             using (var file = new FileAttachment(filePath, fileName))
             {
-                return await FollowupWithFileAsync(file, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll, flags).ConfigureAwait(false);
+                return await FollowupWithFileAsync(file, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll).ConfigureAwait(false);
             }
         }
 #else
         Task<IUserMessage> FollowupWithFileAsync(string filePath, string fileName = null, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null, MessageFlags flags = MessageFlags.None);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
 #endif
         /// <summary>
         ///     Sends a followup message for this interaction.
@@ -352,11 +339,11 @@ Task<IUserMessage> FollowupWithFileAsync(string filePath, string fileName = null
         /// </returns>
 #if NETCOREAPP3_0_OR_GREATER
         Task<IUserMessage> FollowupWithFileAsync(FileAttachment attachment, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null, MessageFlags flags = MessageFlags.None)
-            => FollowupWithFilesAsync(new FileAttachment[] { attachment }, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll, flags);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null)
+            => FollowupWithFilesAsync(new FileAttachment[] { attachment }, text, embeds, isTTS, ephemeral, allowedMentions, components, embed, options, poll);
 #else
         Task<IUserMessage> FollowupWithFileAsync(FileAttachment attachment, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null, MessageFlags flags = MessageFlags.None);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
 #endif
         /// <summary>
         ///     Sends a followup message for this interaction.
@@ -376,7 +363,7 @@ Task<IUserMessage> FollowupWithFileAsync(FileAttachment attachment, string text
         ///     contains the sent message.
         /// </returns>
         Task<IUserMessage> FollowupWithFilesAsync(IEnumerable<FileAttachment> attachments, string text = null, Embed[] embeds = null, bool isTTS = false, bool ephemeral = false,
-            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null, MessageFlags flags = MessageFlags.None);
+            AllowedMentions allowedMentions = null, MessageComponent components = null, Embed embed = null, RequestOptions options = null, PollProperties poll = null);
         /// <summary>
         ///     Gets the original response for this interaction.
         /// </summary>

src/Discord.Net.Core/Entities/Interactions/MessageComponents/ActionRowComponent.cs

@@ -1,27 +1,27 @@
 using System.Collections.Generic;
 
-namespace Discord;
-
-/// <summary>
-///     Represents a <see cref="IMessageComponent"/> Row for child components to live in.
-/// </summary>
-public class ActionRowComponent : IMessageComponent
+namespace Discord
 {
-    /// <inheritdoc/>
-    public ComponentType Type => ComponentType.ActionRow;
-
-    /// <inheritdoc/>
-    public int? Id { get; internal set; }
-
     /// <summary>
-    ///     Gets the child components in this row.
+    ///     Represents a <see cref="IMessageComponent"/> Row for child components to live in.
     /// </summary>
-    public IReadOnlyCollection<IMessageComponent> Components { get; internal set; }
+    public class ActionRowComponent : IMessageComponent
+    {
+        /// <inheritdoc/>
+        public ComponentType Type => ComponentType.ActionRow;
 
-    internal ActionRowComponent() { }
+        /// <summary>
+        ///     Gets the child components in this row.
+        /// </summary>
+        public IReadOnlyCollection<IMessageComponent> Components { get; internal set; }
 
-    internal ActionRowComponent(IReadOnlyCollection<IMessageComponent> components)
-    {
-        Components = components;
+        internal ActionRowComponent() { }
+
+        internal ActionRowComponent(List<IMessageComponent> components)
+        {
+            Components = components;
+        }
+
+        string IMessageComponent.CustomId => null;
     }
 }

src/Discord.Net.Core/Entities/Interactions/MessageComponents/Builders/ActionRowBuilder.cs

@@ -8,12 +8,8 @@ namespace Discord;
 /// <summary>
 ///     Represents a class used to build Action rows.
 /// </summary>
-public class ActionRowBuilder : IMessageComponentBuilder, IInteractableComponentContainer
+public class ActionRowBuilder
 {
-    public ComponentType Type => ComponentType.ActionRow;
-
-    public int? Id { get; set; }
-
     /// <summary>
     ///     The max amount of child components this row can hold.
     /// </summary>
@@ -24,7 +20,7 @@ public class ActionRowBuilder : IMessageComponentBuilder, IInteractableComponent
     /// </summary>
     /// <exception cref="ArgumentNullException" accessor="set"><see cref="Components"/> cannot be null.</exception>
     /// <exception cref="ArgumentException" accessor="set"><see cref="Components"/> count exceeds <see cref="MaxChildCount"/>.</exception>
-    public List<IMessageComponentBuilder> Components
+    public List<IMessageComponent> Components
     {
         get => _components;
         set
@@ -41,29 +37,15 @@ public List<IMessageComponentBuilder> Components
         }
     }
 
-
-    public ActionRowBuilder AddComponents(params IMessageComponentBuilder[] components)
-    {
-        foreach (var component in components)
-            AddComponent(component);
-        return this;
-    }
-
-    public ActionRowBuilder WithComponents(IEnumerable<IMessageComponentBuilder> components)
-    {
-        Components = components.ToList();
-        return this;
-    }
-
-    private List<IMessageComponentBuilder> _components = new ();
+    private List<IMessageComponent> _components = new List<IMessageComponent>();
 
     /// <summary>
     ///     Adds a list of components to the current row.
     /// </summary>
     /// <param name="components">The list of components to add.</param>
     /// <inheritdoc cref="Components"/>
     /// <returns>The current builder.</returns>
-    public ActionRowBuilder WithComponents(List<IMessageComponentBuilder> components)
+    public ActionRowBuilder WithComponents(List<IMessageComponent> components)
     {
         Components = components;
         return this;
@@ -75,7 +57,7 @@ public ActionRowBuilder WithComponents(List<IMessageComponentBuilder> components
     /// <param name="component">The component to add.</param>
     /// <exception cref="InvalidOperationException">Components count reached <see cref="MaxChildCount"/></exception>
     /// <returns>The current builder.</returns>
-    public ActionRowBuilder AddComponent(IMessageComponentBuilder component)
+    public ActionRowBuilder AddComponent(IMessageComponent component)
     {
         if (Components.Count >= MaxChildCount)
             throw new InvalidOperationException($"Components count reached {MaxChildCount}");
@@ -121,11 +103,13 @@ public ActionRowBuilder WithSelectMenu(SelectMenuBuilder menu)
     {
         if (menu.Options is not null && menu.Options.Distinct().Count() != menu.Options.Count)
             throw new InvalidOperationException("Please make sure that there is no duplicates values.");
-        
+
+        var builtMenu = menu.Build();
+
         if (Components.Count != 0)
             throw new InvalidOperationException($"A Select Menu cannot exist in a pre-occupied ActionRow.");
 
-        AddComponent(menu);
+        AddComponent(builtMenu);
 
         return this;
     }
@@ -168,13 +152,15 @@ public ActionRowBuilder WithButton(
     /// <returns>The current builder.</returns>
     public ActionRowBuilder WithButton(ButtonBuilder button)
     {
+        var builtButton = button.Build();
+
         if (Components.Count >= 5)
             throw new InvalidOperationException($"Components count reached {MaxChildCount}");
 
         if (Components.Any(x => x.Type.IsSelectType()))
             throw new InvalidOperationException($"A button cannot be added to a row with a SelectMenu");
 
-        AddComponent(button);
+        AddComponent(builtButton);
 
         return this;
     }
@@ -185,11 +171,10 @@ public ActionRowBuilder WithButton(ButtonBuilder button)
     /// <returns>A <see cref="ActionRowComponent"/> that can be used within a <see cref="ComponentBuilder"/></returns>
     public ActionRowComponent Build()
     {
-        return new ActionRowComponent(_components.Select(x => x.Build()).ToList());
+        return new ActionRowComponent(_components);
     }
-    IMessageComponent IMessageComponentBuilder.Build() => Build();
 
-    internal bool CanTakeComponent(IMessageComponentBuilder component)
+    internal bool CanTakeComponent(IMessageComponent component)
     {
         switch (component.Type)
         {
@@ -210,11 +195,4 @@ internal bool CanTakeComponent(IMessageComponentBuilder component)
                 return false;
         }
     }
-
-
-    IComponentContainer IComponentContainer.AddComponent(IMessageComponentBuilder component) => AddComponent(component);
-
-    IComponentContainer IComponentContainer.AddComponents(params IMessageComponentBuilder[] components) => AddComponents(components);
-
-    IComponentContainer IComponentContainer.WithComponents(IEnumerable<IMessageComponentBuilder> components) => WithComponents(components);
 }