From f3b1952ff8747a6eef1aa6643e8235697a890b21 Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Tue, 14 Jan 2020 15:06:48 +0100
Subject: [sysconfig] AD/LDAP: Default SSL, no 's' prefix

---
 modules-available/sysconfig/addmodule_adauth.inc.php   | 8 +++++---
 modules-available/sysconfig/addmodule_ldapauth.inc.php | 8 +++++---
 modules-available/sysconfig/lang/de/template-tags.json | 2 +-
 modules-available/sysconfig/lang/en/template-tags.json | 2 +-
 4 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/modules-available/sysconfig/addmodule_adauth.inc.php b/modules-available/sysconfig/addmodule_adauth.inc.php
index 80b7cff1..83f4e289 100644
--- a/modules-available/sysconfig/addmodule_adauth.inc.php
+++ b/modules-available/sysconfig/addmodule_adauth.inc.php
@@ -19,9 +19,11 @@ class AdAuth_Start extends AddModule_Base
 			moduleToArray($this->edit, $data, $ADAUTH_COMMON_FIELDS);
 			$data['title'] = $this->edit->title();
 			$data['edit'] = $this->edit->id();
-		}
-		if ($data['fixnumeric'] === false) {
-			$data['fixnumeric'] = 's';
+			if (!isset($data['fixnumeric']) || $data['fixnumeric'] === false) {
+				$data['fixnumeric'] = 's';
+			}
+		} else {
+			$data['ssl'] = true;
 		}
 		postToArray($data, $ADAUTH_COMMON_FIELDS, true);
 		$obdn = Request::post('originalbinddn');
diff --git a/modules-available/sysconfig/addmodule_ldapauth.inc.php b/modules-available/sysconfig/addmodule_ldapauth.inc.php
index 6d612e9e..001353d0 100644
--- a/modules-available/sysconfig/addmodule_ldapauth.inc.php
+++ b/modules-available/sysconfig/addmodule_ldapauth.inc.php
@@ -15,9 +15,11 @@ class LdapAuth_Start extends AddModule_Base
 			moduleToArray($this->edit, $data, $LDAPAUTH_COMMON_FIELDS);
 			$data['title'] = $this->edit->title();
 			$data['edit'] = $this->edit->id();
-		}
-		if (!isset($data['fixnumeric']) || $data['fixnumeric'] === false) {
-			$data['fixnumeric'] = 's';
+			if (!isset($data['fixnumeric']) || $data['fixnumeric'] === false) {
+				$data['fixnumeric'] = 's';
+			}
+		} else {
+			$data['ssl'] = true;
 		}
 		postToArray($data, $LDAPAUTH_COMMON_FIELDS, true);
 		if (isset($data['server']) && preg_match('/^(.*)\:(636|389)$/', $data['server'], $out)) {
diff --git a/modules-available/sysconfig/lang/de/template-tags.json b/modules-available/sysconfig/lang/de/template-tags.json
index 1d0fcc6a..1f6c0bdd 100644
--- a/modules-available/sysconfig/lang/de/template-tags.json
+++ b/modules-available/sysconfig/lang/de/template-tags.json
@@ -120,7 +120,7 @@
     "lang_skip": "Weiter",
     "lang_sshMultipleHeadsup": "Wenn Sie mehrere Pubkeys angeben wollen, k\u00f6nnen Sie entsprechend mehrere Module vom Typ SSH zu einer Systemkonfiguration hinzuf\u00fcgen. In diesem Fall sollten Sie jedoch darauf achten, dass alle Konfigurationen die gleichen Einstellungen f\u00fcr Port und Passwortlogin haben, da ansonsten undefiniert ist, welche der Einstellungen greifen wird.",
     "lang_ssl": "SSL",
-    "lang_sslDescription": "Die Verbindung zum AD-Server mit SSL sichern. (Die Verbindung zwischen Client und Proxy wird in jedem Fall mit SSL abgewickelt.)",
+    "lang_sslDescription": "Die Verbindung zum AD\/LDAP-Server mit SSL sichern. (Die Verbindung zwischen Client und Proxy wird in jedem Fall mit SSL abgewickelt.)",
     "lang_supportedFiles": "Unterst\u00fctzte Archivformate",
     "lang_systemConfiguration": "Systemkonfiguration",
     "lang_systemConfigurationAlert": "Bevor Sie eine Systemkonfiguration erstellen k\u00f6nnen, m\u00fcssen Sie zun\u00e4chst ein Konfigurationsmodul erzeugen.",
diff --git a/modules-available/sysconfig/lang/en/template-tags.json b/modules-available/sysconfig/lang/en/template-tags.json
index 18c32fdb..0e6d0223 100644
--- a/modules-available/sysconfig/lang/en/template-tags.json
+++ b/modules-available/sysconfig/lang/en/template-tags.json
@@ -120,7 +120,7 @@
     "lang_skip": "Next",
     "lang_sshMultipleHeadsup": "If you want to add multiple SSH keys to the system, you can simply create multiple modules of type SSH and add them all to your system config. Note that you should have matching port and password settings for those configs, as it is currently undefined which of the settings will apply to the final configuration.",
     "lang_ssl": "SSL",
-    "lang_sslDescription": "Use SSL encryption to talk to AD server.",
+    "lang_sslDescription": "Use SSL encryption to talk to AD\/LDAP server.",
     "lang_supportedFiles": "Supported File Formats",
     "lang_systemConfiguration": "System Configuration",
     "lang_systemConfigurationAlert": "Before you can create a system configuration, you must first create a configuration module.",
-- 
cgit v1.2.3-55-g7522