Extra checks to make sure we store a valid path for last browser location

timothyschoen · timothyschoen · commit e9565647802b · 2025-07-13T15:06:21.000+02:00
diff --git a/Source/Utility/SettingsFile.cpp b/Source/Utility/SettingsFile.cpp
@@ -244,7 +244,7 @@ ValueTree SettingsFile::getTheme(String const& name) const
 
 void SettingsFile::setLastBrowserPathForId(String const& identifier, File& path)
 {
-    if (identifier.isEmpty())
+    if (identifier.isEmpty() || !path.exists() || path.isRoot())
         return;
 
     settingsTree.getChildWithName("LastBrowserPaths").setProperty(identifier, path.getFullPathName(), nullptr);

Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ ValueTree SettingsFile::getTheme(String const& name) const`
`244`	`244`
`245`	`245`	`void SettingsFile::setLastBrowserPathForId(String const& identifier, File& path)`
`246`	`246`	`{`
`247`		`- if (identifier.isEmpty())`
	`247`	`+ if (identifier.isEmpty() \|\| !path.exists() \|\| path.isRoot())`
`248`	`248`	`return;`
`249`	`249`
`250`	`250`	`settingsTree.getChildWithName("LastBrowserPaths").setProperty(identifier, path.getFullPathName(), nullptr);`