DefaultUserRegistrar

  • don't use field names to iterate over props. While it is a good practice to keep getters in sync with props, it's not guaranteed. Also private props would not be accessible over reflection if running with security manager enabled.
  • The log for catch-all blog should be an error and should print the error stacktrace as well.
  • calling setExtraUserProperties from within updateExtraUSerProperties is incorrect. Missing (unset) properties should be removed on update instead of being set to null.
  • As for deprecation of method w/o user manager, someone could have still overriden them and retried user manager directly from SecuritySupport. You should put that method back call it and leave it empty as it was.
  • error message in RegistrationProcessor should be internationalized.
Change By: Jan Haderka (22/Jun/13 10:51 PM)
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