Problem in CommonDaoWorker::uncacheByIds

[DeryabinSergey]

onphp-framework/src/Main/DAO/Worker/CommonDaoWorker.php

Lines 512 to 523 in 5234ae3

	public function uncacheByIds($ids)
	{
	if (empty($ids))
	return;
	$uncacher = $this->getUncacherById(array_shift($ids));
	foreach ($ids as $id)
	$uncacher->merge($this->getUncacherById($id));
	return $this->registerUncacher($uncacher->uncache());
	}

Определяем $uncacher, мержим его в цикле и передаем $uncacher->uncache() в $this->registerUncacher() вместо $uncacher.

Ошибки из-за этого я пока не получал, но во первых $uncacher->uncache() ничего не возвращает:

onphp-framework/src/Main/DAO/Uncacher/UncacherBaseDaoWorker.php

Lines 53 to 60 in 5234ae3

	public function uncache()
	{
	foreach ($this->classNameMap as $className => $idKeys) {
	foreach ($idKeys as $key) {
	$this->uncacheClassName($className, $idKeys);
	}
	}
	}

и в $this->registerUncacher() пойдет NULL вместо ожидаемого интерфейса UncacherBase. Плюс далее по цепочке в

onphp-framework/src/Core/DB/DB.php

Lines 438 to 440 in 5234ae3

	public function registerUncacher(UncacherBase $uncacher)
	{
	$uncacher->uncache();

вызывается $uncacher->uncache()

По итогу строчку

onphp-framework/src/Main/DAO/Worker/CommonDaoWorker.php

Line 522 in 5234ae3

return $this->registerUncacher($uncacher->uncache());

надо заменить на $this->registerUncacher($uncacher); Возврат тоже убрать, потому как смысла в нем нет.