Varauksen noutopaikan muuttaminen asiakkaan tiedoista antaa virheellisen noutopaikkalistan

[MikkoLiimatainen]

Mikä vikana?

Kun varauksen noutopaikkaa yrittää muuttaa asiakkaan tietojen kautta, Kohan noutopaikka-pudotusvalikkon antaa virheellisen listan noutopaikoista. Vaskin tuotantoympäristössä listalle tulee 20 aakkosissa viimeistä noutopaikkaa ja kun listaa selaa alaspäin, haetaan listaan samat 20 noutopaikkaa, kunnes lista on noutopaikkojen määrän 60 mittainen. Testiympäristössä noutopaikkoja on 61 ja noutopaikkalistaan tulee listan viimeinen kirjasto neljä kertaa. Tästä voi päätellä, että noutopaikkalista sivutetaan 20 noutopaikan mittaiseksi ja lista hakee sivutuksen viimeisen sivun uudelleen ja uudelleen, kunnes sivujen määrä on saavutettu.

Viimeistä sivua lukuunottamatta näkyy listalla myös virheviesti.

Noutopaikan voi hakea kirjoittamalla hakukenttään osan noutopaikan nimestä.

Mitä pitäisi tapahtua

Pudotusvalikon pitäisi hakea ensin aakkosten 20 ensimmäistä noutopaikkaa ja sitten seuraavat 20 jne.

Kuinka toistaa ongelma/asia

No response

Selain

No response

Jotain muuta?

Tiketti Bugzillassa: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34068

[pknuuti]

Vaarassa noutopaikkavalikko antaa kaikki noutopaikat, valikko avautuu hissiin. Valikossa on 40 noutopaikkaa.

[pirliis]

OUTIssa noutopaikkavalikko asiakkaan varauksissa näyttää myös oikealta.

[MikkoLiimatainen]

Vaikuttaakohan tuohon sitten jokin asetus, jos se on nyt vain meillä ongelmana?

[emta001]

Vilkaisin tuota ja eihän tuon pitäisi toimia tuolla tavalla sivuttamalla kuten se Vaskissa nyt toimii. Esim. Vaarassa tehdään vain yksi API-kutsu ilman _page parametriä. Mutta enpä nyt ihan äkkiseltään huomaa, että tuohon vaikuttaisi mikään asetuskaan.

[emta001]

Eikun löytyipäs asetus, RESTdefaultPageSize. Se on Vaskissa 20, mutta Vaarassa 80. Kun laitoin tuohon Vaskin testillä 80, kirjastot latautui oikein. Mutta onpa hölmöä, että se vaikuttaa myös tuohon alasvetolaatikkoon :D

[MikkoLiimatainen]

No hyvä että syyllinen löytyi. Mitähän muita vaikutuksia tuolla mahdollisesti on, että uskaltaisiko tuon muuttaa myös tuotantoon? Ilmeisesti se muualla toimii ihan ongelmitta, joten sinällään varmasti uskaltaa muuttaa, mutta ihmetyttää kyllä tuo kytkös tämän asetuksen ja alasvetovalikon kanssa.

[emta001]

Ilmeisesti aina kun tehdään API-kutsu, niin tuo asetus määrittää sen montako tulosta esitetään kerralla. Ja nuo noutopaikathan tulee APIn kautta. Pitänee kaivella yhteisöä taas, löytyisikö sieltä tuohon tiketti tai tehdä semmoinen. Ei ihan loppuun suunniteltu juttu :D

[MikkoLiimatainen]

No, tämä lienee nyt kuitenkin meidän kohdalta ok. Kävin kommentoimassa ratkaisun myös linkkaamaani bugzillan tikettiin, joten mikäli tämän osalta on tarpeen tehdä jatkokehitystä, niin se hoitunee sitä kautta. Suljen tämän tiketin.