You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by an...@apache.org on 2022/11/24 09:49:18 UTC

[jackrabbit-oak] branch trunk updated: OAK-10012 : Redundant modifier in oak-jackrabbit-api

This is an automated email from the ASF dual-hosted git repository.

angela pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


The following commit(s) were added to refs/heads/trunk by this push:
     new cda4512385 OAK-10012 : Redundant modifier in oak-jackrabbit-api
cda4512385 is described below

commit cda45123851f2cd18555e5aba263f048724bc158
Author: angela <an...@adobe.com>
AuthorDate: Thu Nov 24 10:49:05 2022 +0100

    OAK-10012 : Redundant modifier in oak-jackrabbit-api
---
 .../main/java/org/apache/jackrabbit/api/JackrabbitRepository.java   | 6 +++---
 .../jackrabbit/api/security/JackrabbitAccessControlManager.java     | 5 ++---
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/JackrabbitRepository.java b/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/JackrabbitRepository.java
index 9a84715789..b636187ecf 100644
--- a/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/JackrabbitRepository.java
+++ b/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/JackrabbitRepository.java
@@ -35,19 +35,19 @@ public interface JackrabbitRepository extends Repository {
      * Key to a <code>boolean</code> descriptor. Returns <code>true</code> if
      * and only if user management is supported.
      */
-    public static final String OPTION_USER_MANAGEMENT_SUPPORTED = "option.user.management.supported";
+    String OPTION_USER_MANAGEMENT_SUPPORTED = "option.user.management.supported";
 
     /**
      * Key to a <code>boolean</code> descriptor. Returns <code>true</code> if
      * and only if principal management is supported.
      */
-    public static final String OPTION_PRINCIPAL_MANAGEMENT_SUPPORTED = "option.principal.management.supported";
+    String OPTION_PRINCIPAL_MANAGEMENT_SUPPORTED = "option.principal.management.supported";
 
     /**
      * Key to a <code>boolean</code> descriptor. Returns <code>true</code> if
      * and only if privilege management is supported.
      */
-    public static final String OPTION_PRIVILEGE_MANAGEMENT_SUPPORTED = "option.privilege.management.supported";
+    String OPTION_PRIVILEGE_MANAGEMENT_SUPPORTED = "option.privilege.management.supported";
 
     /**
      * Equivalent to {@code login(credentials, workspaceName)} except that the returned
diff --git a/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/security/JackrabbitAccessControlManager.java b/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/security/JackrabbitAccessControlManager.java
index 50f434c2dc..fa44286cc4 100644
--- a/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/security/JackrabbitAccessControlManager.java
+++ b/oak-jackrabbit-api/src/main/java/org/apache/jackrabbit/api/security/JackrabbitAccessControlManager.java
@@ -133,7 +133,7 @@ public interface JackrabbitAccessControlManager extends AccessControlManager {
      * <code>READ_ACCESS_CONTROL</code> privilege for the <code>absPath</code> node.
      * @throws RepositoryException  if another error occurs.
      */
-    public boolean hasPrivileges(@Nullable String absPath, @NotNull Set<Principal> principals, @NotNull Privilege[] privileges)
+    boolean hasPrivileges(@Nullable String absPath, @NotNull Set<Principal> principals, @NotNull Privilege[] privileges)
             throws PathNotFoundException, AccessDeniedException, RepositoryException;
 
     /**
@@ -171,8 +171,7 @@ public interface JackrabbitAccessControlManager extends AccessControlManager {
      * privilege for the <code>absPath</code> node.
      * @throws RepositoryException  if another error occurs.
      */
-    @NotNull
-    public Privilege[] getPrivileges(@Nullable String absPath, @NotNull Set<Principal> principals)
+    @NotNull Privilege[] getPrivileges(@Nullable String absPath, @NotNull Set<Principal> principals)
             throws PathNotFoundException, AccessDeniedException, RepositoryException;
 
     /**