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

Playlist under edit cannot be refreshed after all the track matches were found #219

Closed
touwys opened this issue Oct 8, 2023 · 4 comments · Fixed by #218
Closed

Playlist under edit cannot be refreshed after all the track matches were found #219

touwys opened this issue Oct 8, 2023 · 4 comments · Fixed by #218
Labels
Could not be reproduced The reported problem could not be reproduced following the provided instructions

Comments

@touwys
Copy link

touwys commented Oct 8, 2023

listFix() v2.9.0

Steps:

  1. Open a playlist for editing in the editor. The playlist should contain a mix of OK tracks (already matched tracks), plus Exclamation-marked tracks (those tracks for which matches are still to be found):

  2. Activate the repair (magic wand) on the playlist.

  3. Once the all the appropriate matches have been found, close the last processing dialogue, i.e. "Select closest matches" by pressing the cancel button to return to the playlist.

  4. Issue upon return to the playlist: the playlist cannot be refreshed. The refresh button is unavailable/dulled. To refresh the playlist, it has to be closed first, and then reopened.

@Borewit
Copy link
Owner

Borewit commented Oct 21, 2023

I cannot reproduce this behavior. It did however came across some code inconsistencies, which I addressed in PR #222.

Can you retest with the build of PR #218 (maybe this is the version you currently have installed)?

@Borewit Borewit added the Could not be reproduced The reported problem could not be reproduced following the provided instructions label Oct 21, 2023
@Borewit
Copy link
Owner

Borewit commented Oct 21, 2023

Duplicate of #195?

@touwys
Copy link
Author

touwys commented Oct 22, 2023

Duplicate of #195?

Same process, but slightly different approach. I think it's safe to combine the two (#195 & #219) as one issue. That said, strangely, I am now also unable to reproduce #219 (using build 2.9.0-1_edf148f). Would you rather combine the two issues into one, so that we can observe its impact on operations?

I am also going to add a note at #195 for additional information.

@Borewit
Copy link
Owner

Borewit commented Oct 22, 2023

Closing this issue as it can no longer be reproduced on a recent build, #218 may have resolved it.

@Borewit Borewit closed this as completed Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Could not be reproduced The reported problem could not be reproduced following the provided instructions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants