Got this in a private message on the forums. Thought I'd pass it along:
You and the rest of the team are doing a great job. This might not be the right way to send this information but I have posted about this before with no results, so here it is for You and the rest of the team to do whatever You want with.
My improvements to the identity section. The easy one is about the sort order in the indentity list. Code:
/program/steps/settings/func.inc function rcmail_indentities_list..... // get contacts from DB ORDER BY standard DESC, name ASC, email ASC // this one is new....
The second one is a bit more. It improves the identity section in two ways.
- You can NOT add an identity 'outside' your own domain. That is if
You are logged in as 'mail@gunfro.com' You could only add identities to that domain. 2. You can NOT add an identity if it already exists in the domain. The domain You are logged into.
I made this in one chunk so it easier to add. It might be better to put the code more where it belongs, hope You get my point.
Code:
/program/steps/settings/save_identities.php
// check input if (empty($_POST['_name']) || empty($_POST['_email'])) { $OUTPUT->show_message('formincomplete', 'warning'); rcmail_overwrite_action('edit-identitiy'); return; }
//********************************************************************* // MOD GUNFRO START $newa = explode("@", $_POST['_email']); $curr = explode("@", $_SESSION['username']); // Check valid domain. if ($newa[1] != $curr[1]) { $OUTPUT->show_message('domainnotvalid', 'warning'); rcmail_overwrite_action($_framed ? 'edit-identity' : 'identities'); return; } // Check if changing email and it exits. if ($_POST['_iid']) { $DB->query("SELECT * FROM ".get_table_name('identities')." WHERE identity_id=? AND del<>1", $_POST['_iid']);
$curri = $DB->fetch_assoc(); if ($curri['email'] <> $_POST['_email']) { $DB->query("SELECT * FROM ".get_table_name('identities')." WHERE email=? AND del<>1", $_POST['_email']);
if ($DB->fetch_assoc() != NULL)
{
$OUTPUT->show_message('emailexists', 'warning');
rcmail_overwrite_action($_framed ? 'edit-identity' : 'identities');
return;
}
} } else { // Just new a identity $DB->query("SELECT * FROM ".get_table_name('identities')." WHERE email=? AND del<>1", $_POST['_email']);
if ($DB->fetch_assoc() != NULL) { $OUTPUT->show_message('emailexists', 'warning'); rcmail_overwrite_action($_framed ? 'edit-identity' : 'identities'); return; } }
// MOD GUNFRO END //*********************************************************************
// update an existing contact if ($_POST['_iid'])
That was my contribution for now. Best regards Gunnar Frödin / GunFro