liza/Influx
liza/Influx
1
0
Fork 1
Code Issues Pull Requests 1 Releases 23 Activity

Added auto enroll feature #1

Open
Asuna wants to merge 4 commits from Asuna/Influx:master into master
pull from: Asuna/Influx:master
merge into: liza:master
Conversation 0 Commits 4 Files Changed 5 +25 -17
3 changed files with 25 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2f10811afe - Show all commits

18
Influx/InfluxPlugin.cs
View File

@ -85,6 +85,8 @@ internal sealed class InfluxPlugin : IDalamudPlugin
_pluginInterface.UiBuilder.OpenConfigUi += _configurationWindow.Toggle;
_pluginInterface.UiBuilder.OpenMainUi += _statisticsWindow.Toggle;
_condition.ConditionChange += UpdateOnLogout;
_clientState.Login += AutoEnrollCharacter;
}
private Configuration LoadConfig()
@ -204,6 +206,22 @@ internal sealed class InfluxPlugin : IDalamudPlugin
return allSubs;
}
private void AutoEnrollCharacter()
{
if (_configuration.AutoEnrollCharacters)
{
_configuration.IncludedCharacters.Add(new Configuration.CharacterInfo
{
LocalContentId = _clientState.LocalContentId,
CachedPlayerName = _clientState.LocalPlayer?.Name.ToString() ?? "??",
CachedWorldName = _clientState.LocalPlayer?.HomeWorld.Value.Name.ToString(),
});
_pluginInterface.SavePluginConfig(_configuration);
}
}
private void UpdateOnLogout(ConditionFlag flag, bool value)
{
if (flag == ConditionFlag.LoggingOut && value)

2
Influx/LocalStatistics/LocalStatsCalculator.cs
View File

@ -243,6 +243,8 @@ internal sealed class LocalStatsCalculator : IDisposable
}
}
private unsafe List<short> ExtractClassJobLevels(PlayerState* playerState)
{
List<short> levels = new();

22
Influx/Windows/ConfigurationWindow.cs
View File

@ -104,9 +104,10 @@ internal sealed class ConfigurationWindow : Window
return;
var refIncludeAll = _configuration.AutoEnrollCharacters;
if (ImGui.Checkbox("Auto enroll characters", ref refIncludeAll))
if (ImGui.Checkbox("Auto enroll characters on login", ref refIncludeAll))
{
_configuration.AutoEnrollCharacters = refIncludeAll;
liza commented 2025-01-06 00:36:27 +00:00
Review
Copy Link

This is probably missing a Save(true); if I'm not mistaken.

This is probably missing a `Save(true);` if I'm not mistaken.
Asuna commented 2025-01-06 17:46:37 +00:00
Review
Copy Link

Thanks!! Fixed and will repush in a sec

Thanks!! Fixed and will repush in a sec
Save(true);
}
if (_clientState is { IsLoggedIn: true, LocalContentId: > 0, LocalPlayer.HomeWorld.RowId: > 0 })
@ -141,7 +142,9 @@ internal sealed class ConfigurationWindow : Window
}
else
{
liza commented 2025-01-06 00:37:47 +00:00
Outdated
Review
Copy Link

The way this is phrased should probably be running in a IClientState.Login event, and not only when you open the configuration AND this tab in the UI.

The way this is phrased should probably be running in a `IClientState.Login` event, and not only when you open the configuration AND this tab in the UI.
Asuna commented 2025-01-06 17:46:51 +00:00
Outdated
Review
Copy Link

Thanks for the input, i made some modifications and will push in a minute

Thanks for the input, i made some modifications and will push in a minute
if (_configuration.AutoEnrollCharacters)
ImGui.TextColored(ImGuiColors.DalamudRed,
"This character is currently excluded.");
if (ImGui.Button("Include current character"))
{
_configuration.IncludedCharacters.Add(new Configuration.CharacterInfo
{
@ -151,21 +154,6 @@ internal sealed class ConfigurationWindow : Window
});
Save();
}
else
{
ImGui.TextColored(ImGuiColors.DalamudRed,
"This character is currently excluded.");
if (ImGui.Button("Include current character"))
{
_configuration.IncludedCharacters.Add(new Configuration.CharacterInfo
{
LocalContentId = _clientState.LocalContentId,
CachedPlayerName = _clientState.LocalPlayer?.Name.ToString() ?? "??",
CachedWorldName = worldName,
});
Save();
}
}
}
ImGui.Unindent(30);