[ie/facebook] preserve thumbnail provided in video info #9060
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
Changes to preserve preferred_thumbnail url obtained from video info
because video_info['thumbnail'] is lost during merge_dicts
example: https://www.facebook.com/events/1509582499515440
Two different thumbnails are provided:
1st one: from "og:image" meta tag
2nd one: preferred_thumbnail in ScheduledServerJS json
They will be put into webpage_info['thumbnail'] and video_info['thumbnail'] respectively.
Due to the use of merge_dicts, "merge_dicts(webpage_info, video_info)", webpage_info['thumbnail'] will be put into the merged dict but video_info['thumbnail'] will be ignored. The changes is made in order to keep both.
Template
Before submitting a pull request make sure you have:
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:
What is the purpose of your pull request?