From 605d0874a4e38772b005beef4a945c3d9610e2bb Mon Sep 17 00:00:00 2001
From: Morris Jobke <hey@morrisjobke.de>
Date: Thu, 18 Jul 2019 11:27:09 +0200
Subject: [PATCH 1/2] Removes unused OC_API::register

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
---
 lib/private/legacy/api.php | 30 ------------------------------
 lib/public/API.php         | 28 ----------------------------
 2 files changed, 58 deletions(-)

diff --git a/lib/private/legacy/api.php b/lib/private/legacy/api.php
index 40bf6132e2..dc8c818574 100644
--- a/lib/private/legacy/api.php
+++ b/lib/private/legacy/api.php
@@ -42,36 +42,6 @@ class OC_API {
 	 */
 	protected static $actions = array();
 
-	/**
-	 * registers an api call
-	 * @param string $method the http method
-	 * @param string $url the url to match
-	 * @param callable $action the function to run
-	 * @param string $app the id of the app registering the call
-	 * @param int $authLevel the level of authentication required for the call
-	 * @param array $defaults
-	 * @param array $requirements
-	 */
-	public static function register($method, $url, $action, $app,
-				$authLevel = API::USER_AUTH,
-				$defaults = array(),
-				$requirements = array()) {
-		$name = strtolower($method).$url;
-		$name = str_replace(array('/', '{', '}'), '_', $name);
-		if(!isset(self::$actions[$name])) {
-			$oldCollection = OC::$server->getRouter()->getCurrentCollection();
-			OC::$server->getRouter()->useCollection('ocs');
-			OC::$server->getRouter()->create($name, $url)
-				->method($method)
-				->defaults($defaults)
-				->requirements($requirements)
-				->action('OC_API', 'call');
-			self::$actions[$name] = array();
-			OC::$server->getRouter()->useCollection($oldCollection);
-		}
-		self::$actions[$name][] = array('app' => $app, 'action' => $action, 'authlevel' => $authLevel);
-	}
-
 	/**
 	 * respond to a call
 	 * @param \OC\OCS\Result $result
diff --git a/lib/public/API.php b/lib/public/API.php
index 9a594f7688..d3c528ee4f 100644
--- a/lib/public/API.php
+++ b/lib/public/API.php
@@ -40,16 +40,6 @@ namespace OCP;
  * @deprecated 9.1.0 Use the AppFramework
  */
 class API {
-
-	/**
-	 * API authentication levels
-	 * @since 8.1.0
-	 */
-	const GUEST_AUTH = 0;
-	const USER_AUTH = 1;
-	const SUBADMIN_AUTH = 2;
-	const ADMIN_AUTH = 3;
-
 	/**
 	 * API Response Codes
 	 * @since 8.1.0
@@ -58,22 +48,4 @@ class API {
 	const RESPOND_SERVER_ERROR = 996;
 	const RESPOND_NOT_FOUND = 998;
 	const RESPOND_UNKNOWN_ERROR = 999;
-
-	/**
-	 * registers an api call
-	 * @param string $method the http method
-	 * @param string $url the url to match
-	 * @param callable $action the function to run
-	 * @param string $app the id of the app registering the call
-	 * @param int $authLevel the level of authentication required for the call (See `self::*_AUTH` constants)
-	 * @param array $defaults
-	 * @param array $requirements
-	 * @since 5.0.0
-	 * @deprecated 9.1.0 Use the AppFramework
-	 */
-	public static function register($method, $url, $action, $app, $authLevel = self::USER_AUTH,
-		$defaults = array(), $requirements = array()){
-		\OC_API::register($method, $url, $action, $app, $authLevel, $defaults, $requirements);
-	}
-
 }

From 54bcd86db7d35acbec2fc4a6f87cfdbceb59e539 Mon Sep 17 00:00:00 2001
From: Morris Jobke <hey@morrisjobke.de>
Date: Thu, 18 Jul 2019 12:03:51 +0200
Subject: [PATCH 2/2] Adjust deprecation tests

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
---
 lib/private/App/CodeChecker/DeprecationCheck.php | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/lib/private/App/CodeChecker/DeprecationCheck.php b/lib/private/App/CodeChecker/DeprecationCheck.php
index 518ab1ec38..e672038f34 100644
--- a/lib/private/App/CodeChecker/DeprecationCheck.php
+++ b/lib/private/App/CodeChecker/DeprecationCheck.php
@@ -38,6 +38,7 @@ class DeprecationCheck extends AbstractCheck {
 		return [
 			'OC_JSON' => '8.2.0',
 
+			'OCP\API' => '9.1.0',
 			'OCP\Contacts' => '8.1.0',
 			'OCP\DB' => '8.1.0',
 			'OCP\JSON' => '8.1.0',
@@ -55,6 +56,15 @@ class DeprecationCheck extends AbstractCheck {
 	 */
 	protected function getLocalConstants() {
 		return [
+			'OCP\API::GUEST_AUTH' => '9.1.0',
+			'OCP\API::USER_AUTH' => '9.1.0',
+			'OCP\API::SUBADMIN_AUTH' => '9.1.0',
+			'OCP\API::ADMIN_AUTH' => '9.1.0',
+			'OCP\API::RESPOND_UNAUTHORISED' => '9.1.0',
+			'OCP\API::RESPOND_SERVER_ERROR' => '9.1.0',
+			'OCP\API::RESPOND_NOT_FOUND' => '9.1.0',
+			'OCP\API::RESPOND_UNKNOWN_ERROR' => '9.1.0',
+
 			'OC_API::GUEST_AUTH' => '8.2.0',
 			'OC_API::USER_AUTH' => '8.2.0',
 			'OC_API::SUBADMIN_AUTH' => '8.2.0',