You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@isis.apache.org by ah...@apache.org on 2022/01/13 19:45:06 UTC

[isis] branch master updated: ISIS-2942: secman: fixes wrong logic with ten. child removal

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

ahuber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/isis.git


The following commit(s) were added to refs/heads/master by this push:
     new a38a150  ISIS-2942: secman: fixes wrong logic with ten. child removal
a38a150 is described below

commit a38a150c0353dadec5bd974df165c6badb7d5900
Author: Andi Huber <ah...@apache.org>
AuthorDate: Thu Jan 13 20:44:51 2022 +0100

    ISIS-2942: secman: fixes wrong logic with ten. child removal
---
 .../secman/applib/tenancy/dom/ApplicationTenancyRepositoryAbstract.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/extensions/security/secman/applib/src/main/java/org/apache/isis/extensions/secman/applib/tenancy/dom/ApplicationTenancyRepositoryAbstract.java b/extensions/security/secman/applib/src/main/java/org/apache/isis/extensions/secman/applib/tenancy/dom/ApplicationTenancyRepositoryAbstract.java
index 24c9193..81381b6 100644
--- a/extensions/security/secman/applib/src/main/java/org/apache/isis/extensions/secman/applib/tenancy/dom/ApplicationTenancyRepositoryAbstract.java
+++ b/extensions/security/secman/applib/src/main/java/org/apache/isis/extensions/secman/applib/tenancy/dom/ApplicationTenancyRepositoryAbstract.java
@@ -202,7 +202,7 @@ implements ApplicationTenancyRepository {
             final @NonNull ApplicationTenancy tenancy) {
         val parent = tenancy.getParent();
         if(parent != null) {
-            parent.getChildren().add(tenancy);
+            parent.getChildren().remove(tenancy);
             tenancy.setParent(null);
         }
     }