BasePersistenceImpl does not unwrap more than one ModelWrapper or handle custom model implementations correctly

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: 7.0.X, Master
• Fix Version/s: 7.0.0 DXP FP46, 7.0.6 CE GA7, 7.0.0 DXP FP47, 7.0.0 DXP SP8, 7.0.X, 7.1.0 Beta 1, 7.1.X, 7.2.X, Master
• Component/s: Core Infrastructure
• Labels:
  • liferay-digital-enterprise-70-sp8
  • liferay-fixpack-de-46-7010
  • liferay-fixpack-de-47-7010
• Branch Version/s:
  7.2.x, 7.0.x
• Backported to Branch:
  Committed
• Fix Priority: 4
• Git Pull Request: https://github.com/brianchandotcom/liferay-portal/pull/58330

BasePersistenceImpl does not unwrap more than one ModelWrapper or handle custom model implementations correctly.

In the case where a model is wrapped more than once only the first model unwrapped. We should unwrap them all.

In the case where a custom implementation of a model has been passed to the update(BaseModel<T> baseModel) method of the persistence the finder cache is not fully cleared.