[Bug in 13.6.0] Speichern von Zeiten (Dauer) im Filterdialog

[cicero]

Sorry, bevor man rügt:

Getestet auf Windows 10 64Bit 1909 unter Nutzung des mitgelieferten Javas.

cicero

[Ein ehemaliger Benutzer]

@le4536
Sehr konstruktive Kritik, alle Achtung.
Warum denn in die Ferne schweifen, denn die Lösung liegt so nah…

[Ein ehemaliger Benutzer]

@cicero Lade doch mal mal den Inhalt deiner settings.xml zwischen
<filter> und
</filter>
als Text hoch. Daraus kann man möglicherweise mehr entnehmen, als aus den Bildern.

Ich habe auch mit den Filtern mal rumgespielt und dabei festgestellt, dass man zum Wechseln immer erst warten muss, bis alle Änderungen durchgeführt wurden, das dauerte meist ein paar bis 15 Sekunden, vor Allem, wenn man die komplette Filmliste ohne weitere Einschränkungen bearbeiten lässt.

[MenchenSued]

@cicero
Ich habe bei mir mit zwei Filtern gespielt und kann das Problem nicht nachvollziehen. Mir ist allerdings etwas aufgefallen, das die Entwickler (z.B. @DerReisende77) verbessern könnten:

Wenn ich Minimalzeit=27 Minuten einstelle, wird in settings.xml irgendeine Gleitkommazahl eingetragen die von der exakten Position des Schiebereglers abhängt. Die Suche scheint die Nachkommastellen zu ignorieren, so dass hier kein Bug vorliegt.

Wenn ich einen neuen Filter anlege, steht der Maximalwert auf unendlich und in settings.xml steht nichts. Wurde der Maximalwert aber mal verändert und dann wieder auf unendlich gestellt, steht 110 Minuten in settings.xml. Auch hier scheint die Suche eine Ausnahme zu machen, denn auch längere Sendungen werden gefunden.

Ich bin kein Freund von festen Ausnahmen, da man beim Erweitern der Funktionen gern etwas übersieht. So ist eventuell auch zu erklären, dass Sendungen ohne Längenangabe nur gefunden werden, wenn Minimalzeit auf 0 steht. Nebenbei bin ich auch kein Freund der Schieberegler, aber das ist eher ein persönliches Problem.

[Ein ehemaliger Benutzer]

@MenchenSued sagte in [Bug in 13.6.0] Speichern von Zeiten (Dauer) im Filterdialog:

Wenn ich einen neuen Filter anlege, steht der Maximalwert auf unendlich und in settings.xml steht nichts. Wurde der Maximalwert aber mal verändert und dann wieder auf unendlich gestellt, steht 110 Minuten in settings.xml. Auch hier scheint die Suche eine Ausnahme zu machen, denn auch längere Sendungen werden gefunden.

Der Wert 110 scheint der Maximalwert zu sein, denn beim Verschieben des Spinners kann man 110 gar nicht auswählen, nach 109 kommt das Unendlich-Zeichen. Dieses Verhalten war auch schon bei den Vorversionen.

[DaDirnbocher]

Ich kann die Beobachtung von @cicero bestätigen und vielleicht etwas detailierter beschreiben.

Das Problem dürfte dann auftreten, wenn man von einem Filter mit einer Mindestdauer > 0 auf einen Filter mit Maximaldauer < unendlich und wieder zurückwechselt. Und zwar auch nur dann, wenn die Maximaldauer des zweiten kleiner ist, als die Mindestdauer des ersten.

Der Wechsel zwischen Filtern mit unterschiedlicher Minimalzeit oder wenn Maxdauer des zweiten > Mindauer des ersten funktioniert problemlos.

Konkret:
Filter 1: 30 - unendlich
Filter 2: 0 - 20
Filter 3: 40 - unendlich
Filter 4: 0 - 50

Wechsel:

• 1 -> 3 -> 1: ok
• 1 -> 4 -> 1: ok
• 1 -> 2 -> 1: Filter 1 hat als Mindestdauer plötzlich 20.

[MenchenSued]

@DaDirnbocher
Hast recht. Mit Filter 1 und Filter 2 kann ich es auch hinstellen.

[cicero]

Sorry, dass ich mich erst jetzt melde. Familie!?

@DaDirnbocher
Danke für deine Unterstützung und Entschuldigung für meine unbeholfene Beschreibung der Testvorgegensweise.

Ich dachte schon, dass das irgendein Seiteneffekt bei mir ist - aus welchem Grund auch immer. Sprich, nicht nur ICH habe ein “Problem” sondern WIR haben ein “Problem”. Es ist zwar blöd, aber es ist schon besser, wenn man einen Sachverhalt reproduzieren kann, statt mit langen Stangen im Nebel zu stochern.

Darf ich vorwitzigerweise fragen, wie es jetzt weitergeht?

@blubber
Ist der Auszug aus der settings.xml zwischen
<filter> und
</filter>
jetzt noch von Belang?

cicero

[Ein ehemaliger Benutzer]

@cicero sagte in [Bug in 13.6.0] Speichern von Zeiten (Dauer) im Filterdialog:

Sorry, dass ich mich erst jetzt melde. Familie!?

@DaDirnbocher
Danke für deine Unterstützung und Entschuldigung für meine unbeholfene Beschreibung der Testvorgegensweise.

Ich dachte schon, dass das irgendein Seiteneffekt bei mir ist - aus welchem Grund auch immer. Sprich, nicht nur ICH habe ein “Problem” sondern WIR haben ein “Problem”. Es ist zwar blöd, aber es ist schon besser, wenn man einen Sachverhalt reproduzieren kann, statt mit langen Stangen im Nebel zu stochern.

Darf ich vorwitzigerweise fragen, wie es jetzt weitergeht?

@blubber
Ist der Auszug aus der settings.xml zwischen
<filter> und
</filter>
jetzt noch von Belang?

Das musst du wissen.

Mich würde an den Werten, Einträgen und der Reihenfolge interessieren, ob das von Belang sein könnte.

[cicero]

@blubber
Dein Wunsch ist mir BefehL.

settings.xml
-----------------------------snip---------------------------------
<filter>
<filter_0d5b724e-88d2-4603-a17a-682775bf244f>
<dont_show>
<audio_versions>true</audio_versions>
<sign_language>true</sign_language>
<trailers>true</trailers>
</dont_show>
<film_length>
<max>10.588235294117643</max>
<min>0.0</min>
</film_length>
<show>
<hd_only>false</hd_only>
<livestreams_only>false</livestreams_only>
<new_only>false</new_only>
<subtitles_only>false</subtitles_only>
<book_marked_only>false</book_marked_only>
</show>
</filter_0d5b724e-88d2-4603-a17a-682775bf244f>
<available>
<filters>
<filter_0d5b724e-88d2-4603-a17a-682775bf244f>Alle Filme (0 - 10 Min)</filter_0d5b724e-88d2-4603-a17a-682775bf244f>
<filter_488c9375-d26a-4157-99cd-d9e9b64ddc8f>Nur neue Filme (>38 Min)</filter_488c9375-d26a-4157-99cd-d9e9b64ddc8f>
<filter_5cf98891-43df-45b7-969e-c88d0e12f786>Alle Filme (>38 Min)</filter_5cf98891-43df-45b7-969e-c88d0e12f786>
<filter_3d54d09d-9376-4a8a-b5ee-8abbe3f5eb9f>Alle Filme (0 - ∞ Min)</filter_3d54d09d-9376-4a8a-b5ee-8abbe3f5eb9f>
<filter_6a86ea53-544d-4d51-b76d-0a00b4e51f44>Alle Filme (10 - 25 Min)</filter_6a86ea53-544d-4d51-b76d-0a00b4e51f44>
</filters>
</available>
<filter_488c9375-d26a-4157-99cd-d9e9b64ddc8f>
<dont_show>
<trailers>true</trailers>
<sign_language>true</sign_language>
<audio_versions>true</audio_versions>
</dont_show>
<film_length>
<min>38.08823529411764</min>
<max>110.0</max>
</film_length>
<show>
<new_only>true</new_only>
<book_marked_only>false</book_marked_only>
</show>
</filter_488c9375-d26a-4157-99cd-d9e9b64ddc8f>
<filter_5cf98891-43df-45b7-969e-c88d0e12f786>
<dont_show>
<trailers>true</trailers>
<sign_language>true</sign_language>
<audio_versions>true</audio_versions>
</dont_show>
<film_length>
<min>38.38714833759591</min>
<max>110.0</max>
</film_length>
<show>
<book_marked_only>false</book_marked_only>
<new_only>false</new_only>
</show>
<zeitraum>∞</zeitraum>
</filter_5cf98891-43df-45b7-969e-c88d0e12f786>
<filter_3d54d09d-9376-4a8a-b5ee-8abbe3f5eb9f>
<dont_show>
<trailers>true</trailers>
<sign_language>true</sign_language>
<audio_versions>true</audio_versions>
</dont_show>
<film_length>
<max>110.0</max>
<min>0.0</min>
</film_length>
<show>
<hd_only>false</hd_only>
<new_only>false</new_only>
<book_marked_only>false</book_marked_only>
</show>
</filter_3d54d09d-9376-4a8a-b5ee-8abbe3f5eb9f>
<filter_6a86ea53-544d-4d51-b76d-0a00b4e51f44>
<dont_show>
<trailers>true</trailers>
<sign_language>true</sign_language>
<audio_versions>true</audio_versions>
</dont_show>
<film_length>
<max>25.294117647058826</max>
<min>10.16304347826087</min>
</film_length>
<show>
<new_only>false</new_only>
</show>
</filter_6a86ea53-544d-4d51-b76d-0a00b4e51f44>
<current>
<filter>5cf98891-43df-45b7-969e-c88d0e12f786</filter>
</current>
</filter>
----------------------------------snip------------------------------------------

voilà

cicero

[Ein ehemaliger Benutzer]

@cicero sagte in [Bug in 13.6.0] Speichern von Zeiten (Dauer) im Filterdialog:

voilà

voilá

Danke. Das werde ich mir ansehen, es kann aber dauern. Ich werde die UUID mal in reale Zeiten umrechnen um zu sehen, ob sich da evtl. was überschnitten hat. Für heute ist mir die Lust vergangen.

[Nicklas2751]

@DaDirnbocher sagte in [Bug in 13.6.0] Speichern von Zeiten (Dauer) im Filterdialog:

Ich kann die Beobachtung von @cicero bestätigen und vielleicht etwas detailierter beschreiben.

Das Problem dürfte dann auftreten, wenn man von einem Filter mit einer Mindestdauer > 0 auf einen Filter mit Maximaldauer < unendlich und wieder zurückwechselt. Und zwar auch nur dann, wenn die Maximaldauer des zweiten kleiner ist, als die Mindestdauer des ersten.

Der Wechsel zwischen Filtern mit unterschiedlicher Minimalzeit oder wenn Maxdauer des zweiten > Mindauer des ersten funktioniert problemlos.

Konkret:
Filter 1: 30 - unendlich
Filter 2: 0 - 20
Filter 3: 40 - unendlich
Filter 4: 0 - 50

Wechsel:

• 1 -> 3 -> 1: ok
• 1 -> 4 -> 1: ok
• 1 -> 2 -> 1: Filter 1 hat als Mindestdauer plötzlich 20.

Ok, dass konnte ich jetzt auch nachstellen. Ich gucks mir mal an.

https://github.com/mediathekview/MediathekView/issues/530

Nicklas2751 created this issue in mediathekview/MediathekView

closed Filter für mindest- & maximallänge beeinflussen sich bei Wechsel #530

[Ein ehemaliger Benutzer]

@Nicklas2751
Meine Analyse kommt zu folgendem Ergebnis:

Wenn der Wert, der im
<current><filter>UUID
als <max> eingetragen ist, kleiner ist als der Wert, der im zu wechselnden Filter als <min>-Wert gelten soll, dann wird dort fälschlicher Weise der <max>-Wert aus dem <current> filter verwendet und auch in settings.xml eingetragen.
HTH

[MenchenSued]

@Nicklas2751
Danke für die Korrektur im Nightly. Nun funktionieren die Längenslider im Filter wie gewünscht.
Kümmerst Du Dich auch um “Sendungen ohne Länge werden nicht gefunden” oder ist das mehr die Domäne von @DerReisende77 ? Da muss vermutlich in der Suchfunktion im Filmtab was erweitert werden, bei den Abos ist das schon korrekt implementiert.