Exclude classes for nette/application - Githubissues

nette / bootstrap

🅱 The simple way to configure and bootstrap your Nette application.

https://doc.nette.org/bootstrap

Other

663 stars 36 forks source link

Exclude classes for nette/application #33

Closed hrach closed 9 years ago

hrach commented 9 years ago

Setting $autowireExcludedClasses (https://github.com/nette/bootstrap/blob/92adef14a2d55eb7205d901a4d0be520f32a27ec/src/Bootstrap/Configurator.php#L57) to Nette\Application\UI\Control should be moved to nette/application package.

Otherwise, if I use nette/bootstrap without nette/application, container compilation fails on https://github.com/nette/di/blob/d090dd3573ab8aeacc83d3eceb0cd1ada11ec657/src/DI/ContainerBuilder.php#L291 with E_WARNING: class_parents(): Class Nette\Application\UI\Control does not exist and could not be loaded.

hrach commented 9 years ago

It's btw about the bad code composition. There should be definitely Container without this defaults, and then it would be totally ok. We said that, but you didn't listen.