Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DeleteAsync with Take on PostgreSQL #808

Closed
r-ising opened this issue Sep 4, 2024 · 4 comments
Closed

DeleteAsync with Take on PostgreSQL #808

r-ising opened this issue Sep 4, 2024 · 4 comments
Assignees

Comments

@r-ising
Copy link

r-ising commented Sep 4, 2024

1. Description

DeleteAsync not working correctly with .Take() after upgrade from 8.103.1 to 8.103.3

2. Any further technical details

Code example

var person1 = await Context.Set<PersonEntity>()
	.AsNoTracking()
	.FirstOrDefaultAsync();

await Context.Set<PersonEntity>()
	.AsNoTracking()
	.Where(p => p.Id == person1.Id)
	.Take(1)
	.DeleteAsync();

var person2 = await Context.Set<PersonEntity>()
	.AsNoTracking()
	.FirstOrDefaultAsync(p => p.Id == person1.Id);
// person2 is not null

If we remove the ".Take(1)" then person2 is null and it works correctly.

Further technical details

  • EF version: [EF Core v8.0.7]
  • EF Plus version: [EF Plus Core v8.103.3]
  • Database Server version: [PostgreSQL 16]
  • Database Provider version (NuGet): [Npgsql v8.0.3]
@JonathanMagnan JonathanMagnan self-assigned this Sep 4, 2024
@JonathanMagnan
Copy link
Member

Hello @r-ising ,

Thank you for reporting.

We will look at it.

Best Regards,

Jon

@JonathanMagnan
Copy link
Member

Hello @r-ising .

Just to confirm that the issue has been fixed.

The fix will be released on September 24.

Best Regards,

Jon

@r-ising
Copy link
Author

r-ising commented Sep 16, 2024

@JonathanMagnan Thanks, that sounds good to me.

@JonathanMagnan
Copy link
Member

Hello @r-ising ,

The v8.103.4.0 has been released today: https://github.com/zzzprojects/EntityFramework-Plus/releases/tag/8.103.4.0

Let me know if my developer correctly fixed your issue.

Best Regards,

Jon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants