Use LocalDateTime instead of LocalDate

Author: Isira-Seneviratne

extractor/src/main/java/org/schabi/newpipe/extractor/localization/DateWrapper.java

@@ -4,6 +4,7 @@
 import java.io.Serializable;
 import java.time.Instant;
 import java.time.LocalDate;
+import java.time.LocalDateTime;
 import java.time.OffsetDateTime;
 import java.time.ZoneId;
 import java.time.ZoneOffset;
@@ -35,12 +36,12 @@ public DateWrapper(@Nonnull final Instant instant, final boolean isApproximation
         this.isApproximation = isApproximation;
     }
 
-    public DateWrapper(@Nonnull final LocalDate localDate) {
-        this(localDate, true);
+    public DateWrapper(@Nonnull final LocalDateTime dateTime) {
+        this(dateTime, false);
     }
 
-    public DateWrapper(@Nonnull final LocalDate localDate, final boolean isApproximation) {
-        this(localDate.atStartOfDay(ZoneId.systemDefault()).toInstant(), isApproximation);
+    public DateWrapper(@Nonnull final LocalDateTime dateTime, final boolean isApproximation) {
+        this(dateTime.atZone(ZoneId.systemDefault()).toInstant(), isApproximation);
     }
 
     /**
@@ -63,8 +64,8 @@ public OffsetDateTime offsetDateTime() {
      * @return the wrapped {@link Instant} as a {@link LocalDate} in the current time zone.
      */
     @Nonnull
-    public LocalDate getLocalDate() {
-        return LocalDate.ofInstant(instant, ZoneId.systemDefault());
+    public LocalDateTime getLocalDateTime() {
+        return LocalDateTime.ofInstant(instant, ZoneId.systemDefault());
     }
 
     /**

extractor/src/main/java/org/schabi/newpipe/extractor/localization/TimeAgoParser.java

@@ -5,7 +5,6 @@
 import org.schabi.newpipe.extractor.utils.Parser;
 
 import java.time.LocalDateTime;
-import java.time.ZoneId;
 import java.time.temporal.ChronoUnit;
 import java.util.Map;
 import java.util.regex.Pattern;
@@ -105,15 +104,13 @@ private boolean textualDateMatches(final String textualDate, final String agoPhr
     }
 
     private DateWrapper getResultFor(final int timeAgoAmount, final ChronoUnit chronoUnit) {
-        final var resolvedDateTime = chronoUnit == ChronoUnit.YEARS
+        final var localDateTime = chronoUnit == ChronoUnit.YEARS
                 // minusDays is needed to prevent `PrettyTime` from showing '12 months ago'.
                 ? now.minusYears(timeAgoAmount).minusDays(1)
                 : now.minus(timeAgoAmount, chronoUnit);
-
-        if (chronoUnit.isDateBased()) {
-            return new DateWrapper(resolvedDateTime.toLocalDate());
-        } else {
-            return new DateWrapper(resolvedDateTime.atZone(ZoneId.systemDefault()).toInstant());
-        }
+        final boolean isApproximate = chronoUnit.isDateBased();
+        final var resolvedDateTime =
+                isApproximate ? localDateTime.truncatedTo(ChronoUnit.DAYS) : localDateTime;
+        return new DateWrapper(resolvedDateTime, isApproximate);
     }
 }