Tweak the receive pack push permission check to be a method

James Moger

2014-03-12 a66312f3d4bb9f1f6aad5ed66d2082057c99f519

 src/main/java/com/gitblit/wicket/pages/TicketPage.java

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

@@ -1203,8 +1203,8 @@
      };

      panel.add(pathsView);



      addPtReviewInstructions(user, repository, panel);

      addGitReviewInstructions(user, repository, panel);

      addPtCheckoutInstructions(user, repository, panel);

      addGitCheckoutInstructions(user, repository, panel);

      panel.add(createMergePanel(user, repository));



      return panel;

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