Tickets :: [#9899] Delete attachments from a message is not working

Summary	Delete attachments from a message is not working
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	goncalo.queiros (at) portugalmail (dot) net
Created	04/13/2011 (5344 days ago)
Due
Updated	05/04/2011 (5323 days ago)
Assigned
Resolved	04/13/2011 (5344 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

04/13/2011 04:29:40 PM	Michael Slusarz	Comment #3 Assigned to Michael Slusarz State ⇒ Resolved	Reply to this comment
This was broken, but the fix was wrong since atc_indices is expected to be an array - multiple attachments may be deleted in a single request. The solution is to make sure we JSON encode/decode atc_indices during the request.

04/13/2011 04:28:26 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9899~~: Fix deleting attachments from compose message in dynamic view 4 files changed, 10 insertions(+), 7 deletions(-) http://git.horde.org/horde-git/-/commit/c6602b202a5af8996b867fed868a333a30f762ae

04/13/2011 04:04:06 PM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #1 Priority ⇒ 1. Low New Attachment: 0001-Fixed-deleting-a-message-attachment.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ IMP Summary ⇒ Delete attachments from a message is not working Type ⇒ Bug State ⇒ Unconfirmed	Reply to this comment
Attached a patch to fix this