Better fix for #17091

Calling self::serializeValue() was breaking other field types, like relation fields

https://github.com/craftcms/cms/actions/runs/14476047922/job/40601978956

Author: brandonkelly

src/base/Field.php

@@ -659,9 +659,7 @@ public function serializeValueForDb(mixed $value, ElementInterface $element): mi
             return Db::prepareDateForDb($value);
         }
 
-        // specifically calling `self::…` here rather than `$this->…`
-        // see https://github.com/craftcms/cms/pull/17091
-        return self::serializeValue($value, $element);
+        return $this->serializeValue($value, $element);
     }
 
     /**

src/fields/Table.php

@@ -16,6 +16,7 @@
 use craft\gql\types\TableRow;
 use craft\helpers\Cp;
 use craft\helpers\DateTimeHelper;
+use craft\helpers\Db;
 use craft\helpers\Json;
 use craft\helpers\StringHelper;
 use craft\validators\ColorValidator;
@@ -557,7 +558,13 @@ public function serializeValueForDb(mixed $value, ElementInterface $element): mi
                     $value = StringHelper::emojiToShortcodes(StringHelper::escapeShortcodes($value));
                 }
 
-                $serializedRow[$colId] = parent::serializeValueForDb($value ?? null, $element);
+                // can't call parent::serializeValueForDb() here because that calls $this->serializeValue()
+                // see https://github.com/craftcms/cms/pull/17091
+                if ($value instanceof DateTime || DateTimeHelper::isIso8601($value)) {
+                    $serializedRow[$colId] = Db::prepareDateForDb($value);
+                } else {
+                    $serializedRow[$colId] = parent::serializeValue($value, $element);
+                }
             }
             $serialized[] = $serializedRow;
         }