Attempt to fix CPD by introducing a common casting function.

Author: patel-vansh

system/BaseModel.php

@@ -911,16 +911,7 @@ public function insertBatch(?array $set = null, ?bool $escape = null, int $batch
             foreach ($set as &$row) {
                 // If casts are used, convert the data first
                 if ($this->useCasts()) {
-                    if (is_array($row)) {
-                        $row = $this->converter->toDataSource($row);
-                    } elseif ($row instanceof stdClass) {
-                        $row = (array) $row;
-                        $row = $this->converter->toDataSource($row);
-                    } elseif ($row instanceof Entity) {
-                        $row = $this->converter->extract($row);
-                    } elseif (is_object($row)) {
-                        $row = $this->converter->extract($row);
-                    }
+                    $row = $this->performCasting($row);
                 } elseif (is_object($row) && ! $row instanceof stdClass) {
                     // If $row is using a custom class with public or protected
                     // properties representing the collection elements, we need to grab
@@ -1065,16 +1056,7 @@ public function updateBatch(?array $set = null, ?string $index = null, int $batc
             foreach ($set as &$row) {
                 // If casts are used, convert the data first
                 if ($this->useCasts()) {
-                    if (is_array($row)) {
-                        $row = $this->converter->toDataSource($row);
-                    } elseif ($row instanceof stdClass) {
-                        $row = (array) $row;
-                        $row = $this->converter->toDataSource($row);
-                    } elseif ($row instanceof Entity) {
-                        $row = $this->converter->extract($row);
-                    } elseif (is_object($row)) {
-                        $row = $this->converter->extract($row);
-                    }
+                    $row = $this->performCasting($row);
                 } elseif (is_object($row) && ! $row instanceof stdClass) {
                     // If $row is using a custom class with public or protected
                     // properties representing the collection elements, we need to grab
@@ -1721,6 +1703,39 @@ protected function trigger(string $event, array $eventData)
         return $eventData;
     }
 
+    /**
+     * If the model is using casts, this will convert the data
+     * in $row according to the rules defined in `$casts`.
+     *
+     * @param object|row_array|null $row Row data
+     *
+     * @return object|row_array|null Converted row data
+     *
+     * @used-by insertBatch()
+     * @used-by updateBatch()
+     *
+     * @deprecated Since 4.6.4, temporary solution - will be removed in 4.7
+     */
+    protected function performCasting(array|object|null $row = null): array|object|null
+    {
+        if (! $this->useCasts()) {
+            return $row;
+        }
+
+        if (is_array($row)) {
+            $row = $this->converter->toDataSource($row);
+        } elseif ($row instanceof stdClass) {
+            $row = (array) $row;
+            $row = $this->converter->toDataSource($row);
+        } elseif ($row instanceof Entity) {
+            $row = $this->converter->extract($row);
+        } elseif (is_object($row)) {
+            $row = $this->converter->extract($row);
+        }
+
+        return $row;
+    }
+
     /**
      * Sets the return type of the results to be as an associative array.
      *