We are using SelfServicing with version 5.11.3.

Consider this code:

var doc = new DocumentEntity();
var bestanden = doc.Bestanden;

In the 2nd line the property 'doc.Bestanden' is called and executes a query caused by the Self Servicing framework to the database:

exec sp_executesql N'SELECT [dbo].[Bestanden].[BestandID] AS [ID], <<...and many other fields... >>
FROM [dbo].[Bestanden] WHERE ( [dbo].[Bestanden].[BestandDocID] = @p1)',N'@p1 uniqueidentifier',@p1='00000000-0000-0000-0000-000000000000'

This unnecessary in our opinion because the document entity is not saved yet, so there could be no record in the Bestanden table. Even the ID of the DocumentEntity is not set, which is the foreign key in the Bestanden table.

We think the execution of the query would only be useful if the DocumentEntity instance 'doc' has a given 'ID' or maybe simpeler, to check if property IsNew is false. Thus if an entity is still new, then there is no reason to query the database for related entities.

What is your opinion on this issue?

We don't check if the entity is new, as people use it to fetch related entities for an entity with an id X without first fetching the entity with id X. (like initializing the doc entity with a pk, and then accessing Bestanden). So changing that will be out of the question.

One could argue that there have to be deeper checks if a field is set to be able to lazy load properly, tho this isn't simple, as for the lazy loading code that starts the whole process it's unknown which fields are involved.

SS's lazy loading can trigger unnecessary DB queries, hence it's discouraged to use it.