Conversation
| videoListe.remove(Resolution.HD); | ||
| } | ||
| } | ||
| // TODO: add more resolutions etc - but for backwards compatibility - set to the current once |
There was a problem hiding this comment.
Tests scheitern wegen dieser Logik.
Vorschlag: in den Crawlern keine Backwards Compatiblity einbauen, sondern erst beim Schreiben der Filmlisten-Datei.
| if (widgets.size() > 1) { | ||
| parseRelatedFilms(filmDto, widgets.get(1).getAsJsonObject()); | ||
| } | ||
| //if (widgets.size() > 1) { |
There was a problem hiding this comment.
ist der auskommentierte Code noch relevant?
There was a problem hiding this comment.
Nein braucht man nicht mehr. Refactor ARD ist ein task welches ich auf die Liste nehmen wollte und jetzt hier nur notdürftig durchgezogen habe
| Map<Resolution, String> newUrls = new EnumMap<>(Resolution.class); | ||
| resolutionUrlMapFromM3U.forEach((key, value) -> newUrls.put(key, value.toString())); | ||
| // | ||
| // TODO: FIXME |
There was a problem hiding this comment.
was ist hier noch zu tun?
Gedanke: die Logik zum Umgang mit fehlenden NORMAL-Urls nicht dezentral im Deserializer einbauen sondern erst wenn Filmlisten-Datei erstellt wird.
There was a problem hiding this comment.
Grundsätzlich Ja aber das führt in der Übergansphase (so lange man noch old format lesen möchte) zu Problemen. Auch alle Merge logiken müssen dann nochmal geprüft wergen weil ggf. auf NORMAL zugegriffen wird. Ich habe schon die getDefaultUrl Methode erweitert um das abzuschwächen - ich glaube aber damit müssen wir uns nochmal beschäftigen.
src/main/java/de/mediathekview/mserver/crawler/ard/json/ArdFilmDeserializer.java
Outdated
Show resolved
Hide resolved
|
|
||
| if (aDTO.getNumberOfClips() > 1) { | ||
| processRelatedFilms(filmDto.getRelatedFilms()); | ||
| //processRelatedFilms(filmDto.getRelatedFilms()); |
There was a problem hiding this comment.
noch relevant oder findet der Crawler mittlerweile über andere Zweige alles?
There was a problem hiding this comment.
Ich denke ja - Es werden jetzt alle Trägerseiten gelesen und im Film model alle Sprachen/subs
src/main/java/de/mediathekview/mserver/crawler/ard/tasks/ArdTopicPageTask.java
Outdated
Show resolved
Hide resolved
| LOG.error("{}", e); | ||
| return resolutions; | ||
| } | ||
| String[] lines = m3uContent.split("\n"); |
There was a problem hiding this comment.
Sie sind zunächst inkompatibel – in Zukunft könnte man jedoch eine gemeinsame Codebase schaffen
src/main/java/de/mediathekview/mserver/crawler/basic/AbstractJsonRestTask.java
Show resolved
Hide resolved
src/test/java/de/mediathekview/mserver/crawler/ard/json/ArdFilmDeserializerTest.java
Outdated
Show resolved
Hide resolved
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| active = false; | ||
| url = "jdbc:postgresql://postgresMV:55432/crawler"; | ||
| username = "crawler"; | ||
| password = "secret"; |
Check failure
Code scanning / SonarCloud
Credentials should not be hard-coded
| active = false; | ||
| url = "jdbc:postgresql://postgresMV:55432/crawler"; | ||
| username = "crawler"; | ||
| password = "secret"; |
Check failure
Code scanning / SonarCloud
Credentials should not be hard-coded High
2 participants
No description provided.