UnifiedContainer clone fix

[avid]

Предлагаю diff по issue #7.
Переопределен метод __clone и добавлен тест на этот случай.

[dovg]

А это нормально, что мы клонируем dao, которое реализует паттерн Singleton?

[AlexeyDsov]

Справедливости ради надо отметить что то dao которое кланируется совсем не синглтон.

[suquant]

Ну оно в контексте Singleton-а юзается )))

[AlexeyDsov]

Что за контекст такой синглтона?

[dovg]

Вот этот пример #57 (comment) я считаю правильным.

Ниже ИМХО:
Мне очевидно, что оба контейнера ссылаются на одну и ту же копию объекта.

[crazedr0m]

а если ....
$container1 = $products->getVariants();
$list1 = $container1->getList();

$products->dropVariants();

$container2 = $products->getVariants()->setCriteria(
Criteria::create()->add(
bla-bla-bla-expression(s)
)
);
$list2 = $container2->getList();

где
Product::dropVariants() {
$this->variants = null;
}

или сделайте фильтр на уже выбраное в $list1. я так понимаю там все а в $list2 - что-то.

На мой взгляд не удачный кейс.
и никто не отменял $products->getVariants()->clean() опять же после $list1 = $container1->getList();

[AlexeyDsov]

Где во всех этих примерах выше клонирование?

[crazedr0m]

я к тому, что пока не понял зачем вообще клонировать контейнер

[AlexeyDsov]

По большому счету контейнер - это просто простой способ получить список объектов которые ссылаются на объект - владелец контейнера а так же сохранить-модифицировать этот список. При этом сейчас в нем использовать setCriteria опасно, т.к. в одном месте сделаешь что б список доставался с дополнительными условиями, а удалить потом не удалишь и в другом месте там где не ожидаешь и где не надо будешь получать список с этими дополнительными условиями...

Кстати, тут походу нужно в таком случае дописать метод __clone также и у UnifiedContainerWorker'а

[crazedr0m]

@Neerrar @soloweb Если еще актуально, сделайте отдельную ветку и с неё пулл реквест.

[AlexeyDsov]

Не могу сказать за авторов, но я все хочу добраться до этого кейса и все никак не добираюсь.

[avid]

@crazedr0m новый pull request: #175

[dovg]

Этот pullrequest доставляет много писем в мою почту. Можно я его закрою?

[avid]

Да, закрывай.
Сам уже закрыл.