KohaSuomi / koha-plugin-intranetjs-holdid

koha-plugin-intranetjs-holdid
GNU General Public License v3.0
0 stars 0 forks source link

Varaustunnuksen luonti ja siirto asiakkaan tiedot-listan loppuun #1

Open lmstrand opened 2 months ago

lmstrand commented 2 months ago

Testaus:

Poista intranetUserJS-asetuksesta rimpsut: // Varaustunnus-asiakasmääreen siirto asiakkaan muokkausnäytöllä ja // Varaustunnuksen automaattinen generointi.

Laita plugin IntranetUserJS: HoldID aktiiviseksi ja katso, että varaustunnus muodostuu uuden asiakkaan luonnissa automaattisesti ja että sitä voi vaihtaa kolmesta pisteestä kentän vierestä ja että varaustunnus-määre on siirtynyt oikeaan paikkaan.

pknuuti commented 2 months ago

Testasin Vaaran nextillä. Poistin mainitut JS-rimpsut ja lisäsin pluginin aktiiviseksi. Tein uuden asiakkaan, jolle varaustunnus-määre oli valmiiksi oikeassa paikassa ja kolmesta pisteestä sen sai vaihdettua. Mutta tyhjennä-painike ei tee mitään missään vaiheessa, ei ennen tietojen tallennusta eikä sen jälkeenkään. Toki tunnuksen vaihtaminen onnistuu kirjoittamalla entisen tunnuksen päälle.

2023kuvat 456

Kati-Helle commented 2 months ago

Helle-nextissä uudelle asiakastiedolle varaustunnus muodostuu automaattisesti.

Varaustunnus-kentän Tyhjennä-painike vie näytön alkuun tyhjentämättä Varaustunnus-kenttää. Tämä tapahtuu sekä tallentamattomalle että tallennetulle asiakastiedolle kuva

*Tiedoksi vielä, että Varaustunnus on hieman eri paikassa kuin tuotannossa

kuva

atarnus commented 2 months ago

Outi-Nextillä myös muodostuu varaustunnus automaattisesti uudelle asiakkaalle, ja kolmesta pisteestä saa aina päivitetyn version myös. Tyhjennä-nappi tosiaan skrollaa vain ruudun yläreunaan, ei tee muuta.

Asiakkaan muokkauksessa ei näy kolmea pistettä, ja Tyhjennä-nappi käyttäytyy samalla tavalla.

lmstrand commented 2 months ago

Korjasin tyhjennä-napin toiminnan, päivitettynä nexteillä.

Kati-Helle commented 2 months ago

Helle-nextissä toimii nyt oikein: Tyhjennä-painike tyhjentää Varaustunnus-kentän eikä liikauta näkymää. Toimii oikein sekä tallentamattomassa asiakastiedossa että tallennetun asiakastiedon muokkauksessa.

MikkoLiimatainen commented 2 months ago

Vaski-nextillä varaustunnuskenttä siirtyy oikeaan kohtaan, varaustunnus generoituu uudelle käyttäjälle automaattisesti ja tunnuksen generointi, sekä kentän tyhjennys toimivat.

Tuosta Hellen kenttien järjestyksen muutoksesta tuli mieleeni, että onko näiden plugareiden suoritusjärjestystä mahdollista muuttaa? IntranetUserJS:ssä koodit pystyttiin pistämään järjestykseen sen mukaan, oliko niissä keskinäisiä riippuvuuksia, mutta voiko plugarit järjestää keskenään ja suhteessa IntranetUserJS:ään niin, että tällaiset asiat voidaan ottaa huomioon? Oletan siis, että Hellessä on tuo käyttökielen siirto tuotannossa suoritettu varaustunnuksen siirron jälkeen ja nextillä plugari suoritetaan IntranetUserJS:n jälkeen, jolloin kenttien järjestys muuttuu.

lmstrand commented 2 months ago

Pluginit haetaan aakkosjärjestyksessä sen mukaan, mikä pluginin plugin_class on taulussa plugin_data. Nuo plugin_classit taas muodostuvat käytännössä pluginin hakemistorakenteesta, eli pluginien javascript-sisällön suorittamisjärjestystä voisi muuttaa tuolla keinoin (nimeämällä pluginien nimet->hakemistot esim. numeroimalla niitä). Ennen pluginien javascript-sisältöjä suoritetaan intranetUserJS-sisältö.

lmstrand commented 2 months ago

Kiitos testeistä. Tämä plugin on ok. Pitää miettiä, mitä tehdä Hellen tapauksessa, jos asiakasmääreiden sijainti tuottaa ongelmia. Kustomoitu versio intranetUserJS:ään?

lmstrand commented 1 week ago

Seuranta kimpoista, joissa rimpsut poistettu intranetUserJS-asetuksesta ja aktivoitu plugin: