matthewkong/liferay-portal — pull request #319 by jonathanmccann (2013-08-14 23:03:44+00:00)
Title: LPS-38694

Hey Matt,

Thanks for taking a look at this. Basically, I reverted http://issues.liferay.com/browse/LPS-25635 because it always returned true even if the user didn't have permission to view/subscribe. I created a new check specifically for discussion type subscriptions to handle that LPS. Also http://issues.liferay.com/browse/LPS-31948 doesn't work anymore in Trunk because the DL class isn't specifically mentioned so it always returned false. I also made a new method that can be called when a user is removed from a site (which could then be used for communities, roles, etc).

Let me know what you think.

Thanks,
Jonathan