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

[Bug]: Cannot navigate between duplicate videos in playlist #57

Closed
3 of 4 tasks
kommunarr opened this issue Sep 9, 2023 · 17 comments
Closed
3 of 4 tasks

[Bug]: Cannot navigate between duplicate videos in playlist #57

kommunarr opened this issue Sep 9, 2023 · 17 comments
Labels
bug Something isn't working

Comments

@kommunarr
Copy link

kommunarr commented Sep 9, 2023

Guidelines

  • I have encountered this bug in the latest release of FreeTube.
  • I have searched the issue tracker for a bug report that matches the one I want to file, without success.
  • I have searched the documentation for information that matches the description of the bug I want to file, without success.

Describe the bug

  1. Save the same video twice to the same playlist
  2. Open that video in the playlist from the Playlist tab
  3. Try to click on duplicate video entry: encounter bug
  4. Try to click on "Play next video" button: encounter bug
  5. Skip to end of video and try to let video autoplay: encounter bug

Screenshot_20230908_225808

Expected Behavior

The duplicate video can be navigated to normally in any case.

Dev notes:
I would reckon this is happening because the videos have the same route. If that's the case, you'll have to add some dupe differentiator to the route (e.g., &duplicate-count={number}).

Issue Labels

usability issue

FreeTube Version

402af7a

Operating System Version

OpenSUSE Tumbleweed

Installation Method

other

Primary API used

Local API

Last Known Working FreeTube Version (If Any)

No response

Additional Information

No response

Nightly Build

@kommunarr kommunarr added the bug Something isn't working label Sep 9, 2023
@PikachuEXE
Copy link
Owner

Fixed in 6e7a627

@PikachuEXE PikachuEXE added this to the Local Playlist v1 milestone Sep 12, 2023
@kommunarr
Copy link
Author

kommunarr commented Sep 12, 2023

While this appears to be fixed for created playlists, I am still encountering the issue with the Favorites playlist.

Edit: It worked once, but now I'm still encountering it on created playlists. The bug is "fixed" in that you can try to navigate to a duplicate if you select the first duplicate in the playlist. But then the UI shows that the first duplicate is still the active playlist video, and you can't navigate to the other duplicates anymore. Tested with a playlist comprised of one video duplicated three times.

@PikachuEXE
Copy link
Owner

I have only fixed the clicking on links in 6e7a627

Please confirm if it's fixed or not first

I will try to take a look at Play next video which needs more changes (and probably more complicated)

@kommunarr
Copy link
Author

To clarify, I'm still experiencing the issue when clicking on links in the watch-video-playlist box.

@PikachuEXE
Copy link
Owner

Cannot reproduce
image
image
image
image
image

Videos:

Even I just create another new playlist now with duplicate I have no issue

Not to mention I might have also fixed play next/prev video in 66e3dbe

Might need help from @efb4f5ff-1298-471a-8973-3d47447115dc

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Collaborator

efb4f5ff-1298-471a-8973-3d47447115dc commented Sep 25, 2023

ave the same video twice to the same playlist
Open that video in the playlist from the Playlist tab
Try to click on duplicate video entry: encounter bug

Did not get a bug but on clicking on nr 4 it went to play nr 4 so i thought but i actually just reloaded nr 3

Capture

Try to click on "Play next video" button: encounter bug

When i got to this step it actually played nr 4

Skip to end of video and try to let video autoplay: encounter bug

no error here

@PikachuEXE
Copy link
Owner

Did not get a bug but on clicking on nr 4 it went to play nr 4 so i thought but i actually just reloaded nr 3

In watch page? or from local playlist view (inside a playlist)? or something else

Let me test both tomorrow

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Collaborator

This is on the watch page

@PikachuEXE
Copy link
Owner

I have tried both (from single local playlist view) & watch page on a playlist with duplicate videos
I clicked on non 1st video (2nd/3rd) and also 1st video to compare the results

The playlist component is showing correct entry being played
image
image
image

So for me it's cannot be reproduced anymore (after fix)

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Collaborator

Still get the bug im on bdb2a94

VirtualBoxVM_0r1JaOEYXv.mp4

@PikachuEXE
Copy link
Owner

OK I see what's wrong
The URLs in different links are different
Fixing...
image

@PikachuEXE
Copy link
Owner

Fixed in 25c75ce

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Collaborator

Can confirm that its fixed. Just to confirm @jasonhenriquez did u experience the same as me when it didn't work or is there still something that needs to be addressed?

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Collaborator

@kommunarr
Copy link
Author

Got this error while moving a duplicate video up in a playlist using the arrow button. But otherwise, looks good
Screenshot_20231021_061747

@PikachuEXE
Copy link
Owner

I think that's one of the callback errors from player after you switch back from watch video view
Please try it without watching any video

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Collaborator

can be closed btw

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

3 participants