From e37df7092bf0a66576fd7b134b5ecbddb616048c Mon Sep 17 00:00:00 2001
From: =?utf8?q?Tim=20D=C3=BCsterhus?= <duesterhus@woltlab.com>
Date: Tue, 23 Feb 2021 14:49:50 +0100
Subject: [PATCH] Add permission to access the GDPR export

Resolves #3993
---
 com.woltlab.wcf/userGroupOption.xml                   | 11 ++++++++++-
 wcfsetup/install/files/acp/templates/userList.tpl     |  2 +-
 .../lib/acp/action/UserExportGdprAction.class.php     |  2 +-
 wcfsetup/install/lang/de.xml                          |  2 ++
 wcfsetup/install/lang/en.xml                          |  2 ++
 5 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/com.woltlab.wcf/userGroupOption.xml b/com.woltlab.wcf/userGroupOption.xml
index 79a68163a5..eeb6a39bfe 100644
--- a/com.woltlab.wcf/userGroupOption.xml
+++ b/com.woltlab.wcf/userGroupOption.xml
@@ -1055,9 +1055,18 @@ webp</defaultvalue>
 				<defaultvalue>1</defaultvalue>
 				<options>module_tagging</options>
 			</option>
+			<option name="admin.user.canExportGdprData">
+				<categoryname>admin.user.user</categoryname>
+				<optiontype>boolean</optiontype>
+				<defaultvalue>0</defaultvalue>
+				<admindefaultvalue>1</admindefaultvalue>
+				<moddefaultvalue>0</moddefaultvalue>
+				<userdefaultvalue>0</userdefaultvalue>
+				<usersonly>1</usersonly>
+			</option>
 		</options>
 	</import>
 	<delete>
-		<option name="user.profile.canMail" />
+		<option name="user.profile.canMail"/>
 	</delete>
 </data>
diff --git a/wcfsetup/install/files/acp/templates/userList.tpl b/wcfsetup/install/files/acp/templates/userList.tpl
index 13b61cd26d..1f92e0ef3b 100644
--- a/wcfsetup/install/files/acp/templates/userList.tpl
+++ b/wcfsetup/install/files/acp/templates/userList.tpl
@@ -132,7 +132,7 @@
 										{/if}
 									{/if}
 									
-									{if $user->editable}
+									{if $user->accessible && $__wcf->session->getPermission('admin.user.canExportGdprData')}
 										<li><a href="{link controller='UserExportGdpr' id=$user->userID}{/link}">{lang}wcf.acp.user.exportGdpr{/lang}</a></li>
 									{/if}
 									
diff --git a/wcfsetup/install/files/lib/acp/action/UserExportGdprAction.class.php b/wcfsetup/install/files/lib/acp/action/UserExportGdprAction.class.php
index 31d08b84e0..3a300ebc91 100644
--- a/wcfsetup/install/files/lib/acp/action/UserExportGdprAction.class.php
+++ b/wcfsetup/install/files/lib/acp/action/UserExportGdprAction.class.php
@@ -100,7 +100,7 @@ class UserExportGdprAction extends AbstractAction
     /**
      * @inheritDoc
      */
-    public $neededPermissions = ['admin.user.canEditUser'];
+    public $neededPermissions = ['admin.user.canExportGdprData'];
 
     /**
      * list of user option names that are excluded from the output, any option that begins with
diff --git a/wcfsetup/install/lang/de.xml b/wcfsetup/install/lang/de.xml
index 29d5e64681..7810ce023a 100644
--- a/wcfsetup/install/lang/de.xml
+++ b/wcfsetup/install/lang/de.xml
@@ -921,6 +921,8 @@ ACHTUNG: Die oben genannten Meldungen sind stark gekürzt. Sie können Details z
 		<item name="wcf.acp.group.option.user.signature.attachment.maxCount"><![CDATA[Maximale Dateianhänge]]></item>
 		<item name="wcf.acp.group.requireMultifactor"><![CDATA[Mehrfaktor-Authentifizierung erzwingen]]></item>
 		<item name="wcf.acp.group.requireMultifactor.description"><![CDATA[Benutzer, die Mitglied dieser Benutzergruppe sind, können die Mehrfaktor-Authentifizierung nicht deaktivieren und können besonders geschützte Bereiche erst betreten, wenn sie die Mehrfaktor-Authentifizierung eingerichtet haben.]]></item>
+		<item name="wcf.acp.group.option.admin.user.canExportGdprData"><![CDATA[Kann persönliche Daten (DSGVO) exportieren]]></item>
+		<item name="wcf.acp.group.option.admin.user.canExportGdprData.description"><![CDATA[Der Export gemäß der europäischen Datenschutzgrundverordnung (DSGVO) enthält detaillierte persönliche Daten des jeweiligen Benutzers. Eine Filterung der im Export enthaltenen Information auf Basis der Zugriffsrechte des Administrators findet <strong>nicht</strong> statt und kann unter Umständen Informationen enthalten, die einem Administrator auf andere Weise nicht zugänglich wären.]]></item>
 	</category>
 	<category name="wcf.acp.index">
 		<item name="wcf.acp.index.credits"><![CDATA[Über WoltLab Suite&trade;]]></item>
diff --git a/wcfsetup/install/lang/en.xml b/wcfsetup/install/lang/en.xml
index 5f5e01a748..1263759d8c 100644
--- a/wcfsetup/install/lang/en.xml
+++ b/wcfsetup/install/lang/en.xml
@@ -898,6 +898,8 @@ ATTENTION: The messages listed above are greatly shortened. You can view details
 		<item name="wcf.acp.group.option.user.signature.attachment.maxCount"><![CDATA[Maximum Attachments]]></item>
 		<item name="wcf.acp.group.requireMultifactor"><![CDATA[Require Multi-factor Authentication]]></item>
 		<item name="wcf.acp.group.requireMultifactor.description"><![CDATA[Users that are members of this user group may not disable multi-factor authentication. They will also be required to set up multi-factor authentication before they may enter protected areas.]]></item>
+		<item name="wcf.acp.group.option.admin.user.canExportGdprData"><![CDATA[Can export personal data (GDPR)]]></item>
+		<item name="wcf.acp.group.option.admin.user.canExportGdprData.description"><![CDATA[The export according to the European General Data Protection Regulation (GDPR) contains detailed personal data of the respective user. The export <strong>will not</strong> be filtered according to the administrator’s permissions and may contain information that would be inaccessible otherwise.]]></item>
 	</category>
 	<category name="wcf.acp.index">
 		<item name="wcf.acp.index.credits"><![CDATA[About WoltLab Suite&trade;]]></item>
-- 
2.20.1