Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Etree entity bundles should be stored in separate database tables #29

Open
dashiwa opened this issue Jan 23, 2019 · 1 comment
Open
Labels
perspective Perspective project features
Milestone

Comments

@dashiwa
Copy link
Collaborator

dashiwa commented Jan 23, 2019

@orion76 Один из способов.

Обьявлять каждый бандл как отдельную простую сущность без бандла (так мы решим проблему с переопределением кода get_base_table)
В route provider сделать роут
Пример
EtreeMain/EtreeFirstBundle
EtreeMain/EtreeSecondBundle

EtreeMain - базовая сущность с апи. То есть Manager класс с апи.
Вероятно можно будет обьявить такую сущность без интерфейса и без таблицы в бд.
Возможно замокать как-то обьявление аннотаций.

EtreeFirstBundle и EtreeSecondBundle будут экстендить ее или имплементировать ее интерфейс..

Я пока не имею представления об общей архитектуре проекта и поэтому не смогу нарисовтаь UML более менее

@dashiwa dashiwa added the perspective Perspective project features label Jan 23, 2019
@dashiwa dashiwa added this to the Refactoring milestone Jan 23, 2019
@orion76
Copy link
Owner

orion76 commented Jan 24, 2019

Так-то да..
Способ с "отдельными типами сущностей" проще и возможно оптимальнее.

1.Из текущей сущности выделить абстрактный класс контент-сущности.
2.Новые сущности наследовать от него.
3.Переписать хендлеры и прочие классы без привязки к конкретному типу контент-сущности.
4.Добавить в аннотацию к контент-сущности доп.параметры, необходимые для работы хендлеров, сервисов и плагинов.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
perspective Perspective project features
Projects
None yet
Development

No branches or pull requests

2 participants