sergiogonzalez/liferay-portal — pull request #325 by KocsisDaniel (2012-08-28 10:35:58+00:00)
Title: [TECH-SUPPORT] LPS-29341

Hi Sergio,

As we discussed I kept the permission checking logic as-is, except I made some SF to fit into the new logic. The new rule to produce the id's is what we agreed on: I use the parentMessage to get the category and threadId if it present and the thread if not. I changed only one thing, I pass the parentMessage groupId to the permissionChecker because if we use the original id it is possible to reply a message in a restricted group with empty category (categoryId = 0) because we check the permission to addMessage in the user site group with categoryId 0 which is true, but of course we will reply a message in the restricted group thread with category 0.

I also deprecated the old method, and re-generated all the related services. Please let me know your opinion about the solution.

Thanks,
Daniel