upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-05-19 08:25:56 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 14

Fix translated app details

Browse source 
Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
Joas Schilling 2021-09-30 09:08:17 +02:00
parent e229cd3d53
commit fd4ff58d62
No known key found for this signature in database
GPG key ID: 7076EA9751AACDDA
4 changed files with 30 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
apps/updatenotification/lib/Controller/APIController.php
View file

@ -35,6 +35,8 @@ use OCP\AppFramework\Http\DataResponse;
use OCP\AppFramework\OCSController;
use OCP\IConfig;
use OCP\IRequest;
use OCP\IUserSession;
use OCP\L10N\IFactory;
class APIController extends OCSController {
@ -47,23 +49,29 @@ class APIController extends OCSController {
/** @var AppFetcher */
protected $appFetcher;
/**
* @param string $appName
* @param IRequest $request
* @param IConfig $config
* @param IAppManager $appManager
* @param AppFetcher $appFetcher
*/
public function __construct($appName,
/** @var IFactory */
protected $l10nFactory;
/** @var IUserSession */
protected $userSession;
/** @var string */
protected $language;
public function __construct(string $appName,
IRequest $request,
IConfig $config,
IAppManager $appManager,
AppFetcher $appFetcher) {
AppFetcher $appFetcher,
IFactory $l10nFactory,
IUserSession $userSession) {
parent::__construct($appName, $request);
$this->config = $config;
$this->appManager = $appManager;
$this->appFetcher = $appFetcher;
$this->l10nFactory = $l10nFactory;
$this->userSession = $userSession;
}
/**
@ -98,7 +106,7 @@ class APIController extends OCSController {
$this->appFetcher->setVersion($newVersion, 'future-apps.json', false);
// Apps available on the app store for that version
$availableApps = array_map(function (array $app) {
$availableApps = array_map(static function (array $app) {
return $app['id'];
}, $this->appFetcher->get());
@ -109,6 +117,8 @@ class APIController extends OCSController {
], Http::STATUS_NOT_FOUND);
}
$this->language = $this->l10nFactory->getUserLanguage($this->userSession->getUser());
$missing = array_diff($installedApps, $availableApps);
$missing = array_map([$this, 'getAppDetails'], $missing);
sort($missing);
@ -129,8 +139,8 @@ class APIController extends OCSController {
* @param string $appId
* @return string[]
*/
protected function getAppDetails($appId): array {
$app = $this->appManager->getAppInfo($appId);
protected function getAppDetails(string $appId): array {
$app = $this->appManager->getAppInfo($appId, false, $this->language);
return [
'appId' => $appId,
'appName' => $app['name'] ?? $appId,

6
apps/updatenotification/lib/Notification/Notifier.php
View file

@ -135,7 +135,7 @@ class Notifier implements INotifier {
$notification->setLink($this->url->linkToRouteAbsolute('settings.AdminSettings.index', ['section' => 'overview']) . '#version');
}
} else {
$appInfo = $this->getAppInfo($notification->getObjectType());
$appInfo = $this->getAppInfo($notification->getObjectType(), $languageCode);
$appName = ($appInfo === null) ? $notification->getObjectType() : $appInfo['name'];
if (isset($this->appVersions[$notification->getObjectType()])) {
@ -195,7 +195,7 @@ class Notifier implements INotifier {
return \OC_App::getAppVersions();
}
protected function getAppInfo($appId) {
return \OC_App::getAppInfo($appId);
protected function getAppInfo($appId, $languageCode) {
return \OC_App::getAppInfo($appId, false, $languageCode);
}
}

6
lib/private/Installer.php
View file

@ -114,9 +114,8 @@ class Installer {
}
$basedir = $app['path'].'/'.$appId;
$info = OC_App::getAppInfo($basedir.'/appinfo/info.xml', true);
$l = \OC::$server->getL10N('core');
$info = OC_App::getAppInfo($basedir.'/appinfo/info.xml', true, $l->getLanguageCode());
if (!is_array($info)) {
throw new \Exception(
@ -171,8 +170,7 @@ class Installer {
//run appinfo/install.php
self::includeAppScript($basedir . '/appinfo/install.php');
$appData = OC_App::getAppInfo($appId);
OC_App::executeRepairSteps($appId, $appData['repair-steps']['install']);
OC_App::executeRepairSteps($appId, $info['repair-steps']['install']);
//set the installed version
\OC::$server->getConfig()->setAppValue($info['id'], 'installed_version', OC_App::getAppVersion($info['id'], false));

5
lib/private/legacy/OC_App.php
View file

@ -980,13 +980,14 @@ class OC_App {
}
\OC::$server->getAppManager()->clearAppsCache();
$appData = self::getAppInfo($appId);
$l = \OC::$server->getL10N('core');
$appData = self::getAppInfo($appId, false, $l->getLanguageCode());
$ignoreMaxApps = \OC::$server->getConfig()->getSystemValue('app_install_overwrite', []);
$ignoreMax = in_array($appId, $ignoreMaxApps, true);
\OC_App::checkAppDependencies(
\OC::$server->getConfig(),
\OC::$server->getL10N('core'),
$l,
$appData,
$ignoreMax
);