Abstract types should always start with "Abstract" - so MailBasedAbstractPasswordStrategy should be named AbstractMailBasedPasswordStrategy.

Also I'd not introduce that supertype unless it's very likely it'll be extended by several types. AFAIK there's only MailChangePasswordLinkStrategy extending it right now.

Change By: Daniel Lipp (06/Mar/13 11:12 AM)
Resolution: Fixed
Status: Resolved Reopened
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
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]>
----------------------------------------------------------------

Reply via email to