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

@@ -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) {