Tickets :: [#14571] Attachments are saved in spite of save_attachments indicating otherwise in mobile modes

6.0.0-beta4

3/9/26

Summary	Attachments are saved in spite of save_attachments indicating otherwise in mobile modes
Queue	IMP
Queue Version	6.2.17
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	wahnes (at) uni-koeln (dot) de
Created	02/02/2017 (3322 days ago)
Due
Updated	10/20/2017 (3062 days ago)
Assigned
Resolved	02/14/2017 (3310 days ago)
Github Issue Link
Github Pull Request
Milestone	6.2.18
Patch	No

10/20/2017 08:33:44 PM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 3812ebf1c5565c05dee6efa73c483cefc12d568e Author: Jan Schneider <jan@horde.org> Date: Tue, 14 Feb 2017 13:02:43 +0100 Only test for checkbox to save attachments if it exists. ~~Bug: 14571~~ M lib/Ajax/Application/Handler/Common.php https://github.com/horde/imp/commit/3812ebf1c5565c05dee6efa73c483cefc12d568e

10/20/2017 08:33:43 PM	Git Commit	Comment #6	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 34d8f7bb37f0e7a284dc92365588b8891925ca01 Author: Jan Schneider <jan@horde.org> Date: Tue, 14 Feb 2017 12:47:59 +0100 Wrap Preparing for ~~bug #14571~~. M lib/Ajax/Application/Handler/Common.php https://github.com/horde/imp/commit/34d8f7bb37f0e7a284dc92365588b8891925ca01

02/14/2017 12:58:23 PM	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git (master): commit 3c5572b585e3a9ef872e809cd362270a0d0ca2e5 Author: Jan Schneider <jan@horde.org> Date: Tue Feb 14 13:02:43 2017 +0100 Only test for checkbox to save attachments if it exists. ~~Bug: 14571~~ imp/lib/Ajax/Application/Handler/Common.php \| 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) http://github.com/horde/horde/commit/3c5572b585e3a9ef872e809cd362270a0d0ca2e5

02/14/2017 12:58:22 PM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git (master): commit 5fbc691fb442c1983110fdce887ce9701c90f3e7 Author: Jan Schneider <jan@horde.org> Date: Tue Feb 14 12:47:59 2017 +0100 Wrap Preparing for ~~bug #14571~~. imp/lib/Ajax/Application/Handler/Common.php \| 32 +++++++++++++++++++++-------- 1 file changed, 24 insertions(+), 8 deletions(-) http://github.com/horde/horde/commit/5fbc691fb442c1983110fdce887ce9701c90f3e7

02/14/2017 12:04:33 PM	Jan Schneider	Assigned to Jan Schneider State ⇒ Resolved Milestone ⇒ 6.2.18

02/14/2017 12:04:25 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 8396aa7fd8edf17c188110fe4e37266bc2a1924b Author: Jan Schneider <jan@horde.org> Date: Tue Feb 14 13:02:43 2017 +0100 Only test for checkbox to save attachments if it exists. ~~Bug: 14571~~ imp/lib/Ajax/Application/Handler/Common.php \| 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) http://github.com/horde/horde/commit/8396aa7fd8edf17c188110fe4e37266bc2a1924b

02/14/2017 12:04:23 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit f0fe81d4213d85e168d07ed53d9c1b415de3965a Author: Jan Schneider <jan@horde.org> Date: Tue Feb 14 12:47:59 2017 +0100 Wrap Preparing for ~~bug #14571~~. imp/lib/Ajax/Application/Handler/Common.php \| 32 +++++++++++++++++++++-------- 1 file changed, 24 insertions(+), 8 deletions(-) http://github.com/horde/horde/commit/f0fe81d4213d85e168d07ed53d9c1b415de3965a

02/02/2017 12:27:10 PM	wahnes (at) uni-koeln (dot) de	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Attachments are saved in spite of save_attachments indicating otherwise in mobile modes Queue ⇒ IMP Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Both in smartmobile and in minimal mode, attachments of sent messages are saved to the "Sent" folder even if the save_attachments preference is set to "never", indicating that attachments should not be saved. This only happens in smartmobile and minimal mode, i.e. in basic and dynamic mode, the save_attachments preference is honored properly.