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

Asiakkaille lähtenyt englanninkielisiä noutoilmoitusten koosteviestejä (asiakas lisännyt itselleen täpän Finnassa) #1309

Closed IrinaHalminen closed 2 months ago

IrinaHalminen commented 3 months ago

Tukipyyntö

Vaarassa ilmeni, että joillain asiakkailla on päällä noutoilmoitusten koostetoiminto, vaikka meillä ei ole sitä käytössä - ei sähköpostissa eikä tekstiviestissä.

Tämä johtuu siitä, että Finnassa viestiasetuksissa on valittavana myös kooste, vaikka sen pitäisi olla piilotettuna. Meiltä ei Päivin loman vuoksi kukaan pysty piilottamaan valintakohtaa ennen ensi viikon maanantaita.

image

Valinnan tehneille on asiakkaille on lähtenyt noutoilmoituksia englanniksi (myös tekstiviestitse), sillä HOLDDGST-pohjassa suomi, englanti ja ruotsi ovat tyhjänä, joten toiminto on käyttänyt englanninkielistä oletuspohjaa.

Pystymmekö saamaan jotenkin kiinni ne asiakkaat, joilla on tuo noutoilmoitusten kooste käytössä ja poistamaan sen heiltä käytöstä?

Jotain muuta?

No response

lmstrand commented 3 months ago

Raportti 864 Vaaran tuotannossa.

lmstrand commented 3 months ago

Näitä saa kiinni raportilla:

select distinct b.borrowernumber, bmp.*, ma.message_name from borrowers b left join borrower_message_preferences bmp on bmp.borrowernumber = b.borrowernumber left join borrower_message_transport_preferences bmtp on bmtp.borrower_message_preference_id = bmp.borrower_message_preference_id left join message_attributes ma on bmp.message_attribute_id = ma.message_attribute_id where bmp.message_attribute_id = 4 and wants_digest = 1

MikkoLiimatainen commented 3 months ago

Vaskissa on tiedoston /themes/custom/templates/myresearch/change-messaging-settings-driver.phtml loppuun laitettu piilotuksia näille viestiasetuksille.

<script nonce="<?=$this->cspNonce()?>">
    $("input#Hold_Reminder_transport_types_sms").parent().parent().css("display", "none");
    $("input[name='Advance_Notice_digest']").prop("checked", true).parent().parent().parent().css("display", "none");
    $("input[name='Item_Due_digest']").prop("checked", true).parent().parent().parent().css("display", "none");
    $("input[name='Hold_Filled_digest']").prop("checked", false).prop("disabled", "disabled").parent().parent().parent().css("display", "none");
  </script>

Edit, lisäsin hieman lisäinfoa skriptin toiminnasta: Hold_Filled_digest rivi piilottaa noutoilmoituksen koosteviestin otsikoineen. Lisäksi .prop("checked", false) kohta varmistaa, ettei valinta ole valittuna ja .prop("disabled", disabled) kohta laittaa asetuksen lukkoon, niin ettei sitä voi muuttaa.

Hold_Reminder_transport_types_sms-rivi piilottaa noutomuistutuksen tekstiviestimahdollisuuden. Advance_Notice_digest-rivi piilottaa erääntymisilmoituksen koostevalinnan ja laittaa sen aina valituksi, jotta nämä lähtevät aina koosteena. Item_Due_digest-rivi piilottaa eräpäivämuistutuksen koostevalinnan ja laittaa sen aina valituksi, jotta nämä lähtevät aina koosteena.

pirliis commented 3 months ago

Näitä saa kiinni raportilla:

select distinct b.borrowernumber, bmp.*, ma.message_name from borrowers b left join borrower_message_preferences bmp on bmp.borrowernumber = b.borrowernumber left join borrower_message_transport_preferences bmtp on bmtp.borrower_message_preference_id = bmp.borrower_message_preference_id left join message_attributes ma on bmp.message_attribute_id = ma.message_attribute_id where bmp.message_attribute_id = 4 and wants_digest = 1

OUTIsta löytyi yksi asiakas, jolta poistin täpät.

IrinaHalminen commented 3 months ago

Kiitos! Minä muokkailen tänään täpät pois niiltä asiakkailta, jotka olivat ne ehtineet laitella (ei onneksi ollut montaa). Meillä ei tosiaan muut kuin Päivi pääse Finnaan tekemään noita muutoksia, joten täytyy odotella ensi viikkoon.

PiaKusmin commented 3 months ago

Kiitos! Lappi tarkistettu (9 kpl) ja korjattu. Ja Finnaan tehty piilotukset.

Kati-Helle commented 3 months ago

Hellessä koosteviesti-täpällisiä asiakastietoja löytyi kaksi. Kooste-täpät on nyt poistettu.

Katja-Lumme commented 3 months ago

Lumpeilla tän oli ehtinyt täppäillä jo 48 asiakasta, JA @MikkoLiimatainen : mitä teillä on muuta tässä tiedostossa /themes/custom/templates/myresearch/change-messaging-settings-driver.phtml ? Koska meillä sellaista tiedostoa myresearch-kansiossa ei ole ollenkaan🙈, joten en osannut korjata Finnan puolella asiaa...

MikkoLiimatainen commented 2 months ago

Lumpeilla tän oli ehtinyt täppäillä jo 48 asiakasta, JA @MikkoLiimatainen : mitä teillä on muuta tässä tiedostossa /themes/custom/templates/myresearch/change-messaging-settings-driver.phtml ? Koska meillä sellaista tiedostoa myresearch-kansiossa ei ole ollenkaan🙈, joten en osannut korjata Finnan puolella asiaa...

Tuo tiedosto on kopio tiedostosta /themes/finna2/templates/myresearch/change-messaging-settings-driver.phtml. Kun tämän kopioi tuohon /themes/custom/templates/myresearch/-kansioon, niin tiedostoa pystyy muokkaamaan ja silloin se ohittaa Finnan oman tiedoston.

Katja-Lumme commented 2 months ago

Kiitos @MikkoLiimatainen 🤩. Lumme-Finna nyt korjattu myös.

IrinaHalminen commented 2 months ago

Hyvä, että kyselystä oli muillekin hyötyä. Suljen tiketin tältä erää. :)