Working on error handling.

James Moger

2011-04-29 bc9d4a0f2266e5ac1a018f0b691c76d53479b9e5

 src/com/gitblit/utils/JGitUtils.java

@@ -94,10 +94,10 @@
            File gitFolder = new File(file, Constants.DOT_GIT);

            boolean isGitRepository = gitFolder.exists() && gitFolder.isDirectory();

            

            // then look for folder.git/HEAD or folder/HEAD and folder/config

            // then look for folder.git/HEAD or folder/HEAD and

            // folder/config

            if (!isGitRepository) {

               if ((file.getName().endsWith(Constants.DOT_GIT_EXT) && new File(file, Constants.HEAD).exists())

                     || (new File(file, "config").exists() && new File(file, Constants.HEAD).exists())) {

               if ((file.getName().endsWith(Constants.DOT_GIT_EXT) && new File(file, Constants.HEAD).exists()) || (new File(file, "config").exists() && new File(file, Constants.HEAD).exists())) {

                  gitFolder = file;

                  isGitRepository = true;

               }

@@ -124,6 +124,9 @@
   }



   public static RevCommit getFirstCommit(Repository r, String branch) {

      if (!hasCommits(r)) {

         return null;

      }

      if (StringUtils.isEmpty(branch)) {

         branch = Constants.HEAD;

      }

@@ -155,17 +158,24 @@
      return null;

   }



   public static boolean hasCommits(Repository r) {

      return new File(r.getDirectory(), Constants.R_HEADS).list().length > 0;

   }



   public static Date getLastChange(Repository r) {      

      RevCommit commit = getCommit(r, Constants.HEAD);

      if (commit == null) {

      if (!hasCommits(r)) {

         // fresh repository

         return new Date(r.getDirectory().lastModified());         

      }

      RevCommit commit = getCommit(r, Constants.HEAD);

      return getCommitDate(commit);

   }



   public static RevCommit getCommit(Repository r, String objectId) {

      RevCommit commit = null;

      if (!hasCommits(r)) {

         return null;

      }

      try {

         if (objectId == null || objectId.trim().length() == 0) {

            objectId = Constants.HEAD;

@@ -176,7 +186,7 @@
         commit = rev;

         walk.dispose();

      } catch (Throwable t) {

         LOGGER.error("Failed to determine last change", t);

         LOGGER.error("Failed to get commit " + objectId, t);

      }

      return commit;

   }

@@ -328,6 +338,10 @@


   public static List<PathChangeModel> getFilesInCommit(Repository r, RevCommit commit) {

      List<PathChangeModel> list = new ArrayList<PathChangeModel>();

      if (commit == null) {

         LOGGER.warn("getFilesInCommit for NULL commit");

         return list;

      }

      try {

         final RevWalk rw = new RevWalk(r);

         RevCommit parent = rw.parseCommit(commit.getParent(0).getId());

@@ -606,6 +620,9 @@


   public static List<RevCommit> getRevLog(Repository r, String objectId, String path, int offset, int maxCount) {

      List<RevCommit> list = new ArrayList<RevCommit>();

      if (!hasCommits(r)) {

         return list;

      }

      try {

         if (objectId == null || objectId.trim().length() == 0) {

            objectId = Constants.HEAD;

@@ -664,6 +681,9 @@
   public static List<RevCommit> searchRevlogs(Repository r, String objectId, String value, final SearchType type, int offset, int maxCount) {

      final String lcValue = value.toLowerCase();

      List<RevCommit> list = new ArrayList<RevCommit>();

      if (!hasCommits(r)) {

         return list;

      }

      try {

         if (objectId == null || objectId.trim().length() == 0) {

            objectId = Constants.HEAD;

@@ -792,13 +812,15 @@
   }



   public static List<Metric> getDateMetrics(Repository r) {

      Metric total = new Metric("TOTAL");

      final Map<String, Metric> metricMap = new HashMap<String, Metric>();

		
      if (hasCommits(r)) {

      final List<RefModel> tags = getTags(r, -1);

      final Map<ObjectId, RefModel> tagMap = new HashMap<ObjectId, RefModel>();

      for (RefModel tag : tags) {

         tagMap.put(tag.getCommitId(), tag);

      }

      Metric total = new Metric("TOTAL");

      final Map<String, Metric> metricMap = new HashMap<String, Metric>();

      try {

         RevWalk walk = new RevWalk(r);

         ObjectId object = r.resolve(Constants.HEAD);

@@ -837,6 +859,7 @@
      } catch (Throwable t) {

         LOGGER.error("Failed to mine log history for metrics", t);

      }

      }

      List<String> keys = new ArrayList<String>(metricMap.keySet());

      Collections.sort(keys);

      List<Metric> metrics = new ArrayList<Metric>();