-
Notifications
You must be signed in to change notification settings - Fork 106
[ENHANCEMENT] Refactored kernel structure #130
Conversation
|
||
protected function getContext() | ||
{ | ||
if (null === $this->context) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can make "context" a kernel parameter that willl automatically be available in the dependency injection container. I believe.
Updatred. Porbley some conflict with ContentBUndle |
@danrot this should be good now, note you will need the referenced PRs above.. i.e. https://github.com/sulu-cmf/SuluMediaBundle/issues/54 |
{ | ||
$bundles = parent::registerBundles(); | ||
$bundles[] = new Symfony\Bundle\SecurityBundle\SecurityBundle(); | ||
$bundles[] = new Bazinga\Bundle\HateoasBundle\BazingaHateoasBundle(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dantleech Would you really put that into the AdminKernel? If some parts of the API are exposed on the website also, this bundle will probably also be needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will move to Abstract
Refactored kernel structure.
Tasks:
Informations: