Nextcloud 31 compatibility

[Mer0me]

The app seems to be broken on Nextcloud 31. Please make it available for this version.

[Bad-and-Mad]

Same here.

"Could not boot drawio: Call to undefined method OC\\Server::getLogger()"

[davidjgraph]

We'll take a look when we get time (which isn't now). Nextcloud are a little keen on breaking changes on major API increments. I know that can mean breaking changes, but someone them tell that they don't have to break the API every single time...

[janLo]

I think this might be a Job for sed: https://github.com/nextcloud/server/pull/40145/files

[davidjgraph]

@janLo have you tested it?

[janLo]

Just about to. If it works, I'll make a PR.

[janLo]

I haven't touched PHP for a long time but the changes are more mechanical - so it should be fine.

I tested it (create/load/save/preview) in my instance and it works again with these changes.
Thank you!

[igorehan]

There is something else that has broken since version 31, in addition to ILogger. This is sharing. When sharing drawio files and transferring the link to another person, the file listing opens briefly (i.e. there is access to the file), after which drawio starts loading and a redirect to the authorization page occurs

[janLo]

That is true, but I think it's unrelated to the logger changes. I also see no drawio-related error in the log for this. The only thing I see is that the bruteForce app causes some exception - but that's just a DEBUG message - I don't know if it has anything to do with the issue.

[janLo]

The request that is made after the file listing is briefly shown redirects without the share token:

I guess, that the plugin then just redirects to the login page:

[janLo]

It's indeed another change in the NC API - I have currently no setup where I can "recompile" the JS part of this plugin for testing, but the path to a fix should be the same as the one the richdocuments plugin took: https://github.com/nextcloud/richdocuments/pull/4030/files

Nevertheless, the logging fix is the first step to make the plugin work again at all ;)

[janLo]

I've provided the changes from richdocuments in #92, but haven't tested them yet.