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

kwin pushed a commit to branch feature/ease-use-of-filterproviderimpl
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


The following commit(s) were added to 
refs/heads/feature/ease-use-of-filterproviderimpl by this push:
     new 0f2cc6d400 Restore nullness annotations
0f2cc6d400 is described below

commit 0f2cc6d400ffde2491ce5227a0a4a00073109824
Author: Konrad Windszus <k...@apache.org>
AuthorDate: Tue Jul 11 15:05:29 2023 +0200

    Restore nullness annotations
    
    Remove non-related code cleanup
---
 .../authorization/principalbased/impl/FilterProviderImpl.java     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java
 
b/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java
index eb903887de..6eb212c7b6 100644
--- 
a/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java
+++ 
b/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java
@@ -81,7 +81,7 @@ public class FilterProviderImpl implements FilterProvider {
      * @param oakPath the repository path where the principals are located
      * @since 1.54
      */
-    public FilterProviderImpl(String oakPath) {
+    public FilterProviderImpl(@NotNull String oakPath) {
         setPath(oakPath);
     }
 
@@ -112,16 +112,16 @@ public class FilterProviderImpl implements FilterProvider 
{
     }
 
     @Activate
-    protected void activate(Configuration configuration) {
+    protected void activate(Configuration configuration, Map<String, Object> 
properties) {
         setPath(configuration.path());
     }
 
     @Modified
-    protected void modified(Configuration configuration) {
+    protected void modified(Configuration configuration, Map<String, Object> 
properties) {
         setPath(configuration.path());
     }
 
-    private void setPath(String path) {
+    private void setPath(@NotNull String path) {
         checkState(isValidPath(path), "Configured path must be a valid 
absolute path.");
         oakPath = path;
     }

Reply via email to