Add todo to remove conditional statement (#7163)

* mark generated by script as false when creating new domain * add todo to remove conditional statement
ORCID · Dec 18, 2024 · 133ce49 · 133ce49
1 parent 5b00024
commit 133ce49
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/orcid-core/src/main/java/org/orcid/core/common/manager/impl/SummaryManagerImpl.java b/orcid-core/src/main/java/org/orcid/core/common/manager/impl/SummaryManagerImpl.java
@@ -524,9 +524,13 @@ public void generateEmailDomainsSummary(RecordSummary recordSummary, String orci
                 for (ProfileEmailDomainEntity ped : emailDomains) {
                     ed = new EmailDomain();
                     ed.setValue(ped.getEmailDomain());
-                    if (!ped.getGeneratedByScript() && ped.getDateCreated() != null) {
-                        VerificationDate verificationDate = new VerificationDate(DateUtils.convertToXMLGregorianCalendar(ped.getDateCreated()));
-                        ed.setVerificationDate(verificationDate);
+                    if (!ped.getGeneratedByScript()) {
+                        //TODO: There is a bug where the date_created is null for some records, so, we need to add this one meanwhile we fix this card https://trello.com/c/qh1uioKO/9557-qa-verified-date-not-displayed-for-new-email-domains
+                        //Remove this if statement once that card get to PROD
+                        if(ped.getDateCreated() != null) {
+                            VerificationDate verificationDate = new VerificationDate(DateUtils.convertToXMLGregorianCalendar(ped.getDateCreated()));
+                            ed.setVerificationDate(verificationDate);
+                        }
                     }
                     edList.add(ed);
                 }