Skip to content

Commit 2f007a0

Browse files
committed
Add more settings to const strings
1 parent 79ab8c7 commit 2f007a0

18 files changed

Lines changed: 83 additions & 76 deletions

File tree

src/UniGetUI.Core.Settings.Tests/SettingsTest.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@ public void Dispose()
5454
[Fact]
5555
public void TestSettingsSaveToNewDirectory()
5656
{
57-
Settings.Set("FreshBoolSetting", true);
58-
Settings.SetValue("FreshValue", "test");
57+
Settings.Set(Settings.FreshBoolSetting, true);
58+
Settings.SetValue(Settings.FreshValue, "test");
5959

6060
Assert.True(File.Exists(GetNewSettingPath("FreshBoolSetting")));
6161
Assert.True(File.Exists(GetNewSettingPath("FreshValue")));

src/UniGetUI.Core.Settings/SettingsEngine_Names.cs

Lines changed: 25 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ public static partial class Settings
88
public const string IconDataBaseURL = "IconDataBaseURL";
99
public const string DisableLangAutoUpdater = "DisableLangAutoUpdater";
1010
public const string DisableDMWThreadOptimizations = "DisableDMWThreadOptimizations";
11-
public const string DisableWidgetsApi = "DisableWidgetsApi";
1211
public const string DisableTelemetry = "DisableTelemetry";
1312
public const string DisableTimeoutOnPackageListingTasks = "DisableTimeoutOnPackageListingTasks";
1413
public const string RemoveAllDesktopShortcuts = "RemoveAllDesktopShortcuts";
@@ -28,7 +27,7 @@ public static partial class Settings
2827
public const string EnableUniGetUIBeta = "EnableUniGetUIBeta";
2928
public const string ShowVersionNumberOnTitlebar = "ShowVersionNumberOnTitlebar";
3029
public const string TransferredOldSettings = "TransferredOldSettings";
31-
public const string /* TODO: NEEDS REVIEW */ DisableApi = "DisableApi";
30+
public const string DisableApi = "DisableApi";
3231
public const string DisableSystemTray = "DisableSystemTray";
3332
public const string MaintainSuccessfulInstalls = "MaintainSuccessfulInstalls";
3433
public const string AutomaticallyUpdatePackages = "AutomaticallyUpdatePackages";
@@ -56,14 +55,29 @@ public static partial class Settings
5655
public const string WindowGeometry = "WindowGeometry";
5756
public const string ChangeBackupFileName = "ChangeBackupFileName";
5857
public const string CurrentSessionToken = "CurrentSessionToken";
59-
/*public const string EnableProxyAuth = "EnableProxyAuth";
60-
public const string EnableProxyAuth = "EnableProxyAuth";
61-
public const string EnableProxyAuth = "EnableProxyAuth";
62-
public const string EnableProxyAuth = "EnableProxyAuth";
63-
public const string EnableProxyAuth = "EnableProxyAuth";
64-
public const string EnableProxyAuth = "EnableProxyAuth";
65-
public const string EnableProxyAuth = "EnableProxyAuth";
66-
public const string EnableProxyAuth = "EnableProxyAuth";
67-
public const string EnableProxyAuth = "EnableProxyAuth";
58+
public const string IgnoredPackageUpdates = "IgnoredPackageUpdates";
59+
public const string DisabledManagers = "DisabledManagers";
60+
public const string DeletableDesktopShortcuts = "DeletableDesktopShortcuts";
61+
public const string WinGetAlreadyUpgradedPackages = "WinGetAlreadyUpgradedPackages";
62+
public const string DependencyManagement = "DependencyManagement";
63+
public const string DisabledPackageManagerNotifications = "DisabledPackageManagerNotifications";
64+
public const string PackageListViewMode = "PackageListViewMode";
65+
public const string DisableInstantSearch = "DisableInstantSearch";
66+
public const string SidepanelWidths = "SidepanelWidths";
67+
public const string HideToggleFilters = "HideToggleFilters";
68+
public const string FreshBoolSetting = "FreshBoolSetting";
69+
public const string FreshValue = "FreshValue";
70+
/*public const string SidepanelWidths = "SidepanelWidths";
71+
public const string SidepanelWidths = "SidepanelWidths";
72+
public const string SidepanelWidths = "SidepanelWidths";
73+
public const string SidepanelWidths = "SidepanelWidths";
74+
public const string SidepanelWidths = "SidepanelWidths";
75+
public const string SidepanelWidths = "SidepanelWidths";
76+
public const string SidepanelWidths = "SidepanelWidths";
77+
public const string SidepanelWidths = "SidepanelWidths";
78+
public const string SidepanelWidths = "SidepanelWidths";
79+
public const string SidepanelWidths = "SidepanelWidths";
80+
public const string SidepanelWidths = "SidepanelWidths";
81+
public const string SidepanelWidths = "SidepanelWidths";
6882
*/
6983
}

src/UniGetUI.Interface.BackgroundApi/BackgroundApi.cs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,13 +43,6 @@ public static bool AuthenticateToken(string? token)
4343

4444
public async Task Start()
4545
{
46-
47-
if (Settings.Get(Settings.DisableWidgetsApi))
48-
{
49-
Logger.Warn("Widgets API is disabled");
50-
return;
51-
}
52-
5346
ApiTokenHolder.Token = CoreTools.RandomString(64);
5447
Settings.SetValue(Settings.CurrentSessionToken, ApiTokenHolder.Token);
5548
Logger.Info("Randomly-generated background API auth token: " + ApiTokenHolder.Token);

src/UniGetUI.PackageEngine.Managers.Chocolatey/Chocolatey.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ protected override ManagerStatus LoadManager()
274274
}
275275

276276
CoreTools.CreateSymbolicLinkDir(old_choco_path, new_choco_path);
277-
Settings.Set("ChocolateySymbolicLinkCreated", true);
277+
Settings.Set(Settings.ChocolateySymbolicLinkCreated, true);
278278
Logger.Info($"Symbolic link created successfully from {old_choco_path} to {new_choco_path}.");
279279

280280
}

src/UniGetUI.PackageEngine.Managers.WinGet/Helpers/WinGetPkgOperationHelper.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -210,11 +210,11 @@ protected override OperationVeredict _getOperationResult(
210210

211211
private static void MarkUpgradeAsDone(IPackage package)
212212
{
213-
Settings.SetDictionaryItem<string, string>("WinGetAlreadyUpgradedPackages", package.Id, package.NewVersionString);
213+
Settings.SetDictionaryItem<string, string>(Settings.WinGetAlreadyUpgradedPackages, package.Id, package.NewVersionString);
214214
}
215215

216216
public static bool UpdateAlreadyInstalled(IPackage package)
217217
{
218-
return Settings.GetDictionaryItem<string, string>("WinGetAlreadyUpgradedPackages", package.Id) == package.NewVersionString;
218+
return Settings.GetDictionaryItem<string, string>(Settings.WinGetAlreadyUpgradedPackages, package.Id) == package.NewVersionString;
219219
}
220220
}

src/UniGetUI.PackageEngine.PackageManagerClasses/Manager/PackageManager.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ public virtual void Initialize()
126126
/// </summary>
127127
public bool IsEnabled()
128128
{
129-
return !Settings.GetDictionaryItem<string, bool>("DisabledManagers", Name);
129+
return !Settings.GetDictionaryItem<string, bool>(Settings.DisabledManagers, Name);
130130
}
131131

132132
/// <summary>

src/UniGetUI.PackageEngine.PackageManagerClasses/Packages/Classes/DesktopShortcutsDatabase.cs

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,12 @@ public enum Status
1616

1717
public static IReadOnlyDictionary<string, bool> GetDatabase()
1818
{
19-
return Settings.GetDictionary<string, bool>("DeletableDesktopShortcuts") ?? new Dictionary<string, bool>();
19+
return Settings.GetDictionary<string, bool>(Settings.DeletableDesktopShortcuts) ?? new Dictionary<string, bool>();
2020
}
2121

2222
public static void ResetDatabase()
2323
{
24-
Settings.ClearDictionary("DeletableDesktopShortcuts");
24+
Settings.ClearDictionary(Settings.DeletableDesktopShortcuts);
2525
}
2626

2727
/// <summary>
@@ -32,9 +32,9 @@ public static void ResetDatabase()
3232
public static void AddToDatabase(string shortcutPath, Status shortcutStatus)
3333
{
3434
if (shortcutStatus is Status.Unknown)
35-
Settings.RemoveDictionaryKey<string, bool>("DeletableDesktopShortcuts", shortcutPath);
35+
Settings.RemoveDictionaryKey<string, bool>(Settings.DeletableDesktopShortcuts, shortcutPath);
3636
else
37-
Settings.SetDictionaryItem<string, bool>("DeletableDesktopShortcuts", shortcutPath, shortcutStatus is Status.Delete);
37+
Settings.SetDictionaryItem<string, bool>(Settings.DeletableDesktopShortcuts, shortcutPath, shortcutStatus is Status.Delete);
3838
}
3939

4040
/// <summary>
@@ -45,10 +45,10 @@ public static void AddToDatabase(string shortcutPath, Status shortcutStatus)
4545
public static bool Remove(string shortcutPath)
4646
{
4747
// Remove the entry if present
48-
if (Settings.DictionaryContainsKey<string, bool>("DeletableDesktopShortcuts", shortcutPath))
48+
if (Settings.DictionaryContainsKey<string, bool>(Settings.DeletableDesktopShortcuts, shortcutPath))
4949
{
5050
// Remove the entry and propagate changes to disk
51-
Settings.SetDictionaryItem("DeletableDesktopShortcuts", shortcutPath, false);
51+
Settings.SetDictionaryItem(Settings.DeletableDesktopShortcuts, shortcutPath, false);
5252
return true;
5353
}
5454

@@ -88,9 +88,9 @@ public static bool DeleteFromDisk(string shortcutPath)
8888
public static Status GetStatus(string shortcutPath)
8989
{
9090
// Check if the package is ignored
91-
if (Settings.DictionaryContainsKey<string, bool>("DeletableDesktopShortcuts", shortcutPath))
91+
if (Settings.DictionaryContainsKey<string, bool>(Settings.DeletableDesktopShortcuts, shortcutPath))
9292
{
93-
bool canDelete = Settings.GetDictionaryItem<string, bool>("DeletableDesktopShortcuts", shortcutPath);
93+
bool canDelete = Settings.GetDictionaryItem<string, bool>(Settings.DeletableDesktopShortcuts, shortcutPath);
9494
return canDelete ? Status.Delete : Status.Maintain;
9595
}
9696

@@ -119,7 +119,7 @@ public static List<string> GetAllShortcuts()
119119
{
120120
var shortcuts = GetShortcutsOnDisk();
121121

122-
foreach (var item in Settings.GetDictionary<string, bool>("DeletableDesktopShortcuts"))
122+
foreach (var item in Settings.GetDictionary<string, bool>(Settings.DeletableDesktopShortcuts))
123123
{
124124
if (!shortcuts.Contains(item.Key))
125125
shortcuts.Add(item.Key);

src/UniGetUI.PackageEngine.PackageManagerClasses/Packages/Classes/IgnoredUpdatesDatabase.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ public string StringRepresentation()
7070

7171
public static IReadOnlyDictionary<string, string> GetDatabase()
7272
{
73-
return Settings.GetDictionary<string, string>("IgnoredPackageUpdates") ?? new Dictionary<string, string>();
73+
return Settings.GetDictionary<string, string>(Settings.IgnoredPackageUpdates) ?? new Dictionary<string, string>();
7474
}
7575

7676
public static string GetIgnoredIdForPackage(IPackage package)
@@ -85,7 +85,7 @@ public static string GetIgnoredIdForPackage(IPackage package)
8585
/// <param name="version">The version to ignore. Use wildcard (`*`) to ignore all versions</param>
8686
public static void Add(string ignoredId, string version = "*")
8787
{
88-
Settings.SetDictionaryItem("IgnoredPackageUpdates", ignoredId, version);
88+
Settings.SetDictionaryItem(Settings.IgnoredPackageUpdates, ignoredId, version);
8989
}
9090

9191
/// <summary>
@@ -96,10 +96,10 @@ public static void Add(string ignoredId, string version = "*")
9696
public static bool Remove(string ignoredId)
9797
{
9898
// Remove the entry if present
99-
if (Settings.DictionaryContainsKey<string, string>("IgnoredPackageUpdates", ignoredId))
99+
if (Settings.DictionaryContainsKey<string, string>(Settings.IgnoredPackageUpdates, ignoredId))
100100
{
101101
// Remove the entry and propagate changes to disk
102-
return Settings.RemoveDictionaryKey<string, string>("IgnoredPackageUpdates", ignoredId) != null;
102+
return Settings.RemoveDictionaryKey<string, string>(Settings.IgnoredPackageUpdates, ignoredId) != null;
103103
}
104104

105105
// Do nothing if the entry was not there
@@ -121,7 +121,7 @@ public static bool Remove(string ignoredId)
121121
/// <returns>True if the package is ignored, false otherwise</returns>
122122
public static bool HasUpdatesIgnored(string ignoredId, string version = "*")
123123
{
124-
string? ignoredVersion = Settings.GetDictionaryItem<string, string>("IgnoredPackageUpdates", ignoredId);
124+
string? ignoredVersion = Settings.GetDictionaryItem<string, string>(Settings.IgnoredPackageUpdates, ignoredId);
125125

126126
if (ignoredVersion != null && ignoredVersion.StartsWith("<"))
127127
{
@@ -151,6 +151,6 @@ public static bool HasUpdatesIgnored(string ignoredId, string version = "*")
151151
/// If **no** versions are ignored, null will be returned.</returns>
152152
public static string? GetIgnoredVersion(string ignoredId)
153153
{
154-
return Settings.GetDictionaryItem<string, string>("IgnoredPackageUpdates", ignoredId);
154+
return Settings.GetDictionaryItem<string, string>(Settings.IgnoredPackageUpdates, ignoredId);
155155
}
156156
}

src/UniGetUI/App.xaml.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -325,7 +325,7 @@ private async Task CheckForMissingDependencies()
325325

326326
if (!isInstalled)
327327
{
328-
if (Settings.GetDictionaryItem<string, string>("DependencyManagement", dependency.Name) == "skipped")
328+
if (Settings.GetDictionaryItem<string, string>(Settings.DependencyManagement, dependency.Name) == "skipped")
329329
{
330330
Logger.Error($"Dependency {dependency.Name} was not found, and the user set it to not be reminded of the missing dependency");
331331
}

src/UniGetUI/MainWindow.xaml.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ private static void TransferOldSettingsFormats()
230230
string SettingName = "Disable" + Manager.Name;
231231
if (Settings.Get(SettingName))
232232
{
233-
Settings.SetDictionaryItem("DisabledManagers", Manager.Name, true);
233+
Settings.SetDictionaryItem(Settings.DisabledManagers, Manager.Name, true);
234234
Settings.Set(SettingName, false);
235235
}
236236
}
@@ -241,22 +241,22 @@ private static void TransferOldSettingsFormats()
241241
{
242242
if (Settings.Get($"HideToggleFilters{Page}Page"))
243243
{
244-
Settings.SetDictionaryItem("HideToggleFilters", Page, true);
244+
Settings.SetDictionaryItem(Settings.HideToggleFilters, Page, true);
245245
Settings.Set($"HideToggleFilters{Page}Page", false);
246246
}
247247

248248
if (Settings.Get($"DisableInstantSearch{Page}Tab"))
249249
{
250-
Settings.SetDictionaryItem("DisableInstantSearch", Page, true);
250+
Settings.SetDictionaryItem(Settings.DisableInstantSearch, Page, true);
251251
Settings.Set($"DisableInstantSearch{Page}Tab", false);
252252
}
253253

254254
if (!int.TryParse(Settings.GetValue($"SidepanelWidth{Page}Page"), out int sidepanelWidth)) sidepanelWidth = 250;
255-
Settings.SetDictionaryItem("SidepanelWidths", Page, sidepanelWidth);
255+
Settings.SetDictionaryItem(Settings.SidepanelWidths, Page, sidepanelWidth);
256256
Settings.Set($"SidepanelWidth{Page}Page", false);
257257
}
258258

259-
Settings.Set("TransferredOldSettings", true);
259+
Settings.Set(Settings.TransferredOldSettings, true);
260260
}
261261
}
262262

@@ -1002,7 +1002,7 @@ private void TitleBar_PaneToggleRequested(WinUIEx.TitleBar sender, object args)
10021002

10031003
if(this.AppWindow.Size.Width >= 1600)
10041004
{
1005-
Settings.Set("CollapseNavMenuOnWideScreen", NavigationPage.NavView.IsPaneOpen);
1005+
Settings.Set(Settings.CollapseNavMenuOnWideScreen, NavigationPage.NavView.IsPaneOpen);
10061006
}
10071007
NavigationPage.NavView.IsPaneOpen = !NavigationPage.NavView.IsPaneOpen;
10081008
}

0 commit comments

Comments
 (0)