search

INL / GTB-html

GTB zonder Flash
0 stars 0 forks source link

SORTEEROPTIES BIJ RESULTATEN: AANTAL ONTBREKEN #76

Closed JessedeDoes closed 6 years ago

JessedeDoes commented 6 years ago

Mijn fout, ik had het FO erbij moeten halen. Maar een gebruiker wijst ons erop dat we een aantal sorteeropties missen. Het functioneel ontwerp zegt:

De gebruiker kan het tussenresultaat sorteren. Er kunnen maximaal 4 sorteersleutels opgegeven worden, en er kan gekozen worden voor alfabetisch oplopend dan wel aflopend. De informatie waarop gesorteerd kan worden is: “Woordenboek”, “Lemma”, “Modern Nederlands lemma”, “Woordsoort”, “Auteur en titel citaat”, “Datering citaat”, Lokalisering citaat”. We missen de laatste drie, en die zijn essentieel. Een gebruiker wees ons erop.

jan-niestadt commented 6 years ago

Zijn het er niet vier dan? Dit zijn nl. de waardes die je aan het backend zou moeten doorgeven:

auteur (Auteur citaat) bron (Titel bron citaat) datering (Datering citaat) loc (Localisering citaat)

jan-niestadt commented 6 years ago

Ik voeg ze wel even toe BTW.

jan-niestadt commented 6 years ago

Ik heb ze op de ontwikkelserver (niet de ATO) toegevoegd, maar sorteren lijkt uberhaupt niet te werken. Als ik bijv. zoek op woord in citaat "test", dan klik op Sorteren, dan bij eerste sleutel "Aantal concordanties" kies, krijg ik de melding "Selecteer a.u.b. tenminste een woordenboek." (andere opties geven zo te zien hetzelfde resultaat)

jan-niestadt commented 6 years ago

Pieter, misschien wil jij even kijken waarom sorteren niet meer lijkt te werken?

(BTW, ik heb het niet kapotgemaakt volgens mij, ook op de ATO (waar ik nu niets aan veranderd heb) werkt het niet)

pmasereeuw commented 6 years ago

Het sorteren gebeurt op de back-end, dus er wordt eigenlijk een nieuwe query afgevuurd. Daarbij wordt gegeken welke woordenboeken geselecteerd zijn, maar op het resultatentabblad ontbreken die checkboxes natuurlijk. Ik zal eens kijken wat er aan te doen is.

De oorsprong van dit issue, de ontbrekende sorteeropties, snap ik nog niet. Ik zie die gewoon staan, zij het iets anders verwoord dan in het FO dat Jesse aanhaalt.

pmasereeuw commented 6 years ago

Branch issue-76 bevat de fix voor "selecteeer tenminste één woordenboek" (de oorzaak was toch iets anders dan hierboven geschetst). De oorspronkelijke vraag moet nog worden onderzocht.

jan-niestadt commented 6 years ago

Wat betreft de sorteeropties: ik had die vier toegevoegd naar aanleiding van de melding, zie commit 8e8c60a7a6.

jan-niestadt commented 6 years ago

Katrien, is dit niet opgelost?

kdepuydt commented 6 years ago

Hoi, ik had per ongelijk in productie gekeken. Echter, als ik een zoekresultaat (zoek naar help in citaten en kies weergave citaten in uitgebreid zoeken ) wil sorteren, eerste sleutel datering, tweede localisering, dan krijg ik de volgende melding: image

jan-niestadt commented 6 years ago

Als ik dit op de ATO test, werkt het bij mij. De "Selecteer a.u.b. tenminste een woordenboek" verschijnt niet meer. Pieter had die ook opgelost volgens mij.