search

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

Lumme: asiakkaan lainarajoituksen ohittaminen ei onnistu (jos liikaa maksuja) #1308

Open Katja-Lumme opened 6 days ago

Katja-Lumme commented 6 days ago

Mikä vikana?

image

Lainaus ei aktivoidu, vaikka painaa Ohita rajoitus tilapäisesti- namiskaa. Ei tapahdu mitään, itseasiassa.

Mitä pitäisi tapahtua

No response

Kuinka toistaa ongelma/asia

No response

Selain

No response

Jotain muuta?

No response

lmstrand commented 6 days ago

Päivitin testille circulation.pl-tiedostoon muutokset, jotka ovat mukana 24.05-versiossa ja lainarajoituksen ohitus toimi. En tiedä, mikä yksittäinen commit korjasi ongelman, enkä löydä tikettiä bugzillasta koskien tätä ongelmaa.

lmstrand commented 6 days ago

(Tiedosto palautettu takaisin alkuperäiseksi ja ongelma on palannut.)

lmstrand commented 5 days ago

Ongelmaan löytyi korjaus bugzillatiketistä https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35518. Vien korjauksen testeille.

lmstrand commented 5 days ago

Korjaus viety testeille.

MikkoLiimatainen commented 4 days ago

Testasin Vaski-testillä ja korjaus näyttäisi toimivan.

Tuossa bugzillan-tiketissä mainittiin toinen ongelma, jonka tämä korjaus aiheuttaa ja joka korjataan tiketissä https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36139.

Tämä ongelma vaatii, että AutoSwitchPatron-asetus on Salli, jolloin lainauksen viivakoodikenttään lukemalla asiakkaan kirjastokortin siirrytään toisen asiakkaan tietoihin. Testillä siirtyminen tapahtuu, mutta Koha herjaa myös, että viivakoodia ei löytynyt, mikä käsittääkseni on tuon toisen tiketin kuvaama ongelma. Vaskissa tämä asetus ei ole käytössä, joten sinällään korjaus vaikuttaa meidän näkökulmastamme tarpeettomalta ja nähdäkseni tulee joka tapauksessa mukaan seuraavassa versiossa. Tämä lähinnä tiedoksi, jos korjaus on tarpeen ottaa mukaan jo nyt.

lmstrand commented 4 days ago

Tuodaan Mikon mainitsema korjaus mukaan bz-tiketistä 36139 samaan feature branchiin ksdev/ks-0163-G1308-fix-bypass-checkout-restriction (AutoSwitchPatron käytössä kimpoissa helle, kyyti ja lumme).

lmstrand commented 4 days ago

AutoSwitchPatron-korjaus viety testeille.

ajrajala commented 10 hours ago

Tähän liittyen tiedoksi aiemmin tehty tiketti siitä, ettei ohittaminen onnistu jos asiakkaalla on jokin rajoitus: https://github.com/KohaSuomi/Koha/issues/1273.

Edit: Tuohon linkkaamaani tikettiin liittyen vielä tiedoksi, että sen osalta testeillä ongelmaa ei alkujaankaan saatu toistumaan, ongelma toistui vain tuotannossa.