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

(List) Drag child item to last position crash #361

Open
andiandi13 opened this issue Feb 2, 2025 · 4 comments
Open

(List) Drag child item to last position crash #361

andiandi13 opened this issue Feb 2, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@andiandi13
Copy link
Contributor

What happened?

Make any item child, drag it below the last item and it'll crash.

App Version

7.1.0-BETA

Android Version (API Level)

29

(Optional) Relevant log output

java.lang.IndexOutOfBoundsException: Asked to get item at 11 but size is 10at h0.z0.f(SourceFile:46)at com.philkes.notallyx.presentation.view.note.listitem.sorting.d.o(SourceFile:21)at com.philkes.notallyx.presentation.view.note.listitem.sorting.d.k(SourceFile:48)at com.philkes.notallyx.presentation.view.note.listitem.h.k(SourceFile:67)at com.philkes.notallyx.presentation.view.note.listitem.c.f(SourceFile:121)at h0.E.p(SourceFile:462)at h0.A.b(SourceFile:121)at androidx.recyclerview.widget.RecyclerView.onTouchEvent(SourceFile:38)at android.view.View.dispatchTouchEvent(View.java:13953)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3054)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2741)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2755)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2755)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2755)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2755)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2755)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2755)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2755)at android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:3060)at android.view.
@andiandi13 andiandi13 added the bug Something isn't working label Feb 2, 2025
@PhilKes
Copy link
Owner

PhilKes commented Feb 2, 2025

Either I am not understanding correctly or there is some step missing:

notallyx_issues_361.webm

@andiandi13
Copy link
Contributor Author

Ah...
I have the bug on a list I created with an older apk, but if I create a new list, no.
Does it make any sense ?

@PhilKes
Copy link
Owner

PhilKes commented Feb 2, 2025

Ah... I have the bug on a list I created with an older apk, but if I create a new list, no. Does it make any sense ?

Mh yes it could be that the internal order fields might have been "corrupted" in some older version containing bugs.
I will try to investigate this issue with some more unit tests

@andiandi13
Copy link
Contributor Author

OK, well from now I'll create a new list for each new apk to prevent this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants