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

Crash when navigating to a release group from an artist, then clicking back #402

Closed
1 task
lydavid opened this issue Sep 1, 2023 · 0 comments
Closed
1 task
Labels
bug Something isn't working

Comments

@lydavid
Copy link
Owner

lydavid commented Sep 1, 2023

Issue did not exist before ktor changes: e219759

Temp fix

Comment out

key = lazyPagingItems.itemKey { it.id },

We only lose the ability to animate sort

Log:

FATAL EXCEPTION: main
Process: io.github.lydavid.musicsearch, PID: 8005
java.lang.IllegalArgumentException: Key "af7347df-ef5a-3c41-826d-d884f8916c17" was already used. If you are using LazyColumn/Row please make sure you provide a unique key for each item.

Problem

Somehow we're inserting a second copy of a rg, but with artist_credit_names.
rg inserted twice

  • Note that this is a query, not the rg table itself, hence why duplicate rg id is possible

    • inspecting artist_credit, we see there's an entry for the empty string
  • Notice that we're matching this rg with the artist_credit_names "" (not NULL) and a-ha

  • why does using ktor vs retrofit cause any issue like this? it's possible it's due to some other changes in that massive PR

@lydavid lydavid added the bug Something isn't working label Sep 1, 2023
@lydavid lydavid closed this as completed in 045bf9e Sep 2, 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
Projects
None yet
Development

No branches or pull requests

1 participant