Project

General

Profile

Actions

Patch #7598

closed

Extensible MailHandler

Added by Yuki Sonoda almost 14 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Normal
Category:
Email receiving
Target version:
Start date:
2011-02-10
Due date:
% Done:

0%

Estimated time:

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.


Files

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

Updated by Jean-Baptiste Barth almost 14 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.

Actions #2

Updated by Jean-Philippe Lang over 13 years ago

  • Status changed from Resolved to Closed

Merged in 1.1-stable.

Actions #3

Updated by Shane Proctor about 13 years ago

SPAM

Actions

Also available in: Atom PDF