Eliminate redundant read() and use accessor instead of direct cache access

James Moger

2013-09-23 b2d7f418bf289e5fa6149ac1b5744cfb03b60dea

 src/main/java/com/gitblit/ConfigUserService.java

@@ -37,6 +37,7 @@
import com.gitblit.Constants.AccessPermission;

import com.gitblit.models.TeamModel;

import com.gitblit.models.UserModel;

import com.gitblit.models.UserRepositoryPreferences;

import com.gitblit.utils.ArrayUtils;

import com.gitblit.utils.DeepCopier;

import com.gitblit.utils.StringUtils;

@@ -88,6 +89,10 @@
   private static final String PRERECEIVE = "preReceiveScript";



   private static final String POSTRECEIVE = "postReceiveScript";

	
   private static final String STARRED = "starred";

	
   private static final String LOCALE = "locale";



   private final File realmFile;



@@ -181,8 +186,10 @@
      if (!StringUtils.isEmpty(model.cookie)) {

         return model.cookie;

      }

      read();

      UserModel storedModel = users.get(model.username.toLowerCase());

      UserModel storedModel = getUserModel(model.username);

      if (storedModel == null) {

         return null;

      }

      return storedModel.cookie;

   }



@@ -203,6 +210,12 @@
      if (cookies.containsKey(hash)) {

         model = cookies.get(hash);

      }

		
      if (model != null) {

         // clone the model, otherwise all changes to this object are

         // live and unpersisted

         model = DeepCopier.copy(model);

      }

      return model;

   }



@@ -215,7 +228,6 @@
    */

   @Override

   public UserModel authenticate(String username, char[] password) {

      read();

      UserModel returnedUser = null;

      UserModel user = getUserModel(username);

      if (user == null) {

@@ -846,6 +858,11 @@
         if (!StringUtils.isEmpty(model.countryCode)) {

            config.setString(USER, model.username, COUNTRYCODE, model.countryCode);

         }

         if (model.getPreferences() != null) {

            if (!StringUtils.isEmpty(model.getPreferences().locale)) {

               config.setString(USER, model.username, LOCALE, model.getPreferences().locale);

            }

         }



         // user roles

         List<String> roles = new ArrayList<String>();

@@ -878,6 +895,14 @@
               }

            }

            config.setStringList(USER, model.username, REPOSITORY, permissions);

         }

			
         // user preferences

         if (model.getPreferences() != null) {

            List<String> starred =  model.getPreferences().getStarredRepositories();

            if (starred.size() > 0) {

               config.setStringList(USER, model.username, STARRED, starred);

            }

         }

      }



@@ -999,6 +1024,7 @@
               user.stateProvince = config.getString(USER, username, STATEPROVINCE);

               user.countryCode = config.getString(USER, username, COUNTRYCODE);

               user.cookie = config.getString(USER, username, COOKIE);

               user.getPreferences().locale = config.getString(USER, username, LOCALE);	
               if (StringUtils.isEmpty(user.cookie) && !StringUtils.isEmpty(user.password)) {

                  user.cookie = StringUtils.getSHA1(user.username + user.password);

               }

@@ -1021,6 +1047,14 @@
                  }

               }



               // starred repositories

               Set<String> starred = new HashSet<String>(Arrays.asList(config

                     .getStringList(USER, username, STARRED)));

               for (String repository : starred) {

                  UserRepositoryPreferences prefs = user.getPreferences().getRepositoryPreferences(repository);

                  prefs.starred = true;

               }



               // update cache

               users.put(user.username, user);

               if (!StringUtils.isEmpty(user.cookie)) {