Projekti

Yleinen

Profile

Bugi #4650

Järjestelmäasetus: ValidatePhoneNumber

Lisännyt Anneli Österman yli vuosi sitten. Päivitetty 5 päivää sitten.

Tila:
Ratkaisu ehdotettu
Prioriteetti:
Kiireellinen
Luokka:
Siirretään
Alku:
12. Marraskuuta 2020
Määräaika:
% Tehty:

90%

Arvioitu aika:
Yhteyshenkilö:
Tiketti Bugzillassa:

Kuvaus

ValidatePhoneNumber-järjestelmäasetukseen pystyy kirjoittamaan RegEx-säännön, jonka perusteella tarkistetaan asiakkaan lisäys/muokkausnäytöllä, että puhelinnumero on säännön mukaisesti tietyssä muodossa. Tämä vähentää esim. näppäilyvirheitä, joissa nollan tilalla on o-kirjain.

Päätös: Säilytetään tavalla tai toisella, koska se parantaa tiedon oikeellisuutta. Tutkitaan mahdollisuutta toteuttaa toiminnallisuus JavaScriptinä IntranetUserJS-järjestelmäasetukseen, jolloin se ei muuta Kohan lähdekoodia.

Helle-testauksen näyttökuvia.PNG (78,5 KB) Helle-testauksen näyttökuvia.PNG Katri Sillgren, 17. Toukokuuta 2022 08:57
5477

Liittyvät tapahtumat

liittyy Versionvaihto - Bugi #5348: Asiakastiedot, puhelinnumeroon saa tallennettua tyhjämerkkejä. Koskee kenttiä Primary phone ja Other phoneRatkaisu ehdotettu19. Huhtikuuta 2022

Historia

#1 Updated by Emmi Takkinen yli vuosi ago

  • Tila changed from Uusi to Ratkaisu ehdotettu

Teknisen dokumentaation puolella on nyt ehdotus javascript-rimpsusta, jolla järjestelmäasetuksen voisi mahdollisesti korvata:

https://tiketti.koha-suomi.fi/projects/koha-suomen-dokumentaatio/wiki/IntranetUserJS#Puhelinnumeron-validointi

#2 Updated by Anneli Österman 4 kuukautta ago

  • Nimetty set to Anneli Österman

#3 Updated by Anneli Österman 2 kuukautta ago

  • % Tehty changed from 0 to 90

Testattava

#4 Updated by Anneli Österman noin 2 kuukautta ago

  • Tila changed from Ratkaisu ehdotettu to Kommentoitu
  • % Tehty changed from 90 to 80

Huomioita:

  • tarkistus on vain phone-kentässä, tarvitaan myös mobile- ja smsalertnumber-kenttiin
  • jos tarkistettavaan kenttään on kertaalleen kirjoittanut väärän tiedon, niin että siitä tulee ilmoitus, muuttuu tallenna-nappi epäaktiiviseksi (tai sitä voi klikata, mutta mitään ei tapahdu). Se ei aktivoidu, vaikka virheellisen tiedon korjaa.

#5 Updated by Emmi Takkinen noin 2 kuukautta ago

  • Tila changed from Kommentoitu to Työn alla
  • Nimetty changed from Anneli Österman to Emmi Takkinen

#6 Updated by Emmi Takkinen noin 2 kuukautta ago

  • Tila changed from Työn alla to Ratkaisu ehdotettu
  • Nimetty changed from Emmi Takkinen to Anneli Österman
  • % Tehty changed from 80 to 90

Skriptiä paranneltu.

#7 Updated by Anneli Österman noin 2 kuukautta ago

Uusi versio lisätty rimpsukirjastoon

#8 Updated by Katri Sillgren noin 2 kuukautta ago

Helle-testissä havaittua:
Hyväksyy ulkomaisen suuntanumeron.

Ensisijainen puhelin -kenttä ja Muu puhelin -kenttä: ylimääräiset tyhjäarvot numeroiden välissä eivät katoa tallentaessa. (Puhelinnumeron jälkeen lisätyt tyhjä-arvot poistuivat tallentaessa.)

Tekstiviesti numeroon -kenttä vaikutti toimivan oikein: ylimääräiset tyhjäarvot numeroiden välissä ja puhelinnumeron jälkeiset tyhjä-merkit häviävät tallentaessa. Mutta: muutaman kerran sain tämän aikaan: kun tallennuksen jälkeen klikkaa kenttää, tulevat ylimääräiset tyhjäarvot näkyviin.

#9 Updated by Anneli Österman noin 2 kuukautta ago

Katri Sillgren kirjoitti:

Helle-testissä havaittua:
Hyväksyy ulkomaisen suuntanumeron.

Siinä on nyt sääntö, joka estää vain kirjaimien kirjoittamisen. Kokeile kopioida tuotannosta ValidatePhoneNumber-kentästä teidän nykyinen sääntö skriptiin phone_reg ja mobile_reg-kohtiin.

Ensisijainen puhelin -kenttä ja Muu puhelin -kenttä: ylimääräiset tyhjäarvot numeroiden välissä eivät katoa tallentaessa. (Puhelinnumeron jälkeen lisätyt tyhjä-arvot poistuivat tallentaessa.)

Tekstiviesti numeroon -kenttä vaikutti toimivan oikein: ylimääräiset tyhjäarvot numeroiden välissä ja puhelinnumeron jälkeiset tyhjä-merkit häviävät tallentaessa. Mutta: muutaman kerran sain tämän aikaan: kun tallennuksen jälkeen klikkaa kenttää, tulevat ylimääräiset tyhjäarvot näkyviin.

Näihin liittyy erilliset skriptit IntranetUserJS:ssä. Ilmeisesti jotain on muuttunut sivulla vanhaan versioon verrattuna, eikä skripti toimi enää oikein. Teetkö näistä erillisen tiketin? :)

#10 Updated by Anneli Österman noin 2 kuukautta ago

  • Prioriteetti changed from Normaali to Kiireellinen

#11 Updated by Katri Sillgren noin kuukausi ago

Anneli Österman kirjoitti:

Katri Sillgren kirjoitti:

Helle-testissä havaittua:
Hyväksyy ulkomaisen suuntanumeron.

Siinä on nyt sääntö, joka estää vain kirjaimien kirjoittamisen. Kokeile kopioida tuotannosta ValidatePhoneNumber-kentästä teidän nykyinen sääntö skriptiin phone_reg ja mobile_reg-kohtiin.

Ensisijainen puhelin -kenttä ja Muu puhelin -kenttä: ylimääräiset tyhjäarvot numeroiden välissä eivät katoa tallentaessa. (Puhelinnumeron jälkeen lisätyt tyhjä-arvot poistuivat tallentaessa.)

Tekstiviesti numeroon -kenttä vaikutti toimivan oikein: ylimääräiset tyhjäarvot numeroiden välissä ja puhelinnumeron jälkeiset tyhjä-merkit häviävät tallentaessa. Mutta: muutaman kerran sain tämän aikaan: kun tallennuksen jälkeen klikkaa kenttää, tulevat ylimääräiset tyhjäarvot näkyviin.

Näihin liittyy erilliset skriptit IntranetUserJS:ssä. Ilmeisesti jotain on muuttunut sivulla vanhaan versioon verrattuna, eikä skripti toimi enää oikein. Teetkö näistä erillisen tiketin? :)

Tein tiketin https://tiketti.koha-suomi.fi/issues/5348

#12 Updated by Anneli Österman 25 päivää ago

Tarkistusskripti muuttaa suomen- ja ruotsinkielisessä versiossa Tallenna-napin nimeksi Save. Saisiko siihen käännökset?

#13 Updated by Anneli Österman 25 päivää ago

  • Nimetty changed from Anneli Österman to Emmi Takkinen

#14 Updated by Emmi Takkinen 25 päivää ago

  • liittyy Bugi #5348: Asiakastiedot, puhelinnumeroon saa tallennettua tyhjämerkkejä. Koskee kenttiä Primary phone ja Other phone added

#15 Updated by Emmi Takkinen 25 päivää ago

Anneli Österman kirjoitti:

Tarkistusskripti muuttaa suomen- ja ruotsinkielisessä versiossa Tallenna-napin nimeksi Save. Saisiko siihen käännökset?

Käännökset lisätty, testattu ja korjattu tänne.

#16 Updated by Anneli Österman 25 päivää ago

Testasin OUTIssa ja minusta toimi oikein. Napissa kääännökset oikein ja puhelinnumeron validointi toimi edelleen.

#17 Updated by Anneli Österman 12 päivää ago

Rimpsuun tuli pieni muutos. Lisäsin uusimman version Järjestelmäasetusmuutoksia -sivulle.

#18 Updated by Pia Kusmin 12 päivää ago

Lappiin tarvitaan oma rimpsu?
Meillä on aiemmin ollut käytössä ValidatePhoneNumber määritys: ^(\+)?[0-9]*$

Meillä on käytössä paljon ulkomaisia puheinnumeroita.

#19 Updated by Anneli Österman 12 päivää ago

Pia Kusmin kirjoitti:

Lappiin tarvitaan oma rimpsu?
Meillä on aiemmin ollut käytössä ValidatePhoneNumber määritys: ^(\+)?[0-9]*$

Meillä on käytössä paljon ulkomaisia puheinnumeroita.

Lisää tuo teidän oma rimpsu niihin "var phone_reg", "var mobile_reg" ja "var SMSnumber_reg" kohtiin. :)

#20 Updated by Katri Sillgren 11 päivää ago

Helle-testissä uusin rimpsu -testaus: puutteita edelleen

Primary phone -kenttään lisätyt ylimääräiset välilyönnit ja lopun ylimääräinen välilyönti: ei ilmoitusta, ei automaattista poistoa. Asiakastiedon saa tallennettua.

Primary phone -kenttään lisätty puhelinnumero: kopioitui SMS-number-kenttään, kun testasin tämän ensimämäisen kerran. Tuon jälkeen en saanut kopioitumaan eri puhelinnumerolla samalla asiakkaalla enkä eri asiakkaalla. ctrl+F5 ei auttanut

Primary phone -kentästä poistettu puhelinnumero ei poistunut SMS-number-kentästä

Other phone -kenttään lisätyt ylimääräiset välilyönnit ja lopun ylimääräinen välilyönti: ei ilmoitusta, ei automaattista poistoa. Asiakastiedon saa tallennetua

Suomenkielisessä käyttöliittymässä tallennuspainike on nyt Tallenna, ei enää Save. Ruotsinkielisessä Spara.

-----Lisätietona-----
Helle-testissä on käytössä myös rimpsu: Kopioi puhelinnumero mobile-kentästä smsalert-kenttään

#21 Updated by Päivi Knuutinen 11 päivää ago

Testasin Vaaran testillä (eng) ja other phone-kentästä tieto ei siirry SMS-number-kentttään. Other phone-numeron pituus voi olla mitä tahansa. Ei taida olla tarkoitus?

#22 Updated by Katri Sillgren 11 päivää ago

Uusin rimpsu vaikuttaa Helle-testissä myös ainakin näin: asiakkaan Tiedot näytöllä näkyvät nyt tiedot asiakkaan noudettavista varauksista, eivätkä ole enää painikkeen takana.

#23 Updated by Emmi Takkinen 11 päivää ago

Kävin korjailemassa rimpsua sen verran, että lisäsin sinne seuraavan:

var save_text; 
if(language == "Suomi"){
 save_text = "Tallenna";
}else if(language == "Svenska"){
 save_text = "Spara";
}else {
 save_text = "Save";
}

Näyttäisi sille, että tuo välilyönneistä nalkuttaja ei enää toimi syystä tai toisesta. Ja aiemmin välilyönnit kentän lopusta automaattisesti poistanut toiminnallisuuskaan ei toimi. Säädetään sitä vielä.

--EDIT--

Ilmeisesti välilyönnit katoavat numeron lopusta tallentaessa edelleen eli sen osalta asia ok.

#24 Updated by Emmi Takkinen 11 päivää ago

Katri Sillgren kirjoitti:

Uusin rimpsu vaikuttaa Helle-testissä myös ainakin näin: asiakkaan Tiedot näytöllä näkyvät nyt tiedot asiakkaan noudettavista varauksista, eivätkä ole enää painikkeen takana.

Tämän ei pitäisi vaikuttaa Tiedot näyttöön, johtuukohan tämä varmasti tästä eikä jostain toisesta rimpsusta?

#25 Updated by Emmi Takkinen 11 päivää ago

Testailin Hellessä ja siellä vaikutti toimivan kuten pitää ilman mitään lisäyksiä. Välilyönnit numeron keskellä aiheuttivat virheilmoituksen ja numero kopioitui oikein other-kentästä SMS-kenttään.

Vaarassa kopionti näyttäisi toimivan nyt, kun otin validointirimpsusta virheellisen Tallenna-napin kielenvaihdon pois. Ja näköjään välilyönneistä ja ylimääräisistä merkeistä nalkutus rupesi toimimaan, kun korvasin rimpsun Hellen vastaavalla, vaikka niissä ei ollut mitään eroa :'D

#26 Updated by Emmi Takkinen 11 päivää ago

Niin ja tiketissä #5348 tekemääni lisäystä noiden välilyöntien tarkistamiseen ei sitten tarvitakaan, vaikuttaisi toimivan ilman niitäkin :D

#27 Updated by Katri Sillgren 7 päivää ago

5477

Emmi Takkinen kirjoitti:

Testailin Hellessä ja siellä vaikutti toimivan kuten pitää ilman mitään lisäyksiä. Välilyönnit numeron keskellä aiheuttivat virheilmoituksen ja numero kopioitui oikein other-kentästä SMS-kenttään.

Helle-testissä en saa toimimaan niin, että ylimääräiset välilyönnit poistuisivat tallentaessa. Lisäksi välilyönnillisten puhelinnumeroarvojen lisäämisen jälkeen Save-painikkeella en saanut tallennettua asiakastietoa. Painike jäi ikään kuin pohjaan, mutta mitään ei tapahtunut. Tulisiko virheelliset puhelinnumeroarvot korjata itse? Vai poistuvatko tallennuksen yhteydessä?
Toimi: Puhelinnumero-arvon lopun ylimääräinen välilyönti hävisi, kun klikkasi jotain muuta kenttää.
Toimi: Asiakkaan Lainaus ja Tiedot näytöillä noudettavat varaukset ovat taas linkin takana.

Lisätietona: asiakastiedon tallennus onnistuu, kun ensin poistaa itse ylimääräiset puhelinnumeroarvon välilyönnit.

#28 Updated by Emmi Takkinen 5 päivää ago

Katri Sillgren kirjoitti:

Emmi Takkinen kirjoitti:

Testailin Hellessä ja siellä vaikutti toimivan kuten pitää ilman mitään lisäyksiä. Välilyönnit numeron keskellä aiheuttivat virheilmoituksen ja numero kopioitui oikein other-kentästä SMS-kenttään.

Helle-testissä en saa toimimaan niin, että ylimääräiset välilyönnit poistuisivat tallentaessa. Lisäksi välilyönnillisten puhelinnumeroarvojen lisäämisen jälkeen Save-painikkeella en saanut tallennettua asiakastietoa. Painike jäi ikään kuin pohjaan, mutta mitään ei tapahtunut. Tulisiko virheelliset puhelinnumeroarvot korjata itse? Vai poistuvatko tallennuksen yhteydessä?
Toimi: Puhelinnumero-arvon lopun ylimääräinen välilyönti hävisi, kun klikkasi jotain muuta kenttää.
Toimi: Asiakkaan Lainaus ja Tiedot näytöillä noudettavat varaukset ovat taas linkin takana.

Lisätietona: asiakastiedon tallennus onnistuu, kun ensin poistaa itse ylimääräiset puhelinnumeroarvon välilyönnit.

Ne pitää poistaa itse sieltä välistä, kai se olisi mahdollista kehitellä tuohon automaatio, mutta vielä en ole kerennyt.

#29 Updated by Emmi Takkinen 5 päivää ago

Päivittelin rimpsua hieman, edellinen esimerkki hyväksyi väliviivat ja uusi Tallenna-painikkeen generointi uupui.

Vie Atom PDF