Skrypt umożliwiający pobieranie materiałów ze znanych serwisów VOD.

Skrypt służący do pobierania materiałów ze znanych serwisów VOD.

< Rückmeldungen auf Skrypt umożliwiający pobieranie materiałów ze znanych serwisów VOD.

Frage/Kommentar

§
Veröffentlicht: 12.05.2020

Drobne usprawnienie gdy nazwa zawiera wiecej niż jeden niedozwolony znak

Gdy nazwa video zawiera więcej niż jeden niedozwolony znak nie można pobierać pliku. Naciśnięcie na przycisk pobierz nie otwiera okna do zapisu pliku. Rozwiązaniem jest drobna modyfikacja linii 83 do postaci: movieTitle = movieTitle.replace(new RegExp(config.notAllowedFileNameCharsMask,'g'), '');

Przykład wideo gdzie można zaobserwować problem: https://sport.tvp.pl/39661427/kapitan-tsubasa-odc-120-biegniemy-po-nasze-marzenia

zacnyVerfasser
§
Veröffentlicht: 12.05.2020

Dziękuję za sugestię. Proszę sprawdzić obecną wersję 6.15.5.

§
Veröffentlicht: 18.05.2020
Bearbeitet: 18.05.2020

W 6.15.5 jest gorzej. String notAllowedFileNameCharsMask w ogóle nie jest teraz maską wyrażenia regularnego. Otoczenie config.notAllowedFileNameCharsMask za pomocą new RegExp nie pomaga. Sugeruje wrócić do poprzedniej wersji z naniesioną zmianą w formie, którą zaproponowałem.

§
Veröffentlicht: 18.05.2020

mozesz tez zerknąć na https://github.com/IonicaBizau/regex-parser.js, zamienia stringa na wyrażenie regularne i wtedy config.notAllowedFileNameCharsMask mogłby zostać jak go zdefiniowałeś w 6.15.5. Pytanie czy warto dorzucać dodatkową logikę do jednego użycia.

zacnyVerfasser
§
Veröffentlicht: 18.05.2020

Dzięki za pomoc. Zrobiłem sobie testy tego wyrażenie i faktycznie było do bani. Domyślam się, że moje wcześniejsze testy nie były miarodajne - filtr chyba zdaje się nie mieć znaczenia na linuksie.

§
Veröffentlicht: 20.05.2020

Cóż linux sporo więcej wybacza w nazwach plików. Warto tu dodać jakiś test automatyczny by upewnić się że RegExp działa jak należy ;-). Link podany w pierwotnym poście dobrze nadaje się do testów (lub jakiś inny dobrany z TVP sport bo tam cudzysłów występuje często).

Antwort schreiben

Anmelden um eine Antwort zu senden.