Currently e-mail event sending process (internally) doesn't represent real world model, which makes it hard to improve this code with new features.
I'm proposing to:
1. create kEmail class with following public methods:
- findEvent($name, $type)
- setParams($params)
- send()
2. make $this->sender refer to kEmailSendingHelper class instance for easy access across all methods
3. move existing code from EmailEventsEventHandler into new kEmail class
4. replace OnEmailEvent event sending with new kEmail class usage
At the end interface of kApplication::EmailEventUser and kApplication::EmailEventAdmin will be almost the same (will return true/false instead of kEvent object), so no need to change code to make it working again.
Usually nobody relies on e-mail sent fact in their code anyway.
- parent of
-
INP-1073 Don't remove In-Portal Tags, when creating Plain Text e-mail from Html one
-
- Closed
-
- relates to
-
INP-224 [ONGOING] Code Cleanup in 5.2.x branch
-
- Closed
-
-
INP-402 Create more control over sent emails
-
- Closed
-
-
INP-971 Improvements to Languages and Phrases Export capabilities
-
- Closed
-
-
INP-1002 Ability to specify both HTML and Plain Text version of e-mail event
-
- Closed
-
-
INP-1068 Record E-mail Event Type to the E-mail Log
-
- Closed
-
-
INP-1276 Fix code, that blocks attempts to send disabled e-mails templates
-
- Closed
-
- mentioned in
-
Page Loading...