Closed angoliar closed 8 years ago
@angoliar, в общем, отличная идея! Чутка доработать только.
Получается этот PR нужно закрыть, а мне сделать форк dotplant/product-bookmarks и доработать его?
Да.
Может включить расширение изначально в CMS? Там ведь нужны вьюхи, переводы, скрипты дотплантовские..
в букмаркс можно заменить все на мое (в.т.ч. миграции)?:-)
Можно, если ты обеспечишь совместимость. Т.е. все сайты с этим расширением должны продолжать корректно работать после обновления. Для этого придется написать миграцию и сохранить названия контролеров. Это навскидку. Может что-то еще.
быть может создать свое расширение и зарегистрировать композитор пакет, что бы все пользователи могли его установить из админки dp2? Когда у пользователей будет возможность выбора, это очень хорошо!
Тогда так и сделаю.. Получается в этом PR останутся только js и переводы?
Давайте все таки включим этот PR в движок? Функционал для магазина весьма стандартный..
Я внес правки.. Посмотрите?
Я все сломал - зашел под админом - накидал списков, потом разлогинился получил ошибку
Integrity constraint violation – yii\db\IntegrityException
SQLSTATE[23000]: Integrity constraint violation: 1452 Cannot add or update a child row: a foreign key constraint fails (`test`.`wishlist`, CONSTRAINT `wishlist_user_id` FOREIGN KEY (`user_id`) REFERENCES `user` (`id`) ON DELETE CASCADE)
The SQL being executed was: INSERT INTO `wishlist` (`user_id`, `title`, `default`) VALUES (0, 'test123', 1)
Error Info: Array
(
[0] => 23000
[1] => 1452
[2] => Cannot add or update a child row: a foreign key constraint fails (`test`.`wishlist`, CONSTRAINT `wishlist_user_id` FOREIGN KEY (`user_id`) REFERENCES `user` (`id`) ON DELETE CASCADE)
)
удалил лишний ключ..
Проверите?
@angoliar, думаю, что лучше всего модифицировать расширение https://github.com/DevGroup-ru/dotplant2-product-bookmarks и там реализовать возможность множественных вишлистов.