Fixes opening an external player on the watch page passes user playlist

[absidue]

Fixes opening an external player on the watch page passes user playlist

Pull Request Type

• Bugfix

Related issue

#5188 (it's a discussion so the close keyword doesn't work)

Description

The code to open an external player in the watch-video-info component was checking this.inUserPlaylist to decide whether or not to pass the playlist information to the external player. Unfortunately though this.inUserPlaylist didn't exist, so it was passing playlist information to the external player. This pull request adds the missing inUserPlaylist property.

Screenshots

Before:

After:

Testing

1. Add this code snippet to the top of the openInExternalPlayer in src/renderer/store/modules/utils.js

   console.log(payload)
   return

2. Open FreeTube
3. Set your external player to anything that isn't None (you don't actually have to have the player installed, because of the code snippet above)
4. Watch a user playlist
5. Click the open in external player button in the video info section
6. Check the console and make sure the logged object doesn't contain any playlist information

Desktop

• OS: Windows
• OS Version: 10
• FreeTube version: da27eab

[absidue]

I've made the parameter required now.