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

Niteiden muokkaukseen/lisäykseen nidetyypin automaattinen valinta #336

Open AnneliO opened 1 year ago

AnneliO commented 1 year ago

Kehitysehdotukseni

Redmine-tiketti 5069

Anneli Österman 21.10.2021

Niteiden muokkaukseen/lisäykseen tarvitaan toiminnallisuus (plugin/value builder), joka valitsee automaattisesti niteelle oikean nidetyypin hyllypaikan ja/tai kokoelmakoodin mukaan.

Esim. jos käyttäjä valitsee hyllypaikaksi lasten hyllypaikan, tulee nidetyypiksi lasten aineiston nidetyyppi. Tai jos valitaan aikuisten hyllypaikka ja kokoelmakoodi Lyhytlaina, tulee nidetyypiksi lyhytlainojen nidetyyppi.

Nidetyyppien mäppäykseen tarvittaneen jokin järjestelmäasetus (json?) tai tietokannan taulu.

--

Katri Sillgren 2022-05-18

Automatiikkaa myös esim. näin: kun niteelle valitaan tietty nidetyyppi, valikoituu niteelle automaattisesti tietty hyllytarkenne ja tietty nidetila.

Helle-esimerkki pikalainasta, joka ei ole varattavissa: niteelle valitaan pikalainan nideyyppi, jolloin niteelle valikoituu automaattisesti Ei varata -nidetila: Lainataan, ei varata Hyllytarkenne: Pikalaina

Kun niteen nidetyyppi vaihdetaan kirjan perusnidetyypiksi, poistuvat niteeltä automaattisesti nämä tarpeettomat: Ei varata -nidetila ja Hyllytarkenne.

--

Koha-Suomen asiantuntijaryhmän päätös 14.11.2022: Toteutetaan alkuperäisen kuvauksen mukainen toiminnallisuus ensin ja palataan toiseen toiveeseen myöhemmin. Toteutetaan versionpäivityksen jälkeen.

emta001 commented 7 months ago

Pääkäyttäjien palaveri 12.3.2024: Jätetään kenttä tyhjäksi. Lisäksi kenttään tehdään tarkistus, että kirjoitettu arvo on olemassa, jotta kantaan ei päädy olemattomia nidetyyppikoodeja.

Tämä on nyt testattavissa testeillä.

Testailkaa näin:

Tuomas-Kyyti commented 7 months ago

Tuo lista nidetyypeistä on hyvä. Myös peruuta-nappi pop-upissa toimii odotuksen mukaisesti ja voi jatkaa niteen muokkaamista. Sen sijaan ok-nappi tallentaa niteen muutokset ja nidetyyppi jää tyhjäksi tai vääräksi. Onko tuon ok-napin pakko olla siinä, ei kai esiinny tarvetta tallentaa tyhjää tai väärin kirjoitettua nidetyyppiä?

emta001 commented 7 months ago

Tuo lista nidetyypeistä on hyvä. Myös peruuta-nappi pop-upissa toimii odotuksen mukaisesti ja voi jatkaa niteen muokkaamista. Sen sijaan ok-nappi tallentaa niteen muutokset ja nidetyyppi jää tyhjäksi tai vääräksi. Onko tuon ok-napin pakko olla siinä, ei kai esiinny tarvetta tallentaa tyhjää tai väärin kirjoitettua nidetyyppiä?

Hmm, kummaa. Miun testillä siinä on vain OK-nappi ja sen painaminen ei tallenna mitään.

EDIT. Kattelin Kyytin testillä tätä ja siellä näkyy vain OK-nappi, joka ei tallenna mitään :thinking:

image

AnneliO commented 7 months ago

outi-testillä näin: kuva

Oisko tää jokin Windows - Linux -ero?

emta001 commented 7 months ago

Sinne oli jäänyt logiikkavirhe, jonka takia ton kentän täyttäminen höpöllä näyttikin confirm-ikkunan eikä alert-ikkunaa. Jos kenttä oli tyhjä, niin näytettiin alert-ikkuna. Korjasin ton nyt.

sksandel commented 7 months ago

Vaskin testillä näin. Vain OK-nappi ja kun sitä painaa pääsee takaisin niteen muokkaukseen. Puutteellinen/virheellinen tieto ei tallennu niteen muokkauksessa tai lisäyksessä.

Image

pknuuti commented 6 months ago

Vaaran testillä testattu toimivaksi. Halutun arvon voi syöttää tai kolmea pistettä klikkaamalla saada sopivan nidetyypin, jonka pystyy vielä vaihtamaan oikeanlaiseksi nidetyypiksi. Jos nidetyyppiä ei ole olemassa, tulee ilmoitus väärästä nidetyypistä.

emta001 commented 6 months ago

Siirretään tämä takaisin työn alle, nidetyypin tarkistuksesta tuli liian geneerinen ja se laukeaa nyt myös silloin, kun kuvailupohjassa ei ole itemtype.pl-plugaria käytössä.

emta001 commented 6 months ago

Siirretään tämä takaisin työn alle, nidetyypin tarkistuksesta tuli liian geneerinen ja se laukeaa nyt myös silloin, kun kuvailupohjassa ei ole itemtype.pl-plugaria käytössä.

Korjattu, nyt voi testailla taas.

Kati-Helle commented 6 months ago

Helle-testissä testasin Kirja-kuvailupohjalla. Toimivalta vaikuttaa. Kun itemtype.pl-plugin ei ole käytössä, tulee uutta nidettä lisätessä niteelle kuvailupohjaan määritellyn oletusarvon kuvaus (Nidetyypin alasvetovalikosta voi tarvittaessa vaihtaa nidetyypin) kuva

Kun itemtype.pl-plugin on käytössä, muodostuu Nidetyyppi-arvoksi nidetyypin tunnus eikä kuvaus kuva

Tietueen Perustiedot-näytöllä näkyy Nidetyypin kuvaus kuva

Tämä oli ilmeisesti tarkoitettukin toimivaksi näin, eli niteen muokkauksessa ei enää näy nidetyypin kuvausta?

emta001 commented 6 months ago

Tämä oli ilmeisesti tarkoitettukin toimivaksi näin?

Juu, näin sen kuuluu toimia. Silloin kun käytetään tuommoista input-kenttää siinä ei valitettavasti voida käyttää nidetyypin kuvausta. Input-kentillä ei ole ns. piilotettua arvoa toisin kuin select-kentillä, josta ohjelma voisi tallentaessa poimia pelkän nidetyypin tunnuksen. Joten se mitä tuossa input-kentässä näkyy, tallentuu kantaan ja jos siinä olisi nidetyypin kuvaus, niin silloin se tallentuisi.

Kati-Helle commented 6 months ago

Lisähuomiona: Nidetyyppi-kentän arvoksi voi kirjoittaa sellaisen olemassa olevan nidetyyppiarvon tunnuksen, joka ei vastaa niteelle valittuja arvoja. Niteen tallennus onnistuu. Tämäkin lienee ok?

emta001 commented 6 months ago

Lisähuomiona: Nidetyyppi-kentän arvoksi voi kirjoittaa sellaisen olemassa olevan nidetyyppiarvon tunnuksen, joka ei vastaa niteelle valittuja arvoja. Niteen tallennus onnistuu. Tämäkin lienee ok?

Juu, kyllähän sitä pitää olla kuitenkin mahdollista tehdä poikkeuksiakin noihin.

Kati-Helle commented 5 months ago

Muodostuuko hankintaportaalin kautta tehdyille tilauksille oikea/oikeantyylinen nidetyyppi, kun nidetyypin automaattinen valinta on käytössä? Onko niin, että tällä hetkellä nidetyyppi muodostuu hankintaportaalin kautta tilatulle niteelle kuvailupohjaan määritellyn 952|y-osakentän oletusarvon perusteella?

AnneliO commented 5 months ago

Tällä hetkellä EDItX-tilauksissa nidetyyppi muodostuu ONIX-koodien perusteella. Niistä olette aikanaan tehneet mäppäyksen, mitä nidetyyppiä mikäkin ONIX-koodi vastaa. Kuvaus EDItX-hankinta-esitelyssä.

Tämän tiketin kommentissa on kirjattuna, että EDItX:n kautta tuleville niteille olisi tarkoitus ottaa käyttöön sama logiikka kuin tässä liitännäisessä. Mutta sen muutosta ei ole vielä aloitettu tekemään ja ei vielä tiedetä, tuleeko se toimimaan.

sksandel commented 5 months ago

Vaskin testillä toimii kuten speksattu. Testasin pluginia muutamalla eri pohjalla (Kirja, Kausijulkaisu, Konsolipeli). Jos itemtype.pl-plugin ei ole käytössä, alasvetovalikko toimii kuten ennenkin.

emta001 commented 2 months ago

Voisiko joku tehdä tiketin tuosta vastaavasta toteutuksesta EditX:ään? Muuten se unohtuu takuuvarmasti.

AnneliO commented 2 months ago

Olit jo itse tehnyt: https://github.com/KohaSuomi/koha-plugin-editx/issues/2 :)

emta001 commented 2 months ago

Olit jo itse tehnyt: KohaSuomi/koha-plugin-editx#2 :)

No kappas, niinpä oonkin :smile:

Tuomas-Kyyti commented 3 weeks ago

Kun nidetyyppien automaattinen määritys on päällä ja menee niteiden erämuokkauksessa muuttamaan nidetyyppiä, pitää nidetyypin lyhenne kirjoittaa manuaalisesti. Jos tässä tekee kirjoitusvirheen, ajaa erämuokkauksen ja menee kyseisen niteen tietueelle, tulee virheilmoitus "Jotain meni pieleen taulua ladattaessa. 500: Internal Server Error." Kyseistä nidettä ei enää näy perustietonäkymässä eikä nidetyyppiä voi täten siinä korjata. Finnassa tämä näkyy hakutuloslistassa "Tapahtui virhe" ja tietuenäytössä "Järjestelmä pois käytöstä. Kirjastojärjestelmä on juuri nyt pois käytöstä. Saatavuustiedot eivät ole juuri nyt käytettävissä. Pahoittelemme tästä aiheutunutta vaivaa. Voitte ottaa yhteyttä:"

Jos viivakoodinumero on tallessa, niin uudella erämuokkauksella voi korjauksen tehdä, mutta jos tätä ei heti huomaa, pitää varmaan jollain raportilla kaivaa esiin niteet, joissa on epäkelpo nidetyyppi. Ja tätähän ei välttämättä huomaa pitkään aikaan.

Voiko niteiden erämuokkaukseen saada validoinnin nidetyyppikentään, kuten niteen muokkausnäkymässäkin on? Vai olisiko tähän jotain muuta ratkaisua?

Testattu sekä 23.11 että 24.05 versioissa.

emta001 commented 3 weeks ago

Tutkin tuota Tuomaksen huomaamaa ongelmaa ja vaikuttaisi sille, että value_builder-pluginien käyttö on mennyt rikki yhteisön bugin 28445 yhteydessä. Ainakin siinä koodiin on tullut seuraava hoksautus:

[%# FIXME - from batchMod-edit, jQuery is included at the end of the template and cataloguing plugins are not working in this situation %]

En vielä sen kummemmin ehtinyt perehtymään siihen kuinka tämän voisi korjata, joten en osaa sanoa kauan korjaaminen kestää. Siihen asti, kannattanee joko olla käyttämättä nidetyypin automaattista generointia tai sitten hoksauttaa käyttäjiä siitä, ettei se toimi erämuokkauksessa ja kehottaa olemaan erityisen tarkkana nidetyyppiä lisätessä.

emta001 commented 2 weeks ago

Nyt on testailtu lisää ja vika onkin itse plugarissa. Jos niteitä viedään erämuokkaukseen listan kautta, sivulle ei välity tietueen biblionumberia. Sitä tarvitaan aineistolajin määrittämiseen, joka on oikeastaan ainut pakollinen ehto nidetyypin generointiin. Tämän takia generointi toimii erämuokkauksessa tällä hetkellä vain silloin, kun niteet tuodaan siihen tietueen perustiedotnäytöltä.

atarnus commented 1 week ago

OUTIssa toivottiin nidetyypit listaavaan virheilmoitukseen mukaan myös nidetyyppien kuvaukset. Nidetyyppien tunnukset ja kuvaukset ovat vuosien saatossa eronneet toisistaan, joten tunnusten opettelua ja muistamista varten olisi mahtavaa jos se kuvauskin vielä sieltä löytyisi.

image (Meillä siis esim. nuo 2LA-tunnukset tarkoittavat nykyään 4 lainaa, joka aiheuttaa ymmärrettävästi hämmennystä.)

Listauksessa tosiaan riittäisi hyvin, että näkyy:

atarnus commented 1 week ago

Katin viestissä yllä on tilanne, jossa nidetyypistä näkyy edelleen kuvaus tietueen kokoelmalistauksessa vaikka tämä plugari on käytössä. Tällä hetkellä kuitenkin sekä testillä että tuotannossa korvaantuu Nidetyypin kuvaus tunnuksella.

image

Olisiko tietuesivulle mahdollista saada takaisin nidetyypin kuvausta? Kuvaus on tosiaan ajantasaisempi kuin aikoinaan luotu tunnus, ja tässä kohdassa hämmennystä aiheuttaa myös se, että muutamilla pohjilla plugari ei ole käytössä, ja näitä pohjia käyttävien niteiden kohdalla näkyy edelleen kuvaus. Eli johdonmukaisuus puuttuu.

emta001 commented 1 week ago

Katin viestissä yllä on tilanne, jossa nidetyypistä näkyy edelleen kuvaus tietueen kokoelmalistauksessa vaikka tämä plugari on käytössä. Tällä hetkellä kuitenkin sekä testillä että tuotannossa korvaantuu Nidetyypin kuvaus tunnuksella.

image

Olisiko tietuesivulle mahdollista saada takaisin nidetyypin kuvausta? Kuvaus on tosiaan ajantasaisempi kuin aikoinaan luotu tunnus, ja tässä kohdassa hämmennystä aiheuttaa myös se, että muutamilla pohjilla plugari ei ole käytössä, ja näitä pohjia käyttävien niteiden kohdalla näkyy edelleen kuvaus. Eli johdonmukaisuus puuttuu.

Tästä oli puhetta versionvaihdon vastaavassa tiketissä ja kommentoinkin sinne selityksen tälle https://github.com/KohaSuomi/Koha-24.05/issues/34#issuecomment-2360673832. Lyhyesti siis, ei valitettavasti onnistu. Yhteisössä halutaan, että tietyissä kuvailupohjan kentissä käytetään nimenomaan auktorisoitua arvoa jotta niiden kuvaus näkyy ja 952$y on yksi näistä kentistä.

atarnus commented 1 week ago

Jahas, onnistuisikohan se ruuvata joku oma JS-rimpsu siihen päälle, jos ei osata elää ilman.

emta001 commented 1 week ago

Rupesinpa aattelemaan tätä tiketöintiä vähän tarkemmin ja tiedottamisen ja muun kannalta voiskin olla parempi, että tehdään näistä parista uudesta ongelmasta omat tikettinsä. @Tuomas-Kyyti ja @atarnus, viittiteks tehä semmoset, niin voidaan sitten sulkea tämä tiketti kokonaan.

Tuomas-Kyyti commented 1 week ago

Ongelma niteiden erämuokkauksessa siirretty tikettiin https://github.com/KohaSuomi/Koha/issues/1510

atarnus commented 1 week ago

Virheilmoituksen kehitysehdotus siirretty: https://github.com/KohaSuomi/Koha/issues/1517

emta001 commented 1 week ago

Jees, eiköhän tämänkin voi nytten sulkea.