nextcloud-scanner icon indicating copy to clipboard operation
nextcloud-scanner copied to clipboard

Compatible with nextcloud 28

Open Boc-chi-no opened this issue 1 year ago • 3 comments

#38 #26 Compatible with nextcloud 28 [U] Refactored project structure to adhere to PSR-4 standards [U] Eliminated sudo usage for enhanced security

Boc-chi-no avatar Nov 30 '23 02:11 Boc-chi-no

@Boc-chi-no Thanks for your efforts to fix the compatibility issue!

I tested your fix and for me still the following Fatal error occurs:

    "File": "/var/www/html/custom_apps/scanner/lib/AppInfo/Application.php",
    "Line": 43,
    "message": "Error during app service registration: Call to undefined method OC\\Server::getEventDispatcher()",
    "exception": [],
    "CustomMessage": "Error during app service registration: Call to undefined method OC\\Server::getEventDispatcher()"

Do you think you could also adjust to the breaking change of the Event API in the NC28 ?

farrad avatar Jul 29 '24 21:07 farrad

@Boc-chi-no Thanks for your efforts to fix the compatibility issue!

I tested your fix and for me still the following Fatal error occurs:

    "File": "/var/www/html/custom_apps/scanner/lib/AppInfo/Application.php",
    "Line": 43,
    "message": "Error during app service registration: Call to undefined method OC\\Server::getEventDispatcher()",
    "exception": [],
    "CustomMessage": "Error during app service registration: Call to undefined method OC\\Server::getEventDispatcher()"

Do you think you could also adjust to the breaking change of the Event API in the NC28 ?

Oh, I see the issue now. I might have mistakenly thought it was compatible with NC28 when it was actually for NC27. I'll update my server, then I'll look into this issue and make it compatible with NC28.

Boc-chi-no avatar Jul 30 '24 14:07 Boc-chi-no

See compatible version for Nextcloud 28 and 29 in this pull request.

Note it is NOT backward compatible.

farrad avatar Sep 12 '24 16:09 farrad