From ba8fbdb9164483d94a343c5eacc80c273a4a1708 Mon Sep 17 00:00:00 2001 From: Joe4evr Date: Thu, 26 Jan 2017 20:02:24 +0100 Subject: [PATCH] Revert back to auto property, fire method at end of SetContext(). --- src/Discord.Net.Commands/ModuleBase.cs | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/src/Discord.Net.Commands/ModuleBase.cs b/src/Discord.Net.Commands/ModuleBase.cs index 808a26caa..9be7d1aeb 100644 --- a/src/Discord.Net.Commands/ModuleBase.cs +++ b/src/Discord.Net.Commands/ModuleBase.cs @@ -8,23 +8,14 @@ namespace Discord.Commands public abstract class ModuleBase : IModuleBase where T : class, ICommandContext { - public T Context - { - get { return _context; } - private set - { - _context = value; - ContextSet(); - } - } - private T _context; + public T Context { get; private set; } protected virtual async Task ReplyAsync(string message, bool isTTS = false, Embed embed = null, RequestOptions options = null) { return await Context.Channel.SendMessageAsync(message, isTTS, embed, options).ConfigureAwait(false); } - protected virtual void ContextSet() + protected virtual void OnContextSet() { } @@ -35,6 +26,7 @@ namespace Discord.Commands if (newValue == null) throw new InvalidOperationException($"Invalid context type. Expected {typeof(T).Name}, got {context.GetType().Name}"); Context = newValue; + OnContextSet(); } } }