deblan/side_menu
deblan/side_menu
4
11
Fork 9
Code Issues 20 Pull requests 5 Projects Releases 137 Wiki Activity

"Refactor Side Menu JsController to Use IAvatarManager via Dependency Injection for Nextcloud 32 Com #442

Merged
deblan merged 4 commits from AndyXheli/side_menu:develop into develop 2025-09-28 19:40:53 +02:00
Conversation 2 Commits 4 Files changed 1 +15 -15

4 commits

Author SHA1 Message Date
AndyXheli
9ad73c7bf7 Update lib/Controller/JsController.php
All checks were successful
ci/woodpecker/pr/build Pipeline was successful
Details
ci/woodpecker/pr/security Pipeline was successful
Details 
Signed-off-by: AndyXheli <andyxheli@gmail.com>
 2025-09-20 16:43:41 +02:00
AndyXheli
acde24ea71 Update lib/Controller/JsController.php
Some checks are pending
ci/woodpecker/pr/build Pipeline is pending approval
Details
ci/woodpecker/pr/security Pipeline is pending approval
Details
2025-09-20 16:42:52 +02:00
AndyXheli
8bd89b4565 Update lib/Controller/JsController.php
Some checks are pending
ci/woodpecker/pr/build Pipeline is pending approval
Details
ci/woodpecker/pr/security Pipeline is pending approval
Details
2025-09-20 16:41:49 +02:00
AndyXheli
58cfbc24af "Refactor Side Menu JsController to Use IAvatarManager via Dependency Injection for Nextcloud 32 Com
All checks were successful
ci/woodpecker/pr/build Pipeline was successful
Details
ci/woodpecker/pr/security Pipeline was successful
Details 
🔧 Problem: The Side Menu app was calling OC\Server::getAvatarManager(), which no longer exists in Nextcloud 32.

✅ Fix:

Injected IAvatarManager using Nextcloud’s public API instead of accessing it via OC::$server.

Updated the constructor to accept IAvatarManager, IUserSession, INavigationManager, and IURLGenerator.

Replaced all direct calls to OC::$server->getAvatarManager() and similar with the injected services.

This makes the app compatible with Nextcloud 32 and avoids deprecated internal method calls.

Signed-off-by: AndyXheli <andyxheli@gmail.com>
 2025-09-20 16:25:58 +02:00