fix: replace logger

[vitormattos]

The previous logger approach was removed at Nextcloud 30 and was needed to be replaced by Psr\Log\LoggerInterface

This PR fix:

Error: Call to undefined method OC\Server::getLogger() in /var/www/html/apps-writable/cms_pico/lib/Migration/Version010000From000908.php:65

[PhrozenByte]

Thank you for your contribution! ❤️

First of all, please note that cms_pico is abandoned dev-wise, thus there isn't going to be a new release in the foreseeable future. There's currently no stable version of cms_pico supporting any of Nextcloud's still officially supported branches; people still using cms_pico do so at their own risk and use custom app builds (usually) based on the cms_pico-1.0 branch.

You can ignore the failing CI, just wanted to see whether it's more broken than last time I checked - it indeed is.

If you're willing to invest more time nevertheless, can you please rebase on the cms_pico-1.0 branch? The master branch is the dev version of cms_pico v2.0. Just switching the PR's base unfortunately doesn't work properly. Did you check all usages of $logger to be compatible with the new LoggerInterface? Unfortunately I don't have the time to do a proper review of your PR, but the changes look good; so, if you affirm that you did check all usages, I'll merge your PR nevertheless. Thanks again!

[vitormattos]

Last changes:

• Moved my commit to a branch from cms_pico-1.0
• Changed the target of this PR to cms_pico-1.0

[PhrozenByte]

Did you check all usages of $logger to be compatible with the new LoggerInterface?

Did you check these? If yes, I'll merge it right away. Thanks again for your contribution! 👍

[vitormattos]

Yes, I didn't made smoke test looking all places of code that use the old log interface, but the piece of code that I tested worked fine and because this I made the replace at all places.

I took care to check the name of methods, because this the place that the code used LogException method, I replaced by a most similar method from LoggerInterface, the other places, the method called have the same name.

[PhrozenByte]

Alright, thanks again for your contribution! 👍