6.0.0-beta1
7/18/25

[#11499] Dynamic view: Save attachments popup not working
Summary Dynamic view: Save attachments popup not working
Queue IMP
Queue Version Git master
Type Bug
State Not A Bug
Priority 1. Low
Owners
Requester thomas.jarosch (at) intra2net (dot) com
Created 10/12/2012 (4662 days ago)
Due
Updated 10/15/2012 (4659 days ago)
Assigned
Resolved 10/13/2012 (4661 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
10/15/2012 08:46:43 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (develop):

commit 5bff6f5a74ad9aa4b762461ed87058b6e6d21511
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Sat Oct 13 16:44:55 2012 -0600

     Remove 'prompt_' options to the save_attachments pref

     See Request #11499

  imp/compose.php                           |    7 +++----
  imp/config/prefs.php                      |   10 ++++------
  imp/docs/UPGRADING                        |    3 +++
  imp/lib/LoginTasks/SystemTask/Upgrade.php |   19 +++++++++++++++++++
  4 files changed, 29 insertions(+), 10 deletions(-)

http://git.horde.org/horde-git/-/commit/5bff6f5a74ad9aa4b762461ed87058b6e6d21511
10/13/2012 10:45:41 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit 5bff6f5a74ad9aa4b762461ed87058b6e6d21511
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Sat Oct 13 16:44:55 2012 -0600

     Remove 'prompt_' options to the save_attachments pref

     See Request #11499

  imp/compose.php                           |    7 +++----
  imp/config/prefs.php                      |   10 ++++------
  imp/docs/UPGRADING                        |    3 +++
  imp/lib/LoginTasks/SystemTask/Upgrade.php |   19 +++++++++++++++++++
  4 files changed, 29 insertions(+), 10 deletions(-)

http://git.horde.org/horde-git/-/commit/5bff6f5a74ad9aa4b762461ed87058b6e6d21511
10/13/2012 10:35:13 PM Michael Slusarz Comment #2
State ⇒ Not A Bug
Reply to this comment
Not a bug - this is simply a feature that has not yet been ported to 
the dynamic view UI.  The preference values are applied correctly (and 
sanely) given the absence of the UI component.

Truthfully, this should probably be changed to a simple yes/no 
preference.  The prompting is overkill.
2. What about setting the default to "always" or at least "prompt_yes".
     Normally the user really wants to keep his data.
No - the default (strip attachments) prevents a user from filling up 
their quota with large sent-mail messages.  It is up to an admin (or 
the user) to decide otherwise.
10/12/2012 03:19:04 PM Thomas Jarosch Comment #1
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Queue ⇒ IMP
Summary ⇒ Dynamic view: Save attachments popup not working
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
Hi,

there's a "save_attachments" pref in IMP to save attachments for sent emails.
The default value is "prompt_no", which means to ask the user and select
"no" as the default button.

When sending a message, the popup is not displayed to the user.
Instead it just uses the default value of the pref. In the default setting,
it discards the attachments.

If you set it to "prompt_yes", there's also no popup, but it will keep 
the attachment.
So the popup is not working right now.

2. What about setting the default to "always" or at least "prompt_yes".
     Normally the user really wants to keep his data.


Cheers,
Thomas

Saved Queries