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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 Bug Report: Query order desc not include null values #7083

Open
2 tasks done
mr-saw opened this issue Nov 6, 2023 · 2 comments
Open
2 tasks done

馃悰 Bug Report: Query order desc not include null values #7083

mr-saw opened this issue Nov 6, 2023 · 2 comments
Assignees
Labels
bug Something isn't working product / databases Fixes and upgrades for the Appwrite Database.

Comments

@mr-saw
Copy link

mr-saw commented Nov 6, 2023

馃憻 Reproduction steps

When I want to retrieve a list of documents with a Query.orderDesc() (having an Integer as an attribute type), which has a pagination with the use of an afterCursor(), it doesn't don't take into account documents with a null attribute.

Here is a more telling example :
id value
1 4
2 1
3 2
4 null
Query: order desc (value) - limit 100
1 3 2 4 (total: 4)
Query: order desc (value) - limit 2
1 3 (total: 4)
Query: order desc (value) - limit 2 - afterCursor(3) # expected
2 4 (total: 4)
Query: order desc (value) - limit 2 - afterCursor(3) # actual
2 (total: 4)

馃憤 Expected behavior

afterCursor() should retrieve any list with null values.

馃憥 Actual Behavior

afterCursor() doesn't retrieve documents with a null value.

馃幉 Appwrite version

Version 1.4.x

馃捇 Operating system

Linux

馃П Your Environment

No response

馃憖 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

馃彚 Have you read the Code of Conduct?

@mr-saw mr-saw added the bug Something isn't working label Nov 6, 2023
@pankaj-2503
Copy link

Hey , i am understanding the issue how can i contribute to it if i have knowledge of some backend with frontend in mern stack

@abnegate
Copy link
Contributor

abnegate commented Nov 7, 2023

Hey , i am understanding the issue how can i contribute to it if i have knowledge of some backend with frontend in mern stack

Hi @pankaj-2503, you are welcome to contribution a fix over in the database repository 馃榿 please check out contribution guide first.

@stnguyen90 stnguyen90 added the product / databases Fixes and upgrades for the Appwrite Database. label Nov 13, 2023
@stnguyen90 stnguyen90 self-assigned this Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working product / databases Fixes and upgrades for the Appwrite Database.
Projects
None yet
Development

No branches or pull requests

4 participants