Fixes findbugs warning - dereferencing null in exception case

Simon Harrer

2013-07-18 fbe265fa787e4be9cb63c6bae2ef30b9400d9afc

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

@@ -32,7 +32,6 @@
import org.apache.wicket.markup.repeater.data.ListDataProvider;

import org.eclipse.jgit.lib.Repository;

import org.eclipse.jgit.revwalk.RevCommit;

import org.wicketstuff.googlecharts.Chart;

import org.wicketstuff.googlecharts.ChartAxis;

import org.wicketstuff.googlecharts.ChartAxisType;

import org.wicketstuff.googlecharts.ChartProvider;

@@ -53,6 +52,7 @@
import com.gitblit.utils.StringUtils;

import com.gitblit.wicket.GitBlitWebSession;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.charting.SecureChart;

import com.gitblit.wicket.panels.BranchesPanel;

import com.gitblit.wicket.panels.LinkPanel;

import com.gitblit.wicket.panels.LogPanel;

@@ -96,12 +96,16 @@
         private static final long serialVersionUID = 1L;

         int counter = 0;

         public void populateItem(final Item<String> item) {

            UserModel ownerModel = GitBlit.self().getUserModel(item.getModelObject());

            String ownername = item.getModelObject();

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

            if (ownerModel != null) {

               item.add(new LinkPanel("owner", null, ownerModel.getDisplayName(), UserPage.class,

                     WicketUtils.newUsernameParameter(ownerModel.username)).setRenderBodyOnly(true));

            } else {

               item.add(new Label("owner").setVisible(false));

               Label owner = new Label("owner", ownername);

               WicketUtils.setCssStyle(owner, "text-decoration: line-through;");

               WicketUtils.setHtmlTooltip(owner,  MessageFormat.format(getString("gb.failedToFindAccount"), ownername));

               item.add(owner);

            }

            counter++;

            item.add(new Label("comma", ",").setVisible(counter < owners.size()));

@@ -112,7 +116,8 @@
      add(ownersView);

      

      add(WicketUtils.createTimestampLabel("repositoryLastChange",

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

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

      add(new Label("repositorySize", getRepositoryModel().size));

      if (metricsTotal == null) {

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

      } else {

@@ -197,9 +202,9 @@
               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));

         provider.addShapeMarker(new ShapeMarker(MarkerType.CIRCLE, Color.decode("#002060"), 1, -1, 5));



         add(new Chart("commitsChart", provider));

         add(new SecureChart("commitsChart", provider));

      } else {

         add(WicketUtils.newBlankImage("commitsChart"));

      }