diff options
author | Roeland Jago Douma <roeland@famdouma.nl> | 2018-03-22 13:20:03 +0100 |
---|---|---|
committer | Roeland Jago Douma <roeland@famdouma.nl> | 2018-03-23 08:50:17 +0100 |
commit | 26073eac0c76cb3c6f6ee4a8688e0e1bada820ff (patch) | |
tree | 829faad3a7e090b0e146151254c5fcdf8e37d6e5 | |
parent | 3dac15a8911fea1e8473af7e7a26d7a57e68328b (diff) | |
download | nextcloud-server-26073eac0c76cb3c6f6ee4a8688e0e1bada820ff.tar.gz nextcloud-server-26073eac0c76cb3c6f6ee4a8688e0e1bada820ff.zip |
Finally remove \OCP\JSON
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
-rw-r--r-- | apps/files_external/ajax/applicable.php | 2 | ||||
-rw-r--r-- | apps/files_versions/ajax/getVersions.php | 4 | ||||
-rw-r--r-- | apps/user_ldap/ajax/wizard.php | 18 | ||||
-rw-r--r-- | lib/public/JSON.php | 166 |
4 files changed, 12 insertions, 178 deletions
diff --git a/apps/files_external/ajax/applicable.php b/apps/files_external/ajax/applicable.php index eb0f754870d..a946962a293 100644 --- a/apps/files_external/ajax/applicable.php +++ b/apps/files_external/ajax/applicable.php @@ -51,4 +51,4 @@ foreach (\OC::$server->getUserManager()->searchDisplayName($pattern, $limit, $of $results = array('groups' => $groups, 'users' => $users); -\\OC_JSON::success($results); +\OC_JSON::success($results); diff --git a/apps/files_versions/ajax/getVersions.php b/apps/files_versions/ajax/getVersions.php index 6bba0a540c6..5811242c236 100644 --- a/apps/files_versions/ajax/getVersions.php +++ b/apps/files_versions/ajax/getVersions.php @@ -47,10 +47,10 @@ if( $versions ) { unset($version['path']); } - \\OC_JSON::success(array('data' => array('versions' => $versions, 'endReached' => $endReached))); + \OC_JSON::success(array('data' => array('versions' => $versions, 'endReached' => $endReached))); } else { - \\OC_JSON::success(array('data' => array('versions' => [], 'endReached' => true))); + \OC_JSON::success(array('data' => array('versions' => [], 'endReached' => true))); } diff --git a/apps/user_ldap/ajax/wizard.php b/apps/user_ldap/ajax/wizard.php index f0ce8f87c39..df178ac0a82 100644 --- a/apps/user_ldap/ajax/wizard.php +++ b/apps/user_ldap/ajax/wizard.php @@ -36,13 +36,13 @@ $l = \OC::$server->getL10N('user_ldap'); if(!isset($_POST['action'])) { - \\OC_JSON::error(array('message' => $l->t('No action specified'))); + \OC_JSON::error(array('message' => $l->t('No action specified'))); } $action = (string)$_POST['action']; if(!isset($_POST['ldap_serverconfig_chooser'])) { - \\OC_JSON::error(array('message' => $l->t('No configuration specified'))); + \OC_JSON::error(array('message' => $l->t('No configuration specified'))); } $prefix = (string)$_POST['ldap_serverconfig_chooser']; @@ -98,10 +98,10 @@ switch($action) { exit; } } catch (\Exception $e) { - \\OC_JSON::error(array('message' => $e->getMessage(), 'code' => $e->getCode())); + \OC_JSON::error(array('message' => $e->getMessage(), 'code' => $e->getCode())); exit; } - \\OC_JSON::error(); + \OC_JSON::error(); exit; break; @@ -114,10 +114,10 @@ switch($action) { exit; } } catch (\Exception $e) { - \\OC_JSON::error(array('message' => $e->getMessage())); + \OC_JSON::error(array('message' => $e->getMessage())); exit; } - \\OC_JSON::error(); + \OC_JSON::error(); exit; break; } @@ -126,14 +126,14 @@ switch($action) { $key = isset($_POST['cfgkey']) ? $_POST['cfgkey'] : false; $val = isset($_POST['cfgval']) ? $_POST['cfgval'] : null; if($key === false || is_null($val)) { - \\OC_JSON::error(array('message' => $l->t('No data specified'))); + \OC_JSON::error(array('message' => $l->t('No data specified'))); exit; } $cfg = array($key => $val); $setParameters = array(); $configuration->setConfiguration($cfg, $setParameters); if(!in_array($key, $setParameters)) { - \\OC_JSON::error(array('message' => $l->t($key. + \OC_JSON::error(array('message' => $l->t($key. ' Could not set configuration %s', $setParameters[0]))); exit; } @@ -144,6 +144,6 @@ switch($action) { \OC_JSON::success(); break; default: - \\OC_JSON::error(array('message' => $l->t('Action does not exist'))); + \OC_JSON::error(array('message' => $l->t('Action does not exist'))); break; } diff --git a/lib/public/JSON.php b/lib/public/JSON.php deleted file mode 100644 index 6138aec1f49..00000000000 --- a/lib/public/JSON.php +++ /dev/null @@ -1,166 +0,0 @@ -<?php -/** - * @copyright Copyright (c) 2016, ownCloud, Inc. - * - * @author Bart Visscher <bartv@thisnet.nl> - * @author Frank Karlitschek <frank@karlitschek.de> - * @author Lukas Reschke <lukas@statuscode.ch> - * @author Morris Jobke <hey@morrisjobke.de> - * @author Roeland Jago Douma <roeland@famdouma.nl> - * @author Thomas Müller <thomas.mueller@tmit.eu> - * @author Thomas Tanghus <thomas@tanghus.net> - * @author Vincent Petry <pvince81@owncloud.com> - * - * @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/> - * - */ - -/** - * Public interface of ownCloud for apps to use. - * JSON Class - */ - -// use OCP namespace for all classes that are considered public. -// This means that they should be used by apps instead of the internal ownCloud classes -namespace OCP; - -/** - * This class provides convenient functions to generate and send JSON data. Useful for Ajax calls - * @deprecated 8.1.0 Use a AppFramework JSONResponse instead - */ -class JSON { - /** - * Check if the user is logged in, send json error msg if not. - * - * This method checks if a user is logged in. If not, a json error - * response will be return and the method will exit from execution - * of the script. - * The returned json will be in the format: - * - * {"status":"error","data":{"message":"Authentication error."}} - * - * Add this call to the start of all ajax method files that requires - * an authenticated user. - * @deprecated 8.1.0 Use annotation based ACLs from the AppFramework instead - * - * @suppress PhanDeprecatedFunction - */ - public static function checkLoggedIn() { - \OC_JSON::checkLoggedIn(); - } - - /** - * Check an ajax get/post call if the request token is valid. - * - * This method checks for a valid variable 'requesttoken' in $_GET, - * $_POST and $_SERVER. If a valid token is not found, a json error - * response will be return and the method will exit from execution - * of the script. - * The returned json will be in the format: - * - * {"status":"error","data":{"message":"Token expired. Please reload page."}} - * - * Add this call to the start of all ajax method files that creates, - * updates or deletes anything. - * In cases where you e.g. use an ajax call to load a dialog containing - * a submittable form, you will need to add the requesttoken first as a - * parameter to the ajax call, then assign it to the template and finally - * add a hidden input field also named 'requesttoken' containing the value. - * @deprecated 8.1.0 Use annotation based CSRF checks from the AppFramework instead - * - * @suppress PhanDeprecatedFunction - */ - public static function callCheck() { - \OC_JSON::callCheck(); - } - - /** - * Send json success msg - * - * Return a json success message with optional extra data. - * @see \OCP\JSON::error() for the format to use. - * - * @param array $data The data to use - * @deprecated 8.1.0 Use a AppFramework JSONResponse instead - * @suppress PhanDeprecatedFunction - */ - public static function success( $data = array() ) { - \OC_JSON::success($data); - } - - /** - * Send json error msg - * - * Return a json error message with optional extra data for - * error message or app specific data. - * - * Example use: - * - * $id = [some value] - * OCP\JSON::error(array('data':array('message':'An error happened', 'id': $id))); - * - * Will return the json formatted string: - * - * {"status":"error","data":{"message":"An error happened", "id":[some value]}} - * - * @param array $data The data to use - * @deprecated 8.1.0 Use a AppFramework JSONResponse instead - * @suppress PhanDeprecatedFunction - */ - public static function error( $data = array() ) { - \OC_JSON::error($data); - } - - /** - * Check if the App is enabled and send JSON error message instead - * - * This method checks if a specific app is enabled. If not, a json error - * response will be return and the method will exit from execution - * of the script. - * The returned json will be in the format: - * - * {"status":"error","data":{"message":"Application is not enabled."}} - * - * Add this call to the start of all ajax method files that requires - * a specific app to be enabled. - * - * @param string $app The app to check - * @deprecated 8.1.0 Use the AppFramework instead. It will automatically check if the app is enabled. - * @suppress PhanDeprecatedFunction - */ - public static function checkAppEnabled( $app ) { - \OC_JSON::checkAppEnabled($app); - } - - /** - * Check if the user is a admin, send json error msg if not - * - * This method checks if the current user has admin rights. If not, a json error - * response will be return and the method will exit from execution - * of the script. - * The returned json will be in the format: - * - * {"status":"error","data":{"message":"Authentication error."}} - * - * Add this call to the start of all ajax method files that requires - * administrative rights. - * - * @deprecated 8.1.0 Use annotation based ACLs from the AppFramework instead - * @suppress PhanDeprecatedFunction - */ - public static function checkAdminUser() { - \OC_JSON::checkAdminUser(); - } -} |