PHP Fatal error: In read_string: expected config create_mailbox_subdirs_hostport to be a string

[tempixx]

The config file parameter 'create_mailbox_subdirs_hostport' must be corrected:
// Normally, the TCP port number does not have to be specified.
// $CONF['create_mailbox_subdirs_hostport']=143;
Until 3.2 was fine, since 3.3 must be:
// $CONF['create_mailbox_subdirs_hostport']='143';

PHP Fatal error: In read_string: expected config create_mailbox_subdirs_hostport to be a string, but received a integer in /var/www/localhost/htdocs/postfixadmin-3.3.4/model/Config.php on line 70,
referer: https://XXX/postfixadmin/public/edit.php?table=mailbox

[DavidGoodwin]

Would this fix it for you ?

diff --git a/model/MailboxHandler.php b/model/MailboxHandler.php
index 4608247a..0fea63e8 100644
--- a/model/MailboxHandler.php
+++ b/model/MailboxHandler.php
@@ -702,14 +702,16 @@ class MailboxHandler extends PFAHandler {
         }
 
         $s_port='';
-        $create_mailbox_subdirs_hostport = Config::read_string('create_mailbox_subdirs_hostport');
-        if (!empty($create_mailbox_subdirs_hostport)) {
-            $s_port = $create_mailbox_subdirs_hostport;
-            if (intval($s_port)!=$s_port) {
-                trigger_error('The $CONF["create_mailbox_subdirs_hostport"] parameter must be an integer', E_USER_ERROR);
-                return false;
+        if (Config::has('create_mailbox_subdirs_hostport')) {
+            $create_mailbox_subdirs_hostport = Config::read('create_mailbox_subdirs_hostport');
+            if (!empty($create_mailbox_subdirs_hostport)) {
+                $s_port = $create_mailbox_subdirs_hostport;
+                if (intval($s_port)!=$s_port) {
+                    trigger_error('The $CONF["create_mailbox_subdirs_hostport"] parameter must be an integer', E_USER_ERROR);
+                    return false;
+                }
+                $s_port=':'.$s_port;
             }
-            $s_port=':'.$s_port;
         }
 
         $s='{'.$s_host.$s_port.$s_options.'}';

[DavidGoodwin]

fix merged. Should be part of 3.3.5 ....