wisydb / wisy

Open Source CMS for Training and Educational Purposes
Other
1 stars 3 forks source link

Anpassung Ortsuche #11

Closed svenkaemper closed 3 years ago

svenkaemper commented 6 years ago

"Die Ortssuche / Datums-Suche verwirrt regelmäßig, wenn das Listenergebnis eine Durchführung zufällig mit einem anderen Ort oben in der Liste zeigt. Pro Zeile genau den gesuchten Ort / Datum in den Durchführungen suchen und anzeigen. Und ggf. trotzdem die Zeile darunter (XX weitere Orte/Termine)"

svenkaemper commented 6 years ago

@meyway @debagel Hier sollte man vor dem Start die exakten Anforderungen genau klären und die Auswirkungen auf die gesamte Suche (Performance, Filter) prüfen… könnte eine umfangreichere Geschichte werden.

Zudem wird "X weitere" ja bereits bei den Terminen mit angezeigt, in der Spalte "Ort" würde ich da weitere Zusatzinfos eher vermeiden wollen.

wisydb commented 6 years ago

@debagel Was ist da der (Entscheidungs-)Stand? Hatten wir (zwei+Redaktionen) glaube ich schon fertig besprochen, im Sinne von genau die DF suchen, die gefilter wird, weil die Redaktionen das so wollten und ressourcenmäßig nicht so schlimm, wir wollten es ausprobieren meine ich.

Priorität: wichtig :-)

debagel commented 6 years ago

Aktueller Wunsch ist das die Anzeige der Kursliste überarbeitet wird: Bei gefilterter Kursliste soll für jeden Kurs soweit möglich eine der zu den gesetzten Filtern passende Durchführung gefunden und deren Daten für Termin, Dauer, Art, Preis, Ort angezeigt werden. Nicht wie bisher einfach die Daten der ersten DF. Theoretisch sollte es ja immer eine DF geben die auf alle relevanten gesetzten Filter passt.

debagel commented 3 years ago

@wisydb Hier gibt es aus meiner Sicht für die Ortsfilterung zwei technisch sehr unterschiedliche Fälle. Relativ einfach ist der Fall, dass nur nach bei: gefiltert wird ohne km: (Umkreis). Dann sollte es ohne weitere Datenbankabfragen möglich sein die Durchführungsdaten anzuzeigen die zum gefilterten Ort passen.

Wenn aber auch nach Umkreis gesucht wird und für den gefilterten Ort direkt keine DF vorliegt (also nur DF im Umkreis, keine im Ort selbst) ist es nicht mehr so einfach zu entscheiden welche die Naheliegendste ist. Dazu bräuchten wir Koordinaten auf Kursebene analog zu x_kurse_latlng. Den Fall lasse ich also erstmal unberücksichtigt.

debagel commented 3 years ago

@wisydb Im aktuellen DEV-Branch ist das jetzt testbar. Wenn nach Ort gefiltert wurde wird nach einer Durchführung gesucht die im gefilterten Ort stattfindet und diese verwendet statt der ersten DF.

Wenn nach Datum gefiltert wird wird nach einer Durchführung gesucht die am oder nach dem gefilterten Datum stattfindet und diese verwendet statt der ersten DF.

Wenn nach Ort UND Datum gefiltert wird wird trotzdem nur versucht den Ort anzupassen.

debagel commented 3 years ago

@wisydb Bitte noch mal testen. Die Anzeige der zum gesuchten Ort passenden Durchführung sollte jetzt besser klappen.

wisydb commented 3 years ago

Stichwort 1, Stichwort 2, Stichwort n, Ort funktioniert, wie erwartet => die passende DF wird angezeigt.

Umkreis-Suche: Genau - sowohl mit km als auch ohne - unberücksichtigt lassen.

"Wenn nach Datum gefiltert wird wird nach einer Durchführung gesucht die am oder nach dem gefilterten Datum stattfindet und diese verwendet statt der ersten DF." Funktioniert nicht mehr (auch ohne Ort nicht). Funktionierte aber schon mal.

"Theoretisch sollte es ja immer eine DF geben die auf alle relevanten gesetzten Filter passt." Leider nicht, denke ich.

"Wenn nach Ort UND Datum gefiltert wird wird trotzdem nur versucht den Ort anzupassen." Da es die anderen Kriterien noch gibt = suboptimal. Gleichzeitig kann man natürlich nicht "magisch" die am passendste bestimmen, wenn mind. 2 Kriterien gefiltert werden und diese sich auf 2 unterschiedlich verteilen.

Folglich wäre informations-transparent für mehrere Kriterien (s. 19.10.18 oben) nur:

Szenario: Kurs hat 3 DF: DF1: Datum: Jan | Ort: Hamburg DF2: Datum: Feb | Ort: Lübeck DF3: Datum: Mrz | Ort: Bremen

Filter: Datum: Jan, Ort Lübeck

Folgt im Ergebnis in der (selben) Kurszeile für die DF-Spalten Termin und Ort: Jan | Hamburg Feb | Lübeck 1 weiterer... |

debagel commented 3 years ago

@wisydb Bitte noch mal testen. Es sollten jetzt für "Termin" und "Ort" zwei Werte untereinander ausgeben werden wenn nötig, wie in deinem Beispiel oben dargelegt.

wisydb commented 3 years ago

Mit der neuen Funktion funktioniert alles, wie hier beschrieben! Danke.

Kleiner Bug mit dem Link "1 weiterer..." etc. => der ist leer (nach /k kommt keine ID), wenn Du z.B. folgende Anfrage, die Live noch funktioniert, mal in der Sandbox1 testet: sh.kursportal.info/search?qs=P%E4dagogik+f%FCr+Ganztagsschulen%2C+Pr%E4senzunterricht%2C+Datum%3A27.08.2021&q=P%E4dagogik+f%FCr+Ganztagsschulen%2C+Pr%E4senzunterricht%2C+datum%3A18.02.2022&qf=