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	4/13/11 (5476 days ago)
Due
Updated	5/4/11 (5455 days ago)
Assigned
Resolved	4/13/11 (5476 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