connect hook to register

[Antipkin-A]

Changed method for connect hook

[julien-nc]

I confirm it solves the sync issue on NC 21 and 22. Hooks are now actually triggered and documents are downloaded again if some file changes were pushed from a client.

It does not affect NC 23 for which it was already working fine.

[juliusknorr]

Also makes sense from a conceptual perspective in how the register and boot methods are caleld during Nextcloud application bootstrap.

[djtilo-ol]

"eneiluj"
It does not affect NC 23 for which it was already working fine.

I installed a fresh NextCloud 23 on my webspace and am sorry to say that the identical issue as in NC22 remains. Synced updates to documents (Office & PDF) are not recognized and an older version is shown.
The plugin is version 7.2.1 & I downloaded the latest NC 23.0.0, OO 6.4.2.
Regards

[julien-nc]

@djtilo-ol Thanks for the feedback. Have you tried to apply this patch to check if it actually fixes the issue in NC 23?

[djtilo-ol]

Hi, I can do that, sure. I am an ambitious rookie, but I can't quite make out what I have to alter in the file application.php as mentioned here
Can I download a patched file somewhere?

[julien-nc]

You can see the diff in the Files changed tab right here.
Or here is a link to the raw file from this branch: https://github.com/ONLYOFFICE/onlyoffice-nextcloud/raw/6bcbce371dcd1ebad02ce68b2454fd6346a046ce/appinfo/application.php

[djtilo-ol]

This fixes the issue in NC22 and NC23 - outstanding & thank you


!

[julien-nc]

@djtilo-ol And thank you for the feedback!

[djtilo-ol]

:) With your help I was able to send the patched file to our NC provider who added it for us, so that the problem is resolved until the new connector is released - that is absolutely amazing! Merry Christmas, should you celebrate it.

[biva]

It works for me as well :)

[9e9d]

Could someone please explain how to patch Nextcloud-VM with ONLYOFFICE integrated. I can't find anything resembling the 'application.php' file.

[djtilo-ol]

Hi, in the NextCloud folder you find the application.php in
/apps/onlyoffice/appinfo
Regards

[9e9d]

Thanks, I had to discover how find the content off /var/www
Unfortunately still big issues, office files won't sync back to windws pc with nextcloud app, reverted to 3.36 even, still not. I reverted this ' fix' again and no w seens better, so weird

[djtilo-ol]

The issue discussed here was that OnlyOffice could not open the latest version once two or three alterations were made locally and synced to NextCloud.
I checked by comparing Hashes of the files, like a fingerprint if you want.
Checksum is an app for NextCloud which allows you to generate i.e. MD5 hash value, then compare to local file and so on. This way I detected that NC was doing its job properly and OO was having a hard time opening the latest version.

[9e9d]

There is some voodoo processing going on to make up for the lack of an an actual process to deal with these document updates. See also: nextcloud/documentserver_community#191 I had to manually add a cron job to get some sort of reliable behavior, unbelievable really.

[9e9d]

Not to mention there is another bug about onlyoffice just stalling for merely trying to check the previous versions.