Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default Sortierreihenfolge #169

Closed
imolb opened this issue May 30, 2020 · 19 comments
Closed

Default Sortierreihenfolge #169

imolb opened this issue May 30, 2020 · 19 comments
Labels
done The issue was soled or implemented feature This issue describes a new feature

Comments

@imolb
Copy link

imolb commented May 30, 2020

Danke für die Default-Sortierung aus #71
Ideal wäre es noch, wenn man auch die Richtung der Sortierung per Default vorgeben kann (auf- oder absteigend). Für mich persönlich ist nämlich meist "Sendedatum absteigend" die hilfreichste Sortierung.

@codingPF codingPF added the feature This issue describes a new feature label May 30, 2020
@codingPF
Copy link
Member

Finde ich auch aber ich habe noch keine Weg gefunden das zu beeinflussen.

@codingPF codingPF reopened this May 30, 2020
@Ch1llb0
Copy link
Contributor

Ch1llb0 commented May 31, 2020

Das unterstütze ich... Musste vorher immer beides manuell verstellen. Es ergibt kaum Sinn, aufsteigend nach Datum zu sortieren, wenn es dann auch noch eine zweite Seite gibt. Wäre super, auch dies könnte anpassbar gemacht werden. 👍

@codingPF
Copy link
Member

Ich habe die Frage mal ins Kodi Forum weitergeleitet...denn ich habe den „Schalter“ dafür nicht gefunden

@maxberghammer
Copy link

Ja, das wäre echt saustark, wenn das gehen würde!

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 12, 2020

Ich habe die Frage mal ins Kodi Forum weitergeleitet...denn ich habe den „Schalter“ dafür nicht gefunden

https://forum.kodi.tv/showthread.php?tid=354867

Wir sind gespannt 👍

@codingPF
Copy link
Member

Ich habe jetzt mehrfach versucht auf verschidene Arten die Sortierung dem Container mitzugeben aber leider erfolglos...
Man kann das ganze auch aus dem Plugin selbst erzwingen aber das verursacht andere Problem...
@Ch1llb0,
hast du dazu noch ein gute Idee?

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 15, 2020

Spontan nicht. Eventuell lässt der Blick in den Code anderer Add-ons Schlüsse zu, wie es machbar sein könnte?

Z.B. hat das YouTube-Add-on einen Sortkey 'Standard', also wohl einen eigens vom Add-on definierten, der 'Datum' in umgekehrter Reihenfolge entspricht - also genau dem, was wir suchen. Eventuell kann das übernommen werden? Und nach Datum zu sortieren ist doch eh als Standard sinnvoll, oder?

@codingPF
Copy link
Member

Ich habe mich schon mal durch den Krypton branch gearbeitet aber nichts gefunden.
Es gibt in den Skins ab und an mal sowas
sortorder="$PARAM[sortorder]"
Aber das setzten dieses Properties aus dem Addon code hat leider nicht das gewünschte Ergebnis.

Das Youtube plugin sortiert selber die Listen (im Phyton code) und gibt diese dann als NoSort an Kodi. Das ist jetzt nicht wirklich elegant.

Serien will man auch mal nach Name sortieren da diese oft auch richtig als 1 / 2 / 3 / .. hinterlegt sind.
Die "dauer" Serien habe ich als Suche hinterlegt und dort genau einmal die Sortierung auf DESC gestellt. Danach merkt er sich das....So ganz einfach ist die Lösung nicht...Leider gibt es keinen SORT_BY_AGE...

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 16, 2020

Ich habe mich schon mal durch den Krypton branch gearbeitet aber nichts gefunden.
Es gibt in den Skins ab und an mal sowas
sortorder="$PARAM[sortorder]"
Aber das setzten dieses Properties aus dem Addon code hat leider nicht das gewünschte Ergebnis.

Das ist eine für Skins reservierte Variante, die nur funktioniert, da jeder einzelne Skin diese Vorgaben/Einstellungen in der Datenbank abspeichert pro jeweiligem Fenster oder jewiliger Add-on-Seite/-Ansicht. Dieses Vorgehen funktioniert nicht, wenn ein Add-on etwas global für all seine Ordner vorgeben möchte.

Das Youtube plugin sortiert selber die Listen (im Phyton code) und gibt diese dann als NoSort an Kodi. Das ist jetzt nicht wirklich elegant.

Naja, das könnte ja aber hier bei diesem Add-on standardmäßig auch so gemacht werden. Was spräche dagegen - ob elegant oder nicht?

Serien will man auch mal nach Name sortieren da diese oft auch richtig als 1 / 2 / 3 / .. hinterlegt sind.

Wie beim YouTube Add-on auch kann man das ja weiterhin tun. Dort kann auch weiterhin 'Titel', 'Datum', etc. manuell ausgewählt werden. Aber 'Standard' ist eben besagtes 'NoSort', was aber real 'Datum' in korrekter Reihenfolge entspricht, was auch hier bei diesem Add-on sinniger Standard sein sollte, meiner Meinung nach.

Eventuell ließe sich ja beides kombinieren - die momentane Variante, mit der für das gesamte Add-on ein Sortkey vorgegeben werden kann plus die Variante wie bei YouTube. Somit hätte man Datum in korrekter Reihenfolge plus alle anderen Varianten der Sortierung als Option. Wäre das was?

@codingPF
Copy link
Member

Ich habe es mal versucht (und die Implementierung ist fertig) aber der Teufel steckt im Detail.
Zum einen, gibt es trotz NO_SORT (oder genau genommen UNSORTED) die Möglichkeit ASC und DESC auszuwählen, was erstmal unschön aussieht aber im wahrsten Sinne des Wortes nichts macht.
Schlimmer aber ist…
Wenn man die Sortierung im UI von NO_SORT auf TITLE wechselt, geht das problemlos.
Aber wenn man von TITLE auf NO_SORT / Default wechselt, dann passiert nichts da das UI ja nichts zum sortieren hat.
Macht Sinn….aber ob das für den Benutzer so nachvollziehbar ist?

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 18, 2020

Das ist merkwürdig, da genau diese beiden Vorgänge beim YouTube-Add-on funktionieren. Es kann von 'Default' (no sort) zu 'Title' oder anderem gewechselt werden und wieder zurück - die Sortierung ändert sich dann auch nachvollziehbar. Ebenso kann bei 'Default' (no sort) die Sortierreihenfolge geändert werden (von ASC zu DESC und umgekehrt) und dies hat auch eine sichtbare Auswirkung. Ohne den Code zu kennen - fehlt eventuell noch was, was bei der Methode des YT-Add-ons gemacht wird, um dies beides zu ermöglichen?

@codingPF
Copy link
Member

Schau dir das bitte nochmal genau an.
Wenn ich das bei mir mache (im YT plugin) passiert genau das was ich beschrieben habe.
Die sortierung "Default" kommt so nicht mehr zurück.

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 19, 2020

War mir tatsächlich nie aufgefallen - hast recht. Frage ist nun: Ist es das kleinere Übel, die Sortierung so zu basteln oder Datum immer nur standardmäßig falsch herum (dass das falsch herum ist, darauf können wir uns einigen, oder?) zu präsentiert bekommen?

Falls es wirklich keine andere Lösung für dieses Problem geben sollte... Es kann doch nicht sein, dass Kodi für Add-ons, wenn nach Datum sortiert wird, immer nur die verkehrte Reihenfolge anbietet und Devs keine Möglichkeit bietet, dies zu ändern.

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 20, 2020

Das Problem wurde hier bereits einmal beschrieben: https://forum.kodi.tv/showthread.php?tid=81155

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 20, 2020

Hilft die Aussage im Kodi Forum, über welches Infolabel die momentane Sortierreihenfolge auszulesen ist, bei der Lösung des Problems?

@codingPF
Copy link
Member

Nicht wirklich...auch Programm dafür zu verwenden finde ich nicht so toll.
Ich habe jetzt die angesprochene Version fertig gemacht.
Wer testen möchte, kann dies gerne tun.
Ob das dann in den devBranch oder das nächste Release geht...weiß ich noch nicht so richtig.
plugin.video.mediathekview-0.6.5.zip

@Ch1llb0
Copy link
Contributor

Ch1llb0 commented Jun 21, 2020

Leider scheint wohl die YouTube-Methode die einzige zu sein, die grundlegend funktioniert, auch wenn sie das angesprochene Problem hat. Ist halt ein Kompromiss.

Ein kurzer Test zeigt, dass der Kompromiss allerdings gar nicht so gravierend ist. Zwar kommt man nicht zurück zur 'Default' (no sort) Ansicht durch Togglen des Sortkeys, allerdings bekommt man es hin, indem man 'Default' wählt, kurz in das darüberliegende Verzeichnis und zurück wechselt. Es geht also... Ich denke, dass der Kompromiss so gangbar ist, wenn es von Seiten Kodis einfach keine sinnvollere Lösung gibt. Und von YT dürfte es Nutzern ja bekannt sein.

Die Testversion hat allerdings jetzt nicht mehr den Toggle für den Contenttype. Beabsichtigt?

@codingPF
Copy link
Member

Ja ist Absicht bzw. liegt das einfach nur daran, dass alle Feature noch nicht gemergt sind/waren. Kommt dann alles zusammen wenn ich die Beta baue...

@codingPF codingPF added the done The issue was soled or implemented label Sep 6, 2020
@codingPF
Copy link
Member

Kodi Repo Check-in: Version 0.6.6 ist live

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done The issue was soled or implemented feature This issue describes a new feature
Projects
None yet
Development

No branches or pull requests

4 participants