Projekti

Yleinen

Profile

Bugi #4043

Kirjastokortittoman asiakkaan muutettuja tietoja ei pysty tallentamaan Muokkaa-linkin kautta

Lisännyt Pirkko-Liisa Lauhikari 10 kuukautta sitten. Päivitetty 7 kuukautta sitten.

Tila:
Kommentoitu
Prioriteetti:
Normaali
Nimetty:
Luokka:
Asiakastiedot
Kohdeversio:
-
Alku:
7. Kesäkuuta 2019
Määräaika:
% Tehty:

0%

Arvioitu aika:
Yhteyshenkilö:
Votes: 0

Kuvaus

Jos asiakkaalle ei ole tallennettu asiakastietoihin Kirjastokortin numero –kenttään mitään, hänen muutettuja tietoja ei pysty tallentamaan Tiedot-sivulla olevien Muokkaa-linkkien kautta (3 kpl).
Tulee ilmoitus:

Muokkaa-painikkeen kautta tallennus onnistuu.

tallennuksen_ilmoitus.PNG (3,76 KB) tallennuksen_ilmoitus.PNG Pirkko-Liisa Lauhikari, 7. Kesäkuuta 2019 14:49
3872

Liittyvät tapahtumat

liittyy Palaute - Koha-Suomi - Bugi #2723: Asiakaslaji Yhteisöasiakas vaatii kirjautumistunnuksen muokkausnäytölläTyön alla 7. Lokakuuta 2019

Historia

#1 Updated by Johanna Räisä 10 kuukautta ago

  • Nimetty set to Emmi Takkinen

Laitetaan harjoittelija korjaamaan. :)

#2 Updated by Johanna Räisä 10 kuukautta ago

Tuohan vaikuttaa, että kannassa on useampi sama käyttäjänimi/salasana. Ei pitäisi liittyä kortinnumeroon. Kokeiltiin tuota ja ei meille antanut virhettä. Voitko laittaa borrowenumberin?

#3 Updated by Johanna Räisä 10 kuukautta ago

  • Tila changed from Uusi to Kommentoitu

#4 Updated by Pirkko-Liisa Lauhikari 10 kuukautta ago

Testitunnus, jolta otin pois kirjastokortin numerokentästä korttitunnuksen, borrowernumber=245956.
Oikea asiakas, jonka kohdalla ongelma tuli vataan, borrowernumber=238766

#5 Updated by Pasi Korkalo 10 kuukautta ago

Vaikuttaisi liittyvän siihen että korttinumero/userid -kenttään yritetään kirjoittaa null-arvon sijaan tyhjä merkkijono ''. Näiden kenttien pitää olla tietokannassa uniikkeja, joten '' voi olla korttinumerona/userid:nä vain yhdellä asiakkalla kerrallaan. Wii häv ö bög, pitää fikata se siten että tyhjän merkkijonon asemesta kantaan kirjoitetaan null silloin kun kenttä on liittymässä tyhjä.

#6 Updated by Johanna Räisä 10 kuukautta ago

Tuolla näyttäis olevan rivi: $newdata{'cardnumber'} = undef if $newdata{'cardnumber'} =~ /^\s*$/;
Sen siis tarkistaa tuon, mutta osaako joku sanoa mitä tuo tarkistaa, ei oo mun regexp niin hyvää että tietäisin mitä tuo tarkottaa.

#7 Updated by Pasi Kallinen 10 kuukautta ago

jos cardnumber sisältää vain tyhjämerkkejä tai on tyhjä merkkijono.

#8 Updated by Johanna Räisä 10 kuukautta ago

Tämä selvisi syvemmältä koodista, on ominaisuus. Ei anna tallentaa NULL-arvoa, koska se ei ole uniikki. Mikä on nyt ihan idioottia, sillä ei kaikille edes täytetä tuota.
https://github.com/KohaSuomi/Koha/commit/426a48dd19c5d3cfd5296b236b4e86f9130e1001

#9 Updated by Pirkko-Liisa Lauhikari 10 kuukautta ago

  • liittyy Bugi #2723: Asiakaslaji Yhteisöasiakas vaatii kirjautumistunnuksen muokkausnäytöllä added

#10 Updated by Johanna Räisä 10 kuukautta ago

Laitettiin testiin triggerit päälle ja silti saatiin muokattua noita. Tuo nyt näyttäisi liittyvän siihen miten noita käyttäjätunnus ja salasana kenttää käsitellään. Vaarassa ei pysty luomaan koko yhteisöasiakasta, koska siellä puuttuu nuo kentät tallennusnäytöltä.

#12 Updated by Pasi Korkalo 7 kuukautta ago

  • Nimetty changed from Emmi Takkinen to Pasi Korkalo

Mie katson tätä samalla kuin tikettiä #2723 (ovat naimisissa)

#13 Updated by Pasi Korkalo 7 kuukautta ago

Korjaus Outin testillä.

Vie Atom PDF