From dccd3d108f3074d0750d2d8d57a59fb58a6985ef Mon Sep 17 00:00:00 2001
From: kohsuke <kohsuke@71c3de6d-444a-0410-be80-ed276b4c234a>
Date: Sat, 16 Jun 2007 01:38:23 +0000
Subject: [PATCH] if we clear it after load, load is meaningless! (issue #600)

git-svn-id: https://hudson.dev.java.net/svn/hudson/trunk/hudson/main@3453 71c3de6d-444a-0410-be80-ed276b4c234a
---
 core/src/main/java/hudson/model/User.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/hudson/model/User.java b/core/src/main/java/hudson/model/User.java
index cf0a2a3201..e4f19a6ec8 100644
--- a/core/src/main/java/hudson/model/User.java
+++ b/core/src/main/java/hudson/model/User.java
@@ -57,6 +57,8 @@ public class User extends AbstractModelObject {
      * Loads the other data from disk if it's available.
      */
     private synchronized void load() {
+        properties.clear();
+
         XmlFile config = getConfigFile();
         try {
             if(config.exists())
@@ -65,7 +67,6 @@ public class User extends AbstractModelObject {
             LOGGER.log(Level.SEVERE, "Failed to load "+config,e);
         }
 
-        properties.clear();
         // allocate default instances if needed.
         // doing so after load makes sure that newly added user properties do get reflected
         for (UserPropertyDescriptor d : UserProperties.LIST) {
-- 
GitLab