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

Kyyti: DUEDGST lähtenyt tekstiviestinä vaikka ei pitäisi #1355

Closed Tuomas-Kyyti closed 2 days ago

Tuomas-Kyyti commented 1 month ago

Mikä vikana?

Muutamalle asiakkaille on lähtenyt DUEDGST eli ilmoitus eräpäivänä -viesti tekstiviestinä, vaikka tekstiviestiä ei siihen voi valita ja näilläkin asiakkailla näkyy vain viiva viestiasetusten tekstiviestisarakkeessa.

Kuitenkin message_queue-taulussa näkyy nämä viestit sent-tilassa ja asiakkaiden ilmoitukset-välilehdellä viestit näkyvät lähteneinä.

Viestit ovat lähteneet englanninkielisinä oletusviesteinä, koska tuota viestiä tekstiviestinä ei ole meillä käännetty, kun sen ei ole koskaan pitänyt olla edes käytössä tekstiviestinä, vaan vain sähköpostina.

Tänä vuonna tekstiviesti on kuitenkin lähtenyt 11 asiakkkaalle. Kaikki viestit ovat lähteneet versionvaihdon jälkeen touko-elokuussa, joten se lienee jotenkin vaikuttanut asiaan.

Yhteistä näille asiakkaille vaikuttaa olevan, että heille lähtee tekstiviestinä jokin ilmoitus (ennakkoilmoitus PREDUEDGST tai saapumisilmoitus HOLD), mutta versionvaihdon jälkeen myös DUEDGST on alkanut lähteä tekstiviestillä.

Viestiasetusmuutoksia ei näy kenelläkään asiakkaalla.

Mitenhän tämä on mahdollista?

Asiakkaat, joille DUEDGST-viestejä on lähtenyt tekstiviesteinä: 275486 273016 271086 272162 208220 212941 180369 141057 273302 204340 145062

Mitä pitäisi tapahtua

DUEDGST pitäisi lähteä vain sähköpostina, jos asiakas on sen halunut. Kohassa ja Kyyti-finnassa ei pysty valitsemaan DUEDGST-viestiä tekstiviestinä.

Kuinka toistaa ongelma/asia

No response

Selain

No response

Jotain muuta?

No response

PiaKusmin commented 1 month ago

Voisko liittyä tähän tikettiin https://github.com/KohaSuomi/Koha/issues/1309?

AnneliO commented 1 month ago

Voisko liittyä tähän tikettiin #1309?

SMS-vaihtoehtoa ei ole näkyvissä Kyyti-Finnassa. Estin JavaScriptit, jotta näkisin, onko jotain piilotettu ja näkyville tuli vain koosteviesti-mahdollisuus, mutta ei tekstiviestimahdollisuutta:

kuva

lmstrand commented 1 month ago

Varmuuden vuoksi DUEDGST-letter codelta voisi poistaa tuon viestimistavan(transport) SMS kuten tiketissä https://github.com/KohaSuomi/Koha/issues/861 kokeiltiin. Poistuu siten Finnasta ja Kohan virkailijaliittymästä. Se löytyy Kyytin tuotannon tietokannasta, joten piilotukset on tehty skripteillä Finnaan ja Kohaan. Sitä en osaa sanoa, että missä vaiheessa noita on pystynyt syntymään asiakkaille noita viestilähetysvalintoja. Kohaan on lisätty muutos, jolla viestiasetusten muutokset ovat alkaneet lokittua asiakkaan tapahtumalokiin 8.3.2024.

lmstrand commented 1 month ago

Hetkinen katsoin väärin. Kyytin tuotannossa löytyy vain vain email, phone ja suomifi -transportit DUEDGST-lettercodelle.

lmstrand commented 1 month ago

Voisiko tässä olla kyseessä jonkin uusi ominaisuus, joka käyttää sitten automaattisesti toissijaista lähetysmuotoa jos vain tuo kooste-täppä on valittuna eikä sitä toimitustapaa ollenkaan...

AnneliO commented 1 month ago

Ainakaan järjestelmäasetuksista en löytänyt mitään tuommoseen liittyvää sanoilla fallback, sms tai message. Mutta voihan se "fallback" olla jossain muuallakin kuin järjestelmäasetuksissa.

lmstrand commented 1 month ago

En minäkään huomaa noissa cronskripteissä mitään sellaista muutosta. Viestiasetusten muutoslokitus on lisätty tuotantoihin sittenkin 13.5. https://github.com/KohaSuomi/Koha/issues/241#issuecomment-2107321030

AnneliO commented 1 month ago

Vastaavia oletusviestejä näyttäisi lähteneen sms:nä myös Lumpeissa, Lapissa ja Vaarassa. Hellessäkin on lähtenyt viestejä, mutta ne näyttää sisällöltään "aiheellisilta".

AnneliO commented 1 month ago

Laskeskelin, kuinka monta sms-tyyppistä DUEDGST-viestiä on lähtenyt eri kimpoissa ja tällaisia lukuja löytyi:

Kimppa Määrä
Lumme 17
Kyyti 17
Lappi 27
Vaara 4

Tein tällaisen kyselyn kantoihin:

select count(*) from message_queue where letter_code='DUEDGST' and message_transport_type='sms';
lmstrand commented 1 month ago

Lapin tuotantoon on lisätty verbose-vipu ajastettuun ajoon advance_noticesille, jos lokille saisi enemmän tietoa, mistä tämä voisi johtua. Lokeilla oli merkintöjä esim. Kyytissä:

No circulation DUEDGST letter transported by sms at /home/koha/Koha/C4/Letters.pm line 585. Use of uninitialized value in concatenation (.) or string at /home/koha/Koha/misc/cronjobs/advance_notices.pl line 624. no letter of type '' found for borrowernumber 275486. Please see sample_notices.sql at /home/koha/Koha/misc/cronjobs/advance_notices.pl line 624.

Tuon perusteella ei olisi kuulunut lähteä viestiä, eli onkohan kysessä jonkinlainen bugi Letters.pm-moduulissa. En näe uusilta lokeilta samanlaisia ilmoituksia enää näistä äskettäin parin päivän sisään syntyneistä viesteistä.

Oletusviestipohjan tyhjentämisen kuuluisi estää viestin syntyminen, mutta jos Lappi jättäisi vielä hetkeksi omansa, niin saatetaan saada kiinni, mistä tämä johtuu.

AnneliO commented 4 weeks ago

Lappiin oli muodostunut yksi sms-duedgst tänä aamuna, mutta lokille ei ollut kirjaantunut mitään hyödyllisiä lisätietoja.

Päätimme kokeilla duedgst-pohjasta sms-viestityypiltä oletuspohjan tyhjentämistä. Se pitäisi estää viestien lähteminen. Tyhjensin Lapista, Vaarasta, Lumpeista ja Kyytistä kyseiset oletuspohjat. Seuraillaan taas tilannetta.

AnneliO commented 3 weeks ago

Tarkistin viestimäärät tänään ja uusia ei ollut vielä ainakaan syntynyt, mutta suht harvoihan noita muutenkin syntyy.

AnneliO commented 3 weeks ago

Tarkistin viestimäärät, ei uusia.

emta001 commented 5 days ago

Voisiko tämän sulkea? Avataan uudelleen, jos näitä tulee lisää.

Tuomas-Kyyti commented 5 days ago

Kyytiin ei ole tullut 12.8.24 jälkeen. Tarkistaako @AnneliO muut kimpat vai suljetaanko pois?

AnneliO commented 4 days ago

Muissa kimpoissakaan ei ole tullut uusia, joten sulje vaan pois. :)

Tuomas-Kyyti commented 4 days ago

Ilmeisesti siis duedgst-pohjasta sms-viestityypiltä oletuspohjan tyhjentäminen auttoi tähän ongelmaan. Suljen tiketin.

PiaKusmin commented 2 days ago

Lappiin on muodostunut muitakin viestejä englannin kielisinä SMS-viesteinä, vaikka näin ei pitäisi tapahtua. Kyseessä on PREDUEDGST ja PREDUE viestit. Molemmissa löytyy eng. kieliset SMS-pohjat. Pitäisikö nekin tyhjentää?

Asiakas: borrowernumber=517596

PiaKusmin commented 2 days ago

Ja vielä DUE-pohjakin. Lapissa SMS viestit pitäisi lähteä vain noutoilmoituksista

lmstrand commented 2 days ago

"Lapissa SMS viestit pitäisi lähteä vain noutoilmoituksista"

Ainakaan Finnasta noita ei voi laittaa päälle:

image

lmstrand commented 2 days ago

DUE-pohjaa ei pitäisi pystyä käyttämään (tai sellaista viestiasetusta ei pitäis pystyä tallentamaan, jossa ei käytetä koostetta), meillä on triggeri joka pakottaa koosteen päälle tietokannassa.

lmstrand commented 2 days ago

Nää on varmaan vanhoja asiakkaita, jotka ovat saaneet laitettua joskus itselleen tuollaiia viestiasetuksia ennen kuin niitä piiloteltiin ja estettiin. Jos viestipohjan tyhjentää, niin viestejä ei synny. Ehkä ratkaisisin asian niin.

PiaKusmin commented 2 days ago

Kävin tyhjentämässä meidän viesteistä nuo SMS-pohjat. Jospa niitä ei enää muodostu. Kiitos selvittelystä. Suljen tiketin uudelleen.