Further code clean up

Author: Avid29

src/API/Discord.API/Models/Enums/Messages/ComponentType.cs

@@ -1,9 +1,5 @@
 // Quarrel © 2022
 
-using System;
-using System.Collections.Generic;
-using System.Text;
-
 namespace Discord.API.Models.Enums.Messages
 {
     public enum ComponentType

src/API/Discord.API/Models/Enums/Messages/MessageFlags.cs

@@ -1,9 +1,5 @@
 // Quarrel © 2022
 
-using System;
-using System.Collections.Generic;
-using System.Text;
-
 namespace Discord.API.Models.Enums.Messages
 {
     public enum MessageFlags

src/Quarrel.Client/Models/Messages/Embeds/Attachment.cs

@@ -5,6 +5,9 @@
 
 namespace Quarrel.Client.Models.Messages.Embeds
 {
+    /// <summary>
+    /// An attachment in a message.
+    /// </summary>
     public class Attachment : SnowflakeItem
     {
         internal Attachment(JsonAttachment jsonAttachment, QuarrelClient context) :
@@ -18,16 +21,34 @@ internal Attachment(JsonAttachment jsonAttachment, QuarrelClient context) :
             Width = jsonAttachment.Width;
         }
 
+        /// <summary>
+        /// Gets the name of the attached file.
+        /// </summary>
         public string Filename { get; }
 
+        /// <summary>
+        /// Gets the size of the attached file.
+        /// </summary>
         public ulong Size { get; }
 
+        /// <summary>
+        /// Gets the url of the attached file.
+        /// </summary>
         public string Url { get; }
 
+        /// <summary>
+        /// Gets the proxy url of the attached file.
+        /// </summary>
         public string ProxyUrl { get; }
 
+        /// <summary>
+        /// Gets the height of the attached file if the file is an image or video.
+        /// </summary>
         public int? Height { get; }
 
+        /// <summary>
+        /// Gets the width of the attached file if the file is an image or video.
+        /// </summary>
         public int? Width { get; }
     }
 }

src/Quarrel.ViewModels/Bindables/Channels/Abstract/BindablePrivateChannel.cs

@@ -1,6 +1,7 @@
 // Quarrel © 2022
 
 using Microsoft.Toolkit.Mvvm.Messaging;
+using Quarrel.Bindables.Abstract;
 using Quarrel.Bindables.Channels.Interfaces;
 using Quarrel.Client.Models.Channels.Abstract;
 using Quarrel.Client.Models.Channels.Interfaces;
@@ -36,6 +37,14 @@ internal BindablePrivateChannel(
         /// <inheritdoc/>
         public IMessageChannel MessageChannel => (IMessageChannel)Channel;
 
+        /// <summary>
+        /// Creates a new instance of a <see cref="BindablePrivateChannel"/> based on the type.
+        /// </summary>
+        /// <param name="messenger">The <see cref="IMessenger"/> to pass to the <see cref="BindableItem"/>.</param>
+        /// <param name="discordService">The <see cref="IDiscordService"/> to pass to the <see cref="BindableItem"/>.</param>
+        /// <param name="localizationService">The <see cref="ILocalizationService"/> to pass to the <see cref="BindableItem"/>.</param>
+        /// <param name="dispatcherService">The <see cref="IDispatcherService"/> to pass to the <see cref="BindableItem"/>.</param>
+        /// <param name="channel">The channel to wrap.</param>
         public static BindablePrivateChannel? Create(
             IMessenger messenger,
             IDiscordService discordService,

src/Quarrel.ViewModels/Bindables/Messages/BindableMessage.cs

@@ -100,6 +100,9 @@ internal BindableMessage(
         /// <inheritdoc/>
         public ulong ChannelId => Message.ChannelId;
 
+        /// <summary>
+        /// Gets the wrapped <see cref="Client.Models.Messages.Message"/>.
+        /// </summary>
         public Message Message
         {
             get => _message;
@@ -110,27 +113,43 @@ public Message Message
             }
         }
 
-        public string Content => Message.Content;
-
+        /// <summary>
+        /// Gets whether or not the message has been deleted.
+        /// </summary>
         public bool IsDeleted
         {
             get => _isDeleted;
-            set => SetProperty(ref _isDeleted, value);
+            private set => SetProperty(ref _isDeleted, value);
         }
 
+        /// <summary>
+        /// Gets the <see cref="IBindableMessageChannel"/> that the message belongs to.
+        /// </summary>
         public IBindableMessageChannel Channel { get; }
 
         /// <summary>
         /// Gets the author of the message as a bindable user.
         /// </summary>
         public BindableUser? Author { get; }
 
+        /// <summary>
+        /// Gets the author of the message as a bindable guild memeber.
+        /// </summary>
         public BindableGuildMember? AuthorMember { get; }
 
+        /// <summary>
+        /// Gets a dictionary of bindable users relavent to 
+        /// </summary>
         public Dictionary<ulong, BindableUser?> Users { get; }
 
+        /// <summary>
+        /// Gets the message attachments.
+        /// </summary>
         public BindableAttachment[] Attachments { get; }
 
+        /// <summary>
+        /// Gets whether or not the message is a continuation.
+        /// </summary>
         public bool IsContinuation => !(
             _previousMessage == null ||
             _message.Type is MessageType.ApplicationCommand or MessageType.ContextMenuCommand || 
@@ -145,19 +164,31 @@ _message.Type is MessageType.ApplicationCommand or MessageType.ContextMenuComman
                  _message.WebhookId != null && _previousMessage.Author?.Username != _message.Author?.Username ||
                  _message.Timestamp.ToUnixTimeMilliseconds() - _previousMessage.Timestamp.ToUnixTimeMilliseconds() >  7 * 60 * 1000)));
 
+        /// <summary>
+        /// Gets whether or not the user can delete the message.
+        /// </summary>
         // TODO: Properly handle deletable condition
         public bool CanDelete => Message.IsOwn;
 
+        /// <summary>
+        /// Gets a command that copies the message id to the clipboard.
+        /// </summary>
         public RelayCommand CopyIdCommand { get; }
 
+        /// <summary>
+        /// Gets a command that copies the message link to the clipboard.
+        /// </summary>
         public RelayCommand CopyLinkCommand { get; }
 
+        /// <summary>
+        /// Gets a command that deletes the message.
+        /// </summary>
         public RelayCommand DeleteCommand { get; }
 
+        /// <inheritdoc/>
         protected virtual void AckUpdate()
         {
             OnPropertyChanged(nameof(Message));
-            OnPropertyChanged(nameof(Content));
         }
     }
 }

src/Quarrel.ViewModels/Bindables/Messages/Embeds/BindableAttachment.cs

@@ -8,6 +8,9 @@
 
 namespace Quarrel.Bindables.Messages.Embeds
 {
+    /// <summary>
+    /// A wrapper of a <see cref="Client.Models.Messages.Embeds.Attachment"/> that can be bound to the UI.
+    /// </summary>
     public class BindableAttachment : BindableItem
     {
         private Attachment _attachment;
@@ -19,13 +22,16 @@ internal BindableAttachment(
             Attachment attachment) :
             base(messenger, discordService, dispatcherService)
         {
-            Attachment = attachment;
+            _attachment = attachment;
         }
 
+        /// <summary>
+        /// Gets the wrapped <see cref="Client.Models.Messages.Embeds.Attachment"/>.
+        /// </summary>
         public Attachment Attachment
         {
             get => _attachment;
-            set => SetProperty(ref _attachment, value);
+            private set => SetProperty(ref _attachment, value);
         }
     }
 }

src/Quarrel.ViewModels/Bindables/Users/BindableGuildMember.cs

@@ -8,6 +8,9 @@
 
 namespace Quarrel.Bindables.Users
 {
+    /// <summary>
+    /// A wrapper of a <see cref="Client.Models.Users.GuildMember"/> that can be bound to the UI.
+    /// </summary>
     public class BindableGuildMember : BindableItem
     {
         private GuildMember _guildMember;
@@ -25,6 +28,9 @@ internal BindableGuildMember(
             _guildMember = guildMember;
         }
 
+        /// <summary>
+        /// Gets the wrapped <see cref="Client.Models.Users.GuildMember"/>.
+        /// </summary>
         public GuildMember GuildMember
         {
             get => _guildMember;

src/Quarrel.ViewModels/Services/Discord/IDiscordService.cs

@@ -59,21 +59,21 @@ public interface IDiscordService
         /// <param name="beforeId">The if of the last message to load messages before, or null.</param>
         /// <returns>An array of <see cref="BindableMessage"/>s from the channel.</returns>
         Task<Message[]> GetChannelMessagesAsync(IBindableMessageChannel channel, ulong? beforeId = null);
-        
+
         /// <summary>
         /// Gets the channels in a guild.
         /// </summary>
         /// <param name="guild">The guild to get the channels for.</param>
         /// <param name="selectedChannel">The selected channel as an <see cref="IBindableSelectableChannel"/>.</param>
-        /// <returns>An array of <see cref="IBindableChannel"/>s from the guild.</returns>
+        /// <returns>An array of <see cref="BindableGuildChannel"/>s from the guild.</returns>
         BindableGuildChannel?[] GetGuildChannels(BindableGuild guild, out IBindableSelectableChannel? selectedChannel);
 
         /// <summary>
         /// Gets the user's direct message channels.
         /// </summary>
         /// <param name="home">The <see cref="BindableHomeItem"/>.</param>
         /// <param name="selectedChannel">The selected channel as an <see cref="IBindableSelectableChannel"/>.</param>
-        /// <returns>An array of <see cref="IBindableChannel"/>s.</returns>
+        /// <returns>An array of <see cref="BindablePrivateChannel"/>s.</returns>
         BindablePrivateChannel?[] GetPrivateChannels(BindableHomeItem home, out IBindableSelectableChannel? selectedChannel);
 
         BindableGuildMember? GetGuildMember(ulong userId, ulong guildId);

src/Quarrel.ViewModels/Services/Localization/ILocalizationService.cs

@@ -29,6 +29,9 @@ public interface ILocalizationService
         /// </summary>
         string CommaList(params string[] args);
 
+        /// <summary>
+        /// Gets or sets the app's language override.
+        /// </summary>
         public string LanguageOverride { get; set; }
 
         /// <summary>

src/Quarrel.ViewModels/ViewModels/Panels/CommandBarViewModel.cs

@@ -9,6 +9,9 @@
 
 namespace Quarrel.ViewModels.Panels
 {
+    /// <summary>
+    /// The view model for the command bar.
+    /// </summary>
     public class CommandBarViewModel : ObservableRecipient
     {
         private readonly IAnalyticsService _analyticsService;
@@ -17,6 +20,9 @@ public class CommandBarViewModel : ObservableRecipient
 
         private IBindableSelectableChannel? _selectedChannel;
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="CommandBarViewModel"/> class.
+        /// </summary>
         public CommandBarViewModel(IAnalyticsService analyticsService, IMessenger messenger, IDiscordService discordService)
         {
             _analyticsService = analyticsService;
@@ -26,10 +32,13 @@ public CommandBarViewModel(IAnalyticsService analyticsService, IMessenger messen
             _messenger.Register<NavigateToChannelMessage<IBindableSelectableChannel>>(this, (_, m) => SelectedChannel = m.Channel);
         }
 
+        /// <summary>
+        /// Gets the selected channel.
+        /// </summary>
         public IBindableSelectableChannel? SelectedChannel
         {
             get => _selectedChannel;
-            set => SetProperty(ref _selectedChannel, value);
+            private set => SetProperty(ref _selectedChannel, value);
         }
     }
 }