fixed additional info can not be save in page OpenVpn settings

amnezia-vpn · Sep 16, 2023 · 9eebee3 · 9eebee3
1 parent c0aca97
commit 9eebee3
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 15 deletions.
diff --git a/client/ui/qml/Controls2/TextAreaType.qml b/client/ui/qml/Controls2/TextAreaType.qml
@@ -6,7 +6,8 @@ Rectangle {
 
     property string placeholderText
     property string text
-    property var onEditingFinished
+    property alias textArea: textArea
+    property alias textAreaText: textArea.text
 
     height: 148
     color: "#1C1D21"
@@ -40,12 +41,6 @@ Rectangle {
             placeholderText: root.placeholderText
             text: root.text
 
-            onEditingFinished: {
-                if (root.onEditingFinished && typeof root.onEditingFinished === "function") {
-                    root.onEditingFinished()
-                }
-            }
-
             wrapMode: Text.Wrap
 
             MouseArea {

diff --git a/client/ui/qml/Pages2/PageProtocolOpenVpnSettings.qml b/client/ui/qml/Pages2/PageProtocolOpenVpnSettings.qml
@@ -312,12 +312,12 @@ PageType {
 
                             visible: additionalClientCommandsSwitcher.checked
 
-                            text: additionalClientCommands
+                            textAreaText: additionalClientCommands
                             placeholderText: qsTr("Commands:")
 
-                            onEditingFinished: {
-                                if (additionalClientCommands !== text) {
-                                    additionalClientCommands = text
+                            textArea.onEditingFinished: {
+                                if (additionalClientCommands !== textAreaText) {
+                                    additionalClientCommands = textAreaText
                                 }
                             }
                         }
@@ -330,6 +330,12 @@ PageType {
                             checked: additionalServerCommands !== ""
 
                             text: qsTr("Additional server configuration commands")
+
+                            onCheckedChanged: {
+                                if (!checked) {
+                                    additionalServerCommands = ""
+                                }
+                            }
                         }
 
                         TextAreaType {
@@ -338,12 +344,12 @@ PageType {
 
                             visible: additionalServerCommandsSwitcher.checked
 
-                            text: additionalServerCommands
+                            textAreaText: additionalServerCommands
                             placeholderText: qsTr("Commands:")
 
-                            onEditingFinished: {
-                                if (additionalServerCommands !== text) {
-                                    additionalServerCommands = text
+                            textArea.onEditingFinished: {
+                                if (additionalServerCommands !== textAreaText) {
+                                    additionalServerCommands = textAreaText
                                 }
                             }
                         }