Improve AbstractEntity::resetChangedColumns for performance reasons

Author: Ivan Vasilkov

src/AbstractEntity.php

@@ -159,8 +159,16 @@ final public function getOldValue(string $columnName): mixed
         return $this->_initialColumns[$columnName] ?? null;
     }
 
-    final public function resetChangedColumns(): void
+    /**
+     * @param array<string, mixed>|null $values An associative array with column names as keys and
+     *                                          their respective values to reset, or null to reset all columns.
+     */
+    final public function resetChangedColumns(?array $values = null): void
     {
-        $this->_initialColumns = $this->toArray();
+        if ($values) {
+            $this->_initialColumns = $values + ($this->_initialColumns ?? []);
+        } else {
+            $this->_initialColumns = $this->toArray();
+        }
     }
 }

tests/AbstractEntityTest.php

@@ -243,4 +243,18 @@ public function __construct(
         $expected = str_replace('Array', 'Composite\Entity\AbstractEntity@anonymous Object', $expected);
         $this->assertEquals($expected, print_r($entity, true));
     }
+
+    public function test_resetChangedColumns(): void
+    {
+        $entity = new TestSubEntity();
+        $this->assertEquals($entity->toArray(), $entity->getChangedColumns());
+
+        $entity->resetChangedColumns();
+        $this->assertEquals([], $entity->getChangedColumns());
+
+        $entity = new TestSubEntity();
+        $entity->resetChangedColumns(['str' => $entity->str]);
+
+        $this->assertEquals(['number' => $entity->number], $entity->getChangedColumns());
+    }
 }