Added null checkinng and logging for edit user permissions (issue 155)

James Moger

2012-10-30 a4231d30c4d2765c80f57df79cd1d4d39a9795ea

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

@@ -45,12 +45,14 @@
import com.gitblit.models.Metric;

import com.gitblit.models.PathModel;

import com.gitblit.models.RepositoryModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.ArrayUtils;

import com.gitblit.utils.JGitUtils;

import com.gitblit.utils.MarkdownUtils;

import com.gitblit.utils.StringUtils;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.panels.BranchesPanel;

import com.gitblit.wicket.panels.LinkPanel;

import com.gitblit.wicket.panels.LogPanel;

import com.gitblit.wicket.panels.RepositoryUrlPanel;

import com.gitblit.wicket.panels.TagsPanel;

@@ -80,7 +82,17 @@


      // repository description

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

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

      String owner = getRepositoryModel().owner;

      if (StringUtils.isEmpty(owner)) {

         add(new Label("repositoryOwner").setVisible(false));

      } else {

         UserModel ownerModel = GitBlit.self().getUserModel(owner);

         if (ownerModel != null) {

            add(new LinkPanel("repositoryOwner", null, ownerModel.getDisplayName(), UserPage.class, WicketUtils.newUsernameParameter(owner)));

         } else {

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

         }

      }



      add(WicketUtils.createTimestampLabel("repositoryLastChange",

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

@@ -130,12 +142,13 @@
      add(new Label("otherUrls", StringUtils.flattenStrings(repositoryUrls, "<br/>"))

      .setEscapeModelStrings(false));



      add(new LogPanel("commitsPanel", repositoryName, getRepositoryModel().HEAD, r, numberCommits, 0));

      add(new LogPanel("commitsPanel", repositoryName, getRepositoryModel().HEAD, r, numberCommits, 0, getRepositoryModel().showRemoteBranches));

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

      add(new BranchesPanel("branchesPanel", getRepositoryModel(), r, numberRefs).hideIfEmpty());

      add(new BranchesPanel("branchesPanel", getRepositoryModel(), r, numberRefs, false).hideIfEmpty());



      if (getRepositoryModel().showReadme) {

         String htmlText = null;

         String markdownText = null;

         String readme = null;

         try {

            RevCommit head = JGitUtils.getCommit(r, null);

@@ -158,11 +171,12 @@
            }

            if (!StringUtils.isEmpty(readme)) {

               String [] encodings = GitBlit.getEncodings();

               String markdownText = JGitUtils.getStringContent(r, head.getTree(), readme, encodings);

               markdownText = JGitUtils.getStringContent(r, head.getTree(), readme, encodings);

               htmlText = MarkdownUtils.transformMarkdown(markdownText);

            }

         } catch (ParseException p) {

            error(p.getMessage());

            markdownText = MessageFormat.format("<div class=\"alert alert-error\"><strong>{0}:</strong> {1}</div>{2}", getString("gb.error"), getString("gb.markdownFailure"), markdownText);

            htmlText = StringUtils.breakLinesForHtml(markdownText);

         }

         Fragment fragment = new Fragment("readme", "markdownPanel");

         fragment.add(new Label("readmeFile", readme));