From acb2f91ed3fc8edf5cf3b908ce58a7f82406f81e Mon Sep 17 00:00:00 2001
From: Florian Schaal <florian@schaal-24.de>
Date: Sat, 13 Feb 2016 13:18:53 +0100
Subject: [PATCH] selected service reconfiguration bug

---
 install/update.php | 39 ++++++++++++++++++++-------------------
 1 file changed, 20 insertions(+), 19 deletions(-)

diff --git a/install/update.php b/install/update.php
index 9b5b9ba7b6..fe08903a89 100644
--- a/install/update.php
+++ b/install/update.php
@@ -321,9 +321,28 @@ if($reconfigure_services_answer == 'yes' || $reconfigure_services_answer == 'sel
 	if($conf['services']['mail']) {
 
 		//** Configure postfix
-		if($inst->reconfigure_app('Postfix', $reconfigure_services_answer)) {
+		if($inst->reconfigure_app('Postfix and IMAP/POP3', $reconfigure_services_answer)) {
 			swriteln('Configuring Postfix');
 			$inst->configure_postfix('dont-create-certs');
+
+			if($conf['dovecot']['installed'] == true) {
+			//* Configure dovecot
+			swriteln('Configuring Dovecot');
+			$inst->configure_dovecot();
+			} elseif ($conf['courier']['installed'] == true) {
+				//** Configure saslauthd
+				swriteln('Configuring SASL');
+				$inst->configure_saslauthd();
+
+				//** Configure PAM
+				swriteln('Configuring PAM');
+				$inst->configure_pam();
+
+				//* Configure courier
+				swriteln('Configuring Courier');
+				$inst->configure_courier();
+			}
+
 		}
 
 		//** Configure mailman
@@ -332,24 +351,6 @@ if($reconfigure_services_answer == 'yes' || $reconfigure_services_answer == 'sel
 			$inst->configure_mailman('update');
 		}
 
-		if($conf['dovecot']['installed'] == true && $inst->reconfigure_app('Dovecot', $reconfigure_services_answer)) {
-			//* Configure dovecot
-			swriteln('Configuring Dovecot');
-			$inst->configure_dovecot();
-		} elseif ($conf['courier']['installed'] == true && $inst->reconfigure_app('Courier', $reconfigure_services_answer)) {
-			//** Configure saslauthd
-			swriteln('Configuring SASL');
-			$inst->configure_saslauthd();
-
-			//** Configure PAM
-			swriteln('Configuring PAM');
-			$inst->configure_pam();
-
-			//* Configure courier
-			swriteln('Configuring Courier');
-			$inst->configure_courier();
-		}
-
 		//** Configure Spamasassin
 		if($inst->reconfigure_app('Spamassassin', $reconfigure_services_answer)) {
 			swriteln('Configuring Spamassassin');
-- 
GitLab