search

sulu / sulu-document-manager

Sulu document manager
sulu.io
6 stars 7 forks source link

Restructure namespaces #77

Open dantleech opened 8 years ago

dantleech commented 8 years ago

We have:

DocumentManager
DocumentManagerIntertace
Events\AbstractDocumentManagerContextEvent
DocumentManagerContext
DocumentManagerRegistry
DocumentManagerRegistryInterface

This should similar to (as an example, it will require more thought)

ManagerInterface
Manager\Context
Manager\Registry
Manager\DocumentManager
Events\AbstractManagerContextEvent.php

This would allow for easier future expansion, e.g. the proposed DocumentManagerContextAccessorInterface name is far too long.

danrot commented 8 years ago

I'd prefer rather long names than having naming collisions. A ManagerInterface can be quite a lot, and it will not always be clear based on fragment of code you'll see.