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]: Shorts Playlist without timestamps cant be copied #7023

Open
6 tasks done
efb4f5ff-1298-471a-8973-3d47447115dc opened this issue Mar 17, 2025 · 5 comments · May be fixed by #7083
Open
6 tasks done

[Bug]: Shorts Playlist without timestamps cant be copied #7023

efb4f5ff-1298-471a-8973-3d47447115dc opened this issue Mar 17, 2025 · 5 comments · May be fixed by #7083

Comments

@efb4f5ff-1298-471a-8973-3d47447115dc

Guidelines

Describe the bug

  1. Go to https://youtube.com/playlist?list=PLnN2bBxGARv7fRxsCcWaxvGE6sn5Ypp1H
  2. Try to copy the playlist
  3. See error

Image

Expected Behavior

Users are able to copy these types of playlists

Issue Labels

inconsistent behavior, feature stopped working

FreeTube Version

v0.23.2-nightly-5735 Beta

Operating System Version

Windows 11 IoT Enterprise LTSC 24H2

Installation Method

.exe

Primary API used

Local API

Last Known Working FreeTube Version (If Any)

No response

Additional Information

No response

Nightly Build

@mohamed-aziz-chamakh

This comment has been minimized.

@akrem023

This comment has been minimized.

@aziztouhami
Copy link

aziztouhami commented Mar 19, 2025

Me to i encountered the same bug , it's coming from showAddToPlaylistPromptForManyVideos function in "/store/modules/utils.js" , as the required video keys are :'videoId',
'title',
'author',
'authorId',
'lengthSeconds'

for this playlist there are some missed keys(author and authorId), so for that " videoDataValid" is set to false, and the error shown in the console is because of this function :
if (!videoDataValid) {
// Print error and abort
const errorMsgText = 'Incorrect videos data passed when opening playlist prompt'
console.error(errorMsgText)
console.error({
videoObjectArray,
missingKeys,
})
throw new Error(errorMsgText)
}

this issue doesn't appear for all shorts playlists,
My operating systems is Windows 10 Professionnel 22H2
and the freetube version is v0.23.2-nightly-5735 Beta

@OmarAbid1
Copy link

I tried this and it's true.
The issue comes from the showAddToPlaylistPromptForManyVideos function in /store/modules/utils.js.
The required keys are videoId, title, author, authorId, and lengthSeconds. In some shorts playlists, author and authorId are missing, causing videoDataValid to be false and triggering the error.

Tested on Windows 11 with FreeTube v0.23.2. I'll try to fix it

@OmarAbid2
Copy link

OmarAbid2 commented Mar 22, 2025

hey @efb4f5ff-1298-471a-8973-3d47447115dc ,sorry to bother , but May i know why my pull request got closed and can't comment anymore ? I started using FreeTube for a while and i absolutely loved it so i learned vue just so i can help contributing , I am serious about this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To assign
6 participants