clones/friendica
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #10708 from annando/peertube-videos 

Better handling of displaying peertube videos
This commit is contained in:
Hypolite Petovan 2021-09-14 15:34:30 -04:00 committed by GitHub
parent 67a2444bc8 f99c4ca704
commit 39e4ab89fa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 21 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/Model/Post/Media.php
View File

@ -547,7 +547,7 @@ class Media
return $attachments; return $attachments;
} }
$height = 0; $heights = [];
$selected = ''; $selected = '';
$previews = []; $previews = [];
@ -591,14 +591,11 @@ class Media
in_array($filetype, ['audio', 'image'])) { in_array($filetype, ['audio', 'image'])) {
$attachments['visual'][] = $medium; $attachments['visual'][] = $medium;
} elseif (($medium['type'] == self::VIDEO) || ($filetype == 'video')) { } elseif (($medium['type'] == self::VIDEO) || ($filetype == 'video')) {
if (strpos($medium['url'], $guid) !== false) { if (!empty($medium['height'])) {
// Peertube videos are delivered in many different resolutions. We pick a moderate one. // Peertube videos are delivered in many different resolutions. We pick a moderate one.
// By checking against the GUID we also ensure to only work this way on Peertube posts. // Since only Peertube provides a "height" parameter, this wouldn't be executed
// This wouldn't be executed when someone for example on Mastodon was sharing multiple videos in a single post. // when someone for example on Mastodon was sharing multiple videos in a single post.
if (empty($height) || ($height > $medium['height']) && ($medium['height'] >= 480)) { $heights[$medium['height']] = $medium['url'];
$height = $medium['height'];
$selected = $medium['url'];
}
$video[$medium['url']] = $medium; $video[$medium['url']] = $medium;
} else { } else {
$attachments['visual'][] = $medium; $attachments['visual'][] = $medium;
@ -607,6 +604,15 @@ class Media
$attachments['additional'][] = $medium; $attachments['additional'][] = $medium;
} }
} }
if (!empty($heights)) {
ksort($heights);
foreach ($heights as $height => $url) {
if (empty($selected) || $height <= 480) {
$selected = $url;
}
}
if (!empty($selected)) { if (!empty($selected)) {
$attachments['visual'][] = $video[$selected]; $attachments['visual'][] = $video[$selected];
unset($video[$selected]); unset($video[$selected]);
@ -614,6 +620,8 @@ class Media
$attachments['additional'][] = $element; $attachments['additional'][] = $element;
} }
} }
}
return $attachments; return $attachments;
} }