Make AppConfig part of the public API
Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
parent
50b6ee67cb
commit
0b1fb180a5
|
@ -26,12 +26,7 @@
|
||||||
*
|
*
|
||||||
*/
|
*/
|
||||||
|
|
||||||
use OCA\Provisioning_API\Apps;
|
return [
|
||||||
use OCA\Provisioning_API\Users;
|
|
||||||
use OCP\API;
|
|
||||||
|
|
||||||
$app = new \OCA\Provisioning_API\AppInfo\Application();
|
|
||||||
$app->registerRoutes($this, [
|
|
||||||
'ocs' => [
|
'ocs' => [
|
||||||
// Apps
|
// Apps
|
||||||
['root' => '/cloud', 'name' => 'Apps#getApps', 'url' => '/apps', 'verb' => 'GET'],
|
['root' => '/cloud', 'name' => 'Apps#getApps', 'url' => '/apps', 'verb' => 'GET'],
|
||||||
|
@ -61,5 +56,11 @@ $app->registerRoutes($this, [
|
||||||
['root' => '/cloud', 'name' => 'Users#addSubAdmin', 'url' => '/users/{userId}/subadmins', 'verb' => 'POST'],
|
['root' => '/cloud', 'name' => 'Users#addSubAdmin', 'url' => '/users/{userId}/subadmins', 'verb' => 'POST'],
|
||||||
['root' => '/cloud', 'name' => 'Users#removeSubAdmin', 'url' => '/users/{userId}/subadmins', 'verb' => 'DELETE'],
|
['root' => '/cloud', 'name' => 'Users#removeSubAdmin', 'url' => '/users/{userId}/subadmins', 'verb' => 'DELETE'],
|
||||||
|
|
||||||
|
// Config
|
||||||
|
['name' => 'AppConfig#getApps', 'url' => '/api/v1/config/apps', 'verb' => 'GET'],
|
||||||
|
['name' => 'AppConfig#getKeys', 'url' => '/api/v1/config/apps/{app}', 'verb' => 'GET'],
|
||||||
|
['name' => 'AppConfig#getValue', 'url' => '/api/v1/config/apps/{app}/{key}', 'verb' => 'GET'],
|
||||||
|
['name' => 'AppConfig#setValue', 'url' => '/api/v1/config/apps/{app}/{key}', 'verb' => 'POST'],
|
||||||
|
['name' => 'AppConfig#deleteKey', 'url' => '/api/v1/config/apps/{app}/{key}', 'verb' => 'DELETE'],
|
||||||
],
|
],
|
||||||
]);
|
];
|
||||||
|
|
|
@ -0,0 +1,158 @@
|
||||||
|
<?php
|
||||||
|
/**
|
||||||
|
* @copyright Copyright (c) 2016 Joas Schilling <coding@schilljs.com>
|
||||||
|
*
|
||||||
|
* @license GNU AGPL version 3 or any later version
|
||||||
|
*
|
||||||
|
* This program is free software: you can redistribute it and/or modify
|
||||||
|
* it under the terms of the GNU Affero General Public License as
|
||||||
|
* published by the Free Software Foundation, either version 3 of the
|
||||||
|
* License, or (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This program is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
* GNU Affero General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU Affero General Public License
|
||||||
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
|
namespace OCA\Provisioning_API\Controller;
|
||||||
|
|
||||||
|
|
||||||
|
use OCP\AppFramework\Http;
|
||||||
|
use OCP\AppFramework\Http\DataResponse;
|
||||||
|
use OCP\AppFramework\OCSController;
|
||||||
|
use OCP\IAppConfig;
|
||||||
|
use OCP\IConfig;
|
||||||
|
use OCP\IRequest;
|
||||||
|
|
||||||
|
class AppConfigController extends OCSController {
|
||||||
|
|
||||||
|
/** @var IAppConfig */
|
||||||
|
protected $config;
|
||||||
|
|
||||||
|
/** @var IConfig */
|
||||||
|
protected $appConfig;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param string $appName
|
||||||
|
* @param IRequest $request
|
||||||
|
* @param IConfig $config
|
||||||
|
* @param IAppConfig $appConfig
|
||||||
|
*/
|
||||||
|
public function __construct($appName,
|
||||||
|
IRequest $request,
|
||||||
|
IConfig $config,
|
||||||
|
IAppConfig $appConfig) {
|
||||||
|
parent::__construct($appName, $request);
|
||||||
|
$this->config = $config;
|
||||||
|
$this->appConfig = $appConfig;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @return DataResponse
|
||||||
|
*/
|
||||||
|
public function getApps() {
|
||||||
|
return new DataResponse([
|
||||||
|
'data' => $this->appConfig->getApps(),
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param string $app
|
||||||
|
* @return DataResponse
|
||||||
|
*/
|
||||||
|
public function getKeys($app) {
|
||||||
|
try {
|
||||||
|
$this->verifyAppId($app);
|
||||||
|
} catch (\InvalidArgumentException $e) {
|
||||||
|
return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN);
|
||||||
|
}
|
||||||
|
return new DataResponse([
|
||||||
|
'data' => $this->config->getAppKeys($app),
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param string $app
|
||||||
|
* @param string $key
|
||||||
|
* @param string $defaultValue
|
||||||
|
* @return DataResponse
|
||||||
|
*/
|
||||||
|
public function getValue($app, $key, $defaultValue = '') {
|
||||||
|
try {
|
||||||
|
$this->verifyAppId($app);
|
||||||
|
} catch (\InvalidArgumentException $e) {
|
||||||
|
return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN);
|
||||||
|
}
|
||||||
|
return new DataResponse([
|
||||||
|
'data' => $this->config->getAppValue($app, $key, $defaultValue),
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @PasswordConfirmationRequired
|
||||||
|
* @param string $app
|
||||||
|
* @param string $key
|
||||||
|
* @param string $value
|
||||||
|
* @return DataResponse
|
||||||
|
*/
|
||||||
|
public function setValue($app, $key, $value) {
|
||||||
|
try {
|
||||||
|
$this->verifyAppId($app);
|
||||||
|
$this->verifyConfigKey($app, $key);
|
||||||
|
} catch (\InvalidArgumentException $e) {
|
||||||
|
return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN);
|
||||||
|
}
|
||||||
|
|
||||||
|
$this->config->setAppValue($app, $key, $value);
|
||||||
|
return new DataResponse();
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @PasswordConfirmationRequired
|
||||||
|
* @param string $app
|
||||||
|
* @param string $key
|
||||||
|
* @return DataResponse
|
||||||
|
*/
|
||||||
|
public function deleteKey($app, $key) {
|
||||||
|
try {
|
||||||
|
$this->verifyAppId($app);
|
||||||
|
$this->verifyConfigKey($app, $key);
|
||||||
|
} catch (\InvalidArgumentException $e) {
|
||||||
|
return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN);
|
||||||
|
}
|
||||||
|
|
||||||
|
return new DataResponse([
|
||||||
|
'data' => $this->config->deleteAppValue($app, $key),
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param string $app
|
||||||
|
* @throws \InvalidArgumentException
|
||||||
|
*/
|
||||||
|
protected function verifyAppId($app) {
|
||||||
|
if (\OC_App::cleanAppId($app) !== $app) {
|
||||||
|
throw new \InvalidArgumentException('Invalid app id given');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param string $app
|
||||||
|
* @param string $key
|
||||||
|
* @throws \InvalidArgumentException
|
||||||
|
*/
|
||||||
|
protected function verifyConfigKey($app, $key) {
|
||||||
|
if (in_array($key, ['installed_version', 'enabled', 'types'])) {
|
||||||
|
throw new \InvalidArgumentException('The given key can not be set');
|
||||||
|
}
|
||||||
|
|
||||||
|
if ($app === 'core' && (strpos($key, 'public_') === 0 || strpos($key, 'remote_') === 0)) {
|
||||||
|
throw new \InvalidArgumentException('The given key can not be set');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
|
@ -1,70 +0,0 @@
|
||||||
<?php
|
|
||||||
/**
|
|
||||||
* @copyright Copyright (c) 2016, ownCloud, Inc.
|
|
||||||
*
|
|
||||||
* @author Lukas Reschke <lukas@statuscode.ch>
|
|
||||||
* @author Morris Jobke <hey@morrisjobke.de>
|
|
||||||
* @author Robin Appelman <robin@icewind.nl>
|
|
||||||
* @author Thomas Müller <thomas.mueller@tmit.eu>
|
|
||||||
*
|
|
||||||
* @license AGPL-3.0
|
|
||||||
*
|
|
||||||
* This code is free software: you can redistribute it and/or modify
|
|
||||||
* it under the terms of the GNU Affero General Public License, version 3,
|
|
||||||
* as published by the Free Software Foundation.
|
|
||||||
*
|
|
||||||
* This program is distributed in the hope that it will be useful,
|
|
||||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
||||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
||||||
* GNU Affero General Public License for more details.
|
|
||||||
*
|
|
||||||
* You should have received a copy of the GNU Affero General Public License, version 3,
|
|
||||||
* along with this program. If not, see <http://www.gnu.org/licenses/>
|
|
||||||
*
|
|
||||||
*/
|
|
||||||
|
|
||||||
OC_Util::checkAdminUser();
|
|
||||||
OCP\JSON::callCheck();
|
|
||||||
|
|
||||||
$action=isset($_POST['action'])?$_POST['action']:$_GET['action'];
|
|
||||||
|
|
||||||
if(isset($_POST['app']) || isset($_GET['app'])) {
|
|
||||||
$app=OC_App::cleanAppId(isset($_POST['app'])? (string)$_POST['app']: (string)$_GET['app']);
|
|
||||||
}
|
|
||||||
|
|
||||||
// An admin should not be able to add remote and public services
|
|
||||||
// on its own. This should only be possible programmatically.
|
|
||||||
// This change is due the fact that an admin may not be expected
|
|
||||||
// to execute arbitrary code in every environment.
|
|
||||||
if($app === 'core' && isset($_POST['key']) &&(substr((string)$_POST['key'],0,7) === 'remote_' || substr((string)$_POST['key'],0,7) === 'public_')) {
|
|
||||||
OC_JSON::error(array('data' => array('message' => 'Unexpected error!')));
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
$result=false;
|
|
||||||
$appConfig = \OC::$server->getAppConfig();
|
|
||||||
switch($action) {
|
|
||||||
case 'getValue':
|
|
||||||
$result=$appConfig->getValue($app, (string)$_GET['key'], (string)$_GET['defaultValue']);
|
|
||||||
break;
|
|
||||||
case 'setValue':
|
|
||||||
$result=$appConfig->setValue($app, (string)$_POST['key'], (string)$_POST['value']);
|
|
||||||
break;
|
|
||||||
case 'getApps':
|
|
||||||
$result=$appConfig->getApps();
|
|
||||||
break;
|
|
||||||
case 'getKeys':
|
|
||||||
$result=$appConfig->getKeys($app);
|
|
||||||
break;
|
|
||||||
case 'hasKey':
|
|
||||||
$result=$appConfig->hasKey($app, (string)$_GET['key']);
|
|
||||||
break;
|
|
||||||
case 'deleteKey':
|
|
||||||
$result=$appConfig->deleteKey($app, (string)$_POST['key']);
|
|
||||||
break;
|
|
||||||
case 'deleteApp':
|
|
||||||
$result=$appConfig->deleteApp($app);
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
OC_JSON::success(array('data'=>$result));
|
|
||||||
|
|
|
@ -1,58 +1,80 @@
|
||||||
/**
|
/**
|
||||||
* Copyright (c) 2011, Robin Appelman <icewind1991@gmail.com>
|
* @copyright Copyright (c) 2016 Joas Schilling <coding@schilljs.com>
|
||||||
* This file is licensed under the Affero General Public License version 3 or later.
|
*
|
||||||
* See the COPYING-README file.
|
* @license GNU AGPL version 3 or any later version
|
||||||
|
*
|
||||||
|
* This program is free software: you can redistribute it and/or modify
|
||||||
|
* it under the terms of the GNU Affero General Public License as
|
||||||
|
* published by the Free Software Foundation, either version 3 of the
|
||||||
|
* License, or (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This program is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
* GNU Affero General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU Affero General Public License
|
||||||
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
*
|
||||||
*/
|
*/
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @namespace
|
* @namespace
|
||||||
|
* @deprecated Use OCP.AppConfig instead
|
||||||
*/
|
*/
|
||||||
OC.AppConfig={
|
OC.AppConfig={
|
||||||
url:OC.filePath('core','ajax','appconfig.php'),
|
/**
|
||||||
getCall:function(action,data,callback){
|
* @deprecated Use OCP.AppConfig.getValue() instead
|
||||||
data.action=action;
|
*/
|
||||||
$.getJSON(OC.AppConfig.url,data,function(result){
|
getValue:function(app,key,defaultValue,callback){
|
||||||
if(result.status==='success'){
|
OCP.AppConfig.getValue(app, key, defaultValue, {
|
||||||
if(callback){
|
success: callback
|
||||||
callback(result.data);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
});
|
});
|
||||||
},
|
},
|
||||||
postCall:function(action,data,callback){
|
|
||||||
data.action=action;
|
/**
|
||||||
$.post(OC.AppConfig.url,data,function(result){
|
* @deprecated Use OCP.AppConfig.setValue() instead
|
||||||
if(result.status==='success'){
|
*/
|
||||||
if(callback){
|
|
||||||
callback(result.data);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
},'json');
|
|
||||||
},
|
|
||||||
getValue:function(app,key,defaultValue,callback){
|
|
||||||
if(typeof defaultValue=='function'){
|
|
||||||
callback=defaultValue;
|
|
||||||
defaultValue=null;
|
|
||||||
}
|
|
||||||
OC.AppConfig.getCall('getValue',{app:app,key:key,defaultValue:defaultValue},callback);
|
|
||||||
},
|
|
||||||
setValue:function(app,key,value){
|
setValue:function(app,key,value){
|
||||||
OC.AppConfig.postCall('setValue',{app:app,key:key,value:value});
|
OCP.AppConfig.setValue(app, key, value);
|
||||||
},
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @deprecated Use OCP.AppConfig.getApps() instead
|
||||||
|
*/
|
||||||
getApps:function(callback){
|
getApps:function(callback){
|
||||||
OC.AppConfig.getCall('getApps',{},callback);
|
OCP.AppConfig.getApps({
|
||||||
|
success: callback
|
||||||
|
});
|
||||||
},
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @deprecated Use OCP.AppConfig.getKeys() instead
|
||||||
|
*/
|
||||||
getKeys:function(app,callback){
|
getKeys:function(app,callback){
|
||||||
OC.AppConfig.getCall('getKeys',{app:app},callback);
|
OCP.AppConfig.getKeys(app, {
|
||||||
|
success: callback
|
||||||
|
});
|
||||||
},
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @deprecated
|
||||||
|
*/
|
||||||
hasKey:function(app,key,callback){
|
hasKey:function(app,key,callback){
|
||||||
OC.AppConfig.getCall('hasKey',{app:app,key:key},callback);
|
console.error('OC.AppConfig.hasKey is not supported anymore. Use OCP.AppConfig.getValue instead.');
|
||||||
},
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @deprecated Use OCP.AppConfig.deleteKey() instead
|
||||||
|
*/
|
||||||
deleteKey:function(app,key){
|
deleteKey:function(app,key){
|
||||||
OC.AppConfig.postCall('deleteKey',{app:app,key:key});
|
OCP.AppConfig.deleteKey(app, key);
|
||||||
},
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @deprecated
|
||||||
|
*/
|
||||||
deleteApp:function(app){
|
deleteApp:function(app){
|
||||||
OC.AppConfig.postCall('deleteApp',{app:app});
|
console.error('OC.AppConfig.deleteApp is not supported anymore.');
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
//TODO OC.Preferences
|
|
||||||
|
|
|
@ -39,6 +39,7 @@
|
||||||
"octemplate.js",
|
"octemplate.js",
|
||||||
"eventsource.js",
|
"eventsource.js",
|
||||||
"config.js",
|
"config.js",
|
||||||
|
"public/appconfig.js",
|
||||||
"multiselect.js",
|
"multiselect.js",
|
||||||
"oc-requesttoken.js",
|
"oc-requesttoken.js",
|
||||||
"setupchecks.js",
|
"setupchecks.js",
|
||||||
|
|
|
@ -58,7 +58,8 @@ function fileDownloadPath(dir, file) {
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @namespace */
|
/** @namespace */
|
||||||
var OC={
|
var OCP = {},
|
||||||
|
OC = {
|
||||||
PERMISSION_CREATE:4,
|
PERMISSION_CREATE:4,
|
||||||
PERMISSION_READ:1,
|
PERMISSION_READ:1,
|
||||||
PERMISSION_UPDATE:2,
|
PERMISSION_UPDATE:2,
|
||||||
|
|
|
@ -0,0 +1,113 @@
|
||||||
|
/**
|
||||||
|
* @copyright Copyright (c) 2016 Joas Schilling <coding@schilljs.com>
|
||||||
|
*
|
||||||
|
* @license GNU AGPL version 3 or any later version
|
||||||
|
*
|
||||||
|
* This program is free software: you can redistribute it and/or modify
|
||||||
|
* it under the terms of the GNU Affero General Public License as
|
||||||
|
* published by the Free Software Foundation, either version 3 of the
|
||||||
|
* License, or (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This program is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
* GNU Affero General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU Affero General Public License
|
||||||
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @namespace
|
||||||
|
* @since 9.2.0
|
||||||
|
*/
|
||||||
|
OCP.AppConfig = {
|
||||||
|
/**
|
||||||
|
* @param {string} method
|
||||||
|
* @param {string} endpoint
|
||||||
|
* @param {Object} [options]
|
||||||
|
* @param {Object} [options.data]
|
||||||
|
* @param {function} [options.success]
|
||||||
|
* @param {function} [options.error]
|
||||||
|
* @internal
|
||||||
|
*/
|
||||||
|
_call: function(method, endpoint, options) {
|
||||||
|
|
||||||
|
$.ajax({
|
||||||
|
type: method.toUpperCase(),
|
||||||
|
url: OC.linkToOCS('apps/provisioning_api/api/v1', 2) + 'config/apps' + endpoint,
|
||||||
|
data: options.data || {},
|
||||||
|
success: options.success,
|
||||||
|
error: options.error
|
||||||
|
})
|
||||||
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param {Object} [options]
|
||||||
|
* @param {function} [options.success]
|
||||||
|
* @since 9.2.0
|
||||||
|
*/
|
||||||
|
getApps: function(options) {
|
||||||
|
this._call('get', '', options);
|
||||||
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param {string} app
|
||||||
|
* @param {Object} [options]
|
||||||
|
* @param {function} [options.success]
|
||||||
|
* @param {function} [options.error]
|
||||||
|
* @since 9.2.0
|
||||||
|
*/
|
||||||
|
getKeys: function(app, options) {
|
||||||
|
this._call('get', '/' + app, options);
|
||||||
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param {string} app
|
||||||
|
* @param {string} key
|
||||||
|
* @param {string|function} defaultValue
|
||||||
|
* @param {Object} [options]
|
||||||
|
* @param {function} [options.success]
|
||||||
|
* @param {function} [options.error]
|
||||||
|
* @since 9.2.0
|
||||||
|
*/
|
||||||
|
getValue: function(app, key, defaultValue, options) {
|
||||||
|
options = options || {};
|
||||||
|
options['data'] = {
|
||||||
|
defaultValue: defaultValue
|
||||||
|
};
|
||||||
|
|
||||||
|
this._call('get', '/' + app + '/' + key, options);
|
||||||
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param {string} app
|
||||||
|
* @param {string} key
|
||||||
|
* @param {string} value
|
||||||
|
* @param {Object} [options]
|
||||||
|
* @param {function} [options.success]
|
||||||
|
* @param {function} [options.error]
|
||||||
|
* @since 9.2.0
|
||||||
|
*/
|
||||||
|
setValue: function(app, key, value, options) {
|
||||||
|
options = options || {};
|
||||||
|
options['data'] = {
|
||||||
|
value: value
|
||||||
|
};
|
||||||
|
|
||||||
|
this._call('post', '/' + app + '/' + key, options);
|
||||||
|
},
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param {string} app
|
||||||
|
* @param {string} key
|
||||||
|
* @param {Object} [options]
|
||||||
|
* @param {function} [options.success]
|
||||||
|
* @param {function} [options.error]
|
||||||
|
* @since 9.2.0
|
||||||
|
*/
|
||||||
|
deleteKey: function(app, key, options) {
|
||||||
|
this._call('delete', '/' + app + '/' + key, options);
|
||||||
|
}
|
||||||
|
};
|
|
@ -66,9 +66,6 @@ $application->registerRoutes($this, [
|
||||||
// Search
|
// Search
|
||||||
$this->create('search_ajax_search', '/core/search')
|
$this->create('search_ajax_search', '/core/search')
|
||||||
->actionInclude('core/search/ajax/search.php');
|
->actionInclude('core/search/ajax/search.php');
|
||||||
// AppConfig
|
|
||||||
$this->create('core_ajax_appconfig', '/core/ajax/appconfig.php')
|
|
||||||
->actionInclude('core/ajax/appconfig.php');
|
|
||||||
// oC JS config
|
// oC JS config
|
||||||
$this->create('js_config', '/core/js/oc.js')
|
$this->create('js_config', '/core/js/oc.js')
|
||||||
->actionInclude('core/js/config.php');
|
->actionInclude('core/js/config.php');
|
||||||
|
|
|
@ -40,6 +40,7 @@
|
||||||
"files",
|
"files",
|
||||||
"dav",
|
"dav",
|
||||||
"federatedfilesharing",
|
"federatedfilesharing",
|
||||||
|
"provisioning_api",
|
||||||
"twofactor_backupcodes",
|
"twofactor_backupcodes",
|
||||||
"workflowengine"
|
"workflowengine"
|
||||||
]
|
]
|
||||||
|
|
|
@ -135,6 +135,7 @@ class OC_Template extends \OC\Template\Base {
|
||||||
OC_Util::addScript("oc-requesttoken", null, true);
|
OC_Util::addScript("oc-requesttoken", null, true);
|
||||||
OC_Util::addScript('search', 'search', true);
|
OC_Util::addScript('search', 'search', true);
|
||||||
OC_Util::addScript("config", null, true);
|
OC_Util::addScript("config", null, true);
|
||||||
|
OC_Util::addScript("public/appconfig", null, true);
|
||||||
OC_Util::addScript("eventsource", null, true);
|
OC_Util::addScript("eventsource", null, true);
|
||||||
OC_Util::addScript("octemplate", null, true);
|
OC_Util::addScript("octemplate", null, true);
|
||||||
OC_Util::addTranslations("core", null, true);
|
OC_Util::addTranslations("core", null, true);
|
||||||
|
|
|
@ -114,31 +114,22 @@ $(document).ready(function(){
|
||||||
});
|
});
|
||||||
|
|
||||||
var savePublicShareDisclaimerText = _.debounce(function(value) {
|
var savePublicShareDisclaimerText = _.debounce(function(value) {
|
||||||
var data = {
|
var options = {
|
||||||
app:'core',
|
success: function() {
|
||||||
key:'shareapi_public_link_disclaimertext'
|
OC.msg.finishedSuccess('#publicShareDisclaimerStatus', t('core', 'Saved'));
|
||||||
};
|
},
|
||||||
if (_.isString(value) && value !== '') {
|
error: function() {
|
||||||
data['action'] = 'setValue';
|
OC.msg.finishedError('#publicShareDisclaimerStatus', t('core', 'Not saved'));
|
||||||
data['value'] = value;
|
|
||||||
} else {
|
|
||||||
data['action'] = 'deleteKey';
|
|
||||||
$('#publicShareDisclaimerText').val('');
|
|
||||||
}
|
}
|
||||||
|
};
|
||||||
|
|
||||||
OC.msg.startSaving('#publicShareDisclaimerStatus');
|
OC.msg.startSaving('#publicShareDisclaimerStatus');
|
||||||
$.post(
|
if (_.isString(value) && value !== '') {
|
||||||
OC.AppConfig.url,
|
OCP.AppConfig.setValue('core', 'shareapi_public_link_disclaimertext', value, options);
|
||||||
data,
|
|
||||||
function(result){
|
|
||||||
if(result.status === 'success'){
|
|
||||||
OC.msg.finishedSuccess('#publicShareDisclaimerStatus', t('core', 'Saved'))
|
|
||||||
} else {
|
} else {
|
||||||
OC.msg.finishedError('#publicShareDisclaimerStatus', t('core', 'Not saved'))
|
$('#publicShareDisclaimerText').val('');
|
||||||
|
OCP.AppConfig.deleteKey('core', 'shareapi_public_link_disclaimertext', options);
|
||||||
}
|
}
|
||||||
},
|
|
||||||
'json'
|
|
||||||
);
|
|
||||||
}, 500);
|
}, 500);
|
||||||
|
|
||||||
$('#publicShareDisclaimerText').on('change, keyup', function() {
|
$('#publicShareDisclaimerText').on('change, keyup', function() {
|
||||||
|
|
|
@ -306,7 +306,16 @@ class ManagerTest extends TestCase {
|
||||||
$this->appConfig->setValue('test1', 'enabled', 'yes');
|
$this->appConfig->setValue('test1', 'enabled', 'yes');
|
||||||
$this->appConfig->setValue('test2', 'enabled', 'no');
|
$this->appConfig->setValue('test2', 'enabled', 'no');
|
||||||
$this->appConfig->setValue('test3', 'enabled', '["foo"]');
|
$this->appConfig->setValue('test3', 'enabled', '["foo"]');
|
||||||
$apps = ['dav', 'federatedfilesharing', 'files', 'test1', 'test3', 'twofactor_backupcodes', 'workflowengine'];
|
$apps = [
|
||||||
|
'dav',
|
||||||
|
'federatedfilesharing',
|
||||||
|
'files',
|
||||||
|
'provisioning_api',
|
||||||
|
'test1',
|
||||||
|
'test3',
|
||||||
|
'twofactor_backupcodes',
|
||||||
|
'workflowengine',
|
||||||
|
];
|
||||||
$this->assertEquals($apps, $this->manager->getInstalledApps());
|
$this->assertEquals($apps, $this->manager->getInstalledApps());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -325,6 +334,7 @@ class ManagerTest extends TestCase {
|
||||||
'dav',
|
'dav',
|
||||||
'federatedfilesharing',
|
'federatedfilesharing',
|
||||||
'files',
|
'files',
|
||||||
|
'provisioning_api',
|
||||||
'test1',
|
'test1',
|
||||||
'test3',
|
'test3',
|
||||||
'twofactor_backupcodes',
|
'twofactor_backupcodes',
|
||||||
|
@ -343,6 +353,7 @@ class ManagerTest extends TestCase {
|
||||||
'dav' => ['id' => 'dav'],
|
'dav' => ['id' => 'dav'],
|
||||||
'files' => ['id' => 'files'],
|
'files' => ['id' => 'files'],
|
||||||
'federatedfilesharing' => ['id' => 'federatedfilesharing'],
|
'federatedfilesharing' => ['id' => 'federatedfilesharing'],
|
||||||
|
'provisioning_api' => ['id' => 'provisioning_api'],
|
||||||
'test1' => ['id' => 'test1', 'version' => '1.0.1', 'requiremax' => '9.0.0'],
|
'test1' => ['id' => 'test1', 'version' => '1.0.1', 'requiremax' => '9.0.0'],
|
||||||
'test2' => ['id' => 'test2', 'version' => '1.0.0', 'requiremin' => '8.2.0'],
|
'test2' => ['id' => 'test2', 'version' => '1.0.0', 'requiremin' => '8.2.0'],
|
||||||
'test3' => ['id' => 'test3', 'version' => '1.2.4', 'requiremin' => '9.0.0'],
|
'test3' => ['id' => 'test3', 'version' => '1.2.4', 'requiremin' => '9.0.0'],
|
||||||
|
@ -386,6 +397,7 @@ class ManagerTest extends TestCase {
|
||||||
'dav' => ['id' => 'dav'],
|
'dav' => ['id' => 'dav'],
|
||||||
'files' => ['id' => 'files'],
|
'files' => ['id' => 'files'],
|
||||||
'federatedfilesharing' => ['id' => 'federatedfilesharing'],
|
'federatedfilesharing' => ['id' => 'federatedfilesharing'],
|
||||||
|
'provisioning_api' => ['id' => 'provisioning_api'],
|
||||||
'test1' => ['id' => 'test1', 'version' => '1.0.1', 'requiremax' => '8.0.0'],
|
'test1' => ['id' => 'test1', 'version' => '1.0.1', 'requiremax' => '8.0.0'],
|
||||||
'test2' => ['id' => 'test2', 'version' => '1.0.0', 'requiremin' => '8.2.0'],
|
'test2' => ['id' => 'test2', 'version' => '1.0.0', 'requiremin' => '8.2.0'],
|
||||||
'test3' => ['id' => 'test3', 'version' => '1.2.4', 'requiremin' => '9.0.0'],
|
'test3' => ['id' => 'test3', 'version' => '1.2.4', 'requiremin' => '9.0.0'],
|
||||||
|
|
|
@ -317,6 +317,7 @@ class AppTest extends \Test\TestCase {
|
||||||
'appforgroup12',
|
'appforgroup12',
|
||||||
'dav',
|
'dav',
|
||||||
'federatedfilesharing',
|
'federatedfilesharing',
|
||||||
|
'provisioning_api',
|
||||||
'twofactor_backupcodes',
|
'twofactor_backupcodes',
|
||||||
'workflowengine',
|
'workflowengine',
|
||||||
),
|
),
|
||||||
|
@ -333,6 +334,7 @@ class AppTest extends \Test\TestCase {
|
||||||
'appforgroup2',
|
'appforgroup2',
|
||||||
'dav',
|
'dav',
|
||||||
'federatedfilesharing',
|
'federatedfilesharing',
|
||||||
|
'provisioning_api',
|
||||||
'twofactor_backupcodes',
|
'twofactor_backupcodes',
|
||||||
'workflowengine',
|
'workflowengine',
|
||||||
),
|
),
|
||||||
|
@ -350,6 +352,7 @@ class AppTest extends \Test\TestCase {
|
||||||
'appforgroup2',
|
'appforgroup2',
|
||||||
'dav',
|
'dav',
|
||||||
'federatedfilesharing',
|
'federatedfilesharing',
|
||||||
|
'provisioning_api',
|
||||||
'twofactor_backupcodes',
|
'twofactor_backupcodes',
|
||||||
'workflowengine',
|
'workflowengine',
|
||||||
),
|
),
|
||||||
|
@ -367,6 +370,7 @@ class AppTest extends \Test\TestCase {
|
||||||
'appforgroup2',
|
'appforgroup2',
|
||||||
'dav',
|
'dav',
|
||||||
'federatedfilesharing',
|
'federatedfilesharing',
|
||||||
|
'provisioning_api',
|
||||||
'twofactor_backupcodes',
|
'twofactor_backupcodes',
|
||||||
'workflowengine',
|
'workflowengine',
|
||||||
),
|
),
|
||||||
|
@ -384,6 +388,7 @@ class AppTest extends \Test\TestCase {
|
||||||
'appforgroup2',
|
'appforgroup2',
|
||||||
'dav',
|
'dav',
|
||||||
'federatedfilesharing',
|
'federatedfilesharing',
|
||||||
|
'provisioning_api',
|
||||||
'twofactor_backupcodes',
|
'twofactor_backupcodes',
|
||||||
'workflowengine',
|
'workflowengine',
|
||||||
),
|
),
|
||||||
|
@ -463,11 +468,11 @@ class AppTest extends \Test\TestCase {
|
||||||
);
|
);
|
||||||
|
|
||||||
$apps = \OC_App::getEnabledApps();
|
$apps = \OC_App::getEnabledApps();
|
||||||
$this->assertEquals(array('files', 'app3', 'dav', 'federatedfilesharing', 'twofactor_backupcodes', 'workflowengine'), $apps);
|
$this->assertEquals(array('files', 'app3', 'dav', 'federatedfilesharing', 'provisioning_api', 'twofactor_backupcodes', 'workflowengine'), $apps);
|
||||||
|
|
||||||
// mock should not be called again here
|
// mock should not be called again here
|
||||||
$apps = \OC_App::getEnabledApps();
|
$apps = \OC_App::getEnabledApps();
|
||||||
$this->assertEquals(array('files', 'app3', 'dav', 'federatedfilesharing', 'twofactor_backupcodes', 'workflowengine'), $apps);
|
$this->assertEquals(array('files', 'app3', 'dav', 'federatedfilesharing', 'provisioning_api', 'twofactor_backupcodes', 'workflowengine'), $apps);
|
||||||
|
|
||||||
$this->restoreAppConfig();
|
$this->restoreAppConfig();
|
||||||
\OC_User::setUserId(null);
|
\OC_User::setUserId(null);
|
||||||
|
|
Loading…
Reference in New Issue