Merge pull request #13 from aHenryJard/790_manage_user

By default managementLink are for administer
aHenryJard · Oct 31, 2019 · 2d232b6 · 2d232b6
2 parents daf7d6b + bb2deeb
commit 2d232b6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/core/src/main/java/hudson/diagnosis/OldDataMonitor.java b/core/src/main/java/hudson/diagnosis/OldDataMonitor.java
@@ -39,6 +39,7 @@
 import hudson.model.listeners.SaveableListener;
 import hudson.security.ACL;
 import hudson.security.ACLContext;
+import hudson.security.Permission;
 import hudson.util.RobustReflectionConverter;
 import hudson.util.VersionNumber;
 import java.io.IOException;
@@ -460,5 +461,10 @@ public String getDescription() {
         public String getDisplayName() {
             return Messages.OldDataMonitor_DisplayName();
         }
+
+        @Override
+        public Permission getRequiredPermission() {
+            return Jenkins.CONFIGURE_JENKINS;
+        }
     }
 }
diff --git a/core/src/main/java/hudson/model/ManagementLink.java b/core/src/main/java/hudson/model/ManagementLink.java
@@ -115,7 +115,7 @@ public boolean getRequiresConfirmation() {
      * @return permission required for user to access this management link, in addition to {@link Jenkins#ADMINISTER}
      */
     public @CheckForNull Permission getRequiredPermission() {
-        return Jenkins.CONFIGURE_JENKINS;
+        return Jenkins.ADMINISTER;
     }
 
     /**