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

Varauksen noutomuistutukset eivät lähde määrittelyiden mukaisesti #990

Open MikkoLiimatainen opened 8 months ago

MikkoLiimatainen commented 8 months ago

Mikä vikana?

Vaskissa on varausten noutomuistutukset määritelty lähtemään viiden päivän päästä noudettavaksi tärppäämisestä.

21 09 * $TRIGGER cronjobs/holds/holds_reminder.pl -v -days 5 --triggered 5 -holidays -c

Triggered vivun pitäisi estää viestien lähteminen uesammin kuin kerran ja holidays vivun ohittaa kirjaston kiinniolopäivät, mutta nyt vaikuttaisi siltä, että aina viidennen päivän varauksen tärppäämisestä osuessa kirjaston kiinniolopäivälle, lähtee noutomuistutus kirjaston ollessa kiinni ja myös seuraavina päivinä varsinaiseen viidenteen aukiolopäivään asti.

Mitä pitäisi tapahtua

No response

Kuinka toistaa ongelma/asia

No response

Selain

No response

Jotain muuta?

No response

lmstrand commented 8 months ago

Yhteisössä on ainakin trigger-vipuun korjaus https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34569. Näyttäisi kyllä siltä, että tuo yhteisökorjaus koskee vain -t -triggeriä, eikä -trigger -vivun käytössä pitäisi olla ongelmaa ilman korjausta. Tuo kalenterin huomiotta jättäminen eli holidays-vipukaan ei siis toimi? Tuosta en kyllä löydä yhteisöstä mainintaa. Voisiko triggerongelma olla siinä, että jos triggerille ei ole toista tapaa ilmoittaa se, triggeri pitää kirjoittaa yhdellä viivalla. Holidays-vipu tosin on jo tällä hetkellä kirjoitettu yhdellä viivalla tuotannon ajastuksissa, mutta sitä ennen on ollut triggered-vivun perässä numero 5.

Muutin tuotantoon cronin muotoon 21 09 * $TRIGGER cronjobs/holds/holds_reminder.pl -v -days 5 -triggered -holidays -c

Koodia tarkastellessa nyt pitäisi -tiggered-vivun toimia (ilman yhteisökorjausta) eli vain viidentenä päivänä varauksen tärppäys-tilasta pitäisi lähteä tuotannossa muistutus. Toivotaan että holidays-vipukin korjaantui tuon -triggered-vivun perässä olleen vitosen poistolla, joka oli jäänyt tuotantoon.

Jos toiminnan testaaminen tuotannossa ei onnistu, niin voidaan kokeilla testillä.

lmstrand commented 8 months ago

Ilmeisesti getOptions-toiminnon, joka kerää nuo asetukset vivuista, ei pitäisi välittää, onko käytetty yhtä tai kahta viivaa tai onko välissä ylimääräisiä parametreja. Se meinaisi, että meillä on tässä kaksi paljastunutta bugia käsissämme...mutta katselkaa nyt ainakin että sama ongelma toistuu edelleen muutoksesta huolimatta tuotannossa, niin saadaan asiaan varmuus.

lmstrand commented 8 months ago

Testasin cronin ajoa Vaara-testillä ja tuolla uudella tyylillä ei ainakaan lähtenyt noutomuistutusta lomapäivänä. Otin triggerin -holidays pois käytöstä ja viesti muodostui.

lmstrand commented 8 months ago

Vaara-testillä ei myöskään kaksi päivää noudettavana ollut varaus luonut muistutusta jos holidays-vipu oli pois käytöstä ja triggered-vipu oli käytössä. Vaara-testillä siis koko viikko merkattu lomapäiviksi (yksittäisiä ei-toistuvia punaisella kalenterissa näkyviä kuten Vaskissakin oli jouluaikaan merkattuna). Skriptin ajoin siis /holds_reminder.pl -v -days 1 -triggered -c jolloin toisena päivänä ei enää kuulu lähteäkään muistutusta. Poistin -triggered-vivun ja viesti muodostui. Eli testien mukaan skripti reagoi oikein kalenteriin holidays-vivulla (viestiä ei muodostunut) ja -triggered-vipukin toimii. Pitäisi toimia testien mukaan kuten halutaan Vaskin tuotannossakin (viimeistään korjatun vipunotaation kanssa, ongelmaa en saanut tosin toistettua myöskään vanhalla tavalla).

MikkoLiimatainen commented 8 months ago

Viikonlopun aikana oli tullut 157 uutta tapausta, joissa näyttäisi samoista varauksista lähteneen useampi noutoilmoitus.

Siellä oli asiakkaalla borrowernumber=5618 erikoisempikin tapaus, jossa varaukset olivat saapuneet noudettaviksi 21.12., mikä olisi pistänyt viidennen päivän tapaninpäivälle. Nämä joulun pyhät oli kuitenkin hypätty ihan oikein yli, mutta viides päivä osui nyt tavallaan lauantaille 30.12., jolloin noutokirjasto on suljettu. Oikea viides aukiolopäivä olisi ollut tänään, mutta näistä varauksista on kuitenkin tuosta 30.12. lähtien lähtenyt noutomuistutus joka päivä tähän päivään asti. Eli muuten tuo hyppää kiinniolopäivät nätisti yli, paitsi sitten kun kyse on tuosta "viidennestä" päivästä.

lmstrand commented 8 months ago

Ilmeisesti ongelma ei ole itse cron-ajossa, vaan jotenkin kalenterin kanssa. En saa toistettua ongelmaa Vaara-testin kalenterin kanssa testaamani yksikön aukioloajoilla, jossa varaus on ollut noudettavana. Viesti lähtee näköjään asiakkaalle myös lomapäivänä jos x päivää on kulunut varauksen tärppäämisestä ja seuraava päivä, jolloin viesti olisi muodostettava on lomapäivä. En saanut toistettua viestin lähetystä useammin kuin kerran triggered-triggerillä. Yritän kokeilla teidän testiympäristössä samaa ja toistaa ongelman, kalenterin pitäisi olla nyt testien päivityksen/redusoinnin jälkeen samassa tilassa kuin tuotannossa.

lmstrand commented 8 months ago

Paitsi että vaski-testiin ei koskettu redusointi/päivitysajossa :D

lmstrand commented 8 months ago

Nyt on kopiotu tuotannosta vaski-testille TUYL-yksikön lomapäivämääritykset special_holidays ja repeatable_holidays -tauluista.

lmstrand commented 8 months ago

Vaski-testillä jostain syystä tapahtuu näin: 1 reserves waiting since 2023-12-21 for TUYL borrower Jekkutausnen, Aaveli has 1 holds triggering notice. borrower Jekkutausnen, Aaveli was sent notices via: email

eli ilmeisesti viikonloppuja ei nyt lasketa, kun ajan muistutusten luonnin "perl holds_reminder.pl -v --days 8 --triggered --holidays -c"? Ennen kuin kopioin Vaskin tuotannosta lomakaenteritiedot, ajo tuntui toimivan oikein ja päivien lukumäärä täsmäsi kalenterin kanssa. Silloin toistettavista lomapäivistä puuttui viikonloput.

lmstrand commented 8 months ago

Lisäsin viikonloppuihin kuvaukset lauantait ja sunnuntait. Sen jälkeen ne huomioitiin kiinniolopäiviksi. tein myös korefreshin, jos sillä oli vaikutusta, ettei pelkkä tietokantakopiointi riittänyt. Nyt siis viesti syntyi tälle päivää yllä olevalle asiakkaalle "perl holds_reminder.pl -v --days 6 --triggered --holidays -c" -ajolla. https://vaski-test.koha-suomi.fi/cgi-bin/koha/members/notices.pl?borrowernumber=44671

lmstrand commented 8 months ago

Lisäsin testiloman 2.1. ja enää ei muodostunut viestiä edellisellä ajolla. "perl holds_reminder.pl -v --days 5 --triggered --holidays -c" taas muodosti viestin.

lmstrand commented 8 months ago

Ainakaan käsin ajettuna eri vuorokausiin siirtämällä tuon varauksen waitingdaten, sille ei muodostunut ylimääräistä viestiä. Jätän ajon " 21 09 * $TRIGGER cronjobs/holds/holds_reminder.pl -v --days 5 --triggered --holidays -c" ajautumaan testillä joka aamulle. Lisäsin myös loppuviikolle joka päivälle yksittäisen lomapäivän TUYL-yksikölle. Katsotaan muodostuuko viestit, vaikkei kuuluisi noilta loppuviikon päiviltä.

lmstrand commented 8 months ago

Sain toistettua ongelman testillä, kun lisäsin yksittäisiä lomapäiviä niin, että lomapäivä osui ensimmäisen (ainoan käytettäessä --triggered) päivän kohdalle, jolloin viestin tuli lähteä. Triggeri ei estänyt viestin lähettämistä uudelleen seuraavana lomapäivänä. Varmaan toistuu yli viikonlopun viestien lähteys, kunnes on ensimmäinen aukiolopäivä maanantaina.

Ongelmasta pitäisi tehdä yhteisöön tiketti.

Ps. kuvausten lisääminen viikonloppulomapäiviin lisäsi ne cacheen, lomapäivien siirto pelkästään tietokantaan ei siksi siis toiminut suorilta.

lmstrand commented 8 months ago

Lisäsin yhteisöön tiketin: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35700

ElinaUotila commented 8 months ago

Kirkesistä voisi ottaa noutomuistutusten lähetyksen pois päältä siksi aikaa, kun asiaan saadaan ratkaisu.

lmstrand commented 8 months ago

Kirkes-tuotannosta poistettu (kommentoitu) noutomuistutusten luonti.

pirliis commented 6 months ago

OUTI-ohjausryhmä päätti ti 27.2.2024 kokouksessa, että otetaan käyttöön noutoilmoitusmuistutus sähköpostiin, kun bugi on saatu korjattua.

Hanna-Lumme commented 5 months ago

Lumpeissa voisi myös ottaa tämän pois päältä siksi aikaa, kunnes bugi on korjaantunut.

lmstrand commented 5 months ago

Lumme-tuotannosta poistettu (kommentoitu) noutomuistutusten luonti.

ElinaUotila commented 3 months ago

Meillä on ilmeisesti muodostunut noita noutomuistutuksia taas. Tuliko tähän versionvaihdossa jokin muutos, joka on mennyt mulla ohi?

lmstrand commented 2 months ago

Meillä on ilmeisesti muodostunut noita noutomuistutuksia taas. Tuliko tähän versionvaihdossa jokin muutos, joka on mennyt mulla ohi?

Teillä on tosiaan tuotannossa tämä cron-ajo päällä. Bugi on siis edelleen olemassa. Otetaanko pois teiltä noiden muodostaminen ja milloin sen voi tehdä (en viitsi nyt suorilta ennen kuittausta tätä tehdä)?

ElinaUotila commented 1 month ago

Meillä on ilmeisesti muodostunut noita noutomuistutuksia taas. Tuliko tähän versionvaihdossa jokin muutos, joka on mennyt mulla ohi?

Teillä on tosiaan tuotannossa tämä cron-ajo päällä. Bugi on siis edelleen olemassa. Otetaanko pois teiltä noiden muodostaminen ja milloin sen voi tehdä (en viitsi nyt suorilta ennen kuittausta tätä tehdä)?

Otetaan ajo pois. Voi ottaa jossain sopivassa välissä tällä viikolla.

johannaraisa commented 1 month ago

Se on nyt pois päältä, tänä aamuna on viimeisen kerran käynnistynyt.

Hanna-Lumme commented 1 month ago

Lumpeissa näytti myös tulevan noutomuistutuksia. Ne voisi ottaa taas pois päältä. :)

johannaraisa commented 1 month ago

Nyt on poistettu Lumpeiltakin.