dantleech

From what you describe this would indeed seem to be a bug in the QueryConverter class. A first step would be to add a functional test for this scenario and...

Thanks for the effort. Using the `jcr:primaryType` would, imo, also be the best thing to do ideally. Unfortunately the Jackrabbit implementation the Java side) does not allow the `jcr:primaryType` to...

Instead of adding a parameter, could we not automatically use the PHPCR primary type IF it has been defined? I can't really think of any reason why that wouldn't be...

Reading metadata will/should not incur a significant overhead, metadata is cached and we read from it hundreds of times per-request. Also we could cache queries. For the second point, the...

Why would we need to load all the metadatas? We only need the metadata for documents being selected from in the query (if I have understood), and this metadata is...

So this UOW is moving the node, then recalculating the changeset, and then registering a delete for the "missing" node and then deleting it because `executeRemoves` is after `executeUpdates` ?...

ok, I know what the problem is. When the changeset if calculated the "isNew" flag is determined by the presence of the object ID in the `$originalData` array. If [isNew...

Had another go at this, but failed. Maybe another day.

> where you specify a translation model that is stored in its own table and used for the translated fields. is that what sylius is using? No, Sylius has its...

+1. Although I should point out the example you gave above is not caused by the identity mapping (at least it is not the principle cause) The problem is that...