fix: Use \DateTimeInterface to check object types instead of `\Date…

…Time`
roadiz · Mar 20, 2024 · 9389865 · 9389865
1 parent 1d30cef
commit 9389865
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 10 deletions.
diff --git a/lib/RoadizCoreBundle/src/Api/Extension/ArchiveExtension.php b/lib/RoadizCoreBundle/src/Api/Extension/ArchiveExtension.php
@@ -104,7 +104,7 @@ public function getResult(
 
         foreach ($paginator as $result) {
             $dateTimeField = reset($result);
-            if ($dateTimeField instanceof \DateTime) {
+            if ($dateTimeField instanceof \DateTimeInterface) {
                 $year = $dateTimeField->format('Y');
                 $month = $dateTimeField->format('Y-m');
 

diff --git a/lib/RoadizCoreBundle/src/Console/NodesDetailsCommand.php b/lib/RoadizCoreBundle/src/Console/NodesDetailsCommand.php
@@ -67,7 +67,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                     if (is_array($data)) {
                         $data = implode(', ', $data);
                     }
-                    if ($data instanceof \DateTime) {
+                    if ($data instanceof \DateTimeInterface) {
                         $data = $data->format('c');
                     }
                     if ($data instanceof \stdClass) {

diff --git a/lib/RoadizCoreBundle/src/CustomForm/CustomFormHelper.php b/lib/RoadizCoreBundle/src/CustomForm/CustomFormHelper.php
@@ -213,7 +213,7 @@ protected function getDocumentFolderForCustomForm(): ?Folder
      */
     private function formValueToString($rawValue): string
     {
-        if ($rawValue instanceof \DateTime) {
+        if ($rawValue instanceof \DateTimeInterface) {
             return $rawValue->format('Y-m-d H:i:s');
         } elseif (is_array($rawValue)) {
             $values = $rawValue;

diff --git a/lib/RoadizCoreBundle/src/Model/AttributeValueTranslationTrait.php b/lib/RoadizCoreBundle/src/Model/AttributeValueTranslationTrait.php
@@ -72,7 +72,7 @@ public function setValue($value)
                 return $this;
             case AttributeInterface::DATETIME_T:
             case AttributeInterface::DATE_T:
-                if ($value instanceof \DateTime) {
+                if ($value instanceof \DateTimeInterface) {
                     $this->value = $value->format('Y-m-d H:i:s');
                 } else {
                     $this->value = (string) $value;

diff --git a/lib/RoadizCoreBundle/src/SearchEngine/NodeSourceSearchHandler.php b/lib/RoadizCoreBundle/src/SearchEngine/NodeSourceSearchHandler.php
@@ -168,15 +168,15 @@ protected function argFqProcess(array &$args): array
          */
         if (isset($args['publishedAt'])) {
             $tmp = "published_at_dt:";
-            if (!is_array($args['publishedAt']) && $args['publishedAt'] instanceof \DateTime) {
+            if (!is_array($args['publishedAt']) && $args['publishedAt'] instanceof \DateTimeInterface) {
                 $tmp .= $this->formatDateTimeToUTC($args['publishedAt']);
             } elseif (
                 isset($args['publishedAt'][0]) &&
                 $args['publishedAt'][0] === "BETWEEN" &&
                 isset($args['publishedAt'][1]) &&
-                $args['publishedAt'][1] instanceof \DateTime &&
+                $args['publishedAt'][1] instanceof \DateTimeInterface &&
                 isset($args['publishedAt'][2]) &&
-                $args['publishedAt'][2] instanceof \DateTime
+                $args['publishedAt'][2] instanceof \DateTimeInterface
             ) {
                 $tmp .= "[" .
                     $this->formatDateTimeToUTC($args['publishedAt'][1]) .
@@ -186,14 +186,14 @@ protected function argFqProcess(array &$args): array
                 isset($args['publishedAt'][0]) &&
                 $args['publishedAt'][0] === "<=" &&
                 isset($args['publishedAt'][1]) &&
-                $args['publishedAt'][1] instanceof \DateTime
+                $args['publishedAt'][1] instanceof \DateTimeInterface
             ) {
                 $tmp .= "[* TO " . $this->formatDateTimeToUTC($args['publishedAt'][1]) . "]";
             } elseif (
                 isset($args['publishedAt'][0]) &&
                 $args['publishedAt'][0] === ">=" &&
                 isset($args['publishedAt'][1]) &&
-                $args['publishedAt'][1] instanceof \DateTime
+                $args['publishedAt'][1] instanceof \DateTimeInterface
             ) {
                 $tmp .= "[" . $this->formatDateTimeToUTC($args['publishedAt'][1]) . " TO *]";
             }

diff --git a/lib/RoadizCoreBundle/src/Xlsx/XlsxExporter.php b/lib/RoadizCoreBundle/src/Xlsx/XlsxExporter.php
@@ -107,7 +107,7 @@ public function exportXlsx($data, $keys = [])
                     continue;
                 }
 
-                if ($value instanceof \DateTime) {
+                if ($value instanceof \DateTimeInterface) {
                     $value = Date::PHPToExcel($value);
                     $activeSheet->getStyle($columnAlpha . ($activeRow))
                         ->getNumberFormat()