LayoutPageTemplateEntries with no LayoutPageTemplateCollectionId are not cleaned up upon group deletion

• Type: Regression Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: 7.2.X, Master
• Fix Version/s: 7.2.10 DXP FP1, 7.2.X, 7.2.1 CE GA2, Master
• Component/s: ~[Archived] WCM
• Labels:
  • liferay-fixpack-dxp-1-7210
• Branch Version/s:
  7.2.x
• Backported to Branch:
  Committed
• Fix Priority: 3
• Git Pull Request: https://github.com/brianchandotcom/liferay-portal/pull/75690

Description
https://github.com/liferay/liferay-portal/blob/master/modules/apps/layout/layout-page-template-service/src/main/java/com/liferay/layout/page/template/internal/model/listener/GroupModelListener.java

When a Group is deleted, all LayoutPageTemplateCollections are cleaned up, which in turn cleans up all LayoutPageTemplateEntries, however since it is now possible to have a LayoutPageTemplateEntry with no LayoutPageTemplateCollectionId, some layoutPageTemplateEntries, mainly Display Pages will not properly get removed from the DB upon group deletion. This will likely affect customers who want to clean up some sites to improve database table query speeds.

Steps to Reproduce

1. Add a LayoutPageTemplateEntry with no layoutPageTemplateCollectionId
2. Delete group LayoutPageTemplateEntry is in
3. Assert layoutPageTemplateEntry is removed from database

Expected Result
LayoutPageTemplateEntry deleted

Actual Result
LayoutPageTemplateEntry present

Tested on Tomcat 9.0.10 + MySQL 5.7
Portal master GIT ID: e4ca8e0fc87b963340338fc413fe068ad88761dc