svn commit: r1858061 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/AllPermissionProviderImpl.java

[angela]

Author: angela
Date: Wed Apr 24 16:28:19 2019
New Revision: 1858061

URL: http://svn.apache.org/viewvc?rev=1858061&view=rev
Log:
OAK-8234 : adding missing annotations with AllPermissionProviderImpl

Modified:
    
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/AllPermissionProviderImpl.java

Modified: 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/AllPermissionProviderImpl.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/AllPermissionProviderImpl.java?rev=1858061&r1=1858060&r2=1858061&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/AllPermissionProviderImpl.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/permission/AllPermissionProviderImpl.java
 Wed Apr 24 16:28:19 2019
@@ -58,8 +58,9 @@ public final class AllPermissionProvider
         immutableRoot = providerCtx.getRootProvider().createReadOnlyRoot(root);
     }
 
+    @NotNull
     @Override
-    public @NotNull Set<String> getPrivileges(@Nullable Tree tree) {
+    public Set<String> getPrivileges(@Nullable Tree tree) {
         return ALL;
     }
 
@@ -68,13 +69,15 @@ public final class AllPermissionProvider
         return true;
     }
 
+    @NotNull
     @Override
-    public @NotNull RepositoryPermission getRepositoryPermission() {
+    public RepositoryPermission getRepositoryPermission() {
         return RepositoryPermission.ALL;
     }
 
+    @NotNull
     @Override
-    public @NotNull TreePermission getTreePermission(@NotNull Tree tree, 
@NotNull TreePermission parentPermission) {
+    public TreePermission getTreePermission(@NotNull Tree tree, @NotNull 
TreePermission parentPermission) {
         return TreePermission.ALL;
     }
 
@@ -89,34 +92,35 @@ public final class AllPermissionProvider
     }
 
     //---------------------------------------< AggregatedPermissionProvider 
>---
-
+    @NotNull
     @Override
-    public @NotNull PrivilegeBits supportedPrivileges(Tree tree, PrivilegeBits 
privilegeBits) {
+    public PrivilegeBits supportedPrivileges(@Nullable Tree tree, @Nullable 
PrivilegeBits privilegeBits) {
         return (privilegeBits != null) ? privilegeBits : new 
PrivilegeBitsProvider(immutableRoot).getBits(PrivilegeConstants.JCR_ALL);
     }
 
     @Override
-    public long supportedPermissions(Tree tree, PropertyState property, long 
permissions) {
+    public long supportedPermissions(@Nullable Tree tree, @Nullable 
PropertyState property, long permissions) {
         return permissions;
     }
 
     @Override
-    public long supportedPermissions(TreeLocation location, long permissions) {
+    public long supportedPermissions(@NotNull TreeLocation location, long 
permissions) {
         return permissions;
     }
 
     @Override
-    public long supportedPermissions(TreePermission treePermission, 
PropertyState property, long permissions) {
+    public long supportedPermissions(@NotNull TreePermission treePermission, 
PropertyState property, long permissions) {
         return permissions;
     }
 
     @Override
-    public boolean isGranted(TreeLocation location, long permissions) {
+    public boolean isGranted(@NotNull TreeLocation location, long permissions) 
{
         return true;
     }
 
+    @NotNull
     @Override
-    public @NotNull TreePermission getTreePermission(Tree tree, TreeType type, 
TreePermission parentPermission) {
+    public TreePermission getTreePermission(@NotNull Tree tree, @NotNull 
TreeType type, @NotNull TreePermission parentPermission) {
         return TreePermission.ALL;
     }
 }