-
Notifications
You must be signed in to change notification settings - Fork 0
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
Comments
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. |
Viety tuotantoon. |
Puuttuvasta versionumerosta tulee edelleen lokimerkintöjä, ainakin apachen lokehin:
|
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. |
Ä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ää. |
Tein uuden tiketin |
Liitännäiselle ei ole määritetty versionumeroa ja sen sijaan kannasta löytyy arvo {VERSION}. Tästä ilmeisesti syntyy lokeille virhe
Lisätään versionumero liitännäiseen (ja tarvittaessa käydään ajamassa muutos myös kantaan).
The text was updated successfully, but these errors were encountered: