You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by dk...@apache.org on 2020/03/19 19:49:34 UTC

[sling-org-apache-sling-app-cms] branch master updated: Minor - Fixing logging issue flagged by SonarQube

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

dklco pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-app-cms.git


The following commit(s) were added to refs/heads/master by this push:
     new 1ff3dfc  Minor - Fixing logging issue flagged by SonarQube
1ff3dfc is described below

commit 1ff3dfc250b18966d7ddac7fb0fb4887bff24c33
Author: Dan Klco <dk...@apache.org>
AuthorDate: Thu Mar 19 15:49:20 2020 -0400

    Minor - Fixing logging issue flagged by SonarQube
---
 .../sling/cms/core/internal/models/AuthorizableWrapperImpl.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/org/apache/sling/cms/core/internal/models/AuthorizableWrapperImpl.java b/core/src/main/java/org/apache/sling/cms/core/internal/models/AuthorizableWrapperImpl.java
index d8c53bc..d6d9a90 100644
--- a/core/src/main/java/org/apache/sling/cms/core/internal/models/AuthorizableWrapperImpl.java
+++ b/core/src/main/java/org/apache/sling/cms/core/internal/models/AuthorizableWrapperImpl.java
@@ -69,7 +69,7 @@ public class AuthorizableWrapperImpl implements AuthorizableWrapper {
                 return empty.iterator();
             }
         } catch (RepositoryException e) {
-            log.error("Failed to get membership of authorizable: {}", authorizable, e);
+            log.error("Failed to get declared members of authorizable: {}", authorizable, e);
             return Collections.emptyIterator();
         }
     }
@@ -79,7 +79,7 @@ public class AuthorizableWrapperImpl implements AuthorizableWrapper {
         try {
             return authorizable.declaredMemberOf();
         } catch (RepositoryException e) {
-            log.error("Failed to get membership of authorizable: {}", authorizable, e);
+            log.error("Failed to get declared membership of authorizable: {}", authorizable, e);
             return Collections.emptyIterator();
         }
     }
@@ -102,7 +102,7 @@ public class AuthorizableWrapperImpl implements AuthorizableWrapper {
         try {
             return authorizable.getID();
         } catch (RepositoryException e) {
-            log.error("Failed to get ID from authorizable: {}", e);
+            log.error("Failed to get ID from authorizable: {}", authorizable, e);
             return null;
         }
     }