[ie/XVideosQuickies] Fix extractor

[JakeFinley96]

IMPORTANT: PRs without the template will be CLOSED

Description of your pull request and other information

XVideos changed their quickies URL format, as such the extractor needs to be updated. The code has been checked using flake8 and the test has been updated and passes.

This pull request introduces the following modifications:

• Updates the quickies URL transformation to conform with the new XVideos URL format https://www.xvideos.com/video.QUICKIES_ID/_.
• Updates the quickies URL filter to be more lenient, since the quickies URL has quite a few variants (e.g. can optionally include: profiles/, amateur-channels/).
• Updates ID match group to allow for letters as well, since XVideos updated their IDs to include letters.

Example valid URLs with this fix:

• https://www.xvideos.com/lili_love#quickies/a/ipdtikh1a4c
• https://www.xvideos.com/profiles/lili_love#quickies/a/ipdtikh1a4c
• https://www.xvideos.com/amateur-channels/lili_love#quickies/a/ipdtikh1a4c

Fixes #6356

Template

Before submitting a pull request make sure you have:

• At least skimmed through contributing guidelines including yt-dlp coding conventions
• Searched the bugtracker for similar pull requests
• Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

• I am the original author of this code and I am willing to release it under Unlicense
• I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

• Fix or improvement to an extractor (Make sure to add/update tests)
• New extractor (Piracy websites will not be accepted)
• Core bug fix/improvement
• New feature (It is strongly recommended to open an issue first)

[pukkandan]

• (e.g. can optionally include: profiles/, amateur-channels/).

Pls add tests for each. only_matching is fine

[pukkandan]

btw, is there any way to get the old numeric id of a video?

[JakeFinley96]

btw, is there any way to get the old numeric id of a video?

It does seem that you can still use the old video ID for downloading the videos, but the URL format must be the old one apparently... e.g. https://www.xvideos.com/video47258683/_, so It seems that they still have legacy support for the old ID. From what I can tell their whole site has moved over to the new ID so new occurrences of the old ID should not happen.

I went ahead and added support for the old numeric IDs as well, I'm not sure if it's implemented in a way that follows yt-dlp's standards, but it works and the old test passes as well.

Also the video in the old test has an updated ID now in their system, so I'm guessing they completely swapped out their IDs (old test video ID: 47258683, new ID for the same video: idhhckbf429). As such when you go to https://www.xvideos.com/video47258683/_ it turns into https://www.xvideos.com/video.idhhckbf429/verification_video.

[JakeFinley96]

What is the status on getting this merged? Is there something blocking this getting merged?

[JakeFinley96]

Alright the code has been updated to use isdigit() and all tests pass and flake8 shows no problems. Any further changes that are needed? Or can this be merged?

[JakeFinley96]

LGTM 👍