Maurycy Markowski

Results 155 comments of Maurycy Markowski

related to https://github.com/dotnet/efcore/issues/23205 and https://github.com/dotnet/efcore/issues/16314

@JoasE easier workaround: add ToList() at the end of subquery defined in the "let": ```cs var defaultLanguageId = 1; var selectedLanguageId = 2; var query = from defaultTranslation in context.Translations...

Note (Smit's comment on the initial PR for the fix): With pending selector in the middle coming in, we should just add a "fake wrapper" during pre-translation phase to match...

@JoasE we decided this issue doesn't meet the bar for a patch, sorry. The workaround is quite straightforward. The exception message indicates which subquery fails the validation (although it incorrectly...

currently querying capabilities are not supported - you can only project the JSON object or it's part. All querying must be done on the client. We track enabling this scenario...

nav expansion produces incorrect tree: ```cs DbSet() .Select(p => IncludeExpression( p, MaterializeCollectionNavigation( Navigation: Post.Tags, subquery: DbSet() .Where(t => EF.Property(p, "Id") != null && object.Equals( objA: (object)EF.Property(p, "Id"), objB: (object)EF.Property(t, "PostId")))...

we should make sure not to expose PK values in the exception messages that will be thrown for unbreakable cycles

related: https://github.com/dotnet/efcore/issues/9630

When accessing sub-element within JsonElement in materialization code we use JsonElement.GetProperty. We should use the TryGetProperty instead