LM-Commons / LmcRbacMvc

Role-based access control Laminas MVC module to provide additional features on top of Laminas\Permissions\Rbac
https://lm-commons.github.io/LmcRbacMvc/
BSD 3-Clause "New" or "Revised" License
11 stars 5 forks source link

Proposed: Move factories to the same folder as the object they create #99

Closed visto9259 closed 4 weeks ago

visto9259 commented 3 months ago

Factories should be closer to the object that it is created. This is will help in understanding the code as one does not have to navigate to the Factory folder.

Refactor all the factories in Factory to the folder of the object that they are creating. For example, LmcRbacMvc\Factory\RoleServiceFactory should be refactored to LmcRbacMvc\Service\RoleServiceFactory. Have the original factory extend the new factory and mark it as deprecated Update ConfigProvider to use the new factories

visto9259 commented 4 weeks ago

Fixed by #117