CP-13374: SetSslLegacyAction must be last because it will disrupt the connection

This commit is contained in:
Stephen Turner 2015-08-19 09:35:51 +01:00
parent 59a432ff40
commit 02aeb0b6d5

View File

@ -420,6 +420,7 @@ namespace XenAdmin.Dialogs
private List<AsyncAction> SaveSettings()
{
List<AsyncAction> actions = new List<AsyncAction>();
AsyncAction finalAction = null;
foreach (IEditPage editPage in verticalTabs.Items)
{
@ -430,9 +431,14 @@ namespace XenAdmin.Dialogs
if (action == null)
continue;
actions.Add(action);
if (action is SetSslLegacyAction)
finalAction = action; // annoying special case: SetSslLegacyAction must be last because it will disrupt the connection and we may lose later actions
else
actions.Add(action);
}
if (finalAction != null)
actions.Add(finalAction);
return actions;
}