[MGNLPUR-56] Strategy "Mail" is not working Created: 20/Dec/11  Updated: 04/Oct/13  Resolved: 04/Oct/13

Status: Closed
Project: Magnolia Public User Registration
Component/s: None
Affects Version/s: 1.3.1
Fix Version/s: None

Type: Bug Priority: Critical
Reporter: Samuel Schmitt Assignee: Unassigned
Resolution: Outdated Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
causality
is causing MGNLPUR-22 make the Mail strategy more flexible Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

In the configuration of PUR we has the possibility to configure a "registrationStrategy".
The Mail strategy info.magnolia.module.publicuserregistration.strategy.Mail is not working for several reasons:

  • in this class the attributes (pagePath, emailTemplate, ect..) are never set, that trigger a NPE in Mail#validateRegistration() on the call of MailUtil.mail()
  • in DefaultUserRegistrar.registerUser() the call to the strategy regStrategy.validateRegistration(user) passes a user where user.properties is empty. The values of this map are needed later in the MailUtil.mail().

Well it seems that this strategy is really not working.
Maybe we could drop it.
If we check the demo instance running on 4.4, http://demoauthor.magnolia-cms.com/demo-project/members-area/registration.html, the concept of the Mail strategy is substitute by the confirmation email message.



 Comments   
Comment by Magnolia International [ 24/Jan/13 ]

The "strategy" is (was?) working. It's the integration with form/stk that completely bypasses it IIRC.

Generated at Mon Feb 12 06:42:39 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.