Skip to content

Commit

Permalink
Minor refactor
Browse files Browse the repository at this point in the history
  • Loading branch information
Plenyx committed Feb 1, 2025
1 parent bbc11c0 commit 1571132
Show file tree
Hide file tree
Showing 18 changed files with 24 additions and 26 deletions.
1 change: 1 addition & 0 deletions Forms/FormArcPluginInfo.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ namespace PlenBotLogUploader;

public partial class FormArcPluginInfo : Form
{
// fields
private readonly ArcDpsComponentHelperClass infoItem;

internal FormArcPluginInfo(ArcDpsComponentHelperClass infoItem)
Expand Down
2 changes: 1 addition & 1 deletion Forms/FormArcPluginManager.cs
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@ namespace PlenBotLogUploader;

public partial class FormArcPluginManager : Form
{
// fields
private readonly List<ArcDpsComponent> componentsToUpdate = [];
private readonly HttpClientController httpController = new();
// fields
private readonly FormMain mainLink;
private int gw2Instances;
private bool updateManual;
Expand Down
1 change: 0 additions & 1 deletion Forms/FormCustomName.cs
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ namespace PlenBotLogUploader;

public partial class FormCustomName : Form
{

// constants
private const string generateOAuthUrl = "https://twitchapps.com/tmi/";
// fields
Expand Down
2 changes: 1 addition & 1 deletion Forms/FormDiscordWebhooks.cs
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@ namespace PlenBotLogUploader;

public partial class FormDiscordWebhooks : Form
{
private readonly Dictionary<int, DiscordWebhookData> allWebhooks;
// fields
private readonly Dictionary<int, DiscordWebhookData> allWebhooks;
private readonly FormMain mainLink;
private readonly TableVisibleBorders tableBorders = TableVisibleBorders.HEADER_ONLY;
private readonly CellStyle tableCellCenterAlign = new(CellHorizontalAlignment.Center);
Expand Down
3 changes: 1 addition & 2 deletions Forms/FormEditAleevaIntegration.cs
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,14 @@ namespace PlenBotLogUploader;

public partial class FormEditAleevaIntegration : Form
{
// fields
private readonly FormAleevaIntegrations aleevaLink;
private readonly List<AleevaChannel> aleevaServerChannels = [];
private readonly List<AleevaServer> aleevaServers = [];
private readonly HttpClientController controller;
private readonly AleevaIntegration data;
// fields
private readonly FormMain mainLink;
private string selectedChannel;

private string selectedServer;
private Team selectedTeam = Teams.Teams.All.Count > 0 ? Teams.Teams.All[0] : null;

Expand Down
3 changes: 1 addition & 2 deletions Forms/FormEditBossData.cs
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,8 @@ namespace PlenBotLogUploader;

public partial class FormEditBossData : Form
{
private readonly BossData data;

// fields
private readonly BossData data;
private readonly FormBossData editLink;
private BossType bossTypeSwitch = BossType.None;

Expand Down
2 changes: 1 addition & 1 deletion Forms/FormEditDiscordWebhook.cs
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ namespace PlenBotLogUploader;

public partial class FormEditDiscordWebhook : Form
{
private readonly DiscordWebhookData data;
// fields
private readonly DiscordWebhookData data;
private readonly FormDiscordWebhooks discordPingLink;
private readonly int reservedId;

Expand Down
10 changes: 4 additions & 6 deletions Forms/FormEditDpsReportUserToken.cs
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,12 @@ namespace PlenBotLogUploader;

public partial class FormEditDpsReportUserToken : Form
{

// constants
private const string dpsReportUserTokenUrl = "https://dps.report/getUserToken";
private readonly HttpClientController httpClientController;
private readonly ApplicationSettingsUploadUserToken data;
// fields
private readonly HttpClientController httpClientController;
private readonly FormDpsReportSettings settingsLink;
private ApplicationSettingsUploadUserToken data;

internal FormEditDpsReportUserToken(FormDpsReportSettings settingsLink, HttpClientController httpClientController, ApplicationSettingsUploadUserToken data = null)
{
Expand Down Expand Up @@ -44,13 +43,12 @@ private void FormEditDPSReportUserToken_FormClosing(object sender, FormClosingEv
}
if (data is null)
{
data = new ApplicationSettingsUploadUserToken
ApplicationSettings.Current.Upload.DpsReportUserTokens.Add(new ApplicationSettingsUploadUserToken
{
Active = false,
Name = textBoxName.Text,
UserToken = textBoxUserToken.Text,
};
ApplicationSettings.Current.Upload.DpsReportUserTokens.Add(data);
});
}
else
{
Expand Down
1 change: 1 addition & 0 deletions Forms/FormEditGw2Api.cs
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ namespace PlenBotLogUploader;

public partial class FormEditGw2Api : Form
{
// fields
private readonly FormGw2Api gw2ApiLink;
private readonly HttpClientController httpClientController = new();
private ApplicationSettingsGw2Api data;
Expand Down
2 changes: 1 addition & 1 deletion Forms/FormEditPing.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,9 @@ namespace PlenBotLogUploader;

public partial class FormEditPing : Form
{
// fields
private readonly bool addNew;
private readonly PingConfiguration config;
// fields
private readonly FormPings pingLink;
private readonly int reservedId;

Expand Down
1 change: 0 additions & 1 deletion Forms/FormEditTeamCondition.cs
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ namespace PlenBotLogUploader;

public partial class FormEditTeamCondition : Form
{

internal FormEditTeamCondition(Team teamData, TeamCondition data)
{
Team = teamData;
Expand Down
4 changes: 2 additions & 2 deletions Forms/FormEditTwitchCommand.cs
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@ namespace PlenBotLogUploader.Forms;

public partial class FormEditTwitchCommand : Form
{
// consts
// constants
private const string variableListLink = "https://github.com/Plenyx/PlenBotLogUploader/blob/main/Twitch/TwitchResponseVariables.md";
private readonly TwitchCommand data;
// fields
private readonly TwitchCommand data;
private readonly FormTwitchCommands editLink;

internal FormEditTwitchCommand(FormTwitchCommands editLink, TwitchCommand data)
Expand Down
1 change: 1 addition & 0 deletions Forms/FormGw2Api.cs
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ namespace PlenBotLogUploader;

public partial class FormGw2Api : Form
{
// fields
private readonly FormMain mainLink;

internal FormGw2Api(FormMain mainLink)
Expand Down
3 changes: 3 additions & 0 deletions Forms/FormGw2Bot.cs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,10 @@ namespace PlenBotLogUploader;

public partial class FormGw2Bot : Form
{
// constants
private const string gw2BotApiBaseUrl = "https://api.gw2bot.info/v1";

// fields
private readonly HttpClientController controller = new();
private readonly FormMain mainLink;

Expand Down
5 changes: 2 additions & 3 deletions Forms/FormMain.cs
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,12 @@ namespace PlenBotLogUploader;

public partial class FormMain : Form
{

// constants
private const int minFileSize = 8192;
private const string plenBotVersionFileUrl = "https://raw.githubusercontent.com/Plenyx/PlenBotLogUploader/master/VERSION";
private const string plenBotDownloadName = "PlenBotLogUploader.exe";

// fields
private readonly FormAleevaIntegrations aleevaLink;
private readonly List<string> allSessionLogs = [];
private readonly FormArcPluginManager arcPluginManagerLink;
Expand Down Expand Up @@ -73,8 +74,6 @@ public partial class FormMain : Form
{ "%channel%", "" },
};
private readonly FormTwitchCommands twitchCommandsLink;

// fields
private readonly FormTwitchNameSetup twitchNameLink;
private readonly Dictionary<string, int> uploadFailCounters = [];
private readonly ArcLogsChangeObserver watcher;
Expand Down
4 changes: 1 addition & 3 deletions Forms/FormPings.cs
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,8 @@ namespace PlenBotLogUploader;

public partial class FormPings : Form
{
// properties
private static readonly string PingJsonFileLocation = $@"{ApplicationSettings.LocalDir}\remote_pings.json";

// fields
private static readonly string PingJsonFileLocation = $@"{ApplicationSettings.LocalDir}\remote_pings.json";
private readonly FormMain mainLink;
private int settingsIdsKey;

Expand Down
2 changes: 1 addition & 1 deletion Forms/FormTeams.cs
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ namespace PlenBotLogUploader;

public partial class FormTeams : Form
{
// fields
private readonly Dictionary<int, Team> allTeams;
private readonly Dictionary<int, DiscordWebhookData> allWebhooks = DiscordWebhooks.All;
// fields
private int teamIdsKey;

internal FormTeams()
Expand Down
3 changes: 2 additions & 1 deletion Forms/FormTwitchNameSetup.cs
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,9 @@ namespace PlenBotLogUploader;

public partial class FormTwitchNameSetup : Form
{

// constants
private const string twitchTvSeparator = "twitch.tv/";

// fields
private readonly FormMain mainLink;

Expand Down

0 comments on commit 1571132

Please sign in to comment.