Documentation

James Moger

2014-05-12 edeab95cac16e5f17cfcd75a9969d8708bf360ab

 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,12 +79,14 @@
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;

import com.gitblit.utils.StringUtils;

import com.gitblit.utils.TimeUtils;

import com.gitblit.wicket.GitBlitWebSession;

import com.gitblit.wicket.TicketsUI;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.panels.BasePanel.JavascriptTextPrompt;

import com.gitblit.wicket.panels.CommentPanel;

@@ -101,7 +103,7 @@
 * @author James Moger

 *

 */

public class TicketPage extends TicketBasePage {

public class TicketPage extends RepositoryPage {



   static final String NIL = "<nil>";



@@ -153,7 +155,7 @@
      String href = urlFor(TicketsPage.class, params).toString();

      add(new ExternalLink("ticketNumber", href, "#" + ticket.number));

      Label headerStatus = new Label("headerStatus", ticket.status.toString());

      WicketUtils.setCssClass(headerStatus, getLozengeClass(ticket.status, false));

      WicketUtils.setCssClass(headerStatus, TicketsUI.getLozengeClass(ticket.status, false));

      add(headerStatus);

      add(new Label("ticketTitle", ticket.title));

      if (currentPatchset == null) {

@@ -316,10 +318,10 @@
       * LARGE STATUS INDICATOR WITH ICON (DISCUSSION TAB->SIDE BAR)

       */

      Fragment ticketStatus = new Fragment("ticketStatus", "ticketStatusFragment", this);

      Label ticketIcon = getStateIcon("ticketIcon", ticket);

      Label ticketIcon = TicketsUI.getStateIcon("ticketIcon", ticket);

      ticketStatus.add(ticketIcon);

      ticketStatus.add(new Label("ticketStatus", ticket.status.toString()));

      WicketUtils.setCssClass(ticketStatus, getLozengeClass(ticket.status, false));

      WicketUtils.setCssClass(ticketStatus, TicketsUI.getLozengeClass(ticket.status, false));

      add(ticketStatus);





@@ -369,7 +371,7 @@
                        setResponsePage(TicketsPage.class, getPageParameters());

                     }

                  };

                  String css = getStatusClass(item.getModel().getObject());

                  String css = TicketsUI.getStatusClass(item.getModel().getObject());

                  WicketUtils.setCssClass(link, css);

                  item.add(link);

               }

@@ -664,7 +666,7 @@
                   */

                  Fragment frag = new Fragment("entry", "statusFragment", this);

                  Label status = new Label("statusChange", entry.getStatus().toString());

                  String css = getLozengeClass(entry.getStatus(), false);

                  String css = TicketsUI.getLozengeClass(entry.getStatus(), false);

                  WicketUtils.setCssClass(status, css);

                  for (IBehavior b : status.getBehaviors()) {

                     if (b instanceof SimpleAttributeModifier) {

@@ -732,16 +734,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 +758,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 +775,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);

@@ -921,7 +937,7 @@
                     case status:

                        // special handling for status

                        Status status = event.getStatus();

                        String css = getLozengeClass(status, true);

                        String css = TicketsUI.getLozengeClass(status, true);

                        value = String.format("<span class=\"%1$s\">%2$s</span>", css, status.toString());

                        break;

                     default:

@@ -988,7 +1004,7 @@
      md = md.replace("${ticketId}", "" + ticketId);

      md = md.replace("${patchset}", "" + 1);

      md = md.replace("${reviewBranch}", Repository.shortenRefName(PatchsetCommand.getTicketBranch(ticketId)));

      String integrationBranch = Repository.shortenRefName(getRepositoryModel().HEAD);

      String integrationBranch = Repository.shortenRefName(getRepositoryModel().mergeTo);

      if (!StringUtils.isEmpty(ticket.mergeTo)) {

         integrationBranch = ticket.mergeTo;

      }

@@ -1069,7 +1085,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)) {

@@ -1133,7 +1149,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;

@@ -1159,7 +1174,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;

               }

@@ -1176,6 +1191,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

@@ -1203,9 +1219,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;

   }

@@ -1278,13 +1293,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));

@@ -1294,7 +1309,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));

@@ -1474,15 +1489,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;

   }



   /**

@@ -1512,14 +1526,14 @@
      switch (type) {

         case Rebase:

         case Rebase_Squash:

            typeCss = getLozengeClass(Status.Declined, false);

            typeCss = TicketsUI.getLozengeClass(Status.Declined, false);

            break;

         case Squash:

         case Amend:

            typeCss = getLozengeClass(Status.On_Hold, false);

            typeCss = TicketsUI.getLozengeClass(Status.On_Hold, false);

            break;

         case Proposal:

            typeCss = getLozengeClass(Status.New, false);

            typeCss = TicketsUI.getLozengeClass(Status.New, false);

            break;

         case FastForward:

         default: