Improve getting changed paths in a commit range

James Moger

2014-02-28 131da2786c1bf275c07ba628af4254a40d1dc42f

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

@@ -133,7 +133,7 @@
    * @return cookie value

    */

   @Override

   public String getCookie(UserModel model) {

   public synchronized String getCookie(UserModel model) {

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

         return model.cookie;

      }

@@ -195,7 +195,7 @@
    * @return true if update is successful

    */

   @Override

   public boolean updateUserModel(UserModel model) {

   public synchronized boolean updateUserModel(UserModel model) {

      return updateUserModel(model.username, model);

   }



@@ -272,6 +272,9 @@
         }

         read();

         originalUser = users.remove(username.toLowerCase());

         if (originalUser != null) {

            cookies.remove(originalUser.cookie);

         }

         users.put(model.username.toLowerCase(), model);

         // null check on "final" teams because JSON-sourced UserModel

         // can have a null teams object

@@ -322,7 +325,7 @@
    * @return true if successful

    */

   @Override

   public boolean deleteUserModel(UserModel model) {

   public synchronized boolean deleteUserModel(UserModel model) {

      return deleteUser(model.username);

   }



@@ -369,7 +372,7 @@
    * @since 0.8.0

    */

   @Override

   public List<String> getAllTeamNames() {

   public synchronized List<String> getAllTeamNames() {

      read();

      List<String> list = new ArrayList<String>(teams.keySet());

      Collections.sort(list);

@@ -444,7 +447,7 @@
    * @since 0.8.0

    */

   @Override

   public boolean updateTeamModel(TeamModel model) {

   public synchronized boolean updateTeamModel(TeamModel model) {

      return updateTeamModel(model.name, model);

   }



@@ -456,7 +459,7 @@
    * @since 1.2.0

    */

   @Override

   public boolean updateTeamModels(Collection<TeamModel> models) {

   public synchronized boolean updateTeamModels(Collection<TeamModel> models) {

      try {

         read();

         for (TeamModel team : models) {

@@ -482,7 +485,7 @@
    * @since 0.8.0

    */

   @Override

   public boolean updateTeamModel(String teamname, TeamModel model) {

   public synchronized boolean updateTeamModel(String teamname, TeamModel model) {

      TeamModel original = null;

      try {

         read();

@@ -511,7 +514,7 @@
    * @since 0.8.0

    */

   @Override

   public boolean deleteTeamModel(TeamModel model) {

   public synchronized boolean deleteTeamModel(TeamModel model) {

      return deleteTeam(model.name);

   }



@@ -523,7 +526,7 @@
    * @since 0.8.0

    */

   @Override

   public boolean deleteTeam(String teamname) {

   public synchronized boolean deleteTeam(String teamname) {

      try {

         // Read realm file

         read();

@@ -542,7 +545,7 @@
    * @return list of all usernames

    */

   @Override

   public List<String> getAllUsernames() {

   public synchronized List<String> getAllUsernames() {

      read();

      List<String> list = new ArrayList<String>(users.keySet());

      Collections.sort(list);