Upgrade from 4.2 to 5.1.3 get error The 'keepSnapshots' option must be enabled to track changes

[Drummi42]

Hello!
I am migrating a project from 4.2 to 5.1.3 phalcon. Get the error Error The 'keepSnapshots' option must be enabled to track changes when using $model->hasChanged(). The function $model->getSnapshotData() returns NULL.

The function $this->keepSnapshots(true); from the docs doesn't work.

class Robots extends Model
{
     public function initialize()
     {
          $this->keepSnapshots(true);
     }
}

The Fix from issue #15837 doesn't work too.

Maybe someone has encountered or have any ideas? Thank you in advance.

Case:

1. Model::findFirst()
2. Fill property
3. Call ->hasChanged()

php 8.1, phalcon 5.1.3, nginx, almalinux

[Drummi42]

If use $model->refresh(), then $model->getSnapshotData() correctly returns an array.

1. Model::findFirst()
2. $model->refresh()
3. Fill property
4. $model->hasChanged()
   

   cphalcon/phalcon/Mvc/Model.zep

   Line 2532 in 9d2d864

   this->setSnapshotData(row, columnMap);

[Drummi42]

__serialization sets snapshot = null in the cache model if attributes are not changed.
https://github.com/phalcon/cphalcon/blob/5.0.x/phalcon/Mvc/Model.zep#L384
If the snapshot is cached, then __unserialize set snapshot = cachedSnapshot. And this condition does not work
https://github.com/phalcon/cphalcon/blob/5.0.x/phalcon/Mvc/Model.zep#L603
temporary fix in the model (im use useDynamicUpdate(true), snapshot enable for all models):

public function __serialize(): array
{
  $arr = parent::__serialize();
  if ($arr['snapshot'] === null) {
      unset($arr['snapshot']);
  }
  return $arr;
}

[Jeckerson]

/cc @niden