]> source.dussan.org Git - nextcloud-server.git/commitdiff
fix(AppManager): Name correct method in error + show error string 45737/head
authorJosh <josh.t.richards@gmail.com>
Sun, 9 Jun 2024 13:36:32 +0000 (09:36 -0400)
committerGitHub <noreply@github.com>
Sun, 9 Jun 2024 13:36:32 +0000 (09:36 -0400)
Signed-off-by: Josh <josh.t.richards@gmail.com>
lib/private/App/AppManager.php

index 99d1c913cedc081f08b36661ff8a152a9478b3e7..0786fbded20fd62d951b3d9eb2fce532de08d1ef 100644 (file)
@@ -319,7 +319,9 @@ class AppManager implements IAppManager {
 
                        if (!is_array($groupIds)) {
                                $jsonError = json_last_error();
-                               $this->logger->warning('AppManger::checkAppForUser - can\'t decode group IDs: ' . print_r($enabled, true) . ' - json error code: ' . $jsonError);
+                               $jsonErrorMsg = json_last_error_msg();
+                               // this really should never happen (if it does, the admin should check the `enabled` key value via `occ config:list` because it's bogus for some reason)
+                               $this->logger->warning('AppManager::checkAppForUser - can\'t decode group IDs listed in app\'s enabled config key: ' . print_r($enabled, true) . ' - JSON error (' . $jsonError . ') ' . $jsonErrorMsg);
                                return false;
                        }
 
@@ -345,7 +347,9 @@ class AppManager implements IAppManager {
 
                        if (!is_array($groupIds)) {
                                $jsonError = json_last_error();
-                               $this->logger->warning('AppManger::checkAppForUser - can\'t decode group IDs: ' . print_r($enabled, true) . ' - json error code: ' . $jsonError);
+                               $jsonErrorMsg = json_last_error_msg();
+                               // this really should never happen (if it does, the admin should check the `enabled` key value via `occ config:list` because it's bogus for some reason)
+                               $this->logger->warning('AppManager::checkAppForGroups - can\'t decode group IDs listed in app\'s enabled config key: ' . print_r($enabled, true) . ' - JSON error (' . $jsonError . ') ' . $jsonErrorMsg);
                                return false;
                        }