pbe
2014-05-05 16653e916fbf6b64baaaed2247d34d8f2372b89e
src/main/java/com/gitblit/wicket/pages/TicketPage.java
@@ -54,7 +54,6 @@
import org.eclipse.jgit.lib.Ref;
import org.eclipse.jgit.lib.Repository;
import org.eclipse.jgit.revwalk.RevCommit;
import org.eclipse.jgit.transport.URIish;
import com.gitblit.Constants;
import com.gitblit.Constants.AccessPermission;
@@ -64,6 +63,7 @@
import com.gitblit.models.PathModel.PathChangeModel;
import com.gitblit.models.RegistrantAccessPermission;
import com.gitblit.models.RepositoryModel;
import com.gitblit.models.RepositoryUrl;
import com.gitblit.models.SubmoduleModel;
import com.gitblit.models.TicketModel;
import com.gitblit.models.TicketModel.Change;
@@ -79,6 +79,7 @@
import com.gitblit.tickets.TicketLabel;
import com.gitblit.tickets.TicketMilestone;
import com.gitblit.tickets.TicketResponsible;
import com.gitblit.utils.ArrayUtils;
import com.gitblit.utils.JGitUtils;
import com.gitblit.utils.JGitUtils.MergeStatus;
import com.gitblit.utils.MarkdownUtils;
@@ -327,7 +328,7 @@
       * UPDATE FORM (DISCUSSION TAB)
       */
      if (user.canEdit(ticket, repository) && app().tickets().isAcceptingTicketUpdates(repository)) {
         if (ticket.isOpen()) {
         if (user.canAdmin(ticket, repository) && ticket.isOpen()) {
            /*
             * OPEN TICKET
             */
@@ -520,7 +521,7 @@
         WicketUtils.setCssClass(votersCount, "badge badge-info");
      }
      add(votersCount);
      if (user.isAuthenticated) {
      if (user.isAuthenticated && app().tickets().isAcceptingTicketUpdates(repository)) {
         Model<String> model;
         if (ticket.isVoter(user.username)) {
            model = Model.of(getString("gb.removeVote"));
@@ -560,7 +561,7 @@
         WicketUtils.setCssClass(watchersCount, "badge badge-info");
      }
      add(watchersCount);
      if (user.isAuthenticated) {
      if (user.isAuthenticated && app().tickets().isAcceptingTicketUpdates(repository)) {
         Model<String> model;
         if (ticket.isWatching(user.username)) {
            model = Model.of(getString("gb.stopWatching"));
@@ -732,16 +733,17 @@
       */
      if (currentPatchset == null) {
         // no patchset available
         if (ticket.isOpen() && app().tickets().isAcceptingNewPatchsets(repository)) {
         RepositoryUrl repoUrl = getRepositoryUrl(user, repository);
         boolean canPropose = repoUrl != null && repoUrl.permission.atLeast(AccessPermission.CLONE) && !UserModel.ANONYMOUS.equals(user);
         if (ticket.isOpen() && app().tickets().isAcceptingNewPatchsets(repository) && canPropose) {
            // ticket & repo will accept a proposal patchset
            // show the instructions for proposing a patchset
            String repoUrl = getRepositoryUrl(user, repository);
            Fragment changeIdFrag = new Fragment("patchset", "proposeFragment", this);
            changeIdFrag.add(new Label("proposeInstructions", MarkdownUtils.transformMarkdown(getString("gb.proposeInstructions"))).setEscapeModelStrings(false));
            changeIdFrag.add(new Label("ptWorkflow", MessageFormat.format(getString("gb.proposeWith"), "Barnum")));
            changeIdFrag.add(new Label("ptWorkflowSteps", getProposeWorkflow("propose_pt.md", repoUrl, ticket.number)).setEscapeModelStrings(false));
            changeIdFrag.add(new Label("ptWorkflowSteps", getProposeWorkflow("propose_pt.md", repoUrl.url, ticket.number)).setEscapeModelStrings(false));
            changeIdFrag.add(new Label("gitWorkflow", MessageFormat.format(getString("gb.proposeWith"), "Git")));
            changeIdFrag.add(new Label("gitWorkflowSteps", getProposeWorkflow("propose_git.md", repoUrl, ticket.number)).setEscapeModelStrings(false));
            changeIdFrag.add(new Label("gitWorkflowSteps", getProposeWorkflow("propose_git.md", repoUrl.url, ticket.number)).setEscapeModelStrings(false));
            add(changeIdFrag);
         } else {
            // explain why you can't propose a patchset
@@ -755,6 +757,12 @@
               reason = getString("gb.repositoryIsFrozen");
            } else if (!repository.acceptNewPatchsets) {
               reason = getString("gb.repositoryDoesNotAcceptPatchsets");
            } else if (!canPropose) {
               if (UserModel.ANONYMOUS.equals(user)) {
                  reason = getString("gb.anonymousCanNotPropose");
               } else {
                  reason = getString("gb.youDoNotHaveClonePermission");
               }
            } else {
               reason = getString("gb.serverDoesNotAcceptPatchsets");
            }
@@ -766,12 +774,19 @@
         Fragment patchsetFrag = new Fragment("patchset", "patchsetFragment", this);
         patchsetFrag.add(new Label("commitsInPatchset", MessageFormat.format(getString("gb.commitsInPatchsetN"), currentPatchset.number)));
         // current revision
         MarkupContainer panel = createPatchsetPanel("panel", repository, user);
         patchsetFrag.add(panel);
         addUserAttributions(patchsetFrag, currentRevision, avatarWidth);
         addUserAttributions(panel, currentRevision, 0);
         addDateAttributions(panel, currentRevision);
         patchsetFrag.add(createMergePanel(user, repository));
         if (ticket.isOpen()) {
            // current revision
            MarkupContainer panel = createPatchsetPanel("panel", repository, user);
            patchsetFrag.add(panel);
            addUserAttributions(patchsetFrag, currentRevision, avatarWidth);
            addUserAttributions(panel, currentRevision, 0);
            addDateAttributions(panel, currentRevision);
         } else {
            // current revision
            patchsetFrag.add(new Label("panel").setVisible(false));
         }
         // commits
         List<RevCommit> commits = JGitUtils.getRevLog(getRepository(), currentPatchset.base, currentPatchset.tip);
@@ -988,7 +1003,11 @@
      md = md.replace("${ticketId}", "" + ticketId);
      md = md.replace("${patchset}", "" + 1);
      md = md.replace("${reviewBranch}", Repository.shortenRefName(PatchsetCommand.getTicketBranch(ticketId)));
      md = md.replace("${integrationBranch}", Repository.shortenRefName(getRepositoryModel().HEAD));
      String integrationBranch = Repository.shortenRefName(getRepositoryModel().mergeTo);
      if (!StringUtils.isEmpty(ticket.mergeTo)) {
         integrationBranch = ticket.mergeTo;
      }
      md = md.replace("${integrationBranch}", integrationBranch);
      return MarkdownUtils.transformMarkdown(md);
   }
@@ -1065,7 +1084,7 @@
      panel.add(reviewsView);
      if (ticket.isOpen() && user.canReviewPatchset(repository)) {
      if (ticket.isOpen() && user.canReviewPatchset(repository) && app().tickets().isAcceptingTicketUpdates(repository)) {
         // can only review open tickets
         Review myReview = null;
         for (Change change : ticket.getReviews(currentPatchset)) {
@@ -1129,7 +1148,6 @@
            WicketUtils.setChangeTypeCssClass(changeType, entry.changeType);
            setChangeTypeTooltip(changeType, entry.changeType);
            item.add(changeType);
            item.add(new DiffStatPanel("diffStat", entry.insertions, entry.deletions, true));
            boolean hasSubmodule = false;
            String submodulePath = null;
@@ -1155,7 +1173,7 @@
               String displayPath = entry.path;
               String path = entry.path;
               if (entry.isSymlink()) {
                  RevCommit commit = JGitUtils.getCommit(getRepository(), Constants.R_TICKETS_PATCHSETS + ticket.number);
                  RevCommit commit = JGitUtils.getCommit(getRepository(), PatchsetCommand.getTicketBranch(ticket.number));
                  path = JGitUtils.getStringContent(getRepository(), commit.getTree(), path);
                  displayPath = entry.path + " -> " + path;
               }
@@ -1172,6 +1190,7 @@
                  item.add(new LinkPanel("pathName", "list", displayPath, BlobDiffPage.class,
                        WicketUtils.newPathParameter(repositoryName, currentPatchset.tip, path), true));
               }
               item.add(new DiffStatPanel("diffStat", entry.insertions, entry.deletions, true));
            }
            // quick links
@@ -1199,9 +1218,8 @@
      };
      panel.add(pathsView);
      addPtReviewInstructions(user, repository, panel);
      addGitReviewInstructions(user, repository, panel);
      panel.add(createMergePanel(user, repository));
      addPtCheckoutInstructions(user, repository, panel);
      addGitCheckoutInstructions(user, repository, panel);
      return panel;
   }
@@ -1274,13 +1292,13 @@
      return x;
   }
   protected void addGitReviewInstructions(UserModel user, RepositoryModel repository, MarkupContainer panel) {
   protected void addGitCheckoutInstructions(UserModel user, RepositoryModel repository, MarkupContainer panel) {
      panel.add(new Label("gitStep1", MessageFormat.format(getString("gb.stepN"), 1)));
      panel.add(new Label("gitStep2", MessageFormat.format(getString("gb.stepN"), 2)));
      String ticketBranch  = Repository.shortenRefName(PatchsetCommand.getTicketBranch(ticket.number));
      String step1 = "git fetch";
      String step1 = "git fetch origin";
      String step2 = MessageFormat.format("git checkout {0} && git pull --ff-only\nOR\ngit checkout {0} && git reset --hard origin/{0}", ticketBranch);
      panel.add(new Label("gitPreStep1", step1));
@@ -1290,7 +1308,7 @@
      panel.add(createCopyFragment("gitCopyStep2", step2.replace("\n", " && ")));
   }
   protected void addPtReviewInstructions(UserModel user, RepositoryModel repository, MarkupContainer panel) {
   protected void addPtCheckoutInstructions(UserModel user, RepositoryModel repository, MarkupContainer panel) {
      String step1 = MessageFormat.format("pt checkout {0,number,0}", ticket.number);
      panel.add(new Label("ptPreStep", step1));
      panel.add(createCopyFragment("ptCopyStep", step1));
@@ -1470,15 +1488,14 @@
    * @param repository
    * @return the primary repository url
    */
   protected String getRepositoryUrl(UserModel user, RepositoryModel repository) {
   protected RepositoryUrl getRepositoryUrl(UserModel user, RepositoryModel repository) {
      HttpServletRequest req = ((WebRequest) getRequest()).getHttpServletRequest();
      String primaryurl = app().gitblit().getRepositoryUrls(req, user, repository).get(0).url;
      String url = primaryurl;
      try {
         url = new URIish(primaryurl).setUser(null).toString();
      } catch (Exception e) {
      List<RepositoryUrl> urls = app().gitblit().getRepositoryUrls(req, user, repository);
      if (ArrayUtils.isEmpty(urls)) {
         return null;
      }
      return url;
      RepositoryUrl primary = urls.get(0);
      return primary;
   }
   /**