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

Finna-liitännäinen: Lisätään liitännäiselle puuttuva versionumero #9

Closed
emta001 opened this issue Nov 21, 2024 · 6 comments
Closed
Assignees

Comments

@emta001
Copy link

emta001 commented Nov 21, 2024

Liitännäiselle ei ole määritetty versionumeroa ja sen sijaan kannasta löytyy arvo {VERSION}. Tästä ilmeisesti syntyy lokeille virhe

Use of uninitialized value $plugin_version in string ne at /home/koha/Koha/Koha/Plugins/Base.pm line 81.

Lisätään versionumero liitännäiseen (ja tarvittaessa käydään ajamassa muutos myös kantaan).

@emta001 emta001 self-assigned this Nov 21, 2024
@emta001
Copy link
Author

emta001 commented Nov 21, 2024

Lisätty ja testattu testeillä. Versionumero pysyy nyt nätisti paikallaan tietokannassa. Siitä tuleeko tuota herjaa enää en osaa sanoa, mutta ehkäpä se selviää kun viedään tää tuotantoon.

@emta001 emta001 moved this to Seuraavaan päivitykseen in Tikettien seuranta Nov 21, 2024
@emta001
Copy link
Author

emta001 commented Dec 2, 2024

Viety tuotantoon.

@emta001 emta001 closed this as completed Dec 2, 2024
@github-project-automation github-project-automation bot moved this from Seuraavaan päivitykseen to Suljettu in Tikettien seuranta Dec 2, 2024
@CodoDerDritte
Copy link

Puuttuvasta versionumerosta tulee edelleen lokimerkintöjä, ainakin apachen lokehin:

Mon Dec 09 00:00:01.578654 2024] [core:notice] [pid 305:tid 140675704138816] AH00094: Command line: '/usr/sbin/apache2'
Use of uninitialized value $plugin_version in string ne at /home/koha/Koha/Koha/Plugins/[Base.pm](http://base.pm/) line 81

@CodoDerDritte CodoDerDritte reopened this Dec 9, 2024
@github-project-automation github-project-automation bot moved this from Suljettu to Kommentoitu in Tikettien seuranta Dec 9, 2024
@github-project-automation github-project-automation bot moved this from Suljettu to Kommentoitu in Tikettien seuranta Dec 9, 2024
@emta001
Copy link
Author

emta001 commented Dec 10, 2024

Puuttuvasta versionumerosta tulee edelleen lokimerkintöjä, ainakin apachen lokehin:

Mon Dec 09 00:00:01.578654 2024] [core:notice] [pid 305:tid 140675704138816] AH00094: Command line: '/usr/sbin/apache2'
Use of uninitialized value $plugin_version in string ne at /home/koha/Koha/Koha/Plugins/[Base.pm](http://base.pm/) line 81

Niin näyttäs tulevan myös plackin lokiin. Pitää varmaan avata ihan oma tikettinsä tälle, kerta tää ei johtunut tästä Finna-plugarista.

@emta001
Copy link
Author

emta001 commented Dec 10, 2024

Äsh, taisin tutkiakin tätä joku hetki sitten ja totesin, että ongelma liittynee globaalien muuttujien käyttämiseen. Meillä on kaikissa plugareissa nyt versionumero paikallaan, mutta miun nähdäkseni sitä versionumeroa ei kutsuta edes kannasta, vaan plugareissa kiinni olevasta muuttujasta our $metadata. Samoin siellä on muuttuja our $VERSION, jota käytetään tuossa $metadata muuttujassa. Pasi muisteli, että plack estää sen, että funktio ei näe funktion ulkopuolisia muuttujia. Eli tässä voi käydä nyt niin, että kun Base.pm kutsuu plugarin metodia new, plack estääkin jommankumman muuttujan kutsumisen. Ja siitä syntyy sitten virhe, kun plugaria syystä tai toisesta yritetään päivittää.

@emta001
Copy link
Author

emta001 commented Dec 10, 2024

@emta001 emta001 closed this as completed Dec 10, 2024
@github-project-automation github-project-automation bot moved this from Kommentoitu to Suljettu in Tikettien seuranta Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants