Browse Source

feature: optional API calling to RestInteraction (#2281)

* Take 2

* Expose channel & guild Id for manual calling

* Make api calling optional at runtime

* Resolve build errors

* Bind runtime option to interaction type

* Expose methods to get channel & guild from API

* Patch out NRE's, test on all int types
tags/3.7.0
Armano den Boef GitHub 3 years ago
parent
commit
a24dde4b19
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
17 changed files with 212 additions and 79 deletions
  1. +15
    -7
      src/Discord.Net.Rest/DiscordRestClient.cs
  2. +2
    -0
      src/Discord.Net.Rest/DiscordRestConfig.cs
  3. +4
    -4
      src/Discord.Net.Rest/Entities/Interactions/CommandBase/RestCommandBase.cs
  4. +4
    -4
      src/Discord.Net.Rest/Entities/Interactions/CommandBase/RestCommandBaseData.cs
  5. +20
    -6
      src/Discord.Net.Rest/Entities/Interactions/CommandBase/RestResolvableData.cs
  6. +5
    -5
      src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/MessageCommands/RestMessageCommand.cs
  7. +3
    -3
      src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/MessageCommands/RestMessageCommandData.cs
  8. +5
    -5
      src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/UserCommands/RestUserCommand.cs
  9. +2
    -2
      src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/UserCommands/RestUserCommandData.cs
  10. +4
    -4
      src/Discord.Net.Rest/Entities/Interactions/MessageComponents/RestMessageComponent.cs
  11. +2
    -2
      src/Discord.Net.Rest/Entities/Interactions/Modals/RestModal.cs
  12. +125
    -18
      src/Discord.Net.Rest/Entities/Interactions/RestInteraction.cs
  13. +2
    -2
      src/Discord.Net.Rest/Entities/Interactions/RestPingInteraction.cs
  14. +2
    -2
      src/Discord.Net.Rest/Entities/Interactions/SlashCommands/RestAutocompleteInteraction.cs
  15. +5
    -5
      src/Discord.Net.Rest/Entities/Interactions/SlashCommands/RestSlashCommand.cs
  16. +4
    -4
      src/Discord.Net.Rest/Entities/Interactions/SlashCommands/RestSlashCommandData.cs
  17. +8
    -6
      src/Discord.Net.Rest/Entities/Users/RestGuildUser.cs

+ 15
- 7
src/Discord.Net.Rest/DiscordRestClient.cs View File

@@ -32,9 +32,15 @@ namespace Discord.Rest
/// Initializes a new <see cref="DiscordRestClient"/> with the provided configuration.
/// </summary>
/// <param name="config">The configuration to be used with the client.</param>
public DiscordRestClient(DiscordRestConfig config) : base(config, CreateApiClient(config)) { }
public DiscordRestClient(DiscordRestConfig config) : base(config, CreateApiClient(config))
{
_apiOnCreation = config.APIOnRestInteractionCreation;
}
// used for socket client rest access
internal DiscordRestClient(DiscordRestConfig config, API.DiscordRestApiClient api) : base(config, api) { }
internal DiscordRestClient(DiscordRestConfig config, API.DiscordRestApiClient api) : base(config, api)
{
_apiOnCreation = config.APIOnRestInteractionCreation;
}

private static API.DiscordRestApiClient CreateApiClient(DiscordRestConfig config)
=> new API.DiscordRestApiClient(config.RestClientProvider, DiscordRestConfig.UserAgent, serializer: Serializer, useSystemClock: config.UseSystemClock, defaultRatelimitCallback: config.DefaultRatelimitCallback);
@@ -82,6 +88,8 @@ namespace Discord.Rest

#region Rest interactions

private readonly bool _apiOnCreation;

public bool IsValidHttpInteraction(string publicKey, string signature, string timestamp, string body)
=> IsValidHttpInteraction(publicKey, signature, timestamp, Encoding.UTF8.GetBytes(body));
public bool IsValidHttpInteraction(string publicKey, string signature, string timestamp, byte[] body)
@@ -113,8 +121,8 @@ namespace Discord.Rest
/// A <see cref="RestInteraction"/> that represents the incoming http interaction.
/// </returns>
/// <exception cref="BadSignatureException">Thrown when the signature doesn't match the public key.</exception>
public Task<RestInteraction> ParseHttpInteractionAsync(string publicKey, string signature, string timestamp, string body)
=> ParseHttpInteractionAsync(publicKey, signature, timestamp, Encoding.UTF8.GetBytes(body));
public Task<RestInteraction> ParseHttpInteractionAsync(string publicKey, string signature, string timestamp, string body, Func<InteractionType, bool> doApiCallOnCreation = null)
=> ParseHttpInteractionAsync(publicKey, signature, timestamp, Encoding.UTF8.GetBytes(body), doApiCallOnCreation);

/// <summary>
/// Creates a <see cref="RestInteraction"/> from a http message.
@@ -127,7 +135,7 @@ namespace Discord.Rest
/// A <see cref="RestInteraction"/> that represents the incoming http interaction.
/// </returns>
/// <exception cref="BadSignatureException">Thrown when the signature doesn't match the public key.</exception>
public async Task<RestInteraction> ParseHttpInteractionAsync(string publicKey, string signature, string timestamp, byte[] body)
public async Task<RestInteraction> ParseHttpInteractionAsync(string publicKey, string signature, string timestamp, byte[] body, Func<InteractionType, bool> doApiCallOnCreation = null)
{
if (!IsValidHttpInteraction(publicKey, signature, timestamp, body))
{
@@ -138,12 +146,12 @@ namespace Discord.Rest
using (var jsonReader = new JsonTextReader(textReader))
{
var model = Serializer.Deserialize<API.Interaction>(jsonReader);
return await RestInteraction.CreateAsync(this, model);
return await RestInteraction.CreateAsync(this, model, doApiCallOnCreation != null ? doApiCallOnCreation(model.Type) : _apiOnCreation);
}
}

#endregion
public async Task<RestApplication> GetApplicationInfoAsync(RequestOptions options = null)
{
return _applicationInfo ??= await ClientHelper.GetApplicationInfoAsync(this, options).ConfigureAwait(false);


+ 2
- 0
src/Discord.Net.Rest/DiscordRestConfig.cs View File

@@ -9,5 +9,7 @@ namespace Discord.Rest
{
/// <summary> Gets or sets the provider used to generate new REST connections. </summary>
public RestClientProvider RestClientProvider { get; set; } = DefaultRestClientProvider.Instance;

public bool APIOnRestInteractionCreation { get; set; } = true;
}
}

+ 4
- 4
src/Discord.Net.Rest/Entities/Interactions/CommandBase/RestCommandBase.cs View File

@@ -39,16 +39,16 @@ namespace Discord.Rest
{
}

internal new static async Task<RestCommandBase> CreateAsync(DiscordRestClient client, Model model)
internal new static async Task<RestCommandBase> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
var entity = new RestCommandBase(client, model);
await entity.UpdateAsync(client, model).ConfigureAwait(false);
await entity.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);
return entity;
}

internal override async Task UpdateAsync(DiscordRestClient client, Model model)
internal override async Task UpdateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
await base.UpdateAsync(client, model).ConfigureAwait(false);
await base.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);
}

/// <summary>


+ 4
- 4
src/Discord.Net.Rest/Entities/Interactions/CommandBase/RestCommandBaseData.cs View File

@@ -27,20 +27,20 @@ namespace Discord.Rest
{
}

internal static async Task<RestCommandBaseData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel)
internal static async Task<RestCommandBaseData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel, bool doApiCall)
{
var entity = new RestCommandBaseData(client, model);
await entity.UpdateAsync(client, model, guild, channel).ConfigureAwait(false);
await entity.UpdateAsync(client, model, guild, channel, doApiCall).ConfigureAwait(false);
return entity;
}

internal virtual async Task UpdateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel)
internal virtual async Task UpdateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel, bool doApiCall)
{
Name = model.Name;
if (model.Resolved.IsSpecified && ResolvableData == null)
{
ResolvableData = new RestResolvableData<Model>();
await ResolvableData.PopulateAsync(client, guild, channel, model).ConfigureAwait(false);
await ResolvableData.PopulateAsync(client, guild, channel, model, doApiCall).ConfigureAwait(false);
}
}



+ 20
- 6
src/Discord.Net.Rest/Entities/Interactions/CommandBase/RestResolvableData.cs View File

@@ -22,7 +22,7 @@ namespace Discord.Rest
internal readonly Dictionary<ulong, Attachment> Attachments
= new Dictionary<ulong, Attachment>();

internal async Task PopulateAsync(DiscordRestClient discord, RestGuild guild, IRestMessageChannel channel, T model)
internal async Task PopulateAsync(DiscordRestClient discord, RestGuild guild, IRestMessageChannel channel, T model, bool doApiCall)
{
var resolved = model.Resolved.Value;

@@ -38,15 +38,26 @@ namespace Discord.Rest

if (resolved.Channels.IsSpecified)
{
var channels = await guild.GetChannelsAsync().ConfigureAwait(false);
var channels = doApiCall ? await guild.GetChannelsAsync().ConfigureAwait(false) : null;

foreach (var channelModel in resolved.Channels.Value)
{
var restChannel = channels.FirstOrDefault(x => x.Id == channelModel.Value.Id);
if (channels != null)
{
var guildChannel = channels.FirstOrDefault(x => x.Id == channelModel.Value.Id);

restChannel.Update(channelModel.Value);
guildChannel.Update(channelModel.Value);

Channels.Add(ulong.Parse(channelModel.Key), restChannel);
Channels.Add(ulong.Parse(channelModel.Key), guildChannel);
}
else
{
var restChannel = RestChannel.Create(discord, channelModel.Value);

restChannel.Update(channelModel.Value);

Channels.Add(ulong.Parse(channelModel.Key), restChannel);
}
}
}

@@ -76,7 +87,10 @@ namespace Discord.Rest
{
foreach (var msg in resolved.Messages.Value)
{
channel ??= (IRestMessageChannel)(Channels.FirstOrDefault(x => x.Key == msg.Value.ChannelId).Value ?? await discord.GetChannelAsync(msg.Value.ChannelId).ConfigureAwait(false));
channel ??= (IRestMessageChannel)(Channels.FirstOrDefault(x => x.Key == msg.Value.ChannelId).Value
?? (doApiCall
? await discord.GetChannelAsync(msg.Value.ChannelId).ConfigureAwait(false)
: null));

RestUser author;



+ 5
- 5
src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/MessageCommands/RestMessageCommand.cs View File

@@ -20,22 +20,22 @@ namespace Discord.Rest
}

internal new static async Task<RestMessageCommand> CreateAsync(DiscordRestClient client, Model model)
internal new static async Task<RestMessageCommand> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
var entity = new RestMessageCommand(client, model);
await entity.UpdateAsync(client, model).ConfigureAwait(false);
await entity.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);
return entity;
}

internal override async Task UpdateAsync(DiscordRestClient client, Model model)
internal override async Task UpdateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
await base.UpdateAsync(client, model).ConfigureAwait(false);
await base.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);

var dataModel = model.Data.IsSpecified
? (DataModel)model.Data.Value
: null;
Data = await RestMessageCommandData.CreateAsync(client, dataModel, Guild, Channel).ConfigureAwait(false);
Data = await RestMessageCommandData.CreateAsync(client, dataModel, Guild, Channel, doApiCall).ConfigureAwait(false);
}

//IMessageCommandInteraction


+ 3
- 3
src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/MessageCommands/RestMessageCommandData.cs View File

@@ -23,15 +23,15 @@ namespace Discord.Rest
/// <b>Note</b> Not implemented for <see cref="RestMessageCommandData"/>
/// </remarks>
public override IReadOnlyCollection<IApplicationCommandInteractionDataOption> Options
=> throw new System.NotImplementedException();
=> throw new NotImplementedException();

internal RestMessageCommandData(DiscordRestClient client, Model model)
: base(client, model) { }

internal new static async Task<RestMessageCommandData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel)
internal new static async Task<RestMessageCommandData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel, bool doApiCall)
{
var entity = new RestMessageCommandData(client, model);
await entity.UpdateAsync(client, model, guild, channel).ConfigureAwait(false);
await entity.UpdateAsync(client, model, guild, channel, doApiCall).ConfigureAwait(false);
return entity;
}



+ 5
- 5
src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/UserCommands/RestUserCommand.cs View File

@@ -23,22 +23,22 @@ namespace Discord.Rest
{
}

internal new static async Task<RestUserCommand> CreateAsync(DiscordRestClient client, Model model)
internal new static async Task<RestUserCommand> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
var entity = new RestUserCommand(client, model);
await entity.UpdateAsync(client, model).ConfigureAwait(false);
await entity.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);
return entity;
}

internal override async Task UpdateAsync(DiscordRestClient client, Model model)
internal override async Task UpdateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
await base.UpdateAsync(client, model).ConfigureAwait(false);
await base.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);

var dataModel = model.Data.IsSpecified
? (DataModel)model.Data.Value
: null;

Data = await RestUserCommandData.CreateAsync(client, dataModel, Guild, Channel).ConfigureAwait(false);
Data = await RestUserCommandData.CreateAsync(client, dataModel, Guild, Channel, doApiCall).ConfigureAwait(false);
}

//IUserCommandInteractionData


+ 2
- 2
src/Discord.Net.Rest/Entities/Interactions/ContextMenuCommands/UserCommands/RestUserCommandData.cs View File

@@ -26,10 +26,10 @@ namespace Discord.Rest
internal RestUserCommandData(DiscordRestClient client, Model model)
: base(client, model) { }

internal new static async Task<RestUserCommandData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel)
internal new static async Task<RestUserCommandData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel, bool doApiCall)
{
var entity = new RestUserCommandData(client, model);
await entity.UpdateAsync(client, model, guild, channel).ConfigureAwait(false);
await entity.UpdateAsync(client, model, guild, channel, doApiCall).ConfigureAwait(false);
return entity;
}



+ 4
- 4
src/Discord.Net.Rest/Entities/Interactions/MessageComponents/RestMessageComponent.cs View File

@@ -37,15 +37,15 @@ namespace Discord.Rest
Data = new RestMessageComponentData(dataModel);
}

internal new static async Task<RestMessageComponent> CreateAsync(DiscordRestClient client, Model model)
internal new static async Task<RestMessageComponent> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
var entity = new RestMessageComponent(client, model);
await entity.UpdateAsync(client, model).ConfigureAwait(false);
await entity.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);
return entity;
}
internal override async Task UpdateAsync(DiscordRestClient discord, Model model)
internal override async Task UpdateAsync(DiscordRestClient discord, Model model, bool doApiCall)
{
await base.UpdateAsync(discord, model).ConfigureAwait(false);
await base.UpdateAsync(discord, model, doApiCall).ConfigureAwait(false);

if (model.Message.IsSpecified && model.ChannelId.IsSpecified)
{


+ 2
- 2
src/Discord.Net.Rest/Entities/Interactions/Modals/RestModal.cs View File

@@ -26,10 +26,10 @@ namespace Discord.Rest
Data = new RestModalData(dataModel);
}

internal new static async Task<RestModal> CreateAsync(DiscordRestClient client, ModelBase model)
internal new static async Task<RestModal> CreateAsync(DiscordRestClient client, ModelBase model, bool doApiCall)
{
var entity = new RestModal(client, model);
await entity.UpdateAsync(client, model);
await entity.UpdateAsync(client, model, doApiCall);
return entity;
}


+ 125
- 18
src/Discord.Net.Rest/Entities/Interactions/RestInteraction.cs View File

@@ -16,6 +16,10 @@ namespace Discord.Rest
/// </summary>
public abstract class RestInteraction : RestEntity<ulong>, IDiscordInteraction
{
// Added so channel & guild methods don't need a client reference
private Func<RequestOptions, ulong, Task<IRestMessageChannel>> _getChannel = null;
private Func<RequestOptions, ulong, Task<RestGuild>> _getGuild = null;

/// <inheritdoc/>
public InteractionType Type { get; private set; }

@@ -31,6 +35,10 @@ namespace Discord.Rest
/// <summary>
/// Gets the user who invoked the interaction.
/// </summary>
/// <remarks>
/// If this user is an <see cref="RestGuildUser"/> and <see cref="DiscordRestConfig.APIOnRestInteractionCreation"/> is set to false,
/// <see cref="RestGuildUser.Guild"/> will return <see langword="null"/>
/// </remarks>
public RestUser User { get; private set; }

/// <inheritdoc/>
@@ -48,14 +56,38 @@ namespace Discord.Rest
public bool IsValidToken
=> InteractionHelper.CanRespondOrFollowup(this);

/// <summary>
/// Gets the ID of the channel this interaction was executed in.
/// </summary>
/// <remarks>
/// <see langword="null"/> if the interaction was not executed in a guild.
/// </remarks>
public ulong? ChannelId { get; private set; } = null;

/// <summary>
/// Gets the channel that this interaction was executed in.
/// </summary>
/// <remarks>
/// <see langword="null"/> if <see cref="DiscordRestConfig.APIOnRestInteractionCreation"/> is set to false.
/// Call <see cref="GetChannelAsync"/> to set this property and get the interaction channel.
/// </remarks>
public IRestMessageChannel Channel { get; private set; }

/// <summary>
/// Gets the guild this interaction was executed in.
/// Gets the ID of the guild this interaction was executed in if applicable.
/// </summary>
/// <remarks>
/// <see langword="null"/> if the interaction was not executed in a guild.
/// </remarks>
public ulong? GuildId { get; private set; } = null;

/// <summary>
/// Gets the guild this interaction was executed in if applicable.
/// </summary>
/// <remarks>
/// This property will be <see langword="null"/> if <see cref="DiscordRestConfig.APIOnRestInteractionCreation"/> is set to false
/// or if the interaction was not executed in a guild.
/// </remarks>
public RestGuild Guild { get; private set; }

/// <inheritdoc/>
@@ -72,11 +104,11 @@ namespace Discord.Rest
: DateTime.UtcNow;
}

internal static async Task<RestInteraction> CreateAsync(DiscordRestClient client, Model model)
internal static async Task<RestInteraction> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
if(model.Type == InteractionType.Ping)
{
return await RestPingInteraction.CreateAsync(client, model);
return await RestPingInteraction.CreateAsync(client, model, doApiCall);
}

if (model.Type == InteractionType.ApplicationCommand)
@@ -90,26 +122,26 @@ namespace Discord.Rest

return dataModel.Type switch
{
ApplicationCommandType.Slash => await RestSlashCommand.CreateAsync(client, model).ConfigureAwait(false),
ApplicationCommandType.Message => await RestMessageCommand.CreateAsync(client, model).ConfigureAwait(false),
ApplicationCommandType.User => await RestUserCommand.CreateAsync(client, model).ConfigureAwait(false),
ApplicationCommandType.Slash => await RestSlashCommand.CreateAsync(client, model, doApiCall).ConfigureAwait(false),
ApplicationCommandType.Message => await RestMessageCommand.CreateAsync(client, model, doApiCall).ConfigureAwait(false),
ApplicationCommandType.User => await RestUserCommand.CreateAsync(client, model, doApiCall).ConfigureAwait(false),
_ => null
};
}

if (model.Type == InteractionType.MessageComponent)
return await RestMessageComponent.CreateAsync(client, model).ConfigureAwait(false);
return await RestMessageComponent.CreateAsync(client, model, doApiCall).ConfigureAwait(false);

if (model.Type == InteractionType.ApplicationCommandAutocomplete)
return await RestAutocompleteInteraction.CreateAsync(client, model).ConfigureAwait(false);
return await RestAutocompleteInteraction.CreateAsync(client, model, doApiCall).ConfigureAwait(false);

if (model.Type == InteractionType.ModalSubmit)
return await RestModal.CreateAsync(client, model).ConfigureAwait(false);
return await RestModal.CreateAsync(client, model, doApiCall).ConfigureAwait(false);

return null;
}

internal virtual async Task UpdateAsync(DiscordRestClient discord, Model model)
internal virtual async Task UpdateAsync(DiscordRestClient discord, Model model, bool doApiCall)
{
IsDMInteraction = !model.GuildId.IsSpecified;

@@ -120,16 +152,23 @@ namespace Discord.Rest
Version = model.Version;
Type = model.Type;

if(Guild == null && model.GuildId.IsSpecified)
if (Guild == null && model.GuildId.IsSpecified)
{
Guild = await discord.GetGuildAsync(model.GuildId.Value);
GuildId = model.GuildId.Value;
if (doApiCall)
Guild = await discord.GetGuildAsync(model.GuildId.Value);
else
{
Guild = null;
_getGuild = new(async (opt, ul) => await discord.GetGuildAsync(ul, opt));
}
}

if (User == null)
{
if (model.Member.IsSpecified && model.GuildId.IsSpecified)
{
User = RestGuildUser.Create(Discord, Guild, model.Member.Value);
User = RestGuildUser.Create(Discord, Guild, model.Member.Value, (Guild is null) ? model.GuildId.Value : null);
}
else
{
@@ -137,18 +176,33 @@ namespace Discord.Rest
}
}

if(Channel == null && model.ChannelId.IsSpecified)
if (Channel == null && model.ChannelId.IsSpecified)
{
try
{
Channel = (IRestMessageChannel)await discord.GetChannelAsync(model.ChannelId.Value);
ChannelId = model.ChannelId.Value;
if (doApiCall)
Channel = (IRestMessageChannel)await discord.GetChannelAsync(model.ChannelId.Value);
else
{
_getChannel = new(async (opt, ul) =>
{
if (Guild is null)
return (IRestMessageChannel)await discord.GetChannelAsync(ul, opt);
else // get a guild channel if the guild is set.
return (IRestMessageChannel)await Guild.GetChannelAsync(ul, opt);
});

Channel = null;
}
}
catch(HttpException x) when(x.DiscordCode == DiscordErrorCode.MissingPermissions) { } // ignore
catch (HttpException x) when (x.DiscordCode == DiscordErrorCode.MissingPermissions) { } // ignore
}

UserLocale = model.UserLocale.IsSpecified
? model.UserLocale.Value
: null;
? model.UserLocale.Value
: null;

GuildLocale = model.GuildLocale.IsSpecified
? model.GuildLocale.Value
: null;
@@ -164,6 +218,59 @@ namespace Discord.Rest
return json.ToString();
}

/// <summary>
/// Gets the channel this interaction was executed in. Will be a DM channel if the interaction was executed in DM.
/// </summary>
/// <remarks>
/// Calling this method succesfully will populate the <see cref="Channel"/> property.
/// After this, further calls to this method will no longer call the API, and depend on the value set in <see cref="Channel"/>.
/// </remarks>
/// <param name="options">The request options for this <see langword="async"/> request.</param>
/// <returns>A Rest channel to send messages to.</returns>
/// <exception cref="InvalidOperationException">Thrown if no channel can be received.</exception>
public async Task<IRestMessageChannel> GetChannelAsync(RequestOptions options = null)
{
if (IsDMInteraction && Channel is null)
{
var channel = await User.CreateDMChannelAsync(options);
Channel = channel;
}

else if (Channel is null)
{
var channel = await _getChannel(options, ChannelId.Value);

if (channel is null)
throw new InvalidOperationException("The interaction channel was not able to be retrieved.");
Channel = channel;

_getChannel = null; // get rid of it, we don't need it anymore.
}

return Channel;
}

/// <summary>
/// Gets the guild this interaction was executed in if applicable.
/// </summary>
/// <remarks>
/// Calling this method succesfully will populate the <see cref="Guild"/> property.
/// After this, further calls to this method will no longer call the API, and depend on the value set in <see cref="Guild"/>.
/// </remarks>
/// <param name="options">The request options for this <see langword="async"/> request.</param>
/// <returns>The guild this interaction was executed in. <see langword="null"/> if the interaction was executed inside DM.</returns>
public async Task<RestGuild> GetGuildAsync(RequestOptions options)
{
if (IsDMInteraction)
return null;

if (Guild is null)
Guild = await _getGuild(options, GuildId.Value);

_getGuild = null; // get rid of it, we don't need it anymore.
return Guild;
}

/// <inheritdoc/>
public abstract string Defer(bool ephemeral = false, RequestOptions options = null);
/// <summary>


+ 2
- 2
src/Discord.Net.Rest/Entities/Interactions/RestPingInteraction.cs View File

@@ -18,10 +18,10 @@ namespace Discord.Rest
{
}

internal static new async Task<RestPingInteraction> CreateAsync(DiscordRestClient client, Model model)
internal static new async Task<RestPingInteraction> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
var entity = new RestPingInteraction(client, model.Id);
await entity.UpdateAsync(client, model);
await entity.UpdateAsync(client, model, doApiCall);
return entity;
}



+ 2
- 2
src/Discord.Net.Rest/Entities/Interactions/SlashCommands/RestAutocompleteInteraction.cs View File

@@ -32,10 +32,10 @@ namespace Discord.Rest
Data = new RestAutocompleteInteractionData(dataModel);
}

internal new static async Task<RestAutocompleteInteraction> CreateAsync(DiscordRestClient client, Model model)
internal new static async Task<RestAutocompleteInteraction> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
var entity = new RestAutocompleteInteraction(client, model);
await entity.UpdateAsync(client, model).ConfigureAwait(false);
await entity.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);
return entity;
}



+ 5
- 5
src/Discord.Net.Rest/Entities/Interactions/SlashCommands/RestSlashCommand.cs View File

@@ -23,22 +23,22 @@ namespace Discord.Rest
{
}

internal new static async Task<RestSlashCommand> CreateAsync(DiscordRestClient client, Model model)
internal new static async Task<RestSlashCommand> CreateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
var entity = new RestSlashCommand(client, model);
await entity.UpdateAsync(client, model).ConfigureAwait(false);
await entity.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);
return entity;
}

internal override async Task UpdateAsync(DiscordRestClient client, Model model)
internal override async Task UpdateAsync(DiscordRestClient client, Model model, bool doApiCall)
{
await base.UpdateAsync(client, model).ConfigureAwait(false);
await base.UpdateAsync(client, model, doApiCall).ConfigureAwait(false);

var dataModel = model.Data.IsSpecified
? (DataModel)model.Data.Value
: null;

Data = await RestSlashCommandData.CreateAsync(client, dataModel, Guild, Channel).ConfigureAwait(false);
Data = await RestSlashCommandData.CreateAsync(client, dataModel, Guild, Channel, doApiCall).ConfigureAwait(false);
}

//ISlashCommandInteraction


+ 4
- 4
src/Discord.Net.Rest/Entities/Interactions/SlashCommands/RestSlashCommandData.cs View File

@@ -14,15 +14,15 @@ namespace Discord.Rest
internal RestSlashCommandData(DiscordRestClient client, Model model)
: base(client, model) { }

internal static new async Task<RestSlashCommandData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel)
internal static new async Task<RestSlashCommandData> CreateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel, bool doApiCall)
{
var entity = new RestSlashCommandData(client, model);
await entity.UpdateAsync(client, model, guild, channel).ConfigureAwait(false);
await entity.UpdateAsync(client, model, guild, channel, doApiCall).ConfigureAwait(false);
return entity;
}
internal override async Task UpdateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel)
internal override async Task UpdateAsync(DiscordRestClient client, Model model, RestGuild guild, IRestMessageChannel channel, bool doApiCall)
{
await base.UpdateAsync(client, model, guild, channel).ConfigureAwait(false);
await base.UpdateAsync(client, model, guild, channel, doApiCall).ConfigureAwait(false);

Options = model.Options.IsSpecified
? model.Options.Value.Select(x => new RestSlashCommandDataOption(this, x)).ToImmutableArray()


+ 8
- 6
src/Discord.Net.Rest/Entities/Users/RestGuildUser.cs View File

@@ -35,7 +35,7 @@ namespace Discord.Rest
/// <inheritdoc />
public DateTimeOffset? PremiumSince => DateTimeUtils.FromTicks(_premiumSinceTicks);
/// <inheritdoc />
public ulong GuildId => Guild.Id;
public ulong GuildId { get; }
/// <inheritdoc />
public bool? IsPending { get; private set; }
/// <inheritdoc />
@@ -80,14 +80,16 @@ namespace Discord.Rest
/// <inheritdoc />
public DateTimeOffset? JoinedAt => DateTimeUtils.FromTicks(_joinedAtTicks);

internal RestGuildUser(BaseDiscordClient discord, IGuild guild, ulong id)
internal RestGuildUser(BaseDiscordClient discord, IGuild guild, ulong id, ulong? guildId = null)
: base(discord, id)
{
Guild = guild;
if (guild is not null)
Guild = guild;
GuildId = guildId ?? Guild.Id;
}
internal static RestGuildUser Create(BaseDiscordClient discord, IGuild guild, Model model)
internal static RestGuildUser Create(BaseDiscordClient discord, IGuild guild, Model model, ulong? guildId = null)
{
var entity = new RestGuildUser(discord, guild, model.User.Id);
var entity = new RestGuildUser(discord, guild, model.User.Id, guildId);
entity.Update(model);
return entity;
}
@@ -116,7 +118,7 @@ namespace Discord.Rest
private void UpdateRoles(ulong[] roleIds)
{
var roles = ImmutableArray.CreateBuilder<ulong>(roleIds.Length + 1);
roles.Add(Guild.Id);
roles.Add(GuildId);
for (int i = 0; i < roleIds.Length; i++)
roles.Add(roleIds[i]);
_roleIds = roles.ToImmutable();


Loading…
Cancel
Save