Varaustunnisteen säilyminen/muuttaminen

[pknuuti]

Kehitysehdotukseni

Redmineen tehty tiketti 24.4.2019, edelleen ongelma:
Tällä hetkellä varaustunnisteen muuttaminen vahingossa on liian helppoa: tunniste muuttuu, kun klikkaa kenttää. Parempi olisi, että muutos tapahtuisi vain tarkoituksella yhdellä tai useammalla tavalla

• jättämättä kenttä tyhjäksi, jolloin tallennuksessa kenttään tulisi numerotunniste
• samanlainen toiminto kuin signumissa eli kolme pistettä kentän perässä ja sitä klikkaamalla saisi uuden numerotunnisteen
• kirjoittamalla haluamansa tunnisteen kenttään

Tiketti saanut 5 peukutusta. Suljen Redminen tiketin.

Jotain muuta?

No response

[lmstrand]

Muokkasin varaustunnuksen generointia Vaara-testille:

/// ALKU ///

// Varaustunnuksen automaattinen generointi/anonymisointi - Adapted from Koha-suomi patch for KD-1452 (commit 1c71b272885d9c510630 from https://github.com/KohaSuomi/Koha/ branch master)
// Tämä generoi patron_attr_2-kenttään. Jos attribuutin arvo muuttuu, pitää se muuttaa jokaiseen kohtaan, jossa se mainitaan. //

$(document).ready(function(){
    if (window.location.pathname == '/cgi-bin/koha/members/memberentry.pl' && window.location.search.includes("?op=add&") || window.location.search.includes("?op=duplicate&")) {
      var unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
      var epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
      $('textarea#patron_attr_2').val(epochdashed);

      $("#patron_attr_2").focus(function() {
        unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
        epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
        $('textarea#patron_attr_2').val(epochdashed);
      });
    }

      if (window.location.pathname == '/cgi-bin/koha/members/memberentry.pl' && window.location.search.includes("?op=modify")) {
      $("#patron_attr_2").focus(function() {
        if (!document.getElementById("patron_attr_2").value){
        unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
        epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
        $('textarea#patron_attr_2').val(epochdashed);
        }
      });
    }
});

/// LOPPU ///

Nyt asiakkaan muokkausnäkymässä varaustunnus vaihtuu uudeksi vain, jos kenttä on tyhjä/tyhjennetty.

[lmstrand]

Pitäisikö sama toiminta olla uutta asiakasta luodessa?

[pknuuti]

Uutta asiakasta luodessa toiminto ei ole niin kriittinen, koska oletusarvoisesti sinne tulee juokseva numero.
Testasin vanhan asiakkaan muokkausnäkymää ja mielestäni se toimii nyt hyvin eli kentän sisältö ei vaihtu, kun sinne klikkaa. Kentän sisällön voi muokata ilman, että koko tunnus muuttuu. Kun kentän sisällön tyhjentää, siihen tarjotaan tallennusvaiheessa uusi juokseva numero, koska kenttä on pakollinen.

[lmstrand]

Tämä muutos olisi varmaan haluttu kaikkiin kimppoihin?

[AnneliO]

Tämä muutos olisi varmaan haluttu kaikkiin kimppoihin?

Kyllä kiitos mielellään kaikissa kimpoissa samanlaiset rimpsut. :) Tämän voisi vielä laittaa testattavaksi niin, että muissakin kimpoissa todetaan toimivaksi ja sen jälkeen sopia, milloin rimpsu vaihdetaan tuotantoihin.

[lmstrand]

Vaskilla näyttääkin olevan oma versionsa testillä: "Koha-Suomen versioon verrattuna erona kentän jälkeen lisätty kolme pistettä, josta muodostus tapahtuu." Muodostus tapahtuu myös, kun uutta asiakasta lisätään, eli kenttä ei jää tyhjäksi.

Olisiko tuo sitten parempi tapa? Tässä vaskin testistä napattu rimpsu:

// Varaustunnuksen automaattinen generointi. Koha-Suomen versioon verrattuna erona kentän jälkeen lisätty kolme pistettä, josta muodostus tapahtuu.
$(document).ready(function(){
    if (window.location.pathname == '/cgi-bin/koha/members/memberentry.pl' && window.location.search.includes("?op=add&") || window.location.search.includes("?op=duplicate&")) {
      var unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
      var epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
      $('textarea#patron_attr_4').val(epochdashed);
      
	  $( '<a class="buttonDot" href="#" id="generate_holdid" title="Luo varaustunnus" style="vertical-align: top;"> ...</a>' ).insertAfter( "#patron_attr_4");
	  $("#generate_holdid").click(function() {
          unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
          epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
          $('textarea#patron_attr_4').val(epochdashed);
		  $("#patron_attr_4").trigger('blur');
      });
    }

      if (window.location.pathname == '/cgi-bin/koha/members/memberentry.pl' && window.location.search.includes("?op=modify")) {
	  $( '<a class="buttonDot" href="#" id="generate_holdid" title="Luo varaustunnus" style="vertical-align: top;"> ...</a>' ).insertAfter( "#patron_attr_4");
	  $("#generate_holdid").click(function() {
          unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
          epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
          $('textarea#patron_attr_4').val(epochdashed);
		  $("#patron_attr_4").trigger('blur');
      });
    }
});
//LOPPU

Näitä voi pääkäyttäjät vapaasti kyllä testata testeillä, kun siellä ei ole checksum-tarkistusta.

[lmstrand]

Virkailijaliittymän toiminnallisen yhtenäisyyden vuoksi eri puolella järjestelmää äänestäisin pisteitä.

[AnneliO]

Lisäsin asian ensi tiistain pääkäyttäjäpalsun listalle.

[AnneliO]

Pääkäyttäjien palaveri 21.11.2023: Pääkäyttäjät testaavat vaihtoehtoja ja tehdään lopullinen päätös parin viikon päästä.

[piiaseme]

OUTI-kimpassa testattiin molempia vaihtoehtoja. Kannatamme Koha-Suomen versiota, koska pisteillä ei ole mielestämme käytännön merkitystä. Lisäksi asiakkaan muokkaussivulla ei ole muissakaan kentissä pisteitä.

[MikkoLiimatainen]

Vaskissa kannatetaan kolmea pistettä. Se on Kohan muiden generoitavien kenttien kanssa yhdenmukainen tapa toimia. Toiminnallisuuksien erottaminen tietokentistä taitaisi muutenkin olla järjestelmän käytettävyyden kannalta järkevämpää. Tuo tyhjään kenttään klikatessa generointi ei ole kovin intuitiivinen tapa toimia ja muistaakseni siitä tuli palautettakin, kun vastaavanlaista toteutusta kokeiltiin meillä ennen nykyiseen päätymistä. Siinä toteutuksessa generointi tapahtui muistaakseni tyhjästä kentästä poistuttaessa, joten toteutus oli hieman erilainen nyt ehdotettuun. Nuo kolme pistettä kuitenkin ohjaavat käyttäjän generointiin, niin ei tarvitse miettiä miten se generointi nyt toimiikaan.

[LeenaKinnunen]

Testasin Lapin testillä: Vaaran rimpsulla vanhan asiakkaan varaustunnus ei muutu kenttää klikkaamalla, mutta ei luo uudelle asiakkaalle varaustunnusta. Vaskin koodi ei myöskään toiminut meillä uudella asiakkaalla.

Lapin kanta sama kuin Vaskilla yhtenäisen käytännön vuoksi, eli käytettäisiin kolmea pistettä, koska kyseessä olisi Kohan luoma koodi.

[AnneliO]

Pääkäyttäjien palaveri 12.12.2023: Otetaan käyttöön kolmen pisteen versio. Käyttöönotto 19.12.2023 päivityksessä.

[AnneliO]

Yhtenäistetty versio rimpsusta lisätty IntranetUserJS-kirjastoon.

[lmstrand]

Lisätty kolmen pisteen versio tuotantoihin. Lumpeessa ongelmana sivun vieritys alkuun uuden varaustunnuksen generoinnin jälkeen (varaustunnus sijaitsee poikkeavasti alempana muiden asiakasmääreiden alla). Pitää tutkia mistä johtuu.

[lmstrand]

Sivun hyppääminen alkuun uuden varaustunnuksen generoinnin yhteydessä korjaantui muuttamalla click-funktiot muotoon:
$("#generate_holdid").click(function(event) {
event.preventDefault();

 //ALKU
// Varaustunnuksen automaattinen generointi. Kentän jälkeen lisätty kolme pistettä, josta muodostus tapahtuu.
// Tarkista oman tietokannan oikea payhdessätron_attr-arvo esim. selaimen Tarkista/Inspect element -toiminnolla.
$(document).ready(function(){
    if (window.location.pathname == '/cgi-bin/koha/members/memberentry.pl' && window.location.search.includes("?op=add&") || window.location.search.includes("?op=duplicate&")) {
      var unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
      var epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
      $('textarea#patron_attr_2').val(epochdashed);
      
	  $( '<a class="buttonDot" href="#" id="generate_holdid" title="Luo varaustunnus" style="vertical-align: top;"> ...</a>' ).insertAfter( "#patron_attr_2");
	  $("#generate_holdid").click(function(event) {
        event.preventDefault();
          unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
          epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
          $('textarea#patron_attr_2').val(epochdashed);
		  $("#patron_attr_2").trigger('blur');
      });
    }

      if (window.location.pathname == '/cgi-bin/koha/members/memberentry.pl' && window.location.search.includes("?op=modify")) {
	  $( '<a class="buttonDot" href="#" id="generate_holdid" title="Luo varaustunnus" style="vertical-align: top;"> ...</a>' ).insertAfter( "#patron_attr_2");
	  $("#generate_holdid").click(function(event) {
        event.preventDefault();
          unixepoch = Math.round( (new Date()).getTime() / 10 ).toString();
          epochdashed = unixepoch.replace( /(....)/g, '$1-').replace(/-$/,'' );
          $('textarea#patron_attr_2').val(epochdashed);
		  $("#patron_attr_2").trigger('blur');
      });
    }
});
//LOPPU

Muutettu Lumme-testille. Lumme-tuotannossa oli molemmat vanha rimpsu ja uusi kolmen pisteen versio, vanha pitää poistaa.

[AnneliO]

Voiko tämän tiketin jo sulkea?

[lmstrand]

Voi.