upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-06-13 18:50:47 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

Properly encapsulate require_once for app.php

Browse source 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
This commit is contained in:
Julius Härtl 2018-02-15 11:20:22 +01:00
parent 364e7fe1be
commit a61608e8c7
No known key found for this signature in database
GPG key ID: 4C614C6ED2CDE6DF
1 changed files with 12 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
lib/private/legacy/app.php
View file

@ -136,6 +136,7 @@ class OC_App {
* load a single app
*
* @param string $app
* @throws Exception
*/
public static function loadApp($app) {
self::$loadedApps[] = $app;
@ -149,7 +150,15 @@ class OC_App {
if (is_file($appPath . '/appinfo/app.php')) {
\OC::$server->getEventLogger()->start('load_app_' . $app, 'Load app: ' . $app);
self::requireAppFile($app);
try {
self::requireAppFile($app);
} catch (Error $ex) {
\OC::$server->getLogger()->logException($ex);
if (!\OC::$server->getAppManager()->isShipped($app)) {
// Only disable apps which are not shipped
self::disable($app);
}
}
if (self::isType($app, array('authentication'))) {
// since authentication apps affect the "is app enabled for group" check,
// the enabled apps cache needs to be cleared to make sure that the
@ -231,16 +240,8 @@ class OC_App {
* @param string $app app name
*/
private static function requireAppFile($app) {
try {
// encapsulated here to avoid variable scope conflicts
require_once $app . '/appinfo/app.php';
} catch (Error $ex) {
\OC::$server->getLogger()->logException($ex);
if (!\OC::$server->getAppManager()->isShipped($app)) {
// Only disable apps which are not shipped
self::disable($app);
}
}
// encapsulated here to avoid variable scope conflicts
require_once $app . '/appinfo/app.php';
}
/**