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

Emotietueesta osakohteeseen siirtyvät tiedot ja RDA #705

Open piiaseme opened 11 months ago

piiaseme commented 11 months ago

Kehitysehdotukseni

Alkuperäinen tiketti: "Pystyykö Kohassa määrittämään, mitkä tiedot siirtyvät emotietueesta automaattisesti osakohteen 773-kenttiin, kun tekee uuden osakohteen?"

"Koska musiikkiaineistonkin kuvailu on RDA:n mukaista, niin määrityksiä tulisi muuttaa. Anneli on joitain vuosia sitten tutkinut tätä ja todennut, ettei käyttöliittymän kautta ei pysty määrittämään, mitä tietoja emosta siirtyy osakohteeseen."

Selvisi, että määrityksille ei ole mitään järjestelmäasetusta olemassa, vaan kentät on kovakoodattu Kohan lähdekoodiin. (C4\Biblio.pm, prepare_host_field).

PasiK teki halutut muutokset ja siirsi ne tuotantoon seuraavasti:

kuva

Jotain muuta?

Siirretty Redminesta: https://tiketti.koha-suomi.fi/issues/3509

Liittyy tikettiin: https://github.com/KohaSuomi/Koha/issues/704

AnneliO commented 4 months ago

Indeksointiryhmä 7.3.2024: Tämä muutos tarvitaan edelleen.

AnneliO commented 1 month ago

Nämä on aiemmin jo korjattu, mutta korjaukset katosivat versiopäivityksen yhteydessä, mutta ovat edelleen tarpeellisia. Tässä uudelleen päivitettynä, miten tiedot pitäisi saada siirtymään emotietueesta osakohteen 773-kenttään:

Emon kenttä Osakohteen kenttä Huomioita
100ab TAI 110ab TAI 111ab 773a Tämän ei tarvitse enää siirtyä
245abnpc 773t
264abc 773d
240 773s Ei tarvita enää
022a 773x
020a 773z
001 773w
300ae 773h
028ba 773o

Päätös: Toteutetaan yllä kuvaillusti ja tarjotaan muutosta myös Koha-yhteisöön.

piiaseme commented 4 weeks ago

Bugzillassa tiketti: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37061

johannaraisa commented 3 weeks ago

Nyt on testeillä PrepareHostField-järjestelmäasetus, mihin voi määritellä halutut kentät. Asetuksen esimerkissä kerrotaan miten määritykset tehdään. Jos ei määrittele mitään, niin pitäisi käyttää kovakoodattua, eli nykyistä tapaa.

AnttHeik commented 3 weeks ago

Kerrassaan mainiota. Ehtisikö @piiaseme tai @AnneliO laittamaan nämä asetukset päälle Testi-TäTissä, niin pääsee testaamaan? :)

piiaseme commented 3 weeks ago

@AnttHeik kävin tallentamassa kentät Koha-testille. Pystytkö testaamaan siellä? Jostain syystä en pääse testi-TäTiin, joten sinne tallennan, kun pääsen sisälle sinne.

johannaraisa commented 3 weeks ago

@AnttHeik kävin tallentamassa kentät Koha-testille. Pystytkö testaamaan siellä? Jostain syystä en pääse testi-TäTiin, joten sinne tallennan, kun pääsen sisälle sinne.

Testi-Täti on redusoitu, toimiikohan tuotantotunnus siellä?

piiaseme commented 3 weeks ago

Anneli tsekkasi, ettei minun tunnusta ole testi-Tätissä. Kodo tekee siirron.

piiaseme commented 3 weeks ago

Nyt kentät on tallennettu myös testi-TäTiin.

AnttHeik commented 3 weeks ago

No se toimii just niin kuin pitääkin.

Tosin OUTIn testillä huomasin, että jos julkaisutiedot ovatkin vanhaan malliin 260-kentässä, niin silloin ei siirry mitään 773d-kenttään. Onko mahdollista määrittää useampi lähtökenttä joihinkin kohtiin eli 260abc | 773d 264abc | 773d

Lisäksi pohdin vielä kahta pientä kehitysjuttua, mutta jos nämä vaikuttavat hankalilta toteuttaviksi, niin ei tarvitse miettiä pidemmälle, koska tämänhetkiselläkin toteutuksella pärjää oikein hyvin :)

(Testi-Tätistä ei muuten tuntunut löytyvän nuotteja ollenkaan. Onkohan ne redusoitu pois?)

piiaseme commented 3 weeks ago
* Entä jos sattuisi olemaan vaikka useampi 028-kenttä tai 020-kenttä, niin olisiko mahdoton ajatus, että nämä kaikki muodostuisi osakohteeseen omiksi 773o- tai 773z-toistumikseen?

* Entä olisiko mahdollista säätää kuvailupohjakohtaisesti, mitä kenttiä 773-kenttiin lähetetään eli katsottaisiin emon kuvailupohja ja sen perusteella sitten lähetettäisiin tiedot?

Jos Johannan mielestä nämä on toteutettavissa, niin teen tiketit näistä ja linkkaan ne tähän tikettiin.

piiaseme commented 3 weeks ago

Tosin OUTIn testillä huomasin, että jos julkaisutiedot ovatkin vanhaan malliin 260-kentässä, niin silloin ei siirry mitään 773d-kenttään. Onko mahdollista määrittää useampi lähtökenttä joihinkin kohtiin eli 260abc | 773d 264abc | 773d

Voin lisätä testeille, niin nähdään miten lisäys reagoi.

AnttHeik commented 3 weeks ago

Tosin OUTIn testillä huomasin, että jos julkaisutiedot ovatkin vanhaan malliin 260-kentässä, niin silloin ei siirry mitään 773d-kenttään. Onko mahdollista määrittää useampi lähtökenttä joihinkin kohtiin eli 260abc | 773d 264abc | 773d

Voin lisätä testeille, niin nähdään miten lisäys reagoi.

Jeps, hyvä juttu, testataan.

piiaseme commented 3 weeks ago

Nyt on lisätty OUTIn testille sekä TäTin testille.

AnttHeik commented 3 weeks ago

Kiitos, hyvin näyttää toimivan, paitsi OUTIssa vissiin unohtui laittaa 260c-kenttä mukaan? TäTin puolella tuli sekin nätisti :)

piiaseme commented 3 weeks ago

(Testi-Tätistä ei muuten tuntunut löytyvän nuotteja ollenkaan. Onkohan ne redusoitu pois?)

On siirtynyt yksi nuotti: 2460995 Löydätkö tällä tiedolla?

Tsekkaan tuon puuttuvan kentän ja lisään, jos puuttuu.

AnttHeik commented 3 weeks ago

(Testi-Tätistä ei muuten tuntunut löytyvän nuotteja ollenkaan. Onkohan ne redusoitu pois?)

On siirtynyt yksi nuotti: 2460995 Löydätkö tällä tiedolla?

Joo, tein tuon just itse testitietueeksi :)

Tsekkaan tuon puuttuvan kentän ja lisään, jos puuttuu.

Ok.

johannaraisa commented 3 weeks ago

Kokeilitteko tuota useampaa lähtökenttää, se saattaa jo toimia tässä. Lisäksi se jos on esim. monta 020-kenttää. Kuvailupohjan mukaan ottaminen vaatii kyllä hieman enemmän, ku se tieto pitäisi hakea jostain, tuo saattaa käyttää oletuspohjaa aina kun tehdään osakohde.

piiaseme commented 3 weeks ago

Kokeilitteko tuota useampaa lähtökenttää, se saattaa jo toimia tässä.

Lisäsin kentät ja Antti taisi jo eilen kokeilla ja todeta, että hyvin toimi sekä OUTin että TäTin testillä.

Lisäksi se jos on esim. monta 020-kenttää.

Tätä ei testattu. Voisin lisätä, niin tulee sekin testattua. Antti, onko jotkin tietyt kentät, joiden haluaisit toistuvan?

Kuvailupohjan mukaan ottaminen vaatii kyllä hieman enemmän, ku se tieto pitäisi hakea jostain, tuo saattaa käyttää oletuspohjaa aina kun tehdään osakohde.

Tehdäänkö tästä oma tiketti?

AnneliO commented 3 weeks ago

Testailin lisäämällä tietueelle kahdet 020a- ja 028a-kentät ja vain ensimmäiset toistumat siirtyivät osakohteeseen. Mäppäykseen ei pystynyt lisäämään toista kertaa samaa kenttää, se valitti tuplatiedosta.

johannaraisa commented 3 weeks ago

Testailin lisäämällä tietueelle kahdet 020a- ja 028a-kentät ja vain ensimmäiset toistumat siirtyivät osakohteeseen. Mäppäykseen ei pystynyt lisäämään toista kertaa samaa kenttää, se valitti tuplatiedosta.

Niin, se on jo avainarvona, mutta mitä jos tietueessa on useampi esiintymä?

johannaraisa commented 3 weeks ago

Kuvailupohjan mukaan ottaminen vaatii kyllä hieman enemmän, ku se tieto pitäisi hakea jostain, tuo saattaa käyttää oletuspohjaa aina kun tehdään osakohde.

Tehdäänkö tästä oma tiketti?

Ehkä tämä on jo uusi tiketti, tuo muutos tekee nyt ainakin sen mitä siihen Bugzillan tikettiin laitoit.

AnneliO commented 3 weeks ago

Testailin lisäämällä tietueelle kahdet 020a- ja 028a-kentät ja vain ensimmäiset toistumat siirtyivät osakohteeseen. Mäppäykseen ei pystynyt lisäämään toista kertaa samaa kenttää, se valitti tuplatiedosta.

Niin, se on jo avainarvona, mutta mites jos tietueessa on useampi esiintymä?

Kuten tuossa ekassa lauseessa kerroin. :) Eli vain ensimmäiset toistumat siirtyivät osakohteeseen.

AnttHeik commented 3 weeks ago

Kiitokset vastauksista ja testailuista. Minusta voidaan mennä näillä säädöillä. Tässä on kuitenkin se hyvä juttu, että käyttäjät voivat nyt itse määritellä, mitä kenttiä haluavat osakohteeseen siirtyvän.

AnneliO commented 2 weeks ago

Suomenkieliset muutokset järjestelmäasetuksiin viety GitHubiin Koha-translations-tietovarantoon.

pirliis commented 1 week ago

Nyt on lisätty OUTIn testille sekä TäTin testille.

Asetukset lisätty OUTIn tuotantoon 18.6. ja TäTin tuotantoon 25.6.