Tietueen kuvailun virheilmoituksen Mene kenttään -linkki ei toimi

[Kati-Helle]

Mikä vikana?

Kuvailu-toiminto ilmoittaa tietuetta tallentaessa, kun tietueelta puuttuu esim. pakollisen kentän arvo.
Ilmoituksessa on mainittuun puuttuvaan kenttään Mene kenttään -linkki, joka ei nyt vie ilmoituksessa mainittuun kenttään eikä muuallekaan.

Esimerkki Helle-tietueesta (biblionumber=1354970). Tietueelta puuttuu pakollinen 084-kenttä.

Mitä pitäisi tapahtua

No response

Kuinka toistaa ongelma/asia

No response

Selain

No response

Jotain muuta?

No response

[Kati-Helle]

Sain tiedon, että 942c eli aineistotyyppikentän osalta Koha oli toiminut vastaavalla tavalla.
Hellen Tietueiden yhdistämissäännöissä 942c-kenttä on suojattu poistamiselta.
(Onko niin, että TäTi-tietueella ei edelleenkään pitäisi olla 942c-kenttää eli aineistotyyppiä?)

[Kati-Helle]

Testasin tietukorvauksen uudestaan, nyt Helle-tietueelle biblionumber=1354970.
(Tällä Helle-tietueella on 084-kenttä, jonka puuttumisesta Koha virheilmoittaa yhdistelyn jälkeen. 084-kenttää ei ole enää tietueyhdistelyn jälkeen, koska kenttä ei ole yhdistämissäännöissä suojattu ja kenttää ei ole Helle-tietueen korvaavassa TäTi-tietueessa.)

Helle-tietueen kentät

TäTi-tietueen kentät

Korvaan Helle-tietueen ja tarkistan 942c-kentän. Helle-tietueella on edelleen aineistotyyppi

Tallennan Helle-tietueen

Klikkaan ilmoituksen Mene-kenttään-linkkiä. Linkki ei vie 084-kenttään, vaan 9-välilehden alkuun

Siirryn 0-välilehdelle, jossa tietueen 084-kenttä tämä

[AnneliO]

Testasin tätä Biblibren sandboxissa ja siellä Mene-linkit toimivat. Kokeilin myös poistaa käytöstä testiltä kaikki IntranetUserJS:t ja IntranetUserCSS:t, eikä niiden poistolla ollut merkitystä. Muistelen myös, että olisin noita linkkejä nexteillä jossain vaiheessa testannut ja ne toimi silloin. Mutta en ole muistikuvasta 100 % varma.

Kommenteissa mainitut yhdistelyt ja tietuekorvaamiset ei ymmärtääkseni liity tähän linkki-ongelmaan, joten teetkö Kati niistä tarvittaessa omat tikettinsä (yksi ongelma/tiketti). :)

[Kati-Helle]

Tarpeetontahan tuo korvaustoiminnon käyttö oli ongelman kannalta eli tarvetta uuteen tikettiin ei Anneli ole :)
Tein tiketin esimerkin Helle-tietueen korvaamalla eli samaan tapaan, kuin ongelmaan törmännyt oli toiminut. Yhdistely ja yhdistelysäännöt toimivat oikein.

[pknuuti]

Itse asiaan eli otsikossa määriteltyyn toimintaan. Tarkoitko Anneli, että versionvaihdon jälkeen tämä ongelma pitäisi olla korjattuna? Näyttäisi liittyvän suomenkieliseen pohjaan eli kun vaihdoin kielen englanniksi, linkki toimi.

[Kati-Helle]

Kokeilin Helle-tuotannon suomenkielisessä Kohassa. Kuvailu-osion Uusi tietue -toiminto avasi uuden tietueen. Klikkasin tietueen Tallenna-painiketta lisäämättä tietueelle tietoja. Tietue-näkymää tallennuksen jälkeen:

Kokeilin kaikkien kuvassa näkyvien kenttien Mene kenttään -linkit tietueen Osa 0 -osiossa:

• 005 toimi
• 008 toimi
• 041 toimi
• 084 toimi
• 245 ei toiminut
• 942 ei toiminut

Siirryin tietueen Osa 2 -osioon

• 245 toimi (molemmat 245-kentät toimivat)

Siirryin tietueen Osa 9 -osioon

• 942 toimi

[AnneliO]

Testailin nyt myös ja kuten Kati tuossa on huomannut, niin linkki toimii, jos ensin menee itse oikealle välilehdelle ja sitten klikkaa Mene-linkkiä. Kielivalinnalla ei ole merkitystä eli sekä suomenkielisessä että englanninkielisessä tapahtuu tuo sama.

PTFS Europen sandboxissa linkki osaa mennä automattisesti myös oikealle välilehdelle.

Kun hiiren vie linkin päälle ja katsoo sen osoitetta, niin linkin osoite on sama sekä meidän versiossa että sandboxissa. Ja jos linkkejä katsoo selaimen Tarkastele-toiminnolla, niin minun silmään ne näyttää samanlaisilta meillä ja sandboxissa.

[atarnus]

Nykyversiossa addbiblio.tt:ssä selectTab() -funktiolle syötettävä parametri on jostain syystä vääränlainen. Nextillä siihen on tullut muutos, ja tabin vaihto toimii taas. Nextin mukainen koodinpätkä ei kuitenkaan toimi vielä nykyversiossa, se on riippuvainen jostain muista muutoksista.

Sekä minun fiksillä että nextissä esiintyy tabi-asian korjaannuttua toista ongelmaa: sivu ei skrollaannu ihan perille asti haluttuun kohtaan tabin vaihtumisen jälkeen (esim. ruutu hyppää kentän 910 kohdalle eikä mene kenttään 942 asti). Asia vaikuttaa korjaantuvan lisäämällä pienen viiveen vaihdon ja skrollauksen väliin (esim. 200 millisekuntia).

Tämä viive on tarpeen siis myös uudessa versiossa, ja kannattaa ottaa testattavaksi.

Fiksattu funktio addbiblio.tt:ssä:

$("body").on("click", ".linkfield", function(e){
  e.preventDefault();
  var tab = $(this).data("tab");
  var field = $(this).data("field");
  var tablink = "#tab" + tab + "XX";
  selectTab( tablink );
  setTimeout(window.scrollTo, 200, 0, getScrollto( field, "toolbar" ));
});

[atarnus]

Githubissa. Vietävä testeille.

[atarnus]

Tosiaan tabin vaihto toimii, mutta skrollaus oikeaan kohtaan sivua ei.

Samanlaista skrollausvikaa saattaa esiintyä muuallakin, kun sivu ei ehdi latautua kokonaan ennen kuin se hakee skrollaukselle koordinaatit. Tähän voisi mahdollisesti olla joku universaalimpi fiksi. Tässä nyt tuo tabin vaihto aiheuttaa vikakoordinaatteja ja timeoutin pitäisi se korjata.

Pienellä timeoutilla näyttäisi ainakin minulla toimivan mainiosti. Eli 23-versiossa korvataan vain funktion viimeinen rivi:
setTimeout(window.scrollTo, 100, 0, getScrollto( field, "toolbar" ) );

Tämä nyt Githubissa. Tuotava nexteille.

[Kati-Helle]

Helle-nextissä toimii.
Seuraavia pakollisia osakenttiä ei täytetty -ilmoituksen Mene kenttään -linkki vie kyseiseen arvottomaan kenttään. Kentässä on keltainen korostusväri.

[MikkoLiimatainen]

Vaski-nextissä toimii samoin kuin Hellessäkin