PortalImpl.getActualLayoutQueryStringComposite retrieves all layouts, but only first one is used

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: 7.0.X, 7.1.X, Master
• Fix Version/s: 7.0.0 DXP FP87, 7.0.10.12 DXP SP12, 7.0.X, 7.1.10 DXP FP14, 7.1.X, 7.2.1 CE GA2, 7.2.X, 7.3.10 DXP GA1, Master
• Component/s: Performance, ~[Archived] WCM
• Labels:
  • ee-ts
  • es-ts
  • liferay-dxp-70-sp12
  • liferay-fixpack-de-87-7010
  • liferay-fixpack-dxp-14-7110
• Branch Version/s:
  7.2.x, 7.1.x, 7.0.x
• Backported to Branch:
  Committed
• Story Points: 1
• Fix Priority: 3
• Git Pull Request: https://github.com/brianchandotcom/liferay-portal/pull/73897

This LPS happens in PortalImpl.java, different from LPS-92844

All layouts are gotten from database or cache even when only a few of them will be shown. This behavior was detected after user login, when layout tree is rendered.

Master: PortalImpl.java#L1106-L1112 
7.1.x: PortalImpl.java#L1108-L1114 
7.0.x: PortalImpl.java#L1079-L1085#L335-L337 

To reproduce it, since when rendering a layout, all layouts are retrieved also in com.liferay.portal.events.ServicePreAction._initThemeDisplay(HttpServletRequest, HttpServletResponse) the script provided in the comments can be used to check if after the method is fixed the layouts are cached after PortalImpl.getActualLayoutQueryStringComposite gets called.