Details
-
Bug
-
Resolution: Unresolved
-
Medium
-
5.0, 5.1
-
None
Description
In eZ\Bundle\EzPublishCoreBundle\Controller, visibility of methods isn't consistent:
- protected function getConfigResolver()
- final protected function getLegacyKernel()
- public function getRepository()
Since this class is in any case, protected and public are identical from a scope perspective. They should either be all public, or all protected.
Considering that some doc tools (SAMI) don't show protected methods, public is an option. It also makes the controller easier to test in terms of public behaviour.