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

[Kehitysehdotus] Uusi työkalu: Varausten erämuokkaustyökalu #249

Open Kati-Helle opened 2 years ago

Kati-Helle commented 2 years ago

Kehitysehdotukseni

Saisiko työkalun/toiminnon, jolla varauksen viimeisen noutopäivän saa muutettua eräajona? Vastaava toiminto on olemassa eräpäiville: Eräpäivien siirto eräajona = Batch extend due dates. Kätevä toiminto kirjaston poikkeuskiinniolotapauksessa.

Noudettavana olevan varauksen found = W Eräajona muutettava arvo = expirationdate

Kohan osio

Työkalut

Jotain muuta?

No response

AnneliO commented 9 months ago

Koha-Suomen asiantuntijaryhmä 15.1.2024: Toteutetaan itse ja tarjotaan yhteisölle.

Mietitään toteutustapa tarkemmin ennen toteutusta. Toteutetaanko esim. saman tyyppiseksti kuin Eräpäivien siirto eräajona vai kuten niteiden muokkaus eräajona, johon tietoja voi viedä raporteilla.

emta001 commented 8 months ago

Kattelin tuossa vanhoja rimpsuja joilla varauksia on aiemmin muutettu. Niissä on käytetty joskus aika monimutkaisiakin ehtoja, esim.:

UPDATE items i LEFT JOIN reserves r ON (i.itemnumber = r.itemnumber) LEFT JOIN branchtransfers bt ON(r.itemnumber = bt.itemnumber) SET i.damaged = "2" WHERE i.damaged NOT IN("2","1","5") AND i.homebranch IN("RUPK","RUVA") AND holdingbranch = "RUPK" AND onloan IS NULL AND r.found IS NULL AND i.itemnumber NOT IN (select itemnumber from branchtransfers where tobranch = "RUPK" and datearrived IS NULL);

Jos tää toteutettas samaan tyyliin kuin "Eräpäivien siirto eräajona", niin mitä kaikkia ehtoja meidän pitäs ottaa huomioon? Eräpäivien siirrossa käytetään vaan asiakaslajia, kirjastoa ja aikaväliä.

Tuolta kannalta siis olis ehkä järkevämpää tehdä tää "Niteiden muokkaus eräajona" mukaan ja syöttää työkalulle reserve_id:t.

emta001 commented 8 months ago

Tein yhteisöön tiketin 36135.

emta001 commented 8 months ago

Kattelin tuossa vanhoja rimpsuja joilla varauksia on aiemmin muutettu. Niissä on käytetty joskus aika monimutkaisiakin ehtoja, esim.:

UPDATE items i LEFT JOIN reserves r ON (i.itemnumber = r.itemnumber) LEFT JOIN branchtransfers bt ON(r.itemnumber = bt.itemnumber) SET i.damaged = "2" WHERE i.damaged NOT IN("2","1","5") AND i.homebranch IN("RUPK","RUVA") AND holdingbranch = "RUPK" AND onloan IS NULL AND r.found IS NULL AND i.itemnumber NOT IN (select itemnumber from branchtransfers where tobranch = "RUPK" and datearrived IS NULL);

Jos tää toteutettas samaan tyyliin kuin "Eräpäivien siirto eräajona", niin mitä kaikkia ehtoja meidän pitäs ottaa huomioon? Eräpäivien siirrossa käytetään vaan asiakaslajia, kirjastoa ja aikaväliä.

Tuolta kannalta siis olis ehkä järkevämpää tehdä tää "Niteiden muokkaus eräajona" mukaan ja syöttää työkalulle reserve_id:t.

Ja tulipas valittua ihan väärä esimerkki :smile:

AnneliO commented 8 months ago

Ideointipalaveri 23.2.2024

Paikalla: Anneli, Mikko, Lasse, Kodo, Emmi, Piia, Päivi, Kati

Varausten erämuokkaustyökalu

Varausten erämuokkaustyökalulla on tarkoitus muuttaa varausten noutokirjastoa, expirationdatea, suspend_until-arvoa ja reservenotea. SQL-raportteihin Eräkäsittele-valikkoon vaihtoehto, jolla saa vietyä tiedot varausten erämuokkaustyökaluun, jos raportilla on reserve_id-sarake.

Tarvitaan uusi käyttöoikeus batch_modify_reserves.

Työkalun aloitussivulla ensin valitaan muutettava joukko.

Hakuehdoiksi

Sen jälkeen siirrytään niteiden erämuokkauksen tyyppisesti toiselle sivulle, jossa valitaan tehtävä muutos

Valinnoiksi:

Jos tiedot tuodaan raportilta, siirrytään suoraan tähän toiselle sivulle.

emta001 commented 8 months ago

Sen verran joudutaan suunnitelmaa muuttamaan, että batch_modify_reserves-oikasta tuleekin batch_modify_holds. Reserves ei ole enää yhteisön käyttämän terminologian mukainen https://wiki.koha-community.org/wiki/Terminology#F

emta001 commented 8 months ago

Tuotu testeille testattavaksi. Huomioikaa, seuraavat asiat kun testaatte:

Testausplääni:

Varausten hakeminen työkalun hakutoiminnoilla

  1. Siirry Tools-sivulle, sieltä pitäisi löytyä linkki "Batch modify holds"-sivulle.
  2. Syötä työkalun kenttiin joitakin hakuehtoja ja paina "Continue".
  3. Tarkista, että taulukossa näkyvät varaukset vastaavat asettamiasi hakuehtoja.
  4. Syötä taulukon alla näkyvään kenttään jokin uusi arvo (esim. uusi vanhenemispäivä) ja paina "Modify selected holds".
  5. Tarkista, että taulukossa näkyvät arvot vastaavat tekemiäsi muutoksia.

Varausten hakeminen SQL-raportilla ja vienti työkaluun

  1. Luo SQL-raportteihin raportti, jossa esiintyy sarake reserves.reserve_id (esim. SELECT reserve_id FROM reserves LIMIT 10).
  2. Aja raportti.
  3. Raportin yläpuolelle tulisi ilmestyä valinta "Batch operations..." ja sen alle valinta "Batch modify holds", paina jälkimmäistä.
  4. Varmista, että työkaluun siirtyneet varaukset ovat vastaavat jotka sait tuloksena raportille.

Tuossa noin yleisesti, mutta kokeilkaa erillaisia hakukombinaatioita ja muokkauksia mahdollisimman monipuolisesti.

AnneliO commented 8 months ago

Nyt en kyllä löydä ainakaan outi-testillä Tools-sivulta Batch modify holds -linkkiä. :)

pknuuti commented 8 months ago

Ei näy Vaarankaan testillä eikä nextillä.

emta001 commented 8 months ago

No eipä kai näy, kun unohin ajaa updatedatabasen :D

emta001 commented 8 months ago

Nyt pitäs näkyä.

Kati-Helle commented 8 months ago

Helle-testissä testasin, mutta vain hieman. Massamuokkaukseen valitsemieni varausten keskeytys ja keskeytyksen syyn lisääminen onnistui massana.

Tulisikohan tässä olla maininta varauksen vanhentumispäivästä eikä asiakkaan? kuva

Lisäämäni hakuehdot kuva

Hakutulos, josta poistin ensin kaikki valinnat ja valitsin muokkaukseen neljä varausta kuva

Varausten massamuokkaus-ehdot kuva

Massamuokkauksen tulos kuva

Tein raportin, jossa hain varaukset joiden hakuarvona on: reservenotes like '%massana%'.

Raportin hakutuloksessa neljä varausta, joista yhtenä tämä kuva

AnneliO commented 8 months ago

Huomioita ja ajatuksia

emta001 commented 8 months ago

Huomioita ja ajatuksia

* Kirjastona on oletuksena valittuna kirjautumiskirjasto. Pitäisikö olla mahdollisuus hakea myös kaikkien kirjastojen varauksia kerralla?

Tuo nykyinen valinta toimii tosiaan niin, että jos haluaa kaikki kirjastot, pitää ne kliksutella mukaan hakuun. Pitänee testailla onnistuuko "Kaikki" valinnan lisäys.

* Varauslistassa ei ole teoksen nimekettä, tarvitsisiko olla? Jos tulee esim. tarve tarkistaa teoksen tiedoista/varausjonosta jotain?

Tähän en sano juuta enkä jaata. Lisäsin nyt vaan ne välttämättömät, jos halutaan lisää niin katsotaan tapauskohtaisesti.

* Voisiko varauslistauksen yläpuolella kertoa rivimäärän/varausten määrän?

Hyvä ehdotus, voidaan minun puolesta lisätä.

Kati-Helle commented 8 months ago

Linkki teostietoon olisi hyvä lisä tarkistuksia varten. Sekä ennen muutosta että jälkeen.

Testasin keskeytetyn varauksen keskeytysajankohdan muuttamisen. Toimi testatessani. kuva

pknuuti commented 8 months ago

Vaaran nextillä ei näy vieläkään tuolla Tools-sivulta Batch modify holds -linkkiä.

emta001 commented 8 months ago

Vaaran nextillä ei näy vieläkään tuolla Tools-sivulta Batch modify holds -linkkiä.

En vielä kerennyt sinne asti, että tää olis nexteilläkin. Testeillä pitäis toimia.

Kati-Helle commented 8 months ago

Kokeilin ohjeistuksessa mainittua raporttia (ohjeen kohta 6). Hakutuloksessa on voimassa olevia sekä noudettavana olevia varauksia.

Raportin hakutuloksesta valitsin Batch modify holds kuva

Varaukset Batch Modify Holds -toiminnossa kuva

Varausten tietoja

1187864 – Item waiting at Pornaisten kunnankirjasto since 13.12.2023 1187865 – (voimassa) Pornaisten kunnankirjasto expiration 11.12.2026 1168296 - Item waiting at Askolan pääkirjasto since 11.12.2023 1172531 – (voimassa) Kevätkummun kirjasto expiration 18.11.2025 1181777 - Item waiting at Kevätkummun kirjasto since 13.12.2023 1186135 - Item waiting at Kevätkummun kirjasto since 13.12.2023 1167356 – (voimassa) Loviisan pääkirjasto expiration 11.11.2025 1150486 – (voimassa) Söderkullan kirjasto expiration 19.10.2025 1132974 – (voimassa) Gammelbackan kirjasto expiration 27.09.2025 1084361 – (voimassa) Porvoon pääkirjasto expiration 19.07.2025

pknuuti commented 8 months ago

Testasin Vaaran testillä ja näyttäisi toimivan hyvin, kun ensin hoksaa miten tämä toimii. Katin esimerkkikuvat auttoi paljon, kiitos niistä!

Kati-Helle commented 8 months ago

Huomiona, että Modify holds -osion kohdasta New pickup library - arvoton/kirjastoton vaihtoehto puuttuu

kuva

Kati-Helle commented 8 months ago

Testasin niin, että varauksia keskeyttäessä on valittuna myös yksi noudettavissa oleva varaus (Waiting) Toiminto ilmoittaa siitä oikein kuva

Ilmoituksen OK-klikkauksen jälkeen tulee edelleen ilmoitus kuva

Muiden valittujen varausten massamuokkaus onnistuu, kun poistaa noudettavan varauksen massamuokkaukseen-valinnan.

pknuuti commented 7 months ago

Milloin tämä työkalu otetaan käyttöön? Meillä on tulossa toukokuussa kirjaston muutto väistötilaan, joten teenkö tarvittavat varausten muutokset itse vai teenkö tiketin kehittäjille?

pknuuti commented 7 months ago

Olisiko tätä mahdollista testata myös nextillä?

emta001 commented 7 months ago

Olisiko tätä mahdollista testata myös nextillä?

Eiköhän tän sinnekin voisi lisäillä.

Milloin tämä työkalu otetaan käyttöön? Meillä on tulossa toukokuussa kirjaston muutto väistötilaan, joten teenkö tarvittavat varausten muutokset itse vai teenkö tiketin kehittäjille?

En tuosta käyttöönotosta osaa/uskalla sanoa oikein mitään, tähänkin on tullut jo pari kehitysehdotusta. Kannattaa tehdä tiketti kehittäjille noista muutoksista.

ReettaSiili commented 7 months ago

Kyllä tämä ainakin lyhyellä testauksella toimii Siilin testillä.

Halutut kirjastot sai valittua valikosta, mutta vaihtoehto kaikki olisi myös hyvä. Teostieto tai linkki teostietoon olisi tärkeä tarkistusta varten. Siirtyminen raportilta työkaluun onnistui kuten piti. Työkalu on tarkoitettu varmaankin varausten muokkaamista varten, joten kokonaan perumis-vaihtoehtoa ei tässä tarvita?

Kati-Helle commented 7 months ago

Sain valittua kaikki kirjastot muokkaukseen klikkaamalla ensin listauksen ensimmäistä kirjastoa ja sen jälkeen Shift-painikkeen kera viimeistä kirjastoa.

Toimi vastaavasti, kun valitsin jostain kohtaa kirjastolistausta peräkkäiset kirjastot muokkaukseen.

Kati-Helle commented 7 months ago

Huomiona, että Modify holds -osion kohdasta New pickup library - arvoton/kirjastoton vaihtoehto puuttuu

kuva

Testasin samalla kertaa useamman eri kirjaston varausten keskeytyksen. Valitsin erämuokkaukseen nämä eri kuntien neljä kirjastoa kuva

Tekemäni toimintovalinnat erämuokkaus -toiminnossa (New pickup library -arvona oli automaattisesti Karjaan kirjasto. Kohaan kirjautumiskirjastoni on Porvoon pääkirjasto,) kuva

Muutoksen myötä kaikkien kirjastojen noutopaikkana on nyt Karjaan kirjasto. kuva

Noutokirjaston muutoskin toimii. Toiminnossa on hyvä olla mahdollisuus siihen, että noutokirjastoa ei muuteta. Näin voi kerralla muokata useamman eri kirjaston varauksien tilan.

emta001 commented 7 months ago

Yhteisöstäkin tuli palauteta. Keräilen kaikki kehitysehdotukset tähän alle:

pknuuti commented 7 months ago

Testattu Vaaran nextillä varausten erämuokkausta ja toimii kuten testilläkin.

emta001 commented 7 months ago

Kyllä tämä ainakin lyhyellä testauksella toimii Siilin testillä.

Halutut kirjastot sai valittua valikosta, mutta vaihtoehto kaikki olisi myös hyvä. Teostieto tai linkki teostietoon olisi tärkeä tarkistusta varten. Siirtyminen raportilta työkaluun onnistui kuten piti. Työkalu on tarkoitettu varmaankin varausten muokkaamista varten, joten kokonaan perumis-vaihtoehtoa ei tässä tarvita?

Hmm, meinaatko siis millä näytöllä? Ihan siinä, kun rajataan varauksia vai kun niitä muokataan?

ReettaSiili commented 7 months ago

Kyllä tämä ainakin lyhyellä testauksella toimii Siilin testillä. Halutut kirjastot sai valittua valikosta, mutta vaihtoehto kaikki olisi myös hyvä. Teostieto tai linkki teostietoon olisi tärkeä tarkistusta varten. Siirtyminen raportilta työkaluun onnistui kuten piti. Työkalu on tarkoitettu varmaankin varausten muokkaamista varten, joten kokonaan perumis-vaihtoehtoa ei tässä tarvita?

Hmm, meinaatko siis millä näytöllä? Ihan siinä, kun rajataan varauksia vai kun niitä muokataan?

Tuo peruminenko? Lähinnä itsekseni ääneen mietin, että tuleeko sellaista tilannetta jossa pitäisi syystä tai toisesta saada kokonaan peruttua isompi joukko valittuja varauksia. Eli muokkaussivulla :)

emta001 commented 7 months ago

Kyllä tämä ainakin lyhyellä testauksella toimii Siilin testillä. Halutut kirjastot sai valittua valikosta, mutta vaihtoehto kaikki olisi myös hyvä. Teostieto tai linkki teostietoon olisi tärkeä tarkistusta varten. Siirtyminen raportilta työkaluun onnistui kuten piti. Työkalu on tarkoitettu varmaankin varausten muokkaamista varten, joten kokonaan perumis-vaihtoehtoa ei tässä tarvita?

Hmm, meinaatko siis millä näytöllä? Ihan siinä, kun rajataan varauksia vai kun niitä muokataan?

Tuo peruminenko? Lähinnä itsekseni ääneen mietin, että tuleeko sellaista tilannetta jossa pitäisi syystä tai toisesta saada kokonaan peruttua isompi joukko valittuja varauksia. Eli muokkaussivulla :)

Aaa, aivan. Enpä osaa sanoa :thinking:

emta001 commented 7 months ago

Yhteisöstäkin tuli palauteta. Keräilen kaikki kehitysehdotukset tähän alle:

* [x]  Kirjastona on oletuksena valittuna kirjautumiskirjasto. Pitäisikö olla mahdollisuus hakea myös kaikkien kirjastojen varauksia kerralla?

* [x]  Varauslistassa ei ole teoksen nimekettä, tarvitsisiko olla? Jos tulee esim. tarve tarkistaa teoksen tiedoista/varausjonosta jotain? Vaihtoehtoina myös viivakoodi, nidenumero tai tietuenumero (yhteisöstä tämä).

* [x]   Voisiko varauslistauksen yläpuolella kertoa rivimäärän/varausten määrän?

* [x]  Mahdollisuus olla muokkaamatta noutokirjastoa (vain jos varauksella ei ole tilaa)

* [x]  Varaukset, joilla on jokin tila, voisi esittää esim. erilaisella taustavärillä.

Nämä on nyt tuotu testeille (ei vielä nexteillä).

MikkoLiimatainen commented 7 months ago

Muuten työkalu näyttäisi toimivan odotetusti, mutta pitäisikö sen noudattaa patron_expiration_datea? Nyt työkalu antaa muuttaa expirationdaten, vaikka varauksella on patron_expiration_date. Tietueen varausjonokin antaa muuttaa expirationdaten samassa tilanteessa, joten tämä vaikuttaisi olevan normaali toimintatapa. Yllä on määritelty, että patron_expiration_datea noudatetaan ja itse näkisin, että se tarkoittaa, ettei expirationdatea voisi muuttaa, jos patron_expiration_date on olemassa. Silloin tulisi kyllä muuttaa varausjononkin toimintaa, jotta nämä toimisivat samalla tavalla.

Tällä on vaikutusta myös siihen, miten tuo vanhentuvien haku toimii työkalussa, kun se käyttää ensisijaisena hakuehtona patron_expiration_datea. Työkalu voi listata varauksia, joissa patron_expiration_date on annetulla aikavälillä, mutta expirationdate voi olla jo jossain aivan muussa päivässä, jos sitä on muutettu esim. varausjonon kautta. Tällaisia varauksia ei välttämättä haluta muuttaa, mutta niitä voi myös olla hankala nähdä työkalun antamasta listasta.

pknuuti commented 7 months ago

Muokkauksen tuloksena näkyvä listaus näyttää valinnat Valitse kaikki / Tyhjennä kaikki, mutta niistä ei tapahdu yhtään mitään. Onko nuo täysin turhaan listauksen yläpuolella? 2023kuvat 357

emta001 commented 7 months ago

Muokkauksen tuloksena näkyvä listaus näyttää valinnat Valitse kaikki / Tyhjennä kaikki, mutta niistä ei tapahdu yhtään mitään. Onko nuo täysin turhaan listauksen yläpuolella? 2023kuvat 357

Kappas, nuo on ihan turhia tosiaan. Oon koppassut taulukon rakenteen näemmä muokkaus-sivulta ja nuo on jääneet huomaamatta.

Kati-Helle commented 7 months ago

Helle-nextissä tietueen nimeke ja niteen viivakoodi niillä varauksilla, joilla on tilana Waiting TAI In transit. Lisäksi korostusväri. *Nimekkeestä linkki tietueeseen, niteen viivakoodista linkki niteeseen. kuva

Kati-Helle commented 7 months ago

Olisiko tarpeellista ilmoittaa myös muokattujen varausten määrä? kuva

Kati-Helle commented 7 months ago

Varauslistauksessa varausten määrä näkyy kuva

emta001 commented 7 months ago

Tein muutaman muutoksen:

Kati-Helle commented 7 months ago

Helle-testissä

tietueen nimeke näkyy myös tilattomissa varauksissa (no status) kuva

muokattujen varausten määrä näkyy muokattujen varausten taulukon yläpuolella ei ole enää "Select all" ja "Clear all" valintoja kuva

ajrajala commented 7 months ago

Muuten työkalu näyttäisi toimivan odotetusti, mutta pitäisikö sen noudattaa patron_expiration_datea? Nyt työkalu antaa muuttaa expirationdaten, vaikka varauksella on patron_expiration_date. Tietueen varausjonokin antaa muuttaa expirationdaten samassa tilanteessa, joten tämä vaikuttaisi olevan normaali toimintatapa. Yllä on määritelty, että patron_expiration_datea noudatetaan ja itse näkisin, että se tarkoittaa, ettei expirationdatea voisi muuttaa, jos patron_expiration_date on olemassa. Silloin tulisi kyllä muuttaa varausjononkin toimintaa, jotta nämä toimisivat samalla tavalla.

Tällä on vaikutusta myös siihen, miten tuo vanhentuvien haku toimii työkalussa, kun se käyttää ensisijaisena hakuehtona patron_expiration_datea. Työkalu voi listata varauksia, joissa patron_expiration_date on annetulla aikavälillä, mutta expirationdate voi olla jo jossain aivan muussa päivässä, jos sitä on muutettu esim. varausjonon kautta. Tällaisia varauksia ei välttämättä haluta muuttaa, mutta niitä voi myös olla hankala nähdä työkalun antamasta listasta.

Mites tämä Mikon huomio, pitäisikö tätä vielä pureskella? 🤔 Tuo patron_expiration_datehän tulee tosiaan kaikkiin verkkokirjaston kautta tehtyihin varauksiin ja siinä kohtaa se viimeistään poikkeaa täysin expirationdatestä kun varaus on noudettavissa, koska expirationdateen päivittyy varauksen viimeinen noutopäivä muttaa patron_expiration_date jää ennalleen.

AnneliO commented 5 months ago

Koha-Suomen pääkäyttäjäpalaveri 4.6.2024: Poistetaan ehto, että noudatetaan patron_expiration_date-arvoa, koska tietueen varausjono-sivullakin muutetaan expirationdatea.

piiaseme commented 5 months ago

OUTIn testillä toolseissa ei näy linkkiä: Batch modify holds. Onko tippunut kyydistä redusoinnissa vai onko otettu pois näkösiltä muokkauksen ajaksi?

emta001 commented 5 months ago

OUTIn testillä toolseissa ei näy linkkiä: Batch modify holds. Onko tippunut kyydistä redusoinnissa vai onko otettu pois näkösiltä muokkauksen ajaksi?

Tää oli tosiaan tipahtanut redusoinnissa matkasta. Kävin nyt ajamassa tuon testeille, paitsi Vaskille, siellä on nyt joku hämy ongelma.

emta001 commented 5 months ago

Ajellaan Vaskiin tää myöhemmin, siellä taitaa olla kannan päivitys just menossa.

AnneliO commented 5 months ago

Nimeke näkyy taas vain sellaisilla varauksilla, joihin liittyy nide. kuva

LeenaKinnunen commented 3 months ago

Lapin testilläkin näyttää toimivan. Pitäisikö muokkauksen tuloksessa näkyä linkki teostietoihin tai nidenumeroon? Meillä ei näy, kuten näkyy tuossa aiemmin Katin esimerkissä Kati-Helle commented on Mar 28 ? Vai jätettiinkö linkit pois?