Takaajan tietoihin tekstimuutosehdotus taattavan maksuista, ehdotus 2

[pirliis]

Kehitysehdotukseni

Tiketissä KohaSuomi/Koha-translations#46 on pyydetty taattavan maksunäkymään tekstimuutosta, jonka voisi ilmeisesti toteuttaa kevyemmin käännösmuutoksena.

Tämä tekstin lisäysehdotus vaatisi ilmeisesti koodimuutoksia.

Voisiko taattavien maksunäkymätekstiin saada lisäksi tiedon, kuinka paljon maksuja taattavilla on yhteensä?
Eli teksti olisi kokonaisuudessaan näin:

"Maksut: Asiakkaan taattavilla on lainauskieltoon vaikuttavia maksuja yhteensä 9,50 €." Yhteensä (sis. maksut, jotka eivät vaikuta lainauskieltoon): 13 € . Näytä kaikki maksut."

Jotain muuta?

No response

[AnneliO]

Asiantuntijaryhmään valmistelua: @pirliis Taattavat maksut -välilehden ongelmasta voisi tehdä erillisen tiketin.

[pirliis]

#1239

[AnneliO]

Koha-Suomen asiantuntijaryhmä 27.5.2024: Toteutetaan itse ja tarjotaan yhteisölle muutos. Edellinen vastaava on mennyt siellä läpi.

[AnneliO]

@pirliis Teetkö tästä tiketin yhteisön Bugzillaan?

[pirliis]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37102

[lmstrand]

Uusi ominaisuus viety testeille. Tämän lisäksi asiakkaalla näkyy (muiden takaajien ja heidän) taattaviensa maksut ylempänäkin samalla logiikalla, kuin itse asiakkaan maksut sis. maksut, jotka eivät vaikuta lainakieltoon (tietoa ei ennen näytetty taattavien tapauksessa):

[lmstrand]

Käyttöön tarvitsee sysprefejä:

NoIssuesChargeGuarantees
Estä asiakasta lainaamasta, jos hänen taattavillaan on maksuja enemmän kuin EUR maksuja.
NoIssuesChargeGuarantorsWithGuarantees
Estä asiakasta lainaamasta, jos asiakkaalla on takaajia ja noiden takaajien taattavilla on yhteensä enemmän kuin EUR maksuja.

[AnneliO]

Suomenkieliset käännökset viety GitHubiin Koha-translations -tietovarantoon.

[pirliis]

Käyttöön tarvitsee sysprefejä:

NoIssuesChargeGuarantees Estä asiakasta lainaamasta, jos hänen taattavillaan on maksuja enemmän kuin EUR maksuja. NoIssuesChargeGuarantorsWithGuarantees Estä asiakasta lainaamasta, jos asiakkaalla on takaajia ja noiden takaajien taattavilla on yhteensä enemmän kuin EUR maksuja.

Nämä on kimppakohtaisesti päätettävissä, onko käytössä vai ei. OUTIssa nämä ei ole olleet koskaan käytössä.

[pirliis]

OUTIn testillä taattavan "ei lainauskieltoon" vaikuttavat maksut näkyvät oikein takaajan tiedoissa eli esim.:

Voisiko OUTIn testille saada ainakin palautuskehotuksen ja noutamattomien varausten maksuajot päälle, jotta näkisi, että myös lainauskieltoon vaikuttavat maksut näkyvät oikein? Myös myöhästymismaksujen ajot olisi mukava saada päälle, kun testillä on niin vähän maksuja.

[lmstrand]

Lisäsin outi-testille seuraavat cronit:

 13 09 * * *    $TRIGGER cronjobs/advance_notices.pl -c --itemscontent date_due,title,part_number,enumchron,author,barcode
 13 09 * * *    $TRIGGER cronjobs/overdue_notices.pl -letternumbers 12 -t -p -s -v
 10 00 * * *        $TRIGGER misc/expire_holds.sh EITILASTO KAUKOLAINA KOTIPALVEL

[lmstrand]

01 00 * * * $TRIGGER cronjobs/fines.pl -v -l -o /var/log/koha/fines ajo on jo testillä ollutkin päällä.

[pirliis]

OUTIn testillä taattavien maksut näkyvät nyt oikein ja halutulla tavalla. Tosin käännöstekstit ovat vielä puutteelliset.

[MikkoLiimatainen]

Vaskin testillä maksut esitetään myös halutulla tavalla. Käännökset tosiaan vielä puutteelliset.

Näytä kaikki maksut linkki ei näytä siirtävän sivua Taattavien maksut kohtaan, kuten olisi tarkoitus. Linkki osoittaa nyt sivun kohtaan #guarantees_finesandcharges, kun pitäisi osoittaa kaiketi kohtaan #guarantees_finesandcharges-tab.

[pirliis]

Vaskin testillä maksut esitetään myös halutulla tavalla. Käännökset tosiaan vielä puutteelliset.

Näytä kaikki maksut linkki ei näytä siirtävän sivua Taattavien maksut kohtaan, kuten olisi tarkoitus. Linkki osoittaa nyt sivun kohtaan #guarantees_finesandcharges, kun pitäisi osoittaa kaiketi kohtaan #guarantees_finesandcharges-tab.

OUTIn testillä kun klikkaa linkkiä "Näytä kaikki maksut", avautuu Taattavien maksut -välilehti, mutta sivua pitää vierittää alas, jotta välilehdet tulevat näkyville. Auttaisiko tuo "tab", että sivu myös pomppaisi välilehtien kohdalle?

[MikkoLiimatainen]

Vaskin testillä maksut esitetään myös halutulla tavalla. Käännökset tosiaan vielä puutteelliset.
Näytä kaikki maksut linkki ei näytä siirtävän sivua Taattavien maksut kohtaan, kuten olisi tarkoitus. Linkki osoittaa nyt sivun kohtaan #guarantees_finesandcharges, kun pitäisi osoittaa kaiketi kohtaan #guarantees_finesandcharges-tab.

OUTIn testillä kun klikkaa linkkiä "Näytä kaikki maksut", avautuu Taattavien maksut -välilehti, mutta sivua pitää vierittää alas, jotta välilehdet tulevat näkyville. Auttaisiko tuo "tab", että sivu myös pomppaisi välilehtien kohdalle?

Juu, testasin selaimen tarkista-toiminnolla muokata tuon linkin tuolla tavalla, niin silloin se siirtää käyttäjän suoraan oikeaan kohtaan. Tässä on varmaan tuon välilehden id muutettu ja linkki osoittaa vielä vanhaan id:hen.

[AnneliO]

Hetken meni ennen kuin tajusin, että tekemäni aiemmin tekemäni käännösmuutos oli puutteellinen. :D Nyt kuitenkin viety koha-translations-tietovarantoon uusi versio, jossa:

Patron's guarantees collectively owe %s.

Asiakkaan taattavilla on yhteensä maksuja %s. -> Asiakkaan taattavilla on lainauskieltoon vaikuttavia maksuja yhteensä %s.

Tulee testeille näkyviin seuraavassa buildissä.

[lmstrand]

Sivun vieritykseen korjaus yhteisössä: Koha-Community/Koha@e2d54b1

[lmstrand]

Käännös ja sivun vieritys kaikkiin maksuihin pitäisi olla korjattuna testeillä.

[AnneliO]

Hmm.. käännösmuutos ei nyt näy jostain syystä vaikka sen mielestäni tein kyllä.

KohaSuomi/Koha-translations@2b67331

Selvittelen vielä.

[AnneliO]

Aaaah, lainauskieltoon liittyviä tekstejä olikin useampia. Tehty myös nämä muutokset.

Patron's guarantees collectively owe %s. %s -> Asiakkaan taattavilla on lainauskieltoon vaikuttavia maksuja yhteensä %s.
Patron's guarantees collectively owe %s. Total (incl. non-blocking): %s. -> Asiakkaan taattavilla on lainauskieltoon vaikuttavia maksuja yhteensä %s. %s
Patron's guarantees collectively owe %s. Total (incl. non-blocking ): %s. Guarantees total (incl. non-blocking ): %s. %s -> Asiakkaan taattavilla on lainauskieltoon vaikuttavia maksuja yhteensä %s. Taattavilla yhteensä (sis. maksut, jotka eivät vaikuta lainakieltoon): %s.
The patron's guarantors and their other guarantees collectively have a debt of %s. -> Asiakkaan taattavilla on lainauskieltoon vaikuttavia maksuja yhteensä %s. Yhteensä (sis. maksut, jotka eivät vaikuta lainakieltoon): %s. Taattavilla yhteensä (sis. maksut, jotka eivät vaikuta lainakieltoon): %s. %s
Patron's guarantors and their other guarantees collectively owe %s. %s -> Asiakkaan takaajilla ja heidän muilla taattavillaan on lainauskieltoon vaikuttavia maksuja yhteensä %s.
Patron's guarantors and their other guarantees collectively owe %s. Total (incl. non-blocking): %s. %s -> Asiakkaan takaajilla ja heidän muilla taattavillaan on lainauskieltoon vaikuttavia maksuja yhteensä %s. %s
Patron's guarantors and their other guarantees collectively owe %s. Total (incl. non-blocking): %s. %s -> Asiakkaan takaajilla ja heidän muilla taattavillaan on lainauskieltoon vaikuttavia maksuja yhteensä %s. Yhteensä (sis. maksut, jotka eivät vaikuta lainakieltoon): %s. %s

[AnneliO]

Testasin tällä asiakkaalla: https://outi-test.koha-suomi.fi/cgi-bin/koha/members/moremember.pl?borrowernumber=91426

Lari ajoi buildin uudelleen ja nyt minusta käännös on nyt oikein kohdillaan. Sivun vierityskin toimii linkistä oikein.

[pirliis]

Jep, OUTIn testillä käännökset näkyvät nyt oikein takaajan Tiedot-sivulla. :)