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.

< Отзив за Skrypt umożliwiający pobieranie materiałów ze znanych serwisów VOD.

Въпрос/коментар

§
Публикуван на: 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

zacnyАвтор
§
Публикуван на: 12.05.2020

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

§
Публикуван на: 18.05.2020
Редактиран на: 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.

§
Публикуван на: 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.

zacnyАвтор
§
Публикуван на: 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.

§
Публикуван на: 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).

Публикувайте отговор

Влезте, за да публикувате отговор.