KocsisDaniel/liferay-portal — pull request #254 by danielreuther (2013-03-06 18:03:35+00:00)
Title: LPS-28990 Test adjustments

Hi Daniel,

QA notified me about an issue with one of our test cases that has been surfaced by the latest changes for LPS-28990 in ee-6.1.x.

The problem here is that the upgrade process runs DB-level queries on data that has not been persisted before. This has not caused issues before because the LayoutTypePortlet.hasPortletId check returned true based on the existence of PortletPreferences entries. The recent change in ee-6.1.x however, has a different logic.

And even though this is currently no problem on trunk, I believe that we should fix it there as well (especially considering that we're still working on the permission logic for embedded portlets).

In essence, the fix simply writes the changed typesettings back to the DB and cleans the caches after the upgrade process (because the data changed on DB-level at that point).

Btw, I didn't really know which LPS number to use; but since the issue was surfaced by the new LPS-28990 logic, I figured it'd be best to use that.

Sorry for the lengthy description—and feel free to quote it, in case you forward this one ;)

Thanks,
Daniel