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

Refactoring #780

Open
jan-stanek opened this issue Jan 8, 2021 · 2 comments
Open

Refactoring #780

jan-stanek opened this issue Jan 8, 2021 · 2 comments
Assignees

Comments

@jan-stanek
Copy link
Member

jan-stanek commented Jan 8, 2021

  • Náhrada většiny Service za CommandBus/QueryBus handlery
    • částečně hotovo
  • Náhrada volání repository z presenterů, formulářů, komponent, ... za CommandBus/QueryBus handlery
    • částečně hotovo
  • Repository dědí z AbstractRepository místo EntityRepository
    • hotovo
  • Odstranění logiky z entit
    • hlavně té složitější, někde to dává smysl
  • V datagridech používat jako zdroj array místo doctrine
    • lepší bude většinou používat předpřipravená query, která se používají i v repository
  • Použití vlastních výjimek
    • částečně hotovo v Model\Program
    • chybové hlášky pro uživatele
  • Použití transakcí, pokud se vykonává více změn
  • Použití doctrine cache místo manuální práce s cache
    • AclService
    • Authenticator?
    • CmsService
@jan-stanek jan-stanek self-assigned this Jan 8, 2021
@petrparolek
Copy link
Collaborator

jaké má výhody takový refactoring z service? 😮

@jan-stanek
Copy link
Member Author

@petrparolek menší jednodušší třídy, který dělají jednu věc + se v konstruktorech předává míň závislostí (Command/Query/Event Bus místo všech service a repository)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

2 participants