The result of the first query (can) have multiple lines

Treat it as multiline and go over all of them, putting it in
email_list before doing the rest of the work.

That way we actually get any identities synced.
pull/1/head
Joerg Jaspert 5 years ago
parent ceeed409a5
commit bf344d86bf

@ -48,17 +48,17 @@ class postfixadmin_user_identities extends rcube_plugin
$qh = $this->db->query($qrystr); $qh = $this->db->query($qrystr);
$result = $this->db->fetch_array($qh); $result = $this->db->fetch_array($qh);
if ($result !== FALSE)
{
$hidden_domains = $this->rc->config->get('postfixadmin_user_identities_hide_domains');
// Set full user name $email_list = array();
$hidden_domains = $this->rc->config->get('postfixadmin_user_identities_hide_domains');
while ($result !== FALSE)
{
$args['user_name'] = $result[0]; $args['user_name'] = $result[0];
$email_list = array();
$email_list[] = array($result[1], $result[2]); $email_list[] = array($result[1], $result[2]);
$result = $this->db->fetch_array($qh);
}
if (count($email_list) > 0)
{
// Fetch alias domains // Fetch alias domains
$t_adom = $this->db->quote_identifier($this->rc->config->get('postfixadmin_user_identities_table_aliasdomain')); $t_adom = $this->db->quote_identifier($this->rc->config->get('postfixadmin_user_identities_table_aliasdomain'));
$c_adom_source = $t_adom . '.' . $this->db->quote_identifier($this->rc->config->get('postfixadmin_user_identities_col_aliasdomain')); $c_adom_source = $t_adom . '.' . $this->db->quote_identifier($this->rc->config->get('postfixadmin_user_identities_col_aliasdomain'));
@ -297,4 +297,4 @@ class postfixadmin_user_identities extends rcube_plugin
return $args; return $args;
} }
} }

Loading…
Cancel
Save