From e84b24969cfd107c0d6fdb7807d9184815f45a4a Mon Sep 17 00:00:00 2001 From: pdontthink Date: Mon, 6 Apr 2009 19:06:45 +0000 Subject: [PATCH] Make default $sent_subfolders_base whatever $sent_folder is Update folder settings and auto-create needed subfolder as soon as settings are changed Update $sent_folder in user prefs when $sent_subfolders_base is not "Sent" folder git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@13531 7612ce4b-ef26-0410-bec9-ea0150e637f0 --- plugins/sent_subfolders/functions.php | 46 +++++++++++++++++++-------- plugins/sent_subfolders/setup.php | 5 +-- 2 files changed, 36 insertions(+), 15 deletions(-) diff --git a/plugins/sent_subfolders/functions.php b/plugins/sent_subfolders/functions.php index d9d44854..7ff8deb1 100644 --- a/plugins/sent_subfolders/functions.php +++ b/plugins/sent_subfolders/functions.php @@ -20,7 +20,7 @@ define('SMOPT_GRP_SENT_SUBFOLDERS','SENT_SUBFOLDERS'); function sent_subfolders_check_handleAsSent_do($mailbox) { - global $handleAsSent_result, $data_dir, $username; + global $handleAsSent_result, $data_dir, $username, $sent_folder; // don't need to bother if it's already special if ($handleAsSent_result) return; @@ -30,7 +30,7 @@ function sent_subfolders_check_handleAsSent_do($mailbox) { $use_sent_subfolders = getPref($data_dir, $username, 'use_sent_subfolders', SMPREF_OFF); $sent_subfolders_base = getPref($data_dir, $username, - 'sent_subfolders_base', SMPREF_NONE); + 'sent_subfolders_base', $sent_folder); /* Only check the folder string if we have been passed a mailbox. */ if ($use_sent_subfolders && !empty($mailbox)) { @@ -50,7 +50,7 @@ function sent_subfolders_check_handleAsSent_do($mailbox) { function sent_subfolders_optpage_loadhook_folders_do() { global $data_dir, $username, $optpage_data, $imapServerAddress, - $imapPort, $show_contain_subfolders_option; + $imapPort, $show_contain_subfolders_option, $sent_folder; /* Get some imap data we need later. */ $imapConnection = sqimap_login($username, false, $imapServerAddress, $imapPort, 0); @@ -82,14 +82,15 @@ function sent_subfolders_optpage_loadhook_folders_do() { global $sent_subfolders_base; $sent_subfolders_base = getPref($data_dir, $username, - 'sent_subfolders_base', SMPREF_NONE); + 'sent_subfolders_base', $sent_folder); $optvals[] = array( 'name' => 'sent_subfolders_base', 'caption' => _("Base Sent Folder"), 'type' => SMOPT_TYPE_FLDRLIST, 'refresh' => SMOPT_REFRESH_FOLDERLIST, 'posvals' => $sent_subfolders_base_values, - 'folder_filter' => 'noinferiors' + 'folder_filter' => 'noinferiors', + 'save' => 'save_option_sent_subfolders_base' ); if ($show_contain_subfolders_option) { @@ -136,6 +137,17 @@ function save_option_sent_subfolders_setting($option) { save_option($option); } +/** + * Update the folder settings/auto-create new subfolder + */ +function save_option_sent_subfolders_base($option) { + // first save the option as normal + save_option($option); + + // now update folder settings and auto-create first subfolder if needed + sent_subfolders_update_sentfolder_do(); +} + /** * Update sent_subfolders settings * @@ -155,7 +167,7 @@ function sent_subfolders_update_sentfolder_do() { 'sent_subfolders_setting', SMPREF_SENT_SUBFOLDERS_DISABLED); $sent_subfolders_base = getPref($data_dir, $username, - 'sent_subfolders_base', SMPREF_NONE); + 'sent_subfolders_base', $sent_folder); if ($use_sent_subfolders || $move_to_sent) { $year = date('Y'); @@ -213,7 +225,7 @@ function sent_subfolders_update_sentfolder_do() { if (sqimap_mailbox_is_noselect($ic,$sent_subfolder,$boxes) || ($level==2 && sqimap_mailbox_is_noinferiors($ic,$year_folder,$boxes)) || sqimap_mailbox_is_noinferiors($ic,$sent_subfolders_base,$boxes)) { - error_box(_("Sent Subfolders plugin is misconfigured.")); + error_box(_("Sent subfolders options are misconfigured.")); } else { if ($level==2) { /* Auto-create the year folder, if it does not yet exist. */ @@ -232,9 +244,18 @@ function sent_subfolders_update_sentfolder_do() { } else if (!sqimap_mailbox_is_subscribed($ic, $sent_subfolder)) { sqimap_subscribe($ic, $sent_subfolder); } - /* Update sent_folder setting. */ - //setPref($data_dir, $username, 'sent_folder', $sent_subfolder); - //setPref($data_dir, $username, 'move_to_sent', SMPREF_ON); + /* Update sent_folder setting in prefs only if the base + subfolders setting is not the same as the normal sent + folder... otherwise, it is quite misleading to the user. + If the sent folder is the same as the subfolders base, it's + OK to leave the sent folder as is. + The sent_folder setting itself needs to be the actual + subfolder (not the base) for proper functionality */ + if ($sent_subfolders_base != $sent_folder) { + setPref($data_dir, $username, 'sent_folder', $sent_subfolders_base); + setPref($data_dir, $username, 'move_to_sent', SMPREF_ON); + setPref($data_dir, $username, 'translate_special_folders', SMPREF_OFF); + } $sent_folder = $sent_subfolder; $move_to_sent = SMPREF_ON; } @@ -289,14 +310,13 @@ function sent_subfolder_getQuarter($month) { * @return boolean 1 - is part of sent_subfolders, 0 - is not part of sent_subfolders */ function sent_subfolders_special_mailbox_do($mb) { - global $data_dir, $username; + global $data_dir, $username, $sent_folder; sqgetGlobalVar('delimiter', $delimiter, SQ_SESSION); $use_sent_subfolders = getPref($data_dir, $username, 'use_sent_subfolders', SMPREF_OFF); -//FIXME: why "na" as the default? Shouldn't this be SMPREF_NONE as it is elsewhere in this file? - $sent_subfolders_base = getPref($data_dir, $username, 'sent_subfolders_base', 'na'); + $sent_subfolders_base = getPref($data_dir, $username, 'sent_subfolders_base', $sent_folder); /** * If sent_subfolders are used and mailbox is equal to subfolder base diff --git a/plugins/sent_subfolders/setup.php b/plugins/sent_subfolders/setup.php index a3e7a6d1..06acb595 100644 --- a/plugins/sent_subfolders/setup.php +++ b/plugins/sent_subfolders/setup.php @@ -24,8 +24,9 @@ function squirrelmail_plugin_init_sent_subfolders() { = 'sent_subfolders_check_handleAsSent'; /* The hooks to automatically update sent subfolders. */ - $squirrelmail_plugin_hooks['left_main_before']['sent_subfolders'] - = 'sent_subfolders_update_sentfolder'; +// hook isn't in 1.5.x; isn't absolutely necessary to run on the folder list anyway +// $squirrelmail_plugin_hooks['left_main_before']['sent_subfolders'] +// = 'sent_subfolders_update_sentfolder'; $squirrelmail_plugin_hooks['compose_send']['sent_subfolders'] = 'sent_subfolders_update_sentfolder'; -- 2.25.1