Patch #7598

Extensible MailHandler

Added by Yuki Sonoda almost 7 years ago. Updated about 6 years ago.

Status:ClosedStart date:2011-02-10
Priority:NormalDue date:
Assignee:Jean-Baptiste Barth% Done:

0%

Category:Email receiving
Target version:1.1.2

Description

app/models/mail_handler is useful but not extensibly from plugins. I think it would be useful if MailHandler were extensible by making a subclass based on it.

I wrote the attached patch. This introduces MailHandler#dispatch_to_default which just calls receive_issue by default. Plugin can extend the dispatch table by overriding #dispatch_to_default.

extensible-mail-handler.patch Magnifier (913 Bytes) Yuki Sonoda, 2011-02-10 08:20

Associated revisions

Revision 4820
Added by Jean-Baptiste Barth almost 7 years ago

Introduced MailHandler#dispatch_to_default method to make MailHandler more extensible. #7598

Contributed by Yuki Sonoda

History

#1 Updated by Jean-Baptiste Barth almost 7 years ago

  • Status changed from New to Resolved
  • Assignee set to Jean-Baptiste Barth
  • Target version set to 1.1.2

If it makes plugin development easier, let's go. Don't hesitate to tell us if you have ideas about a real API (don't know if it's really needed though).

Applied in r4820, thanks.

#2 Updated by Jean-Philippe Lang over 6 years ago

  • Status changed from Resolved to Closed

Merged in 1.1-stable.

#3 Updated by Shane Proctor about 6 years ago

SPAM

Also available in: Atom PDF