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

[Bugi] Hyllyvarausraportilla virhe 500, jos kohasuomi-pendingreserves.tmp tiedostoa ei ole olemassa #254

Closed
emta001 opened this issue Nov 8, 2022 · 4 comments
Assignees
Labels

Comments

@emta001
Copy link
Collaborator

emta001 commented Nov 8, 2022

Mikä vikana?

Jos update_holds_to_pull.pl cronia ei ole ajettu eikä kohasuomi-pendingreserves.tmp ole tästä syystä olemassa, Koha kuolee virheeseen 500. Lokissa:

Template process failed: undef error - The given date (UNKNOWN) does not match the date format (iso) at /home/lmstrand/Koha/Koha/[DateUtils.pm](http://dateutils.pm/) line 193.

Johtuu siis siitä, että raportin koodiin on kovakoodattu "UNKNOWN" Perlin päässä eikä arvoa voida käsitellä Kohan DateUtils objektina templaten puolella.

Mitä pitäisi tapahtua

Hyllyvarausraportin tulisi aueta tyhjänä ja otsikolla "Holds to pull reported on UNKNOWN".

Kuinka toistaa ongelma/asia

No response

Selain

No response

Jotain muuta?

No response

Kohan osio

No response

@emta001 emta001 added the bug label Nov 8, 2022
@CodoDerDritte
Copy link
Collaborator

CodoDerDritte commented Nov 8, 2022

Itseasiassa pendingreserves.pl:ssä on jo olemassa käsittely tyhjälle tiedostolle, mutta tuo olemattoman päiväyksen kuljettaminen dateutilsin läpi rikkoo sen nyt. Ennen versiopäivitystä päiväyksiä ei kuljetettu dateutilsin läpi, jolloin template toimi oikein. Tämä voitanee korjata siten että reported on aikaan tuutataan vaikka nykyinen kellonaika, jolloin template hoksaa, ettei täällä mitää oo ja käskee ajastamaan update-holds-to-pullin.

@CodoDerDritte CodoDerDritte self-assigned this Nov 8, 2022
@CodoDerDritte
Copy link
Collaborator

Tähän on nyt korjaus, pistetään kyytiin ensi viikon tiistain päivitykseen.

@CodoDerDritte
Copy link
Collaborator

Kaipaa vielä käänöksen päivittämistä, koska vanha käännös ei näytä enää tarttuvan.

@CodoDerDritte
Copy link
Collaborator

Avaan käännöksestä uuden tiketin translations-repoon ja suljen tämän.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Suljettu
Development

No branches or pull requests

3 participants