Skip to content

Commit 1aaba46

Browse files
committed
Use constants & data location for code quality
1 parent f1b5e68 commit 1aaba46

File tree

5 files changed

+8
-6
lines changed

5 files changed

+8
-6
lines changed

Flow.Launcher.Infrastructure/Storage/BinaryStorage.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ namespace Flow.Launcher.Infrastructure.Storage
2121
/// </remarks>
2222
public class BinaryStorage<T>
2323
{
24-
const string DirectoryName = "Cache";
24+
const string DirectoryName = Constant.Cache;
2525

2626
const string FileSuffix = ".cache";
2727

Flow.Launcher.Infrastructure/Storage/JsonStorage.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ namespace Flow.Launcher.Infrastructure.Storage
1616
protected T? Data;
1717

1818
// need a new directory name
19-
public const string DirectoryName = "Settings";
19+
public const string DirectoryName = Constant.Settings;
2020
public const string FileSuffix = ".json";
2121

2222
protected string FilePath { get; init; } = null!;

Flow.Launcher.Infrastructure/Storage/PluginJsonStorage.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ public PluginJsonStorage()
1313
// C# related, add python related below
1414
var dataType = typeof(T);
1515
AssemblyName = dataType.Assembly.GetName().Name;
16-
DirectoryPath = Path.Combine(DataLocation.DataDirectory(), DirectoryName, Constant.Plugins, AssemblyName);
16+
DirectoryPath = Path.Combine(DataLocation.PluginSettingsDirectory, AssemblyName);
1717
Helper.ValidateDirectory(DirectoryPath);
1818

1919
FilePath = Path.Combine(DirectoryPath, $"{dataType.Name}{FileSuffix}");

Flow.Launcher.Infrastructure/UserSettings/DataLocation.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,10 @@ public static bool PortableDataLocationInUse()
2525
return false;
2626
}
2727

28+
public static readonly string SettingsDirectorty = Path.Combine(DataDirectory(), Constant.Settings);
2829
public static readonly string PluginsDirectory = Path.Combine(DataDirectory(), Constant.Plugins);
29-
public static readonly string PluginSettingsDirectory = Path.Combine(DataDirectory(), Constant.Settings, Constant.Plugins);
30+
31+
public static readonly string PluginSettingsDirectory = Path.Combine(SettingsDirectorty, Constant.Plugins);
3032
public static readonly string PluginCacheDirectory = Path.Combine(DataDirectory(), Constant.Cache, Constant.Plugins);
3133

3234
public const string PythonEnvironmentName = "Python";

Flow.Launcher/SettingPages/ViewModels/SettingsPaneAboutViewModel.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,13 +77,13 @@ private void AskClearLogFolderConfirmation()
7777
[RelayCommand]
7878
private void OpenSettingsFolder()
7979
{
80-
PluginManager.API.OpenDirectory(Path.Combine(DataLocation.DataDirectory(), Constant.Settings));
80+
PluginManager.API.OpenDirectory(DataLocation.SettingsDirectorty);
8181
}
8282

8383
[RelayCommand]
8484
private void OpenParentOfSettingsFolder(object parameter)
8585
{
86-
string settingsFolderPath = Path.Combine(DataLocation.DataDirectory(), Constant.Settings);
86+
string settingsFolderPath = Path.Combine(DataLocation.SettingsDirectorty);
8787
string parentFolderPath = Path.GetDirectoryName(settingsFolderPath);
8888
PluginManager.API.OpenDirectory(parentFolderPath);
8989
}

0 commit comments

Comments
 (0)