You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2022/08/20 09:13:10 UTC

[maven-dist-tool] 06/07: Improve error reporting

This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-dist-tool.git

commit af7760b826e9a8f6b941948cbc522f29cb243177
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Sat Aug 20 11:11:29 2022 +0200

    Improve error reporting
---
 .../dist/tools/index/DistCheckIndexPageReport.java    | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/maven/dist/tools/index/DistCheckIndexPageReport.java b/src/main/java/org/apache/maven/dist/tools/index/DistCheckIndexPageReport.java
index d7f743d..7d2ddbc 100644
--- a/src/main/java/org/apache/maven/dist/tools/index/DistCheckIndexPageReport.java
+++ b/src/main/java/org/apache/maven/dist/tools/index/DistCheckIndexPageReport.java
@@ -22,6 +22,7 @@ package org.apache.maven.dist.tools.index;
 import java.io.IOException;
 import java.time.LocalDate;
 import java.time.Period;
+import java.time.format.DateTimeParseException;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.LinkedList;
@@ -195,13 +196,21 @@ public class DistCheckIndexPageReport
         sink.tableRow_();
     }
 
-    private boolean isDateSimilar( String date1, String date2 )
+    private boolean isDateSimilar( String metadataDate, String indexDate )
     {
-        LocalDate d1 = LocalDate.parse( date1 );
-        LocalDate d2 = LocalDate.parse( date2 );
+        try
+        {
+            LocalDate d1 = LocalDate.parse( metadataDate );
+            LocalDate d2 = LocalDate.parse( indexDate );
 
-        long daysDifference = Period.between( d1, d2 ).getDays();
-        return Math.abs( daysDifference ) < 7; // ok for 7 days difference
+            long daysDifference = Period.between( d1, d2 ).getDays();
+            return Math.abs( daysDifference ) < 7; // ok for 7 days difference
+        }
+        catch ( DateTimeParseException pe )
+        {
+            getLog().warn( "Unable to parse dates for fields from metadata:\"" + metadataDate + "\" and index:\"" + indexDate + "\"" );
+            return false;
+        }
     }
 
     /** {@inheritDoc} */