Skip to content

Fix re-parse the config. #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
MatthewNeko wants to merge 1 commit into from
Closed

Fix re-parse the config. #81

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 4 additions & 5 deletions src/multiaddonmanager.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -966,11 +966,10 @@ KHook::Return<void> MultiAddonManager::Hook_StartupServer_Post(INetworkServerSer
g_pFullFileSystem->RemoveSearchPath("", "GAME");
g_pFullFileSystem->RemoveSearchPath("", "DEFAULT_WRITE_PATH");

// This has to be done here to replicate the behavior on clients, where they mount addons in the string order
// So if the current map is ID 1 and extra addons are IDs 2 and 3, they would be mounted in that order with ID 3 at the top
// Note that the actual map VPK(s) and any sub-maps like team_select will be even higher, but those usually don't contain any assets that concern us
if (!ParseCfg())
RefreshAddons();
// Don't re-parse the config here: the cvar callbacks replace the addon lists wholesale,
// which would drop addons that other plugins added through the API since Load()
// (e.g. AddAddon() in AllPluginsLoaded). The config was already applied in Load().
RefreshAddons();

return {KHook::Action::Ignore};
}
Expand Down
Loading