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

Method does not exist[BUG] #26

Open
dashiwa opened this issue Jan 21, 2019 · 5 comments
Open

Method does not exist[BUG] #26

dashiwa opened this issue Jan 21, 2019 · 5 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@dashiwa
Copy link
Collaborator

dashiwa commented Jan 21, 2019

The website encountered an unexpected error. Please try again later.

ReflectionException: Method Drupal\etree\Controller\ETreeController::collectionOverview() does not exist in ReflectionMethod->__construct()

@dashiwa dashiwa added the bug Something isn't working label Jan 21, 2019
@dashiwa dashiwa added this to the Refactoring milestone Jan 21, 2019
@dashiwa dashiwa self-assigned this Jan 21, 2019
@dashiwa
Copy link
Collaborator Author

dashiwa commented Jan 21, 2019

@orion76 По зависимостям непонятно
etree сущность является базовой сущностью значит она не должна ни от кого зависеть?
Ну если только не делать инверсию зависимостей
Я бы предложил перерабоать этот модуль чтобы контроллер не зависел от метода находящегоса в etree_links модуле

@orion76
Copy link
Owner

orion76 commented Jan 21, 2019

Про это и был рефакторинг-)
И еще придется "порефакторить"
Да сущность etree должна быть полностью "автономна"..
А весь дополнительный функционал дополнительными связанными модулями.

Просто сначала реализовывался "частный" случай ее использования..
Понемногу появляется понятие как сделать более универсально.

Сейчас думаю надо сделать так чтобы модуль сущности etree предоставлял только саму сущность и API для работы с ней (загрузка-сохранение сущностей в БД)

Еще пришел к такому выводу что бандлы сущности etree должна храниться в отдельных таблицах БД.
Чтобы в пределах приложения не смешивать "разнотипные" данные.

@dashiwa
Copy link
Collaborator Author

dashiwa commented Jan 23, 2019

Еще пришел к такому выводу что бандлы сущности etree должна храниться в отдельных таблицах БД.
Чтобы в пределах приложения не смешивать "разнотипные" данные.

В рамках апи это возможно?
Аннотации затрудняют дебаг данной темы. Тем более большую часть аннотаций 80 процентов не добавили в класс Annotation для документирования.

Но в рамках ООП подхода мне кажется что это возможно. Нужно только найти метод сохраняющий схему данных в бд и попробовать перегрузить его, при условии что там нет кучи зависимого кода который придеться дорабатывать

@dashiwa
Copy link
Collaborator Author

dashiwa commented Jan 23, 2019

По аннотациям.
Это свойства классов которые в целом распиханы по разным классам

@orion76
Copy link
Owner

orion76 commented Jan 24, 2019

В рамках апи это возможно?

В drupal 8 возможно все:
1.Добавить новый функционал (кастомные сервисы, плагины)
2.Добавить функционал к ядерным и контрибным сервисам-плагинам (хуки,события,деривативы, "усилители"(Enhancer) и т.п.)
3.Преопределить ядерные-контрибные сервисы.
4. и еще куча всего, разобраться или как минимум познакомиться с чем еще не дошли руки-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants