Palaute - Koha-Suomi

Kyyti haluaa tämän käyttöön.

Vaaralle käyttöön.

Lappiin myös.

Ja Siilinjärvelle!

Tehdäänkö tämä ennen versionvaihtoa?

Jäikö tämä tekemättä versionvaihdossa?

Liittyykö tämä #5369 ?

Reetta Pihlaja kirjoitti:

Liittyykö tämä #5369 ?

Tavallaan liittyy, mutta on toteutus on järkevämpi, nopeampi ja toimintavarmempi suoraan tietokannassa kuin erillisellä javascriptillä. Se korjaa sillä tavalla suoraan myös Finna-puolen.

No tämäpäs näkyykin sitten Kohan uusissa templateissa olevan rikki, eli ei tottele tietokannan message_transports -taulua. Liittymä antaa iloisesti poistella ja asetella kooste-täpät, vaikka ei-koosteille ei olisi transportteja ollenkaan. Näköjään ongelmaan ei pure edes jos poistaa koko ODUE ja PREDUE -viestipohjat Kohasta. Täpät voi silti liittymästä poistaa. Onpas ärsyttävä bugi.

Täytyy varmaan ratkaista tämä tietokantatriggerillä, joka pakottaa palautuskehotuksille ja eräpäiväennakoille ne kooste-täpät paikalleen tietokantatasolla. Täpät voisi varmaan sitten piilottaa kokonaan Kohasta ja Finnasta.

Koosteviestien pakottaminen päälle on järkevää, en näe mitään syytä miksi asiakkaalle pitäisi lähettää kaikista lainoista erillinen sähköpostiviesti.

Tämmöinen triggeri hoitaa homman:

delimiter $$
create or replace trigger before_message_preferences_insert
before insert on borrower_message_preferences
for each row
begin
  if new.message_attribute_id=1 or new.message_attribute_id=2 then
    set new.wants_digest=1;
  end if;
end $$
delimiter ;

Update-triggeriä ei tarvita, koska viestiasetusten tallennusvaiheessa vanhat asetukset aina poistetaan ja uudet menevät kantaan insertteinä. Laitan tuon paikoilleen outin, vaaran, lapin, kyytin ja siilin testeille.

Triggerit nyt paikallaan ylläolevissa kimpoissa. Tällä hetkellä näkyy käytännössä siten, että vaikka koostetäpät ottaisi pois, ne ovat jälleen paikallaan kun asiakkaan viestiasetuksia katselee seuraavan kerran. Eli tietokanta pakottaa ne paikalleen vaikka ne ottaisi liittymässä pois. Toinen puoli kuviota on se että piilotetaan (tai passivoidaan) liittymässä koostetäpät ja sama täytyy luultavasti tehdä myös Finnassa.

Testatkaa ja kuitatkaa käyttäytyykö tuo joka paikassa halutulla tavalla.

Pikaisella testauksella Vaaran testikannassa: onko tarkoitus, että asiakkaan Muokkaa-näytöllä näkyy koosteviestiruksi, vaikka ei olisi valittuna ollenkaan esim. ennakkoilmoitusta eräpäivästä? Asiakkaan Tiedot-näytöllä koosteviestiruksi ei näy, jos ennakkoilmoitusta ei ole valittuna ollenkaan. Ristiriita näyttöjen välillä siis hämää.

Päivi Knuutinen kirjoitti:

Pikaisella testauksella Vaaran testikannassa: onko tarkoitus, että asiakkaan Muokkaa-näytöllä näkyy koosteviestiruksi, vaikka ei olisi valittuna ollenkaan esim. ennakkoilmoitusta eräpäivästä? Asiakkaan Tiedot-näytöllä koosteviestiruksi ei näy, jos ennakkoilmoitusta ei ole valittuna ollenkaan. Ristiriita näyttöjen välillä siis hämää.

Tuo taitaa liittyä luullakseni Emmin javascript-"rimpsuun" tiketissä #5369. Minusta nuo koostetäpät voisi ehkä piilottaa liittymästä kokonaan nyt kun ne ovat joka tapauksessa aina päällä.

Vaski ei ole huomannut tähän tikettiin aikanaan kommentoida, mutta meillekin triggerin voisi kyllä pistää käyttöön.

Jos/kun koosteviestitäppä laitetaan aina päälle, totteleeko Koha valintaa halutaanko viestiä ollenkaan? Eli kumoaako koosteviestitäppä valinnan ennakkoilmoituksen tai eräpäivämuistutuksen kohdalla?

Ainakin aikaisemmin olen huomannut, että vaikka asiakkaalla ei ollut valintaa sähköpostiviestin kohdalla, koostevalinta (oli oletuksena tallennuspohjassa) sai asiakkaalle lähtemään sähköpostiviestin (tai ainakin yrittämään lähetystä, joka kilpistyi toimimattomaan sähköpostiosoitteeseen).

Siis pitääkö se #5369:n rimpsu ensin ottaa testiltä pois että pääsee tätä testaamaan? Pitääkö tarkistussumma lasketuttaa?

Reetta Pihlaja kirjoitti:

Siis pitääkö se #5369:n rimpsu ensin ottaa testiltä pois että pääsee tätä testaamaan? Pitääkö tarkistussumma lasketuttaa?

Juu, kannattaa ottaa pois se "rimpsu", kun eihän niitä täppiä muuten pääse käpistelemään. Tarkistussummaus on testeillä "vain huomautus" -tilassa, joten se ei lukitse Kohaa vaikkei summa täsmäisikään. Siitä tipahtaa vain meille sähköpostiin hoksautus että se on muuttunut.

Anni Rajala kirjoitti:

Vaski ei ole huomannut tähän tikettiin aikanaan kommentoida, mutta meillekin triggerin voisi kyllä pistää käyttöön.

Nyt on myös vaski-testillä.

Päivi Knuutinen kirjoitti:

Jos/kun koosteviestitäppä laitetaan aina päälle, totteleeko Koha valintaa halutaanko viestiä ollenkaan? Eli kumoaako koosteviestitäppä valinnan ennakkoilmoituksen tai eräpäivämuistutuksen kohdalla?

Ainakin aikaisemmin olen huomannut, että vaikka asiakkaalla ei ollut valintaa sähköpostiviestin kohdalla, koostevalinta (oli oletuksena tallennuspohjassa) sai asiakkaalle lähtemään sähköpostiviestin (tai ainakin yrittämään lähetystä, joka kilpistyi toimimattomaan sähköpostiosoitteeseen).

Tuohon en äkkipäätä osaa vastata, testaappa ;D

Ainakin aikaisemmin olen huomannut, että vaikka asiakkaalla ei ollut valintaa sähköpostiviestin kohdalla, koostevalinta (oli oletuksena tallennuspohjassa) sai asiakkaalle lähtemään sähköpostiviestin (tai ainakin yrittämään lähetystä, joka kilpistyi toimimattomaan sähköpostiosoitteeseen).

Tuohon en äkkipäätä osaa vastata, testaappa ;D

Lähteekö testikannasta viestit?

Siilissä täpät tulee ja jää, aina kun asiakkaan tietoja tallentaa, muokkasi tai ei, olipa viestit valittuna tai ei. Pakotus toimii nähdäkseni siis hyvin.

Päivin havaitsemaan koosteviestin aiheuttamaan sähköpostin lähetysyritykseen en osaa ottaa kantaa, Siilissä ei ole ollut oletuksena asiakaspohjassa näitä, kun olen ajatellut että mahdollisesta sähköpostittomuudesta voi tulla sotkua.

Kodon aiempi viesti: Näköjään ongelmaan ei pure edes jos poistaa koko ODUE ja PREDUE -viestipohjat Kohasta... ei varmaan ole merkitystä enää jos toteutit tämän toisin, mutta tarkoitit varmaan DUE ja PREDUE, ODUEstahan ei koosteita lähetellä.

Reetta Pihlaja kirjoitti:

Siilissä täpät tulee ja jää, aina kun asiakkaan tietoja tallentaa, muokkasi tai ei, olipa viestit valittuna tai ei. Pakotus toimii nähdäkseni siis hyvin.

Päivin havaitsemaan koosteviestin aiheuttamaan sähköpostin lähetysyritykseen en osaa ottaa kantaa, Siilissä ei ole ollut oletuksena asiakaspohjassa näitä, kun olen ajatellut että mahdollisesta sähköpostittomuudesta voi tulla sotkua.

Kodon aiempi viesti: Näköjään ongelmaan ei pure edes jos poistaa koko ODUE ja PREDUE -viestipohjat Kohasta... ei varmaan ole merkitystä enää jos toteutit tämän toisin, mutta tarkoitit varmaan DUE ja PREDUE, ODUEstahan ei koosteita lähetellä.

Kyllä, niin tarkoitin toki. Kiitos oikaisusta :D Itseasiassa tuolla voi Finnan kannalta silti olla merkitys. Kokeilen poistaa nuo DUE ja PREDUE transportit ja viestipohjat testeiltä niin käyttekö sitten sen jälkeen katselemassa miltä testi-Finnat näyttää niiden suhteen.

Päivi Knuutinen kirjoitti:

Ainakin aikaisemmin olen huomannut, että vaikka asiakkaalla ei ollut valintaa sähköpostiviestin kohdalla, koostevalinta (oli oletuksena tallennuspohjassa) sai asiakkaalle lähtemään sähköpostiviestin (tai ainakin yrittämään lähetystä, joka kilpistyi toimimattomaan sähköpostiosoitteeseen).

Tuohon en äkkipäätä osaa vastata, testaappa ;D

Lähteekö testikannasta viestit?

Ei lähde, mutta muodostuu tai voidaan muodostaa tarvittaessa.

Poistin transportit testeiltä lappi, outi, vaara, siili, kyyti ja vaski. Kurkkaatteko miltä testifinnojen puolella näyttää nyt niiden täppien osalta?

Kodo Korkalo kirjoitti:

Poistin transportit testeiltä lappi, outi, vaara, siili, kyyti ja vaski. Kurkkaatteko miltä testifinnojen puolella näyttää nyt niiden täppien osalta?

Vaskin testifinnasta koostetäpät näyttäisivät poistuneen. Advance_Notice_digest ja Item_Due_digest elementtejä ei näytä viestiasetuksiin enää muodostuvan. Varauksen noutomuistutukselle kun saisi saman tehtyä, niin voisin poistaa nuo piilotusskriptit kokonaan.

Mikko Liimatainen kirjoitti:

Kodo Korkalo kirjoitti:

Poistin transportit testeiltä lappi, outi, vaara, siili, kyyti ja vaski. Kurkkaatteko miltä testifinnojen puolella näyttää nyt niiden täppien osalta?

Vaskin testifinnasta koostetäpät näyttäisivät poistuneen. Advance_Notice_digest ja Item_Due_digest elementtejä ei näytä viestiasetuksiin enää muodostuvan. Varauksen noutomuistutukselle kun saisi saman tehtyä, niin voisin poistaa nuo piilotusskriptit kokonaan.

Muistatko mikä sen noutomuistutuksen letter code on?

--Edit--

Täällä on HOLD_REMINDER, mutta sille ei ole eriseen digest-transporttia.

Siilin testi-Finnassa asiakkaalla, jonka tietoja on testi-Kohassa muokattu/tallennettu eli koosteilmoitus on pakotettu, Kooste : Kooste käytössä -valintamahdollisuus on asiakkaalta hävinnyt, eli oikein meni (?).

Asiakkaalla, jolla koosteilmoitus ei vielä ole pakotettuna, valintamahdollisuus on edelleen olemassa.

Reetta Pihlaja kirjoitti:

Siilin testi-Finnassa asiakkaalla, jonka tietoja on testi-Kohassa muokattu/tallennettu eli koosteilmoitus on pakotettu, Kooste : Kooste käytössä -valintamahdollisuus on asiakkaalta hävinnyt, eli oikein meni (?).

Asiakkaalla, jolla koosteilmoitus ei vielä ole pakotettuna, valintamahdollisuus on edelleen olemassa.

Kuulostaa oikealta, ajelenpa siiliin koostetäpät niille asiakkaille joilla niitä ei ole.

--Edit--

Täpät ajettu siiliin.

Kodo Korkalo kirjoitti:

Mikko Liimatainen kirjoitti:

Kodo Korkalo kirjoitti:

Poistin transportit testeiltä lappi, outi, vaara, siili, kyyti ja vaski. Kurkkaatteko miltä testifinnojen puolella näyttää nyt niiden täppien osalta?

Vaskin testifinnasta koostetäpät näyttäisivät poistuneen. Advance_Notice_digest ja Item_Due_digest elementtejä ei näytä viestiasetuksiin enää muodostuvan. Varauksen noutomuistutukselle kun saisi saman tehtyä, niin voisin poistaa nuo piilotusskriptit kokonaan.

Muistatko mikä sen noutomuistutuksen letter code on?

--Edit--

Täällä on HOLD_REMINDER, mutta sille ei ole eriseen digest-transporttia.

Juu, tuo HOLD_REMINDER on se mitä tarkoitin. Tuohan ei meillä ole käytössä, mutta koska toiveita ominaisuuden käyttöönotosta on tullut, niin jätetään tuo nyt ennalleen ainakin siihen asti, että käyttöönotosta tehdään päätös. Verkkokirjaston ylläpidon kannalta olisi vain parempi, jos tuon voisi transporteista poistaa siinä tilanteessa, että päätämme olla ottamatta ominaisuutta käyttöön.

Mikko Liimatainen kirjoitti:

Juu, tuo HOLD_REMINDER on se mitä tarkoitin. Tuohan ei meillä ole käytössä, mutta koska toiveita ominaisuuden käyttöönotosta on tullut, niin jätetään tuo nyt ennalleen ainakin siihen asti, että käyttöönotosta tehdään päätös. Verkkokirjaston ylläpidon kannalta olisi vain parempi, jos tuon voisi transporteista poistaa siinä tilanteessa, että päätämme olla ottamatta ominaisuutta käyttöön.

Kyllä sen periaatteessa voi sieltä poistaa, mutta nähdäkseni kyseessä on kyllä periaatteessa aika hyödyllinen ominaisuus. Noutamattomista varauksista aiheutuu asiakkaille sanktiomaksuja, joten huomautteleminen lienee ihan hyvätapaista :D

Kodo Korkalo kirjoitti:

Kyllä sen periaatteessa voi sieltä poistaa, mutta nähdäkseni kyseessä on kyllä periaatteessa aika hyödyllinen ominaisuus. Noutamattomista varauksista aiheutuu asiakkaille sanktiomaksuja, joten huomautteleminen lienee ihan hyvätapaista :D

Tästä varmaan käydään/on käyty kimpoissa paljonkin keskustelua, hyödyllinen ominaisuus kyllä varmaan, onko liiallista paapomista en tiedä, Siilissä päätettiin ettei käytetä. Siksi täpät on jotenkin Finnassa piilotettu.

Mutta: kaikille asiakkaille ei mielestäni ole ilmestynyt Siilin testissä koostetäppiä, pitikö niiden tulla ihan kaikille vai jollain rajauksella? Vai vain niille, joilla ylipäätään on ruksi item duessa/advance noticessa?

Reetta Pihlaja kirjoitti:

Mutta: kaikille asiakkaille ei mielestäni ole ilmestynyt Siilin testissä koostetäppiä, pitikö niiden tulla ihan kaikille vai jollain rajauksella? Vai vain niille, joilla ylipäätään on ruksi item duessa/advance noticessa?

Koostetäppiä ei pitäisi näkyä lainkaan. Kooste on ainoa mitä me lähetetään, joten se ruksi on turha ja hämää vain asiakasta.

--Edit--

Ja nyt tajusin, että tarkoitit luultavasti Koha-näkymää etkä Finnaa :D Kohassa ne ruksit on myös ajatus piilottaa. Kooste-täppä pakotetaan tietokantatasolla vasta siinä vaiheessa kun kyseinen viesti (palautuskehotus tai eräpäiväennakko) valitaan lähetettäväksi. Jos viestiruksia ei ole, niin kooste-täppääkään ei ole. Jos taas viestiruksi on, niin aina on myös koostetäppä (jota on siten turha oikeastaan edes näyttää, plus että sen ruksin kliksutteleminen käyttöliittymässä ei tee mitään). Kohan pitäisi osata tehdä tuo piilotus itsekkin, mutta siinä näkyy olevan Koha-templateissa virhe, joten täytyy hoitaa css:llä.

Lapissa vaikuttaisi olevan ok. Testi Kohassa täpät palaavat paikalleen, vaikka ne poistaisi. Testi-Finnan puolella koosteviestin valintamahdollisuus on poistunut.

Tämä on siis kesken? Onko triggeriä viety tuotantokantoihin vai onko vain testeillä?

Kyytin testiltä otin tiketin #5369 rimpsun pois ja näyttää triggeri toimivan.
Kyytin tuotannossa on tiketin #5369 rimpsu päällä.

Ilmesesti kumpikaan näistä toteutuksista ei lisää mitenkään automaattisesti koosteviestiä sellaisille asiakkaille, joiden tietoja ei muokata. Pitäisikö se siis lisätä näille asiakkaille (tai käytännössä kaikille) jollain erillisellä ajolla?

Eihän tämä nyt mene sekaisin Githubin tiketin 204 kanssa, kun siellä myös säädetään viestiasetusten kanssa.

Juu, keskenhän tuo ja hieman unohtunut, pahoitteluni. Hoidetaanpa eteenpäin.