Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nidehaun julkaisuvuosi-hakuun ennen/jälkeen -toiminnot #447

Open
Tuomas-Kyyti opened this issue Feb 17, 2023 · 19 comments
Open

Nidehaun julkaisuvuosi-hakuun ennen/jälkeen -toiminnot #447

Tuomas-Kyyti opened this issue Feb 17, 2023 · 19 comments
Assignees
Labels
community enhancement Yhteisössä kehitettävä toiminnallisuus local enhancement Koha-Suomessa kehitettävä toiminnallisuus

Comments

@Tuomas-Kyyti
Copy link

Kehitysehdotukseni

Siirretty Redminen tiketistä #2860
31.12.2018

Updated by Pirkko-Liisa Lauhikari noin 4 vuotta ago
Nidehakuun tullut lisätoivomus, että vosiko Julkaisuvuosi -rajaukseen saada ennen/jälkeen -vaihtoehdon? Voisi hakea vanhentuneita niteitä kätevämmin esim. matkakirjoista.
Kuva vielä lisätoiveeseen. OUTIssa Julkaisuvuosi -rajaus on lisätty Räätälöytyihin hakukenttiin.

Jotain muuta?

julkaisuvuosirajaus

@Tuomas-Kyyti Tuomas-Kyyti added the local enhancement Koha-Suomessa kehitettävä toiminnallisuus label Feb 17, 2023
@AnneliO
Copy link
Contributor

AnneliO commented Mar 7, 2024

Indeksointiryhmän päätös 7.3.2024: Yhteisöön tiketti. Tutkitaan, onko toteutettavissa ja mahdollisesti tarjotaan muutosta itse.

@AnneliO AnneliO added the community enhancement Yhteisössä kehitettävä toiminnallisuus label Mar 7, 2024
@atarnus
Copy link

atarnus commented Mar 26, 2024

Haku lähettää eteenpäin olion tähän tyyliin:
image

Oletuksena tuossa on mahdollista valita operaattoriksi (op) "on" tai "ei ole", jotka vastaa SQL termejä "like" ja "not like". Jos lisää valikkoon vaihtoehdot > ja <, niin haku toimii saman tien. Nämä vaihtoehdot voi lisätä vaikka JS-rimpsullakin (sillä testasin).

Haluttaisiinko tuohon lisää noita vaihtoehtoja, vai onko suurempi ja pienempi kuin vaihtoetojen lisääminen kaikille noille saman pudotusvalikon hakuehdoille huono ajatus? On mahdollista myös tehdä tämän tyyppisiä kenttiä:
image

Tekee sitten koodimuutoksen tai JS-rimpsun, niin ei ole mitenkään iso homma.

Tällä saa nuo vaihtoehdot pudotusvalikkoon:

/// ALKU ///
// Lisää '<' ja '>' operaattorit monivalintavalikkoon nidehaussa
$(document).ready(function() {
  if (window.location.pathname == '/cgi-bin/koha/catalogue/itemsearch.pl') { 
    $('select[name="op"]').append('<option value=">">></option><option value="<"><</option>');
  }
});
/// LOPPU ///

@atarnus atarnus self-assigned this Mar 26, 2024
@atarnus
Copy link

atarnus commented Mar 27, 2024

Muutos voidaan tehdä myös templateen (itemsearch.tt)

    [% IF params.exists('op') %]
        <select name="op" class="form-field-not">
           <option value="like">is</option>
            [% IF params.op == 'not like' %]
               <option value="not like" selected="selected">is not</option>
            [% ELSE %]
               <option value="not like">is not</option>
            [% END %]
            [% IF params.op == '>' %]
               <option value=">" selected="selected">></option>
            [% ELSE %]
               <option value=">">></option>
            [% END %]
            [% IF params.op == '<' %]
               <option value="<" selected="selected"><</option>
            [% ELSE %]
               <option value="<"><</option>
            [% END %]
        </select>
    [% ELSE %]
        <select name="op" class="form-field-not">
           <option value="like">is</option>
           <option value="not like">is not</option>
           <option value=">">></option>
           <option value="<"><</option>
        </select>
    [% END %]

Joka tapauksessa tämä vaihtoehtojen lisääminen olisi minusta yksinkertaisin ratkaisu, ja se tuo lisähyötyä siinä, että noita suurempi ja pienempi kuin määreitä voi sitten käyttää muihinkin kenttiin samassa valikossa. Tietysti kaikkien arvojen kohdalla nuo eivät välttämättä toimi ollenkaan tai oikein, mutta se on tiedonhakijan pääteltävissä.
image

Jos pääkäyttäjillä on aikaa, niin tuon ylemmässä viestissä olevan JS-rimpsun voi lisätä intranetuser.js:ään ja kokeilla miten nuo > ja < operaattorit pelaa eri hakuvaihtoehtojen kanssa. Päivämäärän kohdalla tulee käyttää samaa ISO-muotoa kuten ohjeistetaan lainauspvm:nkin kohdalla, eli YYYY-MM-DD. Ohjeistus voidaan liittää mukaan sitten jos tämä otetaan käyttöön.

@atarnus
Copy link

atarnus commented Mar 27, 2024

Tuolla templatessa määritellään se, että oikea operaattori pysyy valittuna. Jos päivittää sivun, niin tuo js-rimpsu ei tosiaan "muista" valittua arvoa vaan nollaa sen takaisin ylimpään. Tämä kannattanee korjata vielä rimpsuun, jos mennään rimpsulla.

@AnneliO
Copy link
Contributor

AnneliO commented Mar 27, 2024

Testailin noita <>-merkkejä ja minusta ne toimivat ihan hyvin.

Hankinta- ja saapumispäivä 952$d

  • osasi hakea tiettyä päivää aiemmat tai uudemmat päivämäärät
  • onnistui myös haku aikavälillä, kun määritteli saman kentän kahdesti JA-operaattorilla yhdistäen

Viimeksi nähty 952$r

  • osasi hakea pelkällä vvvv-kk-dd haulla, vaikka kentässä on myös kellonaika. Tässä pitää ottaa huomioon, että jos haluaa hakuun mukaan 19.1.2024, pitää päivämääräksi laittaa > 18.1.2024

Aineistotyyppi

  • toimii kyllä. Esim. < DVD hakee aineistotyyppejä, jotka ovat aakkosissa ennen DVD:tä. Mukaan tuli siis esim. ALEHTI.

Tekijä 100$a

  • toimii samoin kyllä aineistotyypin kanssa, eli jos hakee esim. < Aapeli* tulee hakuun mukaan esim. 22-pistepirkko.

@atarnus
Copy link

atarnus commented Apr 23, 2024

Olisiko tässä pääkäyttäjille loogisempaa käyttää kuitenkin operaattoreita >= ja <= ? Eli sisällytetään haettava vuosi hakuun? Tuohon ei kannattane tunkea ihan hirveästi noita lisäoperaattoreita, niin mielestäni valitaan joko setti '< ja >' tai '<= ja =>'.

Tehdään tästä se patch tarjolle yhteisöön, mutta meille tämän saisi varmaan nopeiten käyttöön JS-rimpsuna (mahd. JS-plugarina sitten). Työstän mukaan tuon valinnan säilyvyyden vielä, ja haluaisin mielipiteitä sitten noista operaattoreista.

@atarnus
Copy link

atarnus commented Apr 25, 2024

@Tuomas-Kyyti teetkö tästä tiketin yhteisöön, niin tarjotaan sinne patchia. (Tässä nyt olisi tulossa tuo ominaisuus tietysti moniin muihinkin kenttiin kuin meidän omaan räätälöityyn julkaisuvuoteen.)

Ei myöskään näytä siltä, että tuolla sivulla olisi valmiina oikein mitään, millä saisi säilytettyä valitun operaattorin sivunlatausten yli. En käytä siihen tällä hetkellä ainakaan enempää aikaa. Se toimii koodimuutoksessa, ja jos edes otetaan JS-plugarina käyttöön, niin kyseessä on minimaalinen asia, jota suurin osa käyttäjistä ei todennäköisesti edes huomaa.

@atarnus
Copy link

atarnus commented May 22, 2024

Näyttäisi sandboxissa että nidehaku on vähän muuttunut, mutta tuo pudotusvalikko on koodissa yhä sama, niin yllä tehty template-muutos saattaa hyvinkin toimia vielä.

Me voidaan nyt ottaa käyttöön tämä intranet.js-rimpsu, kunhan nyt vielä joku muukin kuin Anneli testaa sitä:

/// ALKU ///
// Lisää '<' ja '>' operaattorit monivalintavalikkoon nidehaussa
$(document).ready(function() {
  if (window.location.pathname == '/cgi-bin/koha/catalogue/itemsearch.pl') { 
    $('select[name="op"]').append('<option value=">">></option><option value="<"><</option>');
  }
});
/// LOPPU ///

@Tuomas-Kyyti
Copy link
Author

Tiketti tehty Bugzillaan: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36920

@atarnus
Copy link

atarnus commented May 24, 2024

Näköjään Outilla onkin tuotannossa jo tuo rimpsu operaattoreilla >= ja <= 😄
Humpsahtanut sinne Nextin asetuksista.

@AnneliO
Copy link
Contributor

AnneliO commented May 24, 2024

Näköjään Outilla onkin tuotannossa jo tuo rimpsu operaattoreilla >= ja <= 😄 Humpsahtanut sinne Nextin asetuksista.

Tästä tulikin mieleen, että pääkäyttäjäpalaverissa 21.5.2024 päätettiin mennä operaattoreilla > ja < ilman yhtäsuurikuin-merkkejä siis, jotta se olisi johdonmukainen muiden valikoiden kanssa (lähinnä Lainauksien määrä -valikon kanssa).

@atarnus
Copy link

atarnus commented May 24, 2024

Näköjään Outilla onkin tuotannossa jo tuo rimpsu operaattoreilla >= ja <= 😄 Humpsahtanut sinne Nextin asetuksista.

Tästä tulikin mieleen, että pääkäyttäjäpalaverissa 21.5.2024 päätettiin mennä operaattoreilla > ja < ilman yhtäsuurikuin-merkkejä siis, jotta se olisi johdonmukainen muiden valikoiden kanssa (lähinnä Lainauksien määrä -valikon kanssa).

Juu, tuo on ollut ihan joku lapsus, en hoksannut käydä poistelemassa testaamiani juttuja nextistä ennen kuin se kopsattiin tuotantoon.

Olinkin nyt tehnyt tästä näemmä myös ihan oikean branchinkin valmiiksi yhteisön patchia varten, niin ajateltiin että otetaan kuitenkin se testattavaksi ja käyttöön, eikä pelkkä rimpsu.

@atarnus
Copy link

atarnus commented May 27, 2024

Pitäisi olla nyt kaikilla testeillä, rimpsua ei tarvita. (Jos joku on rimpsun laittanut niin kannattaa poistaa.)

Tuo Outin tuotantoon vahingossa mennyt rimpsu tulee poistaa sitten kun tämä branchi laitetaan tuotantoon.

@sksandel
Copy link

sksandel commented May 31, 2024

Vaski-testillä oli myös tuo JS-rimpsu, joka tuplasi >< operaattorit alasvetovalikkoon. Rimpsu poistettu ja nyt vaikuttaisi toimivan kuten pitääkin.

Edit: paitsi jokerimerkkien, - ja %, käyttö ei mielestäni tuo oikeita tuloksia. Saan niillä samat tulokset.
image
saa kokoelmasta riippuen tuloksia esim vuosilta x-2022
image
Ei anna mitään tuloksia.

Edit 2: huonot esimerkit, mutta siis jos käyttää jokerimerkkejä niin ei ole eroa käyttääkö - vai % merkkiä. Esim hakuna 1%9 tai 1-9 tulee sama tulos. Jokerimerkkejä käytettäessä > haun kanssa tulee aina koko tietokanta ja < haulla ei tule mitään tuloksia.

@Kati-Helle
Copy link

Kati-Helle commented Jun 7, 2024

Helle-testissä operaattoreilla > ja < hauilla oikeat hakutulokset.

Jokerimerkki hakuehdossa ei vaikuta Helle-testissäkään toimivan oikein.

Esim. hakuehto
kuva

Hakutulosta Julkaisuvuosi-kentän mukaan järjestettynä
kuva

Testissä on juuri nyt raportilla items-taulusta haettuna 3106 nidettä. Haku rajannee tuon perusteella jotain niteitä pois hakutuloksesta? *Tarkistin: kaikilla niteiden tietueilla ei ole julkaisuvuotta (biblio.copyrightdate).

Esim hakuehto
kuva

Ei hakutulosta
kuva

@AnneliO
Copy link
Contributor

AnneliO commented Jun 7, 2024

Testailin tuota vuosiluvun katkaisua <>-hauissa ja minusta vaikuttaa siltä, että jos haetaan esim. > '202%', haetaan käytännössä numeroita, jotka ovat suurempia kuin 202 eli käytännössä siis koko tietokanta. Jos taas < '202%', niin haetaan pienempiä vuosilukuja kuin 202 eli käytännössä ei löydy mitään.

Julkaisuvuosi-haku taitaa hakea publicationyear-kentästä tietokannasta ja se on tietotyypiltään tekstiä. Opiskelen vielä, onko tuolla merkitystä asiaan.

@AnneliO
Copy link
Contributor

AnneliO commented Jun 10, 2024

Testailin lisää ja juttelimme tästä myös Koha-Suomen palaverissa ja sanoisin, että vuosilukua ei pysty tuolla tavalla katkaisemaan. Enkä kyllä äkkiseltään hoksaa, miksi se pitäisi pystyä. Jos halutaan hakea 2020-luvun vuosia, niin haku tehdään > 2019. Jos haluaa käyttää katkaisumerkkiä, niin silloin on 202_ ja on 202% toimii mielestäni oikein tässä kontekstissa.

Minusta tämä toiminto olisi valmis tuotantoon laitettavaksi, vai huomaako joku vielä jotain omituista? :)

@Kati-Helle
Copy link

Hakuvinkit eivät päde julkaisuvuoden hakuehtoihin > ja <

Image

@AnneliO
Copy link
Contributor

AnneliO commented Jun 11, 2024

Hakuvinkit eivät päde julkaisuvuoden hakuehtoihin > ja <

Ei päde, mutta onko se iso ongelma? Tai pätee, mutta kun vuosiluku katkaistaan, niin haetaan ennen katkaisumerkkiä olevaa merkkijonoa isompia ja pienempiä lukuja.

Jos se on iso ongelma, niin äkkiseltään tuli mieleen, että vihjeen perään voisi laittaa lisäohjeena IntranetUserCSS:llä jotain tällaista:
kuva

/* Lisää nidehakuun huomautus <>-operaattoreiden katkaisusta. */
body#catalog_itemsearch.catalog p.hint:nth-child(2):after {
  content: " \00a0 Vuosilukuja ei voi katkaista, jos käyttää <>-operaattoreita";
  color:red;
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community enhancement Yhteisössä kehitettävä toiminnallisuus local enhancement Koha-Suomessa kehitettävä toiminnallisuus
Projects
Status: Ratkaisu ehdotettu
Development

No branches or pull requests

5 participants