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

Philip L. McMahon

2012-01-27 30f9d25d77ccb5cd978d4cf8fa389ec819e90e95

 src/com/gitblit/wicket/pages/SummaryPage.java

@@ -22,12 +22,9 @@
import java.util.ArrayList;

import java.util.List;



import javax.servlet.http.HttpServletRequest;



import org.apache.wicket.PageParameters;

import org.apache.wicket.markup.html.basic.Label;

import org.apache.wicket.markup.html.link.BookmarkablePageLink;

import org.apache.wicket.protocol.http.WebRequest;

import org.eclipse.jgit.lib.Repository;

import org.eclipse.jgit.revwalk.RevCommit;

import org.wicketstuff.googlecharts.Chart;

@@ -40,20 +37,21 @@
import org.wicketstuff.googlecharts.MarkerType;

import org.wicketstuff.googlecharts.ShapeMarker;



import com.gitblit.Constants;

import com.gitblit.Constants.AccessRestrictionType;

import com.gitblit.GitBlit;

import com.gitblit.Keys;

import com.gitblit.models.Metric;

import com.gitblit.models.PathModel;

import com.gitblit.models.RepositoryModel;

import com.gitblit.utils.ArrayUtils;

import com.gitblit.utils.JGitUtils;

import com.gitblit.utils.MarkdownUtils;

import com.gitblit.utils.MetricUtils;

import com.gitblit.utils.StringUtils;

import com.gitblit.utils.TimeUtils;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.panels.BranchesPanel;

import com.gitblit.wicket.panels.LogPanel;

import com.gitblit.wicket.panels.RepositoryUrlPanel;

import com.gitblit.wicket.panels.TagsPanel;



public class SummaryPage extends RepositoryPage {

@@ -61,41 +59,39 @@
   public SummaryPage(PageParameters params) {

      super(params);



      int numCommitsDef = 20;

      int numRefsDef = 5;



      int numberCommits = GitBlit.getInteger(Keys.web.summaryCommitCount, numCommitsDef);

      int numberCommits = GitBlit.getInteger(Keys.web.summaryCommitCount, 20);

      if (numberCommits <= 0) {

         numberCommits = numCommitsDef;

         numberCommits = 20;

      }



      int numberRefs = GitBlit.getInteger(Keys.web.summaryRefsCount, numRefsDef);

      if (numberRefs <= 0) {

         numberRefs = numRefsDef;

      }

      int numberRefs = GitBlit.getInteger(Keys.web.summaryRefsCount, 5);



      Repository r = getRepository();

      RepositoryModel model = getRepositoryModel();



      List<Metric> metrics = null;

      Metric metricsTotal = null;

      if (GitBlit.getBoolean(Keys.web.generateActivityGraph, true)) {

         metrics = MetricUtils.getDateMetrics(r, null, true, null);

      if (!model.skipSummaryMetrics && GitBlit.getBoolean(Keys.web.generateActivityGraph, true)) {

         metrics = GitBlit.self().getRepositoryDefaultMetrics(model, r);

         metricsTotal = metrics.remove(0);

      }



      addSyndicationDiscoveryLink();



      // repository description

      add(new Label("repositoryDescription", getRepositoryModel().description));

      add(new Label("repositoryOwner", getRepositoryModel().owner));



      add(WicketUtils.createTimestampLabel("repositoryLastChange", JGitUtils.getLastChange(r),

            getTimeZone()));

      add(WicketUtils.createTimestampLabel("repositoryLastChange",

            JGitUtils.getLastChange(r, null), getTimeZone()));

      if (metricsTotal == null) {

         add(new Label("branchStats", ""));			
         add(new Label("branchStats", ""));

      } else {

         add(new Label("branchStats", MessageFormat.format(

               "{0} commits and {1} tags in {2}", metricsTotal.count, metricsTotal.tag,

               TimeUtils.duration(metricsTotal.duration))));

         add(new Label("branchStats",

               MessageFormat.format("{0} commits and {1} tags in {2}", metricsTotal.count,

                     metricsTotal.tag, TimeUtils.duration(metricsTotal.duration))));

      }

      add(new BookmarkablePageLink<Void>("metrics", MetricsPage.class, WicketUtils.newRepositoryParameter(repositoryName)));

      add(new BookmarkablePageLink<Void>("metrics", MetricsPage.class,

            WicketUtils.newRepositoryParameter(repositoryName)));



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



@@ -120,27 +116,18 @@
         default:

            add(WicketUtils.newClearPixel("accessRestrictionIcon").setVisible(false));

         }



         HttpServletRequest req = ((WebRequest) getRequestCycle().getRequest())

               .getHttpServletRequest();

         StringBuilder sb = new StringBuilder();

         sb.append(req.getScheme());

         sb.append("://");

         sb.append(req.getServerName());

         if ((req.getScheme().equals("http") && req.getServerPort() != 80)

               || (req.getScheme().equals("https") && req.getServerPort() != 443)) {

            sb.append(":" + req.getServerPort());

         }

         sb.append(Constants.GIT_SERVLET_PATH);

         sb.append(repositoryName);

         repositoryUrls.add(sb.toString());

         // add the Gitblit repository url

         repositoryUrls.add(getRepositoryUrl(getRepositoryModel()));

      } else {

         add(WicketUtils.newClearPixel("accessRestrictionIcon").setVisible(false));

      }

      repositoryUrls.addAll(GitBlit.self().getOtherCloneUrls(repositoryName));

		
      String primaryUrl = ArrayUtils.isEmpty(repositoryUrls) ? "" : repositoryUrls.remove(0);

      add(new RepositoryUrlPanel("repositoryCloneUrl", primaryUrl));



      add(new Label("repositoryCloneUrl", StringUtils.flattenStrings(repositoryUrls, "<br/>"))

            .setEscapeModelStrings(false));

      add(new Label("otherUrls", StringUtils.flattenStrings(repositoryUrls, "<br/>"))

      .setEscapeModelStrings(false));



      add(new LogPanel("commitsPanel", repositoryName, null, r, numberCommits, 0));

      add(new TagsPanel("tagsPanel", repositoryName, r, numberRefs).hideIfEmpty());

@@ -207,7 +194,6 @@
         commitAxis.setLabels(new String[] { "",

               String.valueOf((int) WicketUtils.maxValue(metrics)) });

         provider.addAxis(commitAxis);



         provider.setLineStyles(new LineStyle[] { new LineStyle(2, 4, 0), new LineStyle(0, 4, 1) });

         provider.addShapeMarker(new ShapeMarker(MarkerType.CIRCLE, Color.BLUE, 1, -1, 5));