From 454a548f312d3dc69e10ec20fc1b8c6ab2c3f941 Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Fri, 5 Mar 2010 11:35:02 -0700
Subject: [PATCH] Remove allow_resume_all_in_drafts config option

---
 imp/config/conf.xml | 5 -----
 imp/docs/UPGRADING  | 4 ++++
 imp/message.php     | 7 +++----
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/imp/config/conf.xml b/imp/config/conf.xml
index f1b9fc80e..b58fbb7e8 100644
--- a/imp/config/conf.xml
+++ b/imp/config/conf.xml
@@ -15,11 +15,6 @@
      choose which folder to save their sent messages to. If this is false, then
      messages will be saved to the folder defined in their
      preferences.">false</configboolean>
-     <configboolean name="allow_resume_all_in_drafts" desc="If this is true, the
-     user will be able to resume any message in their draft mailbox (to
-     compensate for some clients that don't bother setting the draft flag).
-     Otherwise, only messages marked as 'drafts' can be
-     resumed.">false</configboolean>
     </case>
     <case name="false" desc="No" />
    </configswitch>
diff --git a/imp/docs/UPGRADING b/imp/docs/UPGRADING
index 02bc05eb8..751366db3 100644
--- a/imp/docs/UPGRADING
+++ b/imp/docs/UPGRADING
@@ -42,6 +42,10 @@ spellchecking, and login redirection have been moved to Horde.
 The 'show_preview' option has been removed; enabling previews is now handled
 by the 'show_preview' preference.
 
+The 'allow_resume_all_in_drafts' option has been removed; all messages that
+appear in the configured Drafts mailbox always are now always allowed to be
+resumed.
+
 The 'css_files' configuration option for the dynamic (dimp) display has been
 removed.  Equivalent behavior can now be found in the Horde-wide 'cssfiles'
 hook.
diff --git a/imp/message.php b/imp/message.php
index 7b6f74962..ea76b3e9a 100644
--- a/imp/message.php
+++ b/imp/message.php
@@ -514,10 +514,9 @@ if (!empty($conf['user']['allow_view_source'])) {
 }
 
 if (!$disable_compose &&
-    (!empty($conf['user']['allow_resume_all']) ||
-     (!empty($conf['user']['allow_resume_all_in_drafts']) &&
-      $mailbox_name == IMP::folderPref($prefs->getValue('drafts_folder'), true)) ||
-      in_array('\\draft', $flags))) {
+    (in_array('\\draft', $flags) ||
+     !empty($conf['user']['allow_resume_all']) ||
+     ($mailbox_name == IMP::folderPref($prefs->getValue('drafts_folder'), true)))) {
     $a_template->set('resume', Horde::widget(IMP::composeLink(array(), array('actionID' => 'draft') + $compose_params), _("Resume"), 'widget', '', '', _("Resume"), true));
 }
 
-- 
2.11.0