Jasig/AnnouncementsPortlet — pull request #2 by waymirec (2012-11-19 21:34:50+00:00)
Title: ANNPLT-50

Rewrite the UserPermissionChecker class to be instanced, removing any static functionality.

Modify the inRoleForTopic method to increase performance by checking for User based queries before group, thus avoiding unnecessary calls to PortletRequest.isUserInRole.

Added a new cache for the UserPermissionChecker class with a TTL of 5mins.

Add a new UserPermissionCheckerFactory class that is responsible for instantiating the UserPermissionChecker class and caching it based on portlet request user and topic values. This will further reduce unnecesary calls to PortletReques
t.isUserInRole

Refactor all existing classes which reference UserPermissionChecker to now have the factory injected into them and to make calls to the factory create method.