Automaateilla ei voi lainata lapsi, jonka saman takaajan sisarus on lainauskiellossa maksujen takia
Closed this issue · 9 comments
Mikä vikana?
Kyytissä on käytössä asetus NoIssuesChargeGuarantees eli takaaja on lainauskiellossa, jos joku hänen taattavistaan on lainauskiellossa maksujen takia. Sen sijaan asetus NoIssuesChargeGuarantorsWithGuarantees ei ole käytössä.
Virkaija-Kohassa tämä toimii oikein eli liikaa maksuja omaava lapsi ja hänen takaajansa eivät voi lainata, mutta muut saman takaajan lapset voivat lainata.
Lainausautomaateilla tämä ei kuitenkaan toimi näin, vaan automaateilta ei pysty kukaan tästä takaaja-taattava-ryhmästä lainaamaan, vaan automaatti ilmoittaa "Asiakasta ei hyväksytty". Siis esim. lapsi, jonka sisaruksella on liikaa maksuja, ei pysty automaatilla lainaamaan.
Asiasta on tiketti 29755 Bugzillassa
Mitä pitäisi tapahtua
Taattavan, jolla ei ole liikaa maksuja pitää pystä lainaamaan myös automaateilla, vaikka hänen takaajansa muilla taattavilla onkin liikaa maksuja.
Kuinka toistaa ongelma/asia
No response
Selain
Mikroväylän-automaateilla testattu.Jotain muuta?
No response
Viedään yhteisön korjaukset testeille testattavaksi.
Meillä ei ole testillä automaatteja, joten en voi tätä testata.
Vaskilla ja OUTIlla taisi olla testausvalmius. Pyydetään heitä testaamaan, kunhan saan tuotua korjauksen testeille.
Yhteisön tiketti https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33411 taitaa koskea ehkä paremmin tätä tapausta. Ei ole vielä push-tilassa mutta tuon tiketin korjausta voisi kokeilla.
Ja tiketti https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32624 liittyy tähän myös, ehkä nuo pitää kaikki ottaa kyytiin.
Toin korjauksen yhteisön tiketistä 33411, jonka tuonti vaati yhteisön muutoksia mukaan sip-palvelimen koodiin. Lisätty seuraavat commitit omien koha-suomi-muutosten alle (ksdev/ks-0043-K22-15-SIP2):
2023-04-07 18:50 +0000 Nick Clemens o Bug 33411: Don't count related guarantees fees against NoIssuesChargeGuarantees
2022-12-08 10:38 +0000 Matthias Meusburger o Bug 32408: Add unit tests to SIP checkout
2023-04-21 07:09 -0400 Kyle M Hall o Bug 33580: Bring back ability to mark item as seen via SIP2 item information request
2023-03-14 07:06 -0400 Kyle Hall o Bug 33216: Catch and handle RegisterRequired exceptions for SIP fee paid messages
2023-03-08 12:45 +0000 Marcel de Rooy o Bug 32684: (QA follow-up) Move too_many_lost to Patron
2023-01-19 13:45 -0500 Kyle Hall o Bug 32684: Implement SIP patron status field "too many items lost"
2023-02-23 15:33 +0000 Nick Clemens o Bug 33055: (bug 32624 follow-up) Don't send a screen message if not blocked
2023-01-13 19:27 +0000 Nick Clemens o Bug 32624: Don't include guarantee or guarantor fines in BV (fines amount) in SIP messages
2023-02-07 15:19 -0300 Tomas Cohen Arazi o Bug 32612: (QA follow-up) Add POD and fix typo
2023-02-07 06:49 -0500 Kyle Hall o Bug 32612: (QA follow-up) Add BINMODE method to C4::SIP::Trapper
2022-12-27 20:32 +0000 Nick Clemens o Bug 32515: Ignore no-block flag in check-in message
Muutokset ovat nyt testattavissa testeillä. Käynnistän Outin ja Vaskin testeillä sip-palvelimen uusiksi niin noissa kimpoissa voisi yrittää testata onko ongelma poistunut.
Jaa Vaskin testillä ei varmaan ole sip-konfiguraatio kuosissa. No outin testillä on sip-palvelin pystyssä jolla OUTESTI1 ja 2 -tunnukset konfiguraatiossa.
Tuo yhteisön bugikorjaus 29755 on ollut meillä jo matkassa tuotannossa (sen verran vanha tiketti, että koodi on lähtenyt meille matkaan vielä kun alettiin väsätä versionvaihtoa).
Testeille viety korjaus ei ilmeisesti sekään koske suoraan tämän tiketin tilannetta. Automaattikanavalla Mari OUTIsta on testannut näin:
"Testailin Pirkko-Liisan antamat kortit, takaaja ja taattava 1 (jolla maksuja) estetty lainaus, taattava 2 (jolla ei maksuja) voi lainata normaalisti. Sip viesti näyttää hyvin myös syyn."
" ...lapsi, jonka sisaruksella on liikaa maksuja, ei pysty automaatilla lainaamaan."
Vaikka yhteisötiketit ja niiden korjaukset ei kuvausten mukaan suoraan koske tämän meidän tiketin kuvailtua ongelmaa, testien mukaan lainaus nyt kuitenkin onnistuu toiselta taattavalta kun toisella taattavalla on lainauksen estäviä maksuja.
Eli viedään tiistaina tuotantoon tämä muutos.
Testattu nyt tuotannossa ja vaikuttaa toimivan oikein. Suljen tiketin.