JorgeFerrer/liferay-portal — pull request #84 by KocsisDaniel (2012-10-12 09:54:49+00:00)
Title: LPS-28982 Disable add record when staging

Hey Jorge,

as you requested I moved the method from the DDLImpl to a more generic place, but I think since this one is really staging related I put that into stagingUtil/impl. I also renamed, but not to isStagedPortlet() because it should return true only when the portlet is staged however the groupId is the staging group's one not the live, so I used the isStagingPortlet() name instead.

Please let me know your opinion!

I also spent some time on the method itself, and after I checked how the groupImpl.isStagedPortlet works I decided to use the livegroup because it contains the typesettings with the required information and not the stagingGroup and I also use the parentGroup instead of the original group parameter because we follow that pattern on the groupImpl and in this way the method is more generic.

Thanks,
Daniel