Null check on parameters
Merge remote-tracking branch 'origin/Testing_Release_1.2' into Testing_Release_1.2
Merge remote-tracking branch 'origin/Testing_Release_1.2' into Testing_Release_1.2
UF should be listening to a couple more events
A bit of refactoring - just to keep the user feedback stuff together
Renamed one of the methods in IUserFeedback from submitExplicitResponse(…) to submitPrivacyNegotiationResponse(…), and added submitAccessControlResponse(…)
1) Added access control persistence to NotificationsController in webapp.
2) Expanded UserFeedback.clear() method so that it notifies all other nodes that it's being cleared
 3) Tests surrounding UserFeedback.clear()
 4) A lot of code refactoring inside NotificationsController, just to make the code more readable and reusable
Removed some spammy logging statements, added proper synchronisation so we don't get 5x notifications about repeat pubsub events (though this doesn't really fix the fact that we are receiving 5x pubsub events)
Send proper event type in clear() method
Oops, forgot to re-enable the repositories which are there as a backup
Refactoring: Removed redundant PrivacyPolicyNegotiationListener service - it's all wrapped up in the PrivacyPolicyNegotiationController now
Fixing bug where we sometimes get a stack trace when trying to send a Faces Message
No reason why these methods should NOT be static, other than to access the (rather inconveniently non-static) logger
Merge remote-tracking branch 'origin/Testing_Release_1.2' into Testing_Release_1.2
1) Added Access Control GUI.
2) Added notification list items for Access Control.
3) Fixed validation messages for PPN (they're more human-readable now).
4) Small bug when submitting AC responses via user feedback which would prevent the "REMOVE" event being sent
Merge remote-tracking branch 'origin/Testing_Release_1.2' into Testing_Release_1.2