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

Räätälöityihin viesteihin ei saa laitettua dataa tietokantatauluista #158

Open
Tuomas-Kyyti opened this issue Nov 11, 2024 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@Tuomas-Kyyti
Copy link

Mikä vikana?

Asiakkaalle Viesti-toiminnolla lähetettäviin räätälöityihin viesteihin ei pysty laittamaan dataa tietokantataluista. Esim. viestipohjaan laitetusta "Kirjastokorttisi on jäänyt kirjastoon <<branches.branchname>>." jää vain kokonaan pois tuo kirjaston nimi. Tämä puute vaikuttaa paljon toiminnon hyödyllisyyteen.

Yhteisössä on tähän korjaus https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38095. Saako sen tuotua meille? Testasin sitä sandboxissa ja tuntuisi toimivan.

Mitä pitäisi tapahtua

No response

Kuinka toistaa ongelma/asia

No response

Jotain muuta?

No response

Asiasanat/tagit

No response

@Tuomas-Kyyti Tuomas-Kyyti added the bug Something isn't working label Nov 11, 2024
@emta001
Copy link
Contributor

emta001 commented Nov 11, 2024

Tuossa yhteisön tiketissä on esimerkissä käytetty tämmöstä syntaksia: [% branches.branchname %]. Toimiiko tuokaan oikein?

@emta001 emta001 moved this to Kommentoitu in Versionvaihto Nov 11, 2024
@Tuomas-Kyyti
Copy link
Author

Tuolla korjauksella toimi ihan <<branches.branchname>>-koodi. Itse asia tuo [% branches.branchname %] ei toimi. Tätä vähän ihmettelinkin.

@AnneliO
Copy link
Contributor

AnneliO commented Nov 14, 2024

Katselen tässä Bywater Solutionsin esittelyvideota versioiden 23.11 ja 24.05 ominaisuuksista ja siellä sanotaan, että uusissa viestipohjissa ei toimi <<>>-tägit, koska ne tukee vain Template Toolkitia. Ja että vanhat viestipohjat tukee hauenleukoja.

Nyt en kuitenkaan ole varma, kuuluuko nuo räätälöidyt viestit näihin "uusiin" viestipohjiin. Halusin kuitenkin laittaa tämän tiedon johonkin ylös ja tämä tiketti tuli mieleen, kun kuulin tuon väitteen videolla.

@AnneliO
Copy link
Contributor

AnneliO commented Nov 15, 2024

Testailin myös tuota bugia 38095 sandboxissa ja kävin kommentoimassa Bugzillassa. Eihän tuo testausohjeen mukaan toiminut, joten pistin Failed QA siihen.

@atarnus
Copy link
Contributor

atarnus commented Nov 18, 2024

Katselen tässä Bywater Solutionsin esittelyvideota versioiden 23.11 ja 24.05 ominaisuuksista ja siellä sanotaan, että uusissa viestipohjissa ei toimi <<>>-tägit, koska ne tukee vain Template Toolkitia. Ja että vanhat viestipohjat tukee hauenleukoja.

Kurkkasin tämän takia testillä meidän tervetuloviestiä, jossa on määritelty virkailijoille oma viesti näin:
[% IF "<<borrowers.categorycode>>" == "VIRKAILIJA" %]
ja niinpä se näytti laittavan uudelle virkailijalle asiakasviestin, eli tarkistus ei toiminut.
Ei hyväksynyt [% borrowers.categorycode %] tuohon, koska siinä on jo tosiaan nuo toolkitia merkkaavat prosenttimerkit, mutta näyttäisi nyt pikaisella testillä toimivan taas oikein yksinkertaisesti näin (huomaa ettei ole borrowerS taulun mukaan vaan borrower - käsittääkseni sivun koodin mukaan):
[% IF borrower.categorycode == "VIRKAILIJA" %]

Viestin lopussa olleet kirjastotiedot näkyivät oikein, vaikka ne onkin kulmasulkeissa. Eli olisiko nyt näin että tuolla Template Toolkitin "sisällä" pitänee nyt käyttää noita borrower.arvo ja perustekstin seassa nämä kulmasulkeet edelleen pelaa?

@AnneliO
Copy link
Contributor

AnneliO commented Dec 4, 2024

Unohdin kirjata tänne, että bugissa oli testausohjeet väärin. Kun testasin "oikein" eli käytin Template Toolkit- muotoista määritystä [% branch.branchname %], niin korjauskin toimi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Kommentoitu
Development

No branches or pull requests

5 participants