Tickets :: [#1244] Attachment does not work (patch attached)

6.0.0-beta1

12/31/25

Summary	Attachment does not work (patch attached)
Queue	Agora
Type	Bug
State	Resolved
Priority	2. Medium
Owners	Horde Developers (at)
Requester	mathieu.clabaut (at) free (dot) fr
Created	01/25/2005 (7645 days ago)
Due
Updated	01/31/2005 (7639 days ago)
Assigned	01/25/2005 (7645 days ago)
Resolved	01/31/2005 (7639 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

01/31/2005 01:58:25 AM	Chuck Hagenbuch	Comment #6 State ⇒ Resolved	Reply to this comment
Committed, thanks.

01/25/2005 07:40:07 PM	Chuck Hagenbuch	Summary ⇒ Attachment does not work (patch attached)

01/25/2005 05:23:20 PM	Jan Schneider	Assigned to Horde Developers State ⇒ Assigned

01/25/2005 05:18:31 PM	mathieu (dot) clabaut (at) free (dot) fr	Comment #5 New Attachment: agora-lib-message3.diff	Reply to this comment
Ouuups. Sorry for the inconvenience. Attached is a hopefully non empty patch :)

01/25/2005 05:11:53 PM	Jan Schneider	Comment #4 State ⇒ Feedback Priority ⇒ 2. Medium	Reply to this comment
The patches that you uploaded are empty for some reason.

01/25/2005 01:31:36 PM	mathieu (dot) clabaut (at) free (dot) fr	Comment #3 New Attachment: agora-lib-message2.diff	Reply to this comment
I just discover that I was unable to delete an attachment or a message containing attachment. THe $vfs variable was not instanciated in the deleteAttachement function. Patch off all changes attached

01/25/2005 01:14:20 PM	mathieu (dot) clabaut (at) free (dot) fr	Comment #2 New Attachment: agora-lib-message.diff	Reply to this comment
Patch against HEAD attached.

01/25/2005 01:13:51 PM	mathieu (dot) clabaut (at) free (dot) fr	Comment #1 Priority ⇒ 3. High Type ⇒ Bug Summary ⇒ Attachement does not work (patch attached) Queue ⇒ Agora State ⇒ Unconfirmed	Reply to this comment
Saving attachement does not work. The VFS patch is not the same when saving and when retirieving. The problem comes from lib/Message.php where the VFS path is elaborated when message_id is empty. message_id must be elaborated before creating the VFS path. Patch follows in a comment....