mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #26878 from nextcloud/bugfix/noid/dont-break-occ-when-an-app-is-breaking-in-application-class 

Don't break OCC if an app is breaking in it's Application class
This commit is contained in:
Joas Schilling 2021-05-05 11:50:40 +02:00 committed by GitHub
parent 20b34875d9 784b059a01
commit 4d82a9446d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 24 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/private/AppFramework/Bootstrap/Coordinator.php
View File

@ -113,6 +113,7 @@ class Coordinator {
*/ */
$appNameSpace = App::buildAppNamespace($appId); $appNameSpace = App::buildAppNamespace($appId);
$applicationClassName = $appNameSpace . '\\AppInfo\\Application'; $applicationClassName = $appNameSpace . '\\AppInfo\\Application';
try {
if (class_exists($applicationClassName) && in_array(IBootstrap::class, class_implements($applicationClassName), true)) { if (class_exists($applicationClassName) && in_array(IBootstrap::class, class_implements($applicationClassName), true)) {
try { try {
/** @var IBootstrap|App $application */ /** @var IBootstrap|App $application */
@ -121,13 +122,15 @@ class Coordinator {
// Weird, but ok // Weird, but ok
continue; continue;
} }
try {
$application->register($this->registrationContext->for($appId)); $application->register($this->registrationContext->for($appId));
}
} catch (Throwable $e) { } catch (Throwable $e) {
$this->logger->emergency('Error during app service registration: ' . $e->getMessage(), [ $this->logger->emergency('Error during app service registration: ' . $e->getMessage(), [
'exception' => $e, 'exception' => $e,
'app' => $appId,
]); ]);
} continue;
} }
} }

7
lib/private/legacy/OC_App.php
View File

@ -135,7 +135,14 @@ class OC_App {
ob_start(); ob_start();
foreach ($apps as $app) { foreach ($apps as $app) {
if (!isset(self::$loadedApps[$app]) && ($types === [] || self::isType($app, $types))) { if (!isset(self::$loadedApps[$app]) && ($types === [] || self::isType($app, $types))) {
try {
self::loadApp($app); self::loadApp($app);
} catch (\Throwable $e) {
\OC::$server->get(LoggerInterface::class)->emergency('Error during app loading: ' . $e->getMessage(), [
'exception' => $e,
'app' => $app,
]);
}
} }
} }
ob_end_clean(); ob_end_clean();