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.

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

Question / commentaire

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

zacnyAuteur
§
Posté le: 12/05/2020

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

§
Posté le: 18/05/2020
Édité le: 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.

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.

zacnyAuteur
§
Posté le: 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.

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).

Poster une réponse

Connectez-vous pour poster une réponse.