Projekti

Yleinen

Profile

Bugi #5363

Uusi järjestelmäasetus: UpdateNotForLoanStatusOnCheckout

Lisännyt Emmi Takkinen 26 päivää sitten. Päivitetty 12 päivää sitten.

Tila:
Ratkaisu ehdotettu
Prioriteetti:
Korkea
Luokka:
-
Kohdeversio:
-
Alku:
28. Huhtikuuta 2022
Määräaika:
% Tehty:

0%

Arvioitu aika:
Yhteyshenkilö:
Tiketti Bugzillassa:

Kuvaus

Meillä on tähän asti käytetty järjestelmäasetusta UpdateNotForLoanStatusOnCheckin myös lainauksessa. Toiminnallisuus on kuitenkin järkevämpää eriyttää omaksi asetuksekseen varsinkin siksi, että se saataisiin vietyä myös yhteisöön. Lisätään siis uusi järjestelmäasetus UpdateNotForLoanStatusOnCheckout.

Historia

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

  • Prioriteetti changed from Normaali to Korkea

Odottaa sitä, että versionhallinta saadaan kuosiin.

#2 Updated by Emmi Takkinen 18 päivää ago

  • Tila changed from Työn alla to Ratkaisu ehdotettu

Tuotu testeille.

#3 Updated by Tuomas Kunttu 17 päivää ago

Yritin tätä testata, mutta riippumatta siitä onko tähän uuteen asetukseen laitettu arvoja tai ei, niin kun yrittää lainata nidettä, jossa on mikä tahansa notforloan-arvo (positiivinen tai negatiivinen), niin lainauksen varmistuksen jälkeen päädytään virheeseen 500 (An error has occurred! Error 500). Kuitenkin nide on mennyt lainaan.

#4 Updated by Anneli Österman 15 päivää ago

Joko asetuksen päälle menemiseen kuluu aikaa tai sitten se ei toimi OUTIssa. Määritin asetukseen -1: 0 ja yritin lainata niteen, jonka tila on -1. Nide ei mene lainaan, eikä niteen tila vaihdu.

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

Anneli Österman kirjoitti:

Joko asetuksen päälle menemiseen kuluu aikaa tai sitten se ei toimi OUTIssa. Määritin asetukseen -1: 0 ja yritin lainata niteen, jonka tila on -1. Nide ei mene lainaan, eikä niteen tila vaihdu.

OUTIssa oli pois päältä asetus "AllowNotForLoanOverride". Laitoin sen "Allow", niin lainaus poisti niteen tilan -1 ja se vaihtui 0.

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

Tuomas Kunttu kirjoitti:

Yritin tätä testata, mutta riippumatta siitä onko tähän uuteen asetukseen laitettu arvoja tai ei, niin kun yrittää lainata nidettä, jossa on mikä tahansa notforloan-arvo (positiivinen tai negatiivinen), niin lainauksen varmistuksen jälkeen päädytään virheeseen 500 (An error has occurred! Error 500). Kuitenkin nide on mennyt lainaan.

Jaa, onkohan kys. niteen kohdalla ollut sitten jokin muu ongelma. Kokeilin kyytissäkin ja toimi ihan OK.

#7 Updated by Päivi Knuutinen 15 päivää ago

Toimii Vaaran testillä -1 tilassa olevan lainaaminen tai palauttaminen. Palautettaessa tilaksi muuttuu Saatavana (kuten pitääkin), lainattu menee lainaan varmistuskysymyksen jälkeen.

#8 Updated by Tuomas Kunttu 15 päivää ago

Emmi Takkinen kirjoitti:

Tuomas Kunttu kirjoitti:

Yritin tätä testata, mutta riippumatta siitä onko tähän uuteen asetukseen laitettu arvoja tai ei, niin kun yrittää lainata nidettä, jossa on mikä tahansa notforloan-arvo (positiivinen tai negatiivinen), niin lainauksen varmistuksen jälkeen päädytään virheeseen 500 (An error has occurred! Error 500). Kuitenkin nide on mennyt lainaan.

Jaa, onkohan kys. niteen kohdalla ollut sitten jokin muu ongelma. Kokeilin kyytissäkin ja toimi ihan OK.

Jaaha, kyse onkin siitä, että kaikille lapsi-asiakkaille lainattaessa tulee virhe 500, mutta menee silti lainaan. Ei siis ole tämän tiketin asia. En kyllä tiedä mistä johtuu.

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

Tuomas Kunttu kirjoitti:

Emmi Takkinen kirjoitti:

Tuomas Kunttu kirjoitti:

Yritin tätä testata, mutta riippumatta siitä onko tähän uuteen asetukseen laitettu arvoja tai ei, niin kun yrittää lainata nidettä, jossa on mikä tahansa notforloan-arvo (positiivinen tai negatiivinen), niin lainauksen varmistuksen jälkeen päädytään virheeseen 500 (An error has occurred! Error 500). Kuitenkin nide on mennyt lainaan.

Jaa, onkohan kys. niteen kohdalla ollut sitten jokin muu ongelma. Kokeilin kyytissäkin ja toimi ihan OK.

Jaaha, kyse onkin siitä, että kaikille lapsi-asiakkaille lainattaessa tulee virhe 500, mutta menee silti lainaan. Ei siis ole tämän tiketin asia. En kyllä tiedä mistä johtuu.

:D muistelisin, että tällaistä on tapahtunut aiemminkin, mutten kyllä yhtään muista miksi. Saattoi liittyä johonkin javascript-rimpsuun.

#10 Updated by Anni Rajala 15 päivää ago

Testattu Vaskissa asetuksella -1: 0, not for loan -tila poistui lainauksessa kuten pitikin.

#11 Updated by Reetta Pihlaja 15 päivää ago

Toimii Siilissä testatuilla arvoilla, tila poistui lainatessa. Jos arvoa ei oltu määritelty, tila jäi.

Onko tuossa tiketin kuvaustekstissä just väärinpäin: Meillä on tähän asti käytetty järjestelmäasetusta UpdateNotForLoanStatusOnCheckin myös palautuksessa, pitäisikö olla myös lainauksessa?

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

Reetta Pihlaja kirjoitti:

Toimii Siilissä testatuilla arvoilla, tila poistui lainatessa. Jos arvoa ei oltu määritelty, tila jäi.

Onko tuossa tiketin kuvaustekstissä just väärinpäin: Meillä on tähän asti käytetty järjestelmäasetusta UpdateNotForLoanStatusOnCheckin myös palautuksessa, pitäisikö olla myös lainauksessa?

Pitäis juu :D Nää menee mulla jo ilosesti sekasin kumpi on kumpaakin.

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

  • Kuvaus updated (diff)

#14 Updated by Katri Sillgren 14 päivää ago

Helle-testissä toimii oikein miinumerkkisen NOT_LOAN arvon kanssa, jonka tulee poistua lainatessa.
Lainasin testissä niteen, jolla on NOT_LOAN-arvona -7 (Puutteellinen sisältö).
Lainauksen myötä niteeltä poistui NOT_LOAN-arvo Puutteellinen sisältö.

Lainatessa popuppi:
Item is normally not for loan (Puutteellinen sisältö). Check out anyway?

vastausvaihtoehtoinaan:
Yes, check out (Y)
No, don't check out (N)

Vastaamalla Yes, check out (Y) niteen saa lainattua. Lainatulta niteeltä on poistunut NOT_LOAN-nidetila Puutteellinen sisältö.

Tuleeko kaikki, myös plusmerkkiset NOT_LOAN-arvot määritellä asetukseen tyyliin 1: ONLYMESSAGE ?

Kun esim. arvoa 1: ONLYMESSAGE ei ole määritelty asetukseen TAI ko. arvo on määritelty asetukseen, ilmoittaa Koha ainakin Lapsi-asiakastyypille lainatessa Yes, check out (Y) klikkauksen jälkeen:
An error has occurred!
Error 500
This message may have been caused by any of the following reasons:

You made use of an external link to an item that is no longer available
You followed an outdated link e.g. from a search engine or a bookmark
You tried to access a page that needs authentication
An internal link in the client is broken and the page does not exist

What's next?

Use top menu bar to navigate to another part of Koha.
To report a broken link or any other issue, please contact the Koha administrator. Send email

Virheilmoituksesta huolimatta laina on onnistunut ja näkyy lapsen lainoissa. Niteellä pysyy asetukseen pysyväksi ONLYMESSAGE-määritely NOT_LOAN-arvo. Plusmerkkinen NOT_LOAN -nidetila pysyy myös, jos sitä ei ole määritelty asetukseen.

-----Lisätietona-----
Asiakastyypeille Lapsi ja Holhottava aikuinen lainaaminen toimii nyt ongelmitta, tiketti:
https://tiketti.koha-suomi.fi/issues/5383

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

Tässä toteutuksessa ei ole vielä tukea tuolle ONLYMESSAGE toiminnallisuudelle. Jos sille on tarvetta, niin eiköhän sen voi jossain vaiheessa toteuttaa.

#16 Updated by Katri Sillgren 12 päivää ago

5445

Emmi Takkinen kirjoitti:

Tässä toteutuksessa ei ole vielä tukea tuolle ONLYMESSAGE toiminnallisuudelle. Jos sille on tarvetta, niin eiköhän sen voi jossain vaiheessa toteuttaa.

Tarvetta toteutukselle ei liene.

Testasin lainaamalla niteen, jonka NOT_LOAN-arvo=1 eli Ei lainata, ei varata.
Lainatessa ilmoitus:
Nide ei ole lainattavissa (Ei lainata, ei varata). Ohitetaanko lainausesto?

Lainauksen jälkeen niteellä säilyi arvo NOT_LOAN=1 kuten kuuluukin säilyä.

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

Emmi Takkinen kirjoitti:

Anneli Österman kirjoitti:

Joko asetuksen päälle menemiseen kuluu aikaa tai sitten se ei toimi OUTIssa. Määritin asetukseen -1: 0 ja yritin lainata niteen, jonka tila on -1. Nide ei mene lainaan, eikä niteen tila vaihdu.

OUTIssa oli pois päältä asetus "AllowNotForLoanOverride". Laitoin sen "Allow", niin lainaus poisti niteen tilan -1 ja se vaihtui 0.

Ah, no se selittää. Meillä on ihan tarkoituksella tuo asetus päällä, joten voi olla, ettei me voida tätä ominaisuutta varsinaisesti ottaa käyttöön.

Vie Atom PDF