MM-11905: deactivate plugins before unregistering (#9666)

Plugins that listen for config changes might incorrectly re-register
their commands before being /actually/ deactivated, leaving the new
commands alive thereafter.
parent 5bd38ad2
......@@ -120,9 +120,8 @@ func (a *App) removePlugin(id string) *model.AppError {
a.Publish(message)
}
a.UnregisterPluginCommands(id)
a.Plugins.Deactivate(id)
a.UnregisterPluginCommands(id)
err = os.RemoveAll(pluginPath)
if err != nil {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment