Merge commit '787f7d'

internetarchive · Jul 11, 2012 · 1eb5ba7 · 1eb5ba7
2 parents d0add8c + 787f7d9
commit 1eb5ba7
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/modules/src/main/java/org/archive/modules/deciderules/TooManyPathSegmentsDecideRule.java b/modules/src/main/java/org/archive/modules/deciderules/TooManyPathSegmentsDecideRule.java
@@ -18,7 +18,6 @@
  */
 package org.archive.modules.deciderules;
 
-import java.util.logging.Level;
 import java.util.logging.Logger;
 
 import org.archive.modules.CrawlURI;
@@ -33,6 +32,7 @@
 public class TooManyPathSegmentsDecideRule extends PredicatedDecideRule {
 
     private static final long serialVersionUID = 3L;
+    @SuppressWarnings("unused")
     private static final Logger logger = Logger.getLogger(TooManyPathSegmentsDecideRule.class.getName()); 
 
     /** default for this class is to REJECT */
@@ -69,9 +69,8 @@ public TooManyPathSegmentsDecideRule() {
     protected boolean evaluate(CrawlURI curi) {
         String uriPath = curi.getUURI().getEscapedPath();
         if (uriPath == null) {
-            uriPath = curi.toString();
-            // TODO this is a quick fix. figure out why getUURI().getEscapedPath() returns null.
-            logger.log(Level.WARNING, "getUURI().getEscapedPath() returns null for the given CrawlURI: "+ uriPath);
+            // no path means no segments
+            return false;
         }
         int count = 0;
         int threshold = getMaxPathDepth();