From adbf00c9284e5110ce80accd59d220c7b6b22ca4 Mon Sep 17 00:00:00 2001
From: Katarina Valalikova <k.valalikova@evolveum.com>
Date: Tue, 1 Feb 2022 16:44:38 +0100
Subject: [PATCH] stringResourceModel -> BasePanel, do not use component so the
 model can be detached. and do not detach what should not be detached

---
 .../midpoint/gui/api/component/BasePanel.java |  8 ++--
 .../midpoint/gui/api/page/PageBase.java       | 38 +++++--------------
 2 files changed, 13 insertions(+), 33 deletions(-)

diff --git a/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/component/BasePanel.java b/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/component/BasePanel.java
index 4e526aab4f5..59503b76c88 100644
--- a/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/component/BasePanel.java
+++ b/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/component/BasePanel.java
@@ -67,13 +67,13 @@ public String getString(Enum<?> e) {
     }
 
     public StringResourceModel createStringResource(String resourceKey, IModel<?> model, Object... objects) {
-        return new StringResourceModel(resourceKey, this).setModel(model)
+        return new StringResourceModel(resourceKey).setModel(model)
                 .setDefaultValue(resourceKey)
                 .setParameters(objects);
     }
 
     public StringResourceModel createStringResource(String resourceKey, Object... objects) {
-        return new StringResourceModel(resourceKey, this).setModel(null)
+        return new StringResourceModel(resourceKey).setModel(null)
                 .setDefaultValue(resourceKey)
                 .setParameters(objects);
     }
@@ -84,7 +84,7 @@ public StringResourceModel createStringResource(PolyString polystringKey, Object
             // TODO later: use polystringKey.getKey()
             resourceKey = polystringKey.getOrig();
         }
-        return new StringResourceModel(resourceKey, this).setModel(null)
+        return new StringResourceModel(resourceKey).setModel(null)
                 .setDefaultValue(resourceKey)
                 .setParameters(objects);
     }
@@ -95,7 +95,7 @@ public StringResourceModel createStringResource(PolyStringType polystringKey, Ob
             // TODO later: use polystringKey.getKey()
             resourceKey = polystringKey.getOrig();
         }
-        return new StringResourceModel(resourceKey, this).setModel(null)
+        return new StringResourceModel(resourceKey).setModel(null)
                 .setDefaultValue(resourceKey)
                 .setParameters(objects);
     }
diff --git a/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/page/PageBase.java b/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/page/PageBase.java
index c6bb1899d08..7eb97c3cebd 100644
--- a/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/page/PageBase.java
+++ b/gui/admin-gui/src/main/java/com/evolveum/midpoint/gui/api/page/PageBase.java
@@ -683,27 +683,7 @@ public String getObject() {
         pageTitleReal.setRenderBodyOnly(true);
         pageTitle.add(pageTitleReal);
 
-        LoadableDetachableModel<List<Breadcrumb>> breadcrumbsModel = new LoadableDetachableModel<List<Breadcrumb>>() {
-
-            private static final long serialVersionUID = 1L;
-
-            @Override
-            protected List<Breadcrumb> load() {
-                return getBreadcrumbs();
-            }
-
-            @Override
-            protected void onDetach() {
-                for (Breadcrumb b : getObject()) {
-                    b.detach();
-                }
-            }
-
-            //            @Override
-//            public List<Breadcrumb> getObject() {
-//                return getBreadcrumbs();
-//            }
-        };
+        IModel<List<Breadcrumb>> breadcrumbsModel = () -> getBreadcrumbs();
 
         ListView<Breadcrumb> breadcrumbs = new ListView<>(ID_BREADCRUMB, breadcrumbsModel) {
 
@@ -746,14 +726,14 @@ public boolean isVisible() {
                 Label bcName = new Label(ID_BC_NAME, item.getModelObject().getLabel());
                 bcLink.add(bcName);
 
-//                item.add(new VisibleEnableBehaviour() {
-//                    private static final long serialVersionUID = 1L;
-//
-//                    @Override
-//                    public boolean isVisible() {
-//                        return item.getModelObject().isVisible();
-//                    }
-//                });
+                item.add(new VisibleEnableBehaviour() {
+                    private static final long serialVersionUID = 1L;
+
+                    @Override
+                    public boolean isVisible() {
+                        return item.getModelObject().isVisible();
+                    }
+                });
             }
         };
         breadcrumbs.add(new VisibleBehaviour(() -> !isErrorPage()));