[
http://jira.magnolia-cms.com/browse/MGNLPUR-56?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Samuel Schmitt updated MGNLPUR-56:
----------------------------------
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.
was:
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.
> Strategy "Mail" is not working
> ------------------------------
>
> Key: MGNLPUR-56
> URL: http://jira.magnolia-cms.com/browse/MGNLPUR-56
> Project: Magnolia Public User Registration
> Issue Type: Bug
> Affects Versions: 1.3.1
> Reporter: Samuel Schmitt
> Priority: Critical
> Fix For: 1.4
>
>
> 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.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
