UI: Fix crash when creating scene collections with "unsafe" names

Scene collection names that are not considered "safe" by OBS Studio
get a changed JSON file name with incompatible characters replaced.

The refactored scene collection implementation uses the Load function to
either activate an existing scene collection or create a new one if it
does not exist.

The Load function however overwrote the scene collection name set in
the profile with its own variant based off the "safe" file name, which
created a mismatch with the code that created the collection data
model.

As the Load function is only called by ActivateSceneCollection (which
itself already sets the name and filename for the collection), removal
of this superfluous code in the Load function also fixes the issue.
This commit is contained in:
PatTheMav
2024-09-18 23:08:34 +02:00
committed by Ryan Foster
parent 6fed2c2bdd
commit b4137fa65a

View File

@@ -1246,10 +1246,6 @@ void OBSBasic::Load(const char *file, bool remigrate)
}
}
config_set_string(App()->GetUserConfig(), "Basic",
"SceneCollection", name.c_str());
config_set_string(App()->GetUserConfig(), "Basic",
"SceneCollectionFile", name.c_str());
blog(LOG_INFO, "No scene file found, creating default scene");
bool hasFirstRun = config_get_bool(App()->GetUserConfig(),