You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by kw...@apache.org on 2023/07/11 12:50:15 UTC

[jackrabbit-filevault] branch master updated: always close dependent package

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

kwin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jackrabbit-filevault.git


The following commit(s) were added to refs/heads/master by this push:
     new d2451d2e always close dependent package
d2451d2e is described below

commit d2451d2e4b3f8d5985a4ae149622e06418b721aa
Author: Konrad Windszus <kw...@apache.org>
AuthorDate: Tue Jul 11 14:50:07 2023 +0200

    always close dependent package
---
 .../apache/jackrabbit/vault/packaging/impl/JcrPackageImpl.java | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/impl/JcrPackageImpl.java b/vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/impl/JcrPackageImpl.java
index 99bb8242..509ba666 100644
--- a/vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/impl/JcrPackageImpl.java
+++ b/vault-core/src/main/java/org/apache/jackrabbit/vault/packaging/impl/JcrPackageImpl.java
@@ -764,9 +764,13 @@ public class JcrPackageImpl implements JcrPackage {
                     unresolved.add(dep);
                 } else {
                     RegisteredPackage pack = mgr.open(id);
-                    if (pack != null && !pack.isInstalled()) {
-                        unresolved.add(dep);
-                        uninstalled.add(pack);
+                    if (pack != null) {
+                        if (!pack.isInstalled()) {
+                            unresolved.add(dep);
+                            uninstalled.add(pack);
+                        } else {
+                            pack.close();
+                        }
                     }
                 }
             }