feat: enable kirby-log for K3.7 and PHP 8.1.x

[dennisbaum]

Would like to install bvdputte/kirby-log in K3.7.1 with PHP 8.1.x and get the following dependencies-error. I tried to composer require bvdputte/kirby-log -W; that runs without shell-errors, but leads to some panel-errors in '../vendor/katzgrau/klogger/src/Logger.php' There is already an issue @katzgrau/KLogger
So i guess the error comes from PHP 8.1.x Do you have any plans, to update kirby-log for the above requirements or do yo know an alternative logger, working with kirby-logbook? Thank you very much!!

composer require bvdputte/kirby-log

Using version ^2.1 for bvdputte/kirby-log
./composer.json has been updated
Running composer update bvdputte/kirby-log
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - katzgrau/klogger 1.2.0 requires psr/log 1.0.0 -> found psr/log[1.0.0] but the package is fixed to 3.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - katzgrau/klogger 1.2.1 requires psr/log ^1.0.0 -> found psr/log[1.0.0, ..., 1.1.4] but the package is fixed to 3.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - bvdputte/kirby-log 2.1.0 requires katzgrau/klogger ^1.2 -> satisfiable by katzgrau/klogger[1.2.0, 1.2.1].
    - Root composer.json requires bvdputte/kirby-log ^2.1 -> satisfiable by bvdputte/kirby-log[2.1.0].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.
You can also try re-running composer require with an explicit version constraint, e.g. "composer require bvdputte/kirby-log:*" to figure out if any version is installable, or "composer require bvdputte/kirby-log:^2.1" if you know which you need.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

[FynnZW]

katzgrau/klogger just fixed their bug with 1.2.2, so I believe it should work fine with php 8.1 now

[bvdputte]

I've pulled in their fix. Can you please check if it the new release works @dennisbaum ?

(sorry for the late response -- I'm on holiday)

[dennisbaum]

Works for me! Thank you for that!

If i remove the guard in kirby-logbook, that is working also with K3.7.2.1

$kirbyVersion = App::version();
/*
if (
    $kirbyVersion !== null &&
    (
        version_compare($kirbyVersion, '3.6.0-alpha', '<') === true ||
        version_compare($kirbyVersion, '3.7.0-alpha', '>=') === true
    )
) {
    throw new Exception(
        'The installed version of the Kirby LogBook plugin ' .
        'is not compatible with Kirby ' . $kirbyVersion
    );
}*/

If you want me to, i can add an feature-request there too.

Tnx again and i hope you enjoy(ed) your vacation!

[bvdputte]

@dennisbaum ; please check if https://github.com/bvdputte/kirby-logbook/releases/tag/1.0.4 works for you ?

[dennisbaum]

works also on k3.7.2.1. thx again!

[anselmh]

I’m still getting this:

composer require bvdputte/kirby-log    
Using version ^2.2 for bvdputte/kirby-log
./composer.json has been updated
Running composer update bvdputte/kirby-log
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - katzgrau/klogger 1.2.0 requires psr/log 1.0.0 -> found psr/log[1.0.0] but the package is fixed to 3.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - katzgrau/klogger[1.2.1, ..., 1.2.2] require psr/log ^1.0.0 -> found psr/log[1.0.0, ..., 1.1.4] but the package is fixed to 3.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - bvdputte/kirby-log 2.2.0 requires katzgrau/klogger ^1.2 -> satisfiable by katzgrau/klogger[1.2.0, 1.2.1, 1.2.2].
    - Root composer.json requires bvdputte/kirby-log ^2.2 -> satisfiable by bvdputte/kirby-log[2.2.0].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.
You can also try re-running composer require with an explicit version constraint, e.g. "composer require bvdputte/kirby-log:*" to figure out if any version is installable, or "composer require bvdputte/kirby-log:^2.1" if you know which you need.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Any idea what the issue is and how to fix it? Using Kirby 3.8.2 with composer.

[bvdputte]

Ouch :/ I probably done something wrong with composer stuff...

@anselmh, I've delete the composer.lock file and ran composer updateagain. In my quick test, this now works.

Can you please check out version 2.2.1 please?

[anselmh]

@bvdputte this is what I get now:

composer require bvdputte/kirby-log      
Using version ^2.2 for bvdputte/kirby-log
Your requirements could not be resolved to an installable set of packages.
  Problem 1
    - bvdputte/kirby-log 2.2.1 requires katzgrau/klogger ^1.2.2 -> satisfiable by katzgrau/klogger[1.2.2].
    - bvdputte/kirby-log 2.2.0 requires katzgrau/klogger ^1.2 -> satisfiable by katzgrau/klogger[1.2.0, 1.2.1, 1.2.2].
    - katzgrau/klogger 1.2.0 requires psr/log 1.0.0 -> found psr/log[1.0.0] but the package is fixed to 3.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - katzgrau/klogger[1.2.1, ..., 1.2.2] require psr/log ^1.0.0 -> found psr/log[1.0.0, ..., 1.1.4] but the package is fixed to 3.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - Root composer.json requires bvdputte/kirby-log ^2.2 -> satisfiable by bvdputte/kirby-log[2.2.0, 2.2.1].

[anselmh]

https://github.com/katzgrau/KLogger/blob/master/composer.json is sticking to psr/log v1 while somewhere v3 is getting locked in the composer.lock file, I think in Kirby itself. Therefore it runs into conflicts…

[bvdputte]

I'm sorry, as I don't really use composer much, I'm not really getting what's happening? Maybe a more seasoned composer person can help me out here?

FYI; this seems to work though: composer require -W bvdputte/kirby-log.

[bvdputte]

https://github.com/katzgrau/KLogger/blob/master/composer.json is sticking to psr/log v1 while somewhere v3 is getting locked in the composer.lock file, I think in Kirby itself. Therefore it runs into conflicts…

You're probably right. It's most likely not Kirby itself, but one it it's dependencies that requires psr/log at ^3.0.0. I think that it's not breaking for my plugin (or its dependencies) to run composer require -W bvdputte/kirby-log so it also uses psr/log at version 3.