Fixed RD-15247: IS [NOT] NULL generates wrong JQL

raw-labs · Dec 30, 2024 · 7313e49 · 7313e49
1 parent 8530102
commit 7313e49
Show file tree

Hide file tree

Showing 8 changed files with 1,700 additions and 139 deletions.
diff --git a/...-connector/src/main/java/com/rawlabs/das/jira/tables/DASJiraIssueTransformationTable.java b/...-connector/src/main/java/com/rawlabs/das/jira/tables/DASJiraIssueTransformationTable.java
@@ -3,7 +3,11 @@
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.rawlabs.das.sdk.java.exceptions.DASSdkApiException;
 import com.rawlabs.protocol.das.Row;
+import org.joda.time.DateTime;
+import org.joda.time.DateTimeZone;
 
+import java.time.OffsetDateTime;
+import java.time.ZonedDateTime;
 import java.util.*;
 
 @SuppressWarnings("unchecked")
@@ -52,11 +56,12 @@ protected void processFields(
         assignee.map(a -> a.get("displayName")).orElse(null),
         columns);
 
-    addToRow(
-        "created",
-        rowBuilder,
-        maybeFields.map(f -> f.get(names.get("Created"))).orElse(null),
-        columns);
+    String created =
+        maybeFields
+            .map(f -> f.get(names.get("Created")))
+            .map(c -> DateTime.parse(c.toString()).withZone(DateTimeZone.UTC).toString())
+            .orElse(null);
+    addToRow("created", rowBuilder, created, columns);
 
     var creator =
         maybeFields.map(f -> f.get(names.get("Creator"))).map(p -> (Map<String, Object>) p);
@@ -87,11 +92,9 @@ protected void processFields(
       throw new DASSdkApiException("error processing 'description'", e);
     }
 
-    addToRow(
-        "due_date",
-        rowBuilder,
-        maybeFields.map(f -> f.get(names.get("Due date"))).orElse(null),
-        columns);
+    String due_date = maybeFields.flatMap(f -> Optional.ofNullable(f.get(names.get("Due date")))).map(Object::toString).orElse(null);
+
+    addToRow("due_date", rowBuilder, due_date, columns);
 
     var priority =
         maybeFields.map(f -> f.get(names.get("Priority"))).map(p -> (Map<String, Object>) p);
@@ -106,12 +109,20 @@ protected void processFields(
         rowBuilder,
         reporter.map(r -> r.get("accountId")).orElse(null),
         columns);
+
     addToRow(
         "reporter_display_name",
         rowBuilder,
         reporter.map(r -> r.get("displayName")).orElse(null),
         columns);
 
+    String resolved =
+            maybeFields
+                    .flatMap(f -> Optional.ofNullable(f.get(names.get("Resolved"))))
+                    .map(c -> DateTime.parse(c.toString()).withZone(DateTimeZone.UTC).toString())
+                    .orElse(null);
+    addToRow("resolution_date", rowBuilder, resolved, columns);
+
     addToRow(
         "summary",
         rowBuilder,