friendica/friendica-addons
7
2
Fork 13
Code Issues 3 Pull requests 2 Releases 16 Wiki Activity

This addon will replace "youtube.com" with the chosen Invidious instance #1441

Merged
MrPetovan merged 19 commits from :develop into 2023.09-rc 2023-12-08 20:50:23 +01:00
Conversation 3 Commits 19 Files changed 5 -4
1 changed files with 0 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 9f5da18020 - Show all commits

4
invidious/invidious.php
View file

@ -53,10 +53,6 @@ function invidious_render(array &$b)
if (strstr($b['html'], 'https://youtube.com')) {
heluecht commented 2023-12-04 21:41:28 +01:00
Outdated
Review
Copy link

Why is this block duplicated?

Why is this block duplicated?
loma-one commented 2023-12-04 21:49:57 +01:00
Outdated
Review
Copy link

To ensure that different spellings with www and without are intercepted and redirected.

To ensure that different spellings with **www** and without are intercepted and redirected.
MrPetovan commented 2023-12-04 21:51:54 +01:00
Outdated
Review
Copy link

Thanks for the reply, the indentation still needs to be corrected.

Thanks for the reply, the indentation still needs to be corrected.
heluecht commented 2023-12-04 23:08:27 +01:00
Outdated
Review
Copy link

you could combine the str_replace with an array as first parameter.

you could combine the `str_replace` with an array as first parameter.
$b['html'] = str_replace('https://youtube.com', $invidious, $b['html']);
$replaced = true;
}
if (strstr($b['html'], 'https://youtu.be')) {
$b['html'] = str_replace('https://youtu.be', $invidious, $b['html']);
$replaced = true;
}
if ($replaced) {
heluecht commented 2023-12-05 22:14:15 +01:00
Outdated
Review
Copy link

You can simply do

            $b['html'] = str_replace(['https://www.youtube.com', 'https://youtube.com'], $invidious, $b['html']);
You can simply do ``` $b['html'] = str_replace(['https://www.youtube.com', 'https://youtube.com'], $invidious, $b['html']); ```
$b['html'] .= '<hr><p><small>' . DI::l10n()->t('(Invidious addon enabled: YouTube links via %s)', $invidious) . '</small></p>';