search

KohaSuomi / Koha-24.05

(K24) Versioon 24.05 siirtyminen. Wiki-osiossa on erilaisia ohjeistuksia ja tietoa muutoksista ja uusista ominaisuuksista.
GNU General Public License v3.0
0 stars 0 forks source link

SIP muutos - ksdev/ks-0043-K23-15-SIP2 #21

Open emta001 opened 4 months ago

emta001 commented 4 months ago

SIP-muutos.

lmstrand commented 2 months ago

Tuotu puuttuvat commitit kyseiseen branchiin ja fetchattu vaski-nextille.

lmstrand commented 2 months ago

Huom. sisältää yhteisöcommitin Bug 18317: Allow check out of already checked out items through SIP joka ei ollut päätynyt vielä ks24-branchiin, pudotetaan seuraavassa versionvaihdossa.

lmstrand commented 2 months ago

sis. commitit:

Bug 37016: Invalid due date in SIP renew response Bug 18317: Allow check out of already checked out items through SIP G900 Add account locked info to 64 resp and handle message geneneration as expired KD-2608 Handle notforloan status changes on checkin with SIP KD-913 Add SIP2 response message preference for alert KD-4572 Add ss (self service) block support for sip2 63-64 reply

ajrajala commented 2 months ago

Matrixissa oli kyselty testaajaa, Vaskissa onnistuisi ainakin.

ajrajala commented 2 months ago

Bug 37016: Invalid due date in SIP renew response -> Tämä ei näytä toimivan oikein, automaatin näytöllä sekä automaattikuitissa näkyi uusimisen jälkeen edelleen vanha eräpäivä vaikka Koha olikin uusinut niteen.

Bug 18317: Allow check out of already checked out items through SIP -> Tämä ok, laina palautui edelliseltä asiakkaalta ja meni lainaan uudelle asiakkaalle.

G900 Add account locked info to 64 resp and handle message geneneration as expired -> Yritin tunnistautua automaatilla kortilla, jolla "Tili on lukittu" -merkintä. Automaatin lokeissa 64-kentässä pelkästään tieto "Invalid password". Tuotannossa vastaavassa tilanteessa lokissa tieto "ACCOUNT LOCKED -- Invalid password" eli ei nyt toimi nextillä samalla tavalla.

KD-2608 Handle notforloan status changes on checkin with SIP -> Niteellä oleva notforloan-tila poistuu/säilyy automaatilla palauttaessa sen mukaisesti mitä UpdateNotForLoanStatusOnCheckin-järjestelmäasetukseen on Kohassa määritelty eli tämä ok. Jostain syystä tosin ensimmäistä kertaa palauttaessa nidettä, jolla oli palautuksessa säilyvä notforloan-tila, sain virheilmoituksen eikä nide palautunut lainasta. Mutta en onnistunut toistamaan samaa virhettä enää uudelleen eikä siitä näyttänyt jääneen automaatin lokiin merkintää, joten menee ehkä jonkin hetkellisen häiriön piikkiin. 🤔

KD-913 Add SIP2 response message preference for alert KD-4572 Add ss (self service) block support for sip2 63-64 reply -> Näitä kahta en tiennyt miten testata. Ss blockit ei kai Vaskissa käytössä joten jäänyt minulle vieraaksi, raportilla ei ainakaan borrower_ss_blocks-taulusta mitään löydy.

lmstrand commented 2 months ago

Kiitos testailusta. Käyn tutkimassa nuo ongelmatapaukset, että mikä niissä voi olla. Tuo omatoimen esto pitäisi lisätä meidän devaajien käyttäjätunnukselle tuonne borrower_ss_blocks-tauluun. Tuo alert liittyi jotenkin niihin laskutettujen? niteiden lajitteluun.

lmstrand commented 2 months ago

Voitteko testata uusiksi noita ongelmallisia ominaisuuksia. Lokeille ei ole tullut mitään, kun lokitustaso on ollut liian korkealla. Tarkistin, että tarvittavat commitit ovat kyydissä ja käynnistin sip-palvelimen uusiksi lokitusten kera.

ajrajala commented 2 months ago

Bug 37016: Invalid due date in SIP renew response -> Tämä toimii nyt oikein, näytölle ja kuitille päivittyi heti uusi eräpäivä.

G900 Add account locked info to 64 resp and handle message geneneration as expired -> Automaatin lokiin tulee edelleen pelkästään tieto Invalid password eli tuotannossa lokilla näkyvä ACCOUNT LOCKED -tieto puuttuu:

30.08.2024 09:24:03 64 00020240830 092402000000000000000000000000AOTUPK|AA***121|AE***|BLY|CQN|BV0|CC14.99|BD***|BE***|PB***|PC***|PIY|AFGreetings from Koha. -- Invalid password|PA1|AY7AZB3C9

Kaivelin Matrixista keskustelua, joka taisi liittyä tuohon alert-statukseen, keskustelussa oli puhetta siitä että tieto lisättäisiin CL-kenttään. Sellaista ei laskutetulla niteellä ainakaan MV automaatin lokissa näy, laitan nyt tähän vielä kuitenkin miltä lokissa näyttää. Vaskin kaikilla automaateilla laskutettujen niteiden erottelu tapahtuu CV53 perusteella, muulle ei meillä ole ollut tarvetta.

30.08.2024 09:31:51 101YNN20240830 093150AOTUPK|ABLVASK24010900403|AQTAPK|AJYökoulu ja kaamea kosto /|AA***484|CK001|CSLAP 84.2 NOR|CTTAPK|CRHEL|CV53|AY1AZD708

lmstrand commented 2 months ago

Tein muutoksen, jossa tilin lukossa ollessa lisätään tieto sanomaan aina ja sanoma muodostuu samoin, kuin tili olisi vanhentunut. Voitteko kokeilla mitä nyt tapahtuu, jos tili on myös vanhentunut? En ole ihan varma tuosta logiikasta :D

ajrajala commented 2 months ago

Tein muutoksen, jossa tilin lukossa ollessa lisätään tieto sanomaan aina ja sanoma muodostuu samoin, kuin tili olisi vanhentunut. Voitteko kokeilla mitä nyt tapahtuu, jos tili on myös vanhentunut? En ole ihan varma tuosta logiikasta :D

Muutin lukitun kortin vanhentuneeksi, nyt tulee sanomassa näin: AFGreetings from Koha. Your account has expired as of 29.08.2024 ACCOUNT LOCKED -- Invalid password

lmstrand commented 2 months ago

Ok nuo molemmat tapaukset ovat sitten kaiketi kunnossa erikseen ja yhdessä. Pyydän vielä OUTIlaisia testaamaan Lyngsoen vehkeillä ainakin nuo vanhenemisen/lukituksen erikseen ja yhdessä varmuuden vuoksi.