Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#1487718 closed Feature Requests (fixed)

Add part MIME ID to message_part_* hooks

Reported by: brandond Owned by:
Priority: 4 Milestone: 0.5.1
Component: Plugin API Version: 0.5
Severity: normal Keywords: message_part_before plugin api
Cc:

Description

It would be useful if the MIME ID of the message part was passed in to the message_part_before and message_part_after hooks.

This would allow plugins to do things like examine the message structure, parse previously unsupported multipart/alternative subparts, and later replace part text with the output of that parsing.

Attachments (1)

part_mime_id.patch​ (670 bytes) - added by brandond 2 years ago.
Attached patch to add support for part MIME ID

Download all attachments as: .zip

Change History (5)

Changed 2 years ago by brandond

Attached patch to add support for part MIME ID

comment:1 Changed 2 years ago by brandond

  • Component changed from Addressbook to Plugin API
  • Keywords message_part_before plugin api added
  • Milestone changed from later to 0.5.1
  • Priority changed from 5 to 4
  • Type changed from Bugs to Feature Requests

comment:2 Changed 2 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Added in [c9f67353].

comment:3 Changed 2 years ago by brandond

Thanks!

FWIW, I'm not sure that it's necessary to set 'id' again in the message_part_after hook, since it's still contained in the $data array reused from the call to message_part_before.

comment:4 Changed 2 years ago by brandond

... but only if no plugin deletes it or changes it, I suppose. Better safe than sorry.

Note: See TracTickets for help on using tickets.