Make sure sync method is not saved when user click cancel

(reference #118)
Moonshine-IDE · Jan 15, 2025 · f8b3752 · f8b3752
1 parent e6e6a18
commit f8b3752
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/Source/superhuman/components/ConfigPage.hx b/Source/superhuman/components/ConfigPage.hx
@@ -383,7 +383,6 @@ class ConfigPage extends Page {
 
     function _syncMethodCheckTriggered( e:TriggerEvent ) {
         _syncMethodCheck.text = _syncMethodCheck.selected ? SyncMethod.Rsync : SyncMethod.SCP;
-        _server.syncMethod = _syncMethodCheck.selected ? SyncMethod.Rsync : SyncMethod.SCP;
     }
 
     function _inputHostnameChanged( e:Event ) {
@@ -427,7 +426,7 @@ class ConfigPage extends Page {
         // Making sure the event is fired
         var a = _server.roles.value.copy();
         _server.roles.value = a;
-
+        _server.syncMethod = _syncMethodCheck.selected ? SyncMethod.Rsync : SyncMethod.SCP;
         _server.hostname.value = StringTools.trim( _inputHostname.text );
         _server.organization.value = StringTools.trim( _inputOrganization.text );
         var dvv:ProvisionerDefinition = cast _dropdownCoreComponentVersion.selectedItem;
@@ -454,6 +453,13 @@ class ConfigPage extends Page {
 
     }
 
+    override function _cancel( ?e:Dynamic ) {
+        super._cancel(e);
+
+        _syncMethodCheck.text = _server.syncMethod;
+        _syncMethodCheck.selected = _server.syncMethod == SyncMethod.Rsync;
+    }
+
     function _dropdownCoreComponentVersionChangeHandler(event:Event):Void {
         var dvv:ProvisionerDefinition = cast _dropdownCoreComponentVersion.selectedItem;
         _rowSyncMethod.visible = _rowSyncMethod.includeInLayout = dvv.data.version > "0.1.22";