Projekti

Yleinen

Profile

Bugi #5322

Sivutus tietueen Varaukset-sivulle

Lisännyt Anni Rajala noin 2 kuukautta sitten. Päivitetty 6 päivää sitten.

Tila:
Ratkaisu ehdotettu
Prioriteetti:
Välitön
Luokka:
Siirretään
Kohdeversio:
-
Alku:
1. Huhtikuuta 2022
Määräaika:
% Tehty:

80%

Arvioitu aika:
Yhteyshenkilö:
Tiketti Bugzillassa:

Kuvaus

Sivutus tietueen Varaukset-sivulla on Koha-Suomi -lisäys ja tarvitaan myös uuteen versioon, sillä sivun lataaminen on muutoin todella hidasta.

revert.png (31,8 KB) revert.png Anneli Österman, 12. Toukokuuta 2022 15:44
varaustenmaara1.PNG (2,8 KB) varaustenmaara1.PNG Pirkko-Liisa Lauhikari, 13. Toukokuuta 2022 12:41
varaustenmaara2.PNG (2,78 KB) varaustenmaara2.PNG Pirkko-Liisa Lauhikari, 13. Toukokuuta 2022 12:43
5446
5447
5448

Liittyvät tapahtumat

liittyy Versionvaihto - Bugi #5323: Error 500 tietueen varaukset -sivulla kun varauksia 1000<Suljettu 4. Huhtikuuta 2022

Historia

#1 Updated by Anneli Österman noin 2 kuukautta ago

  • liittyy Bugi #5323: Error 500 tietueen varaukset -sivulla kun varauksia 1000< added

#2 Updated by Anneli Österman noin 2 kuukautta ago

  • Prioriteetti changed from Normaali to Välitön

#3 Updated by Johanna Räisä 29 päivää ago

  • Tila changed from Uusi to Työn alla
  • Nimetty set to Johanna Räisä

#4 Updated by Johanna Räisä 29 päivää ago

  • Tiketti Bugzillassa set to 23269

#5 Updated by Johanna Räisä 18 päivää ago

  • Tila changed from Työn alla to Ratkaisu ehdotettu
  • % Tehty changed from 0 to 80

Nyt on testeillä uusi sivutus, toimii vain englanninkielisessä, ku ei ole käännöksiä ajettu. Ladatakaa sivu ctrl+f5:llä, jotta taulukko tulee näkyviin.

Rajasin vielä pois sen jos haluaa muokata useamman teoksen varauksia kerralla, se on hirvee himmeli. Tuossakin on vielä pienien yksityiskohtien säätöä, mutta onpahan nyt suurin osa tehty ja saadaan testiin. Toivottavasti kelpaa yhteisöllekin. :)

#6 Updated by Anni Rajala 18 päivää ago

Minusta itse sivutus toimii (ja nyt myös yli 1000 varausta sisältävän tietueen varaukset saa auki), mutta jos en kuvittele niin nämä ongelmat ilmaantuivat sivutuksen myötä (tai sitten olen ollut aiemmin hyvin sokea :D) :

- varausten noutopaikat ei voi millään pitää paikkaansa, esim. https://vaski-test.koha-suomi.fi/cgi-bin/koha/reserve/request.pl?biblionumber=4135795 melkein kaikkien varausten noutopaikkana (ekana aakkosissa oleva) Kaarina, Auranlaakson koulukirjasto
- myös jos tallentaa uuden varauksen, tallentuu sille tuo Auranlaakson koulukirjasto vaikka olisi valinnut jotain muuta
- viimeisen varauksen jononumero on 0 samassa esimerkkitietueessa

#7 Updated by Johanna Räisä 18 päivää ago

Anni Rajala kirjoitti:

Minusta itse sivutus toimii (ja nyt myös yli 1000 varausta sisältävän tietueen varaukset saa auki), mutta jos en kuvittele niin nämä ongelmat ilmaantuivat sivutuksen myötä (tai sitten olen ollut aiemmin hyvin sokea :D) :

- varausten noutopaikat ei voi millään pitää paikkaansa, esim. https://vaski-test.koha-suomi.fi/cgi-bin/koha/reserve/request.pl?biblionumber=4135795 melkein kaikkien varausten

noutopaikkana (ekana aakkosissa oleva) Kaarina, Auranlaakson koulukirjasto

- myös jos tallentaa uuden varauksen, tallentuu sille tuo Auranlaakson koulukirjasto vaikka olisi valinnut jotain muuta
- viimeisen varauksen jononumero on 0 samassa esimerkkitietueessa

Selvä, katselen mitä siellä tapahtuu.

#8 Updated by Johanna Räisä 18 päivää ago

Johanna Räisä kirjoitti:

Anni Rajala kirjoitti:

Minusta itse sivutus toimii (ja nyt myös yli 1000 varausta sisältävän tietueen varaukset saa auki), mutta jos en kuvittele niin nämä ongelmat ilmaantuivat sivutuksen myötä (tai sitten olen ollut aiemmin hyvin sokea :D) :

- varausten noutopaikat ei voi millään pitää paikkaansa, esim. https://vaski-test.koha-suomi.fi/cgi-bin/koha/reserve/request.pl?biblionumber=4135795 melkein kaikkien varausten

noutopaikkana (ekana aakkosissa oleva) Kaarina, Auranlaakson koulukirjasto

- myös jos tallentaa uuden varauksen, tallentuu sille tuo Auranlaakson koulukirjasto vaikka olisi valinnut jotain muuta
- viimeisen varauksen jononumero on 0 samassa esimerkkitietueessa

Selvä, katselen mitä siellä tapahtuu.

Selvisi ainakin nuo kirjastopisteet, Kohan REST tarjoilee oletuksena vain 20 pistettä, joten Turku ei mahdu enää. Pitää muuttaa, että listaa kaikki.

#9 Updated by Päivi Knuutinen 18 päivää ago

Varaukset näkyvät nyt Varauksia-välilehdellä englanninkielisellä puolella. Oli vähän hidas latautumaan (varauksia 431)
Sija jonossa (priority) valikko ei avautunut ollenkaan. Nimekkeessä, jossa oli vähemmän varauksia, valikko avautui eli syynä oli varmaankin hitaus?

#10 Updated by Katri Sillgren 18 päivää ago

Päivi Knuutinen kirjoitti:

Varaukset näkyvät nyt Varauksia-välilehdellä englanninkielisellä puolella. Oli vähän hidas latautumaan (varauksia 431)
Sija jonossa (priority) valikko ei avautunut ollenkaan. Nimekkeessä, jossa oli vähemmän varauksia, valikko avautui eli syynä oli varmaankin hitaus?

Katsoin Hellessä tietuetta, jolla on vain 8 varausta. Priority-valikko näkyvillä. MUTTA vaikuttaa vaaralliselta ;) Varauksen järjestyksen saa muutettua tuosta. En tosin ole vielä testannut, onko asiakaspalvelutunnuksella samoin silloin, kun tunnuksella ei ole oikeutta muuttaa varausjärjestystä.

#11 Updated by Anneli Österman 18 päivää ago

Johanna Räisä kirjoitti:

Selvisi ainakin nuo kirjastopisteet, Kohan REST tarjoilee oletuksena vain 20 pistettä, joten Turku ei mahdu enää. Pitää muuttaa, että listaa kaikki.

Liittyykö tämä(kin) tähän asetukseen: RESTdefaultPageSize REST API superlibrarianseissa

#12 Updated by Anneli Österman 18 päivää ago

Katri Sillgren kirjoitti:

Katsoin Hellessä tietuetta, jolla on vain 8 varausta. Priority-valikko näkyvillä. MUTTA vaikuttaa vaaralliselta ;) Varauksen järjestyksen saa muutettua tuosta. En tosin ole vielä testannut, onko asiakaspalvelutunnuksella samoin silloin, kun tunnuksella ei ole oikeutta muuttaa varausjärjestystä.

Ainakin aiemmin priority-valikon käyttäminen on vaatinut modify_holds_priority-käyttäjäoikeuden. Sitä tuskin on kovin monella muulla kuin pääkäyttäjillä. Tai näin ainakin OUTIssa. :)

#13 Updated by Johanna Räisä 18 päivää ago

Päivi Knuutinen kirjoitti:

Varaukset näkyvät nyt Varauksia-välilehdellä englanninkielisellä puolella. Oli vähän hidas latautumaan (varauksia 431)
Sija jonossa (priority) valikko ei avautunut ollenkaan. Nimekkeessä, jossa oli vähemmän varauksia, valikko avautui eli syynä oli varmaankin hitaus?

Tää kuulostaa, että vieläkin on se vanha tapa. Tuleeko siihen se Processing-ilmotus?

#14 Updated by Johanna Räisä 18 päivää ago

Anneli Österman kirjoitti:

Katri Sillgren kirjoitti:

Katsoin Hellessä tietuetta, jolla on vain 8 varausta. Priority-valikko näkyvillä. MUTTA vaikuttaa vaaralliselta ;) Varauksen järjestyksen saa muutettua tuosta. En tosin ole vielä testannut, onko asiakaspalvelutunnuksella samoin silloin, kun tunnuksella ei ole oikeutta muuttaa varausjärjestystä.

Ainakin aiemmin priority-valikon käyttäminen on vaatinut modify_holds_priority-käyttäjäoikeuden. Sitä tuskin on kovin monella muulla kuin pääkäyttäjillä. Tai näin ainakin OUTIssa. :)

Joo, pitää jostain kaivaa se tuohon mukaan.

#15 Updated by Johanna Räisä 18 päivää ago

Anneli Österman kirjoitti:

Johanna Räisä kirjoitti:

Selvisi ainakin nuo kirjastopisteet, Kohan REST tarjoilee oletuksena vain 20 pistettä, joten Turku ei mahdu enää. Pitää muuttaa, että listaa kaikki.

Liittyykö tämä(kin) tähän asetukseen: RESTdefaultPageSize REST API superlibrarianseissa

Muokkasin sitä hieman ja ainakin testilläni tuli kaikki, mutta ainakin Vaskissa lista on vajaa. Pitää tutkailla mikä on eritavalla.

#16 Updated by Anneli Österman 18 päivää ago

Johanna Räisä kirjoitti:

Anneli Österman kirjoitti:

Johanna Räisä kirjoitti:

Selvisi ainakin nuo kirjastopisteet, Kohan REST tarjoilee oletuksena vain 20 pistettä, joten Turku ei mahdu enää. Pitää muuttaa, että listaa kaikki.

Liittyykö tämä(kin) tähän asetukseen: RESTdefaultPageSize REST API superlibrarianseissa

Muokkasin sitä hieman ja ainakin testilläni tuli kaikki, mutta ainakin Vaskissa lista on vajaa. Pitää tutkailla mikä on eritavalla.

Sinnehään pitäis tulla vain ne kirjastot, jotka on merkitty noutokirjastoksi. Oisko osa merkitty Vaskissa ei-noutokirjastoksi?

#17 Updated by Anneli Österman 18 päivää ago

Yritin muuttaa yhden varauksen priorityn valikosta ykköseksi, niin tuli poppari "There was an error:error Bad Request". Nuolista klikkaamalla varauksen sai siirrettyä ykköseksi.

#18 Updated by Anneli Österman 18 päivää ago

Puuttuuko Suspend/Unsuspend-nappi?

#19 Updated by Johanna Räisä 15 päivää ago

Johanna Räisä kirjoitti:

Anneli Österman kirjoitti:

Johanna Räisä kirjoitti:

Selvisi ainakin nuo kirjastopisteet, Kohan REST tarjoilee oletuksena vain 20 pistettä, joten Turku ei mahdu enää. Pitää muuttaa, että listaa kaikki.

Liittyykö tämä(kin) tähän asetukseen: RESTdefaultPageSize REST API superlibrarianseissa

Muokkasin sitä hieman ja ainakin testilläni tuli kaikki, mutta ainakin Vaskissa lista on vajaa. Pitää tutkailla mikä on eritavalla.

Nyt pitäisi toimia, näköjään siihen API-kutsuun voi laittaa arvon millä saa kaikki.

#20 Updated by Johanna Räisä 15 päivää ago

Anneli Österman kirjoitti:

Puuttuuko Suspend/Unsuspend-nappi?

Puuttuu, koska tulkitsin siitä vanhasta koodista, että jos on tuo AutoResumeSuspendedHolds, niin sitten lisää aina kentän päivämäärän. Tuossa se päivittää sen kun valitsee vain päivämäärän, turhaa alkaa painelemaan vielä jotain nappeja. Mitä tapahtuu, jos ei valitse päivämäärää ja painaa nappia?

#21 Updated by Johanna Räisä 15 päivää ago

Johanna Räisä kirjoitti:

Anneli Österman kirjoitti:

Puuttuuko Suspend/Unsuspend-nappi?

Puuttuu, koska tulkitsin siitä vanhasta koodista, että jos on tuo AutoResumeSuspendedHolds, niin sitten lisää aina kentän päivämäärän. Tuossa se päivittää sen kun valitsee vain päivämäärän, turhaa alkaa painelemaan vielä jotain nappeja. Mitä tapahtuu, jos ei valitse päivämäärää ja painaa nappia?

Ehkä ne kumpikin pitää näkyä, no korjataan.

#22 Updated by Anneli Österman 12 päivää ago

Testasin nyt varauksen tekemistä sellaisella virkailijatunnuksella, jolla on varauksiin liittyen vain place_holds-oikeus. Sain tehtyä varauksen, mutta en nähnyt varausjonossa olevia varauksia. Sain jonon näkyville vasta, kun lisäsin käyttäjälle myös modify_holds_priority-oikeuden, jota ei haluta käyttäjille antaa.

#23 Updated by Anneli Österman 12 päivää ago

5446

Revert waiting status -nappi ei toimi. Sillä voi palauttaa odottavan varauksen takaisin jonoon esim. kun kiinni jäänyt nide on rikkinäinen.

#24 Updated by Pirkko-Liisa Lauhikari 11 päivää ago

5447
5448

Olisikohan sivutuksessa mennyt rikki, kun nyt varausten kokonaismäärä Varaukset -sivulla näkyy vääriin.
Esim. varauksia yht. 31, näyttää:

Varauksia yht 4:

Samoin pari erikoista tapausta tuli vastaan:
Teoksen "Miehen muotokuva" -teosnäytöllä (https://outi-test.koha-suomi.fi/cgi-bin/koha/catalogue/detail.pl?biblionumber=4676) näkyy, että olisi vain yksi varaus, mutta kun klikkaa Varaukset-painiketta, varausjonossa on 19 varausta. Priorityt ovat myös oudosti, ykkösiä on neljä, kakkosia kolme.
Samanlainen tapaus "Martinmaa, mieshenkilö" (https://outi-test.koha-suomi.fi/cgi-bin/koha/catalogue/detail.pl?biblionumber=5322,

#25 Updated by Johanna Räisä 11 päivää ago

Pirkko-Liisa Lauhikari kirjoitti:

Olisikohan sivutuksessa mennyt rikki, kun nyt varausten kokonaismäärä Varaukset -sivulla näkyy vääriin.
Esim. varauksia yht. 31, näyttää:

Varauksia yht 4:

Samoin pari erikoista tapausta tuli vastaan:
Teoksen "Miehen muotokuva" -teosnäytöllä (https://outi-test.koha-suomi.fi/cgi-bin/koha/catalogue/detail.pl?biblionumber=4676) näkyy, että olisi vain yksi varaus, mutta kun klikkaa Varaukset-painiketta, varausjonossa on 19 varausta. Priorityt ovat myös oudosti, ykkösiä on neljä, kakkosia kolme.
Samanlainen tapaus "Martinmaa, mieshenkilö" (https://outi-test.koha-suomi.fi/cgi-bin/koha/catalogue/detail.pl?biblionumber=5322,

Mielenkiintoista, miksiköhän tuo REST API tarjoilee enemmän varauksia, https://outi-test.koha-suomi.fi/api/v1/holds?biblio_id=4676 ja https://outi-test.koha-suomi.fi/api/v1/holds?biblio_id=4676. Puuttuukoha minulta jokin parametri?

#26 Updated by Anneli Österman 8 päivää ago

Tämän hetken testaukset place_holds-käyttäjäoikeudella:

Varauslistassa:
  • varauksen poisto ei toimi. Antaa popparin, että oletko varma, mutta kun vastaa olevansa, ei tapahdu mitään.
  • Suspend-nappulaa painaessa saa virheilmoituksen: There was an error:error Forbidden
  • Jos valitsee suspend-nappulan vierestä päivämäärän, niin varaus ei suspendaudu eikä valittu päivämäärä jää näkyviin.
  • Details-sarakkeessa pyörii pallero, kun varaus on kuljetettavana tai noudettavana.
  • Revert transit status -nappula ja Revert waiting status -nappula toimii.
  • Expiration ja Date -sarakkeissa ei pysty vaihtamaan tai poistamaan päivämäärää. Jos valitsee toisen päivämäärän, palautuu siihen alkuperäinen päivämäärä. Poisto-nappia painamalla kenttä vilahtaa, mutta siihen palaa alkuperäinen päivämäärä.

#27 Updated by Johanna Räisä 7 päivää ago

Anneli Österman kirjoitti:

Tämän hetken testaukset place_holds-käyttäjäoikeudella:

Varauslistassa:
  • varauksen poisto ei toimi. Antaa popparin, että oletko varma, mutta kun vastaa olevansa, ei tapahdu mitään.
  • Suspend-nappulaa painaessa saa virheilmoituksen: There was an error:error Forbidden
  • Jos valitsee suspend-nappulan vierestä päivämäärän, niin varaus ei suspendaudu eikä valittu päivämäärä jää näkyviin.
  • Details-sarakkeessa pyörii pallero, kun varaus on kuljetettavana tai noudettavana.
  • Revert transit status -nappula ja Revert waiting status -nappula toimii.
  • Expiration ja Date -sarakkeissa ei pysty vaihtamaan tai poistamaan päivämäärää. Jos valitsee toisen päivämäärän, palautuu siihen alkuperäinen päivämäärä. Poisto-nappia painamalla kenttä vilahtaa, mutta siihen palaa alkuperäinen päivämäärä.

Ok, muokkauksethan vaativat jotain oikeuksia. Pitää tarkistaa mitä siellä restissä halutaan noille tapahtumille.

#28 Updated by Johanna Räisä 7 päivää ago

Johanna Räisä kirjoitti:

Anneli Österman kirjoitti:

Tämän hetken testaukset place_holds-käyttäjäoikeudella:

Varauslistassa:
  • varauksen poisto ei toimi. Antaa popparin, että oletko varma, mutta kun vastaa olevansa, ei tapahdu mitään.
  • Suspend-nappulaa painaessa saa virheilmoituksen: There was an error:error Forbidden
  • Jos valitsee suspend-nappulan vierestä päivämäärän, niin varaus ei suspendaudu eikä valittu päivämäärä jää näkyviin.
  • Details-sarakkeessa pyörii pallero, kun varaus on kuljetettavana tai noudettavana.
  • Revert transit status -nappula ja Revert waiting status -nappula toimii.
  • Expiration ja Date -sarakkeissa ei pysty vaihtamaan tai poistamaan päivämäärää. Jos valitsee toisen päivämäärän, palautuu siihen alkuperäinen päivämäärä. Poisto-nappia painamalla kenttä vilahtaa, mutta siihen palaa alkuperäinen päivämäärä.

Ok, muokkauksethan vaativat jotain oikeuksia. Pitää tarkistaa mitä siellä restissä halutaan noille tapahtumille.

Sepä siellä oli, eipä ole siinä APIssakaan otettu huomioon sitä place_holds-oikeutta. Korjasin tilanteen ja tein yhteisöön tiketin. https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30780 Testeille pitäisi olla nämä jo päivitetty.

#29 Updated by Anneli Österman 6 päivää ago

Johanna Räisä kirjoitti:

Sepä siellä oli, eipä ole siinä APIssakaan otettu huomioon sitä place_holds-oikeutta. Korjasin tilanteen ja tein yhteisöön tiketin. https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30780 Testeille pitäisi olla nämä jo päivitetty.

Pikaisella testauksella nämä toimi nyt.

Ei liity varaukset-sivuun, mutta asiakkaan tiedoissa ei voi suspendata/unsuspendata kaikkia varauksia kerralla. Taisin tästä mainita yhteisön tiketissä, joka liittyi varausten poistamiseen tuolla sivulla. Mutta taitaa olla perimmäisenä syynä sama kuin tässä.

Vie Atom PDF