From 793de6a8d0906ec2f01cc3ee293493e5ae1a0d7a Mon Sep 17 00:00:00 2001
From: Arthur Schiwon <blizzz@arthur-schiwon.de>
Date: Tue, 27 Jun 2017 12:21:18 +0200
Subject: move encryption to security (and behind two factor things)

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
---
 apps/encryption/appinfo/info.xml                   |   1 -
 apps/encryption/lib/Settings/Personal.php          |   4 +-
 apps/encryption/lib/Settings/PersonalSection.php   | 119 ---------------------
 .../lib/Settings/Personal.php                      |   2 +-
 4 files changed, 3 insertions(+), 123 deletions(-)
 delete mode 100644 apps/encryption/lib/Settings/PersonalSection.php

diff --git a/apps/encryption/appinfo/info.xml b/apps/encryption/appinfo/info.xml
index 01e42ae370b..7cfdc934386 100644
--- a/apps/encryption/appinfo/info.xml
+++ b/apps/encryption/appinfo/info.xml
@@ -30,7 +30,6 @@
 	<settings>
 		<admin>OCA\Encryption\Settings\Admin</admin>
 		<personal>OCA\Encryption\Settings\Personal</personal>
-		<personal-section>OCA\Encryption\Settings\PersonalSection</personal-section>
 	</settings>
 	<commands>
 		<command>OCA\Encryption\Command\EnableMasterKey</command>
diff --git a/apps/encryption/lib/Settings/Personal.php b/apps/encryption/lib/Settings/Personal.php
index 3d399e4fe13..5b01c224538 100644
--- a/apps/encryption/lib/Settings/Personal.php
+++ b/apps/encryption/lib/Settings/Personal.php
@@ -78,7 +78,7 @@ class Personal implements  ISettings {
 	 * @since 9.1
 	 */
 	public function getSection() {
-		return 'encryption';
+		return 'security';
 	}
 
 	/**
@@ -90,6 +90,6 @@ class Personal implements  ISettings {
 	 * @since 9.1
 	 */
 	public function getPriority() {
-		return 10;
+		return 80;
 	}
 }
diff --git a/apps/encryption/lib/Settings/PersonalSection.php b/apps/encryption/lib/Settings/PersonalSection.php
deleted file mode 100644
index 1af4229bcd2..00000000000
--- a/apps/encryption/lib/Settings/PersonalSection.php
+++ /dev/null
@@ -1,119 +0,0 @@
-<?php
-/**
- * @copyright Copyright (c) 2017 Arthur Schiwon <blizzz@arthur-schiwon.de>
- *
- * @author Arthur Schiwon <blizzz@arthur-schiwon.de>
- *
- * @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\Encryption\Settings;
-
-
-use OCA\Encryption\AppInfo\Application;
-use OCA\Encryption\Session;
-use OCP\IConfig;
-use OCP\IL10N;
-use OCP\IURLGenerator;
-use OCP\IUserSession;
-use OCP\Settings\IIconSection;
-
-class PersonalSection implements IIconSection {
-
-	/** @var IURLGenerator */
-	private $urlGenerator;
-	/** @var IL10N */
-	private $l;
-	/** @var Application */
-	private $app;
-	/** @var IConfig */
-	private $config;
-	/** @var Session */
-	private $session;
-	/** @var IUserSession */
-	private $userSession;
-
-	public function __construct(
-		IURLGenerator $urlGenerator,
-		IL10N $l,
-		Application $app,
-		IConfig $config,
-		Session $session,
-		IUserSession $userSession
-	) {
-		$this->urlGenerator = $urlGenerator;
-		$this->l = $l;
-		$this->app = $app;
-		$this->config = $config;
-		$this->session = $session;
-		$this->userSession = $userSession;
-	}
-
-	/**
-	 * returns the relative path to an 16*16 icon describing the section.
-	 * e.g. '/core/img/places/files.svg'
-	 *
-	 * @returns string
-	 * @since 13.0.0
-	 */
-	public function getIcon() {
-		return $this->urlGenerator->imagePath('settings', 'password.svg');
-	}
-
-	/**
-	 * returns the ID of the section. It is supposed to be a lower case string,
-	 * e.g. 'ldap'
-	 *
-	 * @returns string
-	 * @since 9.1
-	 */
-	public function getID() {
-		// we need to return the proper id while installing/upgrading the app
-		$loggedIn = $this->userSession->isLoggedIn();
-
-		$recoveryAdminEnabled = $this->config->getAppValue('encryption', 'recoveryAdminEnabled');
-		$privateKeySet = $this->session->isPrivateKeySet();
-
-		if ($loggedIn && !$recoveryAdminEnabled && $privateKeySet) {
-			return null;
-		}
-		return 'encryption';
-	}
-
-	/**
-	 * returns the translated name as it should be displayed, e.g. 'LDAP / AD
-	 * integration'. Use the L10N service to translate it.
-	 *
-	 * @return string
-	 * @since 9.1
-	 */
-	public function getName() {
-		return $this->l->t('Encryption');
-	}
-
-	/**
-	 * @return int whether the form should be rather on the top or bottom of
-	 * the settings navigation. The sections are arranged in ascending order of
-	 * the priority values. It is required to return a value between 0 and 99.
-	 *
-	 * E.g.: 70
-	 * @since 9.1
-	 */
-	public function getPriority() {
-		return 10;
-	}
-}
diff --git a/apps/twofactor_backupcodes/lib/Settings/Personal.php b/apps/twofactor_backupcodes/lib/Settings/Personal.php
index 56eead16390..eb28dacb42b 100644
--- a/apps/twofactor_backupcodes/lib/Settings/Personal.php
+++ b/apps/twofactor_backupcodes/lib/Settings/Personal.php
@@ -77,6 +77,6 @@ class Personal implements ISettings {
 	 * @since 9.1
 	 */
 	public function getPriority() {
-		return 85;
+		return 40;
 	}
 }
-- 
cgit v1.2.3