Повторное использование getById

[sahkalit]

Вот такой код:

<?php
$object1 = SomeDao->getById(1);
$object2 = SomeDao->getById(2);

SomeDao->take($object1->setSomeProp('prop1');

$object2 = SomeDao->getById(2);  // Да, мне нужно сделать второй раз getById
SomeDao->take($object1->setSomeProp('prop2');

сохранит в БД только одну первую запись для $object1.
Сделает он это потому, что при первом take, вызывается CommonDaoWorker::uncacheById,
в котором вызывается GenericDAO::uncacheLists. Дальше GenericDAO::dropIdentityMap и в нем есть строчка:
$this->identityMap = array().
Из-за этого, когда мы в следующий раз будем вызывать getById(2), то нам вернется ссылка на объект из кэша(GenericDAO::getById, CommonDaoWorker::getById, BaseDaoWorker::getCachedById) и поэтому, чтобы мы не засетили в него, при попытке take переменная $changed, определенная как
$changed = ($old->$getter() !== $object->$getter()) || ($old->$getter() != $object->$getter());
никогда не будет true, т.к. $old и $object один и тот же объект.

Решается, заменой второй строчки
$object2 = SomeDao->getById(2);
на
$object2 = clone SomeDao->getById(2);

Прокомментируйте пожалуйста, баг ли это и есть ли верное решение.

[AlexeyDsov]

Черт побери, засада :) Странно что никто на это раньше не наткнулся.
Клонировать синглтон однозначно не хорошо. Если проблема лишь в сравнении объектов, то сравнить их можно по id.
Если важно что бы были именно одинаковые объекты, то тут уже нужно делать исправление, а оно пока вырисовывается неоднозначно. С одной стороны uncacheLists должен чистить identityMap, с другой стороны в этом примере CommonDaoWorker действительно должен вызываться uncacheLists, но при этом identityMap удаляться не должен...

[sahkalit]

Ага, засада-) А есть уверенность, что когда мы делает take для конкретного элемента, то надо обнулить весь identityMap? не могу понять, в каком случае мы должны это делать..
И тут
clone SomeDao->getById(2);
я клонирую только объект, который вернулся из кеша, а не весь cинглетон. Тогда $changed = ($old->$getter() !== $object->$getter()) будет true и сработает take.

[AlexeyDsov]

В случае take обнулять idntityMap не надо. GenericDao сам по себе так и не делает. Это все подстраивают злые воркеры :).
При изменении любого объекта необходимо избавиться от старых кэшей всяких листов таблицы. При этом если в других таблицах есть связи на текущую таблицу и в мете они join, а не lazy то списки других таблиц тоже нужно дропнуть. В AutoDao* классов метод uncacheLists переопределяется и там как раз раскэшиваются списки других таблиц. А AutoDao* и есть наследник GenericDao и вот из-за всего этого CommonDaoWorker вызывает метод uncacheLists. К сожалению под раздачу при этом попадает identityMap.

[sahkalit]

стало яснее, но как решать эту проблему правильно непонятно) клоне делать там, где повторно делаю getById?
А если фиксить фм, то тут 3 пути на мой взгляд:

1. както убрать обнуление identityMap в take.
2. обнулять заодно Cache для данного класса(в данном случае у меня это RuntimeMemory)
3. делать дополнительные проверки, что если при getById и getCachedById объекты совпадают, то делать clone.
   Что скажете?

[AlexeyDsov]

Клонирование синглтона не вариант. По хорошему, видимо, решением может быть разделение метода uncacheLists на два. Скажем uncacheLists делает все тоже самое, но при этом есть второй метод, который вызывает CommonDaoWorker и который делает тоже что uncacheLists но при этом не чистит identityMap.

[AlexeyDsov]

На самом деле я бы оставил этот issue открытым (что бы не забыть), так как это все таки бага и ее надо пофиксить.