TypesenseBundle icon indicating copy to clipboard operation
TypesenseBundle copied to clipboard

Published 20 hours ago • verified publisher icon acseo

Fix Entity Hydration Issue for Inherited Entities with DiscriminatorMap

Open james2001 opened this issue 1 year ago • 2 comments

Problem Description

In the existing system, the hydration of entities resulting from a search query is not geared towards entity hierarchies that have a DiscriminatorMap. This leads to a "ResultSetMapping builder does not currently support your inheritance scheme" error when attempting to use ResultSetMappingBuilder with entities that have an inheritance hierarchy.

Proposed Solution

Instead of relying on a native SQL query and ResultSetMappingBuilder, this PR employs DQL to fetch the entities, which natively supports DiscriminatorMap. After fetching the entities using DQL, we use usort in PHP to sort them in the desired order. This allows us to circumvent the aforementioned error while retaining the ability to sort the entities.

This also likely fixes Issue #87 as we no longer use the FIELD function in our queries.

james2001 avatar Oct 19 '23 08:10 james2001

Hello @james2001 thank you for this PR, it looks very interesting.

I'll try review it soon.

npotier avatar Oct 19 '23 08:10 npotier

@npotier Any news on this PR?

james2001 avatar Nov 08 '23 17:11 james2001