KohaSuomi / Koha

(KOHA) Koha is a free software integrated library system (ILS). Koha is distributed under the GNU GPL version 3 or later. Note: this is a synced mirror of the official Koha repo.
https://koha-community.org/
GNU General Public License v3.0
7 stars 13 forks source link

Nidevaraus ei estä uusimista, kun käytössä AllowRenewalIfOtherItemsAvailable-asetus #630

Open ajrajala opened 1 year ago

ajrajala commented 1 year ago

Mikä vikana?

Vaskissa otettiin käyttöön asetus AllowRenewalIfOtherItemsAvailable, joka mahdollistaa uusia varatun teoksen silloin kun saatavilla on riittävästi niteitä jotka voivat täyttää varauksen. Käyttöönoton jälkeen tuli esille, että asetus ei osaa kuitenkaan huomioida nidevarauksia.

Löysin Bugzillasta tuoreen tiketin asiasta: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32878. Saakohan tätä korjausta tuotua meille käyttöön?

Mitä pitäisi tapahtua

Jos lainassa olevaan niteeseen kohdistuu nidevaraus, uusinnan pitäisi olla estetty vaikka tietueella olisikin muita saatavilla olevia niteitä.

Kuinka toistaa ongelma/asia

  1. Etsi tietue, jolla on useampi saatavilla oleva nide eikä varauksia entuudestaan
  2. Lainaa jokin niteistä ja aseta eräpäiväksi esim. seuraava päivä
  3. Tee kyseiseen niteeseen varaus
  4. Mene asiakkaan lainoihin ja huomaa, että nide on varauksesta huolimatta uusittavissa

Selain

No response

Jotain muuta?

No response

Tuomas-Kyyti commented 1 year ago

Sama ongelma huomattiin myös Kyytissä ja meillä myös AllowRenewalIfOtherItemsAvailable-asetuksena Salli. Toivottavasti saadaan korjattua.

AnneliO commented 1 year ago

Koha-Suomen palaveri 10.8.2023: Tuodaan korjaus meidän versioon.

lasse-koha commented 1 year ago

Tuotu testeille.

ajrajala commented 1 year ago

En onnistunut saamaan aikaiseksi tilannetta, jossa nidevaraus estäisi uusimisen. Loin ensin tilanteen, jossa tietueessa oli saatavilla-tilaisia niteitä ja nidevaraus niteeseen joka on lainassa asiakkaalla. Asiakkaan lainoissa nide näkyi uusittavissa.

Lainasin sen jälkeen tietueen muut saatavilla-tilaiset niteet. Edelleenkin nide näkyy asiakkaan tiedoissa uusittavissa olevana. 🤔 Kyseessä tämän testiasiakkaan laina Ukkometso (nide 20203116031).

Tuomas-Kyyti commented 1 year ago

Sama havainto. Kyyti-testillä sai nidevaratun lainan edelleen uusittua.

lasse-koha commented 1 year ago

Omalla testikoneellani toimii oikein, "Uusinta"-sarakkeessa on "Varattu"-linkki niteen varauksiin, nidettä ei pysty valitsemaan uusittavaksi. Olisikohan tässä taas joku välimuistiongelma?

AnneliO commented 1 year ago

Auttaako, jos sivun päivittää ctrl+F5?

lasse-koha commented 1 year ago

Auttaako, jos sivun päivittää ctrl+F5?

Näkyy nyt oikein vaski-testillä.

ajrajala commented 1 year ago

Näyttäisi minusta toimivan halutulla tavalla:

Tuomas-Kyyti commented 1 year ago

Myös Kyyti-testillä vaikuttaa nyt toimivan oikein.

Kati-Helle commented 1 year ago

Toimii myös Helle-testillä. Hienoa, että tämän asetuksen ominaisuus on nyt toimiva ja käyttökelpoinen!

ajrajala commented 1 year ago

Toimii tuotannossa, mutta sellainen huomio tuli nyt sattumalta vastaan että jos nidevaraus kohdistuu niteeseen joka ei ole varattavissa (niteellä vaikkapa ei lainata tai kadonnut -tila) eikä tietueella ole muita saatavilla olevia niteitä, estyy myös toisen lainassa olevan niteen uusiminen. Tilanne tällä testitietueella, testiasiakkaalla VASKI088534 lainassa olevaa nidettä ei pysty tällä hetkellä uusimaan vaikka nidevaraus kohdistuu tietueen toiseen niteeseen.

lasse-koha commented 1 year ago

Erikoinen tilanne, vaikea äkkiseltään sanoa onko tarkoituksellista vai ei, ei kuitenkaan suoraan johdu näistä muutoksista. Varattavuuden tarkistamisen logiikassa ainakin on sekavuuksia/virheitä joihin on mahdollisesti tulossa korjauksia: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32702

ajrajala commented 1 year ago

Suljen tiketin koska viimeisenä mainittu ongelma ei johdu tehdyistä muutoksista. Tehdään ongelmasta oma tiketti, jos koetaan tarpeelliseksi, ongelma kuitenkin aika marginaalinen.

pirliis commented 3 months ago

Sellainen huomio OUTIsta, kun järjestelmäasetus AllowRenewalIfOtherItemsAvailable on päällä eli "Salli", nidevaraus estää muiden lainassa olevien niteiden uusinnan, jos yhtään nidettä ei ole saatavana-tilassa.

Aiemmista testauksista on kommentti: Nidevaraus ei estä muiden niteiden uusimista (riippumatta siitä onko tietueella muita saatavilla olevia niteitä vai ei).

Näyttäisi kuitenkin niin, että jos tietueessa on saatavana olevia niteitä, nidevaraus ei estä muiden lainassa olevien niteiden uusintaa, mutta jos kaikki tietueen niteet ovat lainassa, nidevaraus estää niiden uusinnan.