From cb2bc809ef29f349d38c89e202d821e67bb4c947 Mon Sep 17 00:00:00 2001 From: thomascube <thomas@roundcube.net> Date: Tue, 21 Sep 2010 14:47:55 -0400 Subject: [PATCH] Fix db_mode check in insert_id() --- program/steps/settings/save_identity.inc | 42 ++++++++++++++++++++++++++++++------------ 1 files changed, 30 insertions(+), 12 deletions(-) diff --git a/program/steps/settings/save_identity.inc b/program/steps/settings/save_identity.inc index 1eb9a90..6c2cd81 100644 --- a/program/steps/settings/save_identity.inc +++ b/program/steps/settings/save_identity.inc @@ -5,7 +5,7 @@ | program/steps/settings/save_identity.inc | | | | This file is part of the RoundCube Webmail client | - | Copyright (C) 2005-2007, RoundCube Dev. - Switzerland | + | Copyright (C) 2005-2009, RoundCube Dev. - Switzerland | | Licensed under the GNU GPL | | | | PURPOSE: | @@ -19,16 +19,18 @@ */ +define('IDENTITIES_LEVEL', intval($RCMAIL->config->get('identities_level', 0))); + $a_save_cols = array('name', 'email', 'organization', 'reply-to', 'bcc', 'standard', 'signature', 'html_signature'); $a_html_cols = array('signature'); $a_boolean_cols = array('standard', 'html_signature'); $updated = $default_id = false; // check input -if (empty($_POST['_name']) || empty($_POST['_email'])) +if (empty($_POST['_name']) || (empty($_POST['_email']) && IDENTITIES_LEVEL != 1 && IDENTITIES_LEVEL != 3)) { $OUTPUT->show_message('formincomplete', 'warning'); - rcmail_overwrite_action('edit-identitiy'); + rcmail_overwrite_action('edit-identity'); return; } @@ -50,11 +52,19 @@ $save_data[$col] = 0; } +// unset email address if user has no rights to change it +if (IDENTITIES_LEVEL == 1 || IDENTITIES_LEVEL == 3) + unset($save_data['email']); + // update an existing contact if ($_POST['_iid']) { - if ($updated = $USER->update_identity(get_input_value('_iid', RCUBE_INPUT_POST), $save_data)) + $iid = get_input_value('_iid', RCUBE_INPUT_POST); + $plugin = $RCMAIL->plugins->exec_hook('identity_update', array('id' => $iid, 'record' => $save_data)); + $save_data = $plugin['record']; + + if (!$plugin['abort'] && ($updated = $USER->update_identity($iid, $save_data))) { $OUTPUT->show_message('successfullysaved', 'confirmation'); @@ -67,21 +77,27 @@ // ... } } - else if ($DB->is_error()) + else if ($plugin['abort'] || $DB->is_error()) { // show error message - $OUTPUT->show_message('errorsaving', 'error'); - rcmail_overwrite_action('edit-identitiy'); + $OUTPUT->show_message('errorsaving', 'error', null, false); + rcmail_overwrite_action('edit-identity'); return; } } // insert a new identity record -else +else if (IDENTITIES_LEVEL < 2) { - if ($insert_id = $USER->insert_identity($save_data)) + if (IDENTITIES_LEVEL == 1) + $save_data['email'] = $RCMAIL->user->get_username(); + + $plugin = $RCMAIL->plugins->exec_hook('identity_create', array('record' => $save_data)); + $save_data = $plugin['record']; + + if (!$plugin['abort'] && $save_data['email'] && ($insert_id = $USER->insert_identity($save_data))) { - $OUTPUT->show_message('successfullysaved', 'confirmation'); + $OUTPUT->show_message('successfullysaved', 'confirmation', null, false); $_GET['_iid'] = $insert_id; @@ -91,11 +107,13 @@ else { // show error message - $OUTPUT->show_message('errorsaving', 'error'); + $OUTPUT->show_message('errorsaving', 'error', null, false); rcmail_overwrite_action('edit-identity'); return; } } +else + $OUTPUT->show_message('opnotpermitted', 'error'); // mark all other identities as 'not-default' @@ -105,4 +123,4 @@ // go to next step rcmail_overwrite_action('identities'); -?> \ No newline at end of file + -- Gitblit v1.9.1