From 5853c55ac486dfbe55387b8f08d1b3e4f5e07f82 Mon Sep 17 00:00:00 2001
From: Andrew Auclair <mightymalakai33@gmail.com>
Date: Sat, 17 Aug 2024 15:34:03 -0400
Subject: [PATCH] Fixing a further issue with loading legacy layout files and
 then restoring the default layout. The loading legacy file flag was still set
 internally.

---
 docking-api/src/ModernDocking/api/LayoutPersistenceAPI.java | 6 ++----
 .../src/ModernDocking/internal/DockableProperties.java      | 2 ++
 .../src/ModernDocking/layouts/DockingSimplePanelNode.java   | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/docking-api/src/ModernDocking/api/LayoutPersistenceAPI.java b/docking-api/src/ModernDocking/api/LayoutPersistenceAPI.java
index 70f3282..66c9f8a 100644
--- a/docking-api/src/ModernDocking/api/LayoutPersistenceAPI.java
+++ b/docking-api/src/ModernDocking/api/LayoutPersistenceAPI.java
@@ -148,6 +148,8 @@ else if (next == XMLStreamConstants.END_ELEMENT && reader.getLocalName().equals(
             throw new DockingLayoutException(file, DockingLayoutException.FailureType.LOAD, e);
         }
         finally {
+            DockableProperties.setLoadingLegacyFile(false);
+
             try {
                 if (reader != null) {
                     reader.close();
@@ -517,8 +519,6 @@ private Map<String, Property> readProperties(XMLStreamReader reader) throws XMLS
                         }
                     }
                     else {
-                        DockableProperties.setLoadingLegacyFile(false);
-
                         while (reader.hasNext()) {
                             next = reader.nextTag();
 
@@ -625,8 +625,6 @@ else if (next == XMLStreamConstants.START_ELEMENT && reader.getLocalName().equal
                     }
                 }
                 else {
-                    DockableProperties.setLoadingLegacyFile(false);
-
                     while (reader.hasNext()) {
                         next = reader.nextTag();
 
diff --git a/docking-api/src/ModernDocking/internal/DockableProperties.java b/docking-api/src/ModernDocking/internal/DockableProperties.java
index 0bf06fc..f1ce868 100644
--- a/docking-api/src/ModernDocking/internal/DockableProperties.java
+++ b/docking-api/src/ModernDocking/internal/DockableProperties.java
@@ -72,6 +72,8 @@ public static void configureProperties(DockableWrapper wrapper, Map<String, Prop
                         if (loadingLegacyFile) {
                             Property.StringProperty legacyProp = (Property.StringProperty) properties.get(property.name());
                             prop = parseProperty(prop.getName(), field.getType().getSimpleName().toString(), legacyProp.getValue());
+
+                            properties.put(prop.getName(), prop);
                         }
                         DockableProperties.validateProperty(field, prop);
                     }
diff --git a/docking-api/src/ModernDocking/layouts/DockingSimplePanelNode.java b/docking-api/src/ModernDocking/layouts/DockingSimplePanelNode.java
index 3a0b46f..d4edeab 100644
--- a/docking-api/src/ModernDocking/layouts/DockingSimplePanelNode.java
+++ b/docking-api/src/ModernDocking/layouts/DockingSimplePanelNode.java
@@ -157,7 +157,7 @@ public String getClassName() {
 	 * @return properties map
 	 */
 	public Map<String, Property> getProperties() {
-		return Collections.unmodifiableMap(properties);
+		return properties;
 	}
 
 	public void setProperties(Map<String, Property> properties) {