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

invidious/invidious.php aktualisiert #1497

Open
loma-one wants to merge 11 commits from loma-one/friendica-addons:develop into develop
pull from: loma-one/friendica-addons:develop
merge into: friendica:develop
Conversation 4 Commits 11 Files changed 7 +2 -1
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit fe7cb760cd - Show all commits

3
invidious/invidious.php
View file

@ -5,7 +5,7 @@
* Version: 0.5
* Author: Matthias Ebers <https://loma.ml/profile/feb>
* Author: Michael Vogel <https://pirati.ca/profile/heluecht>
* Status:
loma-one marked this conversation as resolved Outdated
MrPetovan commented 2024-04-16 18:49:10 +02:00
Outdated
Review
Copy link

This was added by us as we encourage people to use the url_replace addon instead. Please leave this label in place. I do not mind you adding changes to this addon but it isn't meant to be officially supported.

This was added by us as we encourage people to use the `url_replace` addon instead. Please leave this label in place. I do not mind you adding changes to this addon but it isn't meant to be officially supported.
loma-one commented 2024-04-16 19:12:39 +02:00
Outdated
Review
Copy link

The addon url_replace is a deterioration in terms of self-administration by users. The feature of being able to specify their own server was very important to them. Many users operate their own invidious servers or have an account on a server other than the one I have preset that they want to use. In addition, it relieves me in the event of an error (accessibility of invidious), as users can switch off independently.

A compromise suggestion: leave it active until the self-administration function desired by the users has been retrofitted in url_replace. In this case, the admin can continue to combine the addon. I would be happy to post an issue on this.

The addon url_replace is a deterioration in terms of self-administration by users. The feature of being able to specify their own server was very important to them. Many users operate their own invidious servers or have an account on a server other than the one I have preset that they want to use. In addition, it relieves me in the event of an error (accessibility of invidious), as users can switch off independently. A compromise suggestion: leave it active until the self-administration function desired by the users has been retrofitted in url_replace. In this case, the admin can continue to combine the addon. I would be happy to post an issue on this.
MrPetovan commented 2024-04-16 19:18:35 +02:00
Outdated
Review
Copy link

This addon isn't going anywhere yet, and admins can choose to show unsupported addons in the list by setting system.show_unsupported_addons. We made the decision of favoring url_replace over the invidious and nitter addons, so it won't be reinstated.

This addon isn't going anywhere yet, and admins can choose to show unsupported addons in the list by setting `system.show_unsupported_addons`. We made the decision of favoring `url_replace` over the `invidious` and `nitter` addons, so it won't be reinstated.
* Status: Unsupported
* Note: Please use the URL Replace addon instead
*/
@ -97,6 +97,7 @@ function invidious_render(array &$b)
$b['html'] = preg_replace("~https?://(?:www\.)?youtube\.com/embed/(.*?)~ism", $server . '/embed/$1', $b['html']);
$b['html'] = preg_replace("~https?://(?:www\.)?youtube\.com/shorts/(.*?)~ism", $server . '/shorts/$1', $b['html']);
$b['html'] = preg_replace ("/https?:\/\/music.youtube.com\/(.*?)/ism", $server . '/watch?v=$1', $b['html']);
$b['html'] = preg_replace ("/https?:\/\/m.youtube.com\/(.*?)/ism", $server . '/watch?v=$1', $b['html']);
$b['html'] = preg_replace("/https?:\/\/youtu.be\/(.*?)/ism", $server . '/watch?v=$1', $b['html']);
if ($original != $b['html']) {