Correct update of HEAD symbolic reference when target is a tag.

Philip L. McMahon

2012-01-27 30f9d25d77ccb5cd978d4cf8fa389ec819e90e95

 src/com/gitblit/GitBlit.java

@@ -69,12 +69,14 @@
import com.gitblit.models.FederationProposal;

import com.gitblit.models.FederationSet;

import com.gitblit.models.Metric;

import com.gitblit.models.RefModel;

import com.gitblit.models.RepositoryModel;

import com.gitblit.models.ServerSettings;

import com.gitblit.models.ServerStatus;

import com.gitblit.models.SettingModel;

import com.gitblit.models.TeamModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.ArrayUtils;

import com.gitblit.utils.ByteFormat;

import com.gitblit.utils.FederationUtils;

import com.gitblit.utils.JGitUtils;

@@ -451,7 +453,7 @@
      List<String> names = new ArrayList<String>(userService.getAllUsernames());

      return names;

   }

	


   /**

    * Returns the list of all users available to the login service.

    * 

@@ -545,7 +547,7 @@
      List<String> teams = new ArrayList<String>(userService.getAllTeamNames());

      return teams;

   }

	


   /**

    * Returns the list of available teams that a user or repository may be

    * assigned to.

@@ -652,21 +654,38 @@
    * @return repository or null

    */

   public Repository getRepository(String repositoryName) {

      return getRepository(repositoryName, true);

   }



   /**

    * Returns the JGit repository for the specified name.

    * 
    * @param repositoryName

    * @param logError

    * @return repository or null

    */

   public Repository getRepository(String repositoryName, boolean logError) {

      Repository r = null;

      try {

         r = repositoryResolver.open(null, repositoryName);

      } catch (RepositoryNotFoundException e) {

         r = null;

         logger.error("GitBlit.getRepository(String) failed to find "

               + new File(repositoriesFolder, repositoryName).getAbsolutePath());

         if (logError) {

            logger.error("GitBlit.getRepository(String) failed to find "

                  + new File(repositoriesFolder, repositoryName).getAbsolutePath());

         }

      } catch (ServiceNotAuthorizedException e) {

         r = null;

         logger.error("GitBlit.getRepository(String) failed to find "

               + new File(repositoriesFolder, repositoryName).getAbsolutePath(), e);

         if (logError) {

            logger.error("GitBlit.getRepository(String) failed to find "

                  + new File(repositoriesFolder, repositoryName).getAbsolutePath(), e);

         }

      } catch (ServiceNotEnabledException e) {

         r = null;

         logger.error("GitBlit.getRepository(String) failed to find "

               + new File(repositoriesFolder, repositoryName).getAbsolutePath(), e);

         if (logError) {

            logger.error("GitBlit.getRepository(String) failed to find "

                  + new File(repositoriesFolder, repositoryName).getAbsolutePath(), e);

         }

      }

      return r;

   }

@@ -768,6 +787,10 @@
         model.mailingLists = new ArrayList<String>(Arrays.asList(config.getStringList(

               "gitblit", null, "mailingList")));

      }

      model.defaultHead = JGitUtils.getDefaultHead(r);

      model.availableHeads = new ArrayList<RefModel>();

      model.availableHeads.addAll(JGitUtils.getLocalBranches(r, true, -1));

      model.availableHeads.addAll(JGitUtils.getTags(r, true, -1));

      r.close();

      return model;

   }

@@ -963,6 +986,9 @@
      // update settings

      if (r != null) {

         updateConfiguration(r, repository);

         if (repository.defaultHead != null) {

            JGitUtils.setDefaultHead(r, repository.defaultHead.reference);

         }

         r.close();

      }

   }

@@ -991,14 +1017,14 @@
      config.setString("gitblit", null, "federationStrategy",

            repository.federationStrategy.name());

      config.setBoolean("gitblit", null, "isFederated", repository.isFederated);

      if (repository.preReceiveScripts != null) {

      if (!ArrayUtils.isEmpty(repository.preReceiveScripts)) {

         config.setStringList("gitblit", null, "preReceiveScript", repository.preReceiveScripts);

      }

      if (repository.postReceiveScripts != null) {

      if (!ArrayUtils.isEmpty(repository.postReceiveScripts)) {

         config.setStringList("gitblit", null, "postReceiveScript",

               repository.postReceiveScripts);

      }

      if (repository.mailingLists != null) {

      if (!ArrayUtils.isEmpty(repository.mailingLists)) {

         config.setStringList("gitblit", null, "mailingList", repository.mailingLists);

      }

      try {

@@ -1741,9 +1767,7 @@
      try {

         // check to see if this "file" is a login service class

         Class<?> realmClass = Class.forName(realm);

         if (IUserService.class.isAssignableFrom(realmClass)) {

            loginService = (IUserService) realmClass.newInstance();

         }

         loginService = (IUserService) realmClass.newInstance();

      } catch (Throwable t) {

         loginService = new GitblitUserService();

      }

@@ -1787,6 +1811,21 @@
            webxmlSettings.applyOverrides(overrideFile);

         }

         configureContext(webxmlSettings, true);



         // Copy the included scripts to the configured groovy folder

         File localScripts = getFileOrFolder(Keys.groovy.scriptsFolder, "groovy");

         if (!localScripts.exists()) {

            File includedScripts = new File(context.getRealPath("/WEB-INF/groovy"));

            if (!includedScripts.equals(localScripts)) {

               try {

                  com.gitblit.utils.FileUtils.copy(localScripts, includedScripts.listFiles());

               } catch (IOException e) {

                  logger.error(MessageFormat.format(

                        "Failed to copy included Groovy scripts from {0} to {1}",

                        includedScripts, localScripts));

               }

            }

         }

      }



      serverStatus.servletContainer = servletContext.getServerInfo();