Merge remote-tracking branch 'upstream/develop' into issue-3229

diff --cc mod/settings.php
index 74dc8936dda148164bf8980938da24033c66d554,8c3ce668426ed6615ee14a81a6ac8817af71b5ae..be121f6a78e51b197b6b939cb909e73535e769ae
--- 1/mod/settings.php
--- 2/mod/settings.php
+++ b/mod/settings.php
@@@ -534,11 -535,29 +536,14 @@@ function settings_post(App $a
                date_default_timezone_set($timezone);
        }
  
-       $str_group_allow   = !empty($_POST['group_allow'])   ? perms2str($_POST['group_allow'])   : '';
-       $str_contact_allow = !empty($_POST['contact_allow']) ? perms2str($_POST['contact_allow']) : '';
-       $str_group_deny    = !empty($_POST['group_deny'])    ? perms2str($_POST['group_deny'])    : '';
-       $str_contact_deny  = !empty($_POST['contact_deny'])  ? perms2str($_POST['contact_deny'])  : '';
+       /** @var ACLFormatter $aclFormatter */
+       $aclFormatter = BaseObject::getClass(ACLFormatter::class);
+ 
+       $str_group_allow   = !empty($_POST['group_allow'])   ? $aclFormatter->toString($_POST['group_allow'])   : '';
+       $str_contact_allow = !empty($_POST['contact_allow']) ? $aclFormatter->toString($_POST['contact_allow']) : '';
+       $str_group_deny    = !empty($_POST['group_deny'])    ? $aclFormatter->toString($_POST['group_deny'])    : '';
+       $str_contact_deny  = !empty($_POST['contact_deny'])  ? $aclFormatter->toString($_POST['contact_deny'])  : '';
  
 -      $openidserver = $a->user['openidserver'];
 -      //$openid = Strings::normaliseOpenID($openid);
 -
 -      // If openid has changed or if there's an openid but no openidserver, try and discover it.
 -      if ($openid != $a->user['openid'] || (strlen($openid) && (!strlen($openidserver)))) {
 -              if (Network::isUrlValid($openid)) {
 -                      Logger::log('updating openidserver');
 -                      $open_id_obj = new LightOpenID($a->getHostName());
 -                      $open_id_obj->identity = $openid;
 -                      $openidserver = $open_id_obj->discover($open_id_obj->identity);
 -              } else {
 -                      $openidserver = '';
 -              }
 -      }
 -
        PConfig::set(local_user(), 'expire', 'items', $expire_items);
        PConfig::set(local_user(), 'expire', 'notes', $expire_notes);
        PConfig::set(local_user(), 'expire', 'starred', $expire_starred);